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

package/README.md

@@ -15,7 +15,7 @@ Azure DevOps CLI focused on work item read/write workflows.
 - Attach a local file to a work item, or remove a named attachment (`add-attachment`, `delete-attachment`)
 - Read/write rich-text fields as markdown (`get-md-field`, `set-md-field`)
 - Download images embedded in rich-text fields, optionally resized for LLM use (`get-item`/`get-md-field` `--download-images`, `--resize-images`)
-- Check branch pull request status, open PRs to `develop` (optionally pre-filled from a repository-defined template), update an existing PR's title or description (`pr update`), abandon or reactivate one (`pr abandon` / `pr reactivate`), list PR comment threads for any PR (`--pr-number`), resolve/reopen threads, link/unlink work items, and add/remove required or optional reviewers — all from the CLI (`pr`)
+- List pull requests with their draft state, dates, reviewers, labels and linked work item ids (`pr list --work-items`), check branch pull request status, open PRs to `develop` (optionally pre-filled from a repository-defined template), update an existing PR's title or description (`pr update`), abandon or reactivate one (`pr abandon` / `pr reactivate`), list PR comment threads for any PR (`--pr-number`), resolve/reopen threads, link/unlink work items, and add/remove required or optional reviewers — all from the CLI (`pr`)
 - Feed long PR titles, descriptions and comment bodies from a file or a pipe instead of the shell (`--title-file`, `--description-file`, `--file`; `-` reads standard input)
 - Persist org/project/default fields in local config (`config`)
 - List all fields of a work item (`list-fields`)
@@ -67,9 +67,11 @@ azdo delete-attachment 12345 screenshot.png --yes         # skip the prompt (scr
 azdo delete-attachment 12345 screenshot.png --id <guid>   # disambiguate when the name is shared
 
 # Find a pull request — one API call, any branch
-azdo pr list                            # active PRs in the repository
-azdo pr list --branch feature/x --json  # id, title, source/target, author, url, description
+azdo pr list                            # active PRs in the repository ([active, draft] marks a draft)
+azdo pr list --branch feature/x --json  # id, title, source/target, author, url, description, isDraft,
+                                        # creationDate, closedDate, reviewers (uniqueName, vote), labels
 azdo pr list --status all --top 50
+azdo pr list --work-items --json        # + workItemIds per PR: the whole PR ↔ work item map in one call
 
 # PR comment threads — list, filter, target by number, resolve or reopen
 azdo pr comments                        # active-branch PR; code-anchored threads show file:line
@@ -85,10 +87,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

