dangernoodle-io / breadboard / 29978854260

96%

push

github

refactor: extract bb_sensor domain primitive from bb_sensor_http; absorb + delete bb_thermal (#1018)

- new components/bb_sensor: HTTP-free cross-HAL collector (bb_sensor_{fan,power,thermal}_snapshot + bb_sensor_fan_apply), portable single TU, REQUIRES bb_core, PRIV_REQUIRES bb_fan/bb_power/bb_temp/bb_log
- bb_sensor_http_wire.c thinned to bind+route: gather/apply hooks now call bb_sensor's snapshot getters and copy fields into the wire structs (renaming aux_target_c->vr_target_c at the boundary); bb_sensor_http_dispatch.c and the espidf bb_sensor_http.c are untouched
- bb_thermal deleted entirely (component, espidf/host platform twins, dead bb_thermal_init stub, its unit test file) -- bb_sensor_thermal_snapshot absorbs bb_thermal_collect verbatim
- test/test_host/test_bb_sensor.c absorbs the 10 moved thermal-collect cases plus fan/power gather/apply domain tests, including the 2 regression pins (no-primary present=false; capability-gap UNSUPPORTED->INVALID_STATE); test_bb_sensor_http.c slims to one wire-copy proof per section plus its existing e2e/dispatch coverage; bb_sensor_reset_for_test is the new reset SSOT
- platformio.ini: -DBB_THERMAL_TESTING -> -DBB_SENSOR_TESTING
- riding cleanup: drop the orphaned CONFIG_BB_FAN_ROUTES_AUTOREGISTER=y from examples/smoke/sdkconfig.defaults.esp32-autofan (zero in-tree refs)
- fence: bb_sensor approved as a new component (new_component family); this PR also closes out a latent baseline gap left by S1 (bb_sensor_http was never recorded under its new name after the bb_sensors rename, only passing via the 1:1 rename heuristic) -- pruned bb_sensors/bb_thermal and approved bb_sensor_http alongside bb_sensor, one deliberate baseline edit

Part of the sensor-family consolidation (S2 of 2, depends on the bb_sensors->bb_sensor_http rename); refs B1-676.


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

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

7386 of 7976 branches covered (92.6%)

Branch coverage included in aggregate %.

83 of 83 new or added lines in 2 files covered. (100.0%)

12561 of 12723 relevant lines covered (98.73%)

2493.46 hits per line