perf(daemon): early-out in note_output_sent_to_local_receivers when no deadlines/breakers

[phil-opp]

Issue

note_output_sent_to_local_receivers runs on every OutputSent message (the per-output notification). Its per-receiver loop has two possible effects:

• refreshing an input deadline — only when input_deadlines is non-empty, and
• circuit-breaker recovery — only when broken_inputs is non-empty.

Both inner branches already guard on those maps being non-empty. But in the common case where neither input_timeout deadlines nor circuit breakers are configured, the whole loop is a no-op — yet the function still did an Instant::now() clock read and, for every receiver, a subscribe_channels lookup plus a channel-capacity check, on every message.

Fix

Add a single early return at the top of the function when both input_deadlines and broken_inputs are empty, skipping the clock read and the entire loop. Behavior is unchanged: the loop had no observable effect in that case.

Validation

• cargo test -p dora-daemon passes, including the existing note_output_sent recovery tests — those configure broken_inputs, so they still exercise the loop past the new guard.
• cargo clippy ... -D warnings and cargo fmt --all --check pass.

---

⚠️ This PR is machine-generated by Claude (Claude Code), an AI assistant. Please review carefully before merging.

🤖 Generated with Claude Code

https://claude.ai/code/session_01D2eu6P1CvTvoKdK9fK3ZSd

---

Generated by Claude Code

[trunk-io]

😎 Merged successfully - details.

[phil-opp]

🤖 Fully automated review by Claude — this review was generated end-to-end by an automated agent with no human vetting. Treat it accordingly.

No issues found.

I traced the full body of note_output_sent_to_local_receivers (binaries/daemon/src/lib.rs:5283-5399). The per-receiver loop has exactly two effects, and both are already independently gated: the deadline refresh on !dataflow.input_deadlines.is_empty() (line 5326) and the circuit-breaker recovery on dataflow.broken_inputs.is_empty() (the continue at line 5350). When both maps are empty the loop performs no observable work, so the added if dataflow.input_deadlines.is_empty() && dataflow.broken_inputs.is_empty() { return; } is behavior-preserving — it also correctly skips the now-unused Instant::now() read, which is only consumed inside those two gated branches. The input_deadlines.insert(...) inside the recovery path (line 5374) is only reachable when broken_inputs is non-empty, so the early return never bypasses it.

---

Generated by Claude Code