dangernoodle-io / breadboard / 27105944470

100%

push

github

fix(bb_power/fan/thermal): route AUTOREGISTER opt-in (default n) to avoid polluting consumers (#361)

Three breadboard HTTP route components auto-registered by default, polluting
every consumer that compiles BB (TaipanMiner compiled all of .breadboard, so
it got /api/power/fan/thermal force-registered, colliding with TM's own
handlers and destabilizing the firmware). Make them opt-in (default n),
matching the hardware-specific satellite pattern.

## Changes
- Flip CONFIG_BB_POWER_ROUTES_AUTOREGISTER default from y to n
- Flip CONFIG_BB_FAN_ROUTES_AUTOREGISTER default from y to n
- Flip CONFIG_BB_THERMAL_AUTOREGISTER default from y to n
- Update each Kconfig help text to note the opt-in pattern
- Update CLAUDE.md power/fan/thermal sections to document opt-in behavior

## Verification
- make test: 1434 tests passed
- make smoke-esp32: SUCCESS (sdkconfig still explicitly sets =y to exercise routes)
- make smoke-tdongle: SUCCESS (sdkconfig still explicitly sets =y to exercise routes)

## Todo (for tracking)
- [x] Flip 3 Kconfig defaults (y → n)
- [x] Update help text for each config (opt-in note)
- [x] Update CLAUDE.md sections for power/fan/thermal
- [x] Run make test and report total
- [x] Run make smoke-esp32 and verify routes still register
- [x] Run make smoke-tdongle and verify routes still register
- [x] Grep-confirm 3 defaults are now n
- [x] Create commit on jae/bb-routes-optin

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

1520 of 1520 branches covered (100.0%)

Branch coverage included in aggregate %.

2566 of 2566 relevant lines covered (100.0%)

960.91 hits per line