dangernoodle-io / breadboard / 27527330824

100%

push

github

fix(bb_section,bb_info,bb_sensors): B1-269 hardening — diag section ordering (P0), dup-name + OOM guards, fan PATCH validation, SSOT (telemetry review) (#445)

- bb_info (P0): decouple freeze from bb_info_init (order 2); add bb_info_freeze_init
  (order 20) so bb_diag_routes (order 7) can register the diag section before freeze
- bb_section: reject duplicate section names with BB_ERR_INVALID_STATE on re-register
- bb_section: NULL-child guard in bb_section_build_get (OOM skip instead of crash)
- bb_thermal (host+espidf): OOM guard in emit_source — skip sub-object on malloc fail
- bb_section: validate-before-apply in bb_section_dispatch_patch (all-or-nothing PATCH)
- bb_sensors (autofan PATCH): validate manual_pct/min_pct ∈ [0,100],
  die_target_c/vr_target_c > 0 before applying any field
- bb_pub_health: replace inline ok with bb_health_compute_ok() (SSOT)
- bb_telemetry: freeze after init, MAX_SECTIONS 4→6, bb_log_w on NO_SPACE,
  delegate dispatch to bb_section_dispatch_patch, assemble_get_schema returns
  fresh allocation (caller-owned)
- bb_manifest: fix values schema {"type":"string"} → {"type":"array","items":{...}}
- tests: dup-name, OOM guard, multi-section PATCH atomicity, mixed null+non-null
  schema_props, P0 diag-before-freeze regression, fan autofan field validation,
  boundary values, atomicity; fix capacity tests to use stable string literals

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

1615 of 1615 branches covered (100.0%)

Branch coverage included in aggregate %.

2702 of 2702 relevant lines covered (100.0%)

1180.48 hits per line