@yemi33/minions 0.1.823 → 0.1.825

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.825 (2026-04-11)
+
+### Features
+-  stale PRD shows Regenerate PRD + Resume as-is buttons
+
+### Fixes
+-  diff-aware plan-to-prd triggered by Resume, not auto-dispatch
+
 ## 0.1.823 (2026-04-11)
 
 ### Features

package/dashboard/js/render-prd.js

@@ -269,7 +269,8 @@ function renderPrdProgress(prog) {
     const staleRecovery = (!isBlocked && g.planStale)
       ? '<div style="width:100%;margin-top:4px;padding:6px 8px;border:1px solid rgba(210,153,34,0.35);border-radius:4px;background:rgba(210,153,34,0.08);display:flex;align-items:center;gap:8px;flex-wrap:wrap">' +
           '<span style="color:var(--orange);font-size:10px;font-weight:600">&#x26A0;&#xFE0F; Source plan was revised. This PRD may be outdated.</span>' +
-          '<span onclick="event.stopPropagation();prdRegenerate(\'' + escHtml(g.file) + '\')" style="color:var(--green);cursor:pointer;font-size:10px;font-weight:700;padding:2px 8px;background:rgba(63,185,80,0.12);border:1px solid rgba(63,185,80,0.35);border-radius:4px" title="Regenerate PRD from latest plan">Regenerate now</span>' +
+          '<span onclick="event.stopPropagation();prdRegenerate(\'' + escHtml(g.file) + '\')" style="color:var(--green);cursor:pointer;font-size:10px;font-weight:700;padding:2px 8px;background:rgba(63,185,80,0.12);border:1px solid rgba(63,185,80,0.35);border-radius:4px" title="Compare revised plan against existing PRD and update items">Regenerate PRD</span>' +
+          '<span onclick="event.stopPropagation();prdResumeWithoutRegen(\'' + escHtml(g.file) + '\')" style="color:var(--muted);cursor:pointer;font-size:10px;padding:2px 8px;background:var(--surface);border:1px solid var(--border);border-radius:4px" title="Resume without regenerating — use current PRD as-is">Resume as-is</span>' +
           '<span onclick="event.stopPropagation();planView(\'' + escHtml(g.sourcePlan || g.file) + '\')" style="color:var(--blue);cursor:pointer;font-size:10px;padding:2px 8px;background:rgba(56,139,253,0.1);border:1px solid rgba(56,139,253,0.3);border-radius:4px" title="Review latest plan changes">Review plan</span>' +
         '</div>'
       : '';
@@ -763,15 +764,32 @@ async function prdItemRequeue(workItemId, source, prdFile) {
 }
 
 async function prdRegenerate(prdFile) {
-  if (!confirm('This PRD is stale because the source plan changed.\n\nRegenerate now from the latest plan?\n\nThis resets PRD status to awaiting-approval and queues a fresh plan-to-prd conversion.')) return;
+  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;
   try {
-    const res = await fetch('/api/prd/regenerate', {
+    const res = await fetch('/api/plans/approve', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ file: prdFile })
     });
     const d = await res.json();
     if (res.ok) {
-      showToast('cmd-toast', 'PRD regeneration queued', true);
+      showToast('cmd-toast', d.diffAwareUpdate ? 'Diff-aware PRD update queued' : 'Plan approved', true);
+      refresh();
+    } else {
+      alert('Failed: ' + (d.error || 'unknown'));
+    }
+  } 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'));

package/dashboard.js

@@ -2183,6 +2183,7 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       if (!body.file) return jsonReply(res, 400, { error: 'file required' });
       const planPath = resolvePlanPath(body.file);
       const plan = safeJsonObj(planPath);
+      const wasStale = !!plan.planStale;
       plan.status = 'approved';
       plan.approvedAt = new Date().toISOString();
       plan.approvedBy = body.approvedBy || os.userInfo().username;
@@ -2227,11 +2228,60 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
         }, { defaultValue: { pending: [], active: [], completed: [] } });
       }
 
