@yemi33/minions 0.1.447 → 0.1.449

package/CHANGELOG.md

@@ -1,11 +1,13 @@
 # Changelog
 
-## 0.1.447 (2026-04-06)
+## 0.1.449 (2026-04-06)
 
 ### Features
 -  notification dot on Notes & KB sidebar when sweep completes/fails
 
 ### Fixes
+-  track ADO head commit for push detection — unblocks re-review cycle
+-  block all fix dispatches while awaiting re-review after a fix
 -  prevent double-dispatch — dedup by work item ID and dispatchKey
 -  pipeline agent is optional — set only when user explicitly requests
 -  remove hardcoded agent assignments from all pipeline JSONs

package/engine/ado.js

@@ -166,6 +166,16 @@ async function pollPrStatus(config) {
       }
     }
 
+    // Track head commit changes to detect new pushes (used for review re-dispatch gating)
+    const headCommit = prData.lastMergeSourceCommit?.commitId || prData.sourceRefName || '';
+    if (headCommit && pr._adoHeadCommit !== headCommit) {
+      if (pr._adoHeadCommit) { // skip first detection — only track changes
+        pr.lastPushedAt = new Date().toISOString();
+      }
+      pr._adoHeadCommit = headCommit;
+      updated = true;
+    }
+
     const reviewers = prData.reviewers || [];
     const votes = reviewers.map(r => r.vote).filter(v => v !== undefined);
     let newReviewStatus = pr.reviewStatus || 'pending';

package/engine.js

@@ -1279,6 +1279,10 @@ function discoverFromPrs(config, project) {
     // minionsReview tracks metadata (reviewer, note) but not the authoritative status
     const reviewStatus = pr.reviewStatus || 'pending';
 
+    // Skip fix dispatch if a fix was recently submitted and awaiting re-review.
+    // The poller holds reviewStatus at 'waiting' until the reviewer acts on the new code.
+    const awaitingReReview = reviewStatus === 'waiting' && !!pr.minionsReview?.fixedAt;
+
     // PRs needing review: pending review status and not already reviewed without new commits
     const autoReview = config.engine?.autoReview !== false;
     // Skip re-dispatch if already reviewed and no new commits pushed since last review
@@ -1298,7 +1302,7 @@ function discoverFromPrs(config, project) {
     }
 
     // PRs with changes requested → route back to author for fix
-    if (reviewStatus === 'changes-requested') {
+    if (reviewStatus === 'changes-requested' && !awaitingReReview) {
       const key = `fix-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
       const agentId = resolveAgent('fix', config, pr.agent);
@@ -1314,7 +1318,7 @@ function discoverFromPrs(config, project) {
     // 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) {
+    if ((pr.humanFeedback?.pendingFix || hasCoalescedFeedback) && !awaitingReReview) {
       const key = humanFixKey;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) {
         // Coalesce: save feedback for next dispatch

package/package.json

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