@yemi33/minions 0.1.828 → 0.1.829

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.828 (2026-04-11)
+## 0.1.829 (2026-04-11)
 
 ### Features
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
@@ -11,6 +11,9 @@
 -  hide conflicting action buttons when stale PRD banner is showing
 -  diff-aware plan-to-prd triggered by Resume, not auto-dispatch
 
+### Other
+- refactor: remove dead criteria variable, DRY plan resume functions
+
 ## 0.1.823 (2026-04-11)
 
 ### Features

package/dashboard/js/render-prd.js

@@ -766,16 +766,16 @@ async function prdItemRequeue(workItemId, source, prdFile) {
   }
 }
 
-async function prdRegenerate(prdFile) {
-  if (!confirm('Source plan was revised.\n\nRegenerate PRD? An agent will compare the updated plan against existing implementation and update items accordingly.\n\nDone items stay done unless modified. New items are added. Removed items are cancelled.')) return;
+async function _planApproveAction(prdFile, skipRegen, confirmMsg, successMsg) {
+  if (!confirm(confirmMsg)) return;
   try {
     const res = await fetch('/api/plans/approve', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ file: prdFile })
+      body: JSON.stringify({ file: prdFile, skipRegen: skipRegen || undefined })
     });
     const d = await res.json();
     if (res.ok) {
-      showToast('cmd-toast', d.diffAwareUpdate ? 'Diff-aware PRD update queued' : 'Plan approved', true);
+      showToast('cmd-toast', successMsg || (d.diffAwareUpdate ? 'Diff-aware PRD update queued' : 'Plan approved'), true);
       refresh();
     } else {
       alert('Failed: ' + (d.error || 'unknown'));
@@ -783,21 +783,15 @@ async function prdRegenerate(prdFile) {
   } catch (e) { alert('Error: ' + e.message); }
 }
 
-async function prdResumeWithoutRegen(prdFile) {
-  if (!confirm('Resume this plan without regenerating the PRD?\n\nThe current PRD items will be used as-is. New work items will be materialized from any missing/planned items.')) return;
-  try {
-    const res = await fetch('/api/plans/approve', {
-      method: 'POST', headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ file: prdFile, skipRegen: true })
-    });
-    const d = await res.json();
-    if (res.ok) {
-      showToast('cmd-toast', 'Plan resumed (PRD unchanged)', true);
-      refresh();
-    } else {
-      alert('Failed: ' + (d.error || 'unknown'));
-    }
-  } catch (e) { alert('Error: ' + e.message); }
+function prdRegenerate(prdFile) {
+  _planApproveAction(prdFile, false,
+    'Source plan was revised.\n\nRegenerate PRD? An agent will compare the updated plan against existing implementation and update items accordingly.\n\nDone items stay done unless modified. New items are added. Removed items are cancelled.');
+}
+
+function prdResumeWithoutRegen(prdFile) {
+  _planApproveAction(prdFile, true,
+    'Resume this plan without regenerating the PRD?\n\nThe current PRD items will be used as-is. New work items will be materialized from any missing/planned items.',
+    'Plan resumed (PRD unchanged)');
 }
 
 function openArchive(i) {

package/engine.js

@@ -1478,7 +1478,6 @@ function materializePlansAsWorkItems(config) {
           if (existingWi && DONE_STATUSES.has(existingWi.status) && shouldReopen) {
             existingWi.status = WI_STATUS.PENDING;
             existingWi._reopened = true;
-            const criteria = (item.acceptance_criteria || []).map(c => `- ${c}`).join('\n');
             existingWi.description = buildWiDescription(item, file);
             existingWi.title = `Implement: ${item.name}`;
             created++;

package/package.json

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