dangernoodle-io / breadboard / 30053990695

97%

push

github

refactor: drive /api/diag/tasks via bb_serialize descriptor (#1035)

Migrates tasks_get_handler's hand-streamed bb_http_resp_json_obj_*
emitter to a bb_serialize descriptor rendered via
bb_http_serialize_stream() (B1-1191, largest B1-1054 conversion).

- new bb_diag_tasks_get_wire_priv.h/.c: unconditional row descriptor
  (13 fields, 8 present-gated via 4 precomputed row-local predicate
  flags) + nested "registry" object, streamed via BB_ARR_STREAM/
  bb_serialize_arr_stream_from_buf (single-pass, no fixed row cap --
  uxTaskGetNumberOfTasks() gives the exact count up front)
- tasks_get_handler rewritten: the two Kconfig gates
  (CONFIG_FREERTOS_VTASKLIST_INCLUDE_COREID/
  CONFIG_FREERTOS_GENERATE_RUN_TIME_STATS) stay in the ESP-IDF-only
  gather, resolved to plain (bool, int64_t) pairs before the portable
  fill fns -- the descriptor itself carries zero #if CONFIG_*
- co-located BB_SERIALIZE_META_HOST meta table + golden test proving
  byte-fidelity against the unchanged hand-authored
  s_tasks_get_responses[] OpenAPI literal
- host tests cover all 16 present/absent predicate combinations
  (proving the present-predicate mechanism works inside a
  BB_ARR_STREAM row) plus a multi-row mixed-presence render test

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

7524 of 8121 branches covered (92.65%)

Branch coverage included in aggregate %.

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

12789 of 12951 relevant lines covered (98.75%)

2505.46 hits per line