Tidepool sync 2026-05-11#450
Conversation
- Point all 18 submodule URLs to https://github.com/loopkitdev/* forks - Minizip and TrueTime.swift retain LoopKit origin (no forks) - Add docs/tidepool-sync-2026-03-10.md (Tidepool sync documentation)
…ce docs - Update all 17 submodule commit pointers to reflect rewritten history (filter-branch changed SHAs when fixing author to LoopKit Developer) - Add Loop build fix commit (LoopDataManager init ordering) - Add LOOPKIT_SYNC_PROCESS.md, SYNC_PROGRESS.md, sync-docs/
Catches the superproject up to all submodule work that has accumulated
on tidepool-sync/2026-03-10 since the initial sync commit, including:
- Loop: restore manual dose entry "+" button on insulin delivery screen
- LoopKit, NightscoutService, TidepoolService, LibreTransmitter:
string catalog updates from Xcode build
- OmniKit: fix reentrant lock crash in isSignalLost
- Various submodules: merge upstream/{dev,main} into tidepool-sync branch
Parallel fix to OmniKit 924f10d — OmniBLE had the identical bug where isSignalLost() read state.lastPumpDataReportDate from inside a lockedState.mutate closure, causing recursive lock acquisition.
Tier 1 - LoopKit: bd30c463 (409 commits from tidepool/dev; 18 source conflicts + 19 pbxproj regions resolved; DIY divergence preserved at BasalRateScheduleEditor) - Loop: 76b6b1e3 (14 commits from tidepool/dev; 3 pbxproj regions) Tier 3 - CGM drivers - CGMBLEKit: 69562e7 - G7SensorKit: d024513 - dexcom-share-client-swift: 541de2f - NightscoutRemoteCGM: b1ea9ee - LibreTransmitter: c99daf1 Tier 3 - Pump drivers - RileyLinkKit: 19f5ae8 - OmniKit: b3b6080 (preserves reentrant lock fix; adopts mutateState) - OmniBLE: 645e0fc (preserves reentrant lock fix and Pod Keep Alive suspend special case; adopts mutateState) - MinimedKit: f994d6e (preserves CAGE/IAGE tracking) Tier 3 - Services - TidepoolService: 5f6a064 (incl. DoseEntry dedupe follow-up) - NightscoutService: 1b5cded (preserves APNS response feature) - AmplitudeService: 77dae3e - LogglyService: 8e18081 - MixpanelService: unchanged (already up to date with tidepool) Tier 3 - Support/Onboarding - LoopSupport: a312dfb - LoopOnboarding: fd7e410 Package.resolved - LoopAlgorithm: tidepool-org/LoopAlgorithm bd1a879 (4 test-only commits)
docs/tidepool-sync-2026-05-11.md describes the merge, conflicts, and divergences. SYNC_PROGRESS.md updated to reflect the new branch and the divergences carried forward.
Pulls in 3 small origin/dev updates that landed after this sync started: - Loop: PR #2410 Live activity plot fixes (manual merge to preserve our post-Tidepool-sync API: kept async/await sample fetch + LoopUnit signature, adopted upstream's presetStart<=presetEnd guard and the adjustedChartStart/End helpers) - CGMBLEKit: PR LoopKit#211 handle inactive CGM sensor sessions (clean merge) - OmniBLE: Lokalise translation update (clean merge)
… charts Picks up loopkitdev/Loop@4eea0b24 — widens the carbStatus.dynamicGlucoseEffects sampling window in fetchCarbAbsorptionReview and getHistoricalChartsData from `from: end` to `from: start` so the model-predicted line spans the full chart again (was only generating samples in now+1h … now+7h, outside the visible window). UI-only; no dosing-path impact.
Bring in dev's workspace changes (CI workflows, Gemfile, lokalise script, VersionOverride). Submodule gitlinks kept at the tidepool-sync/2026-05-11 tips (the unmerged sync branches), to be re-bumped as submodule PRs land.
TestI have been testing this from the loopkitdev repository over the last few weeks.
ConfigurationMost recent test is with LoopWorkspace commit 68ce294. This is behind but still valid for the items I tested. Phone is SE 3rd gen running iOS 26.5. Review prior issues with 213Referring to the table in PR #213 which listed issues with the previous PR. There were 10 items.
Edited to update Item 1: there was a message about invalid Siri Support - Localized description for a few phrases in my email from Apple. Note about preset changeThe presets affects active insulin during the preset duration - not throughout the duration of the insulin lifetime. When you choose a preset that has 170% insulin needs or greater, the new code adds a new safety limit. Instead of using your glucose safety limit and your target range, it limits the lower bound to be 110 mg/dL. See this table. Notice that the recommended dose for a fixed carb entry of 50 g for insulin needs of 200% is less than 150% because the modified minimum threshold (from 70 mg/dL to 110 mg/dL). All other settings used glucose safety of 70 mg/dL. The target thresholds for all value are the same (110-120 mg/dL).
|
Document that upgrading dev→sync migrates the shared store to Modelv6 and cannot be reverted: dev ships only up to Modelv4 and can't open a v6 store, so reverting leaves the app unable to load its data store (no on-disk data loss, but dev needs a delete+reinstall, rebuilding from HealthKit). Also records that the forward migration now preserves dose amounts (value→deliveredUnits). Includes the earlier 2026-03-10 LoopAlgorithm-packaging callout.
TestNot all of the updates listed below were tested in this comment. Summary of recent updatesA few more changes came in after my last test:
ConfigurationiPhone SE 3rd gen running iOS 26.5
Events and Report from 3.14.0Loop_v3.14.0_Export-20260521T163630Z.zip Loop_v3.14.0_Loop Report 2026-05-21 093645-0700.md Test Transition from Loop 3.14 to 3.15.0Build LoopWorkspace tidepool-sync/2026-05-11, commit 14d1987
Events and Report from 3.15.0Loop_v3.15.0_Export-20260521T163957Z.zip Loop_v3.15.0_Loop Report 2026-05-21 094014-0700.md Screenshotsmain and setting screen for 3.14.0 immediately before transition
main and setting screen for 3.15.0 immediately after transition
Browser Build Test✅ The browser build succeeded again. This time there were no warning messages about some items not being supported by Siri. |
|
Thanks @marionbarker ! Also did you see the new Health page in settings? |
Test Live ActivityPart of the testing from #450 (comment) The Live Activity shows up, but the button to always allow was really had to see. I knew it was there so I kept hitting it until I hit the right spot.
|
Repoint workspace LoopAlgorithm package to LoopKit/LoopAlgorithm (main) and bump LoopKit pin.
TestThe test build includes up through commit fc9293e from tidepool-merge/2026-05-11. This was a fresh build of an independent Loop-TP app on my looping phone (different bundle ID) ConfigurationiPhone 15 pro running iOS 26.5 (my personal Looping phone)
Build an independent version on the same phone
Allow Loop-TP (test build) to run in parallel with Loop (app controlling my pump). Attempt to remember to match manual entries of carbs on the test Loop-TP app. |
Test✅ Side-by-side comparison of Loop with G7/Pod with Loop-TP (3.15) with G7/Simulated Pump on same phone looks qualitatively the same
ConfigurationSee #450 (comment) for configuration Morning CoffeeI entered 10 g for 1 hour for my morning coffee. Both Loop and Loop-TP recommended the same 1.15 U dose. Overnight ReviewCompare a few screenshots. Lock ScreenBoth Loop (upper plot) and Loop-TP (lower plot) show the same Live Activity on the lock screen
NightscoutThe graphic on the left is my real Nightscout site and the one on the right is the test site. Because I was previously running a glucose trace uploaded to the test site, the vertical scaling is different. But qualitatively the dosing response is the same.
G7 Sensor transitionThere are 2 grey dots visible in the Loop-TP nightscout site.
|





Refreshed Tidepool → DIY sync for the whole workspace, from the
tidepool-sync/2026-05-11branch. Supersedes and replaces the previous Tidepool merge PR (#213).See Tidepool → LoopKit DIY Sync — 2026-03-10 and Tidepool → LoopKit DIY Sync — 2026-05-11 for more details
Submodule gitlinks currently point at the unmerged per-repo
tidepool-sync/2026-05-11branches. They will be updated to the merged commits as each submodule PR below lands.Submodule PRs
Notes
devwas merged into the Loop sync branch to resolve conflicts (WatchAppInfo.plistLive Activity launch keys +project.pbxprojlocalization file references). Loop builds clean.