@yemi33/minions 0.1.2087 → 0.1.2089

package/dashboard/js/render-prd.js

@@ -160,6 +160,7 @@ function _renderPrdWorkItemIdBadge(workItemId, prdItemId, size, padding) {
 function renderPrdProgress(prog) {
   const el = document.getElementById('prd-progress-content');
   const countEl = document.getElementById('prd-progress-count');
+  const scrollState = captureDashboardScrollState(el, ['.prd-items-list', '.pl-node-chain']);
   if (!prog) { el.innerHTML = ''; countEl.textContent = '—'; return; }
   const visibleItems = (prog.items || []).filter(i => !isDeleted('plan:' + (i.source || '')) && !isDeleted(_prdItemDeleteKey(i.source, i.id)));
 
@@ -564,8 +565,10 @@ function renderPrdProgress(prog) {
     '</div>';
   };
 
-  // E2E / verification PRs — group by sourcePlan
-  const allPrs = window._lastStatus?.pullRequests || [];
+  // E2E / verification PRs — group by sourcePlan. Issue #2949 — pullRequests
+  // moved off /api/status to /api/pull-requests; refresh.js stashes the fresh
+  // fetch into window._lastPullRequests.
+  const allPrs = window._lastPullRequests || [];
   const e2eByPlan = {};
   for (const pr of allPrs) {
     // Match by explicit sourcePlan (robust), or fall back to title/branch heuristic (legacy)
@@ -576,8 +579,11 @@ function renderPrdProgress(prog) {
     e2eByPlan[planKey].push(pr);
   }
 
-  // Find testing guides in prd/ (verify-*.md files)
-  const verifyGuides = (window._lastStatus?.verifyGuides || []);
+  // Find testing guides in prd/ (verify-*.md files). Issue #2949 —
+  // verifyGuides moved off /api/status to /api/verify-guides; refresh.js
+  // fetches it alongside /api/prd in a Promise.all and stashes onto
+  // window._lastVerifyGuides.
+  const verifyGuides = window._lastVerifyGuides || [];
   const guideByPlan = {};
   for (const g of verifyGuides) {
     guideByPlan[g.planFile] = g;
@@ -649,6 +655,7 @@ function renderPrdProgress(prog) {
 
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() or renderMd() by PRD renderers (fields: item ids/names/descriptions, source, projects, PR links, branches, agent labels)
   el.innerHTML = html;
+  restoreDashboardScrollState(el, scrollState);
   restoreNotifBadges();
 }
 
@@ -750,9 +757,11 @@ async function prdItemEdit(source, itemId) {
 
   _prdDescRawCache = item.description || '';
 
-  // Look up work item and dispatch completion info
+  // Look up work item and dispatch completion info. Issue #2949 —
+  // dispatch moved off /api/status to /api/dispatch; refresh.js stashes
+  // the fresh queue into window._lastDispatch.
   const wi = (window._lastWorkItems || []).find(w => w.id === itemId && w.sourcePlan === source);
-  const dispatch = window._lastStatus?.dispatch || {};
+  const dispatch = window._lastDispatch || {};
   const completedEntry = (dispatch.completed || []).find(d =>
     d.meta?.item?.id === itemId && d.meta?.item?.sourcePlan === source
   );

package/dashboard/js/render-prs.js

@@ -4,6 +4,61 @@ let allPrs = [];
 let prPage = 0;
 const PR_PER_PAGE = 25;
 
+// Fetch each project's pull-requests.json straight off disk through the
+// /state/projects/<name>/pull-requests.json static-file passthrough. This
+// is the issue #2949 escape hatch — the engine writes PR status / review
+// state / build state directly into pull-requests.json, and the mtime-ETag
+// on /state/ guarantees a fresh read on the next 4 s poll regardless of
+// whether the /api/status outer cache has busted.
+//
+// Engine-side enrichment is minimal: stamping `_project` from the project
+// name (the filename context) and back-filling `url` from `prUrlBase` when
+// missing. Both are applied here to keep parity with the legacy slice.
+// Other engine-derived fields (build status from CI poll, reviewStatus,
+// _buildStatusStale) are PERSISTED to disk by the engine's PR pollers, so
+// they survive the round-trip naturally.
+async function fetchPullRequestsFromDisk(projects) {
+  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 [];
+    const urlBase = (p && p.prUrlBase) || '';
+    try {
+      const r = await fetch('/state/projects/' + encodeURIComponent(name) + '/pull-requests.json');
+      if (!r.ok) return [];
+      const arr = await r.json();
+      if (!Array.isArray(arr)) return [];
+      return arr.map((pr) => {
+        if (!pr || typeof pr !== 'object' || !pr.id) return null;
+        const stamped = Object.assign({ _project: name }, pr);
+        if (!stamped.url && urlBase && typeof stamped.number === 'number') {
+          stamped.url = urlBase + stamped.number;
+        }
+        return stamped;
+      }).filter(Boolean);
+    } catch {
+      return [];
+    }
+  }));
+  const out = [];
+  const seen = new Set();
+  for (const arr of results) {
+    for (const pr of arr) {
+      if (seen.has(pr.id)) continue;
+      seen.add(pr.id);
+      out.push(pr);
+    }
+  }
+  // Same sort the engine applies: most recent created first, fall back to id.
+  out.sort((a, b) => {
+    const ac = a.created || '';
+    const bc = b.created || '';
+    if (ac && bc) return bc.localeCompare(ac);
+    return (b.id || '').localeCompare(a.id || '');
+  });
+  return out;
+}
+
 function _countPrFollowups(pr) {
   // PR follow-up chip (W-mpej3cox00099466) — counts WIs whose
   // meta.pr_followup.parent_pr_url or parent_pr_id matches this PR.
@@ -152,7 +207,21 @@ function prTableHtml(rows) {
     '</tr></thead><tbody>' + rows + '</tbody></table></div>';
 }
 
-function renderPrs(prs) {
+function _capturePrScrollState(el) {
+  const tableWrap = el ? el.querySelector('.pr-table-wrap') : null;
+  return tableWrap ? { left: tableWrap.scrollLeft || 0, top: tableWrap.scrollTop || 0 } : null;
+}
+
+function _restorePrScrollState(el, state) {
+  if (!state) return;
+  const tableWrap = el ? el.querySelector('.pr-table-wrap') : null;
+  if (!tableWrap) return;
+  tableWrap.scrollLeft = state.left || 0;
+  tableWrap.scrollTop = state.top || 0;
+}
+
+function renderPrs(prs, opts) {
+  opts = opts || {};
   prs = prs.filter(p => !isDeleted('pr:' + p.id));
   allPrs = prs;
   const el = document.getElementById('pr-content');
@@ -163,6 +232,7 @@ function renderPrs(prs) {
     return;
   }
   const totalPages = Math.ceil(prs.length / PR_PER_PAGE);
+  const previousPage = prPage;
   if (prPage >= totalPages) prPage = totalPages - 1;
   const start = prPage * PR_PER_PAGE;
   const pagePrs = prs.slice(start, start + PR_PER_PAGE);
@@ -180,18 +250,16 @@ function renderPrs(prs) {
     '</div>';
   }
 
-  const tableWrap = el.querySelector('.pr-table-wrap');
-  const savedScroll = tableWrap ? tableWrap.scrollLeft : 0;
+  const savedScroll = opts.preserveScroll === false || previousPage !== prPage
+    ? null
+    : _capturePrScrollState(el);
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all PR user data wrapped in escapeHtml() by prRow() (fields: PR id, title, description, agent, branch, review/build/status labels)
   el.innerHTML = prTableHtml(rows) + pager;
-  if (savedScroll) {
-    const newWrap = el.querySelector('.pr-table-wrap');
-    if (newWrap) newWrap.scrollLeft = savedScroll;
-  }
+  _restorePrScrollState(el, savedScroll);
 }
 
-function prPrev() { if (prPage > 0) { prPage--; renderPrs(allPrs); } }
-function prNext() { const totalPages = Math.ceil(allPrs.length / PR_PER_PAGE); if (prPage < totalPages-1) { prPage++; renderPrs(allPrs); } }
+function prPrev() { if (prPage > 0) { prPage--; renderPrs(allPrs, { preserveScroll: false }); } }
+function prNext() { const totalPages = Math.ceil(allPrs.length / PR_PER_PAGE); if (prPage < totalPages-1) { prPage++; renderPrs(allPrs, { preserveScroll: false }); } }
 
 function openAllPrs() {
   const modalEl = document.querySelector('#modal .modal');

package/dashboard/js/render-schedules.js

@@ -247,7 +247,7 @@ function _renderViewToggle() {
 
 function _schedSetView(mode) {
   _schedViewMode = mode;
-  renderSchedules(window._lastSchedules || []);
+  renderSchedules(window._lastSchedules || [], { preserveScroll: false });
 }
 
 function _renderScheduleCalendar(schedules) {
@@ -310,14 +310,17 @@ function _renderScheduleCalendar(schedules) {
   return html;
 }
 
-function renderSchedules(schedules) {
+function renderSchedules(schedules, opts) {
+  opts = opts || {};
   schedules = schedules.filter(function(s) { return !isDeleted('sched:' + s.id); });
   const el = document.getElementById('scheduled-content');
   const countEl = document.getElementById('scheduled-count');
   countEl.textContent = schedules.length;
   window._lastSchedules = schedules;
+  const scrollState = opts.preserveScroll === false ? null : captureDashboardScrollState(el, ['.pr-table-wrap']);
   if (!schedules.length) {
     el.innerHTML = '<p class="empty">No scheduled tasks. Add one to automate recurring work.</p>';
+    restoreDashboardScrollState(el, scrollState);
     return;
   }
 
@@ -371,10 +374,11 @@ function renderSchedules(schedules) {
 
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() by schedule renderers (fields: schedule id/title/cron/type/project/agent/description)
   el.innerHTML = html;
+  restoreDashboardScrollState(el, scrollState);
 }
 
-function _schedPrev() { if (_schedPage > 0) { _schedPage--; renderSchedules(window._lastSchedules || []); } }
-function _schedNext() { var tp = Math.ceil((window._lastSchedules || []).length / SCHED_PER_PAGE); if (_schedPage < tp-1) { _schedPage++; renderSchedules(window._lastSchedules || []); } }
+function _schedPrev() { if (_schedPage > 0) { _schedPage--; renderSchedules(window._lastSchedules || [], { preserveScroll: false }); } }
+function _schedNext() { var tp = Math.ceil((window._lastSchedules || []).length / SCHED_PER_PAGE); if (_schedPage < tp-1) { _schedPage++; renderSchedules(window._lastSchedules || [], { preserveScroll: false }); } }
 
 function openScheduleDetail(id) {
   const s = (window._lastSchedules || []).find(x => x.id === id);

package/dashboard/js/render-utils.js

@@ -405,4 +405,36 @@ function pinButton(pinKey, pinned, source, opts) {
     (pinned ? 'Unpin' : 'Pin') + '</button>';
 }
 
-window.MinionsRenderUtils = { formatToolSummary, renderAgentOutput, renderTerminalBanner, renderPager, pinButton };
+function _scrollNodesForSelector(root, selector) {
+  if (!root || !selector) return [];
+  if (selector === ':self') return [root];
+  var out = [];
+  if (root.matches && root.matches(selector)) out.push(root);
+  if (root.querySelectorAll) {
+    root.querySelectorAll(selector).forEach(function(el) { out.push(el); });
+  }
+  return out;
+}
+
+function captureDashboardScrollState(root, selectors) {
+  var state = [];
+  (selectors || []).forEach(function(selector) {
+    var nodes = _scrollNodesForSelector(root, selector);
+    nodes.forEach(function(el, index) {
+      state.push({ selector: selector, index: index, top: el.scrollTop || 0, left: el.scrollLeft || 0 });
+    });
+  });
+  return state;
+}
+
+function restoreDashboardScrollState(root, state) {
+  (state || []).forEach(function(item) {
+    var nodes = _scrollNodesForSelector(root, item.selector);
+    var el = nodes[item.index];
+    if (!el) return;
+    el.scrollTop = item.top || 0;
+    el.scrollLeft = item.left || 0;
+  });
+}
+
+window.MinionsRenderUtils = { formatToolSummary, renderAgentOutput, renderTerminalBanner, renderPager, pinButton, captureDashboardScrollState, restoreDashboardScrollState };

package/dashboard/js/render-watches.js

@@ -1,5 +1,21 @@
 // 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;
+  }
+}
+
 // ─── Helpers ────────────────────────────────────────────────────────────────
 
 const _WATCH_STATUS_BADGES = {
@@ -117,16 +133,19 @@ function _parseIntervalStr(s) {
 let _watchPage = 0;
 const WATCH_PER_PAGE = 15;
 
-function renderWatches(watchesData) {
+function renderWatches(watchesData, opts) {
+  opts = opts || {};
   var watches = (watchesData || []).filter(function(w) { return !isDeleted('watch:' + w.id); });
   var el = document.getElementById('watches-content');
   var countEl = document.getElementById('watches-count');
   if (!el) return;
   if (countEl) countEl.textContent = watches.length;
   window._lastWatches = watches;
+  var scrollState = opts.preserveScroll === false ? null : captureDashboardScrollState(el, ['.pr-table-wrap']);
 
   if (!watches.length) {
     el.innerHTML = '<p class="empty">No active watches. Create one to monitor PRs or work items.</p>';
+    restoreDashboardScrollState(el, scrollState);
     return;
   }
 
@@ -189,10 +208,11 @@ function renderWatches(watchesData) {
 
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: watch id/target/description/action type/owner/timestamps)
   el.innerHTML = html;
+  restoreDashboardScrollState(el, scrollState);
 }
 
-function _watchPrev() { if (_watchPage > 0) { _watchPage--; renderWatches(window._lastWatches || []); } }
-function _watchNext() { var tp = Math.ceil((window._lastWatches || []).length / WATCH_PER_PAGE); if (_watchPage < tp - 1) { _watchPage++; renderWatches(window._lastWatches || []); } }
+function _watchPrev() { if (_watchPage > 0) { _watchPage--; renderWatches(window._lastWatches || [], { preserveScroll: false }); } }
+function _watchNext() { var tp = Math.ceil((window._lastWatches || []).length / WATCH_PER_PAGE); if (_watchPage < tp - 1) { _watchPage++; renderWatches(window._lastWatches || [], { preserveScroll: false }); } }
 
 // ─── Detail Modal ───────────────────────────────────────────────────────────
 

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

@@ -4,6 +4,76 @@ 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',
+];
+
+// 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;
+}
+
 // 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; }
@@ -107,7 +177,8 @@ function wiRow(item) {
   '</tr>';
 }
 
-function renderWorkItems(items) {
+function renderWorkItems(items, opts) {
+  opts = opts || {};
   items = items.filter(function(w) { return !isDeleted('wi:' + w.id); });
   // Sort: active/dispatched first, then by most recent activity
   const statusOrder = { dispatched: 0, pending: 1, queued: 1, failed: 2, done: 3 };
@@ -128,6 +199,7 @@ function renderWorkItems(items) {
   }
 
   const totalPages = Math.ceil(items.length / WI_PER_PAGE);
+  const previousPage = wiPage;
   if (wiPage >= totalPages) wiPage = totalPages - 1;
   const start = wiPage * WI_PER_PAGE;
   const pageItems = items.slice(start, start + WI_PER_PAGE);
@@ -147,14 +219,12 @@ function renderWorkItems(items) {
     '</div>';
   }
 
-  const tableWrap = el.querySelector('.pr-table-wrap');
-  const savedScroll = tableWrap ? tableWrap.scrollLeft : 0;
+  const scrollState = opts.preserveScroll === false || previousPage !== wiPage
+    ? null
+    : captureDashboardScrollState(el, ['.pr-table-wrap']);
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escapeHtml() by wiRow() (fields: work item id/title/source/status/agent/PR links/dates/follow-up metadata)
   el.innerHTML = html;
-  if (savedScroll) {
-    const newWrap = el.querySelector('.pr-table-wrap');
-    if (newWrap) newWrap.scrollLeft = savedScroll;
-  }
+  restoreDashboardScrollState(el, scrollState);
 }
 
 async function editWorkItem(id, source) {
@@ -367,8 +437,8 @@ async function retryWorkItem(id, source) {
   }
 }
 
-function wiPrev() { if (wiPage > 0) { wiPage--; renderWorkItems(allWorkItems); } }
-function wiNext() { const tp = Math.ceil(allWorkItems.length / WI_PER_PAGE); if (wiPage < tp-1) { wiPage++; renderWorkItems(allWorkItems); } }
+function wiPrev() { if (wiPage > 0) { wiPage--; renderWorkItems(allWorkItems, { preserveScroll: false }); } }
+function wiNext() { const tp = Math.ceil(allWorkItems.length / WI_PER_PAGE); if (wiPage < tp-1) { wiPage++; renderWorkItems(allWorkItems, { preserveScroll: false }); } }
 
 let _feedbackRating = null;
 function feedbackWorkItem(id, source) {

package/dashboard/js/settings.js

@@ -390,11 +390,9 @@ async function openSettings() {
     '<div data-search="routing table playbook agent assignment" style="margin-bottom:16px">' +
       '<textarea id="set-routing" rows="10" style="width:100%;padding:8px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-family:monospace;font-size:var(--text-base);resize:vertical">' + escHtml(data.routing || '') + '</textarea>' +
     '</div>' +
-    '<h4>Inbox &amp; Status Retention</h4>' +
+    '<h4>Inbox</h4>' +
     '<div class="settings-grid-2">' +
       settingsField('Consolidation Threshold', 'set-inboxConsolidateThreshold', e.inboxConsolidateThreshold || 5, 'notes', 'Inbox notes before auto-consolidation') +
-      settingsField('Status WorkItems Retention', 'set-statusWorkItemsRetentionDays', e.statusWorkItemsRetentionDays ?? 0, 'days', 'Optional age-based trim for done/failed/cancelled work items in the /api/status workItems slice (active items are always shipped). Default 0 = no trim — the slim projection that strips description/AC/references already cuts the payload by ~80%. Set to a positive integer to also drop terminal items older than N days.') +
-      settingsField('Status Meetings Retention', 'set-statusMeetingsRetentionDays', e.statusMeetingsRetentionDays ?? 0, 'days', 'Optional age-based trim for completed/archived meetings in the /api/status meetings slice (active meetings are always shipped). Default 0 = no trim — the slim projection that collapses findings/debate/transcript bodies to {agentId: true} sentinels already cuts the payload by ~80%. Set to a positive integer to also drop terminal meetings older than N days. Detail modal still fetches full bodies via /api/meetings/:id.') +
       settingsField('Version Check Interval', 'set-versionCheckInterval', e.versionCheckInterval || 3600000, 'ms', 'How often to check npm for updates (default: 1 hour)') +
     '</div>' +
     '<h4>Operator &amp; Comments</h4>' +
@@ -841,8 +839,6 @@ async function saveSettings() {
       restartGracePeriod: document.getElementById('set-restartGracePeriod').value,
       meetingRoundTimeout: document.getElementById('set-meetingRoundTimeout').value,
       operatorLogin: (document.getElementById('set-operatorLogin')?.value ?? '').trim(),
-      statusWorkItemsRetentionDays: document.getElementById('set-statusWorkItemsRetentionDays').value,
-      statusMeetingsRetentionDays: document.getElementById('set-statusMeetingsRetentionDays').value,
       autoApprovePlans: document.getElementById('set-autoApprovePlans').checked,
       evalLoop: document.getElementById('set-evalLoop').checked,
       autoDecompose: document.getElementById('set-autoDecompose').checked,

package/dashboard/js/state.js

@@ -208,9 +208,15 @@ document.addEventListener('visibilitychange', function() {
 });
 
 function rerenderPrdFromCache() {
-  if (!window._lastStatus || !window._lastStatus.prdProgress) return;
-  renderPrdProgress(window._lastStatus.prdProgress);
-  renderPrd(window._lastStatus.prd, window._lastStatus.prdProgress);
+  // Issue #2949 — prdProgress + prd moved off /api/status to /api/prd;
+  // refresh.js stashes the fresh fetch into window._lastPrdProgress +
+  // window._lastPrdStatus. `!== undefined` (not a truthy check) so a
+  // cache-was-null state (last PRD archived → /api/prd returned
+  // {progress: null}) still triggers a re-render — renderPrd handles
+  // null gracefully and paints the empty 'No PRD found' affordance.
+  if (window._lastPrdProgress === undefined) return;
+  renderPrdProgress(window._lastPrdProgress);
+  renderPrd(window._lastPrdStatus, window._lastPrdProgress);
 }
 
 // Global fetch wrapper with timeout — prevents connection exhaustion from hung requests.

package/dashboard/js/utils.js

@@ -53,8 +53,8 @@ function kbPinKey(cat, file) { return 'knowledge/' + cat + '/' + file; }
 function _togglePinAndRefresh(key, source) {
   var pinned = togglePin(key);
   showToast('cmd-toast', pinned ? 'Pinned to top' : 'Unpinned', true);
-  if (source === 'inbox') renderInbox(inboxData);
-  else if (source === 'kb') renderKnowledgeBase();
+  if (source === 'inbox') renderInbox(inboxData, { preserveScroll: false });
+  else if (source === 'kb') renderKnowledgeBase({ preserveScroll: false });
 }
 
 // Modal navigation stack — enables back button when opening a modal from another modal
@@ -93,8 +93,8 @@ function toggleModalPin() {
   var pinned = togglePin(_modalFilePath);
   showToast('cmd-toast', pinned ? 'Pinned to top' : 'Unpinned', true);
   updateModalPinBtn();
-  if (_modalFilePath.startsWith('notes/inbox/')) renderInbox(inboxData);
-  else if (_modalFilePath.startsWith('knowledge/')) renderKnowledgeBase();
+  if (_modalFilePath.startsWith('notes/inbox/')) renderInbox(inboxData, { preserveScroll: false });
+  else if (_modalFilePath.startsWith('knowledge/')) renderKnowledgeBase({ preserveScroll: false });
 }
 
 // Canonical HTML-escape helper (SEC-03). Use this in all new code and for any user-controlled

package/dashboard-build.js

@@ -29,7 +29,7 @@ function buildDashboardHtml() {
   }
 
   const jsFiles = [
-    'utils', 'state', 'features-client', 'detail-panel', 'live-stream',
+    'utils', 'state', 'features-client', 'render-utils', 'detail-panel', 'live-stream',
     'render-agents', 'render-dispatch', 'render-work-items', 'render-prd',
     'render-prs', 'render-plans', 'render-inbox', 'render-kb', 'render-skills',
     'render-other', 'render-managed', 'render-schedules', 'render-watches', 'render-pipelines', 'render-meetings', 'render-pinned',