dangernoodle-io / breadboard / 25265154476

100%

push

github

fix(bb_manifest): balance brace count in response schema literal (B1-136) (#180)

bb_manifest_register.c L54 had 25 `{` vs 23 `}` in the response
schema literal. cJSON_Parse failed; bb_json_obj_set_raw silently
substituted JSON null. Live `/api/openapi.json` shipped with
`paths."/api/manifest".get.responses."200".content."application/json".schema`
= null.

Three changes:

1. bb_manifest_register.c — append the two missing `}`. Validate
   externally:
     awk '...' | python3 -c 'import json; json.loads(...)'
2. bb_json_obj_set_raw — log via ESP_LOGE / fprintf on
   cJSON_Parse failure with the offending key + offset. Silent
   substitution is what hid this; loud substitution surfaces the
   next regression on the boot log.
3. test_route_schemas.c (new host test) — five cases covering:
   - the manifest schema fixture parses
   - the registry walker validates every schema in the registry
   - the walker correctly flags malformed JSON
   - set_raw writes JSON null on parse failure
   - set_raw preserves valid JSON

The walker is reusable for future descriptor-bearing components;
add a fixture here when a new hand-authored schema literal lands
in production code.

371 tests pass (was 366 + 5 new).

336 of 336 branches covered (100.0%)

Branch coverage included in aggregate %.

557 of 557 relevant lines covered (100.0%)

483.9 hits per line