dangernoodle-io / TaipanMiner / 26764144337

90%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(webui): stream pool/diag + retire the last DOM JSON builders (#480)

Final phase of unifying webui JSON on one host-tested streaming emit path
(after power+fan, stats+settings). The remaining handlers emitted inline while
routes_json.c held host-test-only DOM mirrors — tests validated a copy the
device never ran.

- routes_json.c: build_pool_json/emit_pool_stats_json, build_diag_asic_json,
  build_diag_bench_json (DOM) -> emit_pool_json/emit_diag_asic_json/
  emit_diag_bench_json streaming fns. pool ports the full nesting verbatim:
  configured{primary{},fallback?}, notify{...,merkle_branches[]}, stats[]
- knot: delete the dead test-only build_knot_json (whole-array DOM mirror);
  build_knot_peer_json stays (it IS the runtime path via knot_emit_peer_cb).
  Its tests now capture the real streaming-array path.
- routes.c: pool/diag_asic/diag_benchmark handlers delegate to the emit fns;
  removed dead overlap_str/hwrite_str locals
- tests: rewritten to the bb_http_host_capture harness driving the real emit
  fns; portable dupstr (no POSIX strdup under -std=c99); every null/sentinel/
  empty branch covered -> routes_json.c emit fns at 100%

No build_*_json DOM mirrors remain (only the live build_knot_peer_json). Wire
format unchanged: /api/pool (full nesting), /api/diag/asic, /api/knot
byte-identical on a live bitaxe-601. Completes the JSON SSOT migration.

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

Coverage Stats

2787 of 3488 branches covered (79.9%)

Branch coverage included in aggregate %.

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

3 existing lines in 1 file now uncovered.

5416 of 5592 relevant lines covered (96.85%)

147708.0 hits per line