@yemi33/minions 0.1.151 → 0.1.153

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # Changelog
 
+## 0.1.153 (2026-04-02)
+
+### Engine
+- engine.js
+- engine/lifecycle.js
+- engine/playbook.js
+- engine/queries.js
+
+### Dashboard
+- dashboard.html
+- dashboard.js
+- dashboard/js/modal-qa.js
+- dashboard/js/modal.js
+- dashboard/js/render-inbox.js
+- dashboard/js/render-kb.js
+- dashboard/js/render-plans.js
+- dashboard/js/render-prd.js
+- dashboard/styles.css
+
+### Documentation
+- deprecated.json
+
+### Other
+- test/unit.test.js
+
+## 0.1.152 (2026-04-02)
+
+### Dashboard
+- dashboard.js
+
 ## 0.1.151 (2026-04-02)
 
 ### Engine

package/dashboard/js/modal-qa.js

@@ -2,7 +2,6 @@
 
 let _modalDocContext = { title: '', content: '', selection: '' };
 let _modalFilePath = null; // file path for steering (null = read-only Q&A only)
-let _modalOriginalPlan = null; // tracks original plan file when editing a forked version
 
 function showModalQa() {
   document.getElementById('modal-qa').style.display = '';
@@ -235,9 +234,9 @@ async function _processQaMessage(message, selection) {
         actionDiv.style.cssText = 'margin:8px 0;padding:8px 12px;background:rgba(210,153,34,0.1);border:1px solid rgba(210,153,34,0.3);border-radius:6px;display:flex;flex-wrap:wrap;align-items:center;gap:8px';
         actionDiv.innerHTML =
           '<span style="color:var(--orange);font-weight:600;font-size:12px;width:100%">Execution paused — plan was updated</span>' +
-          '<button onclick="qaReplacePrd(\'' + esc + '\')" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer">Re-execute with new PRD</button>' +
+          '<button onclick="planExecute(\'' + esc + '\', \'\', null)" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer">Re-execute plan</button>' +
           '<button onclick="this.closest(\'div\').innerHTML=\'<span style=color:var(--muted);font-size:11px>Paused. No work dispatched.</span>\'" style="background:var(--surface2);color:var(--text);border:1px solid var(--border);border-radius:4px;padding:4px 12px;font-size:11px;cursor:pointer">Keep paused</button>' +
-          '<span style="color:var(--muted);font-size:10px;width:100%">Re-execute replaces the old PRD with a fresh one from the updated plan.</span>';
+          '<span style="color:var(--muted);font-size:10px;width:100%">Re-execute creates a new PRD from the updated plan.</span>';
         thread.appendChild(actionDiv);
       }
     } else {
@@ -289,45 +288,4 @@ async function _processQaMessage(message, selection) {
   }
 }
 
-async function qaNewPrd(planFile) {
-  qaDisablePrdButtons();
-  const allPlans = window._lastStatus?.plans || [];
-  const mdPlan = allPlans.find(p => p.file === planFile);
-  const project = mdPlan?.project || '';
-
-  planExecute(planFile, project, null);
-
-  const btn = document.getElementById('qa-generate-prd-btn');
-  if (btn) btn.innerHTML = '<span style="color:var(--green);font-size:12px">New PRD dispatched — existing work continues, agent creating fresh PRD from revised plan.</span>';
-}
-
-async function qaReplacePrd(planFile) {
-  qaDisablePrdButtons();
-  const allPlans = window._lastStatus?.plans || [];
-  const mdPlan = allPlans.find(p => p.file === planFile);
-  const project = mdPlan?.project || '';
-  const existingPrd = allPlans.find(p => p.file.endsWith('.json') && p.project === project);
-
-  if (existingPrd) {
-    // Pause first to stop materialization, then clean pending items
-    try {
-      await fetch('/api/plans/pause', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ file: existingPrd.file })
-      });
-    } catch (e) { console.error('plan pause:', e.message); }
-    try {
-      await fetch('/api/plans/regenerate', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ source: existingPrd.file })
-      });
-    } catch (e) { console.error('plan regenerate:', e.message); }
-  }
-
-  planExecute(planFile, project, null);
-
-  const btn = document.getElementById('qa-generate-prd-btn');
-  if (btn) btn.innerHTML = '<span style="color:var(--orange);font-size:12px">Replacing PRD — old items paused, agent regenerating from revised plan.</span>';
-}
-
-window.MinionsQA = { showModalQa, modalAskAboutSelection, clearQaSelection, clearQaConversation, modalSend, qaNewPrd, qaReplacePrd };
+window.MinionsQA = { showModalQa, modalAskAboutSelection, clearQaSelection, clearQaConversation, modalSend };

package/dashboard/js/modal.js

