@yemi33/minions 0.1.958 → 0.1.960

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.958 (2026-04-14)
+## 0.1.960 (2026-04-14)
 
 ### Features
 - surface in-flight tool calls in lastAction (#1064)
@@ -25,6 +25,7 @@
 - add missing branch_name to central dispatch vars (#967)
 
 ### Fixes
+- fix CC queued message 'already processing' and thinking UX stacking
 - enforce --timeout 1 on all azureauth calls to prevent agent orphans (#1063)
 - cancel steering kill watcher on resume spawn (#1052) (#1062)
 - suppress warn for optional template variables (#1061)
@@ -44,9 +45,9 @@
 - show doc-chat badges on Notes and KB items (#1016)
 - steering kill on no-session re-queues dispatch instead of erroring (#1015)
 - inject cached ADO token into spawned agents (#998) (#1012)
--  qaAbort race + skip kill on completed doc-chat
 
 ### Other
+- refactor(dashboard): extract _releaseCCTab helper and CC_LOCK_WAIT_MS constant
 - test(engine): add regression tests for autoFixConflicts flag and DEFAULTS alias
 - refactor(ado): simplify ado-status and extract build/review status helpers
 - refactor(ado): extract stripRefsHeads helper, deduplicate refs/heads/ stripping

package/dashboard/js/command-center.js

@@ -547,8 +547,9 @@ async function _ccDoSend(message, skipUserMsg, forceTabId) {
       // 429 = server still releasing previous request (abort race) — retry silently up to 3 times
       if (res.status === 429 && (!activeTab._429retries || activeTab._429retries < 3)) {
         activeTab._429retries = (activeTab._429retries || 0) + 1;
+        _cleanupStreamDiv(); // remove current thinking div — prevents stacking on each retry
         await new Promise(function(r) { setTimeout(r, 1500); });
-        return await _ccDoSend(message, true); // retry with skipUserMsg=true (already displayed)
+        return await _ccDoSend(message, true, forceTabId || activeTabId); // retry — pass tabId so timer closures don't fight
       }
       activeTab._429retries = 0;
       _cleanupStreamDiv();

package/dashboard.js

@@ -471,7 +471,10 @@ setInterval(() => {
 const CC_SESSION_MAX_TURNS = Infinity;
 let ccSession = { sessionId: null, createdAt: null, lastActiveAt: null, turnCount: 0 };
 const ccInFlightTabs = new Set(); // per-tab in-flight tracking for parallel CC requests
+const ccInFlightAborts = new Map(); // tabId → abortFn — lets a new request kill the stale LLM
 const CC_INFLIGHT_TIMEOUT_MS = 2 * 60 * 1000; // 2 minutes — auto-release if request hangs
+const CC_LOCK_WAIT_MS = 200; // grace period for previous handler's finally to release lock
+function _releaseCCTab(tabId) { ccInFlightTabs.delete(tabId); ccInFlightAborts.delete(tabId); }
 
 // _ccPromptHash computed after CC_STATIC_SYSTEM_PROMPT is defined (see below)
 
@@ -3400,7 +3403,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       // Per-tab concurrency guard
       const tabId = body.tabId || 'default';
       if (ccInFlightTabs.has(tabId)) {
-        return jsonReply(res, 429, { error: 'This tab is already processing — wait or open a new tab.' });
+        await new Promise(r => setTimeout(r, CC_LOCK_WAIT_MS));
+        if (ccInFlightTabs.has(tabId)) {
+          return jsonReply(res, 429, { error: 'This tab is already processing — wait or open a new tab.' });
+        }
       }
       ccInFlightTabs.add(tabId);
 
@@ -3447,9 +3453,9 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         if (sessionReset) reply.sessionReset = true;
         return jsonReply(res, 200, reply);
       } finally {
-        ccInFlightTabs.delete(tabId);
+        _releaseCCTab(tabId);
       }
-    } catch (e) { ccInFlightTabs.delete(tabId); return jsonReply(res, 500, { error: e.message }); }
+    } catch (e) { _releaseCCTab(tabId); return jsonReply(res, 500, { error: e.message }); }
   }
 
   /** Build a lightweight input object for SSE tool events — keeps only the fields formatToolSummary needs, with truncated string values. */
@@ -3471,7 +3477,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!body.message) { res.statusCode = 400; res.end('message required'); return; }
       const tabId = body.tabId || 'default';
       if (ccInFlightTabs.has(tabId)) {
-        res.statusCode = 429; res.end('This tab is already processing'); return;
+        // Previous request still in-flight — abort its LLM (handles keep-alive abort where close event didn't fire)
+        const prevAbort = ccInFlightAborts.get(tabId);
+        if (prevAbort) { prevAbort(); }
+        await new Promise(r => setTimeout(r, CC_LOCK_WAIT_MS)); // let previous finally run and release the lock
+        if (ccInFlightTabs.has(tabId)) {
+          res.statusCode = 429; res.end('This tab is already processing'); return;
+        }
       }
       ccInFlightTabs.add(tabId);
 
@@ -3480,7 +3492,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       let _ccStreamEnded = false;
       // Kill LLM process immediately if client disconnects mid-stream
       req.on('close', () => {
-        ccInFlightTabs.delete(tabId);
+        _releaseCCTab(tabId);
         if (!_ccStreamEnded && _ccStreamAbort) {
           console.log(`[CC-stream] Client disconnected for tab ${tabId} — aborting LLM`);
           _ccStreamAbort();
@@ -3523,6 +3535,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           }
         });
         _ccStreamAbort = llmPromise.abort;
+        ccInFlightAborts.set(tabId, _ccStreamAbort);
         const result = await llmPromise;
         trackUsage('command-center', result.usage);
 
@@ -3604,10 +3617,10 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
         _ccStreamEnded = true; res.end();
       } finally {
-        ccInFlightTabs.delete(tabId);
+        _releaseCCTab(tabId);
       }
     } catch (e) {
-      ccInFlightTabs.delete(tabId);
+      _releaseCCTab(tabId);
       try { res.write('data: ' + JSON.stringify({ type: 'error', error: e.message }) + '\n\n'); } catch {}
       _ccStreamEnded = true; try { res.end(); } catch {}
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.958",
+  "version": "0.1.960",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"