dangernoodle-io / breadboard / 29690687796

96%

push

github

feat: bb_storage_nvs list_entries + get_stats (NVS enumeration backend) (#942)

Implements PR6 of the bb_storage inventory epic: the espidf bb_storage_nvs
backend's list_entries/get_stats vtable pair (PR5, merged), wired to real
ESP-IDF nvs_entry_find/_next/_info/_release + nvs_get_stats calls.

- ns_or_dir == NULL enumerates all namespaces (nvs_entry_find's NULL-filter
  convention, per PR5's documented contract).
- Loud truncation: *count always reports the true total, even past cap.
- nvs_type_t -> bb_storage_enc_t mapping is a new pure, host-testable helper
  (bb_storage_nvs_classify_nvs_type); unrepresentable types fall back to
  BLOB rather than erroring.
- Per-entry value-length probe is best-effort and non-fatal: a probe
  failure logs a WARN and reports len=0 without aborting the enumeration.
- get_stats reports NVS's native 32-byte-entry-slot accounting (*32),
  documented as an approximation, not exact partition-byte accounting.
- Host testability follows the existing bb_storage_nvs_txn_ops_t seam
  pattern: a new bb_storage_nvs_entry_ops_t (find/next/info/release +
  open/probe_len/close + stats), BB_STORAGE_NVS_TESTING-gated, lets a host
  test drive the find->next->info iteration orchestration against a fake
  without any real NVS backing.


Claude-Session: https://claude.ai/code/session_01VA9XvWjJcbAUGCZpSFfp3H

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

7185 of 7792 branches covered (92.21%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 1 file covered. (100.0%)

12189 of 12366 relevant lines covered (98.57%)

1206.74 hits per line