Skip to content

ported 56ff low level protocol from android#32

Open
saksham2001 wants to merge 3 commits into
mainfrom
android-lowlevel-migration
Open

ported 56ff low level protocol from android#32
saksham2001 wants to merge 3 commits into
mainfrom
android-lowlevel-migration

Conversation

@saksham2001

@saksham2001 saksham2001 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Summary

Migrates the low-level (non-UI) features the Android port (foureight84/PulseLoopAndroid) added, into PulseLoop iOS. The iOS architecture already had most of the seams (profile-sync, capability gating, per-driver sync engines), so this is mostly implementing the jring/56ff side of existing protocols and adding new metric kinds.

  • Metrics added: Blood pressure, Fatigue, Stress, Blood sugar, and HRV
  • Fixed mislabeled opcodes
  • Added calibration commands and personal data (required for calibration)
  • Added connection reliability and initial setup

Related issues

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 📟 New / improved wearable support (BLE driver layer)
  • 🤖 Coach / LLM change (tools, prompts, orchestration)
  • 🎨 UI / DesignSystem change
  • 🧹 Refactor / chore (no behavior change)
  • 📝 Docs only
  • ⚠️ Breaking change (existing data, settings, or APIs change)

How was this tested?

Build succeeds, 186 tests pass (49 new), 0 serious lint violations. Changes are staged, not committed.

  • Added / updated unit tests (PulseLoopTests)
  • Ran the test suite locally (⌘U in Xcode)
  • Tested on a physical device with a real ring — model:
  • Tested with demo data (-seedDemo YES, no hardware)
  • N/A (docs / non-code change)

Privacy & data

  • This change does not send health data off-device without explicit user action.
  • No secrets, API keys, or personal data are committed.
  • N/A

Screenshots / recordings

Checklist

  • My code follows the project's style (SwiftLint passes).
  • I ran the tests and they pass.
  • I updated docs / README where relevant.
  • I read the Contributing guide.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant