dangernoodle-io / breadboard / 26012009247

100%

push

github

fix(bb_nv): defer manifest registration to pre_http tier to avoid stack pressure on tight-RAM boards (B1-214) (#271)

PR #270 panicked tdongle-s3 on boot from RAM pressure. On tdongle-s3,
the main task has a 3584-byte stack and bb_nv_config_init runs in the
EARLY tier on top of the deep nvs_flash_init call chain. Calling
bb_manifest_register_nv (with its strcmp loop) from within that tight
context pushed the stack over the limit.

- extract bb_nv_config_manifest_init() that calls bb_manifest_register_nv
- auto-register it at PRE_HTTP tier via BB_NV_CONFIG_MANIFEST_AUTOREGISTER
  (default y, depends on BB_MANIFEST_AUTOREGISTER); runs after EARLY tier
  completes and the main-task stack unwinds
- bb_nv_config_init() now does NVS I/O only (no manifest call at boot)
- s_bb_cfg_keys[] remains static const rodata — zero heap, zero extra BSS
- update test to call bb_nv_config_manifest_init() explicitly; all 878
  host tests pass

Bench-verified on tdongle-s3 (no PSRAM, ~270K SRAM).

998 of 998 branches covered (100.0%)

Branch coverage included in aggregate %.

1784 of 1784 relevant lines covered (100.0%)

1004.71 hits per line