8000 Fix sensor warmup, scan recovery, and live glucose updates by shroominic · Pull Request #35 · shroominic/OpenGlucose · GitHub
[go: up one dir, main page]

Skip to content

Fix sensor warmup, scan recovery, and live glucose updates - #35

Merged
shroominic merged 1 commit into
mainfrom
agent/warmup-history-live-activity
Aug 15, 2026
Merged

shroominic merged 1 commit into
mainfrom
agent/warmup-history-live-activity

Conversation

@shroominic
Copy link
Copy Markdown
Owner

Summary

  • Hide History, Patterns, and Weekly recap during the 60-minute sensor warmup.
  • Exclude sensor minutes 0-59 from displayed history, wellness analytics, messaging, Live Activities, and Apple Health while retaining them in clearly disclosed raw exports.
  • Defer AiDEX history synchronization during warmup, resume it automatically at the boundary, and route automatic failures through safe session error handling.
  • Restore Android warmup countdown notifications and iOS Live Activity countdowns with validated privacy-safe redaction.
  • Add an explicit, reversible setting for post-warmup glucose values in Android live notifications and iOS Live Activities; sensitive content remains off by default and consent failures roll back closed.
  • Replace Bluetooth-off scan StateError output with actionable enable/retry guidance, safe generic fallback copy, partial-result warning UI, and scan-generation race protection.

Root causes

Warmup readings and operational latest-reading state were reused directly by presentation and export integrations. AiDEX history catch-up could also start during warmup and fail without a reliable resume path. Live notification glucose had been intentionally redacted for privacy, but there was no reviewed consent UI to enable it. Scan failures were flattened to text and older scans could mutate UI after retry or connection transitions.

User impact

Users no longer see misleading initial sensor values or raw exception types. Warmup and live-glucose notification behavior is explicit and recoverable. Lock-screen glucose remains private by default and requires informed opt-in; withdrawal removes sensitive surfaces even if preference persistence fails.

Validation

  • make tooling-check
  • make format-check lint test
    • all workspace package tests passed
    • 219 OpenGlucose app tests passed
    • demo lifecycle integration test passed
  • Focused warmup, scan-race, privacy rollback, and live-update regression suites passed
  • make build-web
  • Swift frontend parse for Live Activity controller, shared attributes, and Runner tests
  • git diff --check

Local Android compilation is unavailable because this Mac has no Java runtime, and full iOS XCTest execution requires full Xcode. GitHub Actions is the authoritative native build/test gate for this PR.

@shroominic shroominic added area:release Release readiness, distribution, signing, and hardware validation area:alerts Notifications, freshness, disconnect, and wellness alerts priority:P0 Critical path or release-blocking work risk:R2 Health data, persistence, integration, AI, accessibility, or native risk labels Aug 15, 2026
@shroominic
shroominic marked this pull request as ready for review August 15, 2026 14:28
@shroominic
shroominic merged commit a5e4ea0 into main Aug 15, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5AD1

Labels

area:alerts Notifications, freshness, disconnect, and wellness alerts area:release Release readiness, distribution, signing, and hardware validation priority:P0 Critical path or release-blocking work risk:R2 Health data, persistence, integration, AI, accessibility, or native risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0