@yemi33/minions 0.1.830 → 0.1.832

package/CHANGELOG.md

@@ -1,17 +1,19 @@
 # Changelog
 
-## 0.1.830 (2026-04-11)
+## 0.1.832 (2026-04-11)
 
 ### Features
 -  stale PRD shows Regenerate PRD + Resume as-is buttons
 
 ### Fixes
+-  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
 -  hide conflicting action buttons when stale PRD banner is showing
 -  diff-aware plan-to-prd triggered by Resume, not auto-dispatch
 
 ### Other
+- 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

@@ -2402,31 +2402,19 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       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,
-        });
+      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 (alreadyQueuedId) return jsonReply(res, 200, { id: alreadyQueuedId, alreadyQueued: true });
-      return jsonReply(res, 200, { id, file: plan.source_plan });
+      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 }); }
   }
 
@@ -2440,28 +2428,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 }); }
   }
 

package/engine/shared.js

@@ -587,8 +587,8 @@ const PLAN_STATUS = {
   PAUSED: 'paused', REJECTED: 'rejected', COMPLETED: 'completed',
   REVISION_REQUESTED: 'revision-requested',
 };
-const PRD_ITEM_STATUS = { MISSING: 'missing', PLANNED: 'planned', UPDATED: 'updated', DONE: 'done' };
-const PRD_MATERIALIZABLE = new Set([PRD_ITEM_STATUS.MISSING, PRD_ITEM_STATUS.PLANNED, PRD_ITEM_STATUS.UPDATED]);
+const PRD_ITEM_STATUS = { MISSING: 'missing', UPDATED: 'updated', DONE: 'done' };
+const PRD_MATERIALIZABLE = new Set([PRD_ITEM_STATUS.MISSING, PRD_ITEM_STATUS.UPDATED]);
 const PR_STATUS = { ACTIVE: 'active', MERGED: 'merged', ABANDONED: 'abandoned', CLOSED: 'closed', LINKED: 'linked' };
 // PRs eligible for polling (status/build/comment checks) — excludes terminal statuses
 const PR_POLLABLE_STATUSES = new Set([PR_STATUS.ACTIVE, PR_STATUS.LINKED]);

package/engine.js

@@ -25,7 +25,7 @@ const fs = require('fs');
 const path = require('path');
 const shared = require('./engine/shared');
 const { exec, execAsync, execSilent, runFile, ts, ENGINE_DEFAULTS: DEFAULTS,
-  WI_STATUS, DONE_STATUSES, WORK_TYPE, PLAN_STATUS, PRD_MATERIALIZABLE, PR_STATUS, DISPATCH_RESULT, AGENT_STATUS } = shared;
+  WI_STATUS, DONE_STATUSES, WORK_TYPE, PLAN_STATUS, PRD_ITEM_STATUS, PRD_MATERIALIZABLE, PR_STATUS, DISPATCH_RESULT, AGENT_STATUS } = shared;
 const queries = require('./engine/queries');
 
 // ─── Paths ──────────────────────────────────────────────────────────────────
@@ -1360,8 +1360,8 @@ function materializePlansAsWorkItems(config) {
               if (targetProject) {
                 const queued = shared.queuePlanToPrd({
                   planFile: plan.source_plan, prdFile: file,
-                  title: `Regenerate PRD from revised plan: ${plan.source_plan}`,
-                  description: `Plan file: plans/${plan.source_plan}\nSource plan was revised (PRD was ${prdStatus}) — regenerating.${completedContext}`,
+                  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)`);
@@ -1458,9 +1458,9 @@ function materializePlansAsWorkItems(config) {
 
       mutateWorkItems(wiPath, existingItems => {
         for (const item of projItems) {
-          // Re-open: PRD item set back to missing/planned/updated but work item is done → reset to pending
+          // Re-open: only 'updated' re-opens a done work item (missing = new item, not re-open)
           const existingWi = existingItems.find(w => w.id === item.id);
-          const shouldReopen = PRD_MATERIALIZABLE.has(item.status);
+          const shouldReopen = item.status === PRD_ITEM_STATUS.UPDATED;
           if (existingWi && DONE_STATUSES.has(existingWi.status) && shouldReopen) {
             existingWi.status = WI_STATUS.PENDING;
             existingWi._reopened = true;

package/package.json

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