@dalmasonto/taskflow-mcp 1.0.27 → 1.0.29

package/dist/agent-registry.js

@@ -27,25 +27,20 @@ function detectTmuxPane(pid) {
     return null;
 }
 /** 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. */
+ *  A name is available if no row exists or the existing row's process is dead. */
 function generateName(folderName) {
     const db = getDb();
-    const check = (name) => {
+    const isAvailable = (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
+        return !isAlive(row.pid);
     };
-    if (check(folderName))
+    if (isAvailable(folderName))
         return folderName;
     for (let i = 2; i < 100; i++) {
         const candidate = `${folderName}:${i}`;
-        if (check(candidate))
+        if (isAvailable(candidate))
             return candidate;
     }
     return `${folderName}:${Date.now()}`;
@@ -72,9 +67,12 @@ export function registerAgent(options) {
     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);
+        // If the target name already exists as another row (disconnected from a previous session),
+        // delete it to free the UNIQUE name. Messages stay — they're linked by name string, not FK.
+        const existing = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(newName);
+        if (existing && existing.id !== samePid.id) {
+            db.prepare('DELETE FROM agent_registry WHERE id = ?').run(existing.id);
+        }
         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 } });
@@ -98,23 +96,25 @@ export function registerAgent(options) {
         || (allDisconnected.length === 1 ? allDisconnected[0] : undefined);
     if (disconnected) {
         const newName = options?.customName || disconnected.name;
-        const renamed = newName !== disconnected.name;
-        // Update agent_messages if renaming to preserve history
-        if (renamed) {
-            db.prepare('UPDATE agent_messages SET sender_name = ? WHERE sender_name = ?').run(newName, disconnected.name);
-            db.prepare('UPDATE agent_messages SET recipient_name = ? WHERE recipient_name = ?').run(newName, disconnected.name);
-        }
         db.prepare('UPDATE agent_registry SET name = ?, pid = ?, tmux_pane = ?, status = ?, connected_at = ?, disconnected_at = NULL WHERE id = ?').run(newName, agentPid, tmuxPane, 'connected', ts, disconnected.id);
         const row = db.prepare('SELECT * FROM agent_registry WHERE id = ?').get(disconnected.id);
         broadcast('agent_connected', { entity: 'agent', action: 'agent_connected', payload: row });
         logActivity('agent_connected', `Agent "${newName}" reconnected`, { entityType: 'agent' });
         return newName;
     }
-    // Priority 3: All entries for this path are connected — this is a concurrent agent
-    // Generate a suffixed name to avoid collision
+    // Priority 3: No reusable disconnected entry — this is a new or concurrent agent
+    // Generate a suffixed name to avoid collision with live agents
     const baseName = options?.customName || folderName;
     const name = entries.length === 0 ? baseName : generateName(baseName);
-    db.prepare('INSERT INTO agent_registry (name, project_path, pid, tmux_pane, status, connected_at) VALUES (?, ?, ?, ?, ?, ?)').run(name, projectPath, agentPid, tmuxPane, 'connected', ts);
+    // Upsert: if a dead entry with this name exists (from another project path),
+    // reuse it in-place to preserve its message history. Otherwise insert fresh.
+    const stale = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(name);
+    if (stale) {
+        db.prepare('UPDATE agent_registry SET project_path = ?, pid = ?, tmux_pane = ?, status = ?, connected_at = ?, disconnected_at = NULL WHERE id = ?').run(projectPath, agentPid, tmuxPane, 'connected', ts, stale.id);
+    }
+    else {
+        db.prepare('INSERT INTO agent_registry (name, project_path, pid, tmux_pane, status, connected_at) VALUES (?, ?, ?, ?, ?, ?)').run(name, projectPath, agentPid, tmuxPane, 'connected', ts);
+    }
     const row = db.prepare('SELECT * FROM agent_registry WHERE name = ?').get(name);
     broadcast('agent_connected', { entity: 'agent', action: 'agent_connected', payload: row });
     logActivity('agent_connected', `Agent "${name}" connected`, { entityType: 'agent' });

package/package.json

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