azdo-cli 0.22.0-develop.769 → 0.22.0-develop.787

package/README.md

@@ -80,13 +80,16 @@ azdo pr comments --exclude-system --max-chars 500    # human comments only, trun
 azdo pr comments --thread 148           # a single thread, by id (selector: exit 1 if absent)
 azdo pr comments --contains '"kind":"review-plan"'   # threads holding a literal substring
 azdo pr status                          # PR checks (status + branch policies + pipeline builds) + code-comment counts
+azdo pr status --branch feature/x --json   # same view for another branch, nothing checked out
+azdo pr status --pr-number 64           # same view for one PR by number, any status
 azdo pr comment-resolve 17 --pr-number 64   # idempotent: exit 0 even when already resolved
 azdo pr comment-reopen 17  --pr-number 64
 
-# Write to a PR — new thread, in-place edit, reply
+# Write to a PR — new thread, in-place edit, delete, reply
 azdo pr comments add --file plan.md --pr-number 64 --dry-run   # preview, writes nothing
 azdo pr comments add --file plan.md --pr-number 64             # NEW thread on the overview
 azdo pr comments edit 148 --file plan.md --pr-number 64        # rewrite it in place
+azdo pr comments delete 148 --comment-id 3 --pr-number 64     # delete a comment (irreversible; no prompt)
 azdo pr comments reply 148 "Great suggestion, I'll address it."          # human-readable output
 azdo pr comments reply 148 "Done." --pr-number 64 --json                 # JSON: { pullRequestId, threadId, commentId, content }
 azdo pr comment-reply 148 "Done."  --pr-number 64                        # flat alias, identical behaviour

package/dist/index.js

@@ -4105,6 +4105,19 @@ async function updateThreadComment(context, repo, cred, prId, threadId, commentI
     publishedAt: data.publishedDate ?? null
   };
 }
