@yemi33/minions 0.1.2194 → 0.1.2196

package/dashboard/slim/js/members.js

@@ -69,6 +69,29 @@
     grid.appendChild(frag);
   }
 
+  // Live "Working for" ticker for the agent detail modal. Mirrors the legacy
+  // renderer's .agent-runtime-tick pattern (dashboard/js/render-agents.js): a
+  // data-started element refreshed by a 1s interval, cleared on modal close so
+  // no interval leaks once the popout is dismissed.
+  var _agentDetailTimer = null;
+
+  // Format an elapsed-ms span as 'Xh Ym Zs', dropping the hours segment when 0.
+  function _fmtAgentElapsed(ms) {
+    if (!(ms > 0)) ms = 0;
+    var sec = Math.floor(ms / 1000) % 60, min = Math.floor(ms / 60000) % 60, hr = Math.floor(ms / 3600000);
+    return (hr > 0 ? hr + 'h ' : '') + min + 'm ' + sec + 's';
+  }
+
+  function _tickAgentDetailRuntime() {
+    var el = document.getElementById('slim-agent-working-tick');
+    if (!el) { _stopAgentDetailRuntime(); return; }
+    el.textContent = _fmtAgentElapsed(Date.now() - new Date(el.dataset.started).getTime());
+  }
+
+  function _stopAgentDetailRuntime() {
+    if (_agentDetailTimer) { clearInterval(_agentDetailTimer); _agentDetailTimer = null; }
+  }
+
   // Append a labelled key/value row to the agent detail modal body.
   function appendAgentRow(body, key, value, muted) {
     var row = document.createElement('div');
@@ -92,6 +115,8 @@
     var titleEl = document.getElementById('slim-agent-title');
     if (!a || !modal || !body) return;
     if (titleEl) titleEl.textContent = a.name || a.id;
+    // Clear any ticker left over from a previously-opened agent before rebuild.
+    _stopAgentDetailRuntime();
     body.textContent = '';
 
     // Header: emoji + name + role.
@@ -131,6 +156,24 @@
 
     appendAgentRow(body, 'Last result', a.resultSummary || 'No recent output', !a.resultSummary);
 
+    // Only running agents get a live elapsed-time row; idle/done/error don't.
+    if (a.status === 'working' && a.started_at) {
+      var workRow = document.createElement('div');
+      workRow.className = 'agent-detail-row';
+      var workKey = document.createElement('div');
+      workKey.className = 'agent-detail-key';
+      workKey.textContent = 'Working for';
+      var workVal = document.createElement('div');
+      workVal.className = 'agent-detail-val';
+      workVal.id = 'slim-agent-working-tick';
+      workVal.dataset.started = a.started_at;
+      workRow.appendChild(workKey);
+      workRow.appendChild(workVal);
+      body.appendChild(workRow);
+      _tickAgentDetailRuntime();
+      _agentDetailTimer = setInterval(_tickAgentDetailRuntime, 1000);
+    }
+
     modal.classList.add('open');
   }
 

package/dashboard/slim/js/modals-tiles.js

@@ -12,7 +12,9 @@
       if (ev.key === 'Escape' && modal.classList.contains('open')) close();
     });
   }
-  bindModalClose('slim-agent-modal', 'slim-agent-close');
+  // Stop the agent-detail "Working for" ticker on every dismiss path so the
+  // 1s interval started in openAgentDetail can't leak after the modal closes.
+  bindModalClose('slim-agent-modal', 'slim-agent-close', _stopAgentDetailRuntime);
   bindModalClose('slim-tools-modal', 'slim-tools-close');
   bindModalClose('slim-tile-modal', 'slim-tile-close');
 
