Skip to content

refactor(sensorservice): modernize service runtime - #19

Closed
Foxpace wants to merge 1 commit into
codex/sensorbox-refactor-05-sensorservice-measurementsfrom
codex/sensorbox-refactor-06-sensorservice-runtime
Closed

refactor(sensorservice): modernize service runtime#19
Foxpace wants to merge 1 commit into
codex/sensorbox-refactor-05-sensorservice-measurementsfrom
codex/sensorbox-refactor-06-sensorservice-runtime

Conversation

@Foxpace

@FoxpaceFoxpace commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Refactors the foreground measurement runtime around explicit configuration and session state.

What was added

  • A MeasurementConfig model for service startup options.
  • Updated service controller and foreground MeasurementService lifecycle handling.
  • Notification updates driven by the new runtime state.
  • Removal of the legacy MeasurementStates enum.

How it works

The controller builds a typed configuration and starts the service. MeasurementService uses that configuration to create measurements, updates the shared session store as the lifecycle changes, and keeps the foreground notification synchronized with the active recording state.

Stack position

Layer 6 of 31 in the SensorBox refactor stack. Review this PR against its configured base to see only this layer.

Validation

  • Full stack: ./gradlew test — BUILD SUCCESSFUL (160 tasks)

@FoxpaceFoxpace changed the title codex/sensorbox refactor 06 sensorservice runtimerefactor(sensorservice): modernize service runtimeAug 15, 2026
@Foxpace
Foxpace marked this pull request as ready for review August 15, 2026 21:50
@FoxpaceChatGPT Codex Connector

Copy link
Copy Markdown
OwnerAuthor

Superseded by #46, which targets master and shows the final current tree. Closing this stacked layer so review stays on the consolidated diff; this PR remains available as history.

@FoxpaceFoxpace closed this Aug 19, 2026
@FoxpaceChatGPT Codex Connector

Copy link
Copy Markdown
OwnerAuthor

Stack restored at the author's request. This PR is again part of the original 31-layer refactor chain. The architecture work continues as stacked PRs #47 through #54; consolidated PR #46 is superseded.

@FoxpaceFoxpace reopened this Aug 19, 2026
@FoxpaceFoxpace closed this Aug 23, 2026
@Foxpace
Foxpace deleted the codex/sensorbox-refactor-06-sensorservice-runtime branch August 23, 2026 12:58
Sign up for freeto 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.

1 participant

@Foxpace