@yemi33/minions 0.1.972 → 0.1.974

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.972 (2026-04-14)
+## 0.1.974 (2026-04-14)
 
 ### Features
 -  flush queued CC messages as single combined request
@@ -25,6 +25,7 @@
 - replace magic string 'active' with PR_STATUS.ACTIVE in lifecycle.js (#969)
 
 ### Fixes
+-  skip isAlreadyDispatched in needsReReview to allow re-review within 1hr
 -  loop queue flush so messages queued during combined send aren't orphaned
 -  set lastPushedAt on fix completion to unblock re-review without poller
 -  trigger second-pass re-review after fix agent completes
@@ -44,7 +45,6 @@
 - tighten build query guard to require mergeCommitId
 - restore sourceVersion filter after refs/pull merge ref switch
 - use refs/pull/{id}/merge for build status scoping
-- use repositoryId+reasonFilter for ADO build query
 
 ### Other
 - refactor(dashboard): extract _releaseCCTab helper and CC_LOCK_WAIT_MS constant

package/engine.js

@@ -1984,14 +1984,17 @@ async function discoverFromPrs(config, project) {
       if (item) { newWork.push(item); }
     }
 
-    // Re-review after fix: fall back to fixedAt > lastReviewedAt when lastPushedAt lags poller
+    // Re-review after fix: only trigger when fixedAt > lastReviewedAt (not a broad !alreadyReviewed
+    // fallback — that caused infinite re-review loops on GitHub where self-approval is blocked)
     const fixedAfterReview = !!(pr.minionsReview?.fixedAt &&
       pr.lastReviewedAt && pr.minionsReview.fixedAt > pr.lastReviewedAt);
     const needsReReview = autoReview && reviewStatus === 'waiting' &&
-      (fixedAfterReview || (!alreadyReviewed && !!pr.minionsReview?.fixedAt)) && !evalEscalated;
+      fixedAfterReview && !evalEscalated;
     if (needsReReview) {
       const key = `review-${project?.name || 'default'}-${pr.id}`;
-      if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
+      // Skip isAlreadyDispatched — fixedAfterReview/alreadyReviewed already dedup; the 1hr
+      // completed-dispatch window would block legitimate re-reviews within the hour after a fix
+      if (isOnCooldown(key, cooldownMs)) continue;
 
       // Pre-dispatch live vote check — cached 'waiting' may be stale if reviewer already acted
       try {

package/package.json

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