dangernoodle-io / breadboard / 25082005642

100%

push

github

feat(bb_registry): migrate bb_mdns init to auto-registration via bb_registry (#129)

Add CONFIG_BB_MDNS_AUTOREGISTER Kconfig flag (default-on) that gates
auto-initialization of bb_mdns through bb_registry. The public bb_mdns_init()
function remains available and is now called by the registry init wrapper,
preserving the API for consumers with AUTOREGISTER=n. The setters
(bb_mdns_set_hostname, bb_mdns_set_service_type, bb_mdns_set_instance_name)
are still caller-driven, allowing pre-init configuration before bb_registry_init().

Phase 2.4 completion: all 11 registry-enabled components now migrated.

Changes:
- components/bb_mdns/Kconfig: new CONFIG_BB_MDNS_AUTOREGISTER (default y)
- components/bb_mdns/CMakeLists.txt: add bb_registry, bb_http to PRIV_REQUIRES;
  register force_register when AUTOREGISTER enabled
- platform/espidf/bb_mdns/bb_mdns.c: add thin registry init wrapper calling
  bb_mdns_init(); register via BB_REGISTRY_REGISTER macro
- examples/esp32-wroom-32/main/main.c: remove explicit bb_mdns_init() call
- examples/esp32-wroom-32/sdkconfig.defaults: add CONFIG_BB_MDNS_AUTOREGISTER=y
- CLAUDE.md: update registry section to list bb_mdns; clarify that mdns setters
  remain caller-driven but init is now registry-controlled
- README.md: update components table and Kconfig flags table

Smoke builds (esp32-wroom-32, elecrow-p4-hmi7) pass. Host tests: 301/301 pass.

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

272 of 272 branches covered (100.0%)

Branch coverage included in aggregate %.

431 of 431 relevant lines covered (100.0%)

147.89 hits per line