@yemi33/minions 0.1.2195 → 0.1.2196

package/dashboard/js/modal.js

@@ -1,10 +1,22 @@
 // modal.js — Modal and notification badge functions extracted from dashboard.html
 
 function closeModal() {
+  // P-ce1e5e47 — Esc / X / back-button all route through popModalFrame(),
+  // which itself calls history.back(); the popstate listener then pops one
+  // breadcrumb and restores the previous modal. When the stack still has
+  // depth > 1, the modal stays open with the parent view restored, so we
+  // must NOT tear down Q&A / edit / pin state here — only when the modal is
+  // actually about to be hidden (depth <= 1).
+  var depth = (typeof _modalStackDepth === 'function') ? _modalStackDepth() : 0;
+  if (depth > 1) {
+    if (typeof popModalFrame === 'function') popModalFrame();
+    return;
+  }
   const modalEl = document.querySelector('#modal .modal');
   if (modalEl) modalEl.classList.remove('modal-wide');
   document.getElementById('modal').classList.remove('open');
-  clearModalBackStack();
+  if (typeof popModalFrame === 'function') popModalFrame();
+  else if (typeof clearModalBackStack === 'function') clearModalBackStack();
   // Hide Q&A section (only shown for document modals)
   document.getElementById('modal-qa').style.display = 'none';
   // Remove settings-body marker so the next modal opens with the default
@@ -132,4 +144,26 @@ function renderArchiveButtons(archives) {
   ).join(' ');
 }
 
+// P-3ed68b1e — Esc key closes the active modal layer. Routes through
+// closeModal() so the depth-aware logic (pop one frame vs full teardown) is
+// the single source of truth. Bubble phase + early-out when the QA review
+// modal is on top so qa.js's own Escape handler isn't shadowed. Inputs and
+// textareas inside the modal are exempt so typing Esc inside Q&A / steering
+// inputs still reaches qa.js / cancel handlers (those listeners use
+// stopPropagation; this guard is belt-and-braces for any future inputs).
+if (typeof document !== 'undefined' && typeof document.addEventListener === 'function') {
+  document.addEventListener('keydown', function(ev) {
+    if (ev.key !== 'Escape' && ev.keyCode !== 27) return;
+    if (typeof _qaModalState !== 'undefined' && _qaModalState) return;
+    var modalEl = document.getElementById('modal');
+    if (!modalEl || !modalEl.classList.contains('open')) return;
+    var t = ev.target;
+    if (t && t.tagName) {
+      var tag = String(t.tagName).toUpperCase();
+      if (tag === 'INPUT' || tag === 'TEXTAREA' || t.isContentEditable) return;
+    }
+    closeModal();
+  });
+}
+
 window.MinionsModal = { closeModal, copyModalContent, showNotifBadge, clearNotifBadge, restoreNotifBadges, findCardForFile, renderArchiveButtons };

package/dashboard/js/refresh.js

