dangernoodle-io / breadboard / 28725981588

100%

push

github

fix: bound bb_health_stack task table with mark-and-sweep (#718)

- keys the debounce table by task name via mark-and-sweep instead of insert-only, so short-lived pool tasks (sse_N) no longer leak a permanent slot per name and eventually wedge the table full
- each poll tags every live task's entry with the current scan tick (mark), then frees any entry whose tick doesn't match (sweep)
- extracts the mark/sweep decision into pure, host-testable helpers (bb_health_stack_table_mark / bb_health_stack_table_sweep) mirroring the snapshot/evaluate/writeback split used by bb_task_registry's sw-wdt checker
- adds a grace window (BB_HEALTH_STACK_SWEEP_GRACE) so an entry is only swept after being missed for >1 consecutive scans, since uxTaskGetSystemState can transiently omit a live task between the count call and the snapshot; a grace-held entry's low state is left untouched
- adds host tests covering insert-on-first-sight, refresh-not-duplicate, sweep-on-disappearance, no-wedge-after->max distinct transient task names, and the sweep grace window (single-miss survival with low preserved, two-miss sweep boundary)

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

2358 of 2358 branches covered (100.0%)

Branch coverage included in aggregate %.

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

3866 of 3866 relevant lines covered (100.0%)

1211.16 hits per line