@yemi33/minions 0.1.257 → 0.1.259

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.1.259 (2026-04-03)
+
+### Fixes
+-  CC stream clears ccInFlight on client disconnect + defensive thinking.remove
+-  remove last evaluate references — eval loop now creates review items
+
 ## 0.1.257 (2026-04-03)
 
 ### Features

package/dashboard/js/command-center.js

@@ -192,7 +192,7 @@ async function _ccDoSend(message, skipUserMsg) {
 
     // Create streaming message bubble
     clearInterval(ccTimer);
-    thinking.remove();
+    try { thinking.remove(); } catch { /* may already be removed */ }
     const streamDiv = document.createElement('div');
     streamDiv.className = 'cc-msg assistant';
     streamDiv.innerHTML = '<span style="color:var(--muted);font-size:11px">Thinking...</span>';
@@ -236,7 +236,7 @@ async function _ccDoSend(message, skipUserMsg) {
     }
   } catch (e) {
     clearInterval(ccTimer);
-    thinking.remove();
+    try { thinking.remove(); } catch { /* may already be removed */ }
     const retryId = 'cc-retry-' + Date.now();
     ccAddMessage('assistant', '<span style="color:var(--red)">Error: ' + escHtml(e.message) + '</span>' +
       '<button id="' + retryId + '" onclick="ccRetryLast()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--blue);cursor:pointer;font-size:11px">Retry</button>');

package/dashboard.js

@@ -2993,6 +2993,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       ccInFlightSince = Date.now();
 
       res.writeHead(200, { 'Content-Type': 'text/event-stream', 'Cache-Control': 'no-cache', 'Connection': 'keep-alive' });
+      // Clear ccInFlight if client disconnects mid-stream
+      req.on('close', () => { ccInFlight = false; ccInFlightSince = 0; });
 
       try {
         // Session management — same as non-streaming path

package/engine/lifecycle.js

@@ -1266,8 +1266,8 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
           mutateJsonFileLocked(wiPath, (items) => {
             // For fix items, target the original implement parent; for implement, target self
             const evalTargetId = type === 'fix' ? meta.item._evalParentId : meta.item.id;
-            // Dedup: skip if an evaluate item already exists for this parent
-            const existing = items.find(i => i._evalParentId === evalTargetId && i.type === 'evaluate' && i.status === 'pending');
+            // Dedup: skip if a review item already exists for this parent
+            const existing = items.find(i => i._evalParentId === evalTargetId && i.type === 'review' && i.status === 'pending');
             if (existing) {
               log('info', `Eval loop: evaluate item ${existing.id} already exists for ${evalTargetId}, skipping`);
               return items;
@@ -1276,7 +1276,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
             const evalItem = {
               id: 'W-' + shared.uid(),
               title: `Evaluate: ${parentItem?.title || meta.item.title || evalTargetId}`,
-              type: 'evaluate',
+              type: 'review',
               priority: meta.item.priority || 'high',
               status: 'pending',
               created: ts(),
@@ -1302,7 +1302,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   }
 
   // Evaluate completion: parse verdict and handle eval→fix iteration loop
-  if (isSuccess && type === 'evaluate' && meta?.item?._evalParentId) {
+  if (isSuccess && type === 'review' && meta?.item?._evalParentId) {
     try {
       const verdict = parseEvalVerdict(resultSummary || stdout);
       const evalLoop = config.engine?.evalLoop ?? shared.ENGINE_DEFAULTS.evalLoop;

package/engine/playbook.js

@@ -481,7 +481,7 @@ function selectPlaybook(workType, item) {
   if (workType === 'review' && !item?._pr && !item?.pr_id) {
     return 'work-item';
   }
-  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'evaluate', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
+  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
   return typeSpecificPlaybooks.includes(workType) ? workType : 'work-item';
 }
 

package/package.json

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