agent-coord-mcp 0.26.0 → 0.26.1

package/hooks/tmux-pusher.mjs

@@ -818,15 +818,38 @@ function setRoomOffset(cursor, chan, n) {
   else (cursor.roomOffsets ??= {})[c] = n;
 }
 
+// Channels this agent is a MEMBER of, per rooms.json. #general included only
+// if membership says so.
+//
+// It used to be seeded `new Set([DEFAULT_ROOM])` before the membership loop, so
+// every pusher tailed #general whether or not the agent had ever joined it —
+// membership was honoured for every channel except the one with the most
+// traffic. Observed directly: rooms.json read `general -> 0 members []` while
+// agents joined only to a project room were taking #general cross-project
+// broadcasts into their panes.
+//
+// Deliberate consequence of dropping the seed: no registry (or an unparseable
+// one) now yields NO channels rather than #general. Absence of a registry is
+// absence of evidence of membership, and the safe reading of "I cannot tell
+// whether this agent joined" is not "push to it" — that is the assumption that
+// produced the defect. Nothing is lost either way: offsets are untouched here,
+// so read_messages still serves whatever the pusher did not.
+//
+// Cursor safety: this function only decides what gets TAILED. Offsets advance
+// exclusively in collectRoomChannel, which runs only for the channels returned
+// here — so a channel this agent is not in has its offset left exactly where it
+// was, and joining later resumes from that point. Not stranded (read_messages
+// reads the same cursor), not skipped (nothing advanced it meanwhile), not
+// double-served (the offset is preserved, not reset).
 function joinedRooms() {
   let reg = {};
   try {
     const raw = readFileSync(path.join(ROOT, "rooms.json"), "utf8");
     if (raw.trim()) reg = JSON.parse(raw);
   } catch {
-    // no registry yet → just the default channel
+    // no registry yet → no memberships to honour
   }
-  const out = new Set([DEFAULT_ROOM]);
+  const out = new Set();
   for (const [chan, e] of Object.entries(reg)) {
     if (e && Array.isArray(e.members) && e.members.includes(AGENT_ID)) out.add(chan);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-coord-mcp",
-  "version": "0.26.0",
+  "version": "0.26.1",
   "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": {
@@ -46,7 +46,7 @@
     "@modelcontextprotocol/sdk": "^1.30.0",
     "proper-lockfile": "^4.1.2",
     "zod": "^4.4.3",
-    "@davidbalzan/groundwork-seam": "0.1.3"
+    "@davidbalzan/groundwork-seam": "0.1.4"
   },
   "devDependencies": {
     "@types/node": "^26.2.0",

package/scripts/check-test-count.mjs

@@ -22,7 +22,7 @@
 
 import { spawn } from "node:child_process";
 
-const EXPECTED_TESTS = 312;
+const EXPECTED_TESTS = 321;
 
 const expected = Number(process.env.AGENT_COORD_EXPECTED_TESTS ?? EXPECTED_TESTS);
 // Same glob the suite always used — `--test test/` would recurse differently