@yemi33/minions 0.1.256 → 0.1.258

package/CHANGELOG.md

@@ -1,11 +1,17 @@
 # Changelog
 
-## 0.1.256 (2026-04-03)
+## 0.1.258 (2026-04-03)
+
+### Fixes
+-  remove last evaluate references — eval loop now creates review items
+
+## 0.1.257 (2026-04-03)
 
 ### Features
 -  streaming CC responses — text appears as it arrives via SSE
 
 ### Fixes
+-  streaming CC handler now has full parity with non-streaming
 -  remove dead 'data.actions' code from streaming CC path — caused ReferenceError
 -  CC messages no longer show as "queued" after page refresh
 

package/dashboard.js

@@ -2994,36 +2994,57 @@ 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' });
 
-      const sessionId = ccSessionValid() ? ccSession.sessionId : null;
-      const preamble = buildCCStatePreamble();
-      const prompt = preamble + '\n\n---\n\n' + body.message;
-
-      const { callLLMStreaming, trackEngineUsage: trackUsage } = require('./engine/llm');
-      const result = await callLLMStreaming(prompt, CC_STATIC_SYSTEM_PROMPT, {
-        timeout: 900000, label: 'command-center', model: 'sonnet', maxTurns: 50,
-        allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
-        sessionId,
-        onChunk: (text) => {
-          try { res.write('data: ' + JSON.stringify({ type: 'chunk', text }) + '\n\n'); } catch {}
+      try {
+        // Session management — same as non-streaming path
+        if (body.sessionId && body.sessionId !== ccSession.sessionId) {
+          ccSession = { sessionId: null, createdAt: null, lastActiveAt: null, turnCount: 0 };
         }
-      });
-      trackUsage('command-center', result.usage);
+        const wasResume = !!(ccSessionValid() && ccSession.sessionId);
+        const sessionId = wasResume ? ccSession.sessionId : null;
+        const preamble = buildCCStatePreamble();
+        const prompt = preamble + '\n\n---\n\n' + body.message;
+
+        const { callLLMStreaming, trackEngineUsage: trackUsage } = require('./engine/llm');
+        const result = await callLLMStreaming(prompt, CC_STATIC_SYSTEM_PROMPT, {
+          timeout: 900000, label: 'command-center', model: 'sonnet', maxTurns: 50,
+          allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
+          sessionId,
+          onChunk: (text) => {
+            try { res.write('data: ' + JSON.stringify({ type: 'chunk', text }) + '\n\n'); } catch {}
+          }
+        });
+        trackUsage('command-center', result.usage);
 
-      // Update session
-      const now = Date.now();
-      if (result.sessionId) {
-        ccSession = { sessionId: result.sessionId, createdAt: ccSession.createdAt || now, lastActiveAt: now, turnCount: (ccSession.turnCount || 0) + 1 };
-        safeWrite(path.join(ENGINE_DIR, 'cc-session.json'), ccSession);
-      }
+        // Handle failure — same error reporting as non-streaming
+        if (result.code !== 0 || !result.text) {
+          const debugInfo = result.code !== 0 ? `(exit code ${result.code})` : '(empty response)';
+          const stderrTail = (result.stderr || '').trim().split('\n').filter(Boolean).slice(-3).join(' | ');
+          const retryHint = ccSession.sessionId
+            ? 'Your session is still active — just send your message again to retry.'
+            : 'Try clicking **New Session** and sending your message again.';
+          res.write('data: ' + JSON.stringify({ type: 'done', text: `I had trouble processing that ${debugInfo}. ${stderrTail ? 'Detail: ' + stderrTail : ''}\n\n${retryHint}`, actions: [], sessionId: ccSession.sessionId }) + '\n\n');
+          res.end();
+          return;
+        }
 
-      // Send final result with actions
-      const { text: displayText, actions } = parseCCActions(result.text);
-      res.write('data: ' + JSON.stringify({ type: 'done', text: displayText, actions, sessionId: ccSession.sessionId }) + '\n\n');
-      res.end();
-      ccInFlight = false;
-      ccInFlightSince = 0;
+        // Update session
+        const now = Date.now();
+        if (result.sessionId) {
+          ccSession = { sessionId: result.sessionId, createdAt: ccSession.createdAt || now, lastActiveAt: now, turnCount: (ccSession.turnCount || 0) + 1 };
+          safeWrite(path.join(ENGINE_DIR, 'cc-session.json'), ccSession);
+        }
+
+        // Send final result with actions
+        const { text: displayText, actions } = parseCCActions(result.text);
+        res.write('data: ' + JSON.stringify({ type: 'done', text: displayText, actions, sessionId: ccSession.sessionId, newSession: !wasResume }) + '\n\n');
+        res.end();
+      } finally {
+        ccInFlight = false;
+        ccInFlightSince = 0;
+      }
     } catch (e) {
       ccInFlight = false;
+      ccInFlightSince = 0;
       try { res.write('data: ' + JSON.stringify({ type: 'error', error: e.message }) + '\n\n'); } catch {}
       try { res.end(); } catch {}
     }

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.256",
+  "version": "0.1.258",
   "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"