@yemi33/minions 0.1.701 → 0.1.703

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.703 (2026-04-09)
+
+### Other
+- refactor: move buildFixGracePeriod to ENGINE_DEFAULTS (no magic numbers)
+
+## 0.1.702 (2026-04-09)
+
+### Fixes
+-  build fix grace period — wait for CI before re-dispatching
+
 ## 0.1.701 (2026-04-09)
 
 ### Fixes

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/shared.js

@@ -544,6 +544,7 @@ const ENGINE_DEFAULTS = {
   lockRetries: 2, // retry lock acquisition this many times after initial timeout (total attempts = 1 + lockRetries)
   lockRetryBackoffMs: 500, // base backoff between lock retries (doubles each attempt: 500ms, 1s, 2s, ...)
   maxBuildFixAttempts: 3, // max consecutive auto-fix dispatch cycles per PR before escalation to human
+  buildFixGracePeriod: 600000, // 10min — wait for CI to run after build fix before re-dispatching
   ccModel: 'sonnet', // model for Command Center and doc-chat (sonnet, haiku, opus)
   ccEffort: null, // effort level for CC/doc-chat (null, 'low', 'medium', 'high')
 };

package/engine.js

@@ -1601,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 ?? ENGINE_DEFAULTS.buildFixGracePeriod;
+      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;
 
@@ -1646,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.701",
+  "version": "0.1.703",
   "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"