@yemi33/minions 0.1.10 → 0.1.12

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,19 +1005,26 @@ 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;
   if (type === 'decompose' && isSuccess && meta?.item?.id) {
     const subCount = handleDecompositionResult(stdout, meta, config);
-    if (subCount > 0) {
-      // Parent is marked 'decomposed' by handler — don't overwrite with 'done'
-      return { resultSummary: `Decomposed into ${subCount} sub-items`, taskUsage };
-    }
-    // Fallback: if decomposition produced nothing, mark parent as done to avoid stuck state
+    if (subCount > 0) skipDoneStatus = true; // parent already marked 'decomposed' by handler
+    // If decomposition produced nothing, fall through to mark parent as done
   }
 
-  if (isSuccess && meta?.item?.id) updateWorkItemStatus(meta, 'done', '');
+  if (isSuccess && meta?.item?.id && !skipDoneStatus) updateWorkItemStatus(meta, 'done', '');
   if (!isSuccess && meta?.item?.id) {
     // Auto-retry: read fresh _retryCount from file (not stale dispatch-time snapshot)
     let retries = (meta.item._retryCount || 0);
@@ -1055,12 +1049,29 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
         if (wiPath) {
           const items = safeJson(wiPath) || [];
           const wi = items.find(i => i.id === meta.item.id);
-          if (wi) { wi._retryCount = retries + 1; wi.status = 'pending'; delete wi.dispatched_at; delete wi.dispatched_to; shared.safeWrite(wiPath, items); }
+          if (wi) {
+            wi._retryCount = retries + 1; wi.status = 'pending'; delete wi.dispatched_at; delete wi.dispatched_to;
+            if (type === 'decompose') delete wi._decomposing; // clear so item can retry decomposition
+            shared.safeWrite(wiPath, items);
+          }
         }
       } catch {}
     } else {
       updateWorkItemStatus(meta, 'failed', 'Agent failed (3 retries exhausted)');
     }
+    // Clear _decomposing flag on failure so item doesn't get permanently stuck
+    if (type === 'decompose') {
+      try {
+        const wiPath = meta.source === 'central-work-item' || meta.source === 'central-work-item-fanout'
+          ? path.join(MINIONS_DIR, 'work-items.json')
+          : meta.project?.name ? path.join(MINIONS_DIR, 'projects', meta.project.name, 'work-items.json') : null;
+        if (wiPath) {
+          const items = safeJson(wiPath) || [];
+          const wi = items.find(i => i.id === meta.item.id);
+          if (wi) { delete wi._decomposing; shared.safeWrite(wiPath, items); }
+        }
+      } catch {}
+    }
   }
   // Plan chaining removed — user must explicitly execute plan-to-prd after reviewing the plan
   if (isSuccess && meta?.item?.sourcePlan) checkPlanCompletion(meta, config);

package/engine/scheduler.js

@@ -96,16 +96,11 @@ function shouldRunNow(schedule, lastRunAt) {
   const now = new Date();
   if (!cron.matches(now)) return false;
 
-  // Don't fire again if already ran in this minute window
+  // Don't fire again if already ran within the last 55 seconds
+  // (uses elapsed time instead of field comparison to handle DST/clock adjustments)
   if (lastRunAt) {
     const last = new Date(lastRunAt);
-    if (last.getFullYear() === now.getFullYear() &&
-        last.getMonth() === now.getMonth() &&
-        last.getDate() === now.getDate() &&
-        last.getHours() === now.getHours() &&
-        last.getMinutes() === now.getMinutes()) {
-      return false; // already fired this minute
-    }
+    if (Date.now() - last.getTime() < 55000) return false;
   }
 
   return true;
@@ -120,38 +115,34 @@ function discoverScheduledWork(config) {
   const schedules = config.schedules;
   if (!Array.isArray(schedules) || schedules.length === 0) return [];
 
-  const runs = safeJson(SCHEDULE_RUNS_PATH) || {};
+  // Use file-locked mutation to prevent race conditions on rapid calls
   const work = [];
-
-  for (const sched of schedules) {
-    if (!sched.id || !sched.cron || !sched.title) continue;
-    if (sched.enabled === false) continue;
-
-    const lastRun = runs[sched.id] || null;
-    if (!shouldRunNow(sched, lastRun)) continue;
-
-    work.push({
-      id: `sched-${sched.id}-${Date.now()}`,
-      title: sched.title,
-      type: sched.type || 'implement',
-      priority: sched.priority || 'medium',
-      description: sched.description || sched.title,
-      status: 'pending',
-      created: new Date().toISOString(),
-      createdBy: 'scheduler',
-      agent: sched.agent || null,
-      project: sched.project || null,
-      _scheduleId: sched.id,
-    });
-
-    // Record run time
-    runs[sched.id] = new Date().toISOString();
-  }
-
-  // Persist run times if any schedules fired
-  if (work.length > 0) {
-    safeWrite(SCHEDULE_RUNS_PATH, runs);
-  }
+  mutateJsonFileLocked(SCHEDULE_RUNS_PATH, (runs) => {
+    for (const sched of schedules) {
+      if (!sched.id || !sched.cron || !sched.title) continue;
+      if (sched.enabled === false) continue;
+
+      const lastRun = runs[sched.id] || null;
+      if (!shouldRunNow(sched, lastRun)) continue;
+
+      work.push({
+        id: `sched-${sched.id}-${Date.now()}`,
+        title: sched.title,
+        type: sched.type || 'implement',
+        priority: sched.priority || 'medium',
+        description: sched.description || sched.title,
+        status: 'pending',
+        created: new Date().toISOString(),
+        createdBy: 'scheduler',
+        agent: sched.agent || null,
+        project: sched.project || null,
+        _scheduleId: sched.id,
+      });
+
+      // Record run time inside the lock
+      runs[sched.id] = new Date().toISOString();
+    }
+  }, { defaultValue: {} });
 
   return work;
 }

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;
     }
 
@@ -2626,8 +2710,8 @@ function discoverFromWorkItems(config, project) {
     newWork.push({
       type: workType,
       agent: agentId,
-      agentName: config.agents[agentId]?.name,
-      agentRole: config.agents[agentId]?.role,
+      agentName: config.agents[agentId]?.name || tempAgents.get(agentId)?.name || agentId,
+      agentRole: config.agents[agentId]?.role || tempAgents.get(agentId)?.role || 'Agent',
       task: `[${project?.name || 'project'}] ${item.title || item.description?.slice(0, 80) || item.id}`,
       prompt,
       meta: { dispatchKey: key, source: 'work-item', branch: branchName, branchStrategy: item.branchStrategy || 'parallel', useExistingBranch: !!(item.branchStrategy === 'shared-branch' && item.featureBranch), item, project: { name: project?.name, localPath: project?.localPath } }
@@ -3064,16 +3148,18 @@ function discoverWork(config) {
   try {
     const { discoverScheduledWork } = require('./engine/scheduler');
     const scheduledWork = discoverScheduledWork(config);
-    for (const item of scheduledWork) {
-      // Write scheduled items to central work-items.json so they persist across ticks
+    if (scheduledWork.length > 0) {
       const centralPath = path.join(MINIONS_DIR, 'work-items.json');
       const items = safeJson(centralPath) || [];
-      // Dedupe: don't re-create if same schedule already has a pending/dispatched item
-      if (!items.some(i => i._scheduleId === item._scheduleId && i.status !== 'done' && i.status !== 'failed')) {
-        items.push(item);
-        safeWrite(centralPath, items);
-        log('info', `Scheduled task fired: ${item._scheduleId} → ${item.title}`);
+      let added = 0;
+      for (const item of scheduledWork) {
+        if (!items.some(i => i._scheduleId === item._scheduleId && i.status !== 'done' && i.status !== 'failed')) {
+          items.push(item);
+          added++;
+          log('info', `Scheduled task fired: ${item._scheduleId} → ${item.title}`);
+        }
       }
+      if (added > 0) safeWrite(centralPath, items);
     }
   } catch {}
 
@@ -3394,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.10",
+  "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