feat: implement comment thread functionality for pull requests

Author: Joan-Angelo Enrile

README.md

@@ -85,7 +85,11 @@ ado pr view <pr>
 ado pr view <pr> --comments
 ado pr diff <pr>
 ado pr diff <pr> --name-only
-ado pr comment <pr> --body "Looks good"
+ado pr thread create <pr> --body "Looks good to me"
+ado pr thread create <pr> --file src/api/client.ts --line 42 --body "This could be simplified"
+ado pr thread create <pr> --file src/api/client.ts --line 42-55 --body "Consider extracting this block"
+ado pr thread reply <pr> <thread-id> --body "Fixed in latest commit"
+ado pr thread status <pr> <thread-id> --resolve
 ado pr create --title "My PR" --draft
 ado pr review <pr> --approve
 ado pr review <pr> --reject --comment "Needs changes"

TODO.md

@@ -10,7 +10,7 @@
 - [x] `ado issue view <id>` — work item detail view with optional comments
 - [x] `ado pr list` — list PRs with state/author/repo filters
 - [x] `ado pr view <pr>` — PR detail with optional comment threads
-- [x] `ado pr comment <pr>` — add comment (--body or $EDITOR)
+- [x] `ado pr thread create <pr>` — create a new comment thread (--body or $EDITOR; optionally anchored to --file + --line range)
 - [x] `ado pr diff <pr>` — real unified diff with `--color`, `--exclude`, `--name-only`, `--patch`, `--web` flags matching `gh pr diff`
 - [x] OAuth browser login as default auth (MSAL `acquireTokenInteractive`); device code flow for headless/CI
 - [x] DEP0169 deprecation warning suppression (upstream: azure-devops-node-api#664, fix: PR#662)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "ado-cli",
-  "version": "1.1.1",
+  "version": "1.2.0",
   "description": "An Azure DevOps CLI, mimicking the known and loved Github CLI syntax.",
   "main": "./dist/index.js",
   "bin": {

plugins/azure-devops/skills/azure-devops/reference.md

@@ -143,12 +143,16 @@ Generated from source at `src/commands/**/*.ts`. Update this file whenever a com
 | `--json [fields]` | — | Output as JSON | — |
 | `--web` | `-w` | Open in browser | — |
 
-## ado pr comment \<id\>
+## ado pr thread create \<pr-number\>
+
+Create a new comment thread on a pull request. Optionally anchor the thread to a specific file and line range in the diff (targets the right/head side).
 
 | Flag | Short | Description | Default |
 |------|-------|-------------|---------|
-| `--body <text>` | `-b` | Comment body | — |
-| `--editor` | — | Open `$EDITOR` to write the comment | — |
+| `--body <text>` | `-b` | Thread body | — |
+| `--editor` | — | Open `$EDITOR` to write the thread body | — |
+| `--file <path>` | — | File path to anchor the thread (relative to repo root) | — |
+| `--line <n[-n]>` | — | Line number or range, e.g. `42` or `42-55`; requires `--file` | — |
 | `--repo <repo>` | `-r` | Repository name | — |
 | `--project <project>` | `-p` | Azure DevOps project | — |
 | `--org <url>` | — | Organization URL | — |

src/api/pullRequests.ts

@@ -218,20 +218,28 @@ export async function getPullRequest(
   }
 }
 
-export async function addPrComment(
+export async function createPrThread(
   connection: azdev.WebApi,
   project: string,
   repoName: string,
   prId: number,
-  body: string
+  body: string,
+  fileContext?: { file: string; lineStart: number; lineEnd: number }
 ): Promise<void> {
   const gitApi = await connection.getGitApi();
 
+  const threadContext = fileContext ? {
+    filePath: fileContext.file,
+    rightFileStart: { line: fileContext.lineStart, offset: 1 },
+    rightFileEnd: { line: fileContext.lineEnd, offset: 1 },
+  } : undefined;
+
   try {
     await gitApi.createThread(
       {
         comments: [{ content: body, commentType: 1 }],
         status: 1, // Active
+        threadContext,
       },
       repoName,
       prId,

src/commands/pr/comment.ts

@@ -1,7 +1,6 @@
 import { Command } from 'commander';
 import { registerPrList } from './list.js';
 import { registerPrView } from './view.js';
-import { registerPrComment } from './comment.js';
 import { registerPrDiff } from './diff.js';
 import { registerPrCreate } from './create.js';
 import { registerPrReview } from './review.js';
@@ -11,7 +10,6 @@ export function registerPrCommands(program: Command): void {
   const pr = program.command('pr').description('Manage Azure DevOps pull requests');
   registerPrList(pr);
   registerPrView(pr);
-  registerPrComment(pr);
   registerPrDiff(pr);
   registerPrCreate(pr);
   registerPrReview(pr);

src/commands/pr/index.ts

@@ -4,7 +4,7 @@ import * as os from 'os';
 import * as path from 'path';
 import * as child_process from 'child_process';
 import { getWebApi } from '../../api/client.js';
-import { replyToThread, setThreadStatus, resolveRepo } from '../../api/pullRequests.js';
+import { createPrThread, replyToThread, setThreadStatus, resolveRepo } from '../../api/pullRequests.js';
 import { getConfig } from '../../config/index.js';
 import { AdoError } from '../../errors/index.js';
 
@@ -24,6 +24,60 @@ function openEditor(initial = ''): string {
   return content;
 }
 
+async function prThreadCreateHandler(
+  prId: string,
+  options: { body?: string; editor?: boolean; file?: string; line?: string; repo?: string; project?: string; org?: string }
+): Promise<void> {
+  const numId = parseInt(prId, 10);
+  if (isNaN(numId)) {
+    process.stderr.write(`ado: invalid PR number: ${prId}\n`);
+    process.exit(1);
+  }
+
+  if (options.line && !options.file) {
+    process.stderr.write('ado: --line requires --file\n');
+    process.exit(1);
+  }
+
+  let body = options.body;
+  if (!body || options.editor) {
+    body = openEditor(body ?? '');
+  }
+  if (!body) {
+    throw new AdoError('Thread body cannot be empty.');
+  }
+
+  let fileContext: { file: string; lineStart: number; lineEnd: number } | undefined;
+  if (options.file) {
+    let lineStart = 1;
+    let lineEnd = 1;
+    if (options.line) {
+      const match = options.line.match(/^(\d+)(?:-(\d+))?$/);
+      if (!match) {
+        process.stderr.write(`ado: invalid --line value: ${options.line}  (use N or N-M)\n`);
+        process.exit(1);
+      }
+      lineStart = parseInt(match[1], 10);
+      lineEnd = match[2] !== undefined ? parseInt(match[2], 10) : lineStart;
+      if (lineEnd < lineStart) {
+        process.stderr.write('ado: --line end must be >= start\n');
+        process.exit(1);
+      }
+    }
+    fileContext = { file: options.file, lineStart, lineEnd };
+  }
+
+  const config = getConfig({ orgUrl: options.org, project: options.project });
+  const connection = await getWebApi(config.orgUrl);
+  const repoName = await resolveRepo(connection, config.project, options.repo);
+
+  await createPrThread(connection, config.project, repoName, numId, body, fileContext);
+  const location = fileContext
+    ? ` on ${fileContext.file}:${fileContext.lineStart}${fileContext.lineEnd !== fileContext.lineStart ? `-${fileContext.lineEnd}` : ''}`
+    : '';
+  process.stdout.write(`Created thread on PR #${numId}${location}\n`);
+}
+
 async function prThreadReplyHandler(
   prId: string,
   threadId: string,
@@ -116,6 +170,18 @@ export function registerPrThread(prCmd: Command): void {
     .command('thread')
     .description('Manage pull request comment threads');
 
+  thread
+    .command('create <pr-number>')
+    .description('Create a new comment thread on a pull request')
+    .option('-b, --body <text>', 'Thread body')
+    .option('--editor', 'Open $EDITOR to write the thread body')
+    .option('--file <path>', 'File path to anchor the thread to (relative to repo root)')
+    .option('--line <n[-n]>', 'Line number or range (e.g. 42 or 42-55); requires --file')
+    .option('-r, --repo <repo>', 'Repository name')
+    .option('-p, --project <project>', 'Azure DevOps project (overrides config)')
+    .option('--org <url>', 'Azure DevOps organization URL (overrides config)')
+    .action(prThreadCreateHandler);
+
   thread
     .command('reply <pr-number> <thread-id>')
     .description('Reply to an existing comment thread')