@yemi33/minions 0.1.693 → 0.1.695

package/CHANGELOG.md

@@ -1,8 +1,10 @@
 # Changelog
 
-## 0.1.693 (2026-04-09)
+## 0.1.695 (2026-04-09)
 
 ### Fixes
+-  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
 
 ## 0.1.692 (2026-04-09)

package/engine/ado.js

@@ -99,16 +99,32 @@ 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) {
+async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr) {
   try {
-    // Extract buildId from the targetUrl (e.g. .../_build/results?buildId=12345)
+    // Try extracting buildId from targetUrl (e.g. .../_build/results?buildId=12345)
     const targetUrl = failedStatus?.targetUrl || '';
+    let buildId = null;
     const buildIdMatch = targetUrl.match(/buildId=(\d+)/);
-    if (!buildIdMatch) {
-      log('debug', `No buildId in targetUrl: ${targetUrl.slice(0, 120)}`);
+    if (buildIdMatch) {
+      buildId = buildIdMatch[1];
+    } else {
+      // Fallback: query recent failed builds for this PR's source branch
+      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 builds = await adoFetch(buildsUrl, token);
+          if (builds?.value?.[0]?.id) {
+            buildId = String(builds.value[0].id);
+            log('debug', `Found buildId ${buildId} via branch query for ${branch}`);
+          }
+        }
+      } catch (e) { log('debug', `Branch-based build lookup failed: ${e.message}`); }
+    }
+    if (!buildId) {
+      log('debug', `No buildId from targetUrl or branch query: ${targetUrl.slice(0, 120)}`);
       return null;
     }
-    const buildId = buildIdMatch[1];
 
     // Fetch build timeline to find failed tasks
     const timelineUrl = `${orgBase}/${project.adoProject}/_apis/build/builds/${buildId}/timeline?api-version=7.1`;
@@ -375,7 +391,7 @@ 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);
+        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`);

package/package.json

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