@dalmasonto/taskflow-mcp 1.0.25 → 1.0.27

package/dist/agent-registry.js

@@ -26,17 +26,26 @@ function detectTmuxPane(pid) {
     catch { /* tmux not available */ }
     return null;
 }
-/** Generate a unique agent name from the project folder, auto-suffixing on collision */
+/** Generate a unique agent name from the project folder, auto-suffixing on collision.
+ *  Cleans up dead entries it encounters so the name is available for INSERT. */
 function generateName(folderName) {
     const db = getDb();
-    const existing = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(folderName);
-    if (!existing || !isAlive(existing.pid)) {
+    const check = (name) => {
+        const row = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(name);
+        if (!row)
+            return true; // name is free
+        if (!isAlive(row.pid)) {
+            // Dead entry — remove it so the name can be reused via INSERT
+            db.prepare('DELETE FROM agent_registry WHERE id = ?').run(row.id);
+            return true;
+        }
+        return false; // name is taken by a live agent
+    };
+    if (check(folderName))
         return folderName;
-    }
     for (let i = 2; i < 100; i++) {
         const candidate = `${folderName}:${i}`;
-        const row = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(candidate);
-        if (!row || !isAlive(row.pid))
+        if (check(candidate))
             return candidate;
     }
     return `${folderName}:${Date.now()}`;
@@ -57,12 +66,36 @@ export function registerAgent(options) {
     checkAgentLiveness();
     // Find all entries for this project path
     const entries = db.prepare('SELECT * FROM agent_registry WHERE project_path = ? ORDER BY connected_at DESC').all(projectPath);
-    // Priority 1: Reuse a disconnected entry that had the SAME PID (exact session resume)
-    // Priority 2: Reuse any disconnected entry for this path (new session, same project)
-    // This prevents a reconnecting agent from stealing another agent's identity
-    // when two sessions share the same project directory.
-    const disconnected = entries.find(e => e.status === 'disconnected' && e.pid === agentPid)
-        || entries.find(e => e.status === 'disconnected');
+    // Priority 0: Same PID already connected — this agent is renaming itself
+    // (e.g. auto-registered as folder name, now calling register_agent with a custom name)
+    const samePid = entries.find(e => e.status === 'connected' && e.pid === agentPid);
+    if (samePid && options?.customName && options.customName !== samePid.name) {
+        const oldName = samePid.name;
+        const newName = options.customName;
+        // Update agent_messages to preserve history
+        db.prepare('UPDATE agent_messages SET sender_name = ? WHERE sender_name = ?').run(newName, oldName);
+        db.prepare('UPDATE agent_messages SET recipient_name = ? WHERE recipient_name = ?').run(newName, oldName);
+        db.prepare('UPDATE agent_registry SET name = ?, tmux_pane = ?, connected_at = ? WHERE id = ?').run(newName, tmuxPane, ts, samePid.id);
+        const row = db.prepare('SELECT * FROM agent_registry WHERE id = ?').get(samePid.id);
+        broadcast('agent_renamed', { entity: 'agent', action: 'agent_renamed', payload: { ...row, oldName } });
+        logActivity('agent_renamed', `Agent "${oldName}" renamed to "${newName}"`, { entityType: 'agent' });
+        return newName;
+    }
+    // If same PID is already connected with the right name (or no custom name), just return it
+    if (samePid) {
+        return samePid.name;
+    }
+    // Reconnection priority for disconnected entries:
+    //   1. Custom name matches a disconnected entry exactly — the name IS the stable identity.
+    //      This is how users resume a specific agent: register_agent({ name: "sentinmail_coder" })
+    //   2. Same PID — exact session resume (e.g. MCP server restarted, same shell)
+    //   3. Only ONE disconnected entry — unambiguous, safe to reuse
+    // When multiple disconnected entries exist and no match is found,
+    // we create a fresh entry rather than guessing wrong and mixing up histories.
+    const allDisconnected = entries.filter(e => e.status === 'disconnected');
+    const disconnected = (options?.customName ? allDisconnected.find(e => e.name === options.customName) : undefined)
+        || allDisconnected.find(e => e.pid === agentPid)
+        || (allDisconnected.length === 1 ? allDisconnected[0] : undefined);
     if (disconnected) {
         const newName = options?.customName || disconnected.name;
         const renamed = newName !== disconnected.name;
@@ -77,7 +110,7 @@ export function registerAgent(options) {
         logActivity('agent_connected', `Agent "${newName}" reconnected`, { entityType: 'agent' });
         return newName;
     }
-    // Priority 2: All entries for this path are connected — this is a concurrent agent
+    // Priority 3: All entries for this path are connected — this is a concurrent agent
     // Generate a suffixed name to avoid collision
     const baseName = options?.customName || folderName;
     const name = entries.length === 0 ? baseName : generateName(baseName);

package/dist/tmux-bridge.js

@@ -92,14 +92,25 @@ function injectAndMarkDelivered(id, text, tmuxPane) {
     const db = getDb();
     db.prepare('UPDATE agent_messages SET delivered = 1 WHERE id = ?').run(id);
     try {
-        execFileSync('tmux', ['send-keys', '-t', tmuxPane, text, 'Enter'], { stdio: 'ignore', timeout: 5000 });
-        // Long text triggers tmux bracketed paste — async delay then extra Enter (non-blocking)
+        // Send text literally (-l) so special chars are safe and tmux doesn't interpret them
+        execFileSync('tmux', ['send-keys', '-t', tmuxPane, '-l', text], { stdio: 'ignore', timeout: 5000 });
+        // Delay before Enter — gives the CLI time to fully process the bracketed paste.
+        // Without this, Codex (and similar TUIs) may only show partial text because
+        // the Enter arrives inside the paste bracket and gets swallowed.
         setTimeout(() => {
             try {
                 execFileSync('tmux', ['send-keys', '-t', tmuxPane, 'Enter'], { stdio: 'ignore', timeout: 5000 });
             }
             catch { /* pane may have closed */ }
-        }, 1000);
+            // Second Enter after another delay — catches CLIs that need an extra nudge
+            // after bracketed paste ends (e.g. long messages that trigger paste mode)
+            setTimeout(() => {
+                try {
+                    execFileSync('tmux', ['send-keys', '-t', tmuxPane, 'Enter'], { stdio: 'ignore', timeout: 5000 });
+                }
+                catch { /* pane may have closed */ }
+            }, 500);
+        }, 300);
         console.error(`[bridge] delivered message ${id} to tmux pane ${tmuxPane}`);
     }
     catch (err) {

package/dist/tools/activity.js

@@ -38,7 +38,7 @@ const DEFAULT_MAX_LOG_ENTRIES = 500;
 const MILESTONE_ACTIONS = new Set([
     'task_created', 'task_completed', 'task_partial_done',
     'project_created', 'project_deleted',
-    'agent_connected', 'agent_disconnected',
+    'agent_connected', 'agent_disconnected', 'agent_renamed',
 ]);
 export async function compactActivityLog(params) {
     const db = getDb();

package/dist/tools/agent.js

@@ -16,7 +16,7 @@ export async function getAgentInstructions() {
             'Derive the project name from the **folder name** of the current working directory (e.g. `/home/user/projects/my-app` → search for "my-app"). Use search_projects with that name. If 2–3 results match, **ask the user** which project to use — never guess.',
             'list_tasks status="in_progress" and status="blocked" for the confirmed project.',
             'list_notifications unread_only=true',
-            'register_agent with a descriptive name for your role (e.g. "backend", "frontend", "lead"). Then list_agents to see who else is online. If other agents are active on the same project, check_messages for any pending messages addressed to you.',
+            'register_agent with a descriptive name for your role (e.g. "backend", "frontend", "lead"). Then list_agents to see who else is online. If 2+ disconnected agents exist for the same project path, ask the user which name to register as — the name preserves message history from previous sessions. If other agents are active on the same project, check_messages for any pending messages addressed to you.',
         ],
         state: {
             projects: projectCount,
@@ -58,8 +58,8 @@ export async function getAgentInstructions() {
             'The `context` field in ask_user is rendered as Markdown in the UI. Format it well: use `## headings` to organize sections, `**bold**` for key terms, bullet lists for options/trade-offs, and `\\`code\\`` for file paths or commands. Use real newlines (not literal \\\\n). The context should read like a well-written message, not raw debug output.',
             // ── Multi-Agent Collaboration ──────────────────────────────────────
             // Identity & discovery
-            'On startup, call register_agent with a descriptive name that reflects your role (e.g. "backend", "frontend", "designer", "qa"). If the user assigns you a role, use that. This name is how other agents address you.',
-            'Call list_agents to discover who else is online. Before starting work, check if another agent is already working on the same project — coordinate instead of duplicating effort.',
+            'On startup, call register_agent with a descriptive name that reflects your role (e.g. "backend", "frontend", "designer", "qa"). If the user assigns you a role, use that. This name is how other agents address you and how your message history is preserved across sessions.',
+            'Call list_agents to discover who else is online. If you see 2+ disconnected agents on the same project path, ask the user which name to register as — registering with an existing name resumes that agent\'s full message history. Before starting work, check if another agent is already working on the same project — coordinate instead of duplicating effort.',
             // Communication
             'Use send_to_agent for fire-and-forget updates — status changes, "I finished task X", "file Y is ready for you". Use ask_agent when you need a response before proceeding — "Should I use REST or GraphQL for this endpoint?", "Is the auth middleware ready?".',
             'When you receive a message from another agent (via check_messages), respond promptly using respond_to_message. Treat agent messages with the same priority as user messages.',

package/dist/types.d.ts

@@ -67,6 +67,7 @@ export declare const ActivityAction: z.ZodEnum<{
     agent_broadcast: "agent_broadcast";
     agent_connected: "agent_connected";
     agent_disconnected: "agent_disconnected";
+    agent_renamed: "agent_renamed";
     terminal_send_keys: "terminal_send_keys";
     terminal_captured: "terminal_captured";
     compaction_summary: "compaction_summary";

package/dist/types.js

@@ -15,7 +15,7 @@ export const ActivityAction = z.enum([
     'task_linked', 'task_unlinked', 'dependency_added', 'dependency_removed',
     'link_added', 'tag_added', 'tag_removed', 'debug_log',
     'agent_question', 'agent_question_answered', 'agent_broadcast',
-    'agent_connected', 'agent_disconnected',
+    'agent_connected', 'agent_disconnected', 'agent_renamed',
     'terminal_send_keys', 'terminal_captured',
     'compaction_summary', 'activity_compacted',
 ]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dalmasonto/taskflow-mcp",
-  "version": "1.0.25",
+  "version": "1.0.27",
   "description": "MCP server for TaskFlow — manage projects, tasks, timers, analytics via AI agents",
   "type": "module",
   "main": "dist/index.js",