dangernoodle-io / breadboard / 28605775983

100%

push

github

feat(bb_sub): add cache-only ingress routing core (#667)

- bb_sub_route(topic, payload, len) routes MQTT-style JSON messages into bb_cache, dynamically registering a passthrough serializer per topic on first sight

- len is a real-length contract: no strlen(payload) fallback for len==0 (payload buffers aren't guaranteed NUL-terminated); a zero-length payload is stored as a valid empty payload

- oversized topics (>= BB_SUB_TOPIC_MAX) are rejected with BB_ERR_INVALID_ARG, not silently truncated (avoids two distinct topics colliding into one cache slot)

- per-message snapshot buffer is heap-allocated (freed after use), not a stack-resident struct, removing caller-stack pressure on tight-stack tasks (e.g. the MQTT callback)

- bb_sub_entry_t gains a cache_registered flag so route() skips the redundant idempotent bb_cache_register_ex call once a topic is registered

- aggregate change-notification queue_depth sizing requirement and the aggregate-drop-count gap vs bb_sub_dropped_count() documented in the header; coalescing declined this pass, follow-up noted

- BB_SUB_TESTING gains bb_sub_set_malloc() to exercise the snapshot-alloc-failure branch; bb_sub_reset_for_test() docs the required bb_cache_reset_for_test()-before-bb_sub_reset_for_test() ordering

- ensure_event_topic() now logs on bb_event_topic_register failure instead of silently leaving the aggregate topic unset

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

1932 of 1932 branches covered (100.0%)

Branch coverage included in aggregate %.

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

3324 of 3324 relevant lines covered (100.0%)

1270.56 hits per line