@yemi33/minions 0.1.2447 → 0.1.2449

package/dashboard/js/render-dispatch.js

@@ -349,8 +349,8 @@ function renderDispatch(dispatch, opts) {
     const compStart = _completedPage * COMPLETED_PER_PAGE;
     const pageCompleted = completed.slice(compStart, compStart + COMPLETED_PER_PAGE);
 
-    // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: dispatch id, type, agent, task, result, reason)
-    completedEl.innerHTML = '<table class="pr-table"><thead><tr><th>ID</th><th>Type</th><th>Agent</th><th>Task</th><th>Result</th><th>Completed</th></tr></thead><tbody>' +
+    // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: type, agent, task, result, reason)
+    completedEl.innerHTML = '<table class="pr-table"><thead><tr><th>Type</th><th>Agent</th><th>Task</th><th>Result</th><th>Completed</th></tr></thead><tbody>' +
       pageCompleted.map(d => {
         const isError = d.result === 'error';
         const agentId = (d.agent || '').toLowerCase();
@@ -358,10 +358,9 @@ function renderDispatch(dispatch, opts) {
           ? ' <button class="error-details-btn" data-agent="' + escHtml(agentId) + '" data-reason="' + escHtml(d.reason || 'No reason recorded') + '" data-task="' + escHtml((d.task || '').slice(0, 100)) + '" onclick="showErrorDetails(this.dataset.agent, this.dataset.reason, this.dataset.task)" title="View error details">details</button>'
           : '';
         return '<tr>' +
-          '<td style="font-family:Consolas;font-size:var(--text-sm)" title="' + escHtml(d.id || '') + '">' + escHtml(d.id || '') + '</td>' +
           '<td><span class="dispatch-type ' + (d.type || '') + '">' + escHtml(d.type || '') + '</span></td>' +
           '<td>' + escHtml(d.agentName || d.agent || '') + '</td>' +
-          '<td style="max-width:200px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">' + escHtml((d.task || '').slice(0, 60)) + '</td>' +
+          '<td style="width:100%;max-width:0;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">' + escHtml((d.task || '').slice(0, 60)) + '</td>' +
           '<td style="color:' + (d.result === 'success' ? 'var(--green)' : 'var(--red)') + '">' + escHtml(d.result || '') + errorBtn + '</td>' +
           '<td class="pr-date">' + shortTime(d.completed_at) + '</td>' +
         '</tr>';

package/dashboard/js/render-inbox.js

@@ -3,7 +3,7 @@
 const INBOX_PER_PAGE = 15;
 let _inboxPage = 0;
 
-// Quick relative-time formatter — mirrors engine/queries.js timeSince so the
+// Quick relative-time formatter — mirrors engine/core/queries.js timeSince so the
 // inbox-item "age" chip reads the same as it did via /api/status.
 function _timeSinceMs(mtimeMs) {
   if (!mtimeMs) return '';
@@ -78,7 +78,7 @@ async function fetchNotesFromDisk() {
 // Promote a human-readable title to the inbox card heading. Prefers an
 // explicit `title` field from the API, then a `title:` YAML frontmatter key,
 // then the first markdown heading, then the first non-empty body line. Mirrors
-// the engine's own title derivation (engine/consolidation.js — first `# ` ATX
+// the engine's own title derivation (engine/memory/consolidation.js — first `# ` ATX
 // heading). Returns '' when nothing readable is extractable so the caller can
 // fall back to the raw filename.
 function _extractInboxTitle(item) {

package/dashboard/js/render-other.js

@@ -24,7 +24,7 @@ function renderProjects(projects) {
 }
 
 // Renders the per-project current-branch indicator. Driven by the gitState
-// classifier emitted by getProjectGitStatus() in engine/queries.js.
+// classifier emitted by getProjectGitStatus() in engine/core/queries.js.
 // Adds a title attribute carrying the full branch name so users can still
 // read it on hover after the CSS ellipsis-truncate (W-mpayac6d000b7d33).
 //
@@ -78,7 +78,7 @@ function _renderProjectBranch(p) {
   const detachedSuffix = p.gitDetached ? ' <span class="muted">(detached)</span>' : '';
   // opg-microsoft/minions#664 — the counts are always vs origin/<comparator>
   // (configured mainBranch, falling back to remoteDefaultBranch; see
-  // engine/queries.js _probeProjectGitStatus), never vs the checked-out
+  // engine/core/queries.js _probeProjectGitStatus), never vs the checked-out
   // branch's own upstream. When branchDiffersFromMain the bare "↑N ↓M" chip
   // reads as ambiguous (looks like it could be vs the current branch's
   // tracking ref), so append the comparator name to the VISIBLE text too —
@@ -681,7 +681,7 @@ function renderLlmPerf(metrics) {
 // Excludes:
 //   - 'agent-dispatch' (already counted in per-agent metrics[agentId] totals;
 //     summing both would double-count agent spend in the tile)
-//   - 'test-*' / '_test*' (matching engine/llm.js trackEngineUsage filter so
+//   - 'test-*' / '_test*' (matching engine/agents/llm.js trackEngineUsage filter so
 //     stale test categories don't inflate engineCalls)
 function _aggregateEngineUsageForTokenTile(engine) {
   let cost = 0, input = 0, output = 0, cache = 0, calls = 0;

package/dashboard/js/render-pipelines.js

@@ -881,3 +881,17 @@ async function _submitEditPipeline() {
 }
 
 window.MinionsPipelines = { renderPipelines, openPipelineDetail, openCreatePipelineModal, openEditPipelineModal };
+
+// ── Slim "+ Add" deep-link (W-ms258yw0000ffd8a) ────────────────────────
+// The Slim Automation tile's "+ Add" picker embeds this classic screen at
+// /pipelines?embed=1#create and relies on this hook to auto-open the Create
+// Pipeline dialog — the classic create flow is reused, not forked (CLAUDE.md
+// slim-vs-classic). Fires only on the /pipelines path with a #create hash; the
+// standalone classic screen and every other embed is otherwise untouched.
+// No isReady predicate: unlike the schedule/watch forms this one has no
+// cmdProjects/cmdAgents-backed selects, so it has nothing to wait for.
+// Helper lives in utils.js#bootCreateDeepLink.
+typeof bootCreateDeepLink === 'function' && bootCreateDeepLink({
+  route: '/pipelines',
+  resolve: function() { return typeof openCreatePipelineModal === 'function' ? openCreatePipelineModal : null; },
+});

package/dashboard/js/render-plans.js

@@ -285,7 +285,11 @@ function renderPlans(plans) {
   const archivedPlans = plans.filter(p => p.archived && p.format !== 'prd');
   countEl.textContent = activePlans.length + (archivedPlans.length ? ' + ' + archivedPlans.length + ' archived' : '');
 
-  function renderPlanCard(p) {
+  // Derive the plan's PRD linkage, archive flag, and canonical lifecycle
+  // status. Extracted so the Active/Completed grouping and the card renderer
+  // share one source of truth for "completed" (the derived lifecycle status),
+  // rather than matching on display text.
+  function computePlanStatusContext(p) {
     const prdFile = planToPrdFile[p.file] || (p.file.endsWith('.json') ? p.file : '');
     const isArchived = p.archived;
 
@@ -306,6 +310,11 @@ function renderPlans(plans) {
 
     // Single source of truth: derive status from work items
     const effectiveStatus = isArchived ? 'completed' : derivePlanStatus(prdFile, p.file, prdJsonStatus, allWi);
+    return { prdFile, isArchived, effectiveStatus };
+  }
+
+  function renderPlanCard(p, ctx) {
+    const { prdFile, isArchived, effectiveStatus } = ctx || computePlanStatusContext(p);
 
     const statusLabelsMap = {
       'completed': 'Completed', 'dispatched': 'In Progress', 'converting': 'Converting to PRD',
@@ -370,6 +379,19 @@ function renderPlans(plans) {
     const deleteBtn = !isArchived ? '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 8px;color:var(--red)" ' +
       'onclick="event.stopPropagation();planDelete(\'' + escapeHtml(p.file) + '\')">Delete</button>' : '';
 
+    // Verify status badges stay inline in the meta line as informational
+    // status; the interactive action buttons move to a dedicated row below.
+    const verifyBadgesHtml = verifyWis.length >= 2
+      ? verifyWis.map(v => _renderVerifyBadge(v, { projectLabel: v.project || '' })).join(' ')
+      : (hasVerifyWi ? _renderVerifyBadge(verifyWis[0]) : '');
+    const actionButtons = executeBtn + pauseBtn + resumeBtn + verifyBtn + archiveBtn + deleteBtn;
+    // Dedicated horizontal action row rendered immediately under the status
+    // (meta) line. Skipped entirely when a card has no buttons so empty rows
+    // never render.
+    const actionRow = actionButtons
+      ? '<div class="plan-card-actions plan-card-action-row" onclick="event.stopPropagation()">' + actionButtons + '</div>'
+      : '';
+
     const versionBadge = p.version ? ' <span style="font-size:var(--text-sm);font-weight:700;padding:1px 5px;border-radius:3px;background:rgba(56,139,253,0.15);color:var(--blue);vertical-align:middle">v' + p.version + '</span>' : '';
     const statusColors = { 'completed': 'var(--green)', 'dispatched': 'var(--blue)', 'converting': 'var(--yellow)', 'paused': 'var(--muted)', 'awaiting-approval': 'var(--yellow)', 'approved': 'var(--purple,#a855f7)', 'rejected': 'var(--red)', 'has-failures': 'var(--red)', 'revision-requested': 'var(--purple,#a855f7)', 'active': 'var(--muted)', 'draft': 'var(--muted)' };
     const cardClass = effectiveStatus === 'dispatched' || effectiveStatus === 'converting' ? 'working' : effectiveStatus === 'awaiting-approval' || effectiveStatus === 'paused' ? 'awaiting' : effectiveStatus;
@@ -414,25 +436,51 @@ function renderPlans(plans) {
             (p.updatedAt ? '<span title="Last updated: ' + p.updatedAt + '">Updated ' + timeAgo(p.updatedAt) + '</span>' : '') +
             (p.completedAt ? '<span>' + p.completedAt.slice(0, 10) + '</span>' : '') +
             (p.generatedBy ? '<span>by ' + escapeHtml(p.generatedBy) + '</span>' : '') +
-            executeBtn + pauseBtn + resumeBtn + verifyBtn + (verifyWis.length >= 2 ? verifyWis.map(v => _renderVerifyBadge(v, { projectLabel: v.project || '' })).join(' ') : (hasVerifyWi ? _renderVerifyBadge(verifyWis[0]) : '')) + archiveReadyBadge + archiveBtn + deleteBtn +
+            verifyBadgesHtml + archiveReadyBadge +
           '</div>' +
+          actionRow +
         '</div>' +
       '</div>' +
       actions +
     '</div>';
   }
 
-  const totalPlanPages = Math.ceil(activePlans.length / PLANS_PER_PAGE);
+  // Group displayed plans into Active vs Completed using the canonical derived
+  // lifecycle status (computePlanStatusContext → derivePlanStatus), preserving
+  // the existing order within each group. The combined ordering is
+  // [active…, completed…]; pagination runs over that combined list so a single
+  // pager still bounds the page, and section headers are emitted where the
+  // group changes within the page.
+  const displayEntries = activePlans.map(p => {
+    const ctx = computePlanStatusContext(p);
+    return { plan: p, ctx, completed: ctx.effectiveStatus === 'completed' };
+  });
+  const orderedEntries = displayEntries.filter(e => !e.completed)
+    .concat(displayEntries.filter(e => e.completed));
+
+  const planSectionHeader = (group) =>
+    '<div class="plan-section-header">' + (group === 'completed' ? 'Completed Plans' : 'Active') + '</div>';
+
+  const totalPlanPages = Math.ceil(orderedEntries.length / PLANS_PER_PAGE);
   if (_plansPage >= totalPlanPages) _plansPage = totalPlanPages - 1;
   if (_plansPage < 0) _plansPage = 0;
   const plansStart = _plansPage * PLANS_PER_PAGE;
-  const pagePlans = activePlans.slice(plansStart, plansStart + PLANS_PER_PAGE);
-
-  let html = pagePlans.map(renderPlanCard).join('');
+  const pageEntries = orderedEntries.slice(plansStart, plansStart + PLANS_PER_PAGE);
+
+  let html = '';
+  let lastGroup = null;
+  for (const entry of pageEntries) {
+    const group = entry.completed ? 'completed' : 'active';
+    if (group !== lastGroup) {
+      html += planSectionHeader(group);
+      lastGroup = group;
+    }
+    html += renderPlanCard(entry.plan, entry.ctx);
+  }
 
-  if (activePlans.length > PLANS_PER_PAGE) {
+  if (orderedEntries.length > PLANS_PER_PAGE) {
     html += '<div class="pr-pager">' +
-      '<span class="pr-page-info">' + (plansStart + 1) + '-' + Math.min(plansStart + PLANS_PER_PAGE, activePlans.length) + ' of ' + activePlans.length + '</span>' +
+      '<span class="pr-page-info">' + (plansStart + 1) + '-' + Math.min(plansStart + PLANS_PER_PAGE, orderedEntries.length) + ' of ' + orderedEntries.length + '</span>' +
       '<div class="pr-pager-btns">' +
         '<button class="pr-pager-btn ' + (_plansPage === 0 ? 'disabled' : '') + '" onclick="_plansPrev()">Prev</button>' +
         '<button class="pr-pager-btn ' + (_plansPage >= totalPlanPages - 1 ? 'disabled' : '') + '" onclick="_plansNext()">Next</button>' +
@@ -687,7 +735,7 @@ function _renderPlanModal(normalizedFile, raw, lastMod) {
     modalActions += '<span style="' + bs + ';color:var(--blue)">In Progress</span> ';
   }
   // Verify / Verified badge — cross-repo plans fan out one verify WI per
-  // project (engine/lifecycle.js); collect them all and render one badge
+  // project (engine/orchestration/lifecycle.js); collect them all and render one badge
   // per project so operators can see every repo's verify status.
   const modalVerifyWis = (window._lastWorkItems || []).filter(w => w.itemType === 'verify' && w.sourcePlan === (prdFile || normalizedFile));
   if (effectiveStatus === 'completed' && prdFile && !isArchived && modalVerifyWis.length === 0) {

package/dashboard/js/render-prd.js

@@ -63,6 +63,47 @@ async function _savePrdDesc(source, itemId) {
   if (saveBtn) { saveBtn.disabled = false; saveBtn.textContent = 'Save'; }
 }
 
+// ── PRD item execution-state predicates — single source of truth ────────────
+// engine/core/queries.js#getPrdInfo projects each PRD item's `status` from its
+// materialized work item and maps `pending` -> `missing`, `dispatched` ->
+// `in-progress`, defaulting a blank status to `missing`. So `missing` means the
+// item has NOT STARTED — either no work item was ever materialized, or one
+// exists and is still queued. It is materializability, never execution.
+//
+// Folding `missing` into "active" used to happen independently in the header,
+// the group summary and the item detail modal, so an APPROVED PRD whose items
+// were all `missing` (with zero pending/dispatched work items) rendered as
+// `dispatched` / "In Progress". Every derivation below routes through these
+// predicates so the three paths cannot drift apart again.
+const PRD_ACTIVE_ITEM_STATUSES = ['in-progress'];
+const PRD_NOT_STARTED_ITEM_STATUSES = ['missing'];
+const PRD_ACTIVE_WORK_ITEM_STATUSES = ['pending', 'dispatched'];
+
+// Canonical "this PRD item is executing right now".
+function prdItemIsActive(item) {
+  return !!item && PRD_ACTIVE_ITEM_STATUSES.indexOf(item.status) !== -1;
+}
+
+// Canonical "this PRD item has not started". A blank status counts: the server
+// defaults it to `missing`, so an unset value carries the same meaning.
+function prdItemIsNotStarted(item) {
+  if (!item) return false;
+  return !item.status || PRD_NOT_STARTED_ITEM_STATUSES.indexOf(item.status) !== -1;
+}
+
+// Canonical "this materialized work item is live execution state".
+function prdWorkItemIsActive(wi) {
+  return !!wi && PRD_ACTIVE_WORK_ITEM_STATUSES.indexOf(wi.status) !== -1;
+}
+
+// A PRD is "In Progress" only from canonical active state: a materialized work
+// item in pending/dispatched, or an `in-progress` item projection (which also
+// covers items whose work-item id differs from the PRD feature id and are
+// therefore missed by the feature-id join the callers do).
+function prdHasActiveExecution(items, workItems) {
+  return (workItems || []).some(prdWorkItemIsActive) || (items || []).some(prdItemIsActive);
+}
+
 function renderPrd(prd, prog) {
   const section = document.getElementById('prd-content');
   const badge = document.getElementById('prd-badge');
@@ -94,9 +135,9 @@ function renderPrd(prd, prog) {
     // 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');
-    // Unmaterialized PRD items (status === 'missing') count as active — they still need work.
-    const hasMissing = prdItems.some(i => i.status === 'missing');
-    const hasActive = hasMissing || implementItems.some(w => w.status === 'pending' || w.status === 'dispatched');
+    // Unmaterialized PRD items (status === 'missing') are NOT started, so they
+    // never imply execution on their own — see prdHasActiveExecution above.
+    const hasActive = prdHasActiveExecution(prdItems, implementItems);
     const prdFile = existing[0]?.file || '';
     const prdStatus = existing[0]?.status || '';
     const effectiveStatus = allDone && !hasActive ? 'completed' : hasActive ? 'dispatched' : prdStatus || 'active';
@@ -128,9 +169,9 @@ function renderPrd(prd, prog) {
       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');
-      // Unmaterialized PRD items (status === 'missing') count as active — they still need work.
-      const hasMissing = items.some(i => i.status === 'missing');
-      const hasActive = hasMissing || wiForPrd.some(w => w.status === 'pending' || w.status === 'dispatched');
+      // Unmaterialized PRD items (status === 'missing') are NOT started, so they
+      // never imply execution on their own — see prdHasActiveExecution above.
+      const hasActive = prdHasActiveExecution(items, wiForPrd);
       const s = allDone && !hasActive ? 'completed' : hasActive ? 'dispatched' : p.status || 'active';
       counts[s] = (counts[s] || 0) + 1;
     }
@@ -195,11 +236,11 @@ function renderPrdProgress(prog) {
     const total = items.length;
     if (total === 0) return '';
     const done = items.filter(i => i.status === 'done').length;
-    const inProgress = items.filter(i => i.status === 'in-progress').length;
+    const inProgress = items.filter(prdItemIsActive).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 missing = items.filter(prdItemIsNotStarted).length;
     const pct = (n) => total > 0 ? ((n / total) * 100).toFixed(1) : 0;
 
     const stats = '<div class="prd-stats" style="margin:0">' +
@@ -363,10 +404,17 @@ function renderPrdProgress(prog) {
   };
 
   const timings = prog.planTimings || {};
+  // W-msa0mrus00mh466f — server-derived PRD revision state (`none` |
+  // `in-progress` | `failed`). Derivation lives in
+  // engine/core/shared.js#derivePrdRevisionState; this renderer only paints it.
+  const revisions = prog.planRevisions || {};
 
   const renderGroupHeader = (g) => {
     const done = g.items.filter(i => i.status === 'done').length;
-    const wip = g.items.filter(i => i.status === 'in-progress' || i.status === 'missing').length;
+    // `missing` is not-started, not work-in-progress: count it separately so the
+    // summary never advertises a not-yet-materialized item as "active".
+    const wip = g.items.filter(prdItemIsActive).length;
+    const notStarted = g.items.filter(prdItemIsNotStarted).length;
     const summary = (g.summary || '').replace(/^Convert plan to PRD:\s*/i, '').slice(0, 80);
     const isAwaitingApproval = g.planStatus === 'awaiting-approval';
     const isPaused = g.planStatus === 'paused';
@@ -388,29 +436,61 @@ function renderPrdProgress(prog) {
       : isPaused
         ? '<span style="color:var(--muted);font-weight:600;font-size:var(--text-sm);padding:1px 6px;border:1px solid var(--muted);border-radius:3px">PAUSED</span>'
         : '';
-    const showStale = (!isBlocked || isPaused) && g.planStale;
+    const revision = revisions[g.file] || null;
+    const revisionActive = !!revision && revision.state === 'in-progress';
+    const revisionFailed = !!revision && revision.state === 'failed';
+    // A live revision owns this PRD: never render the generic stale recovery
+    // beside it or the user can launch a competing regenerate/resume path.
+    const showStale = (!isBlocked || isPaused) && g.planStale && !revisionActive && !revisionFailed;
     const staleLabel = showStale
       ? '<span style="color:var(--orange);font-weight:700;font-size:var(--text-sm);padding:1px 6px;border:1px solid var(--orange);border-radius:3px;background:rgba(210,153,34,0.12)" title="Source plan changed after this PRD was generated">STALE</span>'
       : '';
+    const revisionLabel = revisionActive
+      ? '<span style="color:var(--blue);font-weight:700;font-size:var(--text-sm);padding:1px 6px;border:1px solid var(--blue);border-radius:3px;background:rgba(56,139,253,0.12)" title="A revision agent is rewriting this PRD">REVISING</span>'
+      : revisionFailed
+        ? '<span style="color:var(--red);font-weight:700;font-size:var(--text-sm);padding:1px 6px;border:1px solid var(--red);border-radius:3px;background:rgba(248,81,73,0.12)" title="' + escHtml(revision.failReason || 'Revision did not complete') + '">REVISION FAILED</span>'
+        : '';
+    const revisionWiChip = (revision && revision.workItemId && typeof renderArtifactLink === 'function')
+      ? renderArtifactLink({ type: 'wi', id: revision.workItemId, label: revision.workItemId, title: 'Revision work item' })
+      : (revision && revision.workItemId ? '<span class="prd-item-id">' + escHtml(revision.workItemId) + '</span>' : '');
+    const reviewPlanBtn = '<span onclick="event.stopPropagation();planView(\'' + escHtml(g.sourcePlan || g.file) + '\')" style="color:var(--blue);cursor:pointer;font-size:var(--text-sm);padding:2px 8px;background:rgba(56,139,253,0.1);border:1px solid rgba(56,139,253,0.3);border-radius:4px" title="Review latest plan changes">Review plan</span>';
+    const revisionRecovery = revisionActive
+      ? '<div style="width:100%;margin-top:4px;padding:6px 8px;border:1px solid rgba(56,139,253,0.35);border-radius:4px;background:rgba(56,139,253,0.08);display:flex;align-items:center;gap:8px;flex-wrap:wrap">' +
+          '<span style="color:var(--blue);font-size:var(--text-sm);font-weight:600">&#x1F504; Revision in progress. An agent is rewriting this PRD.</span>' +
+          revisionWiChip +
+          (revision.requestedBy ? '<span style="color:var(--muted);font-size:var(--text-sm)">requested by ' + escHtml(revision.requestedBy) + '</span>' : '') +
+          reviewPlanBtn +
+        '</div>'
+      : revisionFailed
+        ? '<div style="width:100%;margin-top:4px;padding:6px 8px;border:1px solid rgba(248,81,73,0.35);border-radius:4px;background:rgba(248,81,73,0.08);display:flex;align-items:center;gap:8px;flex-wrap:wrap">' +
+            '<span style="color:var(--red);font-size:var(--text-sm);font-weight:600">&#x26A0;&#xFE0F; Revision failed: ' + escHtml(revision.failReason || 'the revision did not complete') + '</span>' +
+            revisionWiChip +
+            (revision.workItemId
+              ? '<span onclick="event.stopPropagation();prdRetryRevision(\'' + escHtml(revision.workItemId) + '\')" style="color:var(--green);cursor:pointer;font-size:var(--text-sm);font-weight:700;padding:2px 8px;background:rgba(63,185,80,0.12);border:1px solid rgba(63,185,80,0.35);border-radius:4px" title="Requeue the revision work item">Retry revision</span>'
+              : '<span style="color:var(--muted);font-size:var(--text-sm)">Request a new revision from the Plans page.</span>') +
+            reviewPlanBtn +
+          '</div>'
+        : '';
     const staleRecovery = showStale
       ? '<div style="width:100%;margin-top:4px;padding:6px 8px;border:1px solid rgba(210,153,34,0.35);border-radius:4px;background:rgba(210,153,34,0.08);display:flex;align-items:center;gap:8px;flex-wrap:wrap">' +
           '<span style="color:var(--orange);font-size:var(--text-sm);font-weight:600">&#x26A0;&#xFE0F; Source plan was revised. This PRD may be outdated.</span>' +
           '<span onclick="event.stopPropagation();prdRegenerate(\'' + escHtml(g.file) + '\')" style="color:var(--green);cursor:pointer;font-size:var(--text-sm);font-weight:700;padding:2px 8px;background:rgba(63,185,80,0.12);border:1px solid rgba(63,185,80,0.35);border-radius:4px" title="Compare revised plan against existing PRD and update items">Regenerate PRD</span>' +
           '<span onclick="event.stopPropagation();prdResumeWithoutRegen(\'' + escHtml(g.file) + '\')" style="color:var(--muted);cursor:pointer;font-size:var(--text-sm);padding:2px 8px;background:var(--surface);border:1px solid var(--border);border-radius:4px" title="Resume without regenerating — use current PRD as-is">Resume as-is</span>' +
-          '<span onclick="event.stopPropagation();planView(\'' + escHtml(g.sourcePlan || g.file) + '\')" style="color:var(--blue);cursor:pointer;font-size:var(--text-sm);padding:2px 8px;background:rgba(56,139,253,0.1);border:1px solid rgba(56,139,253,0.3);border-radius:4px" title="Review latest plan changes">Review plan</span>' +
+          reviewPlanBtn +
         '</div>'
       : '';
     const isCompleted = done > 0 && done === g.items.length;
-    // Hide regular action buttons when stale banner is showing — stale banner has its own actions
-    const isStale = showStale;
-    const pauseResumeBtn = isStale ? '' : isAwaitingApproval
+    // Hide regular action buttons when a recovery banner is showing — the banner
+    // owns the recovery actions for that state.
+    const suppressGroupActions = showStale || revisionActive || revisionFailed;
+    const pauseResumeBtn = suppressGroupActions ? '' : isAwaitingApproval
       ? '<span onclick="event.stopPropagation();planApprove(\'' + escHtml(g.file) + '\',this)" style="color:var(--green);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(63,185,80,0.1);border:1px solid rgba(63,185,80,0.3);border-radius:3px">Approve</span>'
       : isPaused
         ? '<span onclick="event.stopPropagation();planApprove(\'' + escHtml(g.file) + '\',this)" style="color:var(--green);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(63,185,80,0.1);border:1px solid rgba(63,185,80,0.3);border-radius:3px">Resume</span>'
         : isCompleted && !(window._lastWorkItems || []).some(w => w.itemType === 'verify' && w.sourcePlan === g.file)
           ? '<span onclick="event.stopPropagation();triggerVerify(\'' + escHtml(g.file) + '\',this)" style="color:var(--green);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(63,185,80,0.1);border:1px solid rgba(63,185,80,0.3);border-radius:3px">Verify</span>'
           : isCompleted ? '' : '<span onclick="event.stopPropagation();planPause(\'' + escHtml(g.file) + '\',this)" style="color:var(--yellow);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(210,153,34,0.1);border:1px solid rgba(210,153,34,0.3);border-radius:3px">Pause</span>';
-    const archiveBtn = (!isStale && (isCompleted || isPaused)) ? '<span onclick="event.stopPropagation();planArchive(\'' + escHtml(g.file) + '\',this)" style="color:var(--muted);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(139,148,158,0.1);border:1px solid rgba(139,148,158,0.3);border-radius:3px">Archive</span>' : '';
+    const archiveBtn = (!suppressGroupActions && (isCompleted || isPaused)) ? '<span onclick="event.stopPropagation();planArchive(\'' + escHtml(g.file) + '\',this)" style="color:var(--muted);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(139,148,158,0.1);border:1px solid rgba(139,148,158,0.3);border-radius:3px">Archive</span>' : '';
     const deleteBtn = '<span onclick="event.stopPropagation();planDelete(\'' + escHtml(g.file) + '\')" style="color:var(--red);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(248,81,73,0.1);border:1px solid rgba(248,81,73,0.3);border-radius:3px">Delete</span>';
     const sourcePlanLink = g.sourcePlan
       ? '<span onclick="event.stopPropagation();planView(\'' + escHtml(g.sourcePlan) + '\')" style="color:var(--blue);cursor:pointer;font-size:var(--text-sm);padding:1px 6px;background:rgba(56,139,253,0.1);border:1px solid rgba(56,139,253,0.3);border-radius:3px" title="View source plan">&#x1F4C4; Plan</span>'
@@ -424,9 +504,10 @@ function renderPrdProgress(prog) {
         : '<span style="font-size:var(--text-sm);padding:1px 5px;border-radius:3px;background:rgba(56,139,253,0.12);color:var(--blue);font-weight:400" title="Each item gets its own branch (work/P-xxx) and PR">&#x1F500; parallel branches</span>') +
       pausedLabel +
       staleLabel +
+      revisionLabel +
       '<span style="font-weight:700;font-size:var(--text-base);color:' + (done === g.items.length && g.items.length > 0 ? 'var(--green)' : 'var(--text)') + '">' + (g.items.length > 0 ? Math.round((done / g.items.length) * 100) : 0) + '%</span>' +
       '<span style="color:var(--muted);font-weight:400;font-size:var(--text-sm)">' + g.items.length + ' items' +
-        (done ? ' · ' + done + ' done' : '') + (wip ? ' · ' + wip + ' active' : '') +
+        (done ? ' · ' + done + ' done' : '') + (wip ? ' · ' + wip + ' active' : '') + (notStarted ? ' · ' + notStarted + ' to do' : '') +
       '</span>' +
       (g.prdUpdatedAt ? '<span style="color:var(--muted);font-weight:400;font-size:var(--text-sm)" title="PRD last updated: ' + g.prdUpdatedAt + '">PRD updated ' + timeAgo(g.prdUpdatedAt) + '</span>' : '') +
       runtimeLabel +
@@ -443,6 +524,7 @@ function renderPrdProgress(prog) {
             deleteBtn +
           '</span>') +
       staleRecovery +
+      revisionRecovery +
     '</div>';
   };
 
@@ -844,9 +926,14 @@ async function prdItemEdit(source, itemId) {
   let completionHtml = '';
   const isDone = item.status === 'done';
   const isFailed = item.status === 'failed';
-  const isActive = item.status === 'in-progress' || item.status === 'missing';
-
-  if (isDone || isFailed || isActive) {
+  const isActive = prdItemIsActive(item);
+  // `missing` is the server's projection for "not materialized, or materialized
+  // but still queued" — it must render as its truthful not-started state, never
+  // as an In Progress completion panel.
+  const isNotStarted = prdItemIsNotStarted(item);
+  const isQueued = isNotStarted && prdWorkItemIsActive(wi);
+
+  if (isDone || isFailed || isActive || isNotStarted) {
     const agent = wi?.dispatched_to || completedEntry?.agent || '';
     const completedAt = wi?.completedAt || completedEntry?.completed_at || '';
     const summary = completedEntry?.resultSummary || '';
@@ -857,8 +944,8 @@ async function prdItemEdit(source, itemId) {
         : '<code>' + escHtml(pr.id) + '</code>';
     }).join(', ');
 
-    const statusColor = isDone ? 'var(--green)' : isFailed ? 'var(--red)' : 'var(--blue)';
-    const statusLabel = isDone ? 'Completed' : isFailed ? 'Failed' : 'In Progress';
+    const statusColor = isDone ? 'var(--green)' : isFailed ? 'var(--red)' : isActive ? 'var(--blue)' : 'var(--muted)';
+    const statusLabel = isDone ? 'Completed' : isFailed ? 'Failed' : isActive ? 'In Progress' : isQueued ? 'Queued' : 'Not started';
 
     completionHtml = '<div style="background:var(--surface);border:1px solid var(--border);border-left:3px solid ' + statusColor + ';border-radius:4px;padding:10px 12px;margin-bottom:12px">' +
       '<div style="display:flex;align-items:center;gap:8px;margin-bottom:6px">' +
@@ -1034,6 +1121,28 @@ function prdResumeWithoutRegen(prdFile) {
     'Plan resumed (PRD unchanged)');
 }
 
+// W-msa0mrus00mh466f — recovery for a failed revision: requeue the SAME
+// revision work item rather than enqueuing a competing operation.
+async function prdRetryRevision(workItemId) {
+  if (!workItemId) return;
+  try {
+    const res = await fetch('/api/work-items/retry', {
+      method: 'POST', headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ id: workItemId })
+    });
+    if (res.ok) {
+      showToast('cmd-toast', 'Revision ' + workItemId + ' requeued', true);
+      if (typeof wakeEngine === 'function') wakeEngine();
+      if (typeof refresh === 'function') refresh();
+    } else {
+      const d = await res.json().catch(() => ({}));
+      showToast('cmd-toast', 'Retry failed: ' + (d.error || 'unknown'), false);
+    }
+  } catch (e) {
+    showToast('cmd-toast', 'Error: ' + e.message, false);
+  }
+}
+
 function openArchive(i) {
   const a = archivedPrds[i];
   if (!a) return;
@@ -1109,7 +1218,7 @@ function openArchive(i) {
 
 // "QA this PRD" — one-shot bridge from a completed/approved PRD card to a QA
 // Session. POSTs the PRD file to /api/qa/from-prd (P-b3e9d4a1), which builds the
-// spec via engine/qa-from-prd.js then reuses createSession + queueSetup. On
+// spec via engine/qa/from-prd.js then reuses createSession + queueSetup. On
 // success we navigate to the QA Sessions page so the operator can watch it run;
 // the endpoint returns a clean 400 (never 500) for any unusable PRD, surfaced
 // here as a toast. Mirrors triggerVerify's button-state + error handling.
@@ -1136,4 +1245,4 @@ async function qaThisPrd(file, btn) {
   }
 }
 
-window.MinionsPrd = { renderPrd, renderPrdProgress, openArchivedPrdModal, showArchivedPrdDetail, prdItemEdit, prdItemSave, prdItemRemove, prdItemRequeue, prdItemReopen, prdRegenerate, openArchive, qaThisPrd, _togglePrdDescEdit, _cancelPrdDescEdit, _savePrdDesc };
+window.MinionsPrd = { renderPrd, renderPrdProgress, openArchivedPrdModal, showArchivedPrdDetail, prdItemEdit, prdItemSave, prdItemRemove, prdItemRequeue, prdItemReopen, prdRegenerate, prdRetryRevision, openArchive, qaThisPrd, _togglePrdDescEdit, _cancelPrdDescEdit, _savePrdDesc };