dangernoodle-io / breadboard / 25285151289

100%

push

github

feat(bb_nv): add batched setter API for atomic multi-key commits (B1-165) (#194)

Per-key bb_nv_set_* opens, sets, commits, and closes its own NVS handle.
Consumers that store related multi-field state (e.g. TaipanMiner's
mining_stats_save_lifetime: 3× bb_nv_set_u32 for shares + hashes_lo/hi)
pay 3× the flash-bus time and produce 3× the wear vs one batched commit.
On the share-accept hot path under concurrent flash use (TLS handshake,
OTA), that contention stalled mining_hw long enough to trip the task
watchdog (TA-347).

Add bb_nv_batch_t opaque struct + begin / set_u8 / set_u16 / set_u32 /
set_str / commit. ESP-IDF backend wraps a single nvs_handle_t with a
sticky-error guard; commit issues nvs_commit and closes the handle in one
pass. Arduino backend forwards each set to the per-key API (no flash-bus
contention to amortize on EEPROM, but API parity preserved). Host stubs
validate args and return BB_OK.

10 new host tests cover null-arg, before-begin / after-commit ordering,
sticky-error semantics, and a 3×u32 happy path mirroring TM's call site.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

336 of 336 branches covered (100.0%)

Branch coverage included in aggregate %.

557 of 557 relevant lines covered (100.0%)

495.68 hits per line