@yemi33/minions 0.1.262 → 0.1.264

package/CHANGELOG.md

@@ -1,11 +1,17 @@
 # Changelog
 
-## 0.1.262 (2026-04-03)
+## 0.1.264 (2026-04-03)
+
+### Fixes
+-  clean up remaining evaluate references in comments and log messages
+
+## 0.1.263 (2026-04-03)
 
 ### Features
 -  real-time token streaming for CC via partial JSON extraction
 
 ### Fixes
+-  all CC messages go through ccAddMessage — no more duplicated styling
 -  streaming CC done event includes copy button matching ccAddMessage
 -  streaming CC message bubble matches ccAddMessage styling
 -  CC stream clears ccInFlight on client disconnect + defensive thinking.remove

package/dashboard/js/command-center.js

@@ -190,14 +190,13 @@ async function _ccDoSend(message, skipUserMsg) {
       return;
     }
 
-    // Create streaming message bubble
+    // Use a temporary streaming div for live updates, then replace with ccAddMessage on completion
     clearInterval(ccTimer);
     try { thinking.remove(); } catch { /* may already be removed */ }
-    const streamDiv = document.createElement('div');
-    streamDiv.className = 'cc-msg-assistant';
-    streamDiv.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:12px;line-height:1.6;max-width:95%;background:var(--surface2);color:var(--text);align-self:flex-start;border:1px solid var(--border);position:relative';
-    streamDiv.innerHTML = '<span style="color:var(--muted);font-size:11px">Thinking...</span>';
-    document.getElementById('cc-messages').appendChild(streamDiv);
+    // Add a temporary placeholder via ccAddMessage so it gets proper styling
+    ccAddMessage('assistant', '<span style="color:var(--muted);font-size:11px">Thinking...</span>', true);
+    const msgs = document.getElementById('cc-messages');
+    const streamDiv = msgs.lastElementChild; // the message we just added
     let streamedText = '';
 
     const reader = res.body.getReader();
@@ -215,23 +214,24 @@ async function _ccDoSend(message, skipUserMsg) {
         try {
           const evt = JSON.parse(line.slice(6));
           if (evt.type === 'chunk') {
-            streamedText = evt.text; // each chunk is the full text so far for this turn
+            streamedText = evt.text;
             streamDiv.innerHTML = renderMd(streamedText);
-            const msgs = document.getElementById('cc-messages');
             if (msgs.scrollHeight - msgs.scrollTop - msgs.clientHeight < 150) msgs.scrollTop = msgs.scrollHeight;
           } else if (evt.type === 'done') {
-            // Final result — replace with rendered markdown + copy button + timer
+            // Replace streaming div with a proper ccAddMessage
+            streamDiv.remove();
+            _ccMessages.pop(); // remove the placeholder we pushed
             const ccElapsed = Math.round((Date.now() - ccStartTime) / 1000);
-            const finalHtml = renderMd(evt.text || streamedText || '');
-            streamDiv.innerHTML = llmCopyBtn() + finalHtml +
-              '<div style="font-size:9px;color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + ccElapsed + 's</div>';
-            _ccMessages.push({ role: 'assistant', html: streamDiv.innerHTML });
+            const rendered = renderMd(evt.text || streamedText || '');
+            ccAddMessage('assistant', rendered + '<div style="font-size:9px;color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + ccElapsed + 's</div>');
             if (evt.sessionId) { _ccSessionId = evt.sessionId; ccSaveState(); ccUpdateSessionIndicator(); }
             if (evt.actions && evt.actions.length > 0) {
               for (const action of evt.actions) { await ccExecuteAction(action); }
             }
           } else if (evt.type === 'error') {
-            streamDiv.innerHTML = '<span style="color:var(--red)">' + escHtml(evt.error) + '</span>';
+            streamDiv.remove();
+            _ccMessages.pop();
+            ccAddMessage('assistant', '<span style="color:var(--red)">' + escHtml(evt.error) + '</span>');
           }
         } catch { /* incomplete JSON */ }
       }

package/engine/lifecycle.js

@@ -1075,7 +1075,7 @@ function resolveWiPath(meta) {
 }
 
 /**
- * Parse structured eval verdict from evaluate agent output.
+ * Parse structured eval verdict from review agent output.
  * Looks for a JSON block with { pass, build, tests, criteria_met, criteria_failed, feedback }.
  * Returns parsed object or null if not found.
  */
@@ -1256,7 +1256,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
 
   if (isSuccess && meta?.item?.id && !skipDoneStatus) updateWorkItemStatus(meta, 'done', '');
 
-  // Auto-dispatch evaluate work item after implement or fix completes successfully
+  // Auto-dispatch review work item after implement or fix completes successfully
   if (isSuccess && !skipDoneStatus && (type === 'implement' || (type === 'fix' && meta?.item?._evalParentId)) && meta?.item?.id) {
     const evalLoop = config.engine?.evalLoop ?? shared.ENGINE_DEFAULTS.evalLoop;
     if (evalLoop) {
@@ -1269,7 +1269,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
             // 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`);
+              log('info', `Eval loop: review item ${existing.id} already exists for ${evalTargetId}, skipping`);
               return items;
             }
             const parentItem = items.find(i => i.id === evalTargetId);

package/engine/shared.js

@@ -356,8 +356,8 @@ const ENGINE_DEFAULTS = {
   autoApprovePlans: false, // auto-approve PRDs without waiting for human approval
   autoReview: true, // auto-dispatch review agents for new PRs (disable for manual review workflow)
   meetingRoundTimeout: 600000, // 10min per meeting round before auto-advance
-  evalLoop: true, // enable evaluate→fix loop after implementation completes
-  evalMaxIterations: 3, // max evaluate→fix cycles before escalating to human
+  evalLoop: true, // enable review→fix loop after implementation completes
+  evalMaxIterations: 3, // max review→fix cycles before escalating to human
   evalMaxCost: null, // USD ceiling per work item across all eval iterations; null = no limit (gather baseline data first)
 };
 

package/package.json

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