agent-coord-mcp 0.4.7 → 0.4.9

package/package.json

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

@@ -44,6 +44,38 @@ const args = parseArgs(process.argv.slice(2));
 const ID = args.id ?? process.env.USER ?? "human";
 const ROOT = args.dir ?? process.env.AGENT_COORD_DIR ?? path.join(homedir(), "agent-coord");
 
+// Message-rendering state + helpers. Declared up here (above the top-level
+// printRecent() call) so they're initialized before first use — const/let
+// don't hoist the way function declarations do.
+
+// Consecutive messages from the same sender within this window are visually
+// grouped: the second one drops its header/blank line and just continues the
+// gutter, Slack-style.
+const GROUP_WINDOW = 2 * 60 * 1000;
+let lastBlock = { who: null, ts: 0, kind: null };
+
+// Whether the ephemeral suggestion hint is currently occupying the slot above
+// the prompt (see drawHint/clearHint). Declared early so say(), called during
+// startup replay, can reference it without tripping the const/let TDZ.
+let hintActive = false;
+
+// Matches "@<this agent>" not followed by a name char, so we can flag messages
+// that ping the current user. ID may contain regex metachars — escape it.
+const SELF_MENTION_RE = new RegExp(
+  "@" + ID.replace(/[.*+?^${}()|[\]\\]/g, "\\$&") + "(?![A-Za-z0-9._-])",
+);
+const mentionsSelf = (text) => SELF_MENTION_RE.test(text ?? "");
+
+// Recency at a glance: "now" / "5m" for fresh messages, falling back to a wall
+// clock for anything over an hour (a stale "63m" reads worse than "08:34").
+function relTime(ts) {
+  const mins = Math.floor((Date.now() - ts) / 60000);
+  if (mins < 1) return "now";
+  if (mins < 60) return `${mins}m`;
+  const d = new Date(ts);
+  return `${String(d.getHours()).padStart(2, "0")}:${String(d.getMinutes()).padStart(2, "0")}`;
+}
+
 const INBOX_DIR = path.join(ROOT, "inbox");
 const CURSOR_DIR = path.join(ROOT, "cursors");
 const TRANSPORT_DIR = path.join(ROOT, "transports");
@@ -136,21 +168,24 @@ function completer(line) {
   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));
+    const ids = onlineAgentIds().filter((id) => id !== 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));
+    const ids = onlineAgentIds().filter((id) => id !== ID && id.startsWith(partial));
     hits = ids.map((id) => `/dm ${id} `);
   } else if (line.startsWith("/")) {
     hits = SLASH_COMMANDS.filter((c) => c.startsWith(line));
   }
-  if (hits.length > 1 && commonPrefix(hits).length <= substr.length) {
+  if (hits.length > 1) {
+    // Show the options as an ephemeral hint and hand readline only the common
+    // prefix — a single-element completion means its native multi-column dump
+    // never lands in scrollback. Tab still advances to the shared prefix.
     const display = hits.map((h) => h.trim()).join("  ");
-    say(A.dim("  ┄ " + display));
+    drawHint(A.dim("  ┄ " + display));
+    const cp = commonPrefix(hits);
+    return [[cp.length >= substr.length ? cp : substr], substr];
   }
   return [hits, substr];
 }
@@ -171,6 +206,23 @@ const rl = readline.createInterface({
   completer,
 });
 
+// Auto-offer the logged-in agents the instant "@" is typed (editor-style),
+// so you don't have to press Tab to discover who's reachable. We only observe
+// keypresses — readline still owns input. setImmediate lets readline insert
+// the "@" into its line buffer before we inspect it.
+if (process.stdin.isTTY) {
+  readline.emitKeypressEvents(process.stdin);
+  process.stdin.on("keypress", (str, key) => {
+    // setImmediate lets readline mutate its line buffer first, then we inspect
+    // it / redraw the slot above the prompt.
+    if (str === "@") { setImmediate(showMentionPicker); return; }
+    // Tab is the completer's — it draws/keeps the hint itself. Every other key
+    // dismisses a showing hint so the view snaps back to a clean separator.
+    if (key && key.name === "tab") return;
+    if (hintActive) setImmediate(clearHint);
+  });
+}
+
 // Banner — printed once on launch. Keep it tight; this is a CLI, not a poster.
 printBanner();
 // Show recent context (last 3 messages from inbox + room) then fast-forward
@@ -322,7 +374,28 @@ function agentColor(id) {
   return AGENT_COLORS[idx];
 }
 
