agent-coord-mcp 0.16.0 → 0.17.0

package/README.md

@@ -303,7 +303,7 @@ Either way, `list_agents` will show the agent with `transport: "tmux-push"` so p
 
 Under the hood: [`hooks/tmux-pusher.mjs`](./hooks/tmux-pusher.mjs) is the daemon. It watches `~/agent-coord/inbox/<id>.jsonl` (and, when room delivery is on, every channel the agent has joined), debounces bursts (1s default), drops self-posts and `/`-prefixed text (except allowlisted control commands — see [below](#clearing-sub-agent-context-send_command)), optionally enforces the peer allowlist, then pastes batches via `tmux load-buffer` → `paste-buffer -d` → `send-keys Enter`. Single-flight so two batches never overlap.
 
-**Delivery tiers.** Not every message deserves an agent turn. Only push-now traffic wakes the pane: `BLOCKER:`, `DAVID_DECISION:`, literal `GO:` work orders, `SCOPE:`/`SCOPE CHANGE:` from a trusted sender (coordinator/gate ids resolved from the registry), `DONE:` when this agent is a gate runner (QA/coordinator — from its registry role, re-resolved every 30s, overridable with `AGENT_COORD_GATE_RUNNER=1|0`), control commands, and server-flagged messages like the post-`/clear` identity reminder. Prefixes are literal and case-sensitive. Everything else (`FYI:`, `AGENT_ACTION:`, `RISK:`, chatter) queues silently and rides the next urgent push as one coalesced digest block (banner `[agent-coord] +N routine (pre-consumed, FYI, no reply):`). Routine backlog is age-bounded: once its oldest message has queued longer than `AGENT_COORD_MAX_QUEUE_MS` (default 5min; `0` disables), the backlog flushes on its own as a routine-only digest, so a quiet bus never sits on undelivered traffic indefinitely. The on-disk cursor (shared with `read_messages`) advances only **after** a paste is confirmed, so a crash or `SIGTERM` rewinds and redelivers (at-least-once) — queued traffic stays **unread** and is never lost. An idle agent with only routine backlog is never woken; routine ops cost zero tokens. `AGENT_COORD_TIERS=0` restores legacy push-everything. The classifier and queue are pure ([`hooks/tier.mjs`](./hooks/tier.mjs)). Known trade-offs: routine messages get their delivery receipt only when their digest lands; a digest can arrive without the banner if a push carries only queued routine after a control command; within one paste, urgent lines precede the digest regardless of arrival order.
+**Delivery tiers.** Not every message deserves an agent turn. All DMs push now — they're point-to-point asks, not broadcast noise. Channel traffic wakes the pane only when push-now: `BLOCKER:`, `DAVID_DECISION:`, literal `GO:` work orders, `SCOPE:`/`SCOPE CHANGE:` from a trusted sender (coordinator/gate ids resolved from the registry), `DONE:` when this agent is a gate runner (QA/coordinator — from its registry role, re-resolved every 30s, overridable with `AGENT_COORD_GATE_RUNNER=1|0`), control commands, and server-flagged messages like the post-`/clear` identity reminder. Prefixes are literal and case-sensitive. Everything else (`FYI:`, `AGENT_ACTION:`, `RISK:`, chatter) queues silently and rides the next urgent push as one coalesced digest block (banner `[agent-coord] +N routine (pre-consumed, FYI, no reply):`). Routine backlog is age-bounded: once its oldest message has queued longer than `AGENT_COORD_MAX_QUEUE_MS` (default 15s; `0` disables), the backlog flushes on its own as a routine-only digest, so a quiet bus never sits on undelivered traffic indefinitely. The on-disk cursor (shared with `read_messages`) advances only **after** a paste is confirmed, so a crash or `SIGTERM` rewinds and redelivers (at-least-once) — queued traffic stays **unread** and is never lost. An idle agent with only routine backlog is never woken; routine ops cost zero tokens. `AGENT_COORD_TIERS=0` restores legacy push-everything. The classifier and queue are pure ([`hooks/tier.mjs`](./hooks/tier.mjs)). Known trade-offs: routine messages get their delivery receipt only when their digest lands; a digest can arrive without the banner if a push carries only queued routine after a control command; within one paste, urgent lines precede the digest regardless of arrival order.
 
 **Caveats — read these.**
 

package/hooks/tier.mjs

@@ -14,6 +14,11 @@ export function classifyTier(m, opts = {}) {
   // Server-set push-now override (post-/clear reminder). send_message builds
   // Messages from fixed fields, so a peer cannot smuggle this flag in.
   if (m.urgent === true) return "urgent";
+  // DMs are always push-now: they're addressed to this agent by a peer who
+  // wants it specifically, and DM volume is tiny next to room traffic. The
+  // tiers exist to absorb broadcast noise, not point-to-point asks (the
+  // liaison relaying a David question must not sit in a digest queue).
+  if (m.kind === "DM") return "urgent";
   if (typeof m.text !== "string") return "routine";
   const text = m.text.trimStart();
   // Control/slash commands are injected raw and must fire immediately.

package/hooks/tmux-pusher.mjs

@@ -19,9 +19,9 @@
  *   AGENT_COORD_DEBOUNCE_MS   coalesce window for bursts (default 1000)
  *   AGENT_COORD_POLL_MS       fallback poll interval (default 1000)
  *   AGENT_COORD_TARGET_GRACE  missed pane probes before self-exit (default 3)
- *   AGENT_COORD_MAX_QUEUE_MS  max time routine traffic may queue without an
- *                             urgent trigger before it flushes as a digest
- *                             (default 300000 = 5min; 0 disables)
+ *   AGENT_COORD_MAX_QUEUE_MS  max time routine (channel) traffic may queue
+ *                             without an urgent trigger before it flushes as
+ *                             a digest (default 15000 = 15s; 0 disables)
  *
  * Safety:
  *   - drops messages where from === AGENT_COORD_ID (no self-echo)
@@ -89,8 +89,9 @@ const TIERS_ENABLED = process.env.AGENT_COORD_TIERS !== "0";
 // Max-age flush: routine traffic never waits longer than this for an urgent
 // trigger — once the oldest queued message is overdue, the backlog flushes as
 // a routine-only digest. Bounds the silent-fleet failure mode where FYI:/DONE:
-// chatter queues forever because nothing urgent ever arrives.
-const MAX_QUEUE_MS = parseInt(process.env.AGENT_COORD_MAX_QUEUE_MS || "300000", 10);
+// chatter queues forever because nothing urgent ever arrives. Only channel
+// traffic queues (DMs are push-now), so the default is short.
+const MAX_QUEUE_MS = parseInt(process.env.AGENT_COORD_MAX_QUEUE_MS || "15000", 10);
 // Gate runners (QA/coordinator) receive DONE: as push-now; countersigned
 // SCOPE: changes are honored only from these trusted ids. Re-resolved from
 // the registry every 30s so a role change doesn't strand a stale pusher;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-coord-mcp",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "File-backed MCP server for coordinating multiple AI coding agents (Claude Code, Cursor, Cline, etc.). Local stdio or networked over Streamable HTTP.",
   "type": "module",
   "bin": {