Uh oh!
There was an error while loading. Please reload this page.
Bubble Transmitter Integration - #7
Closed
bubbledevteam wants to merge 63 commits into
Closed
Conversation
# Conflicts: # .gitmodules # Loop
This reverts commit 85897d6.
bjornoleh referenced
this pull request
in bjornoleh/LoopWorkspace
Jan 22, 2023
Match Loop behavior for default carb entry time
codebymini pushed a commit
to codebymini/LoopWorkspace
that referenced
this pull request
Sep 2, 2025
threecee pushed a commit
to threecee/LoopWorkspace
that referenced
this pull request
May 2, 2026
Picks up: - OmniBLE (b5-start..HEAD, 4 commits): commandsAllowed gate flag on OmniBLEOwnership + callback-based gate in OmniBLEPumpManager enactBolus/enactTempBasal (#1); claimedOwner field on PhoneWatchHeartbeat + PhoneWatchProtocol.currentVersion 2 -> 3 (LoopKit#5); HandoffStatePersistence + state machine restore-on-init (LoopKit#4); command-gate integration sub-test in WatchAlgorithmEndToEndTests. - Loop (b5-start..HEAD, 4 commits): orchestrator effect cases wire ownership.commandsAllowed flips on both iOS + watch (#1); both PhoneWatchSessionCoordinators populate claimedOwner in heartbeats + watch demotes itself silently on split-brain detection (LoopKit#5); watch pump manager construction reads from WatchSettingsCache instead of .watchSideDefault (LoopKit#7); WatchDoseRecoveryStore tripwire in WatchAlgorithmKit + ExtensionDelegate launch-time stale check; test pollution fix on iOS + watch HandoffOrchestratorTests + SettingsSyncEmissionTests + protocol version assertion bump. After this, the audit gaps from the B.4 critical review are closed: - Real protection during handoff transitions (no mid-handoff double- command races) - Split-brain detection with phone-wins arbitration - Crash recovery via persistence (HandoffState survives app restart) + dose-in-flight tripwire on watch - Watch pump manager uses real settings, not hardcoded defaults BEHAVIOR: The new gate and split-brain demotion will fire when expected (forced handoffs, transient WCSession glitches). Worth observing on hardware install of next build. Verified: - OmniBLETests targeted: all suites green incl. WatchAlgorithmEndToEndTests 5/5 - LoopTests targeted: all green (HandoffOrchestratorTests 13/13, HandoffPolicyEngineTests 11/11, SettingsSyncEmissionTests 5/5) - WatchApp ExtensionTests targeted: all green (33/33 across 4 suites) - Release archive: ARCHIVE SUCCEEDED (iOS, generic destination) threecee-claude
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello
Really your branch which support MicroBolus，could you pls add Bubble support so I can get CGM data from Libre directly.
Now I am using the NightscoutAPIClient,but it's online solution.
I have finished the https://github.com/bubbledevteam/bubble-client-swift,so you can use it as your needs