@yemi33/minions 0.1.897 → 0.1.899

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.899 (2026-04-12)
+
+### Fixes
+- add backward-scan in syncPrdItemStatus to re-open done work items on PRD reset (closes #929) (#940)
+
+## 0.1.898 (2026-04-12)
+
+### Fixes
+- preserve updated status across successive diff-aware regenerations (closes #928) (#938)
+
 ## 0.1.897 (2026-04-12)
 
 ### Fixes

package/dashboard/js/render-prd.js

@@ -95,6 +95,7 @@ function renderPrdProgress(prog) {
     const inProgress = items.filter(i => i.status === 'dispatched').length;
     const failed = items.filter(i => i.status === 'failed').length;
     const paused = items.filter(i => i.status === 'paused').length;
+    const updated = items.filter(i => i.status === 'updated').length;
     const missing = items.filter(i => i.status === 'missing' || !i.status).length;
     const pct = (n) => total > 0 ? ((n / total) * 100).toFixed(1) : 0;
 
@@ -103,6 +104,7 @@ function renderPrdProgress(prog) {
       '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--yellow)">' + inProgress + '</div><div class="prd-stat-label">Active</div></div>' +
       (failed ? '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--red)">' + failed + '</div><div class="prd-stat-label">Failed</div></div>' : '') +
       (paused ? '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--muted)">' + paused + '</div><div class="prd-stat-label">Paused</div></div>' : '') +
+      (updated ? '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--purple)">' + updated + '</div><div class="prd-stat-label">Needs Redo</div></div>' : '') +
       '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--muted)">' + missing + '</div><div class="prd-stat-label">To Do</div></div>' +
       '<div class="prd-stat"><div class="prd-stat-num" style="color:var(--text)">' + total + '</div><div class="prd-stat-label">Total</div></div>' +
     '</div>';
@@ -110,6 +112,7 @@ function renderPrdProgress(prog) {
     const bar = '<div class="prd-progress-bar">' +
       '<div class="seg complete" style="width:' + pct(done) + '%"></div>' +
       '<div class="seg dispatched" style="width:' + pct(inProgress) + '%"></div>' +
+      '<div class="seg updated" style="width:' + pct(updated) + '%"></div>' +
       '<div class="seg paused" style="width:' + pct(paused) + '%"></div>' +
       '<div class="seg missing" style="width:' + pct(missing) + '%"></div>' +
     '</div>';
@@ -131,8 +134,9 @@ function renderPrdProgress(prog) {
       'dispatched': 'background:rgba(210,153,34,0.15);color:var(--yellow);animation:wipPulse 1.5s infinite',
       'failed':      'background:rgba(248,81,73,0.15);color:var(--red)',
       'paused':      'background:rgba(139,148,158,0.15);color:var(--muted)',
+      'updated':     'background:rgba(188,140,255,0.15);color:var(--purple)',
     };
-    const labels = { 'done': 'DONE', 'dispatched': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '\u2014' };
+    const labels = { 'done': 'DONE', 'dispatched': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '\u2014', 'updated': 'REDO' };
     const style = styles[s] || 'background:var(--surface);color:var(--muted)';
     const label = labels[s] || '—';
     return '<span style="font-size:9px;font-weight:700;padding:2px 6px;border-radius:3px;letter-spacing:0.5px;white-space:nowrap;' + style + '">' + label + '</span>';
@@ -370,6 +374,7 @@ function renderPrdProgress(prog) {
       if (s === 'done') return 'var(--green)';
       if (s === 'dispatched') return 'var(--yellow)';
       if (s === 'failed') return 'var(--red)';
+      if (s === 'updated') return 'var(--purple)';
       if (s === 'paused') return 'var(--muted)';
       return 'var(--border)';
     };

package/dashboard/styles.css

@@ -171,6 +171,7 @@
   .prd-progress-bar .seg { height: 100%; transition: width 0.5s ease; }
   .prd-progress-bar .seg.complete { background: var(--green); }
   .prd-progress-bar .seg.dispatched { background: var(--yellow); }
+  .prd-progress-bar .seg.updated { background: var(--purple); }
   .prd-progress-bar .seg.paused { background: var(--muted); opacity: 0.5; }
   .prd-progress-bar .seg.missing { background: var(--border); }
   .prd-progress-pct { font-size: 22px; font-weight: 700; color: var(--green); margin-bottom: var(--space-4); }
@@ -179,6 +180,7 @@
   .prd-legend-dot { width: 10px; height: 10px; border-radius: 2px; }
   .prd-legend-dot.complete { background: var(--green); }
   .prd-legend-dot.dispatched { background: var(--yellow); }
+  .prd-legend-dot.updated { background: var(--purple); }
   .prd-legend-dot.missing { background: var(--border); }
 
   /* Pipeline Step-Progress */
@@ -212,6 +214,7 @@
   @keyframes prdWipPulse { 0%, 100% { box-shadow: 0 0 0 0 rgba(210,153,34,0); } 50% { box-shadow: 0 0 0 4px rgba(210,153,34,0.2); } }
   .prd-item-row.st-failed { border-left-color: var(--red); }
   .prd-item-row.st-needs-human-review { border-left-color: var(--orange); }
+  .prd-item-row.st-updated { border-left-color: var(--purple); }
   .prd-item-row.st-paused { border-left-color: var(--muted); opacity: 0.5; }
   .prd-item-id { font-family: Consolas, monospace; color: var(--muted); min-width: 36px; font-size: 0.9em; }
   .prd-item-name { flex: 1; color: var(--text); white-space: nowrap; overflow: hidden; text-overflow: ellipsis; }

package/engine/lifecycle.js

@@ -8,7 +8,7 @@ const path = require('path');
 const os = require('os');
 const shared = require('./shared');
 const { safeRead, safeJson, safeWrite, mutateJsonFileLocked, mutateWorkItems, execSilent, execAsync, projectPrPath, getPrLinks, addPrLink,
-  log, ts, dateStamp, WI_STATUS, DONE_STATUSES, PLAN_TERMINAL_STATUSES, WORK_TYPE, PLAN_STATUS, PR_STATUS, DISPATCH_RESULT,
+  log, ts, dateStamp, WI_STATUS, DONE_STATUSES, PLAN_TERMINAL_STATUSES, WORK_TYPE, PLAN_STATUS, PRD_ITEM_STATUS, PR_STATUS, DISPATCH_RESULT,
   ENGINE_DEFAULTS, DEFAULT_AGENT_METRICS, FAILURE_CLASS } = shared;
 const { trackEngineUsage } = require('./llm');
 const queries = require('./queries');
@@ -637,6 +637,52 @@ function syncPrdItemStatus(itemId, status, sourcePlan) {
   } catch (err) { log('warn', `PRD status sync: ${err.message}`); }
 }
 
+// ─── PRD Backward-Scan Reconciliation (#929) ────────────────────────────────
+// Proactive counterpart to syncPrdItemStatus. Scans all active PRDs and promotes
+// "missing" items to "updated" when a done work item already exists for that ID.
+// This catches cases where a PRD regeneration incorrectly sets items to "missing"
+// and no subsequent work item state change triggers the reactive sync.
+
+function reconcilePrdStatuses(config) {
+  if (!fs.existsSync(PRD_DIR)) return;
+  let prdFiles;
+  try { prdFiles = fs.readdirSync(PRD_DIR).filter(f => f.endsWith('.json')); } catch { return; }
+  if (prdFiles.length === 0) return;
+
+  const allWorkItems = queries.getWorkItems(config);
+  if (allWorkItems.length === 0) return;
+
+  // Index done work items by ID for O(1) lookup
+  const doneWiById = new Map();
+  for (const wi of allWorkItems) {
+    if (wi.id && DONE_STATUSES.has(wi.status)) doneWiById.set(wi.id, wi);
+  }
+  if (doneWiById.size === 0) return;
+
+  for (const file of prdFiles) {
+    try {
+      const fpath = path.join(PRD_DIR, file);
+      const plan = safeJson(fpath);
+      if (!plan?.missing_features) continue;
+      // Skip completed/archived PRDs — no reconciliation needed
+      if (plan.status === PLAN_STATUS.COMPLETED) continue;
+
+      let modified = false;
+      for (const feature of plan.missing_features) {
+        if (feature.status === PRD_ITEM_STATUS.MISSING && doneWiById.has(feature.id)) {
+          feature.status = PRD_ITEM_STATUS.UPDATED;
+          modified = true;
+          log('info', `PRD backward-scan: promoted ${feature.id} from missing→updated in ${file} (done work item exists)`);
+        }
+      }
+
+      if (modified) {
+        safeWrite(fpath, plan);
+      }
+    } catch (err) { log('warn', `PRD backward-scan for ${file}: ${err.message}`); }
+  }
+}
+
 // ─── PR Sync from Output ─────────────────────────────────────────────────────
 
 function syncPrsFromOutput(output, agentId, meta, config) {
@@ -1876,6 +1922,7 @@ module.exports = {
   cleanupPlanWorktrees,
   updateWorkItemStatus,
   syncPrdItemStatus,
+  reconcilePrdStatuses,
   syncPrsFromOutput,
   updatePrAfterReview,
   updatePrAfterFix,

package/engine.js

@@ -135,7 +135,7 @@ const { renderPlaybook, validatePlaybookVars, PLAYBOOK_REQUIRED_VARS,
 
 // ─── Lifecycle (extracted to engine/lifecycle.js) ────────────────────────────
 
-const { runPostCompletionHooks, updateWorkItemStatus, syncPrdItemStatus, handlePostMerge, checkPlanCompletion,
+const { runPostCompletionHooks, updateWorkItemStatus, syncPrdItemStatus, reconcilePrdStatuses, handlePostMerge, checkPlanCompletion,
   syncPrsFromOutput, updatePrAfterReview, updatePrAfterFix, checkForLearnings, extractSkillsFromOutput,
   updateAgentHistory, updateMetrics, createReviewFeedbackForAuthor, parseAgentOutput, syncPrdFromPrs,
   isItemCompleted, classifyFailure, processPendingRebases } = require('./engine/lifecycle');
@@ -2692,6 +2692,7 @@ async function discoverWork(config) {
 
   // Side-effect passes: materialize plans and design docs into work-items.json
   // These write to project work queues — picked up by discoverFromWorkItems below.
+  reconcilePrdStatuses(config); // Backward-scan: correct "missing" PRD items that have done work items (#929)
   materializePlansAsWorkItems(config);
 
   for (const project of projects) {

package/package.json

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

package/playbooks/plan-to-prd.md

@@ -105,7 +105,7 @@ When the engine detects an existing PRD for this plan (`source_plan` match), it
 1. **Parse the existing PRD JSON** — extract all `missing_features` items with their `id`, `status`, and metadata
 2. **Preserve item IDs** — match existing items to current plan items by name/description similarity. Each plan item that corresponds to an existing PRD item MUST reuse that item's `P-<id>`. Do NOT generate new IDs for items that already exist.
 3. **Preserve done items** — any existing item with `"status": "done"` carries forward as `"done"` with the same ID, description, and acceptance criteria. Do NOT reset done items to `"missing"`
-4. **Carry forward in-progress items** — items with `"status": "missing"` or other non-done statuses keep their existing ID and reset to `"missing"`
+4. **Carry forward in-progress items** — items with `"status": "missing"` or other non-done statuses (except `"updated"`) keep their existing ID and reset to `"missing"`. Items with `"status": "updated"` keep their existing ID and remain `"updated"` — do NOT reset to `"missing"`
 5. **New items only** — only generate new `P-<uuid>` IDs for items in the plan that have no match in the existing PRD
 6. **Removed items** — if an existing PRD item has no match in the current plan, drop it from the output
 7. **Preserve plan-level fields** — keep `branch_strategy`, `feature_branch`, and `project` from the existing PRD unless the plan explicitly changes them
@@ -118,6 +118,7 @@ If the task description contains `mode: diff-aware-update`, you are updating an
 
 **Additional diff-aware rules** (on top of the reuse rules above):
 - **Done + modified** (plan added requirements/changed scope) → set `"status": "updated"` with same ID (engine re-opens the work item and dispatches to existing branch)
+- **Already `"updated"`** (was previously flagged for rework and still needs changes) → keep `"status": "updated"` with same ID and revised description. Do NOT reset to `"missing"` — this preserves the existing branch reference so the agent continues from prior work
 - **Pending/failed items** → reset to `"missing"` with updated description if the plan changed their scope
 
 ## Important