dangernoodle-io / breadboard / 28675338100

100%

push

github

feat(bb_nv): tag factory-reset reboot with FACTORY_RESET reason (B1-532) (#694)

- persist a reboot record before esp_restart() in factory_reset_reboot_work_fn
  via bb_nv_reboot_record_save(BB_RESET_SRC_FACTORY_RESET, NULL, 0, uptime_s)
- uptime_s computed via bb_clock_now_ms64()/1000, matching bb_system's convention
- bb_clock.h is already public in bb_core, which bb_nv already REQUIRES, so no
  new component dependency was needed
- BB_RESET_SRC_FACTORY_RESET already existed in the enum (added in PR1); no
  enum change or new host test required — the wire string round-trip was
  already covered by test_bb_reset_source_str_factory_reset
- removes the TODO(B1-527) placeholder — this closes the last un-migrated
  esp_restart() site and completes the reboot-reason SSOT
- no bb_nv->bb_system edge introduced; persistence stays bb_nv->bb_core only
- log a bb_log_w warning (mirroring bb_system_restart_reason_at's idiom) if the
  reboot record persist fails, without blocking the reboot on that failure
- update CLAUDE.md's reboot-reason SSOT section to reflect the completed
  migration instead of the now-stale "deferred exception" framing

2260 of 2260 branches covered (100.0%)

Branch coverage included in aggregate %.

3782 of 3782 relevant lines covered (100.0%)

1201.59 hits per line