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

package/README.md

@@ -85,10 +85,11 @@ azdo pr status --pr-number 64           # same view for one PR by number, any st
 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",
@@ -5229,6 +5242,7 @@ function createPrCommentsCommand() {
   command.addCommand(createPrCommentsReplyCommand());
   command.addCommand(createPrCommentsAddCommand());
   command.addCommand(createPrCommentsEditCommand());
+  command.addCommand(createPrCommentsDeleteCommand());
   return command;
 }
 function parseTargetPrNumber(options) {
@@ -5547,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,
@@ -5566,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) {
@@ -5606,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) => {
@@ -5622,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;
     }
@@ -5684,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) {
@@ -5949,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.782",
+  "version": "0.22.0-develop.787",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {