fix: broaden disconnect retry to all active provisioning phases #9

Closed

schwifty wants to merge 0 commits from schwifty/broaden-disconnect-retry into main

schwifty commented

2026-03-22 00:23:16 +00:00

Collaborator

Problem

PR #8 added disconnect retry for .authenticating and .writing states, but disconnects during .discoveringServices (characteristic discovery) and .verifying (broadcast check) still immediately hard-fail with no retry.

Fix

Extended the retry logic to cover all active provisioning phases: .discoveringServices, .authenticating, .writing, and .verifying. Same reconnect + backoff behavior applies.

Changes

BeaconProvisioner.swift: Replaced 2-state check with isActivePhase covering all 4 active states

🤖 Generated with Claude Code

## Problem PR #8 added disconnect retry for `.authenticating` and `.writing` states, but disconnects during `.discoveringServices` (characteristic discovery) and `.verifying` (broadcast check) still immediately hard-fail with no retry. ## Fix Extended the retry logic to cover **all active provisioning phases**: `.discoveringServices`, `.authenticating`, `.writing`, and `.verifying`. Same reconnect + backoff behavior applies. ## Changes - `BeaconProvisioner.swift`: Replaced 2-state check with `isActivePhase` covering all 4 active states 🤖 Generated with [Claude Code](https://claude.com/claude-code)

schwifty added 1 commit 2026-03-22 00:23:17 +00:00

fix: broaden disconnect retry to cover all active provisioning phases 28aefd1bdf

Previously retry logic only caught disconnects during .authenticating
and .writing states. Beacons can also drop during .discoveringServices
(characteristic discovery) and .verifying (broadcast check), which
would bypass retry and immediately hard-fail.

Now all active provisioning phases get the same reconnect retry
treatment with backoff delays.

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