dangernoodle-io / breadboard / 28674640054

100%

push

github

refactor(bb_core): move reboot-reason codec to bb_core + bb_nv_reboot_record_save; bb_system delegates persist (B1-532) (#693)

- move the pure reboot-reason codec (bb_reset_source_t, BB_RESET_SRC_LIST,
  bb_reboot_record_t/_encode/_decode, bb_reboot_pick_epoch, reboot history
  ring) verbatim from bb_system into a new components/bb_core/include/
  bb_reboot_reason.h + components/bb_core/src/bb_reboot_reason.c; bb_core
  stays a leaf (no new REQUIRES, no esp_/cJSON/bb_json includes)
- bb_system.h re-exports bb_reboot_reason.h so all 17 existing includers,
  bb_diag, and tests see the same types unchanged
- add bb_nv_reboot_record_save(src, detail, epoch_s, uptime_s) to bb_nv,
  a portable helper (bb_nv_set_str + bb_reboot_record_encode are already
  identical on host/ESP-IDF) that owns the NVS persist previously inlined
  in bb_system_restart_reason_at
- bb_system_restart_reason_at now delegates persist to bb_nv, keeping the
  same epoch/uptime computation and esp_restart() call — wire format and
  runtime behavior unchanged, byte-identical
- bb_diag_event_priv.h/bb_diag_event_common.c/bb_diag_routes.c include
  bb_reboot_reason.h directly instead of relying solely on the re-export
- update scripts/native_scaffold.py host source lists to match the move
- add bb_nv_reboot_record_save round-trip host tests

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

2260 of 2260 branches covered (100.0%)

Branch coverage included in aggregate %.

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

3782 of 3782 relevant lines covered (100.0%)

1201.59 hits per line