@@ -3704,7 +3704,12 @@ function mapPullRequest(context, repo, pullRequest) {
     url: pullRequest._links?.web?.href ?? buildPullRequestWebUrl(context, repo, pullRequest.pullRequestId),
     description: pullRequest.description?.trim() || null,
     createdByUniqueName: pullRequest.createdBy?.uniqueName ?? null,
-    createdById: pullRequest.createdBy?.id ?? null
+    createdById: pullRequest.createdBy?.id ?? null,
+    isDraft: pullRequest.isDraft ?? false,
+    creationDate: pullRequest.creationDate ?? null,
+    closedDate: pullRequest.closedDate ?? null,
+    reviewers: (pullRequest.reviewers ?? []).map(mapReviewer),
+    labels: (pullRequest.labels ?? []).filter((label) => label.active !== false).map((label) => label.name)
   };
 }
 function mapPullRequestCheckName(status2) {
@@ -3884,6 +3889,15 @@ async function listRepositoryPullRequests(context, repo, cred, opts) {
   const data = await readJsonResponse(response);
   return data.value.map((pullRequest) => mapPullRequest(context, repo, pullRequest));
 }
+async function getPullRequestWorkItemIds(context, repo, cred, prId) {
+  const url = new URL(
+    `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/workitems`
+  );
+  url.searchParams.set("api-version", "7.1");
+  const response = await fetchWithErrors(url.toString(), { headers: authHeaders(cred) });
+  const data = await readJsonResponse(response);
+  return data.value.map((ref) => Number.parseInt(ref.id, 10)).filter((id) => Number.isInteger(id) && id > 0).sort((a, b) => a - b);
+}
 async function getPullRequestChecks(context, repo, cred, prId) {
   const response = await fetchWithErrors(
     buildPullRequestStatusesUrl(context, repo, prId).toString(),
@@ -4105,6 +4119,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 +5256,7 @@ function createPrCommentsCommand() {
   command.addCommand(createPrCommentsReplyCommand());
   command.addCommand(createPrCommentsAddCommand());
   command.addCommand(createPrCommentsEditCommand());
+  command.addCommand(createPrCommentsDeleteCommand());
   return command;
 }
 function parseTargetPrNumber(options) {
@@ -5547,7 +5575,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 +5613,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 +5648,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 +5660,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,44 +5722,176 @@ 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;
+var WORK_ITEM_LOOKUP_CONCURRENCY = 5;
 function formatPullRequestListEntry(pullRequest) {
-  return [
-    `#${pullRequest.id} [${pullRequest.status}] ${pullRequest.title}`,
+  const state = pullRequest.isDraft ? `${pullRequest.status}, draft` : pullRequest.status;
+  const lines = [
+    `#${pullRequest.id} [${state}] ${pullRequest.title}`,
     `  ${formatBranchName(pullRequest.sourceRefName)} -> ${formatBranchName(pullRequest.targetRefName)}`,
-    `  Author: ${pullRequest.createdBy ?? "Unknown"}`,
-    `  ${pullRequest.url ?? "\u2014"}`
-  ].join("\n");
+    `  Author: ${pullRequest.createdBy ?? "Unknown"}`
+  ];
+  if ("workItemIds" in pullRequest) {
+    const ids = pullRequest.workItemIds.map((id) => `#${id}`).join(", ");
+    lines.push(`  Work items: ${ids || "none"}`);
+  }
+  lines.push(`  ${pullRequest.url ?? "\u2014"}`);
+  return lines.join("\n");
+}
+function parseListOptions(options) {
+  const status2 = options.status ?? "active";
+  if (!LIST_STATUS_VALUES.includes(status2)) {
+    writeError(`Invalid --status "${status2}"; expected one of ${LIST_STATUS_VALUES.join(", ")}.`);
+    return null;
+  }
+  let top = DEFAULT_LIST_TOP;
+  if (options.top !== void 0) {
+    const parsed = parsePositivePrNumber(options.top);
+    if (parsed === null) {
+      writeError(`Invalid --top "${options.top}"; expected a positive integer.`);
+      return null;
+    }
+    top = parsed;
+  }
+  const branch = options.branch?.trim().replace(/^refs\/heads\//, "") || null;
+  return { status: status2, top, branch };
+}
+async function attachWorkItemIds(context, repo, cred, pullRequests) {
+  const result = new Array(pullRequests.length);
+  let next = 0;
+  const worker = async () => {
+    while (next < pullRequests.length) {
+      const index = next;
+      next += 1;
+      const pullRequest = pullRequests[index];
+      const workItemIds = await getPullRequestWorkItemIds(context, repo, cred, pullRequest.id);
+      result[index] = { ...pullRequest, workItemIds };
+    }
+  };
+  await Promise.all(Array.from({ length: Math.min(WORK_ITEM_LOOKUP_CONCURRENCY, pullRequests.length) }, worker));
+  return result;
 }
 function createPrListCommand() {
   const command = new Command12("list");
-  withCommonPrOptions(configureUnwrappedHelp(command)).description("List pull requests in the repository, optionally filtered by source branch").option("--branch <name>", "only pull requests whose source branch is this one (with or without the refs/heads/ prefix)").option("--status <status>", `pull request status filter (${LIST_STATUS_VALUES.join(" | ")})`, "active").option("--top <N>", `maximum number of pull requests to return (default ${DEFAULT_LIST_TOP})`).option("--json", "output JSON").action(async (options) => {
+  withCommonPrOptions(configureUnwrappedHelp(command)).description("List pull requests in the repository, optionally filtered by source branch").option("--branch <name>", "only pull requests whose source branch is this one (with or without the refs/heads/ prefix)").option("--status <status>", `pull request status filter (${LIST_STATUS_VALUES.join(" | ")})`, "active").option("--top <N>", `maximum number of pull requests to return (default ${DEFAULT_LIST_TOP})`).option("--work-items", "also return the ids of each pull request's linked work items (one extra API call per pull request)").option("--json", "output JSON").action(async (options) => {
     validateOrgProjectPair(options);
-    const status2 = options.status ?? "active";
-    if (!LIST_STATUS_VALUES.includes(status2)) {
-      writeError(`Invalid --status "${status2}"; expected one of ${LIST_STATUS_VALUES.join(", ")}.`);
+    const parsed = parseListOptions(options);
+    if (parsed === null) {
       return;
     }
-    let top = DEFAULT_LIST_TOP;
-    if (options.top !== void 0) {
-      const parsed = parsePositivePrNumber(options.top);
-      if (parsed === null) {
-        writeError(`Invalid --top "${options.top}"; expected a positive integer.`);
-        return;
-      }
-      top = parsed;
-    }
-    const branch = options.branch?.trim().replace(/^refs\/heads\//, "") || null;
+    const { status: status2, top, branch } = parsed;
     let context;
     try {
       const resolved = await resolvePrCommandContext(options, { requireBranch: false });
       context = resolved.context;
-      const pullRequests = await listRepositoryPullRequests(resolved.context, resolved.repo, resolved.pat, {
+      const listed = await listRepositoryPullRequests(resolved.context, resolved.repo, resolved.pat, {
         sourceBranch: branch ?? void 0,
         status: status2,
         top
       });
+      const pullRequests = options.workItems ? await attachWorkItemIds(resolved.context, resolved.repo, resolved.pat, listed) : listed;
       const result = {
         repository: resolved.repo,
         branch,
@@ -5949,6 +6119,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.790",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {