@yemi33/minions 0.1.833 → 0.1.835

package/CHANGELOG.md

@@ -1,11 +1,13 @@
 # Changelog
 
-## 0.1.833 (2026-04-11)
+## 0.1.835 (2026-04-11)
 
 ### Features
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
 
 ### Fixes
+-  match playbook trigger phrase for diff-aware PRD updates
+-  engine only flags stale on plan revision, never auto-regenerates
 -  only 'updated' re-opens done work items, remove 'planned' status
 -  revert doc-chat MAX_HEIGHT to 500px
 -  hide plan card action buttons when stale banner is showing

package/dashboard.js

@@ -2251,14 +2251,9 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
             planFile: plan.source_plan, prdFile: body.file,
             title: `Update PRD from revised plan: ${plan.source_plan}`,
             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`,
+              `Source plan was revised. This is a diff-aware update — read the existing PRD and compare against the updated plan.\n\n` +
+              `**Existing implementation state:**\n${implContext}\n\n` +
+              `Follow the "Updating an Existing PRD" section in the playbook. Key: items whose requirements changed must be set to status "updated" (not "done") so the engine re-opens them.`,
             project: targetProject.name, createdBy: 'dashboard:plan-resume',
             extra: { _existingPrdFile: body.file },
           });

package/engine.js

@@ -1331,43 +1331,10 @@ function materializePlansAsWorkItems(config) {
           // Handle PRD based on current status
           const prdStatus = plan.status || (plan.requires_approval ? 'awaiting-approval' : null);
 
-          // Approved/completed/paused PRDs: flag stale — user clicks Resume to trigger diff-aware update
-          if (prdStatus && prdStatus !== 'awaiting-approval') {
+          // Flag stale for all statuses — user decides when to regenerate/resume from dashboard
+          if (prdStatus) {
             plan.planStale = true;
-            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)
-          if (prdStatus === 'awaiting-approval') {
-            log('info', `PRD ${file} invalidated (was awaiting-approval) — queuing regeneration from revised plan`);
-
-            const completedStatuses = new Set(['done', 'in-pr', 'implemented']);
-            const completedItems = (plan.missing_features || [])
-              .filter(f => completedStatuses.has(f.status))
-              .map(f => ({ id: f.id, name: f.name, status: f.status }));
-
-            const completedContext = completedItems.length > 0
-              ? `\nPreviously completed items (preserve their status in the new PRD):\n${completedItems.map(i => `- ${i.id}: ${i.name} [${i.status}]`).join('\n')}`
-              : '';
-
-            try { fs.unlinkSync(path.join(PRD_DIR, file)); } catch { /* cleanup */ }
-
-            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 queued = shared.queuePlanToPrd({
-                  planFile: plan.source_plan, prdFile: file,
-                  title: `Generate PRD from plan: ${plan.source_plan}`,
-                  description: `Plan file: plans/${plan.source_plan}\nSource plan was updated while PRD was awaiting approval — generating fresh PRD.${completedContext}`,
-                  project: targetProject.name, createdBy: 'engine:plan-revision',
-                });
-                if (queued) log('info', `Queued plan-to-prd regeneration for revised plan ${plan.source_plan} (${completedItems.length} completed items to carry over)`);
-              }
-            }
-            continue; // Old PRD deleted — skip safeWrite below
+            log('info', `PRD ${file} flagged as stale (plan revised while ${prdStatus}) — user can regenerate from dashboard`);
           }
 
           safeWrite(path.join(PRD_DIR, file), plan);

package/package.json

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