azdo-cli 0.5.0-017-pr-comments-threads.244 → 0.5.0-017-pr-comments-threads.248

package/README.md

@@ -13,7 +13,7 @@ Azure DevOps CLI focused on work item read/write workflows.
 - Create or update work items from markdown documents (`upsert`)
 - Read and post work item comments (`comments`)
 - Read/write rich-text fields as markdown (`get-md-field`, `set-md-field`)
-- Check branch pull request status, open PRs to `develop`, and review active comments (`pr`)
+- Check branch pull request status, open PRs to `develop`, list PR comment threads for any PR (`--pr-number`), and resolve/reopen threads from the CLI (`pr`)
 - Persist org/project/default fields in local config (`config`)
 - List all fields of a work item (`list-fields`)
 - Store a PAT per Azure DevOps organization in the OS credential store via `azdo auth` (or use `AZDO_PAT`). Inspect with `azdo auth status`, remove with `azdo auth logout`. See [docs/authentication.md](docs/authentication.md).
@@ -40,9 +40,16 @@ azdo set-state 12345 "Active"
 # Create a work item from markdown
 azdo upsert --type "User Story" --content $'---\nTitle: Improve markdown import UX\nState: New\n---'
 
-# Read and post comments
+# Read and post work item comments
 azdo comments list 12345
 azdo comments add 12345 "Investigating the root cause now."