@@ -249,9 +251,9 @@
   // Populate + open the cockpit-tile detail modal from the latest status
   // snapshot. Mirrors the corresponding old-dashboard tab for each tile.
   function openTileModal(key) {
-    // The Pinned-context tile has its own list/editor modal (view + edit + unpin)
-    // rather than the read-only tile detail view.
-    if (key === 'pinned') { openSlimPinnedList(); return; }
+    // The Knowledge tile opens the unified Knowledge control panel (Pinned
+    // Context + Notes + KB tabs) rather than the read-only tile detail view.
+    if (key === 'knowledge') { openKnowledgeModal(); return; }
     var view = TILE_VIEWS[key];
     if (!view) return;
     var modal = document.getElementById('slim-tile-modal');

package/dashboard/slim/js/pinned.js

@@ -66,11 +66,11 @@
       var d = await res.json().catch(function() { return {}; });
       if (!res.ok) throw new Error(d.error || ('HTTP ' + res.status));
       closeSlimPinEditor();
-      scheduleStatusRefresh(400);           // refresh the Pinned-context tile
-      // If the list modal is open, re-render it once the snapshot catches up.
+      scheduleStatusRefresh(400);           // refresh the Knowledge tile count
+      // If the Knowledge modal's Pinned tab is showing, re-render it once the
+      // snapshot catches up.
       setTimeout(function() {
-        var listModal = document.getElementById('slim-pinned-modal');
-        if (listModal && listModal.classList.contains('open')) renderSlimPinnedList();
+        if (document.getElementById('slim-pinned-list')) renderSlimPinnedList();
       }, 700);
     } catch (e) {
       if (msg) { msg.style.color = 'var(--red)'; msg.textContent = 'Error: ' + (e && e.message ? e.message : 'failed'); }
@@ -79,19 +79,10 @@
   }
 
   // ── List / view / unpin ──────────────────────────────────────────
-  function openSlimPinnedList() {
-    var modal = document.getElementById('slim-pinned-modal');
-    if (!modal) return;
-    renderSlimPinnedList();
-    modal.classList.add('open');
-  }
-  function closeSlimPinnedList() {
-    var modal = document.getElementById('slim-pinned-modal');
-    if (modal) modal.classList.remove('open');
-  }
-
+  // Rendered into #slim-pinned-list — the container the Knowledge modal's
+  // Pinned-Context tab builds. No-op when that container isn't mounted.
   function renderSlimPinnedList() {
-    var body = document.getElementById('slim-pinned-body');
+    var body = document.getElementById('slim-pinned-list');
     if (!body) return;
     var entries = slimPinnedEntries();
     body.textContent = '';
@@ -166,13 +157,10 @@
     }
   }
 
-  bindModalClose('slim-pinned-modal', 'slim-pinned-close');
   bindModalClose('slim-pin-edit-modal', 'slim-pin-edit-close');
   (function bindPinnedUi() {
     var pinBtn = document.getElementById('slim-pin-btn');
     if (pinBtn) pinBtn.addEventListener('click', function() { openSlimPinEditor(null); });
-    var addBtn = document.getElementById('slim-pinned-add');
-    if (addBtn) addBtn.addEventListener('click', function() { openSlimPinEditor(null); });
     var tileChip = document.getElementById('slim-tile-pin-chip');
     if (tileChip) tileChip.addEventListener('click', function(ev) { ev.stopPropagation(); openSlimPinEditor(null); });
     var cancel = document.getElementById('slim-pin-cancel');

package/dashboard/slim/js/status.js

@@ -16,11 +16,10 @@
     }, delay || 0);
   }
 
-  // Tiles whose numeric value should fire a one-shot cockpit-flash animation
-  // whenever it changes (W-mqb02bga: status panel). Suppressed on first paint
-  // — only deltas from a previously-seen number trigger the flash, so opening
-  // the dashboard doesn't pulse every tile.
-  var FLASH_ON_CHANGE_KEYS = { dispatches: true, prs: true };
+  // Track each tile's last numeric value so the number element can fade
+  // out-and-back-in whenever it changes (W-mqbaeuhm). Suppressed on first paint
+  // — only deltas from a previously-seen number trigger the fade, so opening
+  // the dashboard doesn't animate every tile.
   var lastTileValues = Object.create(null);
 
   // Apply lit-state and value/detail text to a tile.
@@ -36,17 +35,17 @@
     if (detEl) detEl.textContent = detail || '';
     tile.classList.remove('lit-blue', 'lit-green', 'lit-amber', 'lit-orange', 'lit-red');
     if (lit) tile.classList.add('lit-' + lit);
