@yemi33/minions 0.1.11 → 0.1.12

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 0.1.12 (2026-03-26)
+
+### Engine
+- engine.js
+- engine/ado.js
+- engine/cli.js
+- engine/github.js
+- engine/lifecycle.js
+
+### Dashboard
+- dashboard.html
+- dashboard.js
+
+### Other
+- TODO.md
+- routing.md
+- test/unit.test.js
+
 ## 0.1.11 (2026-03-26)
 
 ### Engine

package/dashboard.html

@@ -45,9 +45,19 @@
   @keyframes pulse { 0%,100%{opacity:1} 50%{opacity:0.3} }
   .timestamp { color: var(--muted); font-size: var(--text-md); font-variant-numeric: tabular-nums; }
 
-  .layout { display: grid; grid-template-columns: 1fr 1fr; gap: 0; max-width: 100vw; overflow-x: hidden; }
+  .layout { display: none; } /* Replaced by page-layout */
   section { padding: var(--space-8) var(--space-9); border-bottom: 1px solid var(--border); overflow: hidden; min-width: 0; }
-  section:nth-child(odd) { border-right: 1px solid var(--border); }
+
+  /* Sidebar navigation */
+  .page-layout { display: flex; height: calc(100vh - 44px); overflow: hidden; }
+  .sidebar { width: 150px; min-width: 150px; background: var(--surface); border-right: 1px solid var(--border); padding: var(--space-4) 0; overflow-y: auto; position: sticky; top: 0; }
+  .sidebar-link { display: flex; align-items: center; justify-content: space-between; padding: 8px 14px; color: var(--muted); text-decoration: none; font-size: var(--text-sm); border-left: 3px solid transparent; transition: all var(--transition-fast); cursor: pointer; }
+  .sidebar-link:hover { color: var(--text); background: var(--surface2); }
+  .sidebar-link.active { color: var(--blue); border-left-color: var(--blue); background: var(--surface2); font-weight: 600; }
+  .sidebar-count { font-size: 9px; color: var(--muted); background: var(--surface2); padding: 1px 5px; border-radius: 8px; min-width: 16px; text-align: center; }
+  .page-content { flex: 1; overflow-y: auto; min-width: 0; }
+  .page { display: none; }
+  .page.active { display: block; }
   section h2 { font-size: var(--text-base); font-weight: 600; text-transform: uppercase; letter-spacing: 1px; color: var(--muted); margin-bottom: 14px; display: flex; align-items: center; gap: var(--space-4); }
   section h2 .count { background: var(--surface2); border: 1px solid var(--border); border-radius: var(--radius-xl); padding: var(--space-1) 7px; font-size: var(--text-base); color: var(--text); }
 
@@ -189,7 +199,7 @@
   .inbox-name { font-weight: 500; font-size: var(--text-md); color: var(--purple); margin-bottom: var(--space-2); display: flex; justify-content: space-between; }
   .inbox-preview { font-size: var(--text-base); color: var(--muted); line-height: 1.5; max-height: 60px; overflow: hidden; }
 
-  .prd-panel, .pr-panel { grid-column: 1 / -1; border-bottom: 1px solid var(--border); overflow: visible; min-width: 0; }
+  .prd-panel, .pr-panel { border-bottom: 1px solid var(--border); overflow: visible; min-width: 0; }
   .prd-inner { display: flex; gap: 16px; align-items: flex-start; }
   .prd-stats { display: flex; gap: 16px; }
   .prd-stat { text-align: center; background: var(--surface2); border: 1px solid var(--border); border-radius: var(--radius-lg); padding: var(--space-6) var(--space-8); }
@@ -331,7 +341,7 @@
   .empty { color: var(--muted); font-style: italic; font-size: var(--text-md); padding: var(--space-4) 0; }
 
   /* Command Center — Unified Input */
-  .cmd-center { grid-column: 1 / -1; overflow: visible !important; }
+  .cmd-center { overflow: visible !important; }
   .cmd-input-wrap {
     position: relative; display: flex; align-items: flex-start; gap: 0;
     background: var(--bg); border: 2px solid var(--border); border-radius: var(--radius-xl);
@@ -642,111 +652,134 @@
   <code style="background:var(--bg);padding:6px 16px;border-radius:4px;font-size:14px;color:var(--blue);border:1px solid var(--border)">minions init</code>
 </div>
 
-<div class="layout">
-  <section class="cmd-center">
-    <h2>Command Center</h2>
-    <div class="cmd-input-wrap" id="cmd-input-wrap">
-      <div class="cmd-highlight-layer" id="cmd-highlight" aria-hidden="true"></div>
-      <textarea id="cmd-input" rows="1" placeholder='What do you need? e.g. "Fix the auth bug @dallas", "explain the dispatch flow", or "/note always use feature flags"'
-        oninput="cmdInputChanged()" onkeydown="cmdKeyDown(event)" onscroll="syncHighlightScroll()"></textarea>
-      <button class="cmd-send-btn" id="cmd-send-btn" onclick="cmdSubmit()">Send <kbd>Ctrl+Enter</kbd></button>
+<!-- Layout replaced by page-layout sidebar navigation -->
+
+<div class="page-layout">
+  <nav class="sidebar" id="sidebar">
+    <a class="sidebar-link" data-page="home" href="/">Home</a>
+    <a class="sidebar-link" data-page="work" href="/work">Work Items <span class="sidebar-count" id="sidebar-wi"></span></a>
+    <a class="sidebar-link" data-page="prd" href="/prd">PRD</a>
+    <a class="sidebar-link" data-page="prs" href="/prs">Pull Requests <span class="sidebar-count" id="sidebar-pr"></span></a>
+    <a class="sidebar-link" data-page="plans" href="/plans">Plans</a>
+    <a class="sidebar-link" data-page="inbox" href="/inbox">Notes & KB</a>
+    <a class="sidebar-link" data-page="schedule" href="/schedule">Schedules</a>
+    <a class="sidebar-link" data-page="engine" href="/engine">Engine</a>
+  </nav>
+  <div class="page-content" id="page-content">
+
+    <div class="page active" id="page-home">
+      <section class="cmd-center">
+        <h2>Command Center</h2>
+        <div class="cmd-input-wrap" id="cmd-input-wrap">
+          <div class="cmd-highlight-layer" id="cmd-highlight" aria-hidden="true"></div>
+          <textarea id="cmd-input" rows="1" placeholder='What do you need? e.g. "Fix the auth bug @dallas", "explain the dispatch flow", or "/note always use feature flags"'
+            oninput="cmdInputChanged()" onkeydown="cmdKeyDown(event)" onscroll="syncHighlightScroll()"></textarea>
+          <button class="cmd-send-btn" id="cmd-send-btn" onclick="cmdSubmit()">Send <kbd>Ctrl+Enter</kbd></button>
+        </div>
+        <div class="cmd-mention-popup" id="cmd-mention-popup"></div>
+        <div class="cmd-meta" id="cmd-meta" style="display:none"></div>
+        <div class="cmd-hints">
+          <span style="color:var(--blue);font-weight:600">Command Center</span>
+          <span>Ask anything, dispatch work, manage plans — powered by Sonnet</span>
+          <button class="cmd-history-btn" onclick="cmdShowHistory()">Past Commands</button>
+        </div>
+        <div class="cmd-toast" id="cmd-toast"></div>
+      </section>
+      <section>
+        <h2>Minions Members <span style="font-size:10px;color:var(--border);font-weight:400;text-transform:none;letter-spacing:0">click for details</span></h2>
+        <div class="agents" id="agents-grid">Loading...</div>
+      </section>
+      <section>
+        <h2>Dispatch Queue</h2>
+        <div class="dispatch-stats" id="dispatch-stats"></div>
+        <div id="dispatch-active"></div>
+        <div id="dispatch-pending"></div>
+      </section>
+      <section class="pr-panel" id="completed-section">
+        <h2>Recent Completions <span class="count" id="completed-count">0</span></h2>
+        <div id="completed-content"><p class="empty">No completed dispatches yet.</p></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-work">
+      <section id="work-items-section" style="overflow:visible">
+        <h2>Work Items <span class="count" id="wi-count">0</span> <button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;margin-left:8px" onclick="toggleWorkItemArchive()">See Archive</button></h2>
+        <div id="work-items-content"><p class="empty">No work items. Add tasks via Command Center above.</p></div>
+        <div id="work-items-archive" style="display:none;margin-top:12px"></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-prd">
+      <section class="prd-panel" id="prd-section">
+        <h2>PRD <span class="count" id="prd-progress-count">0%</span> <span id="prd-badge"></span> <span id="archive-btns"></span></h2>
+        <div id="prd-content"><p class="prd-pending">No PRD found.</p></div>
+        <div id="prd-progress-content" style="margin-top:12px"></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-prs">
+      <section class="pr-panel" id="pr-section">
+        <h2>Pull Requests <span class="count" id="pr-count">0</span></h2>
+        <div id="pr-content"><p class="pr-empty">No pull requests yet.</p></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-plans">
+      <section>
+        <h2>Plans <span class="count" id="plans-count">0</span></h2>
+        <div id="plans-list"><p class="empty">No plans yet. Use /plan in the command center to create one.</p></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-inbox">
+      <section>
+        <h2>Notes Inbox <span class="count" id="inbox-count">0</span> <span style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0">auto-consolidates at 3 notes</span></h2>
+        <div class="inbox-list" id="inbox-list">Loading...</div>
+      </section>
+      <section>
+        <h2 data-file="notes.md" style="position:relative">Team Notes <span style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0" id="notes-updated"></span></h2>
+        <div id="notes-list">Loading...</div>
+      </section>
+      <section>
+        <h2>Knowledge Base <span class="count" id="kb-count">0</span> <button id="kb-sweep-btn" onclick="kbSweep()" style="font-size:9px;padding:2px 8px;background:var(--surface2);border:1px solid var(--border);color:var(--muted);border-radius:4px;cursor:pointer;margin-left:8px;vertical-align:middle">sweep</button><span id="kb-swept-time" style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0;margin-left:8px"></span></h2>
+        <div class="kb-tabs" id="kb-tabs"></div>
+        <div class="kb-list" id="kb-list"><p class="empty">No knowledge entries yet. Notes are classified here after consolidation.</p></div>
+      </section>
+      <section>
+        <h2>Minions Skills <span class="count" id="skills-count">0</span></h2>
+        <div id="skills-list"><p class="empty">No skills yet. Agents create these when they discover repeatable workflows.</p></div>
+      </section>
+    </div>
+
+    <div class="page" id="page-schedule">
+      <section id="scheduled-section">
+        <h2>Scheduled Tasks <span class="count" id="scheduled-count">0</span>
+          <button class="pr-pager-btn" style="font-size:9px;padding:1px 6px;color:var(--green);border-color:var(--green);margin-left:8px" onclick="openCreateScheduleModal()">+ New</button>
+        </h2>
+        <div id="scheduled-content"><p class="empty">No scheduled tasks. Add one to automate recurring work.</p></div>
+      </section>
+      <section>
+        <h2>MCP Servers <span class="count" id="mcp-count">0</span></h2>
+        <div id="mcp-list"><p class="empty">No MCP servers synced.</p></div>
+      </section>
     </div>
-    <div class="cmd-mention-popup" id="cmd-mention-popup"></div>
-    <div class="cmd-meta" id="cmd-meta" style="display:none"></div>
-    <div class="cmd-hints">
-      <span style="color:var(--blue);font-weight:600">Command Center</span>
-      <span>Ask anything, dispatch work, manage plans — powered by Sonnet</span>
-      <button class="cmd-history-btn" onclick="cmdShowHistory()">Past Commands</button>
+
+    <div class="page" id="page-engine">
+      <section>
+        <h2>Engine Log</h2>
+        <div class="log-list" id="engine-log">No log entries yet.</div>
+      </section>
+      <section>
+        <h2>Agent Metrics</h2>
+        <div id="metrics-content"><p class="empty">No metrics yet. Metrics appear after agents complete tasks.</p></div>
+      </section>
+      <section>
+        <h2>Token Usage</h2>
+        <div id="token-usage-content"><p class="empty">No usage data yet.</p></div>
+      </section>
     </div>
-    <div class="cmd-toast" id="cmd-toast"></div>
-  </section>
-
-  <section>
-    <h2>Minions Members <span style="font-size:10px;color:var(--border);font-weight:400;text-transform:none;letter-spacing:0">click for details</span></h2>
-    <div class="agents" id="agents-grid">Loading...</div>
-  </section>
-
-  <section id="work-items-section" style="overflow:visible">
-    <h2>Work Items <span class="count" id="wi-count">0</span> <button class="pr-pager-btn" style="font-size:9px;padding:2px 8px;margin-left:8px" onclick="toggleWorkItemArchive()">See Archive</button></h2>
-    <div id="work-items-content"><p class="empty">No work items. Add tasks via Command Center above.</p></div>
-    <div id="work-items-archive" style="display:none;margin-top:12px"></div>
-  </section>
-
-  <section class="prd-panel" id="prd-section">
-    <h2>PRD <span class="count" id="prd-progress-count">0%</span> <span id="prd-badge"></span> <span id="archive-btns"></span></h2>
-    <div id="prd-content"><p class="prd-pending">No PRD found.</p></div>
-    <div id="prd-progress-content" style="margin-top:12px"></div>
-  </section>
-
-  <section class="pr-panel" id="pr-section">
-    <h2>Pull Requests <span class="count" id="pr-count">0</span></h2>
-    <div id="pr-content"><p class="pr-empty">No pull requests yet.</p></div>
-  </section>
-
-  <section>
-    <h2>Plans <span class="count" id="plans-count">0</span></h2>
-    <div id="plans-list"><p class="empty">No plans yet. Use /plan in the command center to create one.</p></div>
-  </section>
-
-  <section>
-    <h2>Notes Inbox <span class="count" id="inbox-count">0</span> <span style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0">auto-consolidates at 3 notes</span></h2>
-    <div class="inbox-list" id="inbox-list">Loading...</div>
-  </section>
-
-  <section>
-    <h2 data-file="notes.md" style="position:relative">Team Notes <span style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0" id="notes-updated"></span></h2>
-    <div id="notes-list">Loading...</div>
-  </section>
-
-  <section>
-    <h2>Knowledge Base <span class="count" id="kb-count">0</span> <button id="kb-sweep-btn" onclick="kbSweep()" style="font-size:9px;padding:2px 8px;background:var(--surface2);border:1px solid var(--border);color:var(--muted);border-radius:4px;cursor:pointer;margin-left:8px;vertical-align:middle">sweep</button><span id="kb-swept-time" style="font-size:10px;color:var(--muted);font-weight:400;text-transform:none;letter-spacing:0;margin-left:8px"></span></h2>
-    <div class="kb-tabs" id="kb-tabs"></div>
-    <div class="kb-list" id="kb-list"><p class="empty">No knowledge entries yet. Notes are classified here after consolidation.</p></div>
-  </section>
-
-  <section>
-    <h2>Minions Skills <span class="count" id="skills-count">0</span></h2>
-    <div id="skills-list"><p class="empty">No skills yet. Agents create these when they discover repeatable workflows.</p></div>
-  </section>
-
-  <section>
-    <h2>MCP Servers <span class="count" id="mcp-count">0</span></h2>
-    <div id="mcp-list"><p class="empty">No MCP servers synced.</p></div>
-  </section>
-
-  <section id="scheduled-section">
-    <h2>Scheduled Tasks <span class="count" id="scheduled-count">0</span>
-      <button class="pr-pager-btn" style="font-size:9px;padding:1px 6px;color:var(--green);border-color:var(--green);margin-left:8px" onclick="openCreateScheduleModal()">+ New</button>
-    </h2>
-    <div id="scheduled-content"><p class="empty">No scheduled tasks. Add one to automate recurring work.</p></div>
-  </section>
-
-  <section>
-    <h2>Dispatch Queue</h2>
-    <div class="dispatch-stats" id="dispatch-stats"></div>
-    <div id="dispatch-active"></div>
-    <div id="dispatch-pending"></div>
-  </section>
-
-  <section>
-    <h2>Engine Log</h2>
-    <div class="log-list" id="engine-log">No log entries yet.</div>
-  </section>
-
-  <section>
-    <h2>Agent Metrics</h2>
-    <div id="metrics-content"><p class="empty">No metrics yet. Metrics appear after agents complete tasks.</p></div>
-  </section>
-
-  <section>
-    <h2>Token Usage</h2>
-    <div id="token-usage-content"><p class="empty">No usage data yet.</p></div>
-  </section>
-
-  <section class="pr-panel" id="completed-section">
-    <h2>Recent Completions <span class="count" id="completed-count">0</span></h2>
-    <div id="completed-content"><p class="empty">No completed dispatches yet.</p></div>
-  </section>
+
+  </div>
 </div>
 
 <!-- Agent Detail Panel -->
@@ -803,6 +836,34 @@ let inboxData = [];
 let agentData = [];
 let currentAgentId = null;
 let currentTab = 'thought-process';
+
+// Sidebar page navigation — URL-routed
+function getPageFromUrl() {
+  const path = window.location.pathname.replace(/^\//, '') || 'home';
+  if (document.querySelector('.sidebar-link[data-page="' + path + '"]')) return path;
+  return 'home';
+}
+
+let currentPage = getPageFromUrl();
+
+function switchPage(page, pushState) {
+  currentPage = page;
+  document.querySelectorAll('.page').forEach(p => p.classList.remove('active'));
+  const target = document.getElementById('page-' + page);
+  if (target) target.classList.add('active');
+  document.querySelectorAll('.sidebar-link').forEach(l => l.classList.remove('active'));
+  const link = document.querySelector('.sidebar-link[data-page="' + page + '"]');
+  if (link) link.classList.add('active');
+  if (pushState !== false) {
+    const url = page === 'home' ? '/' : '/' + page;
+    history.pushState({ page }, '', url);
+  }
+}
+
+// Browser back/forward navigation
+window.addEventListener('popstate', (e) => {
+  switchPage(e.state?.page || getPageFromUrl(), false);
+});
 window._prdRequeueUi = window._prdRequeueUi || {};
 
 function getPrdRequeueState(workItemId) {
@@ -2388,6 +2449,11 @@ async function refresh() {
     renderSkills(data.skills || []);
     renderMcpServers(data.mcpServers || []);
     renderSchedules(data.schedules || []);
+    // Update sidebar counts
+    const swi = document.getElementById('sidebar-wi');
+    if (swi) swi.textContent = (data.workItems || []).length || '';
+    const spr = document.getElementById('sidebar-pr');
+    if (spr) spr.textContent = (data.pullRequests || []).length || '';
     // Refresh KB and plans less frequently (every 3rd cycle = ~12s)
     if (!window._kbRefreshCount) window._kbRefreshCount = 0;
     if (window._kbRefreshCount++ % 3 === 0) { refreshKnowledgeBase(); refreshPlans(); }
@@ -2397,6 +2463,12 @@ async function refresh() {
 refresh();
 setInterval(refresh, 4000);
 
+// Wire sidebar navigation
+document.querySelectorAll('.sidebar-link').forEach(link => {
+  link.addEventListener('click', e => { e.preventDefault(); switchPage(link.dataset.page); });
+});
+switchPage(currentPage);
+
 // -- Projects --
 function renderProjects(projects) {
   const header = document.getElementById('header-projects');

package/dashboard.js

@@ -2767,6 +2767,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     { method: 'POST', path: '/api/schedules/delete', desc: 'Delete a schedule', params: 'id', handler: handleSchedulesDelete },
 
     // Engine
+    { method: 'POST', path: '/api/engine/wakeup', desc: 'Trigger immediate engine tick via control.json signal', handler: async (req, res) => {
+      const controlPath = path.join(MINIONS_DIR, 'engine', 'control.json');
+      const control = shared.safeJson(controlPath) || {};
+      control._wakeupAt = Date.now();
+      shared.safeWrite(controlPath, control);
+      return jsonReply(res, 200, { ok: true, message: 'Wakeup signal sent' });
+    }},
     { method: 'POST', path: '/api/engine/restart', desc: 'Force-kill engine and restart immediately', handler: handleEngineRestart },
 
     // Settings

package/engine/ado.js

@@ -149,6 +149,20 @@ async function pollPrStatus(config) {
       e.log('info', `PR ${pr.id} reviewStatus: ${pr.reviewStatus} → ${newReviewStatus}`);
       pr.reviewStatus = newReviewStatus;
       updated = true;
+      // Update author metrics when verdict changes to approved/rejected
+      if (newReviewStatus === 'approved' || newReviewStatus === 'changes-requested') {
+        const authorId = (pr.agent || '').toLowerCase();
+        if (authorId) {
+          try {
+            const metricsPath = path.join(__dirname, 'metrics.json');
+            const metrics = shared.safeJson(metricsPath) || {};
+            if (!metrics[authorId]) metrics[authorId] = {};
+            if (newReviewStatus === 'approved') metrics[authorId].prsApproved = (metrics[authorId].prsApproved || 0) + 1;
+            else metrics[authorId].prsRejected = (metrics[authorId].prsRejected || 0) + 1;
+            shared.safeWrite(metricsPath, metrics);
+          } catch {}
+        }
+      }
     }
 
     if (newStatus !== 'active') return updated;

package/engine/cli.js

@@ -306,6 +306,17 @@ const commands = {
 
     // Start tick loop
     const tickTimer = setInterval(() => e.tick(), interval);
+
+    // Fast poll for immediate wakeup signals (checks control.json every 2s)
+    setInterval(() => {
+      const ctrl = getControl();
+      if (ctrl._wakeupAt && Date.now() - ctrl._wakeupAt < 5000) {
+        delete ctrl._wakeupAt;
+        safeWrite(CONTROL_PATH, ctrl);
+        e.tick();
+      }
+    }, 2000);
+
     console.log(`Tick interval: ${interval / 1000}s | Max concurrent: ${config.engine?.maxConcurrent || 5}`);
     console.log('Press Ctrl+C to stop');
 

package/engine/github.js

@@ -127,6 +127,20 @@ async function pollPrStatus(config) {
         e.log('info', `PR ${pr.id} reviewStatus: ${pr.reviewStatus} → ${newReviewStatus}`);
         pr.reviewStatus = newReviewStatus;
         updated = true;
+        // Update author metrics when verdict changes to approved/rejected
+        if (newReviewStatus === 'approved' || newReviewStatus === 'changes-requested') {
+          const authorId = (pr.agent || '').toLowerCase();
+          if (authorId) {
+            try {
+              const metricsPath = path.join(__dirname, 'metrics.json');
+              const metrics = shared.safeJson(metricsPath) || {};
+              if (!metrics[authorId]) metrics[authorId] = {};
+              if (newReviewStatus === 'approved') metrics[authorId].prsApproved = (metrics[authorId].prsApproved || 0) + 1;
+              else metrics[authorId].prsRejected = (metrics[authorId].prsRejected || 0) + 1;
+              shared.safeWrite(metricsPath, metrics);
+            } catch {}
+          }
+        }
       }
     }
 

package/engine/lifecycle.js

@@ -617,28 +617,25 @@ function updatePrAfterReview(agentId, pr, project) {
   const dispatch = getDispatch();
   const completedEntry = (dispatch.completed || []).find(d => d.agent === agentId && d.type === 'review');
 
+  // Set reviewStatus to 'waiting' (single source of truth — synced from ADO/GitHub votes on next poll)
+  target.reviewStatus = 'waiting';
   target.minionsReview = {
-    status: 'waiting',
     reviewer: reviewerName,
     reviewedAt: e.ts(),
     note: completedEntry?.task || ''
   };
-  const minionsVerdict = target.minionsReview.status;
+  // Metrics update: don't track 'waiting' as a verdict — metrics are updated
+  // when pollPrStatus syncs the actual vote to minionsReview.status.
+  // The reviewer's reviewsDone counter is incremented in the main updateMetrics call.
 
+  // Track reviewer for metrics purposes
   const authorAgentId = (pr.agent || '').toLowerCase();
   if (authorAgentId && config.agents?.[authorAgentId]) {
     const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
     const metrics = safeJson(metricsPath) || {};
     if (!metrics[authorAgentId]) metrics[authorAgentId] = { tasksCompleted:0, tasksErrored:0, prsCreated:0, prsApproved:0, prsRejected:0, reviewsDone:0, lastTask:null, lastCompleted:null };
-    if (!metrics[authorAgentId]._reviewedPrs) metrics[authorAgentId]._reviewedPrs = {};
-    const prevVerdict = metrics[authorAgentId]._reviewedPrs[pr.id];
-    if (prevVerdict !== minionsVerdict) {
-      if (prevVerdict === 'approved') metrics[authorAgentId].prsApproved = Math.max(0, (metrics[authorAgentId].prsApproved || 0) - 1);
-      else if (prevVerdict === 'changes-requested') metrics[authorAgentId].prsRejected = Math.max(0, (metrics[authorAgentId].prsRejected || 0) - 1);
-      if (minionsVerdict === 'approved') metrics[authorAgentId].prsApproved++;
-      else if (minionsVerdict === 'changes-requested') metrics[authorAgentId].prsRejected++;
-      metrics[authorAgentId]._reviewedPrs[pr.id] = minionsVerdict;
-    }
+    if (!metrics[agentId]) metrics[agentId] = { tasksCompleted:0, tasksErrored:0, prsCreated:0, prsApproved:0, prsRejected:0, reviewsDone:0, lastTask:null, lastCompleted:null };
+    metrics[agentId].reviewsDone = (metrics[agentId].reviewsDone || 0) + 1;
     shared.safeWrite(metricsPath, metrics);
   }
 
@@ -654,25 +651,15 @@ function updatePrAfterFix(pr, project, source) {
   const target = prs.find(p => p.id === pr.id);
   if (!target) return;
 
+  // Reset reviewStatus to 'waiting' for re-review (single source of truth)
+  target.reviewStatus = 'waiting';
   if (source === 'pr-human-feedback') {
-    // Human feedback fix: clear pendingFix AND reset to waiting for re-review
     if (target.humanFeedback) target.humanFeedback.pendingFix = false;
-    target.minionsReview = {
-      ...target.minionsReview,
-      status: 'waiting',
-      note: 'Fixed human feedback, awaiting re-review',
-      fixedAt: e.ts()
-    };
+    target.minionsReview = { ...target.minionsReview, note: 'Fixed human feedback, awaiting re-review', fixedAt: e.ts() };
     e.log('info', `Updated ${pr.id} → cleared humanFeedback.pendingFix, reset to waiting for re-review`);
   } else {
-    // Review fix: reset to waiting for re-review
-    target.minionsReview = {
-      ...target.minionsReview,
-      status: 'waiting',
-      note: 'Fixed, awaiting re-review',
-      fixedAt: e.ts()
-    };
-    e.log('info', `Updated ${pr.id} → minions review: waiting (fix pushed)`);
+    target.minionsReview = { ...target.minionsReview, note: 'Fixed, awaiting re-review', fixedAt: e.ts() };
+    e.log('info', `Updated ${pr.id} → reviewStatus: waiting (fix pushed)`);
   }
 
   shared.safeWrite(project ? shared.projectPrPath(project) : path.join(path.resolve(MINIONS_DIR, '..'), '.minions', 'pull-requests.json'), prs);
@@ -932,8 +919,8 @@ function updateMetrics(agentId, dispatchItem, result, taskUsage, prsCreatedCount
 // ─── Agent Output Parsing ────────────────────────────────────────────────────
 
 function parseAgentOutput(stdout) {
-  const parsed = shared.parseStreamJsonOutput(stdout, { maxTextLength: 500 });
-  return { resultSummary: parsed.text, taskUsage: parsed.usage };
+  const { text, usage, sessionId } = shared.parseStreamJsonOutput(stdout, { maxTextLength: 2000 });
+  return { resultSummary: text, taskUsage: usage, sessionId };
 }
 
 /**
@@ -1018,7 +1005,16 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   const meta = dispatchItem.meta;
   const isSuccess = code === 0;
   const result = isSuccess ? 'success' : 'error';
-  const { resultSummary, taskUsage } = parseAgentOutput(stdout);
+  const { resultSummary, taskUsage, sessionId } = parseAgentOutput(stdout);
+
+  // Save session for potential resume on next dispatch
+  if (isSuccess && sessionId && agentId && !agentId.startsWith('temp-')) {
+    try {
+      shared.safeWrite(path.join(AGENTS_DIR, agentId, 'session.json'), {
+        sessionId, dispatchId: dispatchItem.id, savedAt: new Date().toISOString()
+      });
+    } catch {}
+  }
 
   // Handle decomposition results — create sub-items from decompose agent output
   let skipDoneStatus = false;

package/engine.js

@@ -166,6 +166,25 @@ function getRoutingTableCached() {
   return _routingCache;
 }
 
+function getMonthlySpend(agentId) {
+  const metrics = safeJson(path.join(ENGINE_DIR, 'metrics.json')) || {};
+  const daily = metrics._daily || {};
+  const now = new Date();
+  const monthPrefix = `${now.getFullYear()}-${String(now.getMonth() + 1).padStart(2, '0')}`;
+  let total = 0;
+  for (const [date, data] of Object.entries(daily)) {
+    if (date.startsWith(monthPrefix)) {
+      total += (data.perAgent?.[agentId]?.costUsd || 0);
+    }
+  }
+  // Fallback: if no per-agent daily data, use cumulative (less accurate for monthly)
+  if (total === 0 && metrics[agentId]?.totalCostUsd) {
+    // Can't distinguish monthly from cumulative — treat as monthly estimate
+    // This path is for backward compat before per-agent daily tracking was added
+  }
+  return total;
+}
+
 function getAgentErrorRate(agentId) {
   const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
   const metrics = safeJson(metricsPath) || {};
@@ -194,7 +213,15 @@ function resolveAgent(workType, config, authorAgent = null) {
   let preferred = route.preferred === '_author_' ? authorAgent : route.preferred;
   let fallback = route.fallback === '_author_' ? authorAgent : route.fallback;
 
-  const isAvailable = (id) => agents[id] && isAgentIdle(id) && !_claimedAgents.has(id);
+  const isAvailable = (id) => {
+    if (!agents[id] || !isAgentIdle(id) || _claimedAgents.has(id)) return false;
+    // Budget check — no budget means infinite (no limit)
+    const budget = agents[id].monthlyBudgetUsd;
+    if (budget && budget > 0) {
+      if (getMonthlySpend(id) >= budget) return false;
+    }
+    return true;
+  };
 
   // Check preferred and fallback first (routing table order)
   if (preferred && isAvailable(preferred)) { _claimedAgents.add(preferred); return preferred; }
@@ -905,6 +932,20 @@ function spawnAgent(dispatchItem, config) {
     args.push('--allowedTools', claudeConfig.allowedTools);
   }
 
+  // Session resume: reuse last session if recent enough (< 2 hours)
+  if (!agentId.startsWith('temp-')) {
+    try {
+      const sessionFile = safeJson(path.join(AGENTS_DIR, agentId, 'session.json'));
+      if (sessionFile?.sessionId && sessionFile.savedAt) {
+        const sessionAge = Date.now() - new Date(sessionFile.savedAt).getTime();
+        if (sessionAge < 2 * 60 * 60 * 1000) { // 2 hour TTL
+          args.push('--resume', sessionFile.sessionId);
+          log('info', `Resuming session ${sessionFile.sessionId} for ${agentId} (age: ${Math.round(sessionAge / 60000)}min)`);
+        }
+      }
+    } catch {}
+  }
+
   // MCP servers: agents inherit from ~/.claude.json directly as Claude Code processes.
   // No --mcp-config needed — avoids redundant config and ensures agents always have latest servers.
 
@@ -1968,6 +2009,27 @@ function setCooldown(key) {
   saveCooldowns();
 }
 
+function setCooldownWithContext(key, context) {
+  const existing = dispatchCooldowns.get(key);
+  const pendingContexts = existing?.pendingContexts || [];
+  if (context) pendingContexts.push(context);
+  dispatchCooldowns.set(key, {
+    timestamp: Date.now(),
+    failures: existing?.failures || 0,
+    pendingContexts
+  });
+  saveCooldowns();
+}
+
+function getCoalescedContexts(key) {
+  const entry = dispatchCooldowns.get(key);
+  const contexts = entry?.pendingContexts || [];
+  if (contexts.length > 0 && entry) {
+    entry.pendingContexts = []; // Clear after retrieval
+  }
+  return contexts;
+}
+
 function setCooldownFailure(key) {
   const existing = dispatchCooldowns.get(key);
   const failures = (existing?.failures || 0) + 1;
@@ -2390,19 +2452,16 @@ function discoverFromPrs(config, project) {
     if (activePrIds.has(pr.id)) continue; // Skip PRs with active dispatch (prevent race)
 
     const prNumber = (pr.id || '').replace(/^PR-/, '');
-    const minionsStatus = pr.minionsReview?.status;
+    // Use reviewStatus as single source of truth (synced from ADO/GitHub votes)
+    // minionsReview tracks metadata (reviewer, note) but not the authoritative status
+    const reviewStatus = pr.reviewStatus || 'pending';
 
-    // PRs needing review
-    const needsReview = !minionsStatus || minionsStatus === 'waiting';
+    // PRs needing review: pending or waiting (review dispatched but no verdict yet)
+    const needsReview = reviewStatus === 'pending' || reviewStatus === 'waiting';
     if (needsReview) {
       const key = `review-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
-      // No self-review: exclude the PR author from review assignment
-      const prAuthor = (pr.agent || '').toLowerCase();
-      let agentId = resolveAgent('review', config);
-      if (agentId && agentId === prAuthor) {
-        agentId = resolveAgent('review', config); // retry — prAuthor now claimed, gets skipped
-      }
+      const agentId = resolveAgent('review', config);
       if (!agentId) continue;
 
       const item = buildPrDispatch(agentId, config, project, pr, 'review', {
@@ -2413,7 +2472,7 @@ function discoverFromPrs(config, project) {
     }
 
     // PRs with changes requested → route back to author for fix
-    if (minionsStatus === 'changes-requested') {
+    if (reviewStatus === 'changes-requested') {
       const key = `fix-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
       const agentId = resolveAgent('fix', config, pr.agent);
@@ -2426,22 +2485,37 @@ function discoverFromPrs(config, project) {
       if (item) { newWork.push(item); setCooldown(key); }
     }
 
-    // PRs with pending human feedback
-    if (pr.humanFeedback?.pendingFix) {
-      const key = `human-fix-${project?.name || 'default'}-${pr.id}`;
-      if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
+    // PRs with pending human feedback (or coalesced comments from while agent was fixing)
+    const humanFixKey = `human-fix-${project?.name || 'default'}-${pr.id}`;
+    const hasCoalescedFeedback = (dispatchCooldowns.get(humanFixKey)?.pendingContexts || []).length > 0;
+    if (pr.humanFeedback?.pendingFix || hasCoalescedFeedback) {
+      const key = humanFixKey;
+      if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) {
+        // Coalesce: save feedback for next dispatch
+        if (pr.humanFeedback?.feedbackContent) {
+          setCooldownWithContext(key, { feedbackContent: pr.humanFeedback.feedbackContent, timestamp: new Date().toISOString() });
+        }
+        continue;
+      }
       const agentId = resolveAgent('fix', config, pr.agent);
       if (!agentId) continue;
 
+      const coalesced = getCoalescedContexts(key);
+      let reviewNote = pr.humanFeedback.feedbackContent || 'See PR thread comments';
+      if (coalesced.length > 0) {
+        const earlier = coalesced.map(c => c.feedbackContent).filter(Boolean).join('\n\n---\n\n');
+        if (earlier) reviewNote = earlier + '\n\n---\n\n' + reviewNote;
+      }
+
       const item = buildPrDispatch(agentId, config, project, pr, 'fix', {
         pr_id: pr.id, pr_number: prNumber, pr_title: pr.title || '', pr_branch: pr.branch || '',
         reviewer: 'Human Reviewer',
-        review_note: pr.humanFeedback.feedbackContent || 'See PR thread comments',
+        review_note: reviewNote,
       }, `Fix PR ${pr.id} — human feedback`, { dispatchKey: key, source: 'pr-human-feedback', pr, branch: pr.branch, project: projMeta });
       if (item) { newWork.push(item); setCooldown(key); }
     }
 
-    // PRs with build failures
+    // PRs with build failures — any agent can pick this up
     if (pr.status === 'active' && pr.buildStatus === 'failing') {
       const key = `build-fix-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
@@ -2565,7 +2639,17 @@ function discoverFromWorkItems(config, project) {
     }
     const agentId = item.agent || resolveAgent(workType, config);
     if (!agentId) {
-      if (item._pendingReason !== 'no_agent') { item._pendingReason = 'no_agent'; needsWrite = true; }
+      // Check if reason is budget
+      const cfgAgents = config.agents || {};
+      const budgetBlocked = Object.keys(cfgAgents).some(id => {
+        const b = cfgAgents[id].monthlyBudgetUsd;
+        return b && b > 0 && getMonthlySpend(id) >= b && isAgentIdle(id);
+      });
+      if (budgetBlocked) {
+        if (item._pendingReason !== 'budget_exceeded') { item._pendingReason = 'budget_exceeded'; needsWrite = true; }
+      } else {
+        if (item._pendingReason !== 'no_agent') { item._pendingReason = 'no_agent'; needsWrite = true; }
+      }
       skipped.noAgent++; continue;
     }
 
@@ -3396,7 +3480,10 @@ module.exports = {
   updateWorkItemStatus, runCleanup, handlePostMerge,
 
   // Cooldowns
-  loadCooldowns,
+  loadCooldowns, setCooldownWithContext, getCoalescedContexts,
+
+  // Budget
+  getMonthlySpend,
 
   // Tick
   tick,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "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"

package/routing.md

@@ -27,7 +27,7 @@ Notes:
 ## Rules
 
 1. **Eager by default** — spawn all agents who can start work, not one at a time
-2. **No self-review** — author cannot review their own PR
+2. **Self-review is allowed** — agents can review their own PRs (useful for single-agent setups)
 3. **Exploration gates implementation** — when exploring, finish before implementing
 4. **Implementation informs PRD** — Lambert reads build summaries before writing PRD
 5. **All rules in `notes.md` apply** — engine injects them into every playbook