dangernoodle-io / breadboard / 28420440078

100%

push

github

fix(bb_net_health): classify heap_state from existing free-heap; gate extra heap walks to HEAPTRACE (B1-433) (#608)

- eval_work_fn previously called three heap_caps walks unconditionally
  every tick: bb_board_heap_free_total(), bb_board_heap_minimum_ever(),
  and bb_board_heap_largest_free_block()
- heap_state classification only needs free-heap; g_min and g_largest
  are only referenced in the BB_NET_HEALTH_HEAP_TRACE log line
- move g_min and g_largest declarations + calls inside the
  #if BB_NET_HEALTH_HEAP_TRACE block so they compile to zero instructions
  on the default (HEAPTRACE-off) path
- unconditional path now makes exactly one heap_caps call (g_free),
  eliminating the stack pressure that caused multi_heap_internal_unlock
  assert on the single-core C3 (bb_timer_disp stack overflow)
- heap_state classification and bb_net_health_set_heap_state remain
  unconditional; /api/health heap_state field stays populated on all boards

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

1839 of 1846 branches covered (99.62%)

Branch coverage included in aggregate %.

3165 of 3177 relevant lines covered (99.62%)

1240.15 hits per line