Alan-Jowett / sonde / 26115437770
82%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 129
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.921% (-0.02%) from 82.936%
26115437770

push

github

web-flow 
fix: reconcile node spec drift — PEER_REQUEST model, storage, boot paths, and test coverage (#995)

* fix: reconcile PEER_REQUEST keying and forwarding model in node specs

Fixes #980 — three spec-drift findings from maintenance audit:

F-ND-011 (Critical): ND-0909 and node-design.md §15.7 incorrectly
described the node as building/encrypting the PEER_REQUEST frame.
The authoritative protocol doc (ble-pairing-protocol.md §7.1) is
clear: the phone builds the complete frame, and the node stores and
forwards it verbatim. Updated both to match.

F-ND-012 (High): ND-0916 listed \phone_psk\ and \phone_key_hint\ as
NVS keys, but the node doesn't need them — it just forwards a
pre-built blob. Removed both; NVS layout now matches §8.4.

F-ND-013 (High): ble-pairing-protocol.md §8.3.1 and node-design.md
described self-healing as unconditional, but ND-0915 correctly
specifies a \peer_payload\ existence guard (clearing \eg_complete\
without a payload to resend is a dead end). Added the guard to
§8.3.1 and node-design.md §15.7 to match the requirement.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: credential storage model and boot path contradictions in node specs

Fixes #981 — three spec-drift findings from maintenance audit:

F-ND-008 (High): ND-0400 said PSK stored in a \dedicated flash
partition\, but code, tests, ND-0906, and ble-pairing-protocol.md §8.4
all use NVS. Updated ND-0400 to say NVS consistently.

F-ND-009 (High): T-N100 step 4 asserted the unpaired node enters deep
sleep, but ND-0900 boot path 2 and the code enter BLE pairing mode.
Updated T-N100 to expect BLE pairing mode.

F-ND-010 (High): T-N914 tested a retired \egistration_proof\ field.
Under AES-256-GCM, successful AEAD decryption constitutes proof — there
is no separate field. Removed T-N914 (AEAD failure already covered by
T-N913) and cleaned up the traceability matrix.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: document ... (continued)

37710 of 45477 relevant lines covered (82.92%)

249.01 hits per line

Coverage Regressions

Lines Coverage ∆ File
7
81.12
 -0.28% crates/sonde-azure-companion/src/main.rs
Jobs
ID Job ID Ran Files Coverage
1 26115437770.1 129
82.92
 GitHub Action Run
Source Files on build 26115437770