@@ -35,8 +35,6 @@ function closeModal() {
   // Clear edit/steer state
   _modalEditable = null;
   _modalFilePath = null;
-  _modalOriginalPlan = null;
-  // steer btn removed — unified send
   const body = document.getElementById('modal-body');
   body.contentEditable = 'false';
   body.style.border = '';

package/dashboard/js/render-inbox.js

@@ -72,7 +72,6 @@ function openNotesModal() {
   _modalEditable = 'notes.md';
   _modalFilePath = 'notes.md'; showModalQa();
   document.getElementById('modal-edit-btn').style.display = '';
-  // steer btn removed — unified send
   document.getElementById('modal').classList.add('open');
 }
 

package/dashboard/js/render-kb.js

@@ -169,7 +169,6 @@ async function kbOpenItem(category, file) {
     // Clear notification badge when opening this document
     const card = findCardForFile(_modalFilePath);
     if (card) clearNotifBadge(card);
-    // steer btn removed — unified send
     document.getElementById('modal').classList.add('open');
   } catch (e) {
     console.error('Failed to load KB item:', e);

package/dashboard/js/render-plans.js

@@ -67,7 +67,7 @@ function derivePlanStatus(prdFile, mdFile, prdJsonStatus, workItems) {
   const hasPendingPrd = wi.some(w => w.type === 'plan-to-prd' && (w.status === 'pending' || w.status === 'dispatched'));
   const hasActiveWork = implementWi.some(w => w.status === 'pending' || w.status === 'dispatched');
   const allDone = implementWi.length > 0 && implementWi.every(w =>
-    w.status === 'done' || w.status === 'in-pr' || w.status === 'implemented' || w.status === 'complete'
+    w.status === 'done'
   );
   const hasFailed = implementWi.some(w => w.status === 'failed');
 
@@ -304,48 +304,6 @@ function openArchivedPlansModal() {
   document.getElementById('modal').classList.add('open');
 }
 
-// Disable all PRD action buttons to prevent double-clicks
-function qaDisablePrdButtons() {
-  const container = document.getElementById('qa-generate-prd-btn');
-  if (container) container.querySelectorAll('button').forEach(b => { b.disabled = true; b.style.opacity = '0.5'; });
-}
-
-// Show plan version action buttons (Run alongside / Replace / Just save)
-function showPlanVersionActions(thread, newFile, originalFile) {
-  const esc = newFile.replace(/'/g, "\\'");
-  // Look up existing PRD for the original plan's project
-  const allPlans = window._lastStatus?.plans || [];
-  const origPlan = allPlans.find(p => p.file === originalFile);
-  const project = origPlan?.project || '';
-  const existingPrd = allPlans.find(p => p.file.endsWith('.json') && p.project === project && p.status !== 'completed');
-
-  const btn = document.createElement('div');
-  btn.id = 'qa-generate-prd-btn';
-  btn.style.cssText = 'margin:8px 0;padding:8px 12px;background:rgba(63,185,80,0.1);border:1px solid rgba(63,185,80,0.3);border-radius:6px;display:flex;flex-wrap:wrap;align-items:center;gap:8px';
-
-  if (existingPrd) {
-    btn.innerHTML = '<span style="color:var(--green);font-weight:600;font-size:12px;width:100%">New plan version created — existing PRD running</span>' +
-      '<button onclick="qaNewPrd(\'' + esc + '\')" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer" title="Execute this plan as a separate PRD alongside the current one">Run alongside</button>' +
-      '<button onclick="qaReplacePrd(\'' + esc + '\')" style="background:var(--orange);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer" title="Pause existing PRD, clean pending items, execute this plan instead">Replace old PRD</button>' +
-      '<button onclick="qaJustSave(this)" style="background:var(--surface2);color:var(--text);border:1px solid var(--border);border-radius:4px;padding:4px 12px;font-size:11px;cursor:pointer" title="Keep the new version saved without dispatching any work">Just save</button>' +
-      '<span style="color:var(--muted);font-size:10px;width:100%">Run alongside keeps current work going. Replace pauses it and starts fresh.</span>';
-  } else {
-    btn.innerHTML = '<span style="color:var(--green);font-weight:600;font-size:12px;width:100%">New plan version created</span>' +
-      '<button onclick="qaNewPrd(\'' + esc + '\')" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer">Execute plan</button>' +
-      '<button onclick="qaJustSave(this)" style="background:var(--surface2);color:var(--text);border:1px solid var(--border);border-radius:4px;padding:4px 12px;font-size:11px;cursor:pointer">Just save</button>' +
-      '<span style="color:var(--muted);font-size:10px">Execute dispatches an agent to create PRD items from this plan</span>';
-  }
-  // Remove any previous action buttons
-  const old = thread.querySelector('#qa-generate-prd-btn');
-  if (old) old.remove();
-  thread.appendChild(btn);
-}
-
-function qaJustSave(el) {
-  const container = el.closest('#qa-generate-prd-btn');
-  if (container) container.innerHTML = '<span style="color:var(--muted);font-size:11px">Saved. No work dispatched.</span>';
-}
-
 async function planExecute(file, project, btn) {
   if (btn) { btn.textContent = 'Executing...'; btn.disabled = true; btn.style.color = 'var(--blue)'; }
   try {
@@ -736,4 +694,4 @@ async function planUnarchive(file, btn) {
   } catch (e) { resetBtn(); alert('Error: ' + e.message); }
 }
 
-window.MinionsPlans = { openCreatePlanModal, refreshPlans, derivePlanStatus, renderPlans, openArchivedPlansModal, qaDisablePrdButtons, showPlanVersionActions, qaJustSave, planExecute, planSubmitRevise, planShowRevise, planHideRevise, planView, planApprove, planArchive, planUnarchive, planDelete, planPause, planReject, planDiscuss, planOpenInDocChat, planRegeneratePRD, openVerifyGuide, triggerVerify };
+window.MinionsPlans = { openCreatePlanModal, refreshPlans, derivePlanStatus, renderPlans, openArchivedPlansModal, planExecute, planSubmitRevise, planShowRevise, planHideRevise, planView, planApprove, planArchive, planUnarchive, planDelete, planPause, planReject, planDiscuss, planOpenInDocChat, planRegeneratePRD, openVerifyGuide, triggerVerify };

package/dashboard/js/render-prd.js

@@ -24,7 +24,7 @@ function renderPrd(prd, prog) {
   if (existing.length <= 1) {
     // Single PRD — show status + actions in header
     const implementItems = allWi.filter(w => prdItems.some(pi => pi.id === w.id));
-    const allDone = implementItems.length > 0 && implementItems.every(w => w.status === 'done' || w.status === 'in-pr' || w.status === 'implemented' || w.status === 'complete');
+    const allDone = implementItems.length > 0 && implementItems.every(w => w.status === 'done');
     const hasActive = implementItems.some(w => w.status === 'pending' || w.status === 'dispatched');
     const prdFile = existing[0]?.file || '';
     const prdStatus = existing[0]?.status || '';
@@ -52,7 +52,7 @@ function renderPrd(prd, prog) {
     for (const p of existing) {
       const items = prdItems.filter(i => i.source === p.file);
       const wiForPrd = allWi.filter(w => items.some(pi => pi.id === w.id));
-      const allDone = wiForPrd.length > 0 && wiForPrd.every(w => w.status === 'done' || w.status === 'in-pr' || w.status === 'implemented' || w.status === 'complete');
+      const allDone = wiForPrd.length > 0 && wiForPrd.every(w => w.status === 'done');
       const hasActive = wiForPrd.some(w => w.status === 'pending' || w.status === 'dispatched');
       const s = allDone && !hasActive ? 'completed' : hasActive ? 'in-progress' : p.status || 'active';
       counts[s] = (counts[s] || 0) + 1;
@@ -73,7 +73,7 @@ function renderPrdProgress(prog) {
   // Compute overall progress from active (non-archived) items
   const activeItems = (prog.items || []).filter(i => !i._archived);
   if (activeItems.length > 0) {
-    const activeDone = activeItems.filter(i => i.status === 'done' || i.status === 'implemented' || i.status === 'in-pr').length;
+    const activeDone = activeItems.filter(i => i.status === 'done').length;
     countEl.textContent = Math.round((activeDone / activeItems.length) * 100) + '%';
   } else {
     countEl.textContent = '—';
@@ -82,7 +82,7 @@ function renderPrdProgress(prog) {
   function renderGroupStats(items) {
     const total = items.length;
     if (total === 0) return '';
-    const done = items.filter(i => i.status === 'done' || i.status === 'implemented' || i.status === 'in-pr').length; // in-pr counted as done for backward compat
+    const done = items.filter(i => i.status === 'done').length;
     const inProgress = items.filter(i => i.status === 'in-progress').length;
     const failed = items.filter(i => i.status === 'failed').length;
     const paused = items.filter(i => i.status === 'paused').length;
@@ -112,13 +112,11 @@ function renderPrdProgress(prog) {
   const statusBadge = (s) => {
     const styles = {
       'done': 'background:rgba(63,185,80,0.15);color:var(--green)',
-      'implemented': 'background:rgba(63,185,80,0.15);color:var(--green)', /* legacy alias */
-      'in-pr':  'background:rgba(63,185,80,0.15);color:var(--green)', /* backward compat: displayed as done */
       'in-progress': '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)',
     };
-    const labels = { 'done': 'DONE', 'implemented': 'DONE', 'in-pr': 'DONE', 'in-progress': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '—' };
+    const labels = { 'done': 'DONE', 'in-progress': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '—' };
     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>';
@@ -197,7 +195,7 @@ function renderPrdProgress(prog) {
   const timings = prog.planTimings || {};
 
   const renderGroupHeader = (g) => {
-    const done = g.items.filter(i => i.status === 'done' || i.status === 'complete' || i.status === 'implemented').length;
+    const done = g.items.filter(i => i.status === 'done').length;
     const wip = g.items.filter(i => i.status === 'in-progress' || i.status === 'dispatched').length;
     const summary = (g.summary || '').replace(/^Convert plan to PRD:\s*/i, '').slice(0, 80);
     const isAwaitingApproval = g.planStatus === 'awaiting-approval';
@@ -304,7 +302,7 @@ function renderPrdProgress(prog) {
     });
 
     const statusColor = (s) => {
-      if (s === 'done' || s === 'implemented' || s === 'in-pr') return 'var(--green)'; // in-pr treated as done
+      if (s === 'done') return 'var(--green)';
       if (s === 'in-progress') return 'var(--yellow)';
       if (s === 'failed') return 'var(--red)';
       if (s === 'paused') return 'var(--muted)';
@@ -475,7 +473,7 @@ function openArchivedPrdModal() {
     // Picker: list archived plans, click to expand
     html = '<div style="margin-bottom:12px;font-size:12px;color:var(--muted)">Select an archived PRD to view:</div>';
     html += groups.map((g, i) => {
-      const done = g.items.filter(it => it.status === 'done' || it.status === 'implemented' || it.status === 'complete').length;
+      const done = g.items.filter(it => it.status === 'done').length;
       const failed = g.items.filter(it => it.status === 'failed').length;
       return '<div class="plan-card" style="cursor:pointer;margin-bottom:8px" onclick="showArchivedPrdDetail(' + i + ')">' +
         '<div class="plan-card-title" style="font-size:13px">' + escHtml(g.summary || g.file) + '</div>' +
@@ -538,7 +536,7 @@ async function prdItemEdit(source, itemId) {
 
   // Build completion summary section
   let completionHtml = '';
-  const isDone = item.status === 'done' || item.status === 'implemented' || item.status === 'complete' || item.status === 'in-pr'; // in-pr treated as done for backward compat
+  const isDone = item.status === 'done';
   const isFailed = item.status === 'failed';
   const isActive = item.status === 'in-progress' || item.status === 'dispatched';
 
@@ -711,7 +709,7 @@ function openArchive(i) {
       html += '<div class="archive-feature">' +
         '<span class="feat-id">' + escHtml(f.id) + '</span> ' +
         '<span class="prd-item-priority ' + pClass + '">' + escHtml(f.priority || '') + '</span>' +
-        (f.status ? ' <span class="pr-badge ' + (f.status === 'complete' || f.status === 'done' || f.status === 'in-pr' ? 'approved' : 'draft') + '" style="font-size:9px">' + escHtml(f.status === 'in-pr' ? 'done' : f.status) + '</span>' : '') +
+        (f.status ? ' <span class="pr-badge ' + (f.status === 'done' ? 'approved' : 'draft') + '" style="font-size:9px">' + escHtml(f.status) + '</span>' : '') +
         '<div class="feat-name">' + escHtml(f.name) + '</div>' +
         '<div class="feat-desc">' + escHtml(f.description || '') + '</div>' +
         (f.rationale ? '<div class="feat-desc" style="margin-top:4px;color:var(--yellow)">Rationale: ' + escHtml(f.rationale) + '</div>' : '') +

package/dashboard/styles.css

@@ -181,8 +181,6 @@
   .prd-items-list { display: flex; flex-direction: column; gap: 3px; max-height: 400px; overflow-y: auto; padding: 0 8px; }
   .prd-item-row { display: flex; align-items: center; gap: 8px; padding: 4px 8px; border-radius: var(--radius-sm); font-size: var(--text-base); background: var(--surface2); border: 1px solid var(--border); border-left: 3px solid var(--border); }
   .prd-item-row.st-done { border-left-color: var(--green); }
-  .prd-item-row.st-implemented { border-left-color: var(--green); } /* legacy alias */
-  .prd-item-row.st-in-pr { border-left-color: var(--green); } /* backward compat: treated as done */
   .prd-item-row.st-in-progress { border-left-color: var(--yellow); animation: prdWipPulse 2s infinite; }
   @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); }

package/dashboard.html

@@ -178,8 +178,6 @@
   .prd-items-list { display: flex; flex-direction: column; gap: 3px; max-height: 400px; overflow-y: auto; padding: 0 8px; }
   .prd-item-row { display: flex; align-items: center; gap: 8px; padding: 4px 8px; border-radius: var(--radius-sm); font-size: var(--text-base); background: var(--surface2); border: 1px solid var(--border); border-left: 3px solid var(--border); }
   .prd-item-row.st-done { border-left-color: var(--green); }
-  .prd-item-row.st-implemented { border-left-color: var(--green); } /* legacy alias */
-  .prd-item-row.st-in-pr { border-left-color: var(--green); } /* backward compat: treated as done */
   .prd-item-row.st-in-progress { border-left-color: var(--yellow); animation: prdWipPulse 2s infinite; }
   @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); }
@@ -1171,7 +1169,7 @@ function renderPrdProgress(prog) {
   // Compute progress from active (non-archived) items only
   const activeItems = (prog.items || []).filter(i => !i._archived);
   if (activeItems.length > 0) {
-    const activeDone = activeItems.filter(i => i.status === 'done' || i.status === 'implemented' || i.status === 'in-pr').length; // in-pr counted as done for backward compat
+    const activeDone = activeItems.filter(i => i.status === 'done').length;
     countEl.textContent = Math.round((activeDone / activeItems.length) * 100) + '%';
   } else {
     countEl.textContent = '—';
@@ -1180,7 +1178,7 @@ function renderPrdProgress(prog) {
   function renderGroupStats(items) {
     const total = items.length;
     if (total === 0) return '';
-    const done = items.filter(i => i.status === 'done' || i.status === 'implemented' || i.status === 'in-pr').length; // in-pr counted as done for backward compat
+    const done = items.filter(i => i.status === 'done').length;
     const inProgress = items.filter(i => i.status === 'in-progress').length;
     const failed = items.filter(i => i.status === 'failed').length;
     const paused = items.filter(i => i.status === 'paused').length;
@@ -1210,13 +1208,11 @@ function renderPrdProgress(prog) {
   const statusBadge = (s) => {
     const styles = {
       'done': 'background:rgba(63,185,80,0.15);color:var(--green)',
-      'implemented': 'background:rgba(63,185,80,0.15);color:var(--green)', /* legacy alias */
-      'in-pr':  'background:rgba(63,185,80,0.15);color:var(--green)', /* backward compat: displayed as done */
       'in-progress': '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)',
     };
-    const labels = { 'done': 'DONE', 'implemented': 'DONE', 'in-pr': 'DONE', 'in-progress': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '—' };
+    const labels = { 'done': 'DONE', 'in-progress': 'WIP', 'failed': 'FAIL', 'paused': 'PAUSED', 'missing': '—' };
     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>';
@@ -1295,7 +1291,7 @@ function renderPrdProgress(prog) {
   const timings = prog.planTimings || {};
 
   const renderGroupHeader = (g) => {
-    const done = g.items.filter(i => i.status === 'done' || i.status === 'complete' || i.status === 'implemented').length;
+    const done = g.items.filter(i => i.status === 'done').length;
     const wip = g.items.filter(i => i.status === 'in-progress' || i.status === 'dispatched').length;
     const summary = (g.summary || '').replace(/^Convert plan to PRD:\s*/i, '').slice(0, 80);
     const isAwaitingApproval = g.planStatus === 'awaiting-approval';
@@ -1394,7 +1390,7 @@ function renderPrdProgress(prog) {
     });
 
     const statusColor = (s) => {
-      if (s === 'done' || s === 'implemented' || s === 'in-pr') return 'var(--green)'; // in-pr treated as done
+      if (s === 'done') return 'var(--green)';
       if (s === 'in-progress') return 'var(--yellow)';
       if (s === 'failed') return 'var(--red)';
       if (s === 'paused') return 'var(--muted)';
@@ -1565,7 +1561,7 @@ function openArchivedPrdModal() {
     // Picker: list archived plans, click to expand
     html = '<div style="margin-bottom:12px;font-size:12px;color:var(--muted)">Select an archived PRD to view:</div>';
     html += groups.map((g, i) => {
-      const done = g.items.filter(it => it.status === 'done' || it.status === 'implemented' || it.status === 'complete').length;
+      const done = g.items.filter(it => it.status === 'done').length;
       const failed = g.items.filter(it => it.status === 'failed').length;
       return '<div class="plan-card" style="cursor:pointer;margin-bottom:8px" onclick="showArchivedPrdDetail(' + i + ')">' +
         '<div class="plan-card-title" style="font-size:13px">' + escHtml(g.summary || g.file) + '</div>' +
@@ -1661,7 +1657,7 @@ async function prdItemEdit(source, itemId) {
 
   // Build completion summary section
   let completionHtml = '';
-  const isDone = item.status === 'done' || item.status === 'implemented' || item.status === 'complete' || item.status === 'in-pr'; // in-pr treated as done for backward compat
+  const isDone = item.status === 'done';
   const isFailed = item.status === 'failed';
   const isActive = item.status === 'in-progress' || item.status === 'dispatched';
 
@@ -1931,7 +1927,6 @@ function openNotesModal() {
   _modalEditable = 'notes.md';
   _modalFilePath = 'notes.md'; showModalQa();
   document.getElementById('modal-edit-btn').style.display = '';
-  // steer btn removed — unified send
   document.getElementById('modal').classList.add('open');
 }
 
@@ -2130,8 +2125,6 @@ function closeModal() {
   // Clear edit/steer state
   _modalEditable = null;
   _modalFilePath = null;
-  _modalOriginalPlan = null;
-  // steer btn removed — unified send
   const body = document.getElementById('modal-body');
   body.contentEditable = 'false';
   body.style.border = '';
@@ -2194,7 +2187,7 @@ function openArchive(i) {
       html += '<div class="archive-feature">' +
         '<span class="feat-id">' + escHtml(f.id) + '</span> ' +
         '<span class="prd-item-priority ' + pClass + '">' + escHtml(f.priority || '') + '</span>' +
-        (f.status ? ' <span class="pr-badge ' + (f.status === 'complete' || f.status === 'done' || f.status === 'in-pr' ? 'approved' : 'draft') + '" style="font-size:9px">' + escHtml(f.status === 'in-pr' ? 'done' : f.status) + '</span>' : '') +
+        (f.status ? ' <span class="pr-badge ' + (f.status === 'done' ? 'approved' : 'draft') + '" style="font-size:9px">' + escHtml(f.status) + '</span>' : '') +
         '<div class="feat-name">' + escHtml(f.name) + '</div>' +
         '<div class="feat-desc">' + escHtml(f.description || '') + '</div>' +
         (f.rationale ? '<div class="feat-desc" style="margin-top:4px;color:var(--yellow)">Rationale: ' + escHtml(f.rationale) + '</div>' : '') +
@@ -3539,7 +3532,6 @@ async function cmdSubmit() {
 
 let _modalDocContext = { title: '', content: '', selection: '' };
 let _modalFilePath = null; // file path for steering (null = read-only Q&A only)
-let _modalOriginalPlan = null; // tracks original plan file when editing a forked version
 
 // ─── Notification Badges ──────────────────────────────────────────────────────
 // Show a red dot on a card/button when a background response arrives
@@ -4131,91 +4123,6 @@ async function planApprove(file) {
   } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); }
 }
 
-// Disable all PRD action buttons to prevent double-clicks
-function qaDisablePrdButtons() {
-  const container = document.getElementById('qa-generate-prd-btn');
-  if (container) container.querySelectorAll('button').forEach(b => { b.disabled = true; b.style.opacity = '0.5'; });
-}
-
-// Show plan version action buttons (Run alongside / Replace / Just save)
-function showPlanVersionActions(thread, newFile, originalFile) {
-  const esc = newFile.replace(/'/g, "\\'");
-  // Look up existing PRD for the original plan's project
-  const allPlans = window._lastStatus?.plans || [];
-  const origPlan = allPlans.find(p => p.file === originalFile);
-  const project = origPlan?.project || '';
-  const existingPrd = allPlans.find(p => p.file.endsWith('.json') && p.project === project && p.status !== 'completed');
-
-  const btn = document.createElement('div');
-  btn.id = 'qa-generate-prd-btn';
-  btn.style.cssText = 'margin:8px 0;padding:8px 12px;background:rgba(63,185,80,0.1);border:1px solid rgba(63,185,80,0.3);border-radius:6px;display:flex;flex-wrap:wrap;align-items:center;gap:8px';
-
-  if (existingPrd) {
-    btn.innerHTML = '<span style="color:var(--green);font-weight:600;font-size:12px;width:100%">New plan version created — existing PRD running</span>' +
-      '<button onclick="qaNewPrd(\'' + esc + '\')" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer" title="Execute this plan as a separate PRD alongside the current one">Run alongside</button>' +
-      '<button onclick="qaReplacePrd(\'' + esc + '\')" style="background:var(--orange);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer" title="Pause existing PRD, clean pending items, execute this plan instead">Replace old PRD</button>' +
-      '<button onclick="qaJustSave(this)" style="background:var(--surface2);color:var(--text);border:1px solid var(--border);border-radius:4px;padding:4px 12px;font-size:11px;cursor:pointer" title="Keep the new version saved without dispatching any work">Just save</button>' +
-      '<span style="color:var(--muted);font-size:10px;width:100%">Run alongside keeps current work going. Replace pauses it and starts fresh.</span>';
-  } else {
-    btn.innerHTML = '<span style="color:var(--green);font-weight:600;font-size:12px;width:100%">New plan version created</span>' +
-      '<button onclick="qaNewPrd(\'' + esc + '\')" style="background:var(--green);color:#fff;border:none;border-radius:4px;padding:4px 12px;font-size:11px;font-weight:600;cursor:pointer">Execute plan</button>' +
-      '<button onclick="qaJustSave(this)" style="background:var(--surface2);color:var(--text);border:1px solid var(--border);border-radius:4px;padding:4px 12px;font-size:11px;cursor:pointer">Just save</button>' +
-      '<span style="color:var(--muted);font-size:10px">Execute dispatches an agent to create PRD items from this plan</span>';
-  }
-  // Remove any previous action buttons
-  const old = thread.querySelector('#qa-generate-prd-btn');
-  if (old) old.remove();
-  thread.appendChild(btn);
-}
-
-function qaJustSave(el) {
-  const container = el.closest('#qa-generate-prd-btn');
-  if (container) container.innerHTML = '<span style="color:var(--muted);font-size:11px">Saved. No work dispatched.</span>';
-}
-
-// Replace existing PRD: pause old, clean pending items, regenerate from revised plan
-async function qaReplacePrd(planFile) {
-  qaDisablePrdButtons();
-  const allPlans = window._lastStatus?.plans || [];
-  const mdPlan = allPlans.find(p => p.file === planFile);
-  const project = mdPlan?.project || '';
-  const existingPrd = allPlans.find(p => p.file.endsWith('.json') && p.project === project);
-
-  if (existingPrd) {
-    // Pause first to stop materialization, then clean pending items
-    try {
-      await fetch('/api/plans/pause', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ file: existingPrd.file })
-      });
-    } catch {}
-    try {
-      await fetch('/api/plans/regenerate', {
-        method: 'POST', headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify({ source: existingPrd.file })
-      });
-    } catch {}
-  }
-
-  planExecute(planFile, project, null);
-
-  const btn = document.getElementById('qa-generate-prd-btn');
-  if (btn) btn.innerHTML = '<span style="color:var(--orange);font-size:12px">Replacing PRD — old items paused, agent regenerating from revised plan.</span>';
-}
-
-// New PRD: keep existing PRD running, create fresh PRD from revised plan
-async function qaNewPrd(planFile) {
-  qaDisablePrdButtons();
-  const allPlans = window._lastStatus?.plans || [];
-  const mdPlan = allPlans.find(p => p.file === planFile);
-  const project = mdPlan?.project || '';
-
-  planExecute(planFile, project, null);
-
-  const btn = document.getElementById('qa-generate-prd-btn');
-  if (btn) btn.innerHTML = '<span style="color:var(--green);font-size:12px">New PRD dispatched — existing work continues, agent creating fresh PRD from revised plan.</span>';
-}
-
 async function planExecute(file, project, btn) {
   if (btn) { btn.textContent = 'Executing...'; btn.disabled = true; btn.style.color = 'var(--blue)'; }
   try {
@@ -4429,7 +4336,6 @@ async function planView(file) {
     // Clear notification badge when opening this document
     const card = findCardForFile(_modalFilePath);
     if (card) clearNotifBadge(card);
-    // steer btn removed — unified send
     document.getElementById('modal').classList.add('open');
   } catch (e) { console.error(e); }
 }
@@ -4568,7 +4474,6 @@ async function kbOpenItem(category, file) {
     // Clear notification badge when opening this document
     const card = findCardForFile(_modalFilePath);
     if (card) clearNotifBadge(card);
-    // steer btn removed — unified send
     document.getElementById('modal').classList.add('open');
   } catch (e) {
     console.error('Failed to load KB item:', e);

package/dashboard.js

@@ -1857,7 +1857,7 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
           for (const w of items) {
             if (w.sourcePlan !== body.file) continue;
             // Keep completed items as-is, reset everything else to pending.
-            if (w.status === 'done' || w.status === 'implemented' || w.status === 'complete' || w.status === 'in-pr') continue;
+            if (w.status === 'done') continue;
 
             if (w.status === 'dispatched') {
               // Kill the agent working on this item, if any.
@@ -2259,148 +2259,6 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 
-  // POST /api/plans/revise-and-regenerate — REMOVED: plan versioning now handled by /api/doc-chat
-  // The "Replace old PRD" flow uses qaReplacePrd (frontend) which calls /api/plans/pause + /api/plans/regenerate + planExecute
-  async function handlePlansReviseAndRegenerate(req, res) {
-    try {
-      const body = await readBody(req);
-      if (!body.source || !body.instruction) return jsonReply(res, 400, { error: 'source and instruction required' });
-
-      // Find the source plan .md file for this PRD
-      // Convention: PRD JSON references plan via plan_summary containing the work item ID,
-      // or the .md file has a matching name prefix
-      const prdPath = path.join(PRD_DIR, body.source);
-      if (!fs.existsSync(prdPath)) return jsonReply(res, 404, { error: 'PRD file not found' });
-
-      // Look for corresponding .md plan file
-      let sourcePlanFile = null;
-      const planFiles = safeReadDir(PLANS_DIR).filter(f => f.endsWith('.md'));
-      if (body.sourcePlan) {
-        // Explicit source plan provided
-        sourcePlanFile = body.sourcePlan;
-      } else {
-        // Heuristic: find .md plan by matching prefix or by reading PRD's generated_from field
-        const prd = JSON.parse(safeRead(prdPath) || '{}');
-        if (prd.source_plan) {
-          sourcePlanFile = prd.source_plan;
-        } else {
-          // Match by prefix: officeagent-2026-03-15.json → plan-*officeagent* or plan-w025*.md
-          const prdBase = body.source.replace('.json', '');
-          for (const f of planFiles) {
-            // Check if plan file mentions the same project or was created around same time
-            const content = safeRead(path.join(PLANS_DIR, f)) || '';
-            if (content.includes(prd.project || '___nomatch___') || content.includes(prd.plan_summary?.slice(0, 40) || '___nomatch___')) {
-              sourcePlanFile = f;
-              break;
-            }
-          }
-          // Last resort: most recent .md plan
-          if (!sourcePlanFile && planFiles.length > 0) {
-            sourcePlanFile = planFiles.sort((a, b) => {
-              try { return fs.statSync(path.join(PLANS_DIR, b)).mtimeMs - fs.statSync(path.join(PLANS_DIR, a)).mtimeMs; } catch { return 0; }
-            })[0];
-          }
-        }
-      }
-
-      if (!sourcePlanFile) {
-        return jsonReply(res, 404, { error: 'No source plan (.md) found for this PRD. You can edit the PRD JSON directly using "Edit Plan".' });
-      }
-
-      const sourcePlanPath = path.join(PLANS_DIR, sourcePlanFile);
-      const planContent = safeRead(sourcePlanPath);
-      if (!planContent) return jsonReply(res, 404, { error: 'Source plan file not readable: ' + sourcePlanFile });
-
-      // Step 1: Steer the source plan with the user's instruction via CC
-      const result = await ccDocCall({
-        message: body.instruction,
-        document: planContent,
-        title: sourcePlanFile,
-        filePath: 'plans/' + sourcePlanFile,
-        selection: body.selection || '',
-        canEdit: true,
-        isJson: false,
-      });
-
-      if (!result.content) {
-        return jsonReply(res, 200, { ok: true, answer: result.answer, updated: false });
-      }
-
-      // Save the revised plan
-      safeWrite(sourcePlanPath, result.content);
-
-      // Step 2: Pause the old PRD so it stops materializing items
-      const prd = JSON.parse(safeRead(prdPath) || '{}');
-      prd.status = 'revision-requested';
-      prd.revision_feedback = body.instruction;
-      prd.revisionRequestedAt = new Date().toISOString();
-      safeWrite(prdPath, prd);
-
-      // Step 3: Clean up pending/failed work items from old PRD
-      let reset = 0, kept = 0;
-      const wiPaths = [{ path: path.join(MINIONS_DIR, 'work-items.json'), label: 'central' }];
-      for (const proj of PROJECTS) {
-        wiPaths.push({ path: shared.projectWorkItemsPath(proj), label: proj.name });
-      }
-      const deletedItemIds = [];
-      for (const wiInfo of wiPaths) {
-        try {
-          const items = safeJson(wiInfo.path);
-          const filtered = [];
-          for (const w of items) {
-            if (w.sourcePlan === body.source) {
-              if (w.status === 'pending' || w.status === 'failed') {
-                reset++;
-                deletedItemIds.push(w.id);
-              } else {
-                kept++;
-                filtered.push(w);
-              }
-            } else {
-              filtered.push(w);
-            }
-          }
-          if (filtered.length < items.length) safeWrite(wiInfo.path, filtered);
-        } catch (e) { console.error('work item deletion:', e.message); }
-      }
-      for (const itemId of deletedItemIds) {
-        cleanDispatchEntries(d =>
-          d.meta?.item?.sourcePlan === body.source && d.meta?.item?.id === itemId
-        );
-      }
-
-      // Step 4: Dispatch plan-to-prd to regenerate PRD from revised plan
-      const centralWiPath = path.join(MINIONS_DIR, 'work-items.json');
-      let centralItems = [];
-      try { centralItems = JSON.parse(safeRead(centralWiPath) || '[]'); } catch {}
-      const wiId = 'W-' + shared.uid();
-      centralItems.push({
-        id: wiId,
-        title: 'Regenerate PRD from revised plan: ' + sourcePlanFile,
-        type: 'plan-to-prd',
-        priority: 'high',
-        description: `The source plan \`${sourcePlanFile}\` has been revised. Convert it into a fresh PRD JSON.\n\nRevision instruction: ${body.instruction}\n\nRead the revised plan, generate updated PRD items (missing_features), and write to \`prd/${body.source}\`. Set status to "approved". Include \`"source_plan": "${sourcePlanFile}"\` in the JSON root.\n\nPreserve items that are already done (status "implemented" or "complete"). Reset or replace items that were pending/failed.`,
-        status: 'pending',
-        created: new Date().toISOString(),
-        createdBy: 'dashboard:revise-and-regenerate',
-        project: prd.project || '',
-        planFile: sourcePlanFile,
-      });
-      safeWrite(centralWiPath, centralItems);
-
-      return jsonReply(res, 200, {
-        ok: true,
-        answer: result.answer,
-        updated: true,
-        sourcePlan: sourcePlanFile,
-        prdPaused: true,
-        reset,
-        kept,
-        workItemId: wiId,
-      });
-    } catch (e) { return jsonReply(res, 500, { error: e.message }); }
-  }
-
   async function handlePlansDiscuss(req, res) {
     try {
       const body = await readBody(req);
@@ -2570,7 +2428,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
                     let changed = false;
                     for (const w of items) {
                       if (w.sourcePlan !== f) continue;
-                      if (w.status === 'done' || w.status === 'implemented' || w.status === 'complete' || w.status === 'in-pr') continue;
+                      if (w.status === 'done') continue;
                       if (w.status === 'dispatched') {
                         const activeEntry = (dispatch.active || []).find(d => d.meta?.item?.id === w.id || d.meta?.dispatchKey?.includes(w.id));
                         if (activeEntry) {
@@ -3078,30 +2936,33 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!config.agents) config.agents = {};
 
       if (body.engine) {
-        // Validate and apply engine settings
         const e = body.engine;
         const D = shared.ENGINE_DEFAULTS;
-        if (e.tickInterval !== undefined) config.engine.tickInterval = Math.max(10000, Number(e.tickInterval) || D.tickInterval);
-        if (e.maxConcurrent !== undefined) config.engine.maxConcurrent = Math.max(1, Math.min(10, Number(e.maxConcurrent) || D.maxConcurrent));
-        if (e.inboxConsolidateThreshold !== undefined) config.engine.inboxConsolidateThreshold = Math.max(1, Number(e.inboxConsolidateThreshold) || D.inboxConsolidateThreshold);
-        if (e.agentTimeout !== undefined) config.engine.agentTimeout = Math.max(60000, Number(e.agentTimeout) || D.agentTimeout);
-        if (e.maxTurns !== undefined) config.engine.maxTurns = Math.max(5, Math.min(500, Number(e.maxTurns) || D.maxTurns));
-        if (e.heartbeatTimeout !== undefined) config.engine.heartbeatTimeout = Math.max(60000, Number(e.heartbeatTimeout) || D.heartbeatTimeout);
-        if (e.worktreeCreateTimeout !== undefined) config.engine.worktreeCreateTimeout = Math.max(60000, Number(e.worktreeCreateTimeout) || D.worktreeCreateTimeout);
-        if (e.worktreeCreateRetries !== undefined) config.engine.worktreeCreateRetries = Math.max(0, Math.min(3, Number(e.worktreeCreateRetries) || D.worktreeCreateRetries));
+        const numericFields = {
+          tickInterval: [10000], maxConcurrent: [1, 10], inboxConsolidateThreshold: [1],
+          agentTimeout: [60000], maxTurns: [5, 500], heartbeatTimeout: [60000],
+          worktreeCreateTimeout: [60000], worktreeCreateRetries: [0, 3],
+          idleAlertMinutes: [1], shutdownTimeout: [30000], restartGracePeriod: [60000],
+          meetingRoundTimeout: [60000],
+        };
+        for (const [key, [min, max]] of Object.entries(numericFields)) {
+          if (e[key] !== undefined) {
+            let val = Number(e[key]) || D[key];
+            val = Math.max(min, val);
+            if (max !== undefined) val = Math.min(max, val);
+            config.engine[key] = val;
+          }
+        }
         if (e.worktreeRoot !== undefined) config.engine.worktreeRoot = String(e.worktreeRoot || D.worktreeRoot);
-        if (e.idleAlertMinutes !== undefined) config.engine.idleAlertMinutes = Math.max(1, Number(e.idleAlertMinutes) || D.idleAlertMinutes);
-        if (e.shutdownTimeout !== undefined) config.engine.shutdownTimeout = Math.max(30000, Number(e.shutdownTimeout) || D.shutdownTimeout);
-        if (e.restartGracePeriod !== undefined) config.engine.restartGracePeriod = Math.max(60000, Number(e.restartGracePeriod) || D.restartGracePeriod);
-        if (e.meetingRoundTimeout !== undefined) config.engine.meetingRoundTimeout = Math.max(60000, Number(e.meetingRoundTimeout) || D.meetingRoundTimeout);
-        if (e.autoApprovePlans !== undefined) config.engine.autoApprovePlans = !!e.autoApprovePlans;
-        if (e.autoDecompose !== undefined) config.engine.autoDecompose = !!e.autoDecompose;
-        if (e.allowTempAgents !== undefined) config.engine.allowTempAgents = !!e.allowTempAgents;
+        for (const key of ['autoApprovePlans', 'autoReview', 'autoDecompose', 'allowTempAgents']) {
+          if (e[key] !== undefined) config.engine[key] = !!e[key];
+        }
       }
 
       if (body.claude) {
-        if (body.claude.allowedTools !== undefined) config.claude.allowedTools = String(body.claude.allowedTools);
-        if (body.claude.outputFormat !== undefined) config.claude.outputFormat = String(body.claude.outputFormat);
+        for (const key of ['allowedTools', 'outputFormat']) {
+          if (body.claude[key] !== undefined) config.claude[key] = String(body.claude[key]);
+        }
       }
 
       if (body.agents) {

package/docs/deprecated.json

@@ -1,83 +1 @@
-[
-  {
-    "id": "status-in-pr",
-    "summary": "in-pr status alias — backward compat shim for done",
-    "deprecated": "2026-03-21",
-    "reason": "Simplified status model: agents mark items done directly, no intermediate in-pr state",
-    "locations": [
-      "dashboard.html: CSS classes, progress bar, status labels, graph colors (~10 locations)",
-      "dashboard.js:1525 completedStatuses set",
-      "engine.js:1165 PRD_MET_STATUSES set",
-      "engine.js:1180 dependency check",
-      "engine.js:1974 completedStatuses set",
-      "engine/lifecycle.js:54,61 plan completion gate",
-      "engine/queries.js:466,580,597 status ordering and counting"
-    ],
-    "cleanup": "Remove in-pr from all status checks, CSS, and display logic. Only keep done."
-  },
-  {
-    "id": "status-implemented",
-    "summary": "implemented status alias — legacy alias for done",
-    "deprecated": "2026-03-21",
-    "reason": "Canonical status is done. implemented was the original name before standardization.",
-    "locations": [
-      "dashboard.html: CSS, progress filters, status labels (~8 locations)",
-      "dashboard.js:1525 completedStatuses set",
-      "engine.js:1165,1974 status sets",
-      "engine/lifecycle.js:694 sets implemented on post-merge"
-    ],
-    "cleanup": "Replace all implemented references with done. Update lifecycle.js post-merge to set done."
-  },
-  {
-    "id": "status-complete",
-    "summary": "complete status alias — another legacy alias for done",
-    "deprecated": "2026-03-21",
-    "reason": "Canonical status is done. complete appeared in early PRD schemas.",
-    "locations": [
-      "dashboard.html: progress filters, completion checks (~4 locations)",
-      "engine.js:1165 PRD_MET_STATUSES set"
-    ],
-    "cleanup": "Remove complete from all status checks. Only keep done."
-  },
-  {
-    "id": "dead-plan-version-actions",
-    "summary": "showPlanVersionActions, qaJustSave, qaReplacePrd, qaNewPrd, qaDisablePrdButtons — dead code",
-    "deprecated": "2026-03-21",
-    "reason": "Doc-chat fork logic removed. These functions are unreachable.",
-    "locations": [
-      "dashboard.html:3775-3860 five function definitions (~80 lines)"
-    ],
-    "cleanup": "Delete the five functions entirely."
-  },
-  {
-    "id": "dead-modal-original-plan",
-    "summary": "_modalOriginalPlan variable — dead code",
-    "deprecated": "2026-03-21",
-    "reason": "Tracked original plan for fork edits. Fork logic removed.",
-    "locations": [
-      "dashboard.html:3187 declaration",
-      "dashboard.html:2018 reset in closeModal"
-    ],
-    "cleanup": "Delete the variable declaration and all references."
-  },
-  {
-    "id": "dead-revise-and-regenerate",
-    "summary": "/api/plans/revise-and-regenerate endpoint — disabled behind if(false)",
-    "deprecated": "2026-03-21",
-    "reason": "Plan versioning handled differently now. Endpoint was explicitly disabled.",
-    "locations": [
-      "dashboard.js:1782-1829 (~45 lines of dead code)"
-    ],
-    "cleanup": "Delete the entire if(false) block."
-  },
-  {
-    "id": "dead-steer-btn-comments",
-    "summary": "// steer btn removed — unified send comments",
-    "deprecated": "2026-03-21",
-    "reason": "Steer button was removed long ago. Comments are noise.",
-    "locations": [
-      "dashboard.html: 4 occurrences"
-    ],
-    "cleanup": "Delete the comments."
-  }
-]
+[]

package/engine/lifecycle.js

@@ -58,7 +58,7 @@ function checkPlanCompletion(meta, config) {
   const unmaterialized = [...planFeatureIds].filter(id => {
     if (workItemById[id]) return false;
     const prdItem = (plan.missing_features || []).find(f => f.id === id);
-    return !(prdItem && (prdItem.status === 'done' || prdItem.status === 'in-pr'));
+    return !(prdItem && prdItem.status === 'done');
   });
   if (unmaterialized.length > 0) {
     log('info', `Plan ${planFile}: ${unmaterialized.length}/${planFeatureIds.size} feature(s) not yet materialized as work items: ${unmaterialized.join(', ')}`);
@@ -68,16 +68,16 @@ function checkPlanCompletion(meta, config) {
   // Check 2: every feature's work item must be done (or PRD item marked done externally)
   const notDone = [...planFeatureIds].filter(id => {
     const w = workItemById[id];
-    if (w && (w.status === 'done' || w.status === 'in-pr')) return false; // in-pr accepted for backward compat
+    if (w && w.status === 'done') return false;
     const prdItem = (plan.missing_features || []).find(f => f.id === id);
-    return !(prdItem && (prdItem.status === 'done' || prdItem.status === 'in-pr'));
+    return !(prdItem && prdItem.status === 'done');
   });
   if (notDone.length > 0) {
     log('info', `Plan ${planFile}: waiting for done on ${notDone.length}/${planFeatureIds.size} item(s): ${notDone.join(', ')}`);
     return;
   }
 
-  const doneItems = planItems.filter(w => w.status === 'done' || w.status === 'in-pr');
+  const doneItems = planItems.filter(w => w.status === 'done');
   const failedItems = planItems.filter(w => w.status === 'failed');
 
   // 1. Mark plan as completed
@@ -740,7 +740,7 @@ async function handlePostMerge(pr, project, config, newStatus) {
   }
 
   if (mergedItemId) {
-    // Mark PRD feature as implemented
+    // Mark PRD feature as done
     const prdDir = path.join(MINIONS_DIR, 'prd');
     try {
       const planFiles = fs.readdirSync(prdDir).filter(f => f.endsWith('.json'));
@@ -749,13 +749,13 @@ async function handlePostMerge(pr, project, config, newStatus) {
         const plan = safeJson(path.join(prdDir, pf));
         if (!plan?.missing_features) continue;
         const feature = plan.missing_features.find(f => f.id === mergedItemId);
-        if (feature && feature.status !== 'implemented') {
-          feature.status = 'implemented';
+        if (feature && feature.status !== 'done') {
+          feature.status = 'done';
           shared.safeWrite(path.join(prdDir, pf), plan);
           updated++;
         }
       }
-      if (updated > 0) log('info', `Post-merge: marked ${mergedItemId} as implemented for ${pr.id}`);
+      if (updated > 0) log('info', `Post-merge: marked ${mergedItemId} as done for ${pr.id}`);
     } catch (err) { log('warn', `Post-merge PRD update: ${err.message}`); }
 
     // Mark work item as done
@@ -1011,7 +1011,8 @@ function updateMetrics(agentId, dispatchItem, result, taskUsage, prsCreatedCount
     if (taskUsage.numTurns > cp.maxTurns) cp.maxTurns = taskUsage.numTurns;
     // Check if this dispatch hit the turn limit
     const engineConfig = require('./queries').getConfig()?.engine || {};
-    const turnLimit = engineConfig.maxTurns || 100;
+    // maxTurns default defined in ENGINE_DEFAULTS (shared.js) — avoid hardcoded fallback
+    const turnLimit = engineConfig.maxTurns || shared.ENGINE_DEFAULTS.maxTurns;
     if (taskUsage.numTurns >= turnLimit) cp.turnLimitHits++;
   }
 
@@ -1206,7 +1207,8 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   if (meta?.item?.id) {
     try {
       const engineConfig = (config.engine || {});
-      const turnLimit = engineConfig.maxTurns || 100;
+      // maxTurns resolved from config, fallback to ENGINE_DEFAULTS (shared.js)
+      const turnLimit = engineConfig.maxTurns || shared.ENGINE_DEFAULTS.maxTurns;
       const turnCount = taskUsage?.numTurns || 0;
       const hitTurnLimit = turnCount >= turnLimit;
       let outputLogSizeBytes = 0;

package/engine/playbook.js

@@ -478,7 +478,7 @@ function selectPlaybook(workType, item) {
   if (workType === 'review' && !item?._pr && !item?.pr_id) {
     return 'work-item';
   }
-  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
+  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'evaluate', 'decompose', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
   return typeSpecificPlaybooks.includes(workType) ? workType : 'work-item';
 }
 

package/engine/queries.js

@@ -506,9 +506,7 @@ function getWorkItems(config) {
     pending: 0,
     queued: 0,
     dispatched: 1,
-    'in-pr': 3, // backward compat — treated as done
     done: 3,
-    implemented: 3,
     failed: 4,
     paused: 5,
   };
@@ -627,7 +625,7 @@ function getPrdInfo(config) {
 
   const byStatus = {};
   items.forEach(item => { const s = item.status || 'missing'; byStatus[s] = byStatus[s] || []; byStatus[s].push(item); });
-  const complete = (byStatus['done'] || []).length + (byStatus['in-pr'] || []).length; // in-pr counted as done for backward compat
+  const complete = (byStatus['done'] || []).length;
   const inProgress = (byStatus['in-progress'] || []).length;
   const paused = (byStatus['paused'] || []).length;
   const missing = (byStatus['missing'] || []).length;
@@ -641,7 +639,7 @@ function getPrdInfo(config) {
     const t = planTimings[wi.sourcePlan];
     if (wi.dispatched_at) { const d = new Date(wi.dispatched_at).getTime(); if (!t.firstDispatched || d < t.firstDispatched) t.firstDispatched = d; }
     if (wi.completedAt) { const c = new Date(wi.completedAt).getTime(); if (!t.lastCompleted || c > t.lastCompleted) t.lastCompleted = c; }
-    if (wi.status !== 'done' && wi.status !== 'in-pr') t.allDone = false; // in-pr treated as done for backward compat
+    if (wi.status !== 'done') t.allDone = false;
   }
 
   const progress = {

package/engine.js

@@ -743,7 +743,7 @@ function areDependenciesMet(item, config) {
     } catch (e) { log('warn', 'read project work items for deps: ' + e.message); }
   }
   // PRD item statuses that count as "done" for dep resolution
-  const PRD_MET_STATUSES = new Set(['done', 'in-pr', 'implemented', 'complete']);
+  const PRD_MET_STATUSES = new Set(['done']);
 
   for (const depId of deps) {
     const depItem = allWorkItems.find(w => w.id === depId);
@@ -977,7 +977,7 @@ function materializePlansAsWorkItems(config) {
             log('info', `PRD ${file} invalidated (was awaiting-approval) — queuing regeneration from revised plan`);
 
             // Collect completed items to carry over to new PRD
-            const completedStatuses = new Set(['done', 'in-pr', 'implemented']); // in-pr kept for backward compat
+            const completedStatuses = new Set(['done']);
             const completedItems = (plan.missing_features || [])
               .filter(f => completedStatuses.has(f.status))
               .map(f => ({ id: f.id, name: f.name, status: f.status }));
@@ -1057,7 +1057,7 @@ function materializePlansAsWorkItems(config) {
     const useCentral = !defaultProject;
 
     const statusFilter = ['missing', 'planned'];
-    // Also materialize in-pr/done items that never got a work item (race with PR status sync)
+    // Also materialize done items that never got a work item (race with PR status sync)
     const allExistingWiIds = new Set();
     for (const p of allProjects) {
       for (const w of (safeJson(projectWorkItemsPath(p)) || [])) {
@@ -1070,7 +1070,7 @@ function materializePlansAsWorkItems(config) {
     }
     const items = plan.missing_features.filter(f =>
       statusFilter.includes(f.status) ||
-      ((f.status === 'in-pr' || f.status === 'done') && f.id && !allExistingWiIds.has(f.id))
+      (f.status === 'done' && f.id && !allExistingWiIds.has(f.id))
     );
 
     // Group items by target project (per-item project field overrides plan-level project)
@@ -2203,7 +2203,7 @@ async function tickInner() {
     try { await ghPollPrStatus(config); } catch (err) { log('warn', `GitHub PR status poll error: ${err?.message || err}${err?.stack ? ' | ' + err.stack.split('\n')[1]?.trim() : ''}`); }
     // Sync PR status back to PRD items (missing → done when active PR exists)
     try { syncPrdFromPrs(config); } catch (err) { log('warn', `PRD sync error: ${err?.message || err}`); }
-    // Check if any plans can be marked completed (all features done/in-pr)
+    // Check if any plans can be marked completed (all features done)
     try {
       const prdFiles = safeReadDir(PRD_DIR).filter(f => f.endsWith('.json'));
       for (const file of prdFiles) {

package/package.json

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