dangernoodle-io / TaipanMiner / 26761047083

90%

push

github

refactor(webui): stream /api/power + /api/fan via host-tested emit fns (#478)

The runtime JSON for these endpoints was emitted inline in routes.c handlers,
while routes_json.c held DOM build_*_json mirrors that were host-test-only
(never called at runtime) — so the tests validated a copy the device never ran.
That is how the expected-efficiency bug shipped (runtime compute had no host
coverage). This unifies on one streaming path: handler gathers a snapshot, then
delegates to a host-testable emit fn that the tests also drive.

- routes_json.c: build_power_json/build_fan_json (DOM) -> emit_power_json/
  emit_fan_json streaming fns (snapshot + bb_http_resp_json_obj_* API,
  host-compilable, no ESP); ported verbatim from the handlers' inline emit
- routes.c: power_handler/fan_handler gather the snapshot then call emit_*_json
  between obj_begin/obj_end
- tests: power/fan cases rewritten to drive the real emit fns through the
  bb_http_host_capture harness (runtime == tested)
- CLAUDE.md: document the gather+delegate / single-emit-path rule; fix stale
  /api/ota/* route paths -> /api/update/*

Wire format unchanged — byte-identical /api/power + /api/fan verified on a live
bitaxe-601 (only fresh-boot transient values differ). Pilot for the
stats/pool/settings/diag/knot migration.

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

2774 of 3492 branches covered (79.44%)

Branch coverage included in aggregate %.

49 of 49 new or added lines in 1 file covered. (100.0%)

5408 of 5588 relevant lines covered (96.78%)

147813.72 hits per line