-    // Flash the tile when its number changes. Remove → force reflow → re-add
-    // restarts the animation even when the class was already present from the
-    // previous tick (otherwise repeated changes would only fire on the first
-    // delta after class application). Only numeric values count as a change;
-    // ignore null/undefined transitions.
-    if (FLASH_ON_CHANGE_KEYS[key] && typeof value === 'number') {
+    // Fade the number out-and-back-in when it changes. Remove → force reflow →
+    // re-add restarts the animation even when the class was already present
+    // from the previous tick (otherwise repeated changes would only fire on the
+    // first delta after class application). Only numeric→different-numeric
+    // counts as a change; ignore null/undefined and unchanged values.
+    if (valEl && typeof value === 'number') {
       var prev = lastTileValues[key];
       if (typeof prev === 'number' && prev !== value) {
-        tile.classList.remove('flash-change');
-        void tile.offsetWidth;
-        tile.classList.add('flash-change');
+        valEl.classList.remove('value-changed');
+        void valEl.offsetWidth;
+        valEl.classList.add('value-changed');
       }
       lastTileValues[key] = value;
     }
@@ -150,14 +149,11 @@
       activeWatches.length ? 'blue' : null
     );
 
-    // ── Pinned context tile ────────────────────────────────────
+    // ── Knowledge tile (Pinned Context + Notes + KB) ──────────────
+    // Pinned count is live from the poll; notes/KB counts are fetched lazily
+    // (see knowledge.js loadKnowledgeCounts) to keep them off the 5s poll.
     var pinned = Array.isArray(data.pinned) ? data.pinned : [];
-    updateTile(
-      'pinned',
-      pinned.length,
-      pinned.length ? (pinned.length === 1 ? '1 note for agents' : pinned.length + ' notes for agents') : 'nothing pinned',
-      pinned.length ? 'blue' : null
-    );
+    if (typeof renderKnowledgeTile === 'function') renderKnowledgeTile(pinned.length);
 
     // ── Team member cards ──────────────────────────────────────
     renderMembers(Array.isArray(data.agents) ? data.agents : []);

package/dashboard/slim/styles.css

@@ -701,7 +701,7 @@
     .tile-section-label { font-weight: 700; letter-spacing: 0.3px; text-transform: uppercase; color: var(--text); }
     .tile-section-count { color: var(--muted); }
 
