Skip to content

Refactor Health pure helper logic - #712

Merged
ryanbr merged 1 commit into
ryanbr:mainfrom
tigercraft4:refactor/health-phase1-logic
Jul 22, 2026
Merged

ryanbr merged 1 commit into
ryanbr:mainfrom
tigercraft4:refactor/health-phase1-logic

Conversation

@tigercraft4

Copy link
Copy Markdown

Summary

  • Extract live heart-rate display/sample helpers into HealthLiveHrLogic.kt.
  • Extract vital detail readings/range helpers into HealthVitalDetailLogic.kt.
  • Extract vital tile derivation and latest-vitals helpers into HealthVitalsLogic.kt.
  • Keep HealthScreen.kt focused on Compose UI wiring; no behavior change intended.

Part of the incremental code-quality split discussed in #665. This is a Phase-1 pure-helper extraction only; UI/composable clusters remain in HealthScreen.kt.

Verification

  • JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home ANDROID_HOME=$HOME/Library/Android/sdk ./gradlew :app:compileFullDebugKotlin
  • JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home ANDROID_HOME=$HOME/Library/Android/sdk ./gradlew :app:testFullDebugUnitTest --tests 'com.noop.ui.VitalRangeGatingTest' --tests 'com.noop.ui.VitalReadingsTableTest' --tests 'com.noop.ui.StepsReadingsMergeTest' --tests 'com.noop.ui.LiveHrSamplingTest' --tests 'com.noop.ui.HrvSnapshotHelpersTest'
  • Same focused test command against :app:testDemoDebugUnitTest

@ryanbr
ryanbr merged commit 4a1cf33 into ryanbr:main Jul 22, 2026
1 check passed
DX23876 pushed a commit to DX23876/noop that referenced this pull request Jul 22, 2026
…-logic

Refactor Health pure helper logic
simoncad7 pushed a commit to simoncad7/noop that referenced this pull request Aug 17, 2026
…-logic

Refactor Health pure helper logic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants