azdo-cli 0.10.0-develop.467 → 0.10.0-develop.479

package/README.md

@@ -58,7 +58,7 @@ azdo pr comments                        # active-branch PR
 azdo pr comments --pr-number 64         # any PR by number (skips branch lookup)
 azdo pr comments --pr-number 64 --hide-resolved      # or --exclude-resolved (alias)
 azdo pr comments --code-related-only    # only file/line-anchored threads
-azdo pr status                          # PR checks (status + branch policies) + code-comment counts
+azdo pr status                          # PR checks (status + branch policies + pipeline builds) + code-comment counts
 azdo pr comment-resolve 17 --pr-number 64   # idempotent: exit 0 even when already resolved
 azdo pr comment-reopen 17  --pr-number 64
 

package/dist/index.js

@@ -2907,6 +2907,16 @@ function buildPolicyEvaluationsUrl(context, projectId, prId) {
   url.searchParams.set("artifactId", `vstfs:///CodeReview/CodeReviewId/${projectId}/${prId}`);
   return url;
 }
+function buildPullRequestBuildsUrl(context, prId) {
+  const url = new URL(
+    `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/build/builds`
+  );
+  url.searchParams.set("branchName", `refs/pull/${prId}/merge`);
+  url.searchParams.set("queryOrder", "queueTimeDescending");
+  url.searchParams.set("$top", "50");
+  url.searchParams.set("api-version", "7.1");
+  return url;
+}
 function mapPullRequest(repo, pullRequest) {
   return {
     id: pullRequest.pullRequestId,
@@ -2966,6 +2976,22 @@ function mapPolicyEvaluationState(status2) {
       return status2;
   }
 }
+function mapBuildToCheckState(build) {
+  if (build.status !== "completed") {
+    return "pending";
+  }
+  switch (build.result) {
+    case "succeeded":
+    case "partiallySucceeded":
+      return "succeeded";
+    case "failed":
+      return "failed";
+    case "canceled":
+      return "error";
+    default:
+      return "pending";
+  }
+}
 function mapPolicyEvaluationName(evaluation) {
   const display = evaluation.configuration?.settings?.displayName?.trim() || evaluation.configuration?.type?.displayName?.trim();
   if (display) {
@@ -2987,7 +3013,8 @@ function mapPolicyEvaluationCheck(evaluation) {
     createdBy: null,
     createdAt: null,
     updatedAt: null,
-    source: "policy"
+    source: "policy",
+    isBlocking: evaluation.configuration?.isBlocking ?? null
   };
 }
 function mapComment(comment) {
@@ -3009,7 +3036,7 @@ function mapThread(thread) {
   }
   return {
     id: thread.id,
-    status: thread.status,
+    status: thread.status ?? "unknown",
     threadContext: thread.threadContext?.filePath ?? null,
     comments
   };
@@ -3017,7 +3044,7 @@ function mapThread(thread) {
 function toActiveCommentThread(thread) {
   return {
     id: thread.id,
-    status: thread.status,
+    status: thread.status ?? "unknown",
     threadContext: thread.threadContext?.filePath ?? null,
     comments: thread.comments.map(mapComment).filter((comment) => comment !== null)
   };
@@ -3088,6 +3115,24 @@ async function getPullRequestPolicyEvaluations(context, cred, projectId, prId) {
   const data = await readJsonResponse(response);
   return data.value.map(mapPolicyEvaluationCheck).filter((check) => check !== null);
 }
+async function getPullRequestBuilds(context, cred, prId) {
+  const response = await fetchWithErrors(buildPullRequestBuildsUrl(context, prId).toString(), {
+    headers: authHeaders(cred)
+  });
+  const data = await readJsonResponse(response);
+  return data.value.map((build) => ({
+    id: build.id,
+    state: mapBuildToCheckState(build),
+    name: build.definition?.name ?? `Build #${build.id}`,
+    description: null,
+    targetUrl: build._links?.web?.href ?? null,
+    createdBy: null,
+    createdAt: build.queueTime ?? null,
+    updatedAt: build.finishTime ?? null,
+    source: "build",
+    isBlocking: null
+  }));
+}
 async function openPullRequest(context, repo, cred, sourceBranch, title, description) {
   const existing = await listPullRequests(context, repo, cred, sourceBranch, {
     status: "active",
@@ -3205,7 +3250,8 @@ function formatPullRequestChecks(checks, checksError) {
   }
   const lines = ["Checks:"];
   for (const check of checks) {
-    lines.push(`- [${check.state}] ${check.name}`);
+    const optionalTag = check.isBlocking === false ? " [optional]" : "";
+    lines.push(`- [${check.state}] ${check.name}${optionalTag}`);
     if ((check.state === "failed" || check.state === "error") && check.description) {
       lines.push(`  Detail: ${check.description}`);
     }
@@ -3258,6 +3304,13 @@ async function buildPullRequestStatusEntry(context, repo, cred, pullRequest, pro
       policyOk = false;
     }
   }
+  let buildChecks = [];
+  let buildsOk = true;
+  try {
+    buildChecks = await getPullRequestBuilds(context, cred, pullRequest.id);
+  } catch {
+    buildsOk = false;
+  }
   let codeCommentCounts;
   try {
     const threads = await getPullRequestThreads(context, repo, cred, pullRequest.id);
@@ -3265,8 +3318,8 @@ async function buildPullRequestStatusEntry(context, repo, cred, pullRequest, pro
   } catch {
     codeCommentCounts = { open: 0, closed: 0 };
   }
-  const checks = [...statusChecks, ...policyChecks];
-  const checksError = checks.length === 0 && (!statusOk || !policyOk) ? "Azure DevOps request failed" : null;
+  const checks = [...statusChecks, ...policyChecks, ...buildChecks];
+  const checksError = checks.length === 0 && (!statusOk || !policyOk || !buildsOk) ? "Azure DevOps request failed" : null;
   return {
     ...pullRequest,
     checks,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.10.0-develop.467",
+  "version": "0.10.0-develop.479",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {