dangernoodle-io / breadboard / 28649050485

100%

push

github

fix(bb_diag): register diag routes after bb_event_routes so diag.boot SSE auto-attach succeeds (B1-527 follow-up) (#691)

- bb_diag_routes now registers via BB_INIT_REGISTER_N(bb_diag_routes, bb_diag_routes_init, 4) instead of the implicit order-0 BB_INIT_REGISTER
- guarantees bb_event_routes_init (order 0, sets s_cfg.initialized = true) has run before bb_diag_routes_init's optional CONFIG_BB_DIAG_AUTO_ATTACH block calls bb_event_routes_attach_ex, fixing the "auto-attach failed for 'diag.boot': 259" race exposed by B1-527's added PRIV_REQUIRES bb_ntp perturbing link order
- mirrors the existing bb_ota_check order-4 precedent (same rationale, documented in CLAUDE.md's Registry section)
- no CMake change needed: bb_init_force_register(comp_lib, name) only encodes the linker force-include symbol, not the numeric order, which lives solely in the BB_INIT_REGISTER_N call

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

2226 of 2226 branches covered (100.0%)

Branch coverage included in aggregate %.

3747 of 3747 relevant lines covered (100.0%)

1205.92 hits per line