meshcore-client

Commit Graph

Author	SHA1	Message	Date
Strycher	7f2548ca95	feat(#135 ): GPS state query via CMD_OFFBAND_GPS (0xC1) + poll flood fix (#140 ) Offband 0xC1 GPS query: 1-byte request -> RESP 0xC1 ASCII reply (enabled/detected/active/fix/baud/lat/lon/alt_cm/sats/time), parsed into OffbandGpsStatus (lat/lon /1e6). New GPS-status section in the Location dialog (live-fix vs stored banner + coords/sats/alt/time). Validated against RedCreek's feat/gps-state-query test firmware. #140: the per-minute GPS poll now uses the lightweight 0xC1 query instead of CMD_APP_START (which re-walked channels + re-drained messages = the BLE re-init flood); a live fix updates self-lat/lon. Validated on b42 (APP_START ~1/min -> ~0, 0xC1 polls clean every 60s, channel re-walk gone). Gemini (fix-then-ship): atomic completer-claim in _handleOffbandGps to shrink the stale-reply race; full elimination needs a protocol sequence id, but the poll shares in-flight via the re-entrancy guard and the query button is disabled during a request, so concurrent requests don't occur in practice. The hardcoded snackbars Gemini flagged in _editLocation are pre-existing l10n debt, not introduced here; all new strings are localized. Closes #135 Closes #140 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	3 months ago

Author

SHA1

Message

Date

Strycher

7f2548ca95

feat(#135 ): GPS state query via CMD_OFFBAND_GPS (0xC1) + poll flood fix (#140 )

Offband 0xC1 GPS query: 1-byte request -> RESP 0xC1 ASCII reply
(enabled/detected/active/fix/baud/lat/lon/alt_cm/sats/time), parsed into
OffbandGpsStatus (lat/lon /1e6). New GPS-status section in the Location dialog
(live-fix vs stored banner + coords/sats/alt/time). Validated against RedCreek's
feat/gps-state-query test firmware.

#140: the per-minute GPS poll now uses the lightweight 0xC1 query instead of
CMD_APP_START (which re-walked channels + re-drained messages = the BLE re-init
flood); a live fix updates self-lat/lon. Validated on b42 (APP_START ~1/min ->
~0, 0xC1 polls clean every 60s, channel re-walk gone).

Gemini (fix-then-ship): atomic completer-claim in _handleOffbandGps to shrink
the stale-reply race; full elimination needs a protocol sequence id, but the
poll shares in-flight via the re-entrancy guard and the query button is disabled
during a request, so concurrent requests don't occur in practice. The hardcoded
snackbars Gemini flagged in _editLocation are pre-existing l10n debt, not
introduced here; all new strings are localized.

Closes #135
Closes #140

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

1 Commits (d56ca6b0371dc3e6876b72053710dab81517c3f8)