dangernoodle-io / breadboard / 29367333348

96%

push

github

refactor: rehome DELETE /api/nvs to a backend-agnostic bb_storage_http (B1-757) (#868)

- add bb_storage_vtable_t.erase_namespace (optional, NULL -> BB_ERR_UNSUPPORTED)
  and the bb_storage_erase_namespace(backend, ns_or_dir) facade fn; wired for
  the nvs backend (bb_storage_nvs_erase_namespace), left NULL for ram/rtc
- new component bb_storage_http (REQUIRES bb_core bb_storage only, never
  bb_storage_nvs) registers DELETE /api/diag/storage against the bb_storage
  facade -- works for any registered backend (nvs today, ram/rtc/a future
  sdcard) with no new route component per backend
- bb_settings exposes bb_settings_ns_is_wifi_creds() so the wipe_wifi safety
  guard asks the namespace owner instead of copying the "bb_cfg" literal
- delete platform/espidf/bb_nv/bb_nv_delete_routes.{c,h} and remove bb_diag's
  private-include boundary violation (target_include_directories(...
  PRIVATE .../bb_nv), the #include, and the manual bb_nv_delete_routes_init
  call) -- bb_diag no longer reaches into bb_nv's implementation directory
- add the missing 501 status-reason mapping to bb_http_status.c (caught by
  a host test asserting a backend without erase_namespace surfaces 501, not
  a silent 200)


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

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

6917 of 7556 branches covered (91.54%)

Branch coverage included in aggregate %.

12 of 12 new or added lines in 3 files covered. (100.0%)

11610 of 11879 relevant lines covered (97.74%)

2441.48 hits per line