@yemi33/minions 0.1.896 → 0.1.898

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 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
+- PRD updated/missing status takes priority over work item done in display (closes #930) (#935)
+
 ## 0.1.896 (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/queries.js

@@ -11,7 +11,7 @@ const shared = require('./shared');
 
 const { safeRead, safeReadDir, safeJson, safeWrite, getProjects,
   projectWorkItemsPath, projectPrPath, parseSkillFrontmatter, KB_CATEGORIES,
-  WI_STATUS, ENGINE_DEFAULTS } = shared;
+  WI_STATUS, DONE_STATUSES, PRD_ITEM_STATUS, ENGINE_DEFAULTS } = shared;
 
 /**
  * Read the first `bytes` and last `bytes` of a file efficiently using byte offsets.
@@ -955,9 +955,12 @@ function getPrdInfo(config) {
   const statusDisplay = { pending: 'missing' };
   for (const item of items) {
     const wi = wiById[item.id];
-    // Work item status is source of truth when available (PRD JSON may lag behind)
+    // PRD 'updated'/'missing' = intentional rework signal — takes priority over a done work item (#930).
+    // Otherwise work item status is source of truth when available (PRD JSON may lag behind).
     // If PRD says dispatched/failed but no work item exists, treat as pending (orphaned — #779)
-    const rawStatus = wi ? (wi.status || item.status)
+    const prdFlaggedForRework = item.status === PRD_ITEM_STATUS.UPDATED || item.status === PRD_ITEM_STATUS.MISSING;
+    const rawStatus = (wi && !(prdFlaggedForRework && DONE_STATUSES.has(wi.status)))
+      ? (wi.status || item.status)
       : ((item.status === WI_STATUS.DISPATCHED || item.status === WI_STATUS.FAILED) ? WI_STATUS.PENDING : item.status);
     item.status = statusDisplay[rawStatus] || rawStatus || 'missing';
     // Attach execution metadata for display (agent, PR link, fail reason)

package/package.json

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