agent-coord-mcp 0.4.3 → 0.4.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-coord-mcp",
-  "version": "0.4.3",
+  "version": "0.4.6",
   "description": "File-backed MCP server for coordinating multiple AI coding agents (Claude Code, Cursor, Cline, etc.) on the same machine.",
   "type": "module",
   "bin": {

package/scripts/coord-chat.mjs

@@ -74,17 +74,15 @@ const A = {
   brightCyan:    (s) => `\x1b[96m${s}\x1b[0m`,
 };
 
-// Stable per-agent color from agentId hash. 10 colors (standard + bright)
-// to reduce collisions. Red is reserved for errors so we skip it.
+// Stable per-agent color via a persistent registry shared by all coord-chat
+// sessions. First time we see an agentId we pick the next unused palette
+// slot — guarantees no collisions until the palette is exhausted. After that
+// we fall back to hashing so behavior stays deterministic.
 const AGENT_COLORS = [
   A.green, A.yellow, A.blue, A.magenta, A.cyan,
   A.brightGreen, A.brightYellow, A.brightBlue, A.brightMagenta, A.brightCyan,
 ];
-function agentColor(id) {
-  let h = 0;
-  for (let i = 0; i < id.length; i++) h = (h * 31 + id.charCodeAt(i)) >>> 0;
-  return AGENT_COLORS[h % AGENT_COLORS.length];
-}
+// Will be initialized after ROOT is set, just below.
 
 // ---------- register and start UI ----------
 
@@ -122,14 +120,27 @@ const SLASH_COMMANDS = [
 ];
 
 const STATUS_FILE_PATH = path.join(ROOT, "status.jsonl");
+const COLOR_MAP_FILE = path.join(ROOT, "chat-colors.json");
 
 function completer(line) {
-  // Tab-complete slash commands and DM targets. On multi-match with no
-  // common-prefix advancement, surface the options on the first Tab via
-  // say() — default readline UX hides them until a second Tab, which most
-  // users assume means "nothing happened."
+  // Tab-complete slash commands, DM targets, and @mentions mid-message.
+  // On multi-match with no common-prefix advancement, surface the options
+  // on the first Tab via say() — default readline UX hides them until a
+  // second Tab, which most users assume means "nothing happened."
   let hits = [];
-  if (line.startsWith("/dm ")) {
+  let substr = line;
+
+  // @mention completion takes priority — checked first because it can
+  // appear inside a slash command argument (e.g. `/dm bob hey @ali`) or
+  // in a plain room message.
+  const mentionMatch = line.match(/@([A-Za-z0-9._-]*)$/);
+  if (mentionMatch) {
+    const partial = mentionMatch[1];
+    const reg = readJsonSafe(AGENTS_FILE, {});
+    const ids = Object.keys(reg).filter((id) => id.startsWith(partial));
+    hits = ids.map((id) => `@${id} `);
+    substr = mentionMatch[0]; // tell readline to replace just the @partial part
+  } else if (line.startsWith("/dm ")) {
     const partial = line.slice(4);
     const reg = readJsonSafe(AGENTS_FILE, {});
     const ids = Object.keys(reg).filter((id) => id !== ID && id.startsWith(partial));
@@ -137,11 +148,11 @@ function completer(line) {
   } else if (line.startsWith("/")) {
     hits = SLASH_COMMANDS.filter((c) => c.startsWith(line));
   }
-  if (hits.length > 1 && commonPrefix(hits).length <= line.length) {
+  if (hits.length > 1 && commonPrefix(hits).length <= substr.length) {
     const display = hits.map((h) => h.trim()).join("  ");
     say(A.dim("  ┄ " + display));
   }
-  return [hits, line];
+  return [hits, substr];
 }
 
 function commonPrefix(strs) {
@@ -162,7 +173,10 @@ const rl = readline.createInterface({
 
 // Banner — printed once on launch. Keep it tight; this is a CLI, not a poster.
 printBanner();
-await drainAndPrint();
+// Show recent context (last 3 messages from inbox + room) then fast-forward
+// the cursor so the same entries don't show up again via the watcher path.
+fastForwardCursors();
+await printRecent(3);
 
 // Lay down the first separator. From this point, async incoming messages
 // (via the watcher → drainAndPrint → say) know they can use the cursor
@@ -280,6 +294,34 @@ function sanitize(s) {
   return s.replace(/[^a-zA-Z0-9._-]/g, "_");
 }
 
+function agentColor(id) {
+  const map = readJsonSafe(COLOR_MAP_FILE, {});
+  const existing = map[id];
+  if (typeof existing === "number" && existing >= 0 && existing < AGENT_COLORS.length) {
+    return AGENT_COLORS[existing];
+  }
+  // First sighting — pick the first unused palette slot.
+  const used = new Set(Object.values(map).filter((v) => typeof v === "number"));
+  let idx = -1;
+  for (let i = 0; i < AGENT_COLORS.length; i++) {
+    if (!used.has(i)) { idx = i; break; }
+  }
+  if (idx === -1) {
+    // Palette exhausted — deterministic hash fallback. No persist (don't
+    // pollute the map with hash assignments that could be wrong).
+    let h = 0;
+    for (let i = 0; i < id.length; i++) h = (h * 31 + id.charCodeAt(i)) >>> 0;
+    return AGENT_COLORS[h % AGENT_COLORS.length];
+  }
+  // Persist. Re-read from disk first to merge any concurrent assignments
+  // by other coord-chat processes; last-writer-wins for a single agentId
+  // is fine since colors are cosmetic.
+  const onDisk = readJsonSafe(COLOR_MAP_FILE, {});
+  onDisk[id] = idx;
+  try { writeJsonAtomic(COLOR_MAP_FILE, onDisk); } catch { /* best effort */ }
+  return AGENT_COLORS[idx];
+}
+
 function say(line) {
   if (lastLineWasSep) {
     // Async path: a separator we own sits directly above the prompt; we own
@@ -373,6 +415,17 @@ async function printWhoami() {
   say(`  ${A.cyan("registered")} ${a ? A.green("yes") : A.red("no")}`);
 }
 
+function fastForwardCursors() {
+  // Move our cursor offsets to end-of-file so anything that existed before
+  // launch is treated as already-seen. printRecent(N) then shows the last N
+  // as historical context, and the watcher path only fires for genuinely
+  // new messages going forward.
+  const cur = readJsonSafe(CURSOR_FILE, {});
+  cur.inboxOffset = readJsonl(INBOX_FILE).length;
+  cur.roomOffset = readJsonl(ROOM_FILE).length;
+  writeJsonAtomic(CURSOR_FILE, cur);
+}
+
 async function printRecent(n) {
   const inbox = readJsonl(INBOX_FILE).slice(-n).map((m) => ({ ...m, _kind: "DM" }));
   const room = readJsonl(ROOM_FILE).slice(-n).map((m) => ({ ...m, _kind: "room" }));
@@ -617,13 +670,49 @@ function printMsg(kind, m, opts = {}) {
   const tag = kind === "DM" ? A.bold(A.cyan("DM")) : A.dim("room");
   const nick = A.bold(color(who));
   const meta = `${A.dim(t)} ${tag} ${nick}`;
-  const body = (m.text ?? "").split("\n").map(formatBody);
-  const indent = "         ";
-  const first = body[0] ?? "";
-  const rest = body.slice(1).map((l) => `${gutter} ${indent}${A.dim("│ ")}${l}`);
   const prefix = opts.history ? A.dim("  ") : "";
-  say(`${prefix}${gutter} ${meta} ${first}`);
-  for (const line of rest) say(`${prefix}${line}`);
+
+  // Visible widths: prefix + "▎ " + meta + " " on the first line;
+  // prefix + "▎ " on continuation lines. We wrap manually so the
+  // colored gutter prepends every wrapped line — terminal auto-wrap
+  // would lose it.
+  const prefixW = visibleLength(prefix);
+  const firstChrome = prefixW + visibleLength(`▎ ${meta} `);
+  const restChrome = prefixW + visibleLength(`▎ `);
+  const firstWidth = Math.max(20, COLS - firstChrome);
+  const restWidth = Math.max(20, COLS - restChrome);
+
+  const text = (m.text ?? "").split("\n").map(formatBody).join("\n");
+  const lines = wrapText(text, firstWidth, restWidth);
+  say(`${prefix}${gutter} ${meta} ${lines[0] ?? ""}`);
+  for (const line of lines.slice(1)) say(`${prefix}${gutter} ${line}`);
+}
+
+function visibleLength(s) {
+  // Strip ANSI SGR sequences so we measure on-screen width, not raw bytes.
+  return s.replace(/\x1b\[[0-9;]*m/g, "").length;
+}
+
+function wrapText(text, firstWidth, restWidth) {
+  if (firstWidth <= 0 || restWidth <= 0) return [text];
+  const out = [];
+  for (const para of text.split("\n")) {
+    const words = para.split(" ");
+    let line = "";
+    let currentWidth = out.length === 0 ? firstWidth : restWidth;
+    for (const word of words) {
+      const proposed = line ? line + " " + word : word;
+      if (visibleLength(proposed) > currentWidth && line) {
+        out.push(line);
+        line = word;
+        currentWidth = restWidth;
+      } else {
+        line = proposed;
+      }
+    }
+    if (line || words.length === 0) out.push(line);
+  }
+  return out;
 }
 
 // Lightweight inline-only "chat markdown" formatter — no dep. Handles bold,