@yemi33/minions 0.1.2298 → 0.1.2299

package/dashboard.js

@@ -11444,25 +11444,19 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
   // W-mq03l6zh0006f0a1-d — read-only diagnostics surface for the per-org ADO
   // throttle tracker. Returns { orgs: { [orgBase]: { throttled, retryAfter,
-  // consecutiveHits } } }. Prefers the per-org getter ado.getAdoThrottleStateAll
-  // when present (introduced by W-mq03l6zh0006f0a1-b). Falls back to the
-  // process-global ado.getAdoThrottleState() under the synthetic key `global`
-  // when the per-org getter is not present, so the endpoint stays live across
-  // the staged rollout of the per-org isolation work.
+  // consecutiveHits } } }. Uses getAdoThrottleState().perOrg, the per-org map
+  // exposed by the aggregate getter (W-mq03l6zh0006f0a1-b).
   async function handleDiagnosticsAdoThrottle(req, res) {
     try {
       let orgs = {};
-      if (typeof ado.getAdoThrottleStateAll === 'function') {
-        const all = ado.getAdoThrottleStateAll() || {};
+      if (typeof ado.getAdoThrottleState === 'function') {
+        const all = ado.getAdoThrottleState().perOrg || {};
         // Defensive copy — handler must never expose internal mutable state.
         for (const [k, v] of Object.entries(all)) {
           if (v && typeof v === 'object') {
             orgs[k] = { throttled: !!v.throttled, retryAfter: Number(v.retryAfter) || 0, consecutiveHits: Number(v.consecutiveHits) || 0 };
           }
         }
-      } else if (typeof ado.getAdoThrottleState === 'function') {
-        const v = ado.getAdoThrottleState() || {};
-        orgs.global = { throttled: !!v.throttled, retryAfter: Number(v.retryAfter) || 0, consecutiveHits: Number(v.consecutiveHits) || 0 };
       }
       return jsonReply(res, 200, { orgs });
     } catch (e) { return jsonReply(res, e.statusCode || 500, { error: e.message }); }

package/engine/ado.js

@@ -2683,14 +2683,6 @@ const getAdoThrottleState = (orgBase) => {
   return { throttled, retryAfter, consecutiveHits, perOrg };
 };
 
-/** Returns the per-org tracker state map keyed by canonical orgBase. */
-const getAdoThrottleStateAll = () => {
-  const out = {};
-  for (const [key, tracker] of _adoThrottlesByOrg) {
-    out[key] = tracker.getState();
-  }
-  return out;
-};
 
 /**
  * Query ADO for an open PR on a specific branch.
@@ -2955,7 +2947,6 @@ module.exports = {
   _setAdoTokenAuthNoteWriterForTest, // exported for testing
   isAdoThrottled,
   getAdoThrottleState,
-  getAdoThrottleStateAll,
   fetchAdoPrMetadata,
   prExists, // issue #246 — confirm a loose ref points at a PR (not a work item) before stamping
   fetchSinglePrBuildStatus,

package/engine/lifecycle.js

@@ -3607,24 +3607,25 @@ function extractSkillsFromOutput(output, agentId, dispatchItem, config, runtimeN
 function updateAgentHistory(agentId, dispatchItem, result) {
 
   const historyPath = path.join(AGENTS_DIR, agentId, 'history.md');
-  let history = safeRead(historyPath) || '# Agent History\n\n';
   const entry = `### ${ts()} — ${result}\n` +
     `- **Task:** ${dispatchItem.task}\n` +
     `- **Type:** ${dispatchItem.type}\n` +
     `- **Project:** ${dispatchItem.meta?.project?.name || 'central'}\n` +
     `- **Branch:** ${dispatchItem.meta?.branch || 'none'}\n` +
     `- **Dispatch ID:** ${dispatchItem.id}\n\n`;
-  const headerEnd = history.indexOf('\n\n');
-  if (headerEnd >= 0) {
-    history = history.slice(0, headerEnd + 2) + entry + history.slice(headerEnd + 2);
-  } else {
-    history += entry;
-  }
-  const entries = history.split('### ').filter(Boolean);
-  const header = entries[0].startsWith('#') ? entries.shift() : '# Agent History\n\n';
-  const trimmed = entries.slice(0, 20);
-  history = header + trimmed.map(e => '### ' + e).join('');
-  shared.safeWrite(historyPath, history);
+  shared.mutateTextFileLocked(historyPath, (history) => {
+    history = history || '# Agent History\n\n';
+    const headerEnd = history.indexOf('\n\n');
+    if (headerEnd >= 0) {
+      history = history.slice(0, headerEnd + 2) + entry + history.slice(headerEnd + 2);
+    } else {
+      history += entry;
+    }
+    const entries = history.split('### ').filter(Boolean);
+    const header = entries[0].startsWith('#') ? entries.shift() : '# Agent History\n\n';
+    const trimmed = entries.slice(0, 20);
+    return header + trimmed.map(e => '### ' + e).join('');
+  }, { defaultValue: '# Agent History\n\n' });
   log('info', `Updated history for ${agentId}`);
 }
 

package/engine.js

@@ -5661,7 +5661,7 @@ function reconcileItemsWithPrs(items, allPrs, { onlyIds } = {}) {
 // ─── Inbox Consolidation (extracted to engine/consolidation.js) ──────────────
 
 const { consolidateInbox } = require('./engine/consolidation');
-const { pollPrStatus, pollPrHumanComments, reconcilePrs, checkLiveReviewStatus: adoCheckLiveReview, checkLiveBuildAndConflict: adoCheckLiveBuildAndConflict, needsAdoPollRetry, getAdoToken, isAdoThrottled, getAdoThrottleStateAll } = require('./engine/ado');
+const { pollPrStatus, pollPrHumanComments, reconcilePrs, checkLiveReviewStatus: adoCheckLiveReview, checkLiveBuildAndConflict: adoCheckLiveBuildAndConflict, needsAdoPollRetry, getAdoToken, isAdoThrottled, getAdoThrottleState } = require('./engine/ado');
 const { pollPrStatus: ghPollPrStatus, pollPrHumanComments: ghPollPrHumanComments, reconcilePrs: ghReconcilePrs, checkLiveReviewStatus: ghCheckLiveReview, checkLiveBuildAndConflict: ghCheckLiveBuildAndConflict, isGhThrottled } = require('./engine/github');
 const { reconcileSharedBranchPrs } = require('./engine/shared-branch-pr-reconcile');
 
@@ -9770,7 +9770,7 @@ async function tickInner() {
       // Per-org throttle skip happens inside forEachActivePr (one log line per skipped project).
       // Top-level short-circuit: when every known ADO org is throttled, skip the whole phase
       // with one log line to avoid the per-project iteration cost.
-      const adoThrottleStates = getAdoThrottleStateAll() || {};
+      const adoThrottleStates = getAdoThrottleState().perOrg || {};
       const adoOrgCount = Object.keys(adoThrottleStates).length;
       const allAdoThrottled = adoOrgCount > 0 && Object.values(adoThrottleStates).every(s => s && s.throttled);
       if (allAdoThrottled) {
@@ -9850,7 +9850,7 @@ async function tickInner() {
       // Per-org throttle skip happens inside forEachActivePr (one log line per skipped project).
       // Top-level short-circuit: when every known ADO org is throttled, skip the whole phase
       // with one log line to avoid the per-project iteration cost.
-      const adoThrottleStates = getAdoThrottleStateAll() || {};
+      const adoThrottleStates = getAdoThrottleState().perOrg || {};
       const adoOrgCount = Object.keys(adoThrottleStates).length;
       const allAdoThrottled = adoOrgCount > 0 && Object.values(adoThrottleStates).every(s => s && s.throttled);
       if (allAdoThrottled) {

package/package.json

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