dangernoodle-io / breadboard / 27930867513

100%

push

github

feat: add AUTOREGISTER opt-in path for bb_temp, bb_mqtt_info, bb_led_info satellites (B1-347) (#526)

adds CONFIG_BB_TEMP_AUTOREGISTER, CONFIG_BB_MQTT_INFO_AUTOREGISTER, and
CONFIG_BB_LED_INFO_AUTOREGISTER (all default n, behavior-neutral) so the
three manual-register satellites can self-register via bb_registry without
requiring an explicit call in app_main.

- new Kconfig file per component with bool option, default n, help documenting
  the auto XOR manual rule (enabling both double-registers the section)
- each ESP-IDF .c file gains a BB_REGISTRY_REGISTER_N init fn (order 1)
  wrapped in #if CONFIG_BB_<X>_AUTOREGISTER; the manual fn is kept as-is
- each CMakeLists.txt adds bb_registry to PRIV_REQUIRES and a conditional
  bb_registry_force_register call
- CLAUDE.md: new "Satellite / extender opt-in convention" section documenting
  the three tiers and the naming split (_AUTOREGISTER vs _AUTO_ATTACH)
- bb_display_info already uses CONFIG_BB_DISPLAY_INFO_AUTO_ATTACH; confirmed
  and documented as the one legacy exception — no behavior change
- default n is required: consumers currently call the manual fns, so default y
  would double-register; flipping to default y + removing manual calls from
  TaipanMiner is a cross-repo follow-up, not part of this PR
- make test: 2432/2432 passed (registry wiring is ESP-IDF-only; host count
  unchanged)


Claude-Session: https://claude.ai/code/session_01Ly8KysyxqixYGEbgtabjMN

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

1760 of 1760 branches covered (100.0%)

Branch coverage included in aggregate %.

2889 of 2889 relevant lines covered (100.0%)

1214.81 hits per line