Suggestion: Split SettingsScreen.kt into domain-specific settings files
Category: Large functions/files
File: composeApp/src/commonMain/kotlin/com/oak/app/ui/settings/SettingsScreen.kt
SettingsScreen.kt is 3,341 lines long — the largest file in the project — and composes settings UI for at least seven distinct domains: general settings, agent configuration, MCP server management, SSH connections, skills, sandbox/file browsing, and heartbeat/scheduling.
Suggested action: Split the file so that each settings section (e.g., GeneralSettingsCard.kt, AgentSettingsCard.kt, McpSettingsCard.kt, SshSettingsCard.kt, SkillSettingsCard.kt) lives in its own file, keeping only the top-level SettingsScreen composable and the tab-switching logic in the original file.
Suggestion: Split SettingsScreen.kt into domain-specific settings files
Category: Large functions/files
File:
composeApp/src/commonMain/kotlin/com/oak/app/ui/settings/SettingsScreen.ktSettingsScreen.ktis 3,341 lines long — the largest file in the project — and composes settings UI for at least seven distinct domains: general settings, agent configuration, MCP server management, SSH connections, skills, sandbox/file browsing, and heartbeat/scheduling.Suggested action: Split the file so that each settings section (e.g.,
GeneralSettingsCard.kt,AgentSettingsCard.kt,McpSettingsCard.kt,SshSettingsCard.kt,SkillSettingsCard.kt) lives in its own file, keeping only the top-levelSettingsScreencomposable and the tab-switching logic in the original file.