dangernoodle-io / breadboard / 31342253926

97%

push

github

fix: stop mirroring bb_data_http's log stream to MQTT (#1264)

- floor_app.c no longer acquires a durable bb_data_http client onto
  the "log" key -- hardware validation of B1-1126 PR1 found it
  self-amplifying: bb_mqtt_client's own outbox-full warning is itself
  a log line, so it re-enters the mirrored stream, gets rejected, and
  logs again (1874/s dropped writes vs 0.17/s with MQTT disabled)
- logs are diagnostic exhaust (unbounded, human-oriented, already
  served by SSE and bb_log's UDP sink), not an MQTT payload; MQTT
  carries device state and domain events
- keep floor_mqtt_egress.[ch] and its host tests as the reusable,
  currently-uncomposed seam for a future genuine domain-event key,
  with a header note recording the precondition (rate-limit the
  bb_mqtt_client rejection warning first) before it is wired up again


Claude-Session: https://claude.ai/code/session_0119nuE2NZME9y2aYEVJTPB5

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

8529 of 9048 branches covered (94.26%)

Branch coverage included in aggregate %.

14293 of 14448 relevant lines covered (98.93%)

2614.08 hits per line