+async function deleteThreadComment(context, repo, cred, prId, threadId, commentId) {
+  const url = new URL(
+    `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/threads/${threadId}/comments/${commentId}`
+  );
+  url.searchParams.set("api-version", "7.1");
+  const response = await fetchWithErrors(url.toString(), {
+    method: "DELETE",
+    headers: authHeaders(cred)
+  });
+  if (!response.ok) {
+    throw httpError(response);
+  }
+}
 async function postThreadComment(context, repo, cred, prId, threadId, content) {
   const response = await fetchWithErrors(buildThreadCommentUrl(context, repo, prId, threadId).toString(), {
     method: "POST",
@@ -4368,7 +4381,7 @@ function parsePositivePrNumber(raw) {
     return null;
   }
   const n = Number.parseInt(raw, 10);
-  return Number.isFinite(n) && n > 0 ? n : null;
+  return Number.isSafeInteger(n) && n > 0 ? n : null;
 }
 function prNumberHelp(branchStatus = "active") {
   const matched = branchStatus === "abandoned" ? "abandoned" : "open";
@@ -4678,16 +4691,59 @@ async function resolvePrCommandContext(options, resolveOpts = {}) {
     pat: credential
   };
 }
+var STATUS_PR_NUMBER_HELP = "show only the pull request with this numeric id, instead of the current branch's pull requests; mutually exclusive with --branch";
+var STATUS_BRANCH_HELP = "show the pull requests whose source branch is this one (with or without the refs/heads/ prefix), instead of the current branch's; no checkout needed. Mutually exclusive with --pr-number";
+function parseStatusTarget(options) {
+  if (options.prNumber !== void 0 && options.branch !== void 0) {
+    writeError("Cannot specify both --pr-number and --branch.");
+    return null;
+  }
+  if (options.branch !== void 0) {
+    const branch = options.branch.trim().replace(/^refs\/heads\//, "");
+    if (branch === "") {
+      writeError("--branch must not be empty.");
+      return null;
+    }
+    return { kind: "branch", branch };
+  }
+  const prId = parseTargetPrNumber(options);
+  if (prId === "invalid") {
+    return null;
+  }
+  return prId === "none" ? { kind: "current" } : { kind: "pr", prId };
+}
+async function findStatusPullRequests(resolved, target) {
+  if (target.kind === "pr") {
+    const pullRequest = await fetchTargetById(resolved, target.prId);
+    return pullRequest === null ? null : { branch: formatBranchName(pullRequest.sourceRefName), pullRequests: [pullRequest] };
+  }
+  const branch = target.kind === "branch" ? target.branch : resolved.branch;
+  const pullRequests = await listPullRequests(resolved.context, resolved.repo, resolved.pat, branch);
+  if (target.kind === "branch" && pullRequests.length === 0) {
+    writeError(
+      `No pull requests found for branch ${branch} in ${resolved.context.org}/${resolved.context.project}/${resolved.repo}.`
+    );
+    return null;
+  }
+  return { branch, pullRequests };
+}
 function createPrStatusCommand() {
   const command = new Command12("status");
-  withCommonPrOptions(command).description("Check pull requests for the current branch").option("--json", "output JSON").action(async (options) => {
+  withCommonPrOptions(configureUnwrappedHelp(command)).description("Check pull requests for the current branch, another branch, or a specific pull request").option("--pr-number <id>", STATUS_PR_NUMBER_HELP).option("--branch <name>", STATUS_BRANCH_HELP).option("--json", "output JSON").action(async (options) => {
     validateOrgProjectPair(options);
+    const target = parseStatusTarget(options);
+    if (target === null) {
+      return;
+    }
     let context;
     try {
-      const resolved = await resolvePrCommandContext(options);
+      const resolved = await resolvePrCommandContext(options, { requireBranch: target.kind === "current" });
       context = resolved.context;
-      const branch = resolved.branch;
-      const pullRequests = await listPullRequests(resolved.context, resolved.repo, resolved.pat, branch);
+      const found = await findStatusPullRequests(resolved, target);
+      if (found === null) {
+        return;
+      }
+      const { branch, pullRequests } = found;
       let projectId = null;
       try {
         projectId = await resolveProjectId(resolved.context, resolved.pat);
@@ -5186,6 +5242,7 @@ function createPrCommentsCommand() {
   command.addCommand(createPrCommentsReplyCommand());
   command.addCommand(createPrCommentsAddCommand());
   command.addCommand(createPrCommentsEditCommand());
+  command.addCommand(createPrCommentsDeleteCommand());
   return command;
 }
 function parseTargetPrNumber(options) {
@@ -5504,7 +5561,26 @@ function createPrCommentAddCommand() {
     'Post a new comment thread on the pull request overview (alias of "azdo pr comments add")'
   );
 }
-async function fetchThreadForEdit(target) {
+function parseExplicitCommentId(options) {
+  if (options.commentId === void 0) {
+    return null;
+  }
+  const parsed = parsePositivePrNumber(options.commentId);
+  if (parsed === null) {
+    writeError(`Invalid --comment-id "${options.commentId}"; expected a positive integer.`);
+    return "invalid";
+  }
+  return parsed;
+}
+function findCommentInThread(thread, commentId, target) {
+  const match = thread.comments.find((comment) => comment.id === commentId);
+  if (match === void 0) {
+    writeError(`Comment #${commentId} not found in thread #${target.threadId} on pull request #${target.pullRequest.id}.`, EXIT_NOT_FOUND);
+    return null;
+  }
+  return match;
+}
+async function fetchTargetThread(target) {
   try {
     return await getPullRequestThread(
       target.context,
@@ -5523,12 +5599,7 @@ async function fetchThreadForEdit(target) {
 }
 function selectEditableComment(thread, explicitCommentId, target) {
   if (explicitCommentId !== null) {
-    const match = thread.comments.find((comment) => comment.id === explicitCommentId);
-    if (match === void 0) {
-      writeError(`Comment #${explicitCommentId} not found in thread #${target.threadId} on pull request #${target.pullRequest.id}.`, EXIT_NOT_FOUND);
-      return null;
-    }
-    return match;
+    return findCommentInThread(thread, explicitCommentId, target);
   }
   const first = [...thread.comments].sort((a, b) => a.id - b.id)[0];
   if (first === void 0) {
@@ -5563,13 +5634,9 @@ async function runCommentEdit(threadIdRaw, text, options) {
     if (body === null) {
       return;
     }
-    let explicitCommentId = null;
-    if (options.commentId !== void 0) {
-      explicitCommentId = parsePositivePrNumber(options.commentId);
-      if (explicitCommentId === null) {
-        writeError(`Invalid --comment-id "${options.commentId}"; expected a positive integer.`);
-        return;
-      }
+    const explicitCommentId = parseExplicitCommentId(options);
+    if (explicitCommentId === "invalid") {
+      return;
     }
     const target = await resolveThreadTarget(threadIdRaw, options, {
       onContextResolved: (resolved) => {
@@ -5579,7 +5646,7 @@ async function runCommentEdit(threadIdRaw, text, options) {
     if (target === null) {
       return;
     }
-    const thread = await fetchThreadForEdit(target);
+    const thread = await fetchTargetThread(target);
     if (thread === null) {
       return;
     }
@@ -5641,6 +5708,107 @@ function createPrCommentEditCommand() {
     'Edit an existing pull request comment in place (alias of "azdo pr comments edit")'
   );
 }
+function selectDeletableComment(thread, explicitCommentId, target) {
+  if (explicitCommentId !== null) {
+    return findCommentInThread(thread, explicitCommentId, target);
+  }
+  const comments = [...thread.comments].sort((a, b) => a.id - b.id);
+  if (comments.length === 1) {
+    return comments[0];
+  }
+  if (comments.length === 0) {
+    writeError(`Thread #${target.threadId} on pull request #${target.pullRequest.id} has no comment to delete.`, EXIT_NOT_FOUND);
+    return null;
+  }
+  const candidates = comments.map((comment) => `#${comment.id} (${comment.author ?? "Unknown"})`).join(", ");
+  writeError(
+    `Thread #${target.threadId} on pull request #${target.pullRequest.id} holds ${comments.length} comments; pass --comment-id to choose one: ${candidates}.`
+  );
+  return null;
+}
+function reportDeleteResult(result, json, comment) {
+  if (json) {
+    process.stdout.write(`${JSON.stringify(result, null, 2)}
+`);
+    return;
+  }
+  if (result.dryRun) {
+    process.stdout.write(
+      `Dry run: would delete comment #${result.commentId} by ${comment.author ?? "Unknown"} (${comment.content.length} chars) from thread #${result.threadId} on pull request #${result.pullRequestId}.
+`
+    );
+    return;
+  }
+  process.stdout.write(
+    `Comment #${result.commentId} deleted from thread #${result.threadId} on pull request #${result.pullRequestId}.
+`
+  );
+}
+async function runCommentDelete(threadIdRaw, options) {
+  let context;
+  try {
+    const explicitCommentId = parseExplicitCommentId(options);
+    if (explicitCommentId === "invalid") {
+      return;
+    }
+    const target = await resolveThreadTarget(threadIdRaw, options, {
+      onContextResolved: (resolved) => {
+        context = resolved;
+      }
+    });
+    if (target === null) {
+      return;
+    }
+    const thread = await fetchTargetThread(target);
+    if (thread === null) {
+      return;
+    }
+    const comment = selectDeletableComment(thread, explicitCommentId, target);
+    if (comment === null) {
+      return;
+    }
+    const dryRun = options.dryRun === true;
+    if (!dryRun) {
+      await deleteThreadComment(
+        target.context,
+        target.repo,
+        target.pat,
+        target.pullRequest.id,
+        target.threadId,
+        comment.id
+      );
+    }
+    reportDeleteResult(
+      {
+        pullRequestId: target.pullRequest.id,
+        threadId: target.threadId,
+        commentId: comment.id,
+        deleted: !dryRun,
+        dryRun
+      },
+      options.json === true,
+      comment
+    );
+  } catch (err) {
+    handlePrCommandError(err, context, "write");
+  }
+}
+function buildCommentDeleteCommand(name, description) {
+  const command = new Command12(name);
+  withCommonPrOptions(configureUnwrappedHelp(command)).description(description).argument("<threadId>", "numeric id of the thread holding the comment").option("--comment-id <N>", "numeric id of the comment to delete; may be omitted only when the thread holds a single comment").option("--dry-run", "resolve the target comment and print what would be deleted, without deleting anything").option("--pr-number <N>", PR_NUMBER_HELP).option("--json", "output JSON").action(async (threadIdRaw, _options, command2) => {
+    await runCommentDelete(threadIdRaw, mergedPrOptions(command2));
+  });
+  return command;
+}
+function createPrCommentsDeleteCommand() {
+  return buildCommentDeleteCommand("delete", "Delete a pull request comment (no confirmation prompt; irreversible)");
+}
+function createPrCommentDeleteCommand() {
+  return buildCommentDeleteCommand(
+    "comment-delete",
+    'Delete a pull request comment (alias of "azdo pr comments delete"; no confirmation prompt; irreversible)'
+  );
+}
 var LIST_STATUS_VALUES = ["active", "completed", "abandoned", "all"];
 var DEFAULT_LIST_TOP = 25;
 function formatPullRequestListEntry(pullRequest) {
@@ -5906,6 +6074,7 @@ function createPrCommand() {
   command.addCommand(createPrWorkItemsCommand());
   command.addCommand(createPrReviewersCommand());
   command.addCommand(createPrCommentEditCommand());
+  command.addCommand(createPrCommentDeleteCommand());
   return command;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.22.0-develop.769",
+  "version": "0.22.0-develop.787",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {