dangernoodle-io / breadboard / 26006550713

100%

push

github

feat(bb_event_routes): honor ?topic= query filter on /api/events (#266)

Previously the `?topic=<name>` query parameter on /api/events was silently
ignored — every connected SSE client received every attached topic. Harmless
today since only `update.available` is attached, but a footgun the moment a
second producer (e.g. ota.progress, mining.share.*) attaches: callers asking
for one topic would silently receive all.

TM's webui has been passing ?topic=update.available since the badge feature
landed, expecting filtering. This PR makes it work.

Adds bb_event_routes_client_acquire_ex(out, topic_filter) — non-NULL filter
subscribes only to topics whose name matches; NULL preserves current
"subscribe to all" behavior. Original bb_event_routes_client_acquire becomes
a thin wrapper around _ex(out, NULL). Mirrors bb_event_routes_attach_ex
naming.

Server parses the query string via httpd_req_get_url_query_str +
httpd_query_key_value and passes the filter into _ex.

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

966 of 966 branches covered (100.0%)

Branch coverage included in aggregate %.

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

1733 of 1733 relevant lines covered (100.0%)

1019.97 hits per line