@yemi33/minions 0.1.818 → 0.1.820

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.818 (2026-04-11)
+## 0.1.820 (2026-04-11)
 
 ### Features
 -  verify workflow handles shared-branch plans and existing E2E PRs
@@ -17,6 +17,8 @@
 -  cap review→fix cycles at evalMaxIterations (default 3)
 
 ### Fixes
+-  prevent duplicate verify WIs per PRD, re-open on modified plan
+-  strip ===ACTIONS=== server-side in streaming chunks (not client)
 -  strip ===ACTIONS=== from CC streamed text during rendering
 -  ccExecuteAction routes action status to correct tab via targetTabId
 -  clear _completionNotified on plan resume for re-completion
@@ -35,8 +37,6 @@
 -  project-scoped skill work items instruct wrong file path format (closes #790) (#804)
 -  move ignoredAuthors construction outside inner comment loop (ADO)
 -  approved is permanent — no path can ever downgrade it
--  allow fix dispatch on approved PRs (only guard is in updatePrAfterFix)
--  bot comment on approved PR can no longer reset vote
 
 ### Other
 - docs: diff-aware PRD update playbook + prioritize team memory in shared rules

package/dashboard/js/command-center.js

@@ -149,10 +149,7 @@ function ccSwitchTab(id) {
         html += '</div>';
       }
       var text = tab._streamedText || '';
-      if (text) {
-        var displayRestore = text.indexOf('===ACTIONS===') >= 0 ? text.slice(0, text.indexOf('===ACTIONS===')).trim() : text;
-        html += renderMd(displayRestore);
-      }
+      if (text) html += renderMd(text);
       var ms = Date.now() - restoreStart;
       var label = 'Thinking...';
       for (var pi = phases.length - 1; pi >= 0; pi--) { if (ms >= phases[pi][0]) { label = phases[pi][1]; break; } }
@@ -513,8 +510,7 @@ async function _ccDoSend(message, skipUserMsg) {
         html += '</div>';
       }
       if (streamedText) {
-        var displayText = streamedText.indexOf('===ACTIONS===') >= 0 ? streamedText.slice(0, streamedText.indexOf('===ACTIONS===')).trim() : streamedText;
-        html += renderMd(displayText);
+        html += renderMd(streamedText);
       }
       html += '<div style="margin-top:' + (streamedText ? '6px' : '0') + '">' + _getThinkingHtml() + '</div>';
       streamDiv.innerHTML = html;

package/dashboard.js

@@ -3524,7 +3524,9 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
           sessionId, effort: streamEffort, direct: true,
           onChunk: (text) => {
-            try { res.write('data: ' + JSON.stringify({ type: 'chunk', text }) + '\n\n'); } catch {}
+            const actIdx = text.indexOf('===ACTIONS===');
+            const display = actIdx >= 0 ? text.slice(0, actIdx).trim() : text;
+            try { res.write('data: ' + JSON.stringify({ type: 'chunk', text: display }) + '\n\n'); } catch {}
           },
           onToolUse: (name, input) => {
             try { res.write('data: ' + JSON.stringify({ type: 'tool', name, input: typeof input === 'string' ? input.slice(0, 200) : JSON.stringify(input).slice(0, 200) }) + '\n\n'); } catch {}

package/engine/lifecycle.js

@@ -185,8 +185,11 @@ function checkPlanCompletion(meta, config) {
   }
 
   // 4. Create verification work item (build, test, start webapp, write testing guide)
+  // Only one verify per PRD — skip if pending/dispatched, re-open if done/failed (PRD was modified)
   const existingVerify = allWorkItems.find(w => w.sourcePlan === planFile && w.itemType === 'verify');
-  if (!existingVerify && doneItems.length > 0) {
+  if (existingVerify && (existingVerify.status === WI_STATUS.PENDING || existingVerify.status === WI_STATUS.DISPATCHED)) {
+    log('info', `Plan ${planFile}: verify WI ${existingVerify.id} already ${existingVerify.status} — skipping`);
+  } else if (doneItems.length > 0) {
     const verifyId = 'PL-' + shared.uid();
     const planSlug = planFile.replace('.json', '');
 
@@ -301,6 +304,24 @@ function checkPlanCompletion(meta, config) {
       const teams = require('./teams');
       teams.teamsNotifyPlanEvent({ name: plan.plan_summary || planFile, file: planFile }, 'verify-created').catch(() => {});
     } catch {}
+  } else if (existingVerify && DONE_STATUSES.has(existingVerify.status) && doneItems.length > 0) {
+    // PRD was modified and re-completed — re-open the existing verify instead of creating a duplicate
+    const verifyProject = existingVerify.project || projectName;
+    const vWiPath = shared.projectWorkItemsPath(
+      projects.find(p => p.name?.toLowerCase() === verifyProject?.toLowerCase()) || primaryProject
+    );
+    mutateWorkItems(vWiPath, items => {
+      const v = items.find(w => w.id === existingVerify.id);
+      if (v && DONE_STATUSES.has(v.status)) {
+        v.status = WI_STATUS.PENDING;
+        v._reopened = true;
+        delete v.completedAt;
+        delete v.dispatched_to;
+        delete v.dispatched_at;
+        v._retryCount = 0;
+      }
+    });
+    log('info', `Re-opened verification work item ${existingVerify.id} for modified plan ${planFile}`);
   }
 
   // Archive deferred until verify completes

package/package.json

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