+
+# PR comment threads — list, filter, target by number, resolve or reopen
+azdo pr comments                        # active-branch PR
+azdo pr comments --pr-number 64         # any PR by number (skips branch lookup)
+azdo pr comments --pr-number 64 --hide-resolved
+azdo pr comment-resolve 17 --pr-number 64   # idempotent: exit 0 even when already resolved
+azdo pr comment-reopen 17  --pr-number 64
 ```
 
 ## Documentation

package/dist/index.js

@@ -2396,6 +2396,14 @@ function mapThread(thread) {
     comments
   };
 }
+function toActiveCommentThread(thread) {
+  return {
+    id: thread.id,
+    status: thread.status,
+    threadContext: thread.threadContext?.filePath ?? null,
+    comments: thread.comments.map(mapComment).filter((comment) => comment !== null)
+  };
+}
 var RESOLVED_THREAD_STATUSES = /* @__PURE__ */ new Set(["fixed", "wontFix", "closed", "byDesign"]);
 function isThreadResolved(status) {
   return RESOLVED_THREAD_STATUSES.has(status);
@@ -2406,6 +2414,22 @@ async function readJsonResponse(response) {
   }
   return response.json();
 }
+async function patchThreadStatus(context, repo, pat, prId, threadId, status) {
+  const url = new URL(
+    `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/threads/${threadId}`
+  );
+  url.searchParams.set("api-version", "7.1");
+  const response = await fetchWithErrors(url.toString(), {
+    method: "PATCH",
+    headers: {
+      ...authHeaders(pat),
+      "Content-Type": "application/json"
+    },
+    body: JSON.stringify({ status })
+  });
+  const data = await readJsonResponse(response);
+  return toActiveCommentThread(data);
+}
 async function getPullRequestById(context, repo, pat, prId) {
   const url = new URL(
     `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}`
@@ -2739,12 +2763,131 @@ function createPrCommentsCommand() {
   });
   return command;
 }
+async function resolveThreadTarget(threadIdRaw, options) {
+  validateOrgProjectPair(options);
+  const threadId = parsePositivePrNumber(threadIdRaw);
+  if (threadId === null) {
+    writeError(`Invalid thread id "${threadIdRaw}"; expected a positive integer.`);
+    return null;
+  }
+  let explicitPrId = null;
+  if (options.prNumber !== void 0) {
+    explicitPrId = parsePositivePrNumber(options.prNumber);
+    if (explicitPrId === null) {
+      writeError(`Invalid --pr-number "${options.prNumber}"; expected a positive integer.`);
+      return null;
+    }
+  }
+  const resolved = await resolvePrCommandContext(options, { requireBranch: explicitPrId === null });
+  let pullRequest;
+  if (explicitPrId !== null) {
+    try {
+      pullRequest = await getPullRequestById(resolved.context, resolved.repo, resolved.pat, explicitPrId);
+    } catch (err) {
+      if (err instanceof Error && err.message.startsWith("NOT_FOUND")) {
+        writeError(`Pull request #${explicitPrId} not found in ${resolved.context.org}/${resolved.context.project}/${resolved.repo}.`);
+        return null;
+      }
+      throw err;
+    }
+  } else {
+    const pullRequests = await listPullRequests(resolved.context, resolved.repo, resolved.pat, resolved.branch, {
+      status: "active"
+    });
+    if (pullRequests.length === 0) {
+      writeError(`No active pull request found for branch ${resolved.branch}.`);
+      return null;
+    }
+    if (pullRequests.length > 1) {
+      const ids = pullRequests.map((pr) => `#${pr.id}`).join(", ");
+      writeError(`Multiple active pull requests found for branch ${resolved.branch}: ${ids}. Use pr status to review them.`);
+      return null;
+    }
+    pullRequest = pullRequests[0];
+  }
+  return { context: resolved.context, repo: resolved.repo, pat: resolved.pat, pullRequest, threadId };
+}
+async function runThreadStateChange(threadIdRaw, options, direction) {
+  let context;
+  try {
+    const target = await resolveThreadTarget(threadIdRaw, options);
+    if (target === null) {
+      return;
+    }
+    context = target.context;
+    const threads = await getPullRequestThreads(target.context, target.repo, target.pat, target.pullRequest.id);
+    const thread = threads.find((t) => t.id === target.threadId);
+    if (!thread) {
+      writeError(`Thread #${target.threadId} not found on pull request #${target.pullRequest.id}.`);
+      return;
+    }
+    const alreadyInTargetState = direction === "resolve" ? isThreadResolved(thread.status) : !isThreadResolved(thread.status);
+    const targetStatus = direction === "resolve" ? "fixed" : "active";
+    if (alreadyInTargetState) {
+      const humanLabel = direction === "resolve" ? "resolved" : "active";
+      const noopResult = {
+        pullRequestId: target.pullRequest.id,
+        threadId: target.threadId,
+        status: targetStatus,
+        noop: true
+      };
+      if (options.json) {
+        process.stdout.write(`${JSON.stringify(noopResult, null, 2)}
+`);
+        return;
+      }
+      process.stdout.write(`Thread #${target.threadId} is already ${humanLabel} on pull request #${target.pullRequest.id}.
+`);
+      return;
+    }
+    const updated = await patchThreadStatus(
+      target.context,
+      target.repo,
+      target.pat,
+      target.pullRequest.id,
+      target.threadId,
+      targetStatus
+    );
+    const result = {
+      pullRequestId: target.pullRequest.id,
+      threadId: target.threadId,
+      status: targetStatus,
+      noop: false
+    };
+    if (options.json) {
+      process.stdout.write(`${JSON.stringify(result, null, 2)}
+`);
+      return;
+    }
+    const verb = direction === "resolve" ? "resolved" : "reopened";
+    process.stdout.write(`Thread #${target.threadId} ${verb} on pull request #${target.pullRequest.id} (status: ${updated.status}).
+`);
+  } catch (err) {
+    handlePrCommandError(err, context, "write");
+  }
+}
+function createPrCommentResolveCommand() {
+  const command = new Command12("comment-resolve");
+  command.description("Mark a pull request comment thread as resolved").argument("<threadId>", "numeric id of the thread to resolve").option("--org <org>", "Azure DevOps organization").option("--project <project>", "Azure DevOps project").option("--pr-number <N>", "target the pull request with this numeric id, instead of the current branch's PR").option("--json", "output JSON").action(async (threadIdRaw, options) => {
+    await runThreadStateChange(threadIdRaw, options, "resolve");
+  });
+  return command;
+}
+function createPrCommentReopenCommand() {
+  const command = new Command12("comment-reopen");
+  command.description("Reopen (set to active) a previously resolved pull request comment thread").argument("<threadId>", "numeric id of the thread to reopen").option("--org <org>", "Azure DevOps organization").option("--project <project>", "Azure DevOps project").option("--pr-number <N>", "target the pull request with this numeric id, instead of the current branch's PR").option("--json", "output JSON").action(async (threadIdRaw, options) => {
+    await runThreadStateChange(threadIdRaw, options, "reopen");
+  });
+  return command;
+}
 function createPrCommand() {
   const command = new Command12("pr");
   command.description("Manage Azure DevOps pull requests");
   command.addCommand(createPrStatusCommand());
   command.addCommand(createPrOpenCommand());
   command.addCommand(createPrCommentsCommand());
+  command.addCommand(createPrCommentResolveCommand());
+  command.addCommand(createPrCommentReopenCommand());
   return command;
 }
 

package/package.json

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