@yemi33/minions 0.1.960 → 0.1.962

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.960 (2026-04-14)
+## 0.1.962 (2026-04-14)
 
 ### Features
+-  add quality standard reminder to all agent and CC prompts
 - surface in-flight tool calls in lastAction (#1064)
 - wire agentBusyReassignMs into settings UI and persist
 -  ADO throttle detection, poll guards, and dashboard banner (#1051)
@@ -22,9 +23,9 @@
 - audit and harden log buffering implementation (#971)
 - replace magic string 'active' with PR_STATUS.ACTIVE in lifecycle.js (#969)
 - fix dashboard plan-pause nested lock violation (#968)
-- add missing branch_name to central dispatch vars (#967)
 
 ### Fixes
+- fix tabId scope and close-event lock race in CC handlers
 - 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)
@@ -44,7 +45,6 @@
 - add unhandledRejection/uncaughtException handlers to engine process (#1019)
 - 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)
 
 ### Other
 - refactor(dashboard): extract _releaseCCTab helper and CC_LOCK_WAIT_MS constant

package/dashboard.js

@@ -3396,12 +3396,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
   async function handleCommandCenter(req, res) {
     if (checkRateLimit('command-center', 10)) return jsonReply(res, 429, { error: 'Rate limited — max 10 requests/minute' });
+    let tabId;
     try {
       const body = await readBody(req);
       if (!body.message) return jsonReply(res, 400, { error: 'message required' });
 
       // Per-tab concurrency guard
-      const tabId = body.tabId || 'default';
+      tabId = body.tabId || 'default';
       if (ccInFlightTabs.has(tabId)) {
         await new Promise(r => setTimeout(r, CC_LOCK_WAIT_MS));
         if (ccInFlightTabs.has(tabId)) {
@@ -3472,10 +3473,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
   async function handleCommandCenterStream(req, res) {
     if (checkRateLimit('command-center', 10)) { res.statusCode = 429; res.end('Rate limited'); return; }
+    let tabId;
     try {
       const body = await readBody(req);
       if (!body.message) { res.statusCode = 400; res.end('message required'); return; }
-      const tabId = body.tabId || 'default';
+      tabId = body.tabId || 'default';
       if (ccInFlightTabs.has(tabId)) {
         // Previous request still in-flight — abort its LLM (handles keep-alive abort where close event didn't fire)
         const prevAbort = ccInFlightAborts.get(tabId);
@@ -3490,12 +3492,16 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       res.writeHead(200, { 'Content-Type': 'text/event-stream', 'Cache-Control': 'no-cache', 'Connection': 'keep-alive' });
       let _ccStreamAbort = null;
       let _ccStreamEnded = false;
-      // Kill LLM process immediately if client disconnects mid-stream
+      // Kill LLM process immediately if client disconnects mid-stream.
+      // Guard with !_ccStreamEnded: when the stream ends normally, finally already released the lock;
+      // without the guard, this close event (which fires after res.end) could wipe a new request's lock.
       req.on('close', () => {
-        _releaseCCTab(tabId);
-        if (!_ccStreamEnded && _ccStreamAbort) {
-          console.log(`[CC-stream] Client disconnected for tab ${tabId} — aborting LLM`);
-          _ccStreamAbort();
+        if (!_ccStreamEnded) {
+          _releaseCCTab(tabId);
+          if (_ccStreamAbort) {
+            console.log(`[CC-stream] Client disconnected for tab ${tabId} — aborting LLM`);
+            _ccStreamAbort();
+          }
         }
       });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.960",
+  "version": "0.1.962",
   "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"

package/playbooks/shared-rules.md

@@ -1,3 +1,7 @@
+## Quality Standard
+
+Codex will review your changes — make sure your implementation is thorough and not lazy.
+
 ## Context Window Awareness
 
 Your context window may be compacted or summarized mid-task by Claude's automatic context management. This is normal and expected for long-running tasks. Do NOT interpret compacted or truncated context as a signal to stop early, wrap up prematurely, or skip remaining work. Continue working toward your stated objective regardless of context window state — re-read key files if needed to recover context.

package/prompts/cc-system.md

@@ -1,6 +1,10 @@
 You are the Command Center AI for "Minions" — a multi-agent software engineering orchestrator.
 You have full CLI power (read, write, edit, shell, builds) plus minions-specific actions to delegate work to agents.
 
+## Quality Standard
+
+Codex will review your changes — make sure your implementation is thorough and not lazy.
+
 ## Guardrails
 READ ONLY — never write/edit: `engine.js`, `engine/*.js`, `dashboard.js`, `dashboard/**`, `minions.js`, `bin/*.js`, `engine/control.json`, `engine/dispatch.json`, `config.json`.
 CAN modify: notes, plans, knowledge, work items, pull-requests.json, routing.md, charters, skills, playbooks, project repos.