dangernoodle-io / breadboard / 28248155320

100%

push

github

fix(bb_version): unify version source + fix /api/update/status truncation (TA-462) (#561)

Two root causes, one PR:

1. cmake/bb_version.cmake reimplemented the full version logic independently and
   was not updated by PR #560, which rewrote scripts/bb_version.py with the new
   format (dev-<tm-ref>-bb-<ref>). As a result /api/info (esp_app_desc.version /
   PROJECT_VER) showed the stale old format (dev-g<sha>[-dirty]-bb-g<sha>[-dirty]).
   Fix: add --emit CLI to bb_version.py (prints bare version to stdout, no log
   line) and rewrite cmake/bb_version.cmake to delegate via execute_process so
   there is ONE implementation. BB_FW_VERSION env override fast-path preserved.

2. bb_update_check_status_t.current was char[24], truncating dev-format versions
   to ~22 chars (observed "dev-f218d41+7634-bb-mai" vs expected "…bb-main").
   The longest dev format "dev-<7sha>+<4hash>-bb-<7sha>+<4hash>" is 36 chars.
   Fix: widen current and latest to char[BB_UPDATE_CHECK_VERSION_BUF] (48). Add
   BB_UPDATE_CHECK_VERSION_BUF constant + static_assert in the emit test.

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

1752 of 1752 branches covered (100.0%)

Branch coverage included in aggregate %.

2924 of 2924 relevant lines covered (100.0%)

1268.34 hits per line