dangernoodle-io / breadboard / 26002222338

100%

push

github

fix(bb_event_ring): right-size replay snapshot to ring->max_entry (#256)

The snapshot allocation in bb_event_ring_subscribe_with_replay was over-allocating
by 2x due to a fixed 512-byte payload per entry. With the default ring capacity of
16 and max_entry of 256, this consumed 8384 bytes (16 × 524) instead of the optimal
4288 bytes.

Replace the fixed-size snapshot_entry_t with a flat two-buffer design:
- snapshot_header_t array: metadata only (id, size)
- Separate payload buffer: sized to capacity × max_entry

Byte allocation before: 16 capacity × (4 bytes id + 8 bytes size + 512 bytes payload) = 8384 bytes
Byte allocation after: (16 × 12 bytes headers) + (16 × 256 bytes payloads) = 4288 bytes
Reduction: ~49% smaller allocation, fixes OOM on constrained devices (bitaxe-650).

Adds tests for the second-alloc-failure cleanup branch and for the
bb_event_routes_attach_ex retained=true path (the latter introduced in #255
and uncovered until this PR exercised it).

Fixes B1-203.

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

958 of 958 branches covered (100.0%)

Branch coverage included in aggregate %.

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

1717 of 1717 relevant lines covered (100.0%)

1018.51 hits per line