dangernoodle-io / breadboard / 28454904692

100%

push

github

fix(bb_pub): move telem_buf off the bb_pub task stack to static (C3/S3 stack overflow from SERIALIZE_MAX=1024) (#609)

- char telem_buf[1024] in bb_pub_tick_once Phase 2b was stack-allocated,
  adding 1 KB to the bb_pub task's frame after #603 raised SERIALIZE_MAX
  512 → 1024
- on c3-supermini and tdongle-s3 (tighter stacks) this pushed bb_pub over
  the edge: bb_health_stack reported "task 'bb_pub' stack low: 20/120 bytes
  free" within ~2 min, followed by panic-loop; wroom32 (.81) survived
- static is safe: bb_pub_tick_once runs under the single-worker guarantee
  (documented at buffer_capture ~L353: "only one worker thread ever calls
  it concurrently"); no re-entrancy risk
- serialize_max stays at 1024 so dashboard /api/stats serialization is
  unchanged; only the stack frame shrinks

validated: 2886/2886 host tests pass; all 3 boards (wroom32/c3/s3)
clean-rebuilt with serialize_max=1024 confirmed in sdkconfig; .10 and .32
stable at 10+ min uptime, zero wdt_resets, no stack-low warnings, no
panic; /api/stats 200 on all 3; /api/diag/boot wdt_resets=0 on all 3

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