azdo-cli 0.5.0-017-pr-comments-threads.237 → 0.5.0-017-pr-comments-threads.240

package/dist/index.js

@@ -2340,7 +2340,7 @@ function mapPullRequest(repo, pullRequest) {
     targetRefName: pullRequest.targetRefName,
     status: pullRequest.status,
     createdBy: pullRequest.createdBy?.displayName ?? null,
-    url: pullRequest._links.web.href
+    url: pullRequest._links?.web?.href ?? null
   };
 }
 function mapPullRequestCheckName(status) {
@@ -2385,9 +2385,6 @@ function mapComment(comment) {
   };
 }
 function mapThread(thread) {
-  if (thread.status !== "active" && thread.status !== "pending") {
-    return null;
-  }
   const comments = thread.comments.map(mapComment).filter((comment) => comment !== null);
   if (comments.length === 0) {
     return null;
@@ -2480,25 +2477,30 @@ function formatBranchName(refName) {
 function writeError(message) {
   process.stderr.write(`Error: ${message}
 `);
-  process.exit(1);
+  process.exitCode = 1;
 }
 function handlePrCommandError(err, context, mode = "read") {
   const error = err instanceof Error ? err : new Error(String(err));
   if (error.message === "AUTH_FAILED") {
     const scopeLabel = mode === "write" ? "Code (Read & Write)" : "Code (Read)";
     writeError(`Authentication failed. Check that your PAT is valid and has the "${scopeLabel}" scope.`);
+    return;
   }
   if (error.message === "PERMISSION_DENIED") {
     writeError(`Access denied. Your PAT may lack ${mode} permissions for project "${context?.project}".`);
+    return;
   }
   if (error.message === "NETWORK_ERROR") {
     writeError("Could not connect to Azure DevOps. Check your network connection.");
+    return;
   }
   if (error.message.startsWith("NOT_FOUND")) {
     writeError(`Azure DevOps repository not found in ${context?.org}/${context?.project}.`);
+    return;
   }
   if (error.message.startsWith("HTTP_")) {
     writeError(`Azure DevOps request failed with ${error.message}.`);
+    return;
   }
   writeError(error.message);
 }
@@ -2519,7 +2521,7 @@ function formatPullRequestBlock(pullRequest) {
   return [
     `#${pullRequest.id} [${pullRequest.status}] ${pullRequest.title}`,
     `${formatBranchName(pullRequest.sourceRefName)} -> ${formatBranchName(pullRequest.targetRefName)}`,
-    pullRequest.url,
+    pullRequest.url ?? "\u2014",
     ...formatPullRequestChecks(pullRequest.checks)
   ].join("\n");
 }
@@ -2587,10 +2589,12 @@ function createPrOpenCommand() {
     const title = options.title?.trim();
     if (!title) {
       writeError("--title is required for pull request creation.");
+      return;
     }
     const description = options.description?.trim();
     if (!description) {
       writeError("--description is required for pull request creation.");
+      return;
     }
     let context;
     try {
@@ -2598,6 +2602,7 @@ function createPrOpenCommand() {
       context = resolved.context;
       if (resolved.branch === "develop") {
         writeError("Pull request creation requires a source branch other than develop.");
+        return;
       }
       const result = await openPullRequest(
         resolved.context,
@@ -2614,19 +2619,20 @@ function createPrOpenCommand() {
       }
       if (result.created) {
         process.stdout.write(`Created pull request #${result.pullRequest.id}: ${result.pullRequest.title}
-${result.pullRequest.url}
+${result.pullRequest.url ?? "\u2014"}
 `);
         return;
       }
       process.stdout.write(
         `Active pull request already exists for ${resolved.branch} -> develop: #${result.pullRequest.id}
-${result.pullRequest.url}
+${result.pullRequest.url ?? "\u2014"}
 `
       );
     } catch (err) {
       if (err instanceof Error && err.message.startsWith("AMBIGUOUS_PRS:")) {
         const ids = err.message.replace("AMBIGUOUS_PRS:", "").split(",").map((id) => `#${id}`).join(", ");
         writeError(`Multiple active pull requests already exist for this branch targeting develop: ${ids}. Use pr status to review them.`);
+        return;
       }
       handlePrCommandError(err, context, "write");
     }
@@ -2646,10 +2652,12 @@ function createPrCommentsCommand() {
       });
       if (pullRequests.length === 0) {
         writeError(`No active pull request found for branch ${resolved.branch}.`);
+        return;
       }
       if (pullRequests.length > 1) {
         const ids = pullRequests.map((pullRequest2) => `#${pullRequest2.id}`).join(", ");
         writeError(`Multiple active pull requests found for branch ${resolved.branch}: ${ids}. Use pr status to review them.`);
+        return;
       }
       const pullRequest = pullRequests[0];
       const threads = await getPullRequestThreads(resolved.context, resolved.repo, resolved.pat, pullRequest.id);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.5.0-017-pr-comments-threads.237",
+  "version": "0.5.0-017-pr-comments-threads.240",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {