fix: reconnect retry for unexpected BLE disconnects #8

Merged

payfrit merged 1 commit from schwifty/reconnect-on-unexpected-disconnect into main

2026-03-22 00:17:30 +00:00

schwifty commented

2026-03-21 23:54:27 +00:00

Collaborator

Summary

Adds reconnect-retry logic (2 attempts with backoff) for disconnects during .authenticating or .writing states, instead of failing immediately
Resets passwordIndex on reconnect so re-auth starts from password #1 (fixes @koda's catch where burned password attempts caused retry failures)
Also fixes missing passwordIndex reset in the device-info safety-net reconnect path

What was happening

Any BLE disconnect during authentication or early config writing hit the generic "Unexpected disconnect" handler which failed immediately with no retry. On flaky beacons this killed the entire provisioning.

Test plan

Build from this branch and provision a beacon that has flaky BLE
Verify provisioning recovers from 1-2 disconnects during auth/write
Verify it still fails cleanly after 2 retries exhausted

🤖 Generated with Claude Code

## Summary - Adds reconnect-retry logic (2 attempts with backoff) for disconnects during `.authenticating` or `.writing` states, instead of failing immediately - Resets `passwordIndex` on reconnect so re-auth starts from password #1 (fixes @koda's catch where burned password attempts caused retry failures) - Also fixes missing `passwordIndex` reset in the device-info safety-net reconnect path ## What was happening Any BLE disconnect during authentication or early config writing hit the generic "Unexpected disconnect" handler which failed immediately with no retry. On flaky beacons this killed the entire provisioning. ## Test plan - [ ] Build from this branch and provision a beacon that has flaky BLE - [ ] Verify provisioning recovers from 1-2 disconnects during auth/write - [ ] Verify it still fails cleanly after 2 retries exhausted 🤖 Generated with [Claude Code](https://claude.com/claude-code)

schwifty added 1 commit 2026-03-21 23:54:27 +00:00

fix: add reconnect retry for unexpected disconnects during auth/write df6601c50b

Instead of immediately failing on disconnect during authenticating or
writing states, retry up to 2 times with backoff. Resets passwordIndex
on reconnect so re-auth starts fresh (fixes issue where burned password
attempts caused retry failures). Also fixes passwordIndex reset in the
device-info safety-net reconnect path.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>