@yemi33/minions 0.1.2276 → 0.1.2278

package/docs/deprecated.json

@@ -223,6 +223,8 @@
     "id": "ado-throttle-arg-less-shim",
     "description": "Arg-less form of isAdoThrottled() in engine/ado.js. Introduced by W-mq03l6zh0006f0a1-b as a back-compat shim during the per-org ADO throttle isolation rollout: pre-rollout, isAdoThrottled() collapsed the single process-global tracker to one boolean; post-rollout, the canonical form is isAdoThrottled(orgBase) against the per-org Map. The arg-less call site is preserved transiently so engine code (and any in-process callers) that haven't yet been threaded through with a per-org `orgBase` keep returning the safe global-OR (true if ANY org is currently throttled) — preventing a regression where new poll work bypasses a still-warm throttle backoff on an unrelated noisy org.",
     "deprecated": "2026-06-04",
+    "status": "removed",
+    "removedDate": "2026-06-25",
     "code": [
       {
         "file": "engine/ado.js",

package/engine/ado.js

@@ -2630,18 +2630,13 @@ async function fetchSinglePrBuildStatus(project, prNumber) {
 
 // ─── ADO Throttle Queries ────────────────────────────────────────────────────
 
-/** Returns true if ADO is throttled. If orgBase is provided, checks that org's
- *  tracker only; if omitted, returns true when ANY tracked org is throttled
- *  (back-compat OR semantics for existing call sites). Auto-clears stale state. */
+/** Returns true if the given org is currently throttled. Auto-clears stale state.
+ *  orgBase is required — callers must pass shared.getAdoOrgBase(project).
+ *  (The arg-less back-compat OR shim was removed: docs/deprecated.json id: ado-throttle-arg-less-shim) */
 const isAdoThrottled = (orgBase) => {
-  if (orgBase != null) {
-    const tracker = _adoThrottlesByOrg.get(canonicalAdoOrgKey(orgBase));
-    return tracker ? tracker.isThrottled() : false;
-  }
-  for (const tracker of _adoThrottlesByOrg.values()) {
-    if (tracker.isThrottled()) return true;
-  }
-  return false;
+  if (orgBase == null) throw new TypeError('isAdoThrottled requires an orgBase argument');
+  const tracker = _adoThrottlesByOrg.get(canonicalAdoOrgKey(orgBase));
+  return tracker ? tracker.isThrottled() : false;
 };
 
 /** Returns a snapshot of the throttle state.

package/engine/lifecycle.js

@@ -5353,6 +5353,8 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
       try { stampPrdItemWorkItemId(meta.item.id, meta.item.sourcePlan); } catch (err) { log('warn', `stampPrdItemWorkItemId: ${err.message}`); }
     }
     promoteCompletionArtifacts(meta, agentId, dispatchItem.id, structuredCompletion, { resultSummary });
+    // M003 — auto-dispatch a live-validation WI when a coding WI completes.
+    try { autoDispatchLiveValidationWi(meta, config); } catch (err) { log('warn', `autoDispatchLiveValidationWi: ${err.message}`); }
   }
   // Failure retry is handled by completeDispatch in dispatch.js — not duplicated here.
   // Only clear _decomposing flag on failure so decompose items don't get permanently stuck.
@@ -6124,6 +6126,78 @@ function pruneScopeMismatchDuplicatePrs(config) {
   return { pruned, scanned };
 }
 
+// M003 — After a coding WI completes successfully, auto-dispatch a live-validation
+// WI when project.liveValidation.autoDispatch === true and the completed item is
+// a coding WI (not the validation type itself). Skips if the coding WI has no PR.
+// Deduplicates: a non-terminal WI with meta.liveValidationFor === codingWiId
+// blocks a second dispatch.
+function autoDispatchLiveValidationWi(meta, config) {
+  const item = meta?.item;
+  if (!item?.id || !item?.type) return;
+
+  const projectName = meta?.project?.name;
+  if (!projectName) return;
+
+  // Look up the full project config (which carries liveValidation) from config.
+  const projects = shared.getProjects(config || {});
+  const project = projects.find(p => p && p.name === projectName);
+  if (!project) return;
+
+  const lv = project.liveValidation;
+  if (!lv || lv.autoDispatch !== true || !lv.type) return;
+
+  // Only auto-dispatch for coding WIs — skip if this IS the validation type.
+  if (item.type === lv.type) return;
+
+  // Resolve PR reference: prefer the canonical stamped _pr field (set by
+  // stampWiPrRef which runs earlier in runPostCompletionHooks), then fall
+  // back to extractWorkItemPrRef for structured fields / references[].
+  const prRef = item._pr || item._prUrl || shared.extractWorkItemPrRef(item) || null;
+  if (!prRef) return;
+
+  const codingWiId = item.id;
+  const wiPath = resolveWorkItemPath(meta);
+  if (!wiPath) return;
+
+  try {
+    mutateWorkItems(wiPath, items => {
+      if (!Array.isArray(items)) return items;
+
+      // Dedup: skip if a non-terminal WI already tracks this coding WI.
+      const existing = items.find(i =>
+        i &&
+        i.meta &&
+        i.meta.liveValidationFor === codingWiId &&
+        !PLAN_TERMINAL_STATUSES.has(i.status)
+      );
+      if (existing) {
+        log('info', `liveValidation: dedup — non-terminal validation WI ${existing.id} already exists for ${codingWiId}`);
+        return items;
+      }
+
+      const validationWi = {
+        id: 'W-' + shared.uid(),
+        title: 'Validate: ' + (item.title || codingWiId),
+        type: lv.type,
+        status: WI_STATUS.PENDING,
+        depends_on: [codingWiId],
+        references: [prRef],
+        meta: { liveValidationFor: codingWiId },
+        project: projectName,
+        priority: item.priority || 'medium',
+        created: ts(),
+        createdBy: 'lifecycle:live-validation-auto-dispatch',
+      };
+
+      items.push(validationWi);
+      log('info', `liveValidation: auto-dispatched validation WI ${validationWi.id} (type=${lv.type}) for coding WI ${codingWiId}`);
+      return items;
+    });
+  } catch (err) {
+    log('warn', `autoDispatchLiveValidationWi for ${codingWiId}: ${err.message}`);
+  }
+}
+
 module.exports = {
   checkPlanCompletion,
   archivePlan,
@@ -6198,4 +6272,6 @@ module.exports = {
   // W-mqtplpk6001oe6d5 — stamp PR ref + workItemId onto WI and PRD item at completion time.
   stampWiPrRef,
   stampPrdItemWorkItemId,
+  // M003 — auto-dispatch live-validation WI after coding WI completion.
+  autoDispatchLiveValidationWi,
 };

package/engine.js

@@ -6700,7 +6700,7 @@ async function discoverFromPrs(config, project) {
     ? (config.engine?.adoPollEnabled ?? ENGINE_DEFAULTS.adoPollEnabled)
     : (config.engine?.ghPollEnabled ?? ENGINE_DEFAULTS.ghPollEnabled));
   const evalLoopEnabled = config.engine?.evalLoop !== false;
-  const fixThrottled = isAdoProject ? isAdoThrottled() : isGhThrottled();
+  const fixThrottled = isAdoProject ? isAdoThrottled(shared.getAdoOrgBase(project)) : isGhThrottled();
   const autoReviewPrs = config.engine?.autoReviewPrs ?? ENGINE_DEFAULTS.autoReviewPrs;
   const autoReReviewPrs = config.engine?.autoReReviewPrs ?? ENGINE_DEFAULTS.autoReReviewPrs;
   // P-b2e5d8c7: engine.autoFixPaused is a hard-stop master switch over every

package/package.json

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