@yemi33/minions 0.1.945 → 0.1.947

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## 0.1.945 (2026-04-14)
+## 0.1.947 (2026-04-14)
 
 ### Features
 -  make ADO poll frequency configurable and ungate reconcilePrs
@@ -47,6 +47,8 @@
 - prioritize error_max_turns over permission-blocked in classifyFailure (#1001)
 
 ### Other
+- refactor(ado): simplify comments, fix declaration order, promote ado import
+- refactor(ado): move PR enrichment fetch into fetchAdoPrMetadata helper
 - refactor: Make renderLiveChatMessage a thin wrapper over renderAgentOutput
 - test(preflight): add unit tests for findClaudeBinary, runPreflight, printPreflight, checkOrExit (#953)
 - test(meeting): add unit tests for key functions (#957)

package/dashboard.js

@@ -21,6 +21,7 @@ const _dashboardVersion = {
 const shared = require('./engine/shared');
 const queries = require('./engine/queries');
 const teams = require('./engine/teams');
+const ado = require('./engine/ado');
 const os = require('os');
 
 const { safeRead, safeReadDir, safeWrite, safeJson, safeJsonObj, safeJsonArr, safeUnlink, mutateJsonFileLocked, mutateWorkItems, getProjects: _getProjects, DONE_STATUSES, WI_STATUS, reopenWorkItem } = shared;
@@ -4228,14 +4229,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           } else if (adoMatch) {
             const [, adoOrg, adoProj, adoRepo] = adoMatch;
             try {
-              const { getAdoToken } = require('./engine/ado');
-              const token = await getAdoToken();
-              if (token) {
-                const apiUrl = `https://dev.azure.com/${adoOrg}/${adoProj}/_apis/git/repositories/${adoRepo}/pullrequests/${prNum}?api-version=7.1`;
-                const result = await shared.execAsync(`curl -s --max-time 10 -H "Authorization: Bearer ${token}" "${apiUrl}"`, { encoding: 'utf-8', timeout: 15000, windowsHide: true });
-                const d = JSON.parse(result);
-                prData = { title: d.title, description: d.description, branch: d.sourceRefName?.replace('refs/heads/', ''), author: d.createdBy?.displayName };
-              }
+              prData = await ado.fetchAdoPrMetadata(prNum, adoOrg, adoProj, adoRepo);
             } catch { /* ADO token may not be available */ }
           }
           if (!prData) return;

package/engine/ado.js

@@ -371,19 +371,17 @@ async function pollPrStatus(config) {
 
     if (newStatus !== PR_STATUS.ACTIVE) return updated;
 
-    // Query builds API directly — status checks (/statuses) are unreliable (stale codecoverage postbacks)
-    // Use refs/pull/{id}/merge — the synthetic merge ref ADO creates for each PR — to scope builds exactly
+    // Query builds API directly — /statuses is unreliable (stale codecoverage postbacks).
+    // refs/pull/{id}/merge scopes server-side to this PR; sourceVersion narrows to the current
+    // merge commit (same ref accumulates builds across all prior pushes to the PR).
     const prNumber = pr.prNumber;
+    const mergeCommitId = prData.lastMergeCommit?.commitId;
     let buildStatus = 'none';
     let buildFailReason = '';
     let buildStatuses = []; // for error log fetching
 
-    const mergeCommitId = prData.lastMergeCommit?.commitId;
     if (prNumber && mergeCommitId) {
       try {
-        // branchName=refs/pull/{id}/merge scopes server-side to this PR's builds.
-        // sourceVersion filter then narrows to the current merge commit — a PR updated
-        // multiple times accumulates builds on the same ref across different commits.
         const mergeRef = encodeURIComponent(`refs/pull/${prNumber}/merge`);
         const buildsUrl = `${orgBase}/${project.adoProject}/_apis/build/builds?branchName=${mergeRef}&repositoryId=${project.repositoryId}&repositoryType=TfsGit&$top=25&api-version=7.1`;
         const buildsData = await adoFetch(buildsUrl, token);
@@ -768,6 +766,20 @@ async function checkLiveReviewStatus(pr, project) {
   }
 }
 
+async function fetchAdoPrMetadata(prNum, adoOrg, adoProj, adoRepo) {
+  const token = await getAdoToken();
+  if (!token) return null;
+  const url = `https://dev.azure.com/${adoOrg}/${adoProj}/_apis/git/repositories/${adoRepo}/pullrequests/${prNum}?api-version=7.1`;
+  const pr = await adoFetch(url, token);
+  if (!pr) return null;
+  return {
+    title: pr.title || '',
+    description: pr.description || '',
+    branch: pr.sourceRefName?.replace('refs/heads/', '') || '',
+    author: pr.createdBy?.displayName || '',
+  };
+}
+
 module.exports = {
   getAdoToken,
   adoFetch,
@@ -777,5 +789,6 @@ module.exports = {
   checkLiveReviewStatus,
   needsAdoPollRetry,
   isAdoAuthError, // exported for testing
+  fetchAdoPrMetadata,
 };
 

package/package.json

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