+// Ephemeral suggestion line (the @mention / completion picker). It borrows the
+// separator slot directly above the prompt: drawn there, then wiped back to a
+// real separator on the next keystroke — so it never piles up in scrollback.
+function drawHint(content) {
+  if (typeof rl === "undefined" || !lastLineWasSep) return;
+  process.stdout.write("\x1b[1A\r\x1b[2K");  // up to the slot, clear it
+  process.stdout.write(content + "\n");       // draw the hint in place of the sep
+  rl.prompt(true);                            // redraw prompt + preserved input
+  hintActive = true;
+}
+
+function clearHint() {
+  if (typeof rl === "undefined" || !lastLineWasSep) { hintActive = false; return; }
+  if (!hintActive) return;
+  process.stdout.write("\x1b[1A\r\x1b[2K");  // up to the hint slot, clear it
+  process.stdout.write(sepLine() + "\n");     // restore the separator
+  rl.prompt(true);
+  hintActive = false;
+}
+
 function say(line) {
+  hintActive = false; // any real output reclaims the slot the hint borrowed
   if (lastLineWasSep) {
     // Async path: a separator we own sits directly above the prompt; we own
     // that line. Replace it with the incoming message, drop a new sep, and
@@ -661,29 +734,6 @@ async function drainAndPrint() {
   if (changed) writeJsonAtomic(CURSOR_FILE, cursor);
 }
 
-// Consecutive messages from the same sender within this window are visually
-// grouped: the second one drops its header/blank line and just continues the
-// gutter, Slack-style.
-const GROUP_WINDOW = 2 * 60 * 1000;
-let lastBlock = { who: null, ts: 0, kind: null };
-
-// Matches "@<this agent>" not followed by a name char, so we can flag messages
-// that ping the current user. ID may contain regex metachars — escape it.
-const SELF_MENTION_RE = new RegExp(
-  "@" + ID.replace(/[.*+?^${}()|[\]\\]/g, "\\$&") + "(?![A-Za-z0-9._-])",
-);
-const mentionsSelf = (text) => SELF_MENTION_RE.test(text ?? "");
-
-// Recency at a glance: "now" / "5m" for fresh messages, falling back to a wall
-// clock for anything over an hour (a stale "63m" reads worse than "08:34").
-function relTime(ts) {
-  const mins = Math.floor((Date.now() - ts) / 60000);
-  if (mins < 1) return "now";
-  if (mins < 60) return `${mins}m`;
-  const d = new Date(ts);
-  return `${String(d.getHours()).padStart(2, "0")}:${String(d.getMinutes()).padStart(2, "0")}`;
-}
-
 function printMsg(kind, m, opts = {}) {
   const who = m.from ?? "?";
   const color = agentColor(who);
@@ -836,6 +886,38 @@ function pidAlive(pid) {
   catch (e) { return e?.code === "EPERM"; }
 }
 
+// Agents considered "logged in": a live transport process, or a heartbeat
+// within the stale window. Shared by the @mention picker and completer so we
+// only ever offer reachable agents.
+function onlineAgentIds() {
+  const reg = readJsonSafe(AGENTS_FILE, {});
+  const now = Date.now();
+  const STALE = 5 * 60 * 1000;
+  return Object.keys(reg)
+    .filter((id) => {
+      const a = reg[id];
+      const marker = readJsonSafe(path.join(TRANSPORT_DIR, `${sanitize(id)}.json`), null);
+      const live = marker && marker.pid && pidAlive(marker.pid);
+      return live || now - (a?.lastHeartbeat ?? 0) < STALE;
+    })
+    .sort();
+}
+
+// Pop the list of logged-in agents the moment "@" starts a mention token, so
+// you can see who's reachable without hunting through /list. The list is
+// dim/cosmetic and re-renders above the preserved input line.
+function showMentionPicker() {
+  if (typeof rl === "undefined") return;
+  const before = (rl.line ?? "").slice(0, rl.cursor ?? (rl.line ?? "").length);
+  // Only when the just-typed "@" opens a fresh token (start of line or after
+  // whitespace) — avoids firing inside emails or mid-word.
+  if (!/(^|\s)@$/.test(before)) return;
+  const ids = onlineAgentIds().filter((id) => id !== ID);
+  if (!ids.length) return;
+  const list = ids.map((id) => A.green("●") + agentColor(id)(`@${id}`)).join("  ");
+  drawHint(A.dim("  ┄ ") + list + A.dim("   · Tab to complete"));
+}
+
 function readJsonl(file) {
   if (!existsSync(file)) return [];
   return readFileSync(file, "utf8").split("\n").filter(Boolean).map((l) => {