@yemi33/minions 0.1.832 → 0.1.834

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.832 (2026-04-11)
+## 0.1.834 (2026-04-11)
 
 ### Features
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
 
 ### Fixes
+-  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
@@ -13,6 +14,7 @@
 -  diff-aware plan-to-prd triggered by Resume, not auto-dispatch
 
 ### Other
+- remove: /api/prd/regenerate endpoint (destructive delete + fresh gen)
 - refactor: migrate handlePrdRegenerate and handlePlansExecute to queuePlanToPrd
 - refactor: add PRD_ITEM_STATUS/PRD_MATERIALIZABLE constants, extract queuePlanToPrd
 - refactor: remove dead criteria variable, DRY plan resume functions

package/dashboard.js

@@ -2362,61 +2362,7 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 
-  async function handlePrdRegenerate(req, res) {
-    try {
-      const body = await readBody(req);
-      if (!body.file) return jsonReply(res, 400, { error: 'file is required' });
-      shared.sanitizePath(body.file, PRD_DIR);
-
-      const prdPath = path.join(PRD_DIR, body.file);
-      const plan = safeJson(prdPath);
-      if (!plan) return jsonReply(res, 404, { error: 'PRD file not found' });
-      if (!plan.source_plan) return jsonReply(res, 400, { error: 'PRD has no source_plan — cannot regenerate' });
-
-      const sourcePlanPath = path.join(PLANS_DIR, plan.source_plan);
-      if (!fs.existsSync(sourcePlanPath)) return jsonReply(res, 400, { error: `Source plan not found: ${plan.source_plan}` });
-
-      // Collect completed item IDs from the old PRD to carry over
-      const completedStatuses = new Set(['done', 'in-pr', 'implemented']); // in-pr kept for backward compat
-      const completedItems = (plan.missing_features || [])
-        .filter(f => completedStatuses.has(f.status))
-        .map(f => ({ id: f.id, name: f.name, status: f.status }));
-
-      // Clean pending/failed work items from old PRD (keep done items)
-      const { getProjects, projectWorkItemsPath } = shared;
-      const config = queries.getConfig();
-      for (const p of getProjects(config)) {
-        const projWiPath = projectWorkItemsPath(p);
-        try {
-          mutateWorkItems(projWiPath, items => {
-            const filtered = items.filter(w => {
-              if (w.sourcePlan !== body.file) return true; // different plan, keep
-              return completedStatuses.has(w.status); // keep completed, remove pending/failed
-            });
-            if (filtered.length < items.length) return filtered;
-          });
-        } catch { /* project may not have work items */ }
-      }
-
-      // Delete old PRD — agent will write replacement at same path
-      try { fs.unlinkSync(prdPath); } catch { /* cleanup */ }
-
-      // Queue plan-to-prd regeneration with instructions to preserve completed items
-      const completedContext = completedItems.length > 0
-        ? `\n\n**Previously completed items (preserve their status in the new PRD):**\n${completedItems.map(i => `- ${i.id}: ${i.name} [${i.status}]`).join('\n')}`
-        : '';
-
-      const queued = shared.queuePlanToPrd({
-        planFile: plan.source_plan, prdFile: body.file,
-        title: `Regenerate PRD: ${plan.plan_summary || plan.source_plan}`,
-        description: `Plan file: plans/${plan.source_plan}\nTarget PRD filename: ${body.file}\nRegeneration requested by user after plan revision.${completedContext}`,
-        project: plan.project || '', createdBy: 'dashboard:regenerate',
-        extra: { _targetPrdFile: body.file },
-      });
-      if (!queued) return jsonReply(res, 200, { alreadyQueued: true });
-      return jsonReply(res, 200, { ok: true, file: plan.source_plan });
-    } catch (e) { return jsonReply(res, 500, { error: e.message }); }
-  }
+  // handlePrdRegenerate removed — destructive delete+regen replaced by diff-aware update via /api/plans/approve
 
   async function handlePlansExecute(req, res) {
     if (checkRateLimit('plans-execute', 5)) return jsonReply(res, 429, { error: 'Rate limited — max 5 requests/minute' });
@@ -4154,7 +4100,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'POST', path: '/api/prd-items', desc: 'Create a PRD item as a plan file in prd/ (auto-approved)', params: 'name, description?, priority?, estimated_complexity?, project?, id?', handler: handlePrdItemsCreate },
     { method: 'POST', path: '/api/prd-items/update', desc: 'Edit a PRD item in its source plan JSON', params: 'source, itemId, name?, description?, priority?, estimated_complexity?, status?', handler: handlePrdItemsUpdate },
     { method: 'POST', path: '/api/prd-items/remove', desc: 'Remove a PRD item from plan + cancel materialized work item', params: 'source, itemId', handler: handlePrdItemsRemove },
-    { method: 'POST', path: '/api/prd/regenerate', desc: 'Regenerate PRD from revised source plan', params: 'file', handler: handlePrdRegenerate },
+    // /api/prd/regenerate removed — use /api/plans/approve which does diff-aware update
 
     // Agents
     { method: 'POST', path: '/api/pull-requests/link', desc: 'Manually link an external PR for tracking', params: 'url, title?, project?, autoObserve?, context?', handler: async (req, res) => {

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.832",
+  "version": "0.1.834",
   "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"