Repository navigation
The watch loads G7SensorKit at run time - #110
Open
jeremybarnum wants to merge 2 commits into
Open
jeremybarnum wants to merge 2 commits into
jeremybarnum wants to merge 2 commits into
Conversation
On the watch the kit is its own plugin: a watch-only Info.plist names G7SensorWatchPlugin as its principal class and G7CGMManager as its CGM manager, under the keys the phone's plugin uses. G7CGMManager adopts LoopKit's DirectReadCGMManager, and its statusDidChange is that protocol's notification. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Problem: - The G7 told the watch what it needs through a watch-only protocol and its own notification: whether it is searching for its sensor, the note for the user, the sensor's name and its last connection. Solution: - The G7 implements the new CGMManager members. ensureConnectionActiveOrPending() is the former recheckAcquisition(), and isSearchingForSensor is unchanged. shortStatusNote is the watch's note, so on the phone it keeps the default: no note. - The G7 supports status observers, as LibreLoop and the mock CGM do, and tells them only when its status, its search or its note changes. The notification is gone. - Its status now carries lastCommunicationDate: when it last connected to the sensor. On the phone: the status kept with each dosing decision now carries that date, and the status screen refreshes when the G7's status changes, about once per reading. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
For LoopKit/Loop#2655. On the watch the kit is its own plugin: a watch-only Info.plist names
G7SensorWatchPluginand theG7CGMManageridentifier, under the keys the phone's plugin uses.G7CGMManageradopts LoopKit'sDirectReadCGMManager. Nothing changes for the phone. Needs LoopKit/LoopKit#687.🤖 Generated with Claude Code