refactor: Address detekt code quality suppressions (technical debt)

[Wikijito7]

Context

During the migration from ktlint to detekt, many code quality issues were temporarily suppressed to allow the migration to proceed without blocking all development.

Update (2026-03-21)

A new issue #108 has been created specifically for the InvalidPackageDeclaration problem. The issue #107 now focuses only on the architecture violations that were suppressed.

Technical Debt - Refactoring Required

The following suppressions were added with // TODO: Refactor... (issue: #detekt-suppress) comments:

1. TooManyFunctions - GuildLavaPlayerService.kt

• Current: 39 functions in class
• Threshold: 20

2. LongParameterList - PaginatedEmbedUtils.kt

• Current: 11 parameters

3. LongParameterList - CommandHandlerServiceImpl

• Current: 16 constructor parameters

4. LongMethod + CyclomaticComplexMethod + NestedBlockDepth - AudioPlayerManagerProvider.kt

5. CyclomaticComplexMethod - CommandHandlerServiceImpl.onExecute

6. LongMethod - ConfigSetCommand.onExecute

7. LongMethod - PlayerCommons.kt

8. ThrowsCount - GuildQueueService.getOrCreateLavaPlayerService

9. SpreadOperator - BotException.kt

• Uses vararg which triggers warning when forwarding

Related Issues

• refactor: Fix InvalidPackageDeclaration in 87 files (flattened package structure) #108: InvalidPackageDeclaration - 87 files affected