Suggestion: Split AppSettings.kt into domain-specific settings files
Category: Large file
File: composeApp/src/commonMain/kotlin/com/oak/app/data/AppSettings.kt
At 1292 lines, AppSettings.kt handles settings across multiple domains including service credentials, email accounts, MCP server config, notification preferences, heartbeat schedules, and UI theme options. Storing all of these in one file makes it harder to maintain and navigate as new settings categories are added.
Suggested action: Decompose AppSettings.kt into domain-specific files such as ServiceSettings.kt, EmailSettings.kt, McpSettings.kt, NotificationSettings.kt, and UiSettings.kt, each owning a focused subset of the settings keys and accessors.
Suggestion: Split AppSettings.kt into domain-specific settings files
Category: Large file
File:
composeApp/src/commonMain/kotlin/com/oak/app/data/AppSettings.ktAt 1292 lines, AppSettings.kt handles settings across multiple domains including service credentials, email accounts, MCP server config, notification preferences, heartbeat schedules, and UI theme options. Storing all of these in one file makes it harder to maintain and navigate as new settings categories are added.
Suggested action: Decompose AppSettings.kt into domain-specific files such as
ServiceSettings.kt,EmailSettings.kt,McpSettings.kt,NotificationSettings.kt, andUiSettings.kt, each owning a focused subset of the settings keys and accessors.