@@ -153,7 +153,7 @@ const RENDER_VERSIONS = {
   mcpServers: 1,
   harnessDiag: 1,
   schedules: 1,
-  watches: 2,
+  watches: 3,
   meetings: 1,
   pipelines: 1,
   pinned: 1,

package/dashboard/js/render-agents.js

@@ -71,6 +71,24 @@ function renderAgents(agents) {
   restoreDashboardScrollState(grid, scrollState);
 }
 
+/**
+ * Open the right-side agent detail panel.
+ *
+ * P-30b6cf8a — Detail-panel exception. This panel is intentionally NOT a
+ * modal-stack frame; it is a peer of the stack. Routing comes through
+ * openArtifact('agent', id) in dashboard/js/render-utils.js, which:
+ *   1. Calls resetModalStack() + _physicallyCloseModal() so the modal-stack
+ *      identity goes back to empty (any prior modal is dismissed).
+ *   2. Calls openAgentDetail(id), which opens this slide-in panel.
+ *
+ * Reverse direction is symmetric: clicking a WI/PR/plan chip from inside
+ * the panel calls openArtifact('<type>', id), which closes this panel and
+ * starts a fresh modal stack. This is the one documented exception to
+ * in-stack chip behaviour — do NOT refactor it into a normal modal frame.
+ * The panel UI (tabs, live-output stream, charter editor) does not fit
+ * the single-shell modal contract; trying to force it into pushModalFrame
+ * would lose the tabbed layout and the persistent live-stream socket.
+ */
 async function openAgentDetail(id) {
   const agent = agentData.find(a => a.id === id);
   if (!agent) return;

package/dashboard/js/render-kb.js

@@ -251,8 +251,33 @@ async function kbOpenItem(category, file) {
     const display = content.replace(/^---[\s\S]*?---\n*/m, '');
     document.getElementById('modal-title').textContent = file;
     const modalBody = document.getElementById('modal-body');
-    // eslint-disable-next-line no-unsanitized/property -- reason: renderMd() escapes all user-controlled fields before assembling HTML (see dashboard/js/utils.js)
-    modalBody.innerHTML = renderMd(display);
+    // P-34fa5d79 — KB modal renders "From note: …" and "From WI: …" chips
+    // when the entry was promoted via /api/inbox/promote-kb (which stamps
+    // source_note + source_wi). Parse the frontmatter block off the raw
+    // fetched content (we already strip it for `display`).
+    var _fmChipsHtml = '';
+    var _fmMatch = String(content || '').match(/^---\n([\s\S]*?)\n---/);
+    if (_fmMatch) {
+      var _fm = {};
+      _fmMatch[1].split('\n').forEach(function(line) {
+        var lm = line.match(/^([\w-]+):\s*(.*)$/);
+        if (lm) _fm[lm[1]] = lm[2].trim();
+      });
+      var _chips = [];
+      if (_fm.source_note) {
+        _chips.push('<span style="color:var(--muted);font-size:var(--text-sm);text-transform:uppercase;letter-spacing:0.5px;margin-right:4px">From note</span>' +
+          renderArtifactLink({ type: 'note', id: _fm.source_note, label: _fm.source_note.replace(/\.md$/, '').slice(0, 30), title: 'Source note: ' + _fm.source_note }));
+      }
+      if (_fm.source_wi) {
+        _chips.push('<span style="color:var(--muted);font-size:var(--text-sm);text-transform:uppercase;letter-spacing:0.5px;margin-right:4px">From WI</span>' +
+          renderArtifactLink({ type: 'wi', id: _fm.source_wi, label: _fm.source_wi, title: 'Source work item: ' + _fm.source_wi }));
+      }
+      if (_chips.length > 0) {
+        _fmChipsHtml = '<div style="margin-bottom:10px;padding:6px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:var(--radius-sm);display:flex;flex-wrap:wrap;gap:12px;align-items:center">' + _chips.join('') + '</div>';
+      }
+    }
+    // eslint-disable-next-line no-unsanitized/property -- reason: renderMd() escapes all user-controlled fields before assembling HTML (see dashboard/js/utils.js); _fmChipsHtml is built from renderArtifactLink() chips which escape id+label via escapeHtml
+    modalBody.innerHTML = _fmChipsHtml + renderMd(display);
     _modalDocContext = { title: file, content: display, selection: '' };
     _modalFilePath = 'knowledge/' + category + '/' + file; showModalQa();
     // Clear notification badge when opening this document

package/dashboard/js/render-meetings.js

@@ -196,6 +196,30 @@ function _renderMeetingDetail(m) {
         '<span style="font-size:var(--text-sm);color:var(--muted)">' + escHtml(m.createdAt?.slice(0, 16).replace('T', ' ') || '') + '</span>' +
       '</div>';
 
+      // P-30b6cf8a — Linked-artifact chip row. Renders the meeting.linkedPlan
+      // (derived by _findLinkedPlan from conclusion text / title-slug match)
+      // and the participants list as in-stack renderArtifactLink chips.
+      // Plan chip routes through openArtifact -> planView (in-stack modal
+      // push, browser Back returns here). Agent chips route through the
+      // detail-panel exception in openArtifact (resetModalStack + open the
+      // slide-in panel); see render-utils.js openArtifact comment.
+      const _linkedPlanForChip = _findLinkedPlan(m);
+      const _hasChipRenderer = typeof renderArtifactLink === 'function';
+      if (_hasChipRenderer && (_linkedPlanForChip || (m.participants || []).length > 0)) {
+        let chipRow = '<div style="display:flex;gap:6px;flex-wrap:wrap;align-items:center;font-size:var(--text-sm)">';
+        if (_linkedPlanForChip) {
+          chipRow += '<span style="color:var(--muted)">Linked plan:</span> ' +
+            renderArtifactLink({ type: 'plan', id: _linkedPlanForChip.file, label: _linkedPlanForChip.file, title: _linkedPlanForChip.summary || _linkedPlanForChip.file });
+        }
+        if ((m.participants || []).length > 0) {
+          chipRow += (_linkedPlanForChip ? ' <span style="color:var(--border)">·</span> ' : '') +
+            '<span style="color:var(--muted)">Participants:</span> ' +
+            (m.participants || []).map(a => renderArtifactLink({ type: 'agent', id: a, label: a, title: 'Open ' + a + ' detail panel' })).join(' ');
+        }
+        chipRow += '</div>';
+        html += chipRow;
+      }
+
       // Agenda — render markdown-like formatting
       html += '<div style="background:var(--surface2);padding:8px 12px;border-radius:6px;font-size:var(--text-md);white-space:pre-wrap;line-height:1.6">' +
         '<strong>Agenda:</strong>\n' + renderMd(m.agenda || '') + '</div>';
@@ -203,7 +227,13 @@ function _renderMeetingDetail(m) {
       // Per-agent panels
       for (const agent of (m.participants || [])) {
         html += '<div style="border:1px solid var(--border);border-radius:6px;overflow:hidden">';
-        html += '<div style="background:var(--surface2);padding:6px 12px;font-weight:600;font-size:var(--text-md)">' + escHtml(agent) + '</div>';
+        // P-30b6cf8a — agent header is a chip so it routes through the
+        // detail-panel exception in openArtifact (resetModalStack + open
+        // slide-in panel).
+        const _agentChip = _hasChipRenderer
+          ? renderArtifactLink({ type: 'agent', id: agent, label: agent, title: 'Open ' + agent + ' detail panel' })
+          : escHtml(agent);
+        html += '<div style="background:var(--surface2);padding:6px 12px;font-weight:600;font-size:var(--text-md)">' + _agentChip + '</div>';
 
         // Findings
         if (m.findings?.[agent]) {
@@ -322,8 +352,11 @@ function openMeetingDetail(id) {
             if (d.meeting && _meetingPollId === id) {
               const hash = JSON.stringify(d.meeting);
               if (hash === _lastMeetingHash) return; // no change — skip re-render
-              _lastMeetingHash = hash;
-              _renderMeetingDetail(d.meeting);
+              // P-ce1e5e47 — withTopFrame skips the re-render when the user
+              // has stacked another modal on top of this meeting view.
+              const apply = function() { _lastMeetingHash = hash; _renderMeetingDetail(d.meeting); };
+              if (typeof withTopFrame === 'function') withTopFrame('meeting', id, apply);
+              else apply();
             }
           })
           .catch(function() {});

package/dashboard/js/render-pipelines.js

@@ -98,29 +98,34 @@ function _getPipelineDisplayTitle(pipeline) {
 
 /**
  * Render clickable artifact links for a pipeline stage.
- * Each artifact type gets an icon and navigates to the relevant detail view.
+ * Navigation chips (workItems / meetings / plans / prds) route through
+ * renderArtifactLink → openArtifact() so the modal stack push and URL hash
+ * come for free (P-e265cd31). PRs / notes / subStages stay inline: PRs and
+ * notes page-switch instead of opening a detail modal, and subStages are
+ * non-clickable — none has an openArtifact path, so the "View PR" / "View
+ * Note" pills render identically to today.
  * @param {Object} artifacts - artifact map from stage run
- * @param {string} [pipelineId] - pipeline ID for modal back-navigation
+ * @param {string} [pipelineId] - retained for back-compat; unused now that
+ *   openArtifact captures the current visible modal automatically.
  */
-function _renderArtifactLinks(artifacts, pipelineId) {
+function _renderArtifactLinks(artifacts /* , pipelineId — unused since P-e265cd31 */) {
   if (!artifacts) return '';
   var links = [];
   var linkStyle = 'display:inline-flex;align-items:center;gap:2px;padding:1px 6px;border-radius:10px;font-size:var(--text-sm);cursor:pointer;text-decoration:none;color:var(--blue);background:color-mix(in srgb, var(--blue) 10%, transparent);border:1px solid color-mix(in srgb, var(--blue) 20%, transparent)';
 
-  // Pushes current pipeline modal onto back stack so detail modals can navigate back
-  var backFn = pipelineId ? "pushModalBack(function(){openPipelineDetail('" + escHtml(pipelineId) + "')});" : '';
-
   (artifacts.workItems || []).forEach(function(id) {
-    links.push('<span style="' + linkStyle + '" onclick="event.stopPropagation();' + backFn + 'openWorkItemDetail(\'' + escHtml(id) + '\')" title="Open work item ' + escHtml(id) + '">⚙ ' + escHtml(id) + '</span>');
+    links.push(renderArtifactLink({ type: 'wi', id: id, label: id, title: 'Open work item ' + id }));
   });
   (artifacts.meetings || []).forEach(function(id) {
-    links.push('<span style="' + linkStyle + '" onclick="event.stopPropagation();' + backFn + 'openMeetingDetail(\'' + escHtml(id) + '\')" title="Open meeting ' + escHtml(id) + '">💬 ' + escHtml(id) + '</span>');
+    links.push(renderArtifactLink({ type: 'meeting', id: id, label: id, title: 'Open meeting ' + id }));
   });
   (artifacts.plans || []).forEach(function(name) {
-    links.push('<span style="' + linkStyle + '" onclick="event.stopPropagation();' + backFn + 'planView(\'' + escHtml(name) + '\')" title="Plan: ' + escHtml(name) + '">📋 ' + escHtml(name.replace(/\.md$/, '').slice(0, 30)) + '</span>');
+    var label = String(name).replace(/\.md$/, '').slice(0, 30);
+    links.push(renderArtifactLink({ type: 'plan', id: name, label: label, title: 'Plan: ' + name }));
   });
   (artifacts.prds || []).forEach(function(name) {
-    links.push('<span style="' + linkStyle + '" onclick="event.stopPropagation();' + backFn + 'planView(\'' + escHtml(name) + '\')" title="PRD: ' + escHtml(name) + '">📄 ' + escHtml(name.replace(/\.json$/, '').slice(0, 30)) + '</span>');
+    var label = String(name).replace(/\.json$/, '').slice(0, 30);
+    links.push(renderArtifactLink({ type: 'prd', id: name, label: label, title: 'PRD: ' + name }));
   });
   (artifacts.prs || []).forEach(function(id) {
     links.push('<span style="' + linkStyle + '" onclick="event.stopPropagation();closeModal();switchPage(\'prs\')" title="Pull request ' + escHtml(id) + '">🔀 PR-' + escHtml(id) + '</span>');

package/dashboard/js/render-plans.js

@@ -697,11 +697,73 @@ function _renderPlanModal(normalizedFile, raw, lastMod) {
     modalActions += '<button class="pr-pager-btn" style="' + bs + ';color:var(--red)" onclick="planDelete(\'' + escapeHtml(normalizedFile) + '\')">Delete</button>';
   }
 
+  // P-79b47b0c — surface linked artifacts (PRD + verify-guide) as in-stack
+  // renderArtifactLink chips alongside the plan modal action buttons.
+  // P-30b6cf8a — also surface the source-meeting chip when the PRD JSON
+  // carries a `meetingId` field or the .md plan body contains the
+  // `**Source Meeting:** MTG-…` header line (the dashboard convention
+  // written by POST /api/plans/create when a plan is created from a
+  // meeting). Chip routes through openArtifact → openMeetingDetail for
+  // in-stack push (Back returns here).
+  let linkedChips = '';
+  if (typeof renderArtifactLink === 'function') {
+    if (prdFile && prdFile !== normalizedFile) {
+      linkedChips += renderArtifactLink({ type: 'prd', id: prdFile, label: 'PRD', title: prdFile }) + ' ';
+    }
+    let meetingChipId = '';
+    if (normalizedFile.endsWith('.json')) {
+      try {
+        const _parsedForMtg = JSON.parse(raw);
+        if (_parsedForMtg && typeof _parsedForMtg.meetingId === 'string' && /^MTG-[\w-]+$/.test(_parsedForMtg.meetingId)) {
+          meetingChipId = _parsedForMtg.meetingId;
+        }
+      } catch { /* JSON parse already surfaced upstream */ }
+    } else {
+      const _mtgMatch = (raw || '').match(/\*\*Source Meeting:\*\*\s+(MTG-[\w-]+)/);
+      if (_mtgMatch) meetingChipId = _mtgMatch[1];
+    }
+    if (meetingChipId) {
+      linkedChips += renderArtifactLink({ type: 'meeting', id: meetingChipId, label: meetingChipId, title: 'Source meeting: ' + meetingChipId }) + ' ';
+    }
+    const guidesForChip = window._lastVerifyGuides || [];
+    const guideForChip = guidesForChip.find(g => g && g.planFile === (prdFile || normalizedFile));
+    if (guideForChip && guideForChip.file) {
+      // Verify guides live at prd/guides/verify-<slug>.md and are fetched via
+      // /api/plans/<file>, so a plain {type:'plan'} chip resolves correctly
+      // through openArtifact -> planView. openVerifyGuide() is the legacy
+      // helper that does the same thing — we keep the modal-stack semantics
+      // by routing through openArtifact instead.
+      linkedChips += renderArtifactLink({ type: 'plan', id: guideForChip.file, label: 'Verify Guide', icon: '&#x1F4D6;', title: 'Manual testing guide (openVerifyGuide)' }) + ' ';
+    }
+    // P-e6093f70 — cross-repo plan ↔ plan sibling chips. When the plan
+    // fans out across ≥2 projects (lifecycle.checkPlanCompletion creates
+    // one verify WI per touched project — see CLAUDE.md §"Cross-repo
+    // plans" invariant 5), surface a clickable WI chip per sibling so
+    // operators can hop between the per-project verify WIs without
+    // leaving the modal stack. The chip label is the project slug so each
+    // sibling is identifiable at a glance (mirrors _renderVerifyBadge's
+    // projectLabel handling at line ~1042). Gate matches the existing
+    // per-project verify-badge fanout gate at line ~689 so single-project
+    // plans are unaffected.
+    if (modalVerifyWis.length >= 2) {
+      for (const v of modalVerifyWis) {
+        const projLabel = v.project || 'verify';
+        linkedChips += renderArtifactLink({
+          type: 'wi',
+          id: v.id,
+          label: projLabel,
+          title: 'Sibling verify WI for ' + projLabel + ' (cross-repo plan fanout)',
+        }) + ' ';
+      }
+    }
+  }
+
   const lastModLabel = lastMod ? '<div style="font-size:var(--text-sm);color:var(--muted);font-weight:400;margin-top:2px">Last updated: ' + formatLocalDateTime(lastMod) + '</div>' : '';
+  const linkedChipsRow = linkedChips ? '<div style="display:flex;gap:4px;flex-wrap:wrap;margin-top:4px">' + linkedChips + '</div>' : '';
   const actionBtns = '<div style="display:flex;gap:4px;flex-wrap:wrap;margin-top:4px">' + modalActions + '</div>';
 
-  // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escapeHtml() (fields: plan title, version label, action target files)
-  document.getElementById('modal-title').innerHTML = escapeHtml(title) + (versionLabel ? ' <span style="font-size:var(--text-base);font-weight:700;padding:1px 6px;border-radius:3px;background:rgba(56,139,253,0.15);color:var(--blue)">' + escapeHtml(versionLabel) + '</span>' : '') + lastModLabel + actionBtns;
+  // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escapeHtml() (fields: plan title, version label, action target files); linkedChipsRow output is renderArtifactLink chips (escapes internally) or empty
+  document.getElementById('modal-title').innerHTML = escapeHtml(title) + (versionLabel ? ' <span style="font-size:var(--text-base);font-weight:700;padding:1px 6px;border-radius:3px;background:rgba(56,139,253,0.15);color:var(--blue)">' + escapeHtml(versionLabel) + '</span>' : '') + lastModLabel + linkedChipsRow + actionBtns;
   const modalBody = document.getElementById('modal-body');
   const scrollTop = modalBody.scrollTop;
   if (normalizedFile.endsWith('.json')) {
@@ -748,7 +810,15 @@ async function planView(file) {
       }
       fetch('/api/plans/' + encodeURIComponent(normalizedFile))
         .then(function(r) { return r.text().then(function(raw) { return { raw: raw, lastMod: r.headers.get('Last-Modified') }; }); })
-        .then(function(d) { if (_planPollFile === normalizedFile && d.raw !== _planPollLastRaw) { _planPollLastRaw = d.raw; _renderPlanModal(normalizedFile, d.raw, d.lastMod); } })
+        .then(function(d) {
+          // P-ce1e5e47 — withTopFrame guards a stacked modal: if the user
+          // navigated to a deeper modal on top of this plan view, skip the
+          // re-render so we don't clobber the visible top frame.
+          if (_planPollFile !== normalizedFile || d.raw === _planPollLastRaw) return;
+          var apply = function() { _planPollLastRaw = d.raw; _renderPlanModal(normalizedFile, d.raw, d.lastMod); };
+          if (typeof withTopFrame === 'function') withTopFrame('plan', normalizedFile, apply);
+          else apply();
+        })
         .catch(function() {});
     }, 3000);
   } catch (e) { console.error(e); }

package/dashboard/js/render-prd.js

@@ -144,9 +144,16 @@ function renderPrd(prd, prog) {
 
 function _renderPrLink(pr, opts) {
   var size = (opts && opts.size) || '10px';
-  var statusColor = pr.status === 'merged' ? 'var(--green)' : pr.status === 'abandoned' ? 'var(--red)' : 'var(--blue)';
   var statusIcon = pr.status === 'merged' ? '✓' : pr.status === 'abandoned' ? '✗' : '○';
-  return '<a href="' + escHtml(pr.url || '#') + '" target="_blank" rel="noopener" onclick="event.stopPropagation()" style="font-size:' + size + ';color:' + statusColor + ';text-decoration:underline;cursor:pointer;margin-left:4px" title="' + escHtml((pr.title || '') + ' (' + (pr.status || 'active') + ')') + '">' + statusIcon + ' ' + escHtml(pr.id) + '</a>';
+  // P-79b47b0c — render as an in-stack modal chip via renderArtifactLink
+  // (was a raw <a target="_blank">). The status icon stays as a non-link
+  // prefix so the chip body remains the canonical id, matching the WI/PRD
+  // chip pattern. Click is gated by openArtifact -> openPrDetail.
+  var chip = (typeof renderArtifactLink === 'function')
+    ? renderArtifactLink({ type: 'pr', id: pr.id, label: pr.id, title: (pr.title || '') + ' (' + (pr.status || 'active') + ')' })
+    : '<code>' + escHtml(pr.id) + '</code>';
+  return '<span style="font-size:' + size + ';margin-left:4px;display:inline-flex;align-items:center;gap:3px">' +
+    '<span title="' + escHtml(pr.status || 'active') + '">' + statusIcon + '</span>' + chip + '</span>';
 }
 
 function _renderPrdWorkItemIdBadge(workItemId, prdItemId, size, padding) {
@@ -288,9 +295,19 @@ function renderPrdProgress(prog) {
       ? '<span onclick="event.stopPropagation();prdItemReopen(\'' + escHtml(i.source || '') + '\',\'' + escHtml(i.id) + '\')" style="color:var(--blue);cursor:pointer;font-size:var(--text-xs);padding:1px 5px;background:rgba(56,139,253,0.1);border:1px solid rgba(56,139,253,0.3);border-radius:3px" title="Re-open: set to updated so engine re-dispatches on existing branch">re-open</span>'
       : '';
 
+    // P-79b47b0c — replace the bare prd-item-id span with a renderArtifactLink
+    // chip pointing at the backing WI (when one exists). The chip's onclick
+    // calls event.stopPropagation, so the parent row's prdItemEdit() still
+    // fires only on row-body clicks; clicking the chip opens the WI detail
+    // modal in-stack. When there's no backing WI, fall back to the legacy
+    // bare-id span (clicking the row still opens prdItemEdit).
+    const idChip = (wi && typeof renderArtifactLink === 'function')
+      ? renderArtifactLink({ type: 'wi', id: wi.id, label: i.id, title: i.name || wi.title || i.id })
+      : '<span class="prd-item-id">' + escHtml(i.id) + '</span>';
+
     return '<div class="prd-item-row st-' + (i.status || 'missing') + '" style="flex-wrap:wrap;cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;prdItemEdit(\'' + src + '\',\'' + iid + '\')">' +
       statusBadge(i.status, i.id) +
-      '<span class="prd-item-id">' + escHtml(i.id) + '</span>' +
+      idChip +
       '<span class="prd-item-name" title="' + escHtml(i.name) + '">' + escHtml(i.name) + '</span>' +
       wiLabel +
       agentLabel +
@@ -600,9 +617,13 @@ function renderPrdProgress(prog) {
       html += '<div style="font-size:var(--text-sm);font-weight:600;color:var(--blue);margin-bottom:4px">E2E Aggregate PRs</div>';
       html += prs.map(pr => {
         const statusColor = pr.status === 'active' ? 'var(--green)' : pr.status === 'merged' ? 'var(--purple)' : 'var(--muted)';
+        // P-79b47b0c — render PR id as in-stack chip (was raw <a target="_blank">).
+        const prChip = (typeof renderArtifactLink === 'function')
+          ? renderArtifactLink({ type: 'pr', id: pr.id, label: pr.id, title: pr.title || pr.id })
+          : '<code>' + escHtml(pr.id) + '</code>';
         return '<div style="display:flex;align-items:center;gap:6px;padding:2px 0;font-size:var(--text-base)">' +
           '<span style="color:' + statusColor + ';font-size:var(--text-xs);font-weight:600;padding:1px 4px;border:1px solid;border-radius:3px">' + escHtml(pr.status || 'active') + '</span>' +
-          '<a href="' + escHtml(pr.url || '#') + '" target="_blank" rel="noopener" style="color:var(--blue);text-decoration:underline;font-weight:500">' + escHtml(pr.id) + '</a>' +
+          prChip +
           '<span style="color:var(--text);flex:1;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">' + escHtml(pr.title || '') + '</span>' +
           '<span style="color:var(--muted);font-size:var(--text-xs)">' + escHtml(pr._project || '') + '</span>' +
         '</div>';
@@ -791,7 +812,10 @@ async function prdItemEdit(source, itemId) {
     const completedAt = wi?.completedAt || completedEntry?.completed_at || '';
     const summary = completedEntry?.resultSummary || '';
     const prLinks = (item.prs || []).map(function(pr) {
-      return '<a href="' + escHtml(pr.url || '#') + '" target="_blank" rel="noopener" style="color:var(--green);text-decoration:underline">' + escHtml(pr.id) + '</a>';
+      // P-79b47b0c — render as in-stack chip (was raw <a target="_blank">).
+      return (typeof renderArtifactLink === 'function')
+        ? renderArtifactLink({ type: 'pr', id: pr.id, label: pr.id, title: pr.title || pr.id })
+        : '<code>' + escHtml(pr.id) + '</code>';
     }).join(', ');
 
     const statusColor = isDone ? 'var(--green)' : isFailed ? 'var(--red)' : 'var(--blue)';

package/dashboard/js/render-prs.js

@@ -204,7 +204,8 @@ function prRow(pr) {
     '<td><span class="pr-badge ' + statusClass + '" title="' + escapeHtml(statusLabel) + '">' + escapeHtml(statusLabel) + '</span></td>' +
     '<td>' + (observeBtn || '<span style="color:var(--muted);font-size:var(--text-base)">—</span>') + '</td>' +
     '<td><span class="pr-date" title="' + escapeHtml(createdLabel) + '">' + escapeHtml(createdLabel) + '</span></td>' +
-    '<td><button class="btn-destructive" style="padding:1px 5px" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
+    '<td><button class="pr-info-pill" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();openPrDetail(this.dataset.prId)" title="Open PR detail (in-stack modal)">&#x2197;</button>' +
+      '<button class="btn-destructive" style="padding:1px 5px;margin-left:4px" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
     '</tr>';
 }
 
@@ -224,7 +225,7 @@ const PRS_COLGROUP =
     '<col style="width:110px">' +   // Status
     '<col style="width:100px">' +   // Observe
     '<col style="width:130px">' +   // Created
-    '<col style="width:50px">' +    // Actions
+    '<col style="width:80px">' +    // Actions (info-pill + remove)
   '</colgroup>';
 
 function prTableHtml(rows) {
@@ -300,6 +301,176 @@ function openAllPrs() {
   document.getElementById('modal').classList.add('open');
 }
 
+// P-79b47b0c — in-stack PR detail modal. Routed from openArtifact('pr', id)
+// in render-utils.js and from the PR-row info-pill in prRow(). Read-only:
+// renders title, status/review/build badges, reviewers, linked-WI chips
+// (joined via item._pr === pr.id from queries enrichment), description, and
+// an "Open on GitHub" secondary button. No POSTs from this modal.
+function openPrDetail(prId) {
+  if (window.getSelection && window.getSelection().toString().length > 0) return;
+  if (!prId) return;
+  // Render an immediate skeleton from the in-memory PR cache (allPrs) so the
+  // modal feels instant. Then hit GET /api/prs/<id> for the canonical record
+  // and re-render via withTopFrame() to respect modal-stack semantics.
+  const cached = (allPrs || []).find(p => p && p.id === prId) ||
+                 ((window._lastPrs) || []).find(p => p && p.id === prId);
+  document.getElementById('modal-title').textContent = (cached && (cached.title || cached.id)) || prId;
+  // eslint-disable-next-line no-unsanitized/property -- reason: _renderPrDetail() escapes all user fields (title, description, agent, reviewers, branch) via escapeHtml() / renderMd() / renderArtifactLink() before assembly
+  document.getElementById('modal-body').innerHTML = _renderPrDetail(cached || { id: prId, _loading: true });
+  document.getElementById('modal-body').style.fontFamily = "'Segoe UI', system-ui, sans-serif";
+  document.getElementById('modal-body').style.whiteSpace = 'normal';
+  document.getElementById('modal').classList.add('open');
+
+  fetch('/api/prs/' + encodeURIComponent(prId))
+    .then(function(r) { if (!r.ok) throw new Error('HTTP ' + r.status); return r.json(); })
+    .then(function(data) {
+      const pr = data && data.pr;
+      if (!pr) return;
+      const apply = function() {
+        document.getElementById('modal-title').textContent = pr.title || pr.id || prId;
+        // eslint-disable-next-line no-unsanitized/property -- reason: _renderPrDetail() escapes all user fields (title, description, agent, reviewers, branch) via escapeHtml() / renderMd() / renderArtifactLink() before assembly
+        document.getElementById('modal-body').innerHTML = _renderPrDetail(pr);
+      };
+      if (typeof withTopFrame === 'function') withTopFrame('pr', prId, apply);
+      else apply();
+    })
+    .catch(function() {
+      const body = document.getElementById('modal-body');
+      if (body) {
+        body.innerHTML = '<p style="color:var(--red)">Failed to load PR ' + escapeHtml(String(prId)) + '.</p>';
+      }
+    });
+}
+
+function _renderPrDetail(pr) {
+  if (!pr) return '<p style="color:var(--muted)">No data.</p>';
+  if (pr._loading) {
+    return '<p style="color:var(--muted)">Loading PR ' + escapeHtml(String(pr.id || '')) + '...</p>';
+  }
+  const sq = pr.minionsReview || {};
+  const reviewLabel = sq.status === 'waiting' ? 'reviewing (minions)' : sq.status ? sq.status + ' (minions)' : (pr.reviewStatus || 'pending');
+  const reviewClass = (sq.status || pr.reviewStatus) === 'approved' ? 'approved' :
+                      ((sq.status || pr.reviewStatus) === 'changes-requested' || (sq.status || pr.reviewStatus) === 'rejected') ? 'rejected' :
+                      (sq.status || pr.reviewStatus) === 'waiting' ? 'building' : 'draft';
+  const buildLabel = pr.buildStatus || 'none';
+  const buildClass = pr._buildStatusStale ? 'build-stale' :
+                     pr.buildStatus === 'passing' ? 'build-pass' :
+                     pr.buildStatus === 'failing' ? 'build-fail' :
+                     pr.buildStatus === 'running' ? 'building' : 'no-build';
+  const statusLabel = pr.status || 'active';
+  const statusClass = statusLabel === 'merged' ? 'pr-merged' :
+                      statusLabel === 'abandoned' ? 'pr-abandoned' :
+                      statusLabel === 'draft' ? 'draft' : 'pr-active';
+
+  const badges =
+    '<span class="pr-badge ' + statusClass + '" title="status">' + escapeHtml(String(statusLabel)) + '</span> ' +
+    '<span class="pr-badge ' + reviewClass + '" title="review">' + escapeHtml(String(reviewLabel)) + '</span> ' +
+    '<span class="pr-badge ' + buildClass + '" title="build">' + escapeHtml(String(buildLabel)) + '</span>';
+
+  // Linked WIs — join via item._pr === pr.id (stamped by engine/queries.js).
+  const wis = (window._lastWorkItems) || [];
+  const linkedWis = wis.filter(w => w && w._pr === pr.id);
+  let linkedChips = '';
+  if (linkedWis.length && typeof renderArtifactLink === 'function') {
+    linkedChips = '<div style="margin-top:10px"><strong style="color:var(--muted);font-size:var(--text-base)">Linked Work Items:</strong><br>' +
+      linkedWis.map(w => renderArtifactLink({ type: 'wi', id: w.id, label: w.id, title: w.title || w.id })).join(' ') +
+      '</div>';
+  }
+
+  // P-e6093f70 — PR ↔ PR linkage. Surface (a) a "Parent PR (follow-up)"
+  // chip when this PR record carries meta.pr_followup.parent_pr_url /
+  // parent_pr_id (mirrors the WI-side rendering at render-work-items.js
+  // _wiRenderDetail, P-79b47b0c) and (b) one chip per pr.depends_on[]
+  // entry. Both surfaces use renderArtifactLink({type:'pr'}) so click
+  // opens the parent / dep PR modal in-stack with browser-Back support.
+  //
+  // Graceful degradation: when parent_pr_url cannot be parsed into a
+  // canonical id (unknown host), fall back to a plain external <a>.
+  // depends_on chips consult the local PR cache (allPrs / window._lastPrs
+  // / window._lastPullRequests) and pass deleted:true when the dep is
+  // unknown so the chip renders struck-out (non-clickable) instead of
+  // silently linking to a 404.
+  let parentChip = '';
+  const followup = pr.meta && pr.meta.pr_followup;
+  if (followup && (followup.parent_pr_id || followup.parent_pr_url) && typeof renderArtifactLink === 'function') {
+    let parentPrId = followup.parent_pr_id || '';
+    if (!parentPrId && followup.parent_pr_url && typeof _wiDeriveCanonicalPrIdFromUrl === 'function') {
+      parentPrId = _wiDeriveCanonicalPrIdFromUrl(followup.parent_pr_url) || '';
+    }
+    if (parentPrId) {
+      parentChip = '<div style="margin-top:10px"><strong style="color:var(--muted);font-size:var(--text-base)">Parent PR (follow-up):</strong> ' +
+        renderArtifactLink({ type: 'pr', id: parentPrId, label: parentPrId, title: 'Follow-up from ' + (followup.parent_pr_url || parentPrId) }) +
+        '</div>';
+    } else if (followup.parent_pr_url) {
+      parentChip = '<div style="margin-top:10px"><strong style="color:var(--muted);font-size:var(--text-base)">Parent PR (follow-up):</strong> ' +
+        '<a href="' + escapeHtml(followup.parent_pr_url) + '" target="_blank" rel="noopener" style="color:var(--blue)">' + escapeHtml(followup.parent_pr_url) + '</a>' +
+        '</div>';
+    }
+  }
+
+  let dependsChips = '';
+  if (Array.isArray(pr.depends_on) && pr.depends_on.length && typeof renderArtifactLink === 'function') {
+    const knownPrIds = new Set();
+    const knownSources = [
+      (typeof allPrs !== 'undefined' && Array.isArray(allPrs)) ? allPrs : null,
+      window._lastPrs,
+      window._lastPullRequests,
+    ];
+    for (const src of knownSources) {
+      if (Array.isArray(src)) for (const p of src) { if (p && p.id) knownPrIds.add(p.id); }
+    }
+    dependsChips = '<div style="margin-top:10px"><strong style="color:var(--muted);font-size:var(--text-base)">Depends On:</strong> ' +
+      pr.depends_on.map(function(d) {
+        const depId = String(d);
+        const isDeleted = knownPrIds.size > 0 && !knownPrIds.has(depId);
+        return renderArtifactLink({
+          type: 'pr',
+          id: depId,
+          label: depId,
+          title: isDeleted ? 'Dependency PR ' + depId + ' is no longer in local state' : 'Depends on ' + depId,
+          deleted: isDeleted,
+        });
+      }).join(' ') +
+      '</div>';
+  }
+
+  // Reviewers list
+  let reviewers = '';
+  if (Array.isArray(pr.reviewers) && pr.reviewers.length) {
+    reviewers = '<div style="margin-top:6px"><strong style="color:var(--muted);font-size:var(--text-base)">Reviewers:</strong> ' +
+      pr.reviewers.map(r => '<code>' + escapeHtml(String(typeof r === 'string' ? r : (r.displayName || r.name || ''))) + '</code>').join(' ') +
+      '</div>';
+  }
+
+  const agent = pr.agent ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Agent:</strong> ' + escapeHtml(String(pr.agent)) + '</div>' : '';
+  const branch = pr.branch ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Branch:</strong> <code>' + escapeHtml(String(pr.branch)) + '</code></div>' : '';
+  const created = pr.created ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Created:</strong> ' + escapeHtml(String(pr.created)) + '</div>' : '';
+  const project = pr._project ? '<div><strong style="color:var(--muted);font-size:var(--text-base)">Project:</strong> ' + escapeHtml(String(pr._project)) + '</div>' : '';
+
+  const descSafe = typeof renderMd === 'function' ? renderMd(pr.description || '') : escapeHtml(pr.description || '');
+  const descBlock = pr.description ?
+    '<div style="margin-top:14px"><strong style="color:var(--muted);font-size:var(--text-base)">Description:</strong><div style="margin-top:6px;font-size:var(--text-md);line-height:1.6">' + descSafe + '</div></div>' :
+    '';
+
+  const openOnHost = pr.url ?
+    '<a class="btn-secondary" style="display:inline-block;margin-top:14px;padding:6px 12px;text-decoration:none" href="' + escapeHtml(safeUrl(pr.url)) + '" target="_blank" rel="noopener">Open on GitHub &#x2197;</a>' :
+    '';
+
+  return '<div style="font-family:\'Segoe UI\',system-ui,sans-serif">' +
+    '<div style="margin-bottom:10px">' +
+      '<code style="color:var(--muted);font-size:var(--text-base)">' + escapeHtml(String(pr.id || '')) + '</code><br>' +
+      badges +
+    '</div>' +
+    agent + branch + project + created +
+    reviewers +
+    linkedChips +
+    parentChip +
+    dependsChips +
+    descBlock +
+    openOnHost +
+  '</div>';
+}
+
 function openModal(i) {
   const item = inboxData[i];
   if (!item) return;
@@ -490,4 +661,4 @@ async function resumePausedCause(btn) {
   }
 }
 
-window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, unlinkPr, togglePrObserve, resumePausedCause };
+window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, openPrDetail, unlinkPr, togglePrObserve, resumePausedCause };

package/dashboard/js/render-schedules.js

@@ -405,6 +405,18 @@ function openScheduleDetail(id) {
     '<div><strong style="color:var(--muted)">Agent:</strong> ' + escHtml(s.agent || 'auto') + '</div>' +
     '<div><strong style="color:var(--muted)">Status:</strong> ' + enabledLabel + '</div>' +
     '<div><strong style="color:var(--muted)">Last Run:</strong> ' + escHtml(lastRun) + '</div>' +
+    // P-c549d07e — "Recent dispatches" chip row. Each chip routes through
+    // renderArtifactLink({type:'wi'}) → openArtifact() so clicking pushes
+    // the WI detail modal onto the stack. Hard-cap to 5 chips even if the
+    // API returned more (defensive — the API already slices).
+    (Array.isArray(s._recentWorkItemIds) && s._recentWorkItemIds.length > 0
+      ? '<div><strong style="color:var(--muted)">Recent dispatches:</strong>' +
+          '<div style="display:flex;flex-wrap:wrap;gap:4px;margin-top:4px">' +
+            s._recentWorkItemIds.slice(0, 5).map(function(wi) {
+              return renderArtifactLink({ type: 'wi', id: String(wi), label: String(wi), title: 'Open work item ' + wi });
+            }).join('') +
+          '</div></div>'
+      : '') +
     (s.description ? '<div><strong style="color:var(--muted)">Description:</strong><div style="margin-top:4px;padding:8px;background:var(--surface2);border-radius:4px">' + renderMd(s.description) + '</div></div>' : '') +
   '</div>';