dangernoodle-io / breadboard / 24961026922

100%

push

github

fix(bb_mdns): tolerant service/proto matching in browse notifier (#91)

internal_notifier did exact strcmp on r->service_type / r->proto against
the registered subscription strings to dispatch results. IDF passes those
back inconsistently — sometimes with leading underscore, sometimes
without, depending on the result path. Subscriptions registered as
"_taipanminer" / "_tcp" silently dropped every result whose service_type
came back as "taipanminer", so on_peer never fired and consumers saw
zero peers despite peers actively announcing.

Strip a leading '_' on both sides before comparing. Also log at debug
when a result fails to match any subscription so future regressions
are visible.

Verified locally: TaipanMiner Knot tab now sees peers.

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

81 of 81 branches covered (100.0%)

Branch coverage included in aggregate %.

113 of 113 relevant lines covered (100.0%)

289.66 hits per line