@yemi33/minions 0.1.694 → 0.1.696

package/CHANGELOG.md

@@ -1,8 +1,10 @@
 # Changelog
 
-## 0.1.694 (2026-04-09)
+## 0.1.696 (2026-04-09)
 
 ### Fixes
+-  ADO build error log fetches all failing pipelines, not just first
+-  verify/implement playbooks must include PR URL in final message
 -  ADO build error log fallback when targetUrl lacks buildId
 -  syncPrsFromOutput missed PRs mentioned in assistant text blocks
 

package/engine/ado.js

@@ -99,7 +99,7 @@ const BUILD_ERROR_LOG_MAX_LINES = 150;
  * for failed tasks, and fetches their logs.
  * Returns truncated log text or null if unavailable.
  */
-async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr) {
+async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr, seenBuildIds) {
   try {
     // Try extracting buildId from targetUrl (e.g. .../_build/results?buildId=12345)
     const targetUrl = failedStatus?.targetUrl || '';
@@ -112,10 +112,11 @@ async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr)
       try {
         const branch = pr?.branch || pr?.sourceRefName?.replace('refs/heads/', '');
         if (branch) {
-          const buildsUrl = `${orgBase}/${project.adoProject}/_apis/build/builds?branchName=refs/heads/${encodeURIComponent(branch)}&statusFilter=completed&resultFilter=failed&$top=1&api-version=7.1`;
+          const buildsUrl = `${orgBase}/${project.adoProject}/_apis/build/builds?branchName=refs/heads/${encodeURIComponent(branch)}&statusFilter=completed&resultFilter=failed&$top=3&api-version=7.1`;
           const builds = await adoFetch(buildsUrl, token);
-          if (builds?.value?.[0]?.id) {
-            buildId = String(builds.value[0].id);
+          const fresh = (builds?.value || []).find(b => !seenBuildIds?.has(String(b.id)));
+          if (fresh?.id) {
+            buildId = String(fresh.id);
             log('debug', `Found buildId ${buildId} via branch query for ${branch}`);
           }
         }
@@ -125,6 +126,8 @@ async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr)
       log('debug', `No buildId from targetUrl or branch query: ${targetUrl.slice(0, 120)}`);
       return null;
     }
+    if (seenBuildIds?.has(buildId)) return null; // already fetched this build
+    if (seenBuildIds) seenBuildIds.add(buildId);
 
     // Fetch build timeline to find failed tasks
     const timelineUrl = `${orgBase}/${project.adoProject}/_apis/build/builds/${buildId}/timeline?api-version=7.1`;
@@ -390,11 +393,16 @@ async function pollPrStatus(config) {
 
       // Fetch actual compiler/build error logs when transitioning to failing
       if (buildStatus === 'failing') {
-        const failedStatusObj = buildStatuses.find(s => s.state === 'failed' || s.state === 'error');
-        const errorLog = await fetchAdoBuildErrorLog(orgBase, project, failedStatusObj, token, pr);
-        if (errorLog) {
-          pr.buildErrorLog = errorLog;
-          log('info', `PR ${pr.id}: fetched ${errorLog.split('\n').length} lines of build error log`);
+        const failedStatusObjs = buildStatuses.filter(s => s.state === 'failed' || s.state === 'error').slice(0, 3);
+        const logParts = [];
+        const seenBuildIds = new Set();
+        for (const failedStatusObj of failedStatusObjs) {
+          const errorLog = await fetchAdoBuildErrorLog(orgBase, project, failedStatusObj, token, pr, seenBuildIds);
+          if (errorLog) logParts.push(errorLog);
+        }
+        if (logParts.length > 0) {
+          pr.buildErrorLog = logParts.join('\n\n');
+          log('info', `PR ${pr.id}: fetched error logs from ${logParts.length} failing pipeline(s)`);
         }
       }
     }

package/package.json

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

package/playbooks/implement.md

@@ -81,4 +81,4 @@ Include build/test status and run instructions in the PR description. If the pro
 
 ## When to Stop
 
-Your task is complete once you have: (1) confirmed build and tests pass, (2) pushed your branch, and (3) created the PR. Do NOT continue exploring, refactoring, or adding features beyond the task description. Stop immediately.
+Your task is complete once you have: (1) confirmed build and tests pass, (2) pushed your branch, and (3) created the PR. Your final message MUST include the PR URL so the engine can track it. Stop immediately after.

package/playbooks/verify.md

@@ -153,4 +153,13 @@ Use subagents only for genuinely parallel, independent tasks (e.g., building sep
 
 ## When to Stop
 
-Your task is complete once you have: (1) merged dependency branches, (2) built and tested, (3) written the verification report to both locations, and (4) created the E2E PR(s). Stop after creating PRs.
+Your task is complete once you have: (1) merged dependency branches, (2) built and tested, (3) written the verification report to both locations, and (4) created the E2E PR(s).
+
+**IMPORTANT: Your final message MUST include the E2E PR URL(s) so the engine can track them.** Example final message:
+
+```
+Verification complete. E2E PR created: https://github.com/org/repo/pull/123
+Testing guide saved to prd/guides/verify-plan-name.md
+```
+
+Stop after confirming the PR was created.