@yemi33/minions 0.1.822 → 0.1.824

package/CHANGELOG.md

@@ -1,6 +1,11 @@
 # Changelog
 
-## 0.1.822 (2026-04-11)
+## 0.1.824 (2026-04-11)
+
+### Fixes
+-  diff-aware plan-to-prd triggered by Resume, not auto-dispatch
+
+## 0.1.823 (2026-04-11)
 
 ### Features
 -  add resume-plan CC action for diff-aware plan updates
@@ -18,6 +23,7 @@
 -  cap review→fix cycles at evalMaxIterations (default 3)
 
 ### Fixes
+-  clear planStale on approve/resume so materialization proceeds
 -  doc-chat resize cap raised, modal overflow + flex body layout
 -  prevent duplicate verify WIs per PRD, re-open on modified plan
 -  strip ===ACTIONS=== server-side in streaming chunks (not client)
@@ -37,7 +43,6 @@
 - allow PRD item retry when work item is missing (closes #781) (#807)
 -  handle force-pushed dep branches on retry (closes #738) (#806)
 -  project-scoped skill work items instruct wrong file path format (closes #790) (#804)
--  move ignoredAuthors construction outside inner comment loop (ADO)
 
 ### Other
 - docs: diff-aware PRD update playbook + prioritize team memory in shared rules

package/dashboard.js

@@ -961,6 +961,7 @@ const server = http.createServer(async (req, res) => {
         if (!plan) return jsonReply(res, 500, { error: 'Could not parse PRD file' });
         plan.status = 'approved';
         delete plan.completedAt;
+        delete plan.planStale;
         safeWrite(activePath, plan);
       }
 
@@ -2186,7 +2187,8 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       plan.approvedAt = new Date().toISOString();
       plan.approvedBy = body.approvedBy || os.userInfo().username;
       delete plan.pausedAt;
-      delete plan._completionNotified; // Allow re-completion after new work finishes
+      delete plan.planStale;
+      delete plan._completionNotified;
       safeWrite(planPath, plan);
 
       // Resume paused work items across all projects
@@ -2225,11 +2227,63 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
         }, { defaultValue: { pending: [], active: [], completed: [] } });
       }
 
+      // Diff-aware PRD update: if plan has done work and a source_plan .md, dispatch plan-to-prd
+      // to compare the revised plan against existing implementation
+      let diffAwareQueued = false;
+      if (plan.source_plan && plan.missing_features) {
+        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 doneWis = planWis.filter(w => shared.DONE_STATUSES.has(w.status));
+        if (doneWis.length > 0) {
+          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 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: 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.822",
+  "version": "0.1.824",
   "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"