+      // Diff-aware PRD update: if plan was stale (source .md revised), dispatch plan-to-prd
+      // to compare the revised plan against existing PRD and produce an updated version
+      let diffAwareQueued = false;
+      if (plan.source_plan && plan.missing_features && wasStale && !body.skipRegen) {
+        const config = queries.getConfig();
+        const allWorkItems = queries.getWorkItems(config);
+        const planWis = allWorkItems.filter(w => w.sourcePlan === body.file && w.itemType !== 'pr' && w.itemType !== 'verify');
+        const allPrs = PROJECTS.flatMap(p => shared.safeJson(shared.projectPrPath(p)) || []);
+        const prLinks = shared.getPrLinks();
+        const implContext = (plan.missing_features || []).map(f => {
+          const wi = planWis.find(w => w.id === f.id);
+          const pr = allPrs.find(p => prLinks[p.id] === f.id || (p.prdItems || []).includes(f.id));
+          return `- **${f.id}**: ${f.name} [status: ${wi?.status || f.status}]${pr ? ` (PR: ${pr.id}, branch: \`${pr.branch}\`)` : ''}`;
+        }).join('\n');
+
+        const projectName = plan.project || body.file.replace(/-\d{4}-\d{2}-\d{2}\.json$/, '');
+        const targetProject = PROJECTS.find(p => p.name?.toLowerCase() === projectName.toLowerCase()) || PROJECTS[0];
+        if (targetProject) {
+          const centralWiPath = path.join(MINIONS_DIR, 'work-items.json');
+          mutateJsonFileLocked(centralWiPath, items => {
+            if (!Array.isArray(items)) items = [];
+            if (items.some(w => w.type === 'plan-to-prd' && w.planFile === plan.source_plan && (w.status === 'pending' || w.status === 'dispatched'))) return items;
+            items.push({
+              id: 'W-' + shared.uid(),
+              title: `Update PRD from revised plan: ${plan.source_plan}`,
+              type: 'plan-to-prd',
+              priority: 'high',
+              description: `Plan file: plans/${plan.source_plan}\nPRD file: prd/${body.file}\n\n` +
+                `Source plan was revised. Compare the updated plan against existing PRD and produce an updated version.\n\n` +
+                `**Current PRD implementation state:**\n${implContext}\n\n` +
+                `**Rules for updating:**\n` +
+                `- Items that are done and unchanged in the plan → keep status "done" (preserve their ID)\n` +
+                `- Items that are done but modified in the plan (new requirements) → set status "updated" (engine will re-open)\n` +
+                `- New items in the plan → set status "missing" (engine will materialize)\n` +
+                `- Items removed from the plan → drop from PRD (engine will cancel pending WIs)\n` +
+                `- Preserve all existing item IDs for unchanged/modified items — do NOT generate new IDs for them`,
+              status: 'pending',
+              created: new Date().toISOString(),
+              createdBy: 'dashboard:plan-resume',
+              project: targetProject.name,
+              planFile: plan.source_plan,
+              _existingPrdFile: body.file,
+            });
+            diffAwareQueued = true;
+            return items;
+          }, { defaultValue: [] });
+        }
+      }
+
       // Teams notification for plan approval — non-blocking
       try { teams.teamsNotifyPlanEvent({ name: plan.plan_summary || body.file, file: body.file }, 'plan-approved').catch(() => {}); } catch {}
 
       invalidateStatusCache();
-      return jsonReply(res, 200, { ok: true, status: 'approved', resumedWorkItems: resumed });
+      return jsonReply(res, 200, { ok: true, status: 'approved', resumedWorkItems: resumed, diffAwareUpdate: diffAwareQueued });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 

package/engine.js

@@ -1331,65 +1331,10 @@ function materializePlansAsWorkItems(config) {
           // Handle PRD based on current status
           const prdStatus = plan.status || (plan.requires_approval ? 'awaiting-approval' : null);
 
-          // Approved/completed PRDs with existing work: auto-dispatch diff-aware plan-to-prd update
-          if (prdStatus === PLAN_STATUS.APPROVED || prdStatus === PLAN_STATUS.COMPLETED) {
-            const allWorkItems = queries.getWorkItems(config);
-            const planWis = allWorkItems.filter(w => w.sourcePlan === file && w.itemType !== 'pr' && w.itemType !== 'verify');
-            const doneWis = planWis.filter(w => DONE_STATUSES.has(w.status));
-            if (doneWis.length > 0) {
-              const allPrs = getProjects(config).flatMap(p => safeJson(projectPrPath(p)) || []);
-              const prLinks = shared.getPrLinks();
-              const implContext = (plan.missing_features || []).map(f => {
-                const wi = planWis.find(w => w.id === f.id);
-                const pr = allPrs.find(p => prLinks[p.id] === f.id || (p.prdItems || []).includes(f.id));
-                return `- **${f.id}**: ${f.name} [status: ${wi?.status || f.status}]${pr ? ` (PR: ${pr.id}, branch: \`${pr.branch}\`)` : ''}`;
-              }).join('\n');
-
-              const planContent = safeRead(path.join(PLANS_DIR, plan.source_plan));
-              if (planContent) {
-                const projectName = plan.project || file.replace(/-\d{4}-\d{2}-\d{2}\.json$/, '');
-                const allProjects = getProjects(config);
-                const targetProject = allProjects.find(p => p.name?.toLowerCase() === projectName.toLowerCase()) || allProjects[0];
-                if (targetProject) {
-                  const centralWiPath = path.join(MINIONS_DIR, 'work-items.json');
-                  let queued = false;
-                  mutateJsonFileLocked(centralWiPath, items => {
-                    if (!Array.isArray(items)) items = [];
-                    if (items.some(w => w.type === WORK_TYPE.PLAN_TO_PRD && w.planFile === plan.source_plan && (w.status === WI_STATUS.PENDING || w.status === WI_STATUS.DISPATCHED))) return items;
-                    items.push({
-                      id: 'W-' + shared.uid(),
-                      title: `Update PRD from revised plan: ${plan.source_plan}`,
-                      type: 'plan-to-prd',
-                      priority: 'high',
-                      description: `Plan file: plans/${plan.source_plan}\nPRD file: prd/${file}\n\n` +
-                        `Source plan was revised. Compare the updated plan against existing implementation and update the PRD.\n\n` +
-                        `**Existing implementation state:**\n${implContext}\n\n` +
-                        `**Rules for updating:**\n` +
-                        `- Items that are done and unchanged in the plan → keep status "done" (preserve their ID)\n` +
-                        `- Items that are done but modified in the plan (new requirements) → set status "updated" (engine will re-open)\n` +
-                        `- New items in the plan → set status "missing" (engine will materialize)\n` +
-                        `- Items removed from the plan → drop from PRD (engine will cancel pending WIs)\n` +
-                        `- Preserve all existing item IDs for unchanged/modified items — do NOT generate new IDs for them`,
-                      status: 'pending',
-                      created: ts(),
-                      createdBy: 'engine:plan-revision',
-                      project: targetProject.name,
-                      planFile: plan.source_plan,
-                      _existingPrdFile: file,
-                    });
-                    queued = true;
-                    return items;
-                  }, { defaultValue: [] });
-                  if (queued) log('info', `Queued diff-aware plan-to-prd update for ${plan.source_plan} (${doneWis.length} done, ${planWis.length} total items)`);
-                }
-              }
-            } else {
-              plan.planStale = true;
-              log('info', `PRD ${file} flagged as stale (plan revised while ${prdStatus}, no done items) — user can re-execute from dashboard`);
-            }
-          } else if (prdStatus === PLAN_STATUS.PAUSED) {
+          // Approved/completed/paused PRDs: flag stale — user clicks Resume to trigger diff-aware update
+          if (prdStatus && prdStatus !== 'awaiting-approval') {
             plan.planStale = true;
-            log('info', `PRD ${file} flagged as stale (plan revised while paused) — user can re-execute from dashboard`);
+            log('info', `PRD ${file} flagged as stale (plan revised while ${prdStatus}) — user can resume from dashboard`);
           }
 
           // Awaiting-approval PRDs: auto-regenerate (no work started yet, safe to replace)

package/package.json

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