@yemi33/minions 0.1.831 → 0.1.833

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.831 (2026-04-11)
+## 0.1.833 (2026-04-11)
 
 ### Features
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
@@ -13,6 +13,8 @@
 -  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,73 +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 wiPath = path.join(MINIONS_DIR, 'work-items.json');
-
-      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 id = 'W-' + shared.uid();
-      let alreadyQueuedId = null;
-      mutateWorkItems(wiPath, items => {
-        // Dedup: check if already queued
-        const alreadyQueued = items.find(w =>
-          w.type === 'plan-to-prd' && w.planFile === plan.source_plan && (w.status === WI_STATUS.PENDING || w.status === WI_STATUS.DISPATCHED)
-        );
-        if (alreadyQueued) { alreadyQueuedId = alreadyQueued.id; return; }
-        items.push({
-          id, title: `Regenerate PRD: ${plan.plan_summary || plan.source_plan}`,
-          type: 'plan-to-prd', priority: 'high',
-          description: `Plan file: plans/${plan.source_plan}\nTarget PRD filename: ${body.file}\nRegeneration requested by user after plan revision.${completedContext}`,
-          status: WI_STATUS.PENDING, created: new Date().toISOString(), createdBy: 'dashboard:regenerate',
-          project: plan.project || '', planFile: plan.source_plan,
-          _targetPrdFile: body.file,
-        });
-      });
-      if (alreadyQueuedId) return jsonReply(res, 200, { id: alreadyQueuedId, alreadyQueued: true });
-      return jsonReply(res, 200, { id, 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' });
@@ -2440,28 +2374,15 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       const planPath = path.join(MINIONS_DIR, 'plans', body.file);
       if (!fs.existsSync(planPath)) return jsonReply(res, 404, { error: 'plan file not found' });
 
-      // Atomic check-and-insert to prevent duplicates and races with engine
-      const centralPath = path.join(MINIONS_DIR, 'work-items.json');
-      let existingId = null;
-      const id = 'W-' + shared.uid();
-      mutateJsonFileLocked(centralPath, (items) => {
-        if (!Array.isArray(items)) items = [];
-        // Only block if actively pending/dispatched — allow re-execute after completion
-        const existing = items.find(w => w.type === 'plan-to-prd' && w.planFile === body.file && (w.status === WI_STATUS.PENDING || w.status === WI_STATUS.DISPATCHED));
-        if (existing) { existingId = existing.id; return items; }
-        items.push({
-          id, title: 'Convert plan to PRD: ' + body.file.replace('.md', ''),
-          type: 'plan-to-prd', priority: 'high',
-          description: 'Plan file: plans/' + body.file,
-          status: WI_STATUS.PENDING, created: new Date().toISOString(),
-          createdBy: 'dashboard:execute', project: body.project || '',
-          planFile: body.file,
-        });
-        return items;
-      }, { defaultValue: [] });
-      if (existingId) return jsonReply(res, 200, { ok: true, id: existingId, alreadyQueued: true });
+      const queued = shared.queuePlanToPrd({
+        planFile: body.file,
+        title: 'Convert plan to PRD: ' + body.file.replace('.md', ''),
+        description: 'Plan file: plans/' + body.file,
+        project: body.project || '', createdBy: 'dashboard:execute',
+      });
+      if (!queued) return jsonReply(res, 200, { ok: true, alreadyQueued: true });
       invalidateStatusCache();
-      return jsonReply(res, 200, { ok: true, id });
+      return jsonReply(res, 200, { ok: true });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 
@@ -4179,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/package.json

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