Suggestion: Split HeartbeatSection.kt into separate section files

[adrielGGmotion]

Suggestion: Split HeartbeatSection.kt into separate section files

Category: Large functions/files
File: composeApp/src/commonMain/kotlin/com/oak/app/ui/settings/HeartbeatSection.kt:1

HeartbeatSection.kt is 889 lines and contains four distinct composable sections: HeartbeatSection, EmailSection, SmsSection, and NotificationsSection, along with several private helper functions. Each section manages a different domain (heartbeat polling, email polling, SMS polling, and app notifications) with its own state and UI logic.

Suggested action: Split the file into separate files such as HeartbeatSection.kt, EmailSection.kt, SmsSection.kt, and NotificationsSection.kt, each containing only its respective composable and helpers.

[adrielGGmotion]

@OpenHands all file split issues are approved

[openhands-ai]

I'm on it! adrielGGmotion can track my progress at all-hands.dev

[openhands-ai]

All done! Here's a summary of what was accomplished:

Changes Made

Issue #110 — Split the 889-line HeartbeatSection.kt into four separate files, each containing its own composable and helpers:

New files created:

• EmailSection.kt (167 lines) — EmailSection composable + formatPollRelative private helper
• SmsSection.kt (204 lines) — SmsSection composable + PermissionRequiredRow + formatPollRelative
• NotificationsSection.kt (119 lines) — NotificationsSection composable + PermissionRequiredRow

Modified file:

• HeartbeatSection.kt (498 lines, down from 889) — now contains only the HeartbeatSection composable and the cron-related helpers (describeCron, formatCronTime, dayName)

Verification:

• ✅ Desktop compilation passes (./gradlew :composeApp:compileKotlinDesktop)
• ✅ All tests pass (./gradlew :composeApp:allTests)
• All composables remain internal in the same package, so SettingsScreen.kt requires no changes

PR:

👉 PR #120 opened at #120