23359021289
82%

Ran 20 Mar 2026 07:25PM UTC

Jobs 1

Files 71

Run time 1min

Badge

Embed ▾

Committed 20 Mar 2026 07:22PM UTC coverage: 82.633%. Remained the same

Build # 23359021289

Build Type

push

github

Committed by

web-flow

Commit Message

fix(ble): use on_notify_tx for confirmation-driven indication pacing (#326)

The modem was clearing waiting_confirm unconditionally every poll cycle
(1 ms) in dvance_indication(), causing burst-sent indications that
overwhelmed NimBLE buffers on phones with longer BLE connection intervals
(7.5-30 ms).  This produced esource exhaustion errors and pairing
failures.

Wire up esp32-nimble's on_notify_tx() callback on the Gateway Command
characteristic to clear waiting_confirm only when the BLE client ACKs
the indication (ATT Handle Value Confirmation).  This ensures true
spec-compliant pacing — one indication per ATT round-trip — regardless
of the client's connection interval (MD-0403).

On timeout or failure, the callback clears the indication queue to avoid
stalling on a disconnected or misbehaving client.

Also adds validation test cases T-0623 (confirmation pacing) and T-0624
(slow client pacing) to modem-validation.md to close the gap that allowed
this regression.

Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Coverage Stats

16525 of 19998 relevant lines covered (82.63%)

49.65 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	23359021289.1	20 Mar 2026 07:25PM UTC	71	82.63	GitHub Action Run

Alan-Jowett / sonde / 23359021289
82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 23359021289

Alan-Jowett / sonde / 23359021289 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 23359021289

Alan-Jowett / sonde / 23359021289
82%

README BADGES
x