dangernoodle-io / breadboard / 27393296277

100%

push

github

fix(bb_display_info): defer health.display attach to registry order 4 (#393)

- bb_display_register_info() is called early in app_main, before the
  regular-tier registry walk runs bb_event_routes_init; calling
  bb_event_routes_attach_ex there returns ESP_ERR_INVALID_STATE (259)
  because bb_event_routes has not yet set s_cfg.initialized = true
- matches the bb_update_check / bb_ota_hooks pattern: move the
  bb_event_routes_attach_ex call (and the initial retained snapshot
  publish) into a new bb_display_info_register_init registered at
  order 4, which runs after bb_event_routes at order 0
- bb_display_register_info() still calls bb_info_register_extender_ex
  (must happen before bb_info_init freezes extenders) and
  bb_event_topic_register (topic must exist for runtime posts), and sets
  s_registered=true so the deferred init can gate itself safely
- add bb_registry to PRIV_REQUIRES and bb_registry_force_register to
  CMakeLists.txt (guarded by CONFIG_BB_DISPLAY_INFO_AUTO_ATTACH) to
  prevent garbage-collection under PlatformIO
- sibling check: bb_led_info and bb_temp have no bb_event_routes_attach
  calls; bb_health_stack_monitor_init is called from bb_health_init
  (registry order 1) so its attach is already deferred correctly;
  bb_diag and bb_ota_hooks attach inside their registry inits — no other
  inline-attach-before-routes-init bugs found

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

1665 of 1665 branches covered (100.0%)

Branch coverage included in aggregate %.

2772 of 2772 relevant lines covered (100.0%)

1002.01 hits per line