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

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
@@ -121,6 +123,8 @@ azdo pr work-items unlink 1234 --pr-number 64
 azdo pr reviewers add jane@example.com --pr-number 64             # optional by default
 azdo pr reviewers add jane@example.com --pr-number 64 --required  # required (or promotes in place)
 azdo pr reviewers remove jane@example.com --pr-number 64
+azdo pr reviewers list --pr-number 64                             # who must review, and how each voted
+azdo pr reviewers list --pr-number 64 --json                      # { pullRequestId, reviewers: [{ id, uniqueName, isRequired, vote, voteState, ... }] }
 
 # Any pr subcommand can target another repository
 azdo pr comments --repo other-repo --pr-number 12

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(),
@@ -4247,13 +4261,34 @@ function buildPullRequestReviewerUrl(context, repo, prId, reviewerId) {
   url.searchParams.set("api-version", "7.1");
   return url;
 }
+function reviewerVoteState(vote) {
+  switch (vote) {
+    case 10:
+      return "approved";
+    case 5:
+      return "approved-with-suggestions";
+    case 0:
+      return "no-vote";
+    case -5:
+      return "waiting-for-author";
+    case -10:
+      return "rejected";
+    case 15:
+      return "bypassed";
+    default:
+      return "unknown";
+  }
+}
 function mapReviewer(data) {
+  const vote = data.vote ?? 0;
   return {
     id: data.id,
     displayName: data.displayName ?? null,
     uniqueName: data.uniqueName ?? null,
     isRequired: data.isRequired ?? false,
-    vote: data.vote ?? 0
+    vote,
+    voteState: reviewerVoteState(vote),
+    hasDeclined: data.hasDeclined ?? false
   };
 }
 async function getPullRequestReviewers(context, repo, cred, prId) {
@@ -5811,42 +5846,73 @@ function createPrCommentDeleteCommand() {
 }
 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,
@@ -6042,9 +6108,50 @@ async function runReviewerRemove(reviewer, options) {
     handlePrCommandError(err, context, "write");
   }
 }
+function formatReviewerLine(reviewer) {
+  const name = reviewer.displayName ?? reviewer.uniqueName ?? reviewer.id;
+  const unique = reviewer.uniqueName && reviewer.uniqueName !== name ? ` <${reviewer.uniqueName}>` : "";
+  const kind = reviewer.isRequired ? "required" : "optional";
+  const declined = reviewer.hasDeclined ? ", declined" : "";
+  return `${name}${unique} \u2014 ${reviewer.voteState} (${kind}${declined})`;
+}
+async function runReviewerList(options) {
+  let context;
+  try {
+    const target = await resolvePullRequestTarget(options, {
+      onContextResolved: (resolved) => {
+        context = resolved;
+      }
+    });
+    if (target === null) {
+      return;
+    }
+    const reviewers = await getPullRequestReviewers(target.context, target.repo, target.pat, target.pullRequest.id);
+    const result = { pullRequestId: target.pullRequest.id, reviewers };
+    if (options.json) {
+      process.stdout.write(`${JSON.stringify(result, null, 2)}
+`);
+      return;
+    }
+    if (reviewers.length === 0) {
+      process.stdout.write(`No reviewers on pull request #${target.pullRequest.id}.
+`);
+      return;
+    }
+    process.stdout.write(`${reviewers.map(formatReviewerLine).join("\n")}
+`);
+  } catch (err) {
+    handlePrCommandError(err, context, "read");
+  }
+}
 function createPrReviewersCommand() {
   const command = new Command12("reviewers");
   command.description("Manage pull request reviewers");
+  const list = new Command12("list");
+  withCommonPrOptions(configureUnwrappedHelp(list)).description("List the pull request reviewers with their votes (approved, waiting-for-author, \u2026) and required flag").option("--pr-number <N>", PR_NUMBER_HELP).option("--json", "output JSON").action(async (_options, command2) => {
+    await runReviewerList(mergedPrOptions(command2));
+  });
+  command.addCommand(list);
   const add = new Command12("add");
   withCommonPrOptions(configureUnwrappedHelp(add)).description("Add a reviewer to the pull request (optional by default)").argument("<reviewer>", "reviewer email or unique name").option("--required", "mark the reviewer as required instead of optional").option("--pr-number <N>", PR_NUMBER_HELP).option("--json", "output JSON").action(async (reviewer, _options, command2) => {
     await runReviewerAdd(reviewer, mergedPrOptions(command2));

package/package.json

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