-    /* Pinned-context list rows (slim-pinned-modal). */
+    /* Pinned-context list rows (Knowledge modal → Pinned Context tab). */
     .pinned-row {
       border: 1px solid var(--border);
       border-radius: var(--radius);
@@ -724,6 +724,85 @@
     .pinned-row-actions .btn-secondary { padding: 4px 12px; font-size: var(--text-base); }
     .pinned-row-unpin { color: var(--red); border-color: var(--red); }
 
+    /* Knowledge control panel (slim-knowledge-modal): Pinned Context / Notes /
+       KB tabs in one box. Reuses .tile-* and .pinned-row primitives. */
+    .slim-knowledge-modal-inner { width: 720px; max-width: calc(100vw - 32px); }
+    .kn-tabs { display: flex; gap: 6px; margin-left: 16px; }
+    .kn-tab {
+      border: 1px solid var(--border);
+      background: var(--surface2);
+      color: var(--muted);
+      font-size: var(--text-sm);
+      padding: 4px 12px;
+      border-radius: 999px;
+      cursor: pointer;
+      white-space: nowrap;
+    }
+    .kn-tab:hover { color: var(--text); border-color: var(--blue); }
+    .kn-tab.active {
+      color: var(--text);
+      border-color: var(--blue);
+      background: color-mix(in srgb, var(--blue) 14%, var(--surface2));
+    }
+    .kn-toolbar { display: flex; align-items: center; gap: 8px; margin-bottom: 10px; }
+    .kn-toolbar-end { justify-content: flex-end; }
+    .kn-section-head {
+      margin: 16px 0 8px;
+      font-size: var(--text-md);
+      font-weight: 600;
+      color: var(--text);
+      border-top: 1px solid var(--border);
+      padding-top: 12px;
+    }
+    .kn-msg { font-size: var(--text-base); min-height: 16px; color: var(--muted); }
+    .kn-msg-inline { margin-right: auto; }
+    .kn-notes-textarea { font-family: var(--mono, monospace); white-space: pre; }
+    .kn-create-form {
+      border: 1px solid var(--border);
+      border-radius: var(--radius);
+      background: var(--surface2);
+      padding: 12px;
+      margin-bottom: 12px;
+    }
+    /* KB list rows */
+    .kb-row {
+      border: 1px solid var(--border);
+      border-radius: var(--radius);
+      background: var(--surface2);
+      padding: 8px 10px;
+      margin-bottom: 8px;
+      cursor: pointer;
+    }
+    .kb-row:hover { border-color: var(--blue); }
+    .kb-row-pinned { border-color: var(--amber); }
+    .kb-row-top { display: flex; align-items: center; gap: 8px; }
+    .kb-row-title {
+      flex: 1; min-width: 0;
+      font-weight: 600; font-size: var(--text-md); color: var(--text);
+      overflow: hidden; text-overflow: ellipsis; white-space: nowrap;
+    }
+    .kb-pin-btn {
+      flex: 0 0 auto;
+      border: none; background: transparent; cursor: pointer;
+      color: var(--muted); font-size: var(--text-lg); line-height: 1; padding: 0 2px;
+    }
+    .kb-pin-btn.on { color: var(--amber); }
+    .kb-pin-btn:hover { color: var(--amber); }
+    .kb-row-meta { font-size: var(--text-base); color: var(--muted); margin-top: 3px; }
+    .kb-row-preview {
+      margin-top: 5px;
+      font-size: var(--text-base); color: var(--muted); line-height: 1.5;
+      word-break: break-word;
+    }
+    .kb-entry-content {
+      white-space: pre-wrap; word-break: break-word;
+      font-family: var(--mono, monospace); font-size: var(--text-base);
+      color: var(--text); line-height: 1.55;
+      background: var(--surface2); border: 1px solid var(--border);
+      border-radius: var(--radius); padding: 12px; margin: 0;
+      max-height: 60vh; overflow-y: auto;
+    }
+
     .chat-thinking {
       color: var(--muted);
       font-size: var(--text-md);
@@ -893,9 +972,9 @@
     .cockpit-grid > [data-tile="queued"],
     .cockpit-grid > [data-tile="dispatches"],
     .cockpit-grid > [data-tile="prs"] { grid-column: span 2; }
-    /* Row 3: Watches | Pinned — two equal columns (3 of 6 each). */
+    /* Row 3: Watches | Knowledge — two equal columns (3 of 6 each). */
     .cockpit-grid > [data-tile="watches"],
-    .cockpit-grid > [data-tile="pinned"] { grid-column: span 3; }
+    .cockpit-grid > [data-tile="knowledge"] { grid-column: span 3; }
     .cockpit-tile {
       background: var(--surface2);
       border: 1px solid var(--border);
@@ -905,37 +984,26 @@
       flex-direction: column;
       gap: 4px;
       transition: border-color 0.2s, background 0.2s;
-      /* Per-state flash color (rgb triplet, no rgba wrapper) consumed by
-         `cockpit-flash` so the change-flash ring matches the tile's lit state
-         (amber for working dispatches, orange for queued, red for failing
-         PRs, etc.). Each lit-* class below overrides this. Default is blue
-         for unlit / generic tiles. */
-      --flash-color: 88, 166, 255;
     }
     .cockpit-tile.lit-blue {
       border-color: rgba(88, 166, 255, 0.6);
       background: rgba(88, 166, 255, 0.06);
-      --flash-color: 88, 166, 255;
     }
     .cockpit-tile.lit-green {
       border-color: rgba(63, 185, 80, 0.6);
       background: rgba(63, 185, 80, 0.06);
-      --flash-color: 63, 185, 80;
     }
     .cockpit-tile.lit-amber {
       border-color: rgba(210, 153, 34, 0.6);
       background: rgba(210, 153, 34, 0.06);
-      --flash-color: 210, 153, 34;
     }
     .cockpit-tile.lit-orange {
       border-color: rgba(234, 88, 12, 0.6);
       background: rgba(234, 88, 12, 0.08);
-      --flash-color: 234, 88, 12;
     }
     .cockpit-tile.lit-red {
       border-color: rgba(248, 81, 73, 0.6);
       background: rgba(248, 81, 73, 0.08);
-      --flash-color: 248, 81, 73;
     }
     .cockpit-label {
       font-size: var(--text-base);
@@ -998,32 +1066,22 @@
     .cockpit-tile[data-tile="dispatches"].lit-amber {
       animation: slim-pulse 1.6s ease-in-out infinite;
     }
-    /* Change-flash: brief one-shot ring around a tile whose value just
-       changed, used by status.js for the `dispatches` and `prs` tiles. Layered
-       on a pseudo-element so it composes cleanly with `slim-pulse` above
-       (running both on the tile itself would force a single `animation:` rule
-       and clobber one of them). The ring expands then fades — fully visible
-       even when the underlying tile is already lit. Respects
+    /* Value-fade: when a cockpit tile's number changes, status.js toggles
+       `.value-changed` on the `.cockpit-value` element to fade the number out
+       then back in around the freshly-swapped text. Applies to every numeric
+       tile (engine/dispatches/prs/watches/pinned). Respects
        prefers-reduced-motion. */
-    .cockpit-tile::after {
-      content: '';
-      position: absolute;
-      inset: -1px;
-      border-radius: inherit;
-      pointer-events: none;
-      box-shadow: 0 0 0 0 rgba(var(--flash-color), 0);
-    }
-    .cockpit-tile.flash-change::after {
-      animation: cockpit-flash 0.75s ease-out;
-    }
-    @keyframes cockpit-flash {
-      0%   { box-shadow: 0 0 0 0 rgba(var(--flash-color), 0); }
-      30%  { box-shadow: 0 0 0 6px rgba(var(--flash-color), 0.7); }
-      100% { box-shadow: 0 0 0 0 rgba(var(--flash-color), 0); }
+    .cockpit-value.value-changed {
+      animation: cockpit-value-fade 0.45s ease;
+    }
+    @keyframes cockpit-value-fade {
+      0%   { opacity: 1; transform: translateY(0); }
+      45%  { opacity: 0; transform: translateY(-2px); }
+      100% { opacity: 1; transform: translateY(0); }
     }
     @media (prefers-reduced-motion: reduce) {
       .cockpit-tile[data-tile="dispatches"].lit-amber { animation: none; }
-      .cockpit-tile.flash-change::after { animation: none; }
+      .cockpit-value.value-changed { animation: none; }
     }
 
     /* ── Status panel sub-divisions: Team cards over System tiles,

package/dashboard/styles.css

@@ -1042,6 +1042,40 @@
   .modal-close { background: none; border: none; color: var(--muted); font-size: var(--text-2xl); cursor: pointer; padding: var(--space-2) var(--space-4); }
   .modal-close:hover { color: var(--text); }
 
+  /* P-ce1e5e47 — modal stack chrome (back button + depth chip). The back
+     button reuses .modal-copy chrome; modal-back-btn just tightens the
+     leading icon. The depth chip is a small badge that shows n/5 when the
+     modal stack has depth > 1. */
+  .modal-back-btn { color: var(--blue); }
+  .modal-back-btn:hover { color: var(--text); border-color: var(--blue); }
+  .modal-stack-chip {
+    display: inline-flex; align-items: center; padding: var(--space-1) var(--space-4);
+    border-radius: 10px; font-size: var(--text-xs); color: var(--muted);
+    background: var(--surface2); border: 1px solid var(--border);
+    font-variant-numeric: tabular-nums;
+  }
+
+  /* P-ce1e5e47 — artifact-link chips. Re-usable inline pill used by chip
+     callsites (renderArtifactLink in P-e265cd31 routes through this class).
+     Hover lifts via border accent. .deleted shows a struck-out non-clickable
+     hint so dead refs still render in context. */
+  .artifact-chip {
+    display: inline-flex; align-items: center; gap: var(--space-1);
+    padding: var(--space-1) var(--space-4); border-radius: 10px;
+    font-size: var(--text-sm); cursor: pointer;
+    background: var(--surface2); border: 1px solid var(--border); color: var(--text);
+    transition: border-color var(--transition-base), color var(--transition-base);
+    user-select: none;
+  }
+  .artifact-chip:hover { border-color: var(--blue); color: var(--blue); }
+  .artifact-chip-icon { font-size: var(--text-sm); line-height: 1; }
+  .artifact-chip-label { font-size: var(--text-sm); }
+  .artifact-chip.deleted {
+    cursor: not-allowed; text-decoration: line-through;
+    color: var(--muted); opacity: 0.6;
+  }
+  .artifact-chip.deleted:hover { border-color: var(--border); color: var(--muted); }
+
   /* Custom confirm dialog (W-mq5dk1lq) — promise-based replacement for
      native window.confirm(). Scaffold in layout.html (#confirm-dialog),
      behavior in dashboard/js/confirm-dialog.js. Reuses .modal-bg / .modal

package/dashboard-build.js

@@ -71,7 +71,7 @@ function buildDashboardHtml() {
 // original single-file source) inside the wrapper that layout.html provides.
 const SLIM_JS_ORDER = [
   'helpers', 'settings', 'link-pr', 'chat', 'projects',
-  'command-send', 'status', 'members', 'modals-tiles', 'history', 'pinned',
+  'command-send', 'status', 'members', 'modals-tiles', 'history', 'pinned', 'knowledge',
 ];
 
 // Cache for the assembled slim source fragments (layout/css/body/js). Keyed on

package/dashboard.js

@@ -6050,6 +6050,21 @@ const server = http.createServer(async (req, res) => {
     } catch (e) { return jsonReply(res, 500, { error: e.message }); }
   }
 
+  // GET /api/prs/<id> — return a single fully-enriched PR record by canonical
+  // id (`<host>:<slug>#<number>`) or by bare number (P-79b47b0c). The in-stack
+  // PR modal (renderPrs.openPrDetail) calls this on demand. Always returns the
+  // record exactly as queries.getPullRequests() produces it (no slimming).
+  async function handlePrsById(req, res, match) {
+    try {
+      const id = decodeURIComponent(match[1] || '').trim();
+      if (!id) return jsonReply(res, 400, { error: 'id required' });
+      const prs = queries.getPullRequests();
+      const found = prs.find(p => p && (p.id === id || String(p.number) === id));
+      if (!found) return jsonReply(res, 404, { error: 'pr not found' });
+      return jsonReply(res, 200, { pr: found });
+    } catch (e) { return jsonReply(res, 500, { error: e.message }); }
+  }
+
   async function handleWorkItemsReopen(req, res) {
     try {
       const body = await readBody(req);
@@ -8413,10 +8428,31 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       // Add frontmatter if not present
       const today = new Date().toISOString().slice(0, 10);
       let kbContent = content;
+      // P-34fa5d79 — Note ↔ KB linkage. Parse the inbox note's own frontmatter
+      // (if any) to recover the WI it cited so we can stamp the new KB file
+      // with source_note + source_wi. This is the inverse-link the KB modal
+      // renders as "From note:" / "From WI:" chips.
+      const _inboxFm = queries._parseNoteFrontmatter(content) || {};
+      const _sourceWi = queries._wiIdFromNoteFrontmatter(_inboxFm);
       if (!content.startsWith('---')) {
         const titleMatch = content.match(/^#+ (.+)$/m);
         const title = titleMatch ? titleMatch[1].trim() : name.replace('.md', '');
-        kbContent = `---\ntitle: ${title}\ncategory: ${category}\ndate: ${today}\nsource: inbox/${name}\n---\n\n${content}`;
+        const wiLine = _sourceWi ? `source_wi: ${_sourceWi}\n` : '';
+        kbContent = `---\ntitle: ${title}\ncategory: ${category}\ndate: ${today}\nsource: inbox/${name}\nsource_note: ${name}\n${wiLine}---\n\n${content}`;
+      } else {
+        // Inject source_note / source_wi into the existing frontmatter block.
+        // Replace the inbox frontmatter terminator with the new lines + the
+        // terminator; preserves the rest of the body verbatim.
+        const fmMatch = content.match(/^---\n([\s\S]*?)\n---\n?/);
+        if (fmMatch) {
+          const existingFm = fmMatch[1];
+          const additions = [];
+          if (!/^source_note:/m.test(existingFm)) additions.push(`source_note: ${name}`);
+          if (_sourceWi && !/^source_wi:/m.test(existingFm)) additions.push(`source_wi: ${_sourceWi}`);
+          if (additions.length > 0) {
+            kbContent = `---\n${existingFm}\n${additions.join('\n')}\n---\n` + content.slice(fmMatch[0].length);
+          }
+        }
       }
 
       // Write to knowledge base
@@ -12124,6 +12160,11 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         builder: () => queries.getPullRequests(),
       });
     }},
+    // GET /api/prs/<id> — single fully-enriched PR record by canonical id
+    // (`<host>:<slug>#<number>`, URL-encode the `#`) or by bare number.
+    // Backs renderPrs.openPrDetail() in the in-stack PR modal (P-79b47b0c).
+    // Regex uses (.+) (not [^/?]+) because canonical ids embed `/` in the slug.
+    { method: 'GET', path: /^\/api\/prs\/(.+)$/, template: '/api/prs/<id>', desc: 'Fetch a single fully-enriched PR record by canonical id (`<host>:<slug>#<number>`, URL-encode the `#`) or by bare number. Backs the in-stack PR detail modal.', handler: handlePrsById },
     { method: 'GET', path: '/api/dispatch', desc: 'Live dispatch queue with completion-report summaries (pending/active/completed slices)', handler: (req, res) => {
       return serveFreshJson(req, res, {
         tag: 'dispatch',
@@ -12220,7 +12261,14 @@ What would you like to discuss or change? When you're happy, say "approve" and I
             const runEntry = runs[s.id];
             const _lastRun = typeof runEntry === 'string' ? runEntry : (runEntry?.lastRun || runEntry?.lastCompletedAt || null);
             const extra = typeof runEntry === 'object' && runEntry ? { _lastWorkItemId: runEntry.lastWorkItemId, _lastResult: runEntry.lastResult, _lastCompletedAt: runEntry.lastCompletedAt } : {};
-            return { ...s, _lastRun, ...extra };
+            // P-c549d07e — surface the ring buffer of recent WI ids (newest
+            // first, hard cap 5) so the schedule detail modal can render a
+            // "Recent dispatches" chip row routed through openArtifact('wi').
+            // Empty / missing array on legacy entries → empty list.
+            const _recentWorkItemIds = (typeof runEntry === 'object' && runEntry && Array.isArray(runEntry.recentWorkItemIds))
+              ? runEntry.recentWorkItemIds.slice(0, 5)
+              : [];
+            return { ...s, _lastRun, ...extra, _recentWorkItemIds };
           });
         },
       });

package/engine/lifecycle.js

@@ -5144,7 +5144,13 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
       const itemId = meta.item.id;
       const schedRunsPath = path.join(ENGINE_DIR, 'schedule-runs.json');
       mutateJsonFileLocked(schedRunsPath, (runs) => {
+        // P-c549d07e — preserve existing fields (esp. recentWorkItemIds ring
+        // populated at dispatch time by engine/scheduler.writeScheduleRunEntry).
+        // The literal-object replacement that lived here previously silently
+        // clobbered the ring on every successful completion.
+        const existing = typeof runs[scheduleId] === 'object' && runs[scheduleId] ? runs[scheduleId] : {};
         runs[scheduleId] = {
+          ...existing,
           lastRun: typeof runs[scheduleId] === 'string' ? runs[scheduleId] : (runs[scheduleId]?.lastRun || ts()),
           lastWorkItemId: itemId,
           lastResult: effectiveSuccess ? DISPATCH_RESULT.SUCCESS : DISPATCH_RESULT.ERROR,

package/engine/pipeline.js

@@ -427,6 +427,12 @@ function executeTaskStage(stage, stageState, run, config, pipeline = {}) {
           ...(wiBranch ? { branch: wiBranch } : {}),
           _pipelineRun: run.runId,
           _pipelineStage: stage.id,
+          // P-c549d07e — back-link to the dispatcher (pipeline:<id>) so the
+          // WI detail modal can render a clickable chip via openArtifact()
+          // → openPipelineDetail. Kept alongside the existing createdBy /
+          // _pipelineRun / _pipelineStage fields so we don't disturb any
+          // existing readers — meta.spawnedBy is purely additive.
+          meta: { spawnedBy: 'pipeline:' + run.pipelineId },
         });
         return workItems;
       });
@@ -561,6 +567,8 @@ async function executePlanStage(stage, stageState, run, config, pipeline = {}) {
               planFile: existingPlanFile, created: ts(), createdBy: 'pipeline:' + run.pipelineId,
               // W-mp8ho6w500034a58: PLAN_TO_PRD is read-only — no branch needed.
               _pipelineRun: run.runId, _pipelineStage: stage.id,
+              // P-c549d07e — back-link chip (see executeTaskStage above).
+              meta: { spawnedBy: 'pipeline:' + run.pipelineId },
               ...(project ? { project: project.name } : {}),
             });
           }
@@ -657,6 +665,8 @@ async function executePlanStage(stage, stageState, run, config, pipeline = {}) {
           // W-mp8ho6w500034a58: PLAN_TO_PRD is read-only — no branch needed.
           _pipelineRun: run.runId,
           _pipelineStage: stage.id,
+          // P-c549d07e — back-link chip (see executeTaskStage above).
+          meta: { spawnedBy: 'pipeline:' + run.pipelineId },
           ...(project ? { project: project.name } : {}),
         });
       }