agent-coord-mcp 0.13.0 → 0.14.0

package/README.md

@@ -293,7 +293,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 `[agent-coord] digest` block. 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. 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):`). 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

@@ -68,25 +68,33 @@ export class TierQueue {
   }
 }
 
+// The per-message PARSE CONTRACT line. Agent harnesses read from/room/text
+// back out of this, so its shape is load-bearing and MUST stay byte-identical
+// to coord-pusher.mjs's `injectLine`. Compact form (v0.14.0, salvaged from
+// v0.8.10): `  [<kind> <HH:MM> <from>] <text>` where kind drops the leading
+// "room " ("room #general" → "#general"), the timestamp is HH:MM UTC, and the
+// "from=" label is dropped (bare id). kind/time/from never contain spaces
+// (ids are sanitized), so a parser splits on the first "] " unambiguously.
+export function injectLine(m) {
+  const tag = String(m.kind ?? "").replace(/^room /, "");
+  const d = new Date(m.ts ?? 0);
+  const hhmm = `${String(d.getUTCHours()).padStart(2, "0")}:${String(d.getUTCMinutes()).padStart(2, "0")}`;
+  return `  [${tag} ${hhmm} ${m.from}] ${m.text ?? ""}`;
+}
+
 // Render one delivery: urgent verbatim under the banner, then at most ONE
 // coalesced digest block carrying the queued routine messages.
 export function formatBatch(batch) {
-  const line = (m) =>
-    `  [${m.kind} ${new Date(m.ts ?? 0).toISOString()} from=${m.from}] ${m.text ?? ""}`;
   const urgent = batch.filter((m) => m.tier !== "routine");
   const routine = batch.filter((m) => m.tier === "routine");
   const lines = [];
   if (urgent.length > 0) {
-    lines.push(
-      "[agent-coord] incoming peer messages — already consumed from your inbox, do not call read_messages for them:",
-    );
-    for (const m of urgent) lines.push(line(m));
+    lines.push("[agent-coord] msgs (pre-consumed, don't re-read):");
+    for (const m of urgent) lines.push(injectLine(m));
   }
   if (routine.length > 0) {
-    lines.push(
-      `[agent-coord] digest — ${routine.length} routine message${routine.length === 1 ? "" : "s"} coalesced with this push (also consumed; FYI only, no reply expected):`,
-    );
-    for (const m of routine) lines.push(line(m));
+    lines.push(`[agent-coord] +${routine.length} routine (pre-consumed, FYI, no reply):`);
+    for (const m of routine) lines.push(injectLine(m));
   }
   return lines.join("\n");
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-coord-mcp",
-  "version": "0.13.0",
+  "version": "0.14.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": {

package/scripts/coord-pusher.mjs

@@ -168,13 +168,23 @@ async function flush() {
   }
 }
 
+// The per-message PARSE CONTRACT line — MUST stay byte-identical to
+// hooks/tier.mjs's injectLine (agent harnesses parse from/room/text out of
+// it). Compact form (v0.14.0): `  [<kind> <HH:MM> <from>] <text>`, kind drops
+// the leading "room ", timestamp is HH:MM UTC, no "from=" label. This pusher
+// is standalone (may deploy without hooks/), so the helper is duplicated
+// rather than imported; test/tier.test.mjs locks both to the same shape.
+function injectLine(m) {
+  const tag = String(m.kind ?? "").replace(/^room /, "");
+  const d = new Date(m.ts ?? 0);
+  const hhmm = `${String(d.getUTCHours()).padStart(2, "0")}:${String(d.getUTCMinutes()).padStart(2, "0")}`;
+  return `  [${tag} ${hhmm} ${m.from}] ${m.text ?? ""}`;
+}
+
 function formatBatch(batch) {
-  const lines = [
-    "[agent-coord] incoming peer messages — already consumed from your inbox, do not call read_messages for them:",
-  ];
+  const lines = ["[agent-coord] msgs (pre-consumed, don't re-read):"];
   for (const m of batch) {
-    const ts = new Date(m.ts ?? Date.now()).toISOString();
-    lines.push(`  [${m.kind} ${ts} from=${m.from}] ${m.text ?? ""}`);
+    lines.push(injectLine(m));
   }
   return lines.join("\n");
 }