Summary

This started as a narrower fix (a bare calendar Declined: was being treated as noise the same as Accepted:) but on reflection the whole Tier-1 guard doesn't belong here , removing it entirely.

No other system agent (crm-updater, next-steps, drafter, inbound-email-notifier) has a hand-maintained pattern-match guard like this. They all rely on the model's own judgment to recognize noise cheaply, since they're already fed event content directly rather than needing forced reads. Strategist's own prompt (agent-defaults/strategist.ts, "Each run" step 0, added in #2691) already covers the exact same two cases via judgment , "an automated system/tooling notification" and "a scheduling accept with no message content" are both named explicitly , so the code-level guard was redundant with a check that already works, not filling a real gap.

It was also the source of both bugs raised against this branch: an unmatched-event edge case Greptile caught on #2691, and the Declined-RSVP issue that started this PR. A hardcoded regex is exactly the kind of thing that quietly drifts wrong in a way a model's own judgment doesn't.

Net effect of removing it: those two noise categories (Cedar's own delivery-failure alerts, bare calendar accepts) go from costing 0 LLM calls to costing 1 , Tier 2's judgment pass still catches them, just not for free anymore. That's exactly how every other agent in the fleet already operates.

Removes isNonDealSystemOrCalendarNoise, its two supporting constants, its call site in strategist-updates.ts, its two comment references in runSubagentTool.ts, and its now-obsolete test file.

Test plan

• pnpm --filter @zero/server run types , 0 errors
• pnpm exec vitest run across touched test dirs , 285 passed, 0 failed
• oxlint --deny-warnings , exit 0 (the check that failed CI on #2691 initially)

🤖 Generated with Claude Code

https://claude.ai/code/session_01E3AhwqADAFeqe9wEhz6Mk7