@yemi33/minions 0.1.2014 → 0.1.2016

package/dashboard/js/refresh.js

@@ -74,6 +74,17 @@ function _processStatusUpdate(data) {
   const threshEl = document.getElementById('inbox-threshold');
   if (threshEl && data.autoMode?.inboxThreshold) threshEl.textContent = data.autoMode.inboxThreshold;
 
+  // Publish window._last* snapshots BEFORE any renderer runs. Several renderers
+  // (renderProjects, renderPrd, renderPrs, refreshPlans, and others) read
+  // window._lastStatus / window._lastWorkItems / window._lastDispatch during
+  // their render path. If we assign these after the renderer calls, every tick
+  // consumes the PREVIOUS tick's globals — a one-tick (~4 s) staleness lag
+  // baked into the polling loop (W-mpgb0x81000cf8df, audit F4). No renderer in
+  // this function mutates `data`, so hoisting is safe.
+  window._lastDispatch = data.dispatch;
+  window._lastWorkItems = data.workItems || [];
+  window._lastStatus = data;
+
   // Render only changed sections
   if (_changed('agents', data.agents)) { renderAgents(data.agents); cmdUpdateAgentList(data.agents); }
   if (_changed('prdProgress', data.prdProgress) || _changed('prdPrs', data.pullRequests?.length)) { renderPrdProgress(data.prdProgress); _cachePrdItems(data.prdProgress); }
@@ -81,13 +92,19 @@ function _processStatusUpdate(data) {
   if (_changed('projects', data.projects)) { cmdUpdateProjectList(data.projects || []); renderProjects(data.projects || []); }
   // FRE banner — safe to call every tick (idempotent + cheap). Pass the full
   // status payload so the runtime-CLI explainer reads autoMode.defaultCli from
-  // THIS tick rather than the previous one (window._lastStatus is set later).
+  // THIS tick (window._lastStatus is hoisted above, but renderFre takes the
+  // payload directly to avoid the window-global indirection).
   if (typeof renderFre === 'function') {
     try { renderFre(data); } catch { /* expected on first load */ }
   }
   if (_changed('notes', data.notes)) renderNotes(data.notes);
   if (_changed('prd', [data.prd, data.prdProgress])) renderPrd(data.prd, data.prdProgress);
-  if (_changed('prs', data.pullRequests)) renderPrs(data.pullRequests || []);
+  // Capture prs + workItems change signals once — also reused by the cross-slice
+  // render triggers at the bottom of this function (F1/F3, W-mpgb0xbh000e3b86).
+  // _changed mutates _sectionCache so it must be called exactly once per key.
+  var _prsChanged = _changed('prs', data.pullRequests);
+  var _workItemsChanged = _changed('workItems', data.workItems);
+  if (_prsChanged) renderPrs(data.pullRequests || []);
   if (_changed('archivedPrds', data.archivedPrds)) renderArchiveButtons(data.archivedPrds || []);
   if (_changed('engine', data.engine)) {
     renderEngineStatus(data.engine);
@@ -103,9 +120,6 @@ function _processStatusUpdate(data) {
   if (_changed('adoThrottle', data.adoThrottle)) renderAdoThrottleAlert(data.adoThrottle);
   if (_changed('ghThrottle', data.ghThrottle)) renderGhThrottleAlert(data.ghThrottle);
   if (_changed('dispatch', data.dispatch)) renderDispatch(data.dispatch);
-  window._lastDispatch = data.dispatch;
-  window._lastWorkItems = data.workItems || [];
-  window._lastStatus = data;
   prunePrdRequeueState(window._lastWorkItems);
   if (_changed('engineLog', data.engineLog)) renderEngineLog(data.engineLog || []);
   if (_changed('metrics', data.metrics)) renderMetrics(data.metrics || {});
@@ -127,7 +141,7 @@ function _processStatusUpdate(data) {
       .catch(function () { /* keep render even if managed fetch failed — getLastItems() returns the last good cache (or []) */ })
       .then(function () { try { renderKeepProcesses(); } catch {} });
   }
-  if (_changed('workItems', data.workItems)) renderWorkItems(data.workItems || []);
+  if (_workItemsChanged) renderWorkItems(data.workItems || []);
   if (_changed('skills', data.skills)) renderSkills(data.skills || []);
   if (_changed('mcpServers', data.mcpServers)) renderMcpServers(data.mcpServers || []);
   if (_changed('schedules', data.schedules)) renderSchedules(data.schedules || []);
@@ -144,6 +158,27 @@ function _processStatusUpdate(data) {
   if (!window._kbRefreshCount) window._kbRefreshCount = 0;
   if (window._kbRefreshCount++ % 3 === 0) { refreshKnowledgeBase(); refreshPlans(); }
 
+  // Cross-slice render triggers (F1/F3, W-mpgb0xbh000e3b86): renderPrs reads
+  // window._lastWorkItems for the +N follow-up chip count and derivePlanStatus
+  // reads window._lastStatus.pullRequests for verify-follow-up reconciliation.
+  // When workItems OR pullRequests change, re-render the dependents against the
+  // freshest cached data — without waiting up to 12s for the next refreshPlans
+  // cycle (F3) or for a PR object to mutate (F1). Runs after the window._last*
+  // assignments above so the cached globals these renderers consult are fresh.
+  if (_workItemsChanged && !_prsChanged) {
+    // F1: only the work-item slice moved this tick — renderPrs wasn't called
+    // above, so the +N follow-up chip would otherwise stay stale until the
+    // next PR mutation.
+    renderPrs(data.pullRequests || []);
+  }
+  if ((_workItemsChanged || _prsChanged) && Array.isArray(window._lastPlans) && typeof renderPlans === 'function') {
+    // F3: derivePlanStatus + _renderVerifyBadge derive from pullRequests +
+    // workItems. Re-render against the cached plan list so plan status flips
+    // within one /api/status tick (~4s) instead of one refreshPlans poll
+    // (~12s). No-op when _lastPlans hasn't been populated yet (first load).
+    renderPlans(window._lastPlans);
+  }
+
   // Sidebar activity indicators — show red dot on pages with new activity
   try {
     const changes = _detectPageChanges(data);
@@ -164,7 +199,15 @@ let _knownDashboardStartId = null;
 // downstream consumers of `data`.
 let _lastStatusEtag = null;
 let _lastStatusData = null;
+// In-flight gate (S5 — dashboard-refresh-audit). setInterval(refresh, 4000)
+// can interleave two ticks inside _processStatusUpdate if a refresh takes
+// >4 s (cold server cache + slow JSON + network blip). Skip the second tick
+// when the first hasn't finished — the next interval fire picks up the
+// fresh state anyway.
+let _refreshInFlight = false;
 async function refresh() {
+  if (_refreshInFlight) return;
+  _refreshInFlight = true;
   try {
     const headers = {};
     if (_lastStatusEtag) headers['If-None-Match'] = _lastStatusEtag;
@@ -191,6 +234,7 @@ async function refresh() {
     if (dashId) _knownDashboardStartId = dashId;
     _processStatusUpdate(data);
   } catch(e) { console.error('refresh error', e); }
+  finally { _refreshInFlight = false; }
 }
 
 refresh();

package/dashboard/js/render-kb.js

@@ -32,8 +32,16 @@ function _formatSweepElapsed(startedAt) {
 
 async function refreshKnowledgeBase() {
   try {
+    // Always update _kbData — cross-slice readers (e.g. command-center.js
+    // suggestion lookup) depend on it staying fresh even when the renderer is
+    // skipped (S1).
     _kbData = await fetch('/api/knowledge').then(r => r.json());
-    renderKnowledgeBase();
+    // Skip the full kb-list/kb-tabs rewrite when /api/knowledge returned the
+    // same payload as last tick. Dedicated cache key 'kbPayload' avoids
+    // colliding with any other tracked slice (S1).
+    if (typeof _changed !== 'function' || _changed('kbPayload', _kbData)) {
+      renderKnowledgeBase();
+    }
   } catch (e) { console.error('kb refresh:', e.message); }
 }
 

package/dashboard/js/render-meetings.js

@@ -1,7 +1,7 @@
 // render-meetings.js — Team meeting rendering
 
 let _showArchived = false;
-const MTG_PER_PAGE = 10;
+const MTG_PER_PAGE = 25;
 let _mtgPage = 0;
 let _lastMeetingHash = '';
 let _lastMeetingsForPaging = [];

package/dashboard/js/render-pipelines.js

@@ -3,6 +3,9 @@
 let _pipelinesData = [];
 let _pipelinePollId = null;
 let _pipelinePollInterval = null;
+const PIPELINE_PER_PAGE = 25;
+let _pipelinePage = 0;
+let _pipelineTotalPages = 1;
 function _stopPipelinePoll() { if (_pipelinePollInterval) { clearInterval(_pipelinePollInterval); _pipelinePollInterval = null; } _pipelinePollId = null; }
 
 /**
@@ -293,11 +296,18 @@ function renderPipelines(pipelines) {
   if (!pipelines || pipelines.length === 0) {
     countEl.textContent = '0';
     el.innerHTML = '<p class="empty">No pipelines yet. Create one to chain stages like audit \u2192 meeting \u2192 plan \u2192 merge.</p>';
+    _pipelineTotalPages = 1;
     return;
   }
   countEl.textContent = pipelines.length;
 
-  el.innerHTML = pipelines.map(function(p) {
+  const totalPages = Math.ceil(pipelines.length / PIPELINE_PER_PAGE);
+  _pipelineTotalPages = totalPages;
+  if (_pipelinePage >= totalPages) _pipelinePage = Math.max(0, totalPages - 1);
+  const start = _pipelinePage * PIPELINE_PER_PAGE;
+  const pageItems = pipelines.slice(start, start + PIPELINE_PER_PAGE);
+
+  el.innerHTML = pageItems.map(function(p) {
     const activeRun = _getPipelineActiveRun(p);
     const lastRun = (p.runs || []).slice(-1)[0];
     const statusColor = activeRun ? 'var(--blue)' : lastRun?.status === 'completed' ? 'var(--green)' : lastRun?.status === 'failed' ? 'var(--red)' : lastRun?.status === 'stopped' ? 'var(--yellow)' : 'var(--muted)';
@@ -336,6 +346,29 @@ function renderPipelines(pipelines) {
       progressHtml +
     '</div>';
   }).join('');
+
+  if (pipelines.length > PIPELINE_PER_PAGE) {
+    el.insertAdjacentHTML('beforeend', '<div class="pr-pager">' +
+      '<span class="pr-page-info">Showing ' + (start + 1) + ' to ' + Math.min(start + PIPELINE_PER_PAGE, pipelines.length) + ' of ' + pipelines.length + '</span>' +
+      '<div class="pr-pager-btns">' +
+        '<button class="pr-pager-btn ' + (_pipelinePage === 0 ? 'disabled' : '') + '" onclick="_pipelinePrev()">Prev</button>' +
+        '<button class="pr-pager-btn ' + (_pipelinePage >= totalPages - 1 ? 'disabled' : '') + '" onclick="_pipelineNext()">Next</button>' +
+      '</div></div>');
+  }
+}
+
+function _pipelinePrev() {
+  if (_pipelinePage > 0) {
+    _pipelinePage--;
+    renderPipelines(_pipelinesData);
+  }
+}
+
+function _pipelineNext() {
+  if (_pipelinePage < _pipelineTotalPages - 1) {
+    _pipelinePage++;
+    renderPipelines(_pipelinesData);
+  }
 }
 
 function openPipelineDetail(id) {

package/dashboard/js/render-plans.js

@@ -56,8 +56,16 @@ async function _submitCreatePlan(e) {
 async function refreshPlans() {
   try {
     const plans = await fetch('/api/plans').then(r => r.json());
-    window._lastPlans = plans; // Store globally so plan modal can access
-    renderPlans(plans);
+    // Cross-slice readers (renderMeetings, plan modal, command-center, etc.)
+    // depend on _lastPlans staying current — assign BEFORE the change gate so
+    // they see fresh data even when the renderer is skipped (F2/F3).
+    window._lastPlans = plans;
+    // Skip the full plans-list innerHTML rewrite when /api/plans returned the
+    // same payload as last tick. Dedicated cache key 'plansPayload' avoids
+    // colliding with the 'plans' slice tracked by _processStatusUpdate (F6).
+    if (typeof _changed !== 'function' || _changed('plansPayload', plans)) {
+      renderPlans(plans);
+    }
   } catch (e) { console.error('plans refresh:', e.message); }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2014",
+  "version": "0.1.2016",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"