@yemi33/minions 0.1.1105 → 0.1.1106

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.1105 (2026-04-18)
+## 0.1.1106 (2026-04-18)
 
 ### Features
 - seed realActivityMap at spawn time, stamp pid in live-output (#1200)
 
 ### Fixes
+- preserve buildErrorLog through transient build states (#1232) (#1274)
 - annotate fast-exit empty-output failures with diagnostic hint (#1276)
 - invalidate PRD cache on pr-links.json change + guard aggregate PRs (#1220) (#1272)
 - pass --add-dir for minions + ~/.claude to agents (#1271)

package/engine/ado.js

@@ -465,6 +465,11 @@ async function pollPrStatus(config) {
       } catch (e) { log('warn', `ADO build query for ${pr.id}: ${e.message}`); }
     }
 
+    // Record actual poll time — makes lastBuildCheck reflect when the engine last
+    // talked to ADO, not when the agent was dispatched. Issue #1233.
+    pr.lastBuildCheck = ts();
+    updated = true;
+
     if (pr.buildStatus !== buildStatus) {
       log('info', `PR ${pr.id} build: ${pr.buildStatus || 'none'} → ${buildStatus}${buildFailReason ? ' (' + buildFailReason + ')' : ''}`);
       pr.buildStatus = buildStatus;
@@ -475,9 +480,17 @@ async function pollPrStatus(config) {
       if (buildStatus === 'failing') delete pr._autoCompleted;
       if (buildStatus !== 'failing') {
         delete pr._buildFailNotified;
-        delete pr.buildErrorLog;
-        // Reset build fix retry counter on recovery — allows fresh auto-fix cycles if build breaks again
-        if (pr.buildFixAttempts) { delete pr.buildFixAttempts; delete pr.buildFixEscalated; }
+        // Preserve buildErrorLog + buildFixAttempts through transient 'none'/'running'
+        // transitions — only clear on confirmed 'passing' recovery. Issue #1232: 'none'
+        // can also occur when ADO recomputes the merge commit after a target-branch
+        // update but no new builds have been triggered yet (filter by sourceVersion
+        // returns []), which previously wiped the last known error log and caused
+        // fix agents to be dispatched blind.
+        if (buildStatus === 'passing') {
+          delete pr.buildErrorLog;
+          // Reset build fix retry counter on recovery — allows fresh auto-fix cycles if build breaks again
+          if (pr.buildFixAttempts) { delete pr.buildFixAttempts; delete pr.buildFixEscalated; }
+        }
       }
       updated = true;
 

package/engine/github.js

@@ -422,6 +422,10 @@ async function pollPrStatus(config) {
     if (prData.state === 'open' && prData.head?.sha) {
       const checksData = await ghApi(`/commits/${prData.head.sha}/check-runs`, slug);
       if (checksData && checksData.check_runs) {
+        // Record actual poll time — makes lastBuildCheck reflect when the engine last
+        // talked to GitHub, not when the agent was dispatched. Issue #1233.
+        pr.lastBuildCheck = ts();
+        updated = true;
         const runs = checksData.check_runs;
         let buildStatus = 'none';
         let buildFailReason = '';
@@ -452,9 +456,15 @@ async function pollPrStatus(config) {
           if (buildStatus === 'failing') delete pr._autoCompleted; // allow re-merge after fix
           if (buildStatus !== 'failing') {
             delete pr._buildFailNotified;
-            delete pr.buildErrorLog;
-            // Reset build fix retry counter on recovery — allows fresh auto-fix cycles if build breaks again
-            if (pr.buildFixAttempts) { delete pr.buildFixAttempts; delete pr.buildFixEscalated; }
+            // Preserve buildErrorLog + buildFixAttempts through transient 'none'/'running'
+            // transitions — only clear on confirmed 'passing' recovery. Issue #1232:
+            // clearing on every non-failing transition blinded the next fix agent
+            // while a queued build was still running.
+            if (buildStatus === 'passing') {
+              delete pr.buildErrorLog;
+              // Reset build fix retry counter on recovery — allows fresh auto-fix cycles if build breaks again
+              if (pr.buildFixAttempts) { delete pr.buildFixAttempts; delete pr.buildFixEscalated; }
+            }
           }
           updated = true;
 

package/package.json

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