@yemi33/minions 0.1.2423 → 0.1.2425

package/dashboard/js/render-skills.js

@@ -63,7 +63,7 @@ function renderSkills(skills) {
   let html = '<div style="display:flex;gap:4px;margin-bottom:8px;flex-wrap:wrap">';
   for (const t of tabs) {
     const active = _skillsTab === t.key;
-    html += '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px;' +
+    html += '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 8px;' +
       (active ? 'background:var(--green);color:#fff;border-color:var(--green)' : '') +
       '" onclick="_skillsSelectTab(' + _jsArg(t.key) + ')">' + escHtml(t.label) + '</button>';
   }
@@ -73,7 +73,7 @@ function renderSkills(skills) {
   // native locations (~/.claude/skills, ~/.copilot/skills, plugin skills) are
   // only visible to that runtime. The "agent" tab (~/.agents/skills) is the
   // cross-runtime portable bucket and IS visible to every runtime.
-  html += '<div style="font-size:var(--text-xs);font-style:italic;color:var(--muted);margin-bottom:8px;line-height:1.4">' +
+  html += '<div style="font-size:var(--text-sm);font-style:italic;color:var(--muted);margin-bottom:8px;line-height:1.4">' +
     'Skills are reference docs agents read on demand — they are not injected wholesale into prompts. ' +
     'Each tab reflects what the matching runtime would see; runtime-native skills are NOT cross-visible. ' +
     'The agent tab (~/.agents/skills) is the cross-runtime portable bucket — visible to every runtime.' +
@@ -90,10 +90,10 @@ function renderSkills(skills) {
   html += '<div style="display:flex;flex-direction:column;gap:6px">';
   for (const r of page) {
     const meta = _skillMetaOf(r.source);
-    const autoTag = r.autoGenerated ? '<span style="font-size:var(--text-xs);background:rgba(63,185,80,0.15);color:var(--green);padding:1px 4px;border-radius:3px;margin-left:4px">auto</span>' : '';
+    const autoTag = r.autoGenerated ? '<span style="font-size:var(--text-sm);background:rgba(63,185,80,0.15);color:var(--green);padding:1px 4px;border-radius:3px;margin-left:4px">auto</span>' : '';
     html += '<div class="inbox-item" onclick="openSkill(' + _jsArg(r.file) + ',' + _jsArg(r.source || 'claude-code') + ',' + _jsArg(r.dir || '') + ')" style="border-left-color:var(--green)">' +
       '<div class="inbox-name"><span style="color:var(--green);font-weight:700">' + meta.icon + ' ' + escHtml(r.name) + '</span>' + autoTag +
-        '<span style="font-size:var(--text-xs);color:var(--muted);margin-left:auto">' + escHtml(meta.label) + '</span>' +
+        '<span style="font-size:var(--text-sm);color:var(--muted);margin-left:auto">' + escHtml(meta.label) + '</span>' +
       '</div>' +
       (r.description ? '<div class="inbox-preview" style="font-size:var(--text-sm);color:var(--text)">' + escHtml(r.description) + '</div>' : '') +
     '</div>';
@@ -103,11 +103,11 @@ function renderSkills(skills) {
   // Pagination controls
   if (totalPages > 1) {
     html += '<div style="display:flex;align-items:center;gap:8px;margin-top:6px;justify-content:center">';
-    html += '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px" ' +
+    html += '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 8px" ' +
       (_skillsPage === 0 ? 'disabled' : '') +
       ' onclick="_skillsPage--;renderSkills(window._lastSkills)">&laquo;</button>';
     html += '<span style="font-size:var(--text-sm);color:var(--muted)">' + (_skillsPage + 1) + ' / ' + totalPages + '</span>';
-    html += '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:2px 8px" ' +
+    html += '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:2px 8px" ' +
       (_skillsPage >= totalPages - 1 ? 'disabled' : '') +
       ' onclick="_skillsPage++;renderSkills(window._lastSkills)">&raquo;</button>';
     html += '</div>';

package/dashboard/js/render-utils.js

@@ -260,7 +260,7 @@ function _renderJsonObj(obj, state) {
     if (!text || text.length <= 10) return '';
     var truncated = text.length > 3000;
     var displayText = truncated ? text.slice(0, 3000) + '...' : text;
-    return '<div style="background:var(--surface);border-left:2px solid var(--border);padding:2px 8px;margin:0 0 2px 16px;font-size:var(--text-xs);font-family:monospace;color:var(--muted);max-height:160px;overflow-y:auto;white-space:pre-wrap;cursor:pointer" onclick="this.style.maxHeight=this.style.maxHeight===\'160px\'?\'none\':\'160px\'">' + escHtml(displayText) + '</div>';
+    return '<div style="background:var(--surface);border-left:2px solid var(--border);padding:2px 8px;margin:0 0 2px 16px;font-size:var(--text-sm);font-family:monospace;color:var(--muted);max-height:160px;overflow-y:auto;white-space:pre-wrap;cursor:pointer" onclick="this.style.maxHeight=this.style.maxHeight===\'160px\'?\'none\':\'160px\'">' + escHtml(displayText) + '</div>';
   }
   function toolKey(name, input) {
     try { return String(name || 'tool') + '|' + JSON.stringify(input || {}); }
@@ -406,7 +406,7 @@ function renderAgentOutput(text) {
       var msg = trimmed.replace('[human-steering] ', '');
       fragments.push('<div style="align-self:flex-end;background:var(--blue);color:#fff;padding:6px 12px;border-radius:12px 12px 2px 12px;max-width:80%;margin:4px 0;font-size:var(--text-md)">' +
         '<span style="margin-right:4px">❯</span>' + escHtml(msg) +
-        '<div style="font-size:var(--text-xs);opacity:0.7;margin-top:2px">\u2713 Queued</div></div>');
+        '<div style="font-size:var(--text-sm);opacity:0.7;margin-top:2px">\u2713 Queued</div></div>');
       continue;
     }
 
@@ -444,7 +444,7 @@ function renderAgentOutput(text) {
 
     // Stderr
     if (trimmed.startsWith('[stderr]')) {
-      fragments.push('<div style="font-size:var(--text-xs);color:var(--red);font-family:monospace;padding:1px 4px">' + escHtml(trimmed) + '</div>');
+      fragments.push('<div style="font-size:var(--text-sm);color:var(--red);font-family:monospace;padding:1px 4px">' + escHtml(trimmed) + '</div>');
     } else {
       // Plain text fallback
       fragments.push('<div style="font-size:var(--text-sm);color:var(--muted);font-family:monospace;padding:1px 4px">' + escHtml(trimmed) + '</div>');
@@ -550,7 +550,7 @@ function renderPager(opts) {
 function pinButton(pinKey, pinned, source, opts) {
   var extraStyle = (opts && opts.extraStyle) || '';
   return '<button class="pr-pager-btn pin-btn' + (pinned ? ' pinned' : '') +
-    '" style="font-size:var(--text-xs);padding:2px 8px;' + extraStyle +
+    '" style="font-size:var(--text-sm);padding:2px 8px;' + extraStyle +
     '" data-pin-key="' + escHtml(pinKey) +
     '" onclick="event.stopPropagation();_togglePinAndRefresh(this.dataset.pinKey,\'' + source + '\')">' +
     (pinned ? 'Unpin' : 'Pin') + '</button>';

package/dashboard/js/render-watches.js

@@ -1,20 +1,8 @@
 // render-watches.js — Watch rendering and CRUD functions for the dashboard
 
-// Pull watches.json straight off disk through /state/engine/watches.json
-// instead of waiting on the /api/status outer cache (issue #2949). The
-// engine writes watch status / _history transitions directly into
-// watches.json, so mtime+size ETag surfaces them on the next 4 s poll.
-// Falls back to data.watches when /state/ returns null.
-async function fetchWatchesFromDisk() {
-  try {
-    const r = await fetch('/state/engine/watches.json');
-    if (!r.ok) return null;
-    const arr = await r.json();
-    return Array.isArray(arr) ? arr : null;
-  } catch {
-    return null;
-  }
-}
+// Watch loading lives in dashboard/shared/watches-source.js
+// (MinionsWatchesSource) so the classic and Slim surfaces resolve watches from
+// the same SQL-backed endpoint. See refresh.js for the polling call site.
 
 // ─── Helpers ────────────────────────────────────────────────────────────────
 
@@ -256,7 +244,7 @@ function renderWatches(watchesData, opts) {
     // Pause/Resume button (F7: marks as `disabled` while POST in flight)
     var pauseBusyCls = _watchToggleInFlight.has(w.id) ? ' disabled' : '';
     if (w.status === 'active') {
-      html += '<button class="pr-pager-btn' + pauseBusyCls + '" style="font-size:var(--text-xs);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',true)" title="Pause">&#x23F8;</button>';
+      html += '<button class="pr-pager-btn' + pauseBusyCls + '" style="font-size:var(--text-sm);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',true)" title="Pause">&#x23F8;</button>';
     } else if (w.status === 'paused') {
       html += '<button class="btn-action' + pauseBusyCls + '" style="margin-right:4px" onclick="event.stopPropagation();toggleWatchPause(\'' + escHtml(w.id) + '\',false)" title="Resume">&#x25B6;</button>';
     }

package/dashboard/js/render-work-items.js

@@ -4,28 +4,6 @@ let allWorkItems = [];
 let wiPage = 0;
 const WI_PER_PAGE = 20;
 
-// Engine-enrichment field names that live on the /api/status workItems slice
-// but are NOT on disk in projects/<name>/work-items.json. The /state/ fetch
-// recovers raw disk state — these get overlaid from the most recent
-// /api/status snapshot (window._lastWorkItems) so the table still renders the
-// PR chip, pending-reason badge, blockedBy hint, etc. The disk fields win
-// for everything else, so a status flip from `pending` → `done` lands within
-// one /state poll regardless of how stale /api/status got.
-const _WI_ENRICHMENT_FIELDS = [
-  '_pr', '_prUrl', '_pendingReason', '_skipReason', '_blockedBy',
-  '_humanFeedback', '_reopened', '_managedSpawnPartial', '_securityFlag',
-  '_artifacts', 'referencesCount', 'acceptanceCriteriaCount',
-  // W-mq5xg5e9000nec0e — slim shape marker so the detail/edit modals know
-  // to hydrate the full description via GET /api/work-items/<id> rather than
-  // pre-filling textareas with the truncated snippet.
-  '_descriptionTruncated',
-  // W-mq08kuog001110a6 — _preDispatchEval IS persisted to disk by
-  // engine/dispatch.js#_persistInvalidWorkItem, but the slim /api/status
-  // overlay may drop it on later passes. Carry it across the overlay so
-  // the Needs-Attention badge survives /state polls.
-  '_preDispatchEval',
-];
-
 // W-mq08kuog001110a6 — Needs-Attention badge for stuck pending work items.
 //
 // A pending WI is "needs attention" when the engine is intentionally refusing
@@ -40,10 +18,21 @@ const _WI_ENRICHMENT_FIELDS = [
 // Does NOT trigger for transient reasons that auto-clear:
 //   * cooldown / retry_cooldown — wait it out
 //   * already_dispatched — engine reconciles on next tick
+//   * agents_busy — an eligible agent exists but every one is currently busy;
+//       the engine auto-dispatches on a later tick when a slot frees. This is
+//       the transient sibling of no_agent (W-mrzlzgju0002e6f0): the engine
+//       classifies the two at the source via resolveAgentReservation, so the
+//       UI never has to guess whether a "no agent" pending state is actionable.
 //   * branch_locked — wait for the holding dispatch
+//   * worktree_branch_held — configured checkout is re-checked each tick
 //   * dependency_unmet — auto-clears when upstream dep finishes (engine discovery loop + DEPENDENCY_MET watch)
 //   * live_checkout_busy — one live-mode dispatch at a time; clears when running dispatch finishes
 //
+// no_agent (kept as a trigger) now means the engine found NO eligible agent
+// that could EVER take this work (empty/over-budget agent roster or an
+// unresolvable route) — a genuine operator-actionable state. Transient
+// capacity exhaustion is emitted as agents_busy instead.
+//
 // Returns { kind, short, full } when the item needs attention, else null.
 // Pure helper (no DOM / escapeHtml deps) so it can be unit-tested in Node.
 function needsAttentionInfo(item) {
@@ -75,7 +64,8 @@ function needsAttentionInfo(item) {
     return { kind: 'pr_not_found', short: prMsg, full: prMsg };
   }
   if (reason === 'no_agent') {
-    var noAgentMsg = 'No agent assigned and routing.md has no match for type=' + (item.type || 'unknown') + '.';
+    var noAgentMsg = 'No eligible agent can take this work (type=' + (item.type || 'unknown') +
+      '). The agent roster is empty, over budget, or routing.md has no resolvable match — assign or configure an agent.';
     return { kind: 'no_agent', short: noAgentMsg, full: noAgentMsg };
   }
   if (reason === 'budget_exceeded') {
@@ -86,63 +76,27 @@ function needsAttentionInfo(item) {
   return null;
 }
 
-// Pull each project's work-items.json straight off disk through
-// /state/projects/<name>/work-items.json. The endpoint is a static-file
-// passthrough with mtime+size ETag — there is no server-side cache in the
-// way, so an engine write to `status: 'done'` is visible to the next poll
-// (issue #2949 root fix). Enrichment fields are overlaid from a cached
-// id→item map sourced from window._lastWorkItems (the last /api/status
-// snapshot). All fetches run in parallel; a single project's failure does
-// not block the others.
-async function fetchWorkItemsFromDisk(projects, cachedById) {
-  if (!Array.isArray(projects) || !projects.length) return null;
-  const results = await Promise.all(projects.map(async (p) => {
-    const name = p && (p.name || p.path);
-    if (!name) return [];
-    try {
-      const r = await fetch('/state/projects/' + encodeURIComponent(name) + '/work-items.json');
-      if (!r.ok) return [];
-      const arr = await r.json();
-      if (!Array.isArray(arr)) return [];
-      return arr.map((item) => {
-        if (!item || typeof item !== 'object') return null;
-        const stamped = Object.assign({}, item, { _source: name });
-        const cached = cachedById && cachedById.get(item.id);
-        if (cached) {
-          for (const k of _WI_ENRICHMENT_FIELDS) {
-            if (stamped[k] === undefined && cached[k] !== undefined) {
-              stamped[k] = cached[k];
-            }
-          }
-        }
-        // referencesCount / acceptanceCriteriaCount don't live on disk;
-        // derive from the raw arrays if present so the table icons render
-        // even before the cached snapshot catches up.
-        if (stamped.referencesCount == null && Array.isArray(stamped.references)) {
-          stamped.referencesCount = stamped.references.length;
-        }
-        if (stamped.acceptanceCriteriaCount == null && Array.isArray(stamped.acceptanceCriteria)) {
-          stamped.acceptanceCriteriaCount = stamped.acceptanceCriteria.length;
-        }
-        return stamped;
-      }).filter(Boolean);
-    } catch {
-      return [];
-    }
-  }));
-  // Flatten + dedupe by id (a project rename mid-flight could double-list).
-  const out = [];
-  const seen = new Set();
-  for (const arr of results) {
-    for (const item of arr) {
-      if (!item.id || seen.has(item.id)) continue;
-      seen.add(item.id);
-      out.push(item);
-    }
-  }
-  return out;
+function formatPendingReason(item, detailed) {
+  var reason = item && item._pendingReason ? String(item._pendingReason) : '';
+  if (!reason) return '';
+  if (reason === 'already_dispatched') return 'Queued — waiting for available agent slot';
+  if (reason === 'agents_busy') return 'Waiting for an available agent — all eligible agents are busy';
+  if (reason !== 'worktree_branch_held') return reason.replace(/_/g, ' ');
+  if (!detailed) return 'operator checkout holds branch';
+
+  var diagnostic = item._worktreeBranchHeld || {};
+  var branch = diagnostic.branch || item.branch || '(unknown branch)';
+  var holderPath = diagnostic.holderPath || '(unknown checkout path)';
+  return 'Branch ' + branch + ' is checked out by configured operator checkout ' + holderPath +
+    '. The work item remains pending and will dispatch automatically once that checkout releases the branch.';
 }
 
+// Work items are served fresh by GET /api/work-items (see refresh.js), which
+// re-runs queries.getWorkItems() server-side so _pr/_prUrl/_pendingReason/
+// _preDispatchEval/_descriptionTruncated are always current. The per-project
+// work-items.json mirrors this file used to read were retired by the SQL-only
+// cutover, and the client-side enrichment overlay they fed is gone with them.
+
 // Track retry state per work item so loading/success/error survives re-renders
 const _wiRetryState = {}; // { [id]: { status: 'pending'|'done'|'error', message?, until? } }
 function setWiRetryState(id, state) { _wiRetryState[id] = state; }
@@ -156,15 +110,15 @@ function getWiRetryState(id) {
 function wiRetryBtn(item) {
   const rs = getWiRetryState(item.id);
   if (rs && rs.status === 'pending') {
-    return '<span style="font-size:var(--text-xs);padding:1px 6px;color:var(--yellow);border:1px solid rgba(210,153,34,0.35);background:rgba(210,153,34,0.1);border-radius:3px;cursor:wait;margin-left:4px">Retrying\u2026</span>';
+    return '<span style="font-size:var(--text-sm);padding:1px 6px;color:var(--yellow);border:1px solid rgba(210,153,34,0.35);background:rgba(210,153,34,0.1);border-radius:3px;cursor:wait;margin-left:4px">Retrying\u2026</span>';
   }
   if (rs && rs.status === 'done') {
-    return '<span style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border:1px solid rgba(63,185,80,0.35);background:rgba(63,185,80,0.1);border-radius:3px;margin-left:4px">Requeued</span>';
+    return '<span style="font-size:var(--text-sm);padding:1px 6px;color:var(--green);border:1px solid rgba(63,185,80,0.35);background:rgba(63,185,80,0.1);border-radius:3px;margin-left:4px">Requeued</span>';
   }
   if (rs && rs.status === 'error') {
-    return '<span style="font-size:var(--text-xs);padding:1px 6px;color:var(--red);border:1px solid rgba(248,81,73,0.35);background:rgba(248,81,73,0.1);border-radius:3px;margin-left:4px;cursor:pointer" title="' + escapeHtml(rs.message || 'Retry failed') + ' — click to try again" onclick="event.stopPropagation();retryWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')">Retry failed</span>';
+    return '<span style="font-size:var(--text-sm);padding:1px 6px;color:var(--red);border:1px solid rgba(248,81,73,0.35);background:rgba(248,81,73,0.1);border-radius:3px;margin-left:4px;cursor:pointer" title="' + escapeHtml(rs.message || 'Retry failed') + ' — click to try again" onclick="event.stopPropagation();retryWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')">Retry failed</span>';
   }
-  return '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-left:4px" onclick="event.stopPropagation();retryWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')">Retry</button>';
+  return '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:1px 6px;color:var(--yellow);border-color:var(--yellow);margin-left:4px" onclick="event.stopPropagation();retryWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')">Retry</button>';
 }
 
 function wiRow(item) {
@@ -177,13 +131,13 @@ function wiRow(item) {
   const prLink = item._pr
     ? '<a class="pr-title" href="' + escapeHtml(item._prUrl || '#') + '" target="_blank" style="font-size:var(--text-sm)">' + escapeHtml(item._pr) + '</a>'
     : (item.branchStrategy === 'shared-branch' && item.status === 'done')
-      ? '<span style="font-size:var(--text-xs);color:var(--muted)" title="Part of shared branch — aggregate PR created at verify stage">shared branch</span>'
-      : '<span style="color:var(--muted)">—</span>';
+      ? '<span style="font-size:var(--text-sm);color:var(--muted)" title="Part of shared branch — aggregate PR created at verify stage">shared branch</span>'
+      : (item.failReason ? '' : '<span style="color:var(--muted)">—</span>');
   // W-mqrdn6qq / W-mrkqc0pl — ALL failure/error snippets (retryable AND
   // non-retryable) surface their red failReason snippet in the PR column,
   // never the Agent column.
   var failSnippet = item.failReason
-    ? '<span style="display:block;font-size:var(--text-xs);color:var(--red)" title="' + escapeHtml(item.failReason) + '">' + escapeHtml(item.failReason.slice(0, 30)) + '</span>'
+    ? '<span style="display:block;font-size:var(--text-sm);color:var(--red)" title="' + escapeHtml(item.failReason) + '">' + escapeHtml(item.failReason.slice(0, 30)) + '</span>'
     : '';
   // PR follow-up chip (W-mpej3cox00099466) — surfaced when the WI was spun
   // off from a PR comment via meta.pr_followup. Links to the parent PR.
@@ -193,16 +147,16 @@ function wiRow(item) {
     var prRef = prFollowup.parent_pr_id || prFollowup.parent_pr_url;
     var prNumMatch = String(prRef).match(/(\d+)(?!.*\d)/);
     var prLabel = prNumMatch ? ('PR #' + prNumMatch[1]) : 'parent PR';
-    followupChip = ' <a class="pr-badge draft" style="font-size:var(--text-xs);text-decoration:none" target="_blank" rel="noopener" href="' + escapeHtml(prFollowup.parent_pr_url) + '" title="Follow-up dispatched from ' + escapeHtml(prRef) + (prFollowup.parent_comment_author ? ' by ' + escapeHtml(prFollowup.parent_comment_author) : '') + '" onclick="event.stopPropagation()">&#8617; from ' + escapeHtml(prLabel) + '</a>';
+    followupChip = ' <a class="pr-badge draft" style="font-size:var(--text-sm);text-decoration:none" target="_blank" rel="noopener" href="' + escapeHtml(prFollowup.parent_pr_url) + '" title="Follow-up dispatched from ' + escapeHtml(prRef) + (prFollowup.parent_comment_author ? ' by ' + escapeHtml(prFollowup.parent_comment_author) : '') + '" onclick="event.stopPropagation()">&#8617; from ' + escapeHtml(prLabel) + '</a>';
   }
   return '<tr data-wi-id="' + escapeHtml(item.id) + '" style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openWorkItemDetail(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')">' +
     '<td style="min-width:280px;max-width:320px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap" title="' + escapeHtml((item.title || '').slice(0, 200)) + '">' + escapeHtml(item.title || '') + followupChip + '</td>' +
     '<td><span style="font-size:var(--text-sm);color:var(--muted)">' + escapeHtml(item._source || '') + '</span>' +
-      (item.scope === 'fan-out' ? ' <span class="pr-badge ' + (item.status === 'done' || item.status === 'failed' ? 'draft' : 'building') + '" style="font-size:var(--text-xs)">fan-out</span>' : '') + '</td>' +
+      (item.scope === 'fan-out' ? ' <span class="pr-badge ' + (item.status === 'done' || item.status === 'failed' ? 'draft' : 'building') + '" style="font-size:var(--text-sm)">fan-out</span>' : '') + '</td>' +
     '<td>' + typeBadge(item.type) + '</td>' +
     '<td>' + priBadge(item.priority) + '</td>' +
     '<td>' + statusBadge(item.status || 'pending') +
-      (item._reopened ? ' <span style="font-size:var(--text-xs);color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') +
+      (item._reopened ? ' <span style="font-size:var(--text-sm);color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') +
       (function() {
         // W-mq08kuog001110a6 — Needs-Attention badge for stuck pending WIs.
         // Click anywhere on the row to open the modal where the full reason
@@ -210,12 +164,12 @@ function wiRow(item) {
         var info = needsAttentionInfo(item);
         if (!info) return '';
         var tooltip = (info.short || '').replace(/\s+/g, ' ').slice(0, 120);
-        return ' <span class="pr-badge needs-attention" style="font-size:var(--text-xs);margin-left:4px" title="' + escapeHtml(tooltip) + ' — click row for full reason">&#x26A0; Needs attention</span>';
+        return ' <span class="pr-badge needs-attention" style="font-size:var(--text-sm);margin-left:4px" title="' + escapeHtml(tooltip) + ' — click row for full reason">&#x26A0; Needs attention</span>';
       })() +
-      (item._pendingReason && item.status === 'pending' && item._pendingReason !== 'already_dispatched' ? ' <span style="font-size:var(--text-xs);color:var(--muted);margin-left:4px" title="Pending reason: ' + escapeHtml(item._pendingReason) + '">' + escapeHtml(item._pendingReason.replace(/_/g, ' ')) + '</span>' : '') +
-      (item._pendingReason === 'already_dispatched' && item.status === 'pending' ? ' <span style="font-size:var(--text-xs);color:var(--blue);margin-left:4px" title="In dispatch queue, waiting to be assigned">queued</span>' : '') +
+      (item._pendingReason && item.status === 'pending' && item._pendingReason !== 'already_dispatched' ? ' <span style="font-size:var(--text-sm);color:var(--muted);margin-left:4px" title="Pending reason: ' + escapeHtml(formatPendingReason(item, true)) + '">' + escapeHtml(formatPendingReason(item, false)) + '</span>' : '') +
+      (item._pendingReason === 'already_dispatched' && item.status === 'pending' ? ' <span style="font-size:var(--text-sm);color:var(--blue);margin-left:4px" title="In dispatch queue, waiting to be assigned">queued</span>' : '') +
       (item._managedSpawnPartial && Array.isArray(item._managedSpawnPartial.failed) && item._managedSpawnPartial.failed.length
-        ? ' <span style="font-size:var(--text-xs);color:var(--yellow);margin-left:4px;border:1px solid var(--yellow);padding:0 4px;border-radius:6px" title="managed-spawn: '
+        ? ' <span style="font-size:var(--text-sm);color:var(--yellow);margin-left:4px;border:1px solid var(--yellow);padding:0 4px;border-radius:6px" title="managed-spawn: '
             + escapeHtml(((item._managedSpawnPartial.healthy || []).length) + '/' + (((item._managedSpawnPartial.healthy || []).length) + item._managedSpawnPartial.failed.length))
             + ' healthy — failed: '
             + escapeHtml(item._managedSpawnPartial.failed.map(function(f){ return (f && f.name) || '?'; }).join(', '))
@@ -223,7 +177,7 @@ function wiRow(item) {
             + escapeHtml(((item._managedSpawnPartial.healthy || []).length) + '/' + (((item._managedSpawnPartial.healthy || []).length) + item._managedSpawnPartial.failed.length))
             + ' healthy</span>'
         : '') +
-      (item._skipReason && item.status === 'pending' && item._skipReason !== 'live_checkout_busy' ? ' <span style="font-size:var(--text-xs);color:var(--yellow);margin-left:4px" title="Dispatch blocked: ' + escapeHtml(item._skipReason) + (item._blockedBy ? ' (by ' + escapeHtml(item._blockedBy) + ')' : '') + '">' + escapeHtml(item._skipReason.replace(/_/g, ' ')) + (item._blockedBy ? ' <span style="color:var(--muted)">(' + escapeHtml(item._blockedBy) + ')</span>' : '') + '</span>' : '') +
+      (item._skipReason && item.status === 'pending' && item._skipReason !== 'live_checkout_busy' ? ' <span style="font-size:var(--text-sm);color:var(--yellow);margin-left:4px" title="Dispatch blocked: ' + escapeHtml(item._skipReason) + (item._blockedBy ? ' (by ' + escapeHtml(item._blockedBy) + ')' : '') + '">' + escapeHtml(item._skipReason.replace(/_/g, ' ')) + (item._blockedBy ? ' <span style="color:var(--muted)">(' + escapeHtml(item._blockedBy) + ')</span>' : '') + '</span>' : '') +
       (item.status === 'failed' ? ' ' + wiRetryBtn(item) : '') +
     '</td>' +
     '<td>' +
@@ -233,7 +187,7 @@ function wiRow(item) {
     '</td>' +
     '<td>' + prLink + (failSnippet ? failSnippet : '') + '</td>' +
     '<td><span class="pr-date">' + escapeHtml((item.created || '').slice(0, 16).replace('T', ' ')) + '</span></td>' +
-    '<td style="white-space:nowrap;font-size:var(--text-xs);color:var(--muted)">' +
+    '<td style="white-space:nowrap;font-size:var(--text-sm);color:var(--muted)">' +
       (function() {
         // /api/status slims the full arrays into integer counts
         // (dashboard.js:1796-1797: referencesCount / acceptanceCriteriaCount).
@@ -249,15 +203,15 @@ function wiRow(item) {
                (acCount ? '<span title="' + acCount + ' acceptance criteria">&#x2611;' + acCount + '</span>' : '');
       })() +
     '</td>' +
-    '<td style="white-space:nowrap">' +
+    '<td style="white-space:nowrap;text-align:right">' +
       // W-mqqa76pi — "View live output" opens the agent slide-in panel (live-output
       // tab auto-selected when the agent is working). Only for genuinely active
       // dispatches with a resolvable assigned agent id.
-      ((item.status === 'dispatched' && (item.dispatched_to || item.agent)) ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--green);border-color:var(--green);margin-right:4px" onclick="event.stopPropagation();openArtifact(\'agent\',\'' + escapeHtml(item.dispatched_to || item.agent) + '\')" title="View live output">&#x1F4FA;</button>' : '') +
-      ((item.status === 'pending' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--blue);border-color:var(--blue);margin-right:4px" onclick="event.stopPropagation();editWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Edit work item">&#x270E;</button>' : '') +
-      ((item.status === 'done' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--muted);border-color:var(--border);margin-right:4px" onclick="event.stopPropagation();archiveWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Archive work item">&#x1F4E6;</button>' : '') +
-      ((item.status === 'done' || item.status === 'failed') && !item._humanFeedback ? '<button class="btn-action" style="margin-right:4px" onclick="event.stopPropagation();feedbackWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Give feedback">&#x1F44D;&#x1F44E;</button>' : (item._humanFeedback ? '<span style="font-size:var(--text-xs)" title="Feedback given">' + (item._humanFeedback.rating === 'up' ? '&#x1F44D;' : '&#x1F44E;') + '</span> ' : '')) +
-      ((item.status === 'pending' || item.status === 'dispatched' || item.status === 'queued' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-xs);padding:1px 6px;color:var(--orange);border-color:var(--orange);margin-right:4px" onclick="event.stopPropagation();cancelWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Cancel work item and kill agent">&#x1F6AB;</button>' : '') +
+      ((item.status === 'dispatched' && (item.dispatched_to || item.agent)) ? '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:1px 6px;color:var(--green);border-color:var(--green);margin-right:4px" onclick="event.stopPropagation();openArtifact(\'agent\',\'' + escapeHtml(item.dispatched_to || item.agent) + '\')" title="View live output">&#x1F4FA;</button>' : '') +
+      ((item.status === 'pending' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:1px 6px;color:var(--blue);border-color:var(--blue);margin-right:4px" onclick="event.stopPropagation();editWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Edit work item">&#x270E;</button>' : '') +
+      ((item.status === 'done' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:1px 6px;color:var(--muted);border-color:var(--muted);margin-right:4px" onclick="event.stopPropagation();archiveWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Archive work item">&#x1F4E6;</button>' : '') +
+      ((item.status === 'done' || item.status === 'failed') && !item._humanFeedback ? '<button class="btn-action" style="margin-right:4px" onclick="event.stopPropagation();feedbackWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Give feedback">&#x1F44D;&#x1F44E;</button>' : (item._humanFeedback ? '<span style="font-size:var(--text-sm)" title="Feedback given">' + (item._humanFeedback.rating === 'up' ? '&#x1F44D;' : '&#x1F44E;') + '</span> ' : '')) +
+      ((item.status === 'pending' || item.status === 'dispatched' || item.status === 'queued' || item.status === 'failed') ? '<button class="pr-pager-btn" style="font-size:var(--text-sm);padding:1px 6px;color:var(--orange);border-color:var(--orange);margin-right:4px" onclick="event.stopPropagation();cancelWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Cancel work item and kill agent">&#x1F6AB;</button>' : '') +
       '<button class="btn-destructive" onclick="event.stopPropagation();deleteWorkItem(\'' + escapeHtml(item.id) + '\',\'' + escapeHtml(item._source || '') + '\')" title="Delete work item and kill agent">&#x2715;</button>' +
     '</td>' +
   '</tr>';
@@ -705,6 +659,24 @@ function _wiDeriveCanonicalPrIdFromUrl(url) {
   return null;
 }
 
+// W-ms0ytfdd000d5a26 — provenance hint for the WI detail modal's Model field.
+// Explains WHERE the shown model came from so an operator can tell a recorded
+// fact from a projection. Mirrors the `source` values produced by
+// engine/execution-model.js#resolveWorkItemModel.
+function _wiModelSourceHint(m) {
+  var runtime = m.runtime || 'runtime';
+  var hint;
+  if (m.source === 'runtime-reported') hint = 'reported by ' + runtime;
+  else if (m.source === 'requested') hint = 'sent to ' + runtime + ' at dispatch';
+  else if (m.source === 'runtime-default') hint = 'no model pinned';
+  else if (m.source === 'configured') hint = 'pinned in current config';
+  else hint = 'no runtime recorded';
+  // Nothing was persisted for this item, so the value is what the NEXT dispatch
+  // would use — not a record of a past run.
+  if (!m.persisted) hint += ' · projected (no dispatch record)';
+  return hint;
+}
+
 function _wiRenderDetail(item) {
   const field = (label, value) => value ? '<div style="margin-bottom:8px"><span style="color:var(--muted);font-size:var(--text-sm);text-transform:uppercase;letter-spacing:0.5px">' + label + '</span><div style="margin-top:2px">' + value + '</div></div>' : '';
   const badge = (cls, text) => '<span class="pr-badge ' + cls + '">' + escapeHtml(text) + '</span>';
@@ -713,14 +685,14 @@ function _wiRenderDetail(item) {
   let html = '<div style="display:flex;flex-direction:column;gap:4px;font-size:var(--text-lg)">';
   html += '<div style="display:flex;gap:8px;align-items:center;margin-bottom:8px">' +
     badge(statusCls, item.status || 'pending') +
-    (item._reopened ? ' <span style="font-size:var(--text-xs);color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') + ' ' +
+    (item._reopened ? ' <span style="font-size:var(--text-sm);color:var(--purple);margin-left:4px" title="This item was reopened from a previously completed state">reopened</span>' : '') + ' ' +
     '<span class="dispatch-type ' + (item.type || 'implement') + '">' + escapeHtml(item.type || 'implement') + '</span>' +
     '<span class="prd-item-priority ' + (item.priority || '') + '">' + escapeHtml(item.priority || 'medium') + '</span>' +
     (function() {
       // W-mq08kuog001110a6 — Needs-Attention chip in the detail modal header.
       var info = needsAttentionInfo(item);
       if (!info) return '';
-      return ' <span class="pr-badge needs-attention" style="font-size:var(--text-xs);margin-left:4px" title="See &quot;Why this is blocked&quot; below">&#x26A0; Needs attention</span>';
+      return ' <span class="pr-badge needs-attention" style="font-size:var(--text-sm);margin-left:4px" title="See &quot;Why this is blocked&quot; below">&#x26A0; Needs attention</span>';
     })() +
     '</div>';
   // W-mqqa76pi — Prominent "View live output" button for an actively-dispatched
@@ -764,6 +736,22 @@ function _wiRenderDetail(item) {
   }
   html += field('Description', '<div id="wi-detail-desc" style="font-size:var(--text-md);max-height:320px;overflow-y:auto;padding:8px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:var(--radius-sm)">' + _descHtml + '</div>');
   html += field('Agent', escapeHtml(item.dispatched_to || item.agent || 'Auto'));
+  // W-ms0ytfdd000d5a26 — the model this work item actually ran on. Resolved
+  // server-side (engine/queries.js#resolveWorkItemModelInfo) so persisted
+  // dispatch evidence — what the runtime reported, else what Minions requested
+  // — always beats current config. Items with no dispatch record show the
+  // projection from current config, marked as such. When nothing concrete is
+  // known we render "<runtime> default" rather than blanking or guessing.
+  (function() {
+    var m = item._model;
+    if (!m || typeof m !== 'object') return;
+    var label = m.label || m.model;
+    if (!label) return;
+    var hint = _wiModelSourceHint(m);
+    html += field('Model',
+      '<code style="font-size:var(--text-sm);background:var(--surface2);padding:2px 6px;border-radius:var(--radius-sm)">' + escapeHtml(String(label)) + '</code>' +
+      (hint ? ' <span style="color:var(--muted);font-size:var(--text-sm)">' + escapeHtml(hint) + '</span>' : ''));
+  })();
   html += field('Project', escapeHtml(item._source || 'central'));
   // P-714ef144 — surface meta.workdir in the detail modal so operators can
   // confirm the agent's actual cwd. Renders nothing when unset (which is
@@ -791,7 +779,7 @@ function _wiRenderDetail(item) {
       (item.cancelledAt ? '<div style="margin-top:2px;color:var(--muted);font-size:var(--text-sm)">' + escapeHtml(formatLocalDateTime(item.cancelledAt)) + '</div>' : '');
     html += field('Cancellation Reason', _cancelBody);
   }
-  if (item._pendingReason && item.status === 'pending') html += field('Pending Reason', item._pendingReason === 'already_dispatched' ? 'Queued — waiting for available agent slot' : escapeHtml(item._pendingReason.replace(/_/g, ' ')));
+  if (item._pendingReason && item.status === 'pending') html += field('Pending Reason', escapeHtml(formatPendingReason(item, true)));
   if (item._managedSpawnPartial && Array.isArray(item._managedSpawnPartial.failed) && item._managedSpawnPartial.failed.length) {
     var _msp = item._managedSpawnPartial;
     var _mspHealthy = Array.isArray(_msp.healthy) ? _msp.healthy : [];
@@ -899,8 +887,8 @@ function _wiRenderDetail(item) {
   // Artifacts — branch (non-navigation) + navigation chips routed through
   // renderArtifactLink so chip click → openArtifact() → modal-stack push +
   // URL hash + browser-Back integration come for free (P-e265cd31).
-  // NOTE (issue #603): item._source is the WI's *project name* (stamped by
-  // fetchWorkItemsFromDisk above / used by every action button to route
+  // NOTE (issue #603): item._source is the WI's *project name* (stamped
+  // server-side by queries.getWorkItems / used by every action button to route
   // retry/cancel/delete/etc.), NOT a plan/PRD file id. A prior "Source" chip
   // here treated item._source as if it held a genuine source-plan/PRD
   // filename, which rendered a bogus chip labeled with the project name on
@@ -1189,9 +1177,9 @@ function _wiInjectSourceChipFromFrontmatter(content, containerId) {
 }
 
 if (typeof window !== 'undefined') {
-  window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, submitFeedback, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput, openInboxNote, needsAttentionInfo };
+  window.MinionsWork = { wiRow, renderWorkItems, editWorkItem, submitWorkItemEdit, deleteWorkItem, archiveWorkItem, toggleWorkItemArchive, retryWorkItem, wiPrev, wiNext, feedbackWorkItem, openCreateWorkItemModal, openWorkItemDetail, openAllWorkItems, viewAgentOutput, openInboxNote, needsAttentionInfo };
 }
 // exported for testing — pure helpers safe to require under Node (no DOM deps).
 if (typeof module !== 'undefined' && module.exports) {
-  module.exports = { needsAttentionInfo };
+  module.exports = { needsAttentionInfo, formatPendingReason };
 }