@yemi33/minions 0.1.700 → 0.1.702

package/CHANGELOG.md

@@ -1,8 +1,14 @@
 # Changelog
 
-## 0.1.700 (2026-04-09)
+## 0.1.702 (2026-04-09)
 
 ### Fixes
+-  build fix grace period — wait for CI before re-dispatching
+
+## 0.1.701 (2026-04-09)
+
+### Fixes
+-  prevent duplicate fix dispatch when review-fix and human-feedback fire on same tick
 -  PRD E2E section not updating when pullRequests change
 -  ADO build error log fetches all failing pipelines, not just first
 -  verify/implement playbooks must include PR URL in final message

package/engine/ado.js

@@ -383,6 +383,8 @@ async function pollPrStatus(config) {
       pr.buildStatus = buildStatus;
       if (buildFailReason) pr.buildFailReason = buildFailReason;
       else delete pr.buildFailReason;
+      // Build transitioned — clear grace period so next failure can trigger immediately
+      delete pr._buildFixPushedAt;
       if (buildStatus !== 'failing') {
         delete pr._buildFailNotified;
         delete pr.buildErrorLog;

package/engine/github.js

@@ -393,6 +393,8 @@ async function pollPrStatus(config) {
           pr.buildStatus = buildStatus;
           if (buildFailReason) pr.buildFailReason = buildFailReason;
           else delete pr.buildFailReason;
+          // Build transitioned — clear grace period so next failure can trigger immediately
+          delete pr._buildFixPushedAt;
           if (buildStatus !== 'failing') {
             delete pr._buildFailNotified;
             delete pr.buildErrorLog;

package/engine.js

@@ -1556,6 +1556,7 @@ async function discoverFromPrs(config, project) {
     }
 
     // PRs with changes requested → route back to author for fix
+    let fixDispatched = false;
     if (reviewStatus === 'changes-requested' && !awaitingReReview) {
       const key = `fix-${project?.name || 'default'}-${pr.id}`;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) continue;
@@ -1566,13 +1567,13 @@ async function discoverFromPrs(config, project) {
         pr_id: pr.id, pr_branch: pr.branch || '',
         review_note: pr.minionsReview?.note || pr.reviewNote || 'See PR thread comments',
       }, `Fix PR ${pr.id} review feedback`, { dispatchKey: key, source: 'pr', pr, branch: pr.branch, project: projMeta });
-      if (item) { newWork.push(item); setCooldown(key); }
+      if (item) { newWork.push(item); setCooldown(key); fixDispatched = true; }
     }
 
-    // PRs with pending human feedback (or coalesced comments from while agent was fixing)
+    // PRs with pending human feedback (skip if review-fix already dispatched above)
     const humanFixKey = `human-fix-${project?.name || 'default'}-${pr.id}`;
     const hasCoalescedFeedback = (dispatchCooldowns.get(humanFixKey)?.pendingContexts || []).length > 0;
-    if ((pr.humanFeedback?.pendingFix || hasCoalescedFeedback) && !awaitingReReview) {
+    if ((pr.humanFeedback?.pendingFix || hasCoalescedFeedback) && !awaitingReReview && !fixDispatched) {
       const key = humanFixKey;
       if (isAlreadyDispatched(key) || isOnCooldown(key, cooldownMs)) {
         // Coalesce: save feedback for next dispatch
@@ -1600,6 +1601,12 @@ async function discoverFromPrs(config, project) {
     }
 
     // PRs with build failures — route to author (has session context from implementing)
+    // Grace period: after a build fix push, wait for CI to run before re-dispatching
+    // Skip if build hasn't transitioned since last fix (still showing the old failure)
+    if (pr._buildFixPushedAt && pr.buildStatus === 'failing') {
+      const gracePeriodMs = config.engine?.buildFixGracePeriod ?? 600000; // 10 min default
+      if (Date.now() - new Date(pr._buildFixPushedAt).getTime() < gracePeriodMs) continue;
+    }
     if (pr.status === PR_STATUS.ACTIVE && pr.buildStatus === 'failing') {
       const maxBuildFix = config.engine?.maxBuildFixAttempts ?? ENGINE_DEFAULTS.maxBuildFixAttempts;
 
@@ -1645,7 +1652,10 @@ async function discoverFromPrs(config, project) {
           const prPath = projectPrPath(project);
           mutatePullRequests(prPath, prs => {
             const target = prs.find(p => p.id === pr.id);
-            if (target) target.buildFixAttempts = (target.buildFixAttempts || 0) + 1;
+            if (target) {
+              target.buildFixAttempts = (target.buildFixAttempts || 0) + 1;
+              target._buildFixPushedAt = ts();
+            }
           });
         } catch (e) { log('warn', 'increment build fix attempts: ' + e.message); }
       }

package/package.json

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