@yemi33/minions 0.1.959 → 0.1.960

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.959 (2026-04-14)
+## 0.1.960 (2026-04-14)
 
 ### Features
 - surface in-flight tool calls in lastAction (#1064)
@@ -47,6 +47,7 @@
 - inject cached ADO token into spawned agents (#998) (#1012)
 
 ### 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

@@ -473,6 +473,8 @@ let ccSession = { sessionId: null, createdAt: null, lastActiveAt: null, turnCoun
 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)
 
@@ -3401,7 +3403,7 @@ 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)) {
-        await new Promise(r => setTimeout(r, 200));
+        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.' });
         }
@@ -3451,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. */
@@ -3478,7 +3480,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         // 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, 200)); // let previous finally run and release the lock
+        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;
         }
@@ -3490,8 +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);
-        ccInFlightAborts.delete(tabId);
+        _releaseCCTab(tabId);
         if (!_ccStreamEnded && _ccStreamAbort) {
           console.log(`[CC-stream] Client disconnected for tab ${tabId} — aborting LLM`);
           _ccStreamAbort();
@@ -3534,7 +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); // store so a new request for this tab can abort this LLM
+        ccInFlightAborts.set(tabId, _ccStreamAbort);
         const result = await llmPromise;
         trackUsage('command-center', result.usage);
 
@@ -3616,12 +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);
-        ccInFlightAborts.delete(tabId);
+        _releaseCCTab(tabId);
       }
     } catch (e) {
-      ccInFlightTabs.delete(tabId);
-      ccInFlightAborts.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.959",
+  "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"