azdo-cli 0.24.0-rescue-develop-20261106T154004Z.916 → 0.25.0-develop.933

package/README.md

@@ -162,6 +162,7 @@ azdo relations list 1000 --json             # JSON: { workItemId, relations: [..
 | Linux credential store setup | [docs/linux-credential-store.md](docs/linux-credential-store.md) |
 | Full command reference — every command, alias and option, credential resolution order, and the `--json` output contracts | [docs/commands.md](docs/commands.md) |
 | Development setup | [docs/development.md](docs/development.md) |
+| Project operations and automation (Dev Container, VM, Hermes) | [instructions/README.md](instructions/README.md) |
 
 ## License
 

package/dist/{chunk-DNOOZIXA.js → chunk-WCO6SF63.js}

@@ -1047,6 +1047,17 @@ var DpapiEntry = class {
   }
 };
 
+// src/lib/async.ts
+function runSequentially(items, task) {
+  return items.reduce(
+    (previous, item, index) => previous.then(() => task(item, index)),
+    Promise.resolve()
+  );
+}
+function asPromise(fn) {
+  return Promise.resolve().then(fn);
+}
+
 // src/services/credential-store.ts
 var SERVICE = "azdo-cli";
 var LEGACY_ACCOUNT = "pat";
@@ -1187,7 +1198,7 @@ function serializeCredential(cred) {
   }
   return JSON.stringify(cred);
 }
-async function maybeMigrateLegacy(targetOrg) {
+function maybeMigrateLegacy(targetOrg) {
   if (activeCredentialStore() === "dpapi") return null;
   const config = loadConfig();
   if (!config.org || config.org !== targetOrg) {
@@ -1235,30 +1246,31 @@ async function getPat(org) {
   if (cred.kind === "pat") return cred.token;
   return null;
 }
-async function getStoredCredential(org) {
-  const entry = entryFor(accountFor(org));
-  const value = wrapUnavailable(() => entry.getPassword());
-  if (value === null) {
-    const migrated = await maybeMigrateLegacy(org);
-    if (migrated === null) return null;
-    return parseStoredValue(migrated);
-  }
-  return parseStoredValue(value);
+function getStoredCredential(org) {
+  return asPromise(() => {
+    const entry = entryFor(accountFor(org));
+    const value = wrapUnavailable(() => entry.getPassword()) ?? maybeMigrateLegacy(org);
+    return value === null ? null : parseStoredValue(value);
+  });
 }
-async function storePat(org, pat) {
-  const cred = { kind: "pat", token: pat };
-  const entry = entryFor(accountFor(org));
-  wrapUnavailable(() => entry.setPassword(serializeCredential(cred)));
-  appendAuthAuditEvent({
-    event: "auth.store",
-    org,
-    backend: probeBackend(),
-    masked_pat: maskedDisplay(pat)
+function storePat(org, pat) {
+  return asPromise(() => {
+    const cred = { kind: "pat", token: pat };
+    const entry = entryFor(accountFor(org));
+    wrapUnavailable(() => entry.setPassword(serializeCredential(cred)));
+    appendAuthAuditEvent({
+      event: "auth.store",
+      org,
+      backend: probeBackend(),
+      masked_pat: maskedDisplay(pat)
+    });
   });
 }
-async function storeOAuthCredential(org, cred) {
-  const entry = entryFor(accountFor(org));
-  wrapUnavailable(() => entry.setPassword(serializeCredential(cred)));
+function storeOAuthCredential(org, cred) {
+  return asPromise(() => {
+    const entry = entryFor(accountFor(org));
+    wrapUnavailable(() => entry.setPassword(serializeCredential(cred)));
+  });
 }
 async function deletePat(org) {
   const entry = entryFor(accountFor(org));
@@ -1290,13 +1302,16 @@ async function deletePat(org) {
   }
   try {
     const { unlinkSync: unlinkSync2 } = await import("fs");
-    const { lockPath: lockPath2 } = await import("./oauth-token-refresh-RLDAAR5D.js");
+    const { lockPath: lockPath2 } = await import("./oauth-token-refresh-4WISWRPJ.js");
     unlinkSync2(lockPath2(org));
   } catch {
   }
   return true;
 }
-async function listOrgsWithStoredPat() {
+function listOrgsWithStoredPat() {
+  return asPromise(listOrgsWithStoredPatSync);
+}
+function listOrgsWithStoredPatSync() {
   const seen = /* @__PURE__ */ new Set();
   for (const ev of readAuditEvents()) {
     if (ev.event === "auth.store" || ev.event === "oauth-login-success") {
@@ -1387,25 +1402,26 @@ function lockPath(org) {
 async function defaultAcquireLock(org, waitMs = DEFAULT_LOCK_WAIT_MS) {
   const path4 = lockPath(org);
   mkdirSync(locksDir(), { recursive: true, mode: 448 });
-  const deadline = Date.now() + waitMs;
-  for (; ; ) {
-    try {
-      const fd = openSync2(path4, "wx");
-      closeSync2(fd);
-      return {
-        release: () => {
-          try {
-            unlinkSync(path4);
-          } catch {
-          }
+  return tryAcquireLock(path4, Date.now() + waitMs);
+}
+async function tryAcquireLock(path4, deadline) {
+  try {
+    const fd = openSync2(path4, "wx");
+    closeSync2(fd);
+    return {
+      release: () => {
+        try {
+          unlinkSync(path4);
+        } catch {
         }
-      };
-    } catch (err) {
-      if (err.code !== "EEXIST") throw err;
-      if (Date.now() > deadline) return null;
-      await new Promise((r) => setTimeout(r, 100));
-    }
+      }
+    };
+  } catch (err) {
+    if (err.code !== "EEXIST") throw err;
+    if (Date.now() > deadline) return null;
   }
+  await new Promise((r) => setTimeout(r, 100));
+  return tryAcquireLock(path4, deadline);
 }
 function classifyRefreshFailure(error) {
   const code = typeof error === "string" ? error : error.error ?? "";
@@ -1543,6 +1559,7 @@ export {
   CredentialStoreUnavailableError,
   CredentialMissingError,
   CredentialRefreshError,
+  runSequentially,
   appendAuthAuditEvent,
   readAuditEvents,
   SETTINGS,

package/dist/index.js

@@ -37,6 +37,7 @@ import {
   resolveOAuthConfig,
   resolveScopedConfig,
   runAuthCodeFlow,
+  runSequentially,
   setConfigValue,
   setOrgScopedValue,
   storeOAuthCredential,
@@ -45,7 +46,7 @@ import {
   tokenResponseToCredential,
   unsetConfigValue,
   unsetOrgScopedValue
-} from "./chunk-DNOOZIXA.js";
+} from "./chunk-WCO6SF63.js";
 
 // src/program.ts
 import { Command as Command20 } from "commander";
@@ -154,6 +155,14 @@ function handleCommandError(err, id, context, scope = "write", exit = true) {
     return;
   }
   const error = err instanceof Error ? err : new Error(String(err));
+  writeCommandError(error, id, context, scope);
+  if (exit) {
+    process.exit(1);
+  } else {
+    process.exitCode = 1;
+  }
+}
+function writeCommandError(error, id, context, scope) {
   const msg = error.message;
   const scopeLabel = scope === "read" ? "Work Items (read)" : "Work Items (Read & Write)";
   if (isSentinel(msg, "AUTH_FAILED")) {
@@ -193,11 +202,6 @@ function handleCommandError(err, id, context, scope = "write", exit = true) {
     process.stderr.write(`Error: ${msg}
 `);
   }
-  if (exit) {
-    process.exit(1);
-  } else {
-    process.exitCode = 1;
-  }
 }
 var STDIN_PATH = "-";
 function readTextSource(file) {
@@ -340,9 +344,8 @@ async function downloadImagesFromFields(fields, args, credential) {
     }
   }
   const results = [];
-  let index = 0;
-  for (const reference of references) {
-    index += 1;
+  await runSequentially(references, async (reference, position) => {
+    const index = position + 1;
     try {
       const bytes = await downloadAttachment(reference.url, credential);
       const processed = await processImageBytes(bytes, options.maxWidth);
@@ -363,7 +366,7 @@ async function downloadImagesFromFields(fields, args, credential) {
         error: err instanceof Error ? err.message : String(err)
       });
     }
-  }
+  });
   return results;
 }
 async function runImageDownload(fields, args, credential) {
@@ -846,23 +849,24 @@ async function getWorkItemFieldValue(context, id, cred, fieldName) {
   }
   return stringifyFieldValue(value);
 }
+async function listWorkItemCommentPages(context, id, cred, continuationToken, comments) {
+  const response = await fetchWithErrors(
+    buildWorkItemCommentsListUrl(context, id, continuationToken ?? void 0).toString(),
+    { headers: authHeaders(cred) }
+  );
+  if (!response.ok) {
+    throw httpError(response);
+  }
+  const data = await response.json();
+  comments.push(
+    ...(data.comments ?? []).map((comment) => mapWorkItemComment(comment, id)).filter((comment) => !comment.isDeleted)
+  );
+  const next = readContinuationToken(response, data);
+  if (next === null) return comments;
+  return listWorkItemCommentPages(context, id, cred, next, comments);
+}
 async function listWorkItemComments(context, id, cred) {
-  const comments = [];
-  let continuationToken = null;
-  do {
-    const response = await fetchWithErrors(
-      buildWorkItemCommentsListUrl(context, id, continuationToken ?? void 0).toString(),
-      { headers: authHeaders(cred) }
-    );
-    if (!response.ok) {
-      throw httpError(response);
-    }
-    const data = await response.json();
-    comments.push(
-      ...(data.comments ?? []).map((comment) => mapWorkItemComment(comment, id)).filter((comment) => !comment.isDeleted)
-    );
-    continuationToken = readContinuationToken(response, data);
-  } while (continuationToken !== null);
+  const comments = await listWorkItemCommentPages(context, id, cred, null, []);
   return {
     workItemId: id,
     count: comments.length,
@@ -1442,29 +1446,31 @@ async function loginWithOAuth(org, opts = {}) {
 async function logout(opts = {}) {
   if (opts.all) {
     const orgs = await listOrgsWithStoredPat();
-    const removed = [];
-    for (const o of orgs) {
-      const cred2 = await getStoredCredential(o);
-      const ok2 = await deletePat(o);
-      if (ok2 && cred2 !== null) {
-        removed.push({ org: o, kind: cred2.kind });
-      }
-    }
-    return { removed };
+    return { removed: await logoutOrgsInOrder(orgs, 0, []) };
   }
   if (!opts.org) {
     throw new Error("logout requires an org or --all");
   }
-  const cred = await getStoredCredential(opts.org);
-  const ok = await deletePat(opts.org);
-  return { removed: ok && cred !== null ? [{ org: opts.org, kind: cred.kind }] : [] };
+  return { removed: await logoutOrg(opts.org) };
+}
+async function logoutOrgsInOrder(orgs, index, removed) {
+  const org = orgs[index];
+  if (org === void 0) return removed;
+  removed.push(...await logoutOrg(org));
+  return logoutOrgsInOrder(orgs, index + 1, removed);
+}
+async function logoutOrg(org) {
+  const cred = await getStoredCredential(org);
+  const ok = await deletePat(org);
+  return ok && cred !== null ? [{ org, kind: cred.kind }] : [];
 }
 async function status() {
   const orgs = await listOrgsWithStoredPat();
+  const creds = await Promise.all(orgs.map((org) => getStoredCredential(org)));
   const out = [];
-  for (const org of orgs) {
-    const cred = await getStoredCredential(org);
-    if (cred === null) continue;
+  orgs.forEach((org, i) => {
+    const cred = creds[i];
+    if (cred === null) return;
     if (cred.kind === "pat") {
       out.push({ org, kind: "pat", backend: probeBackend() });
     } else {
@@ -1477,7 +1483,7 @@ async function status() {
         backend: probeBackend()
       });
     }
-  }
+  });
   return { orgs: out };
 }
 async function exportCredential(org) {
@@ -3128,10 +3134,8 @@ ${renderSettingsHelp()}`);
     const ask = createAsk(rl);
     process.stderr.write("Azure DevOps CLI - Configuration Wizard\n");
     process.stderr.write("=======================================\n\n");
-    for (const setting of SETTINGS) {
-      if (setting.key === "credentialStore") continue;
-      await promptForSetting(cfg, setting, ask);
-    }
+    const wizardSettings = SETTINGS.filter((setting) => setting.key !== "credentialStore");
+    await runSequentially(wizardSettings, (setting) => promptForSetting(cfg, setting, ask));
     rl.close();
     process.stderr.write("Configuration complete!\n");
   });
@@ -3771,7 +3775,8 @@ function stringifyValue(value) {
   if (value === null || value === void 0) return "";
   if (typeof value === "string") return value;
   if (typeof value === "object") return JSON.stringify(value);
-  if (typeof value === "symbol" || typeof value === "function") return value.toString();
+  if (typeof value === "symbol") return value.toString();
+  if (typeof value === "function") return `[function ${value.name || "anonymous"}]`;
   return String(value);
 }
 function formatRichValue(raw) {
@@ -4296,7 +4301,7 @@ async function getPullRequestThread(context, repo, cred, prId, threadId) {
   const data = await readJsonResponse(response);
   return toActiveCommentThread(data);
 }
-async function createPullRequestThread(context, repo, cred, prId, content, status2) {
+async function createPullRequestThread(context, repo, cred, prId, content, status2, anchor) {
   const url = new URL(
     `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/threads`
   );
@@ -4307,6 +4312,10 @@ async function createPullRequestThread(context, repo, cred, prId, content, statu
   if (status2 !== void 0) {
     payload.status = status2;
   }
+  if (anchor !== void 0) {
+    payload.threadContext = anchor.threadContext;
+    payload.pullRequestThreadContext = anchor.pullRequestThreadContext;
+  }
   const response = await fetchWithErrors(url.toString(), {
     method: "POST",
     headers: {
@@ -4318,6 +4327,42 @@ async function createPullRequestThread(context, repo, cred, prId, content, statu
   const data = await readJsonResponse(response);
   return toActiveCommentThread(data);
 }
+function buildIterationsUrl(context, repo, prId, suffix = "") {
+  const url = new URL(
+    `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/iterations${suffix}`
+  );
+  url.searchParams.set("api-version", "7.1");
+  return url;
+}
+async function getLatestPullRequestIterationId(context, repo, cred, prId) {
+  const url = buildIterationsUrl(context, repo, prId);
+  const response = await fetchWithErrors(url.toString(), { headers: authHeaders(cred) });
+  const data = await readJsonResponse(response);
+  const ids = (data.value ?? []).map((iteration) => iteration.id);
+  if (ids.length === 0) {
+    throw new Error(`Pull request #${prId} has no iterations.`);
+  }
+  return Math.max(...ids);
+}
+async function getPullRequestIterationChanges(context, repo, cred, prId, iterationId) {
+  const changes = [];
+  let skip = 0;
+  for (; ; ) {
+    const url = buildIterationsUrl(context, repo, prId, `/${iterationId}/changes`);
+    url.searchParams.set("$top", "2000");
+    if (skip > 0) {
+      url.searchParams.set("$skip", String(skip));
+    }
+    const response = await fetchWithErrors(url.toString(), { headers: authHeaders(cred) });
+    const page = await readJsonResponse(response);
+    changes.push(...page.changeEntries ?? []);
+    const next = page.nextSkip ?? 0;
+    if (next <= skip) {
+      return changes;
+    }
+    skip = next;
+  }
+}
 function buildThreadCommentUrl(context, repo, prId, threadId) {
   const url = new URL(
     `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/git/repositories/${encodeURIComponent(repo)}/pullRequests/${prId}/threads/${threadId}/comments`
@@ -4641,6 +4686,39 @@ async function resolvePullRequestTemplate(context, repo, cred, defaultBranch, ta
   return firstMatchingTemplate(context, repo, cred, defaultBranch, defaultTemplateCandidates(), "default");
 }
 
+// src/services/thread-anchor.ts
+var LINE_END_OFFSET = 1e3;
+function normalizeThreadFilePath(raw) {
+  if (raw.trim() === "") {
+    return null;
+  }
+  let path5 = raw.replaceAll("\\", "/");
+  while (path5.startsWith("./")) {
+    path5 = path5.slice(2);
+  }
+  path5 = path5.replace(/^\/+/, "");
+  return path5 === "" ? null : `/${path5}`;
+}
+function buildThreadAnchorContext(filePath, line, endLine, side) {
+  const start = { line, offset: 1 };
+  const end = { line: endLine, offset: LINE_END_OFFSET };
+  return side === "right" ? { filePath, rightFileStart: start, rightFileEnd: end } : { filePath, leftFileStart: start, leftFileEnd: end };
+}
+function buildPullRequestThreadContext(iterationId, changeTrackingId) {
+  const context = {
+    iterationContext: { firstComparingIteration: iterationId, secondComparingIteration: iterationId }
+  };
+  if (changeTrackingId !== void 0) {
+    context.changeTrackingId = changeTrackingId;
+  }
+  return context;
+}
+function findChangeTrackingId(changes, filePath, side) {
+  const byPath = (change) => change.item?.path === filePath;
+  const match = side === "left" ? changes.find((change) => change.originalPath === filePath) ?? changes.find(byPath) : changes.find(byPath);
+  return match?.changeTrackingId;
+}
+
 // src/commands/pr.ts
 function parsePositivePrNumber(raw) {
   if (!/^\d+$/.test(raw)) {
@@ -5794,6 +5872,130 @@ var CREATABLE_THREAD_STATUSES = [
   "byDesign",
   "pending"
 ];
+var THREAD_ANCHOR_SIDES = ["right", "left"];
+function parseCommentAddStatus(raw) {
+  if (raw === void 0) {
+    return void 0;
+  }
+  const match = CREATABLE_THREAD_STATUSES.find((candidate) => candidate === raw);
+  if (match === void 0) {
+    writeError(`Invalid --status "${raw}"; expected one of ${CREATABLE_THREAD_STATUSES.join(", ")}.`);
+    return null;
+  }
+  return match;
+}
+function parseLineOption(flag, raw) {
+  const value = parsePositivePrNumber(raw);
+  if (value === null) {
+    writeError(`Invalid ${flag} "${raw}"; expected a positive integer (1-based line number).`);
+  }
+  return value;
+}
+function parseCommentAnchorOptions(options) {
+  if (options.filePath === void 0) {
+    const stray = [
+      ["--line", options.line],
+      ["--end-line", options.endLine],
+      ["--side", options.side],
+      ["--allow-unchanged-file", options.allowUnchangedFile]
+    ].find(([, value]) => value !== void 0);
+    if (stray !== void 0) {
+      writeError(`${stray[0]} requires --file-path.`);
+      return null;
+    }
+    return void 0;
+  }
+  if (options.line === void 0) {
+    writeError("--file-path requires --line.");
+    return null;
+  }
+  const filePath = normalizeThreadFilePath(options.filePath);
+  if (filePath === null) {
+    writeError(`Invalid --file-path "${options.filePath}"; expected a repository file path.`);
+    return null;
+  }
+  const line = parseLineOption("--line", options.line);
+  if (line === null) {
+    return null;
+  }
+  const endLine = options.endLine === void 0 ? line : parseLineOption("--end-line", options.endLine);
+  if (endLine === null) {
+    return null;
+  }
+  if (endLine < line) {
+    writeError(`--end-line (${endLine}) must not be less than --line (${line}).`);
+    return null;
+  }
+  const side = THREAD_ANCHOR_SIDES.find((candidate) => candidate === (options.side ?? "right"));
+  if (side === void 0) {
+    writeError(`Invalid --side "${options.side}"; expected ${THREAD_ANCHOR_SIDES.join(" or ")}.`);
+    return null;
+  }
+  return { filePath, line, endLine, side, allowUnchangedFile: options.allowUnchangedFile === true };
+}
+async function resolveCommentAnchor(target, request) {
+  const prId = target.pullRequest.id;
+  const iterationId = await getLatestPullRequestIterationId(target.context, target.repo, target.pat, prId);
+  const changes = await getPullRequestIterationChanges(target.context, target.repo, target.pat, prId, iterationId);
+  const changeTrackingId = findChangeTrackingId(changes, request.filePath, request.side);
+  if (changeTrackingId === void 0 && !request.allowUnchangedFile) {
+    writeError(
+      `${request.filePath} is not among the files changed by pull request #${prId} (iteration ${iterationId}). Check the path, or pass --allow-unchanged-file to post the comment anyway.`,
+      3
+    );
+    return null;
+  }
+  return {
+    request,
+    iterationId,
+    changeTrackingId,
+    threadContext: buildThreadAnchorContext(request.filePath, request.line, request.endLine, request.side),
+    pullRequestThreadContext: buildPullRequestThreadContext(iterationId, changeTrackingId)
+  };
+}
+function describeCommentAnchor(anchor) {
+  const { filePath, line, endLine, side } = anchor.request;
+  const lines = line === endLine ? `line ${line}` : `lines ${line}-${endLine}`;
+  const tracking = anchor.changeTrackingId === void 0 ? "no changeTrackingId" : `changeTrackingId ${anchor.changeTrackingId}`;
+  return ` at ${filePath} ${lines} (${side} side, iteration ${anchor.iterationId}, ${tracking})`;
+}
+function warnIfUntracked(anchor) {
+  if (anchor !== void 0 && anchor.changeTrackingId === void 0) {
+    process.stderr.write(
+      `Note: ${anchor.request.filePath} is not among the pull request's changed files, so the thread is posted without a changeTrackingId and will not follow the line across later pushes.
+`
+    );
+  }
+}
+function withAnchorFields(result, anchor) {
+  if (anchor === void 0) {
+    return result;
+  }
+  return {
+    ...result,
+    threadContext: anchor.threadContext,
+    pullRequestThreadContext: anchor.pullRequestThreadContext,
+    changeTracked: anchor.changeTrackingId !== void 0
+  };
+}
+function writeCommentAddDryRun(prId, body, status2, anchor, json) {
+  if (json) {
+    const dryResult = withAnchorFields(
+      { pullRequestId: prId, threadId: null, commentId: null, status: status2 ?? null, content: body, dryRun: true },
+      anchor
+    );
+    process.stdout.write(`${JSON.stringify(dryResult, null, 2)}
+`);
+    return;
+  }
+  const statusSuffix = status2 === void 0 ? "" : ` with status ${status2}`;
+  const anchorSuffix = anchor === void 0 ? "" : describeCommentAnchor(anchor);
+  process.stdout.write(
+    `Dry run: would post a new comment thread${statusSuffix} on pull request #${prId}${anchorSuffix} (${body.length} chars).
+${body}
+`
+  );
+}
 async function runCommentAdd(text, options) {
   let context;
   try {
@@ -5801,16 +6003,13 @@ async function runCommentAdd(text, options) {
     if (body === null) {
       return;
     }
-    let status2;
-    if (options.status !== void 0) {
-      const match = CREATABLE_THREAD_STATUSES.find((candidate) => candidate === options.status);
-      if (match === void 0) {
-        writeError(
-          `Invalid --status "${options.status}"; expected one of ${CREATABLE_THREAD_STATUSES.join(", ")}.`
-        );
-        return;
-      }
-      status2 = match;
+    const status2 = parseCommentAddStatus(options.status);
+    if (status2 === null) {
+      return;
+    }
+    const anchorRequest = parseCommentAnchorOptions(options);
+    if (anchorRequest === null) {
+      return;
     }
     const target = await resolvePullRequestTarget(options, {
       onContextResolved: (resolved) => {
@@ -5820,26 +6019,13 @@ async function runCommentAdd(text, options) {
     if (target === null) {
       return;
     }
+    const anchor = anchorRequest === void 0 ? void 0 : await resolveCommentAnchor(target, anchorRequest);
+    if (anchor === null) {
+      return;
+    }
+    warnIfUntracked(anchor);
     if (options.dryRun === true) {
-      const dryResult = {
-        pullRequestId: target.pullRequest.id,
-        threadId: null,
-        commentId: null,
-        status: status2 ?? null,
-        content: body,
-        dryRun: true
-      };
-      if (options.json) {
-        process.stdout.write(`${JSON.stringify(dryResult, null, 2)}
-`);
-        return;
-      }
-      const statusSuffix = status2 === void 0 ? "" : ` with status ${status2}`;
-      process.stdout.write(
-        `Dry run: would post a new comment thread${statusSuffix} on pull request #${target.pullRequest.id} (${body.length} chars).
-${body}
-`
-      );
+      writeCommentAddDryRun(target.pullRequest.id, body, status2, anchor, options.json);
       return;
     }
     const thread = await createPullRequestThread(
@@ -5848,24 +6034,29 @@ ${body}
       target.pat,
       target.pullRequest.id,
       body,
-      status2
+      status2,
+      ...anchor === void 0 ? [] : [{ threadContext: anchor.threadContext, pullRequestThreadContext: anchor.pullRequestThreadContext }]
     );
     const created = thread.comments[0];
-    const result = {
-      pullRequestId: target.pullRequest.id,
-      threadId: thread.id,
-      commentId: created?.id ?? null,
-      status: thread.status,
-      content: created?.content ?? body,
-      dryRun: false
-    };
+    const result = withAnchorFields(
+      {
+        pullRequestId: target.pullRequest.id,
+        threadId: thread.id,
+        commentId: created?.id ?? null,
+        status: thread.status,
+        content: created?.content ?? body,
+        dryRun: false
+      },
+      anchor
+    );
     if (options.json) {
       process.stdout.write(`${JSON.stringify(result, null, 2)}
 `);
       return;
     }
+    const anchorSuffix = anchor === void 0 ? "" : describeCommentAnchor(anchor);
     process.stdout.write(
-      `Comment posted to pull request #${target.pullRequest.id} (thread #${thread.id}).
+      `Comment posted to pull request #${target.pullRequest.id} (thread #${thread.id})${anchorSuffix}.
 `
     );
   } catch (err) {
@@ -5877,18 +6068,21 @@ function buildCommentAddCommand(name, description) {
   withCommonPrOptions(configureUnwrappedHelp(command)).description(description).argument("[text]", "body of the new comment; omit when using --file").option("--file <path>", "read the comment body from a UTF-8 file instead of the inline argument").option(
     "--status <status>",
     `thread status (${CREATABLE_THREAD_STATUSES.join(" | ")}); omit for a plain, non-resolvable comment`
+  ).option("--file-path <path>", "anchor the thread to this repository file (Files tab); requires --line").option("--line <n>", "first line of the anchored range (1-based)").option("--end-line <n>", "last line of the anchored range (default: --line)").option("--side <side>", "right (new version, default) or left (old version, for deleted lines)").option(
+    "--allow-unchanged-file",
+    "post even when --file-path is not among the pull request's changed files (the thread will not follow the line across pushes)"
   ).option("--dry-run", "resolve the target pull request and print what would be posted, without writing anything").option("--pr-number <N>", PR_NUMBER_HELP).option("--json", "output JSON").action(async (text, _options, command2) => {
     await runCommentAdd(text, mergedPrOptions(command2));
   });
   return command;
 }
 function createPrCommentsAddCommand() {
-  return buildCommentAddCommand("add", "Post a new comment thread on the pull request overview");
+  return buildCommentAddCommand("add", "Post a new comment thread on the pull request overview, or on a file and line");
 }
 function createPrCommentAddCommand() {
   return buildCommentAddCommand(
     "comment-add",
-    'Post a new comment thread on the pull request overview (alias of "azdo pr comments add")'
+    'Post a new comment thread on the pull request overview, or on a file and line (alias of "azdo pr comments add")'
   );
 }
 function parseExplicitCommentId(options) {
@@ -6178,13 +6372,15 @@ 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 };
+    if (next >= pullRequests.length) {
+      return;
     }
+    const index = next;
+    next += 1;
+    const pullRequest = pullRequests[index];
+    const workItemIds = await getPullRequestWorkItemIds(context, repo, cred, pullRequest.id);
+    result[index] = { ...pullRequest, workItemIds };
+    await worker();
   };
   await Promise.all(Array.from({ length: Math.min(WORK_ITEM_LOOKUP_CONCURRENCY, pullRequests.length) }, worker));
   return result;
@@ -6680,26 +6876,27 @@ async function getTestSummary(context, cred, buildId) {
 var MAX_FAILED_TESTS = 50;
 async function getFailedTests(context, cred, buildId) {
   const testRuns = await listTestRuns(context, cred, buildId);
-  const failed = [];
-  for (const testRun of testRuns) {
-    if (failed.length >= MAX_FAILED_TESTS) break;
-    const resultsUrl = withApiVersion(
-      new URL(`${orgProjectBase(context)}/_apis/test/runs/${testRun.id}/results`)
-    );
-    resultsUrl.searchParams.set("outcomes", "Failed");
-    resultsUrl.searchParams.set("$top", String(MAX_FAILED_TESTS - failed.length));
-    const resultsResponse = await fetchWithErrors(resultsUrl.toString(), {
-      headers: authHeaders(cred)
+  return collectFailedTests(context, cred, testRuns, 0, []);
+}
+async function collectFailedTests(context, cred, testRuns, index, failed) {
+  const testRun = testRuns[index];
+  if (testRun === void 0 || failed.length >= MAX_FAILED_TESTS) return failed;
+  const resultsUrl = withApiVersion(
+    new URL(`${orgProjectBase(context)}/_apis/test/runs/${testRun.id}/results`)
+  );
+  resultsUrl.searchParams.set("outcomes", "Failed");
+  resultsUrl.searchParams.set("$top", String(MAX_FAILED_TESTS - failed.length));
+  const resultsResponse = await fetchWithErrors(resultsUrl.toString(), {
+    headers: authHeaders(cred)
+  });
+  const resultsData = await readJsonResponse2(resultsResponse);
+  for (const result of resultsData.value) {
+    failed.push({
+      name: result.testCaseTitle ?? result.automatedTestName ?? "(unnamed test)",
+      errorMessage: result.errorMessage ?? null
     });
-    const resultsData = await readJsonResponse2(resultsResponse);
-    for (const result of resultsData.value) {
-      failed.push({
-        name: result.testCaseTitle ?? result.automatedTestName ?? "(unnamed test)",
-        errorMessage: result.errorMessage ?? null
-      });
-    }
   }
-  return failed;
+  return collectFailedTests(context, cred, testRuns, index + 1, failed);
 }
 function secondsBetween(start, finish) {
   if (!start || !finish) return null;
@@ -6972,6 +7169,17 @@ function sleep(ms) {
     setTimeout(resolve2, ms);
   });
 }
+async function pollRunUntilDone(context, cred, runId, timing) {
+  const status2 = await getBuildStatus(context, cred, runId);
+  if (status2.state === "completed") {
+    return { id: runId, state: status2.state, result: status2.result, timedOut: false };
+  }
+  if (Date.now() >= timing.deadline) {
+    return { id: runId, state: status2.state, result: status2.result, timedOut: true };
+  }
+  await sleep(timing.pollMs);
+  return pollRunUntilDone(context, cred, runId, timing);
+}
 function formatTable(rows, rightAlign = /* @__PURE__ */ new Set()) {
   const widths = [];
   for (const row of rows) {
@@ -7134,19 +7342,10 @@ function createPipelineWaitCommand() {
       const resolved = await resolvePipelineContext(options);
       context = resolved.context;
       const deadline = Date.now() + timeoutSec * 1e3;
-      let waitResult = null;
-      for (; ; ) {
-        const status2 = await getBuildStatus(resolved.context, resolved.cred, runId);
-        if (status2.state === "completed") {
-          waitResult = { id: runId, state: status2.state, result: status2.result, timedOut: false };
-          break;
-        }
-        if (Date.now() >= deadline) {
-          waitResult = { id: runId, state: status2.state, result: status2.result, timedOut: true };
-          break;
-        }
-        await sleep(pollSec * 1e3);
-      }
+      const waitResult = await pollRunUntilDone(resolved.context, resolved.cred, runId, {
+        deadline,
+        pollMs: pollSec * 1e3
+      });
       applyWaitExitCode(waitResult);
       if (options.json) {
         process.stdout.write(`${JSON.stringify(waitResult, null, 2)}

package/dist/{oauth-token-refresh-RLDAAR5D.js → oauth-token-refresh-4WISWRPJ.js}

@@ -5,7 +5,7 @@ import {
   lockPath,
   locksDir,
   refreshIfNeeded
-} from "./chunk-DNOOZIXA.js";
+} from "./chunk-WCO6SF63.js";
 export {
   _resetInFlight,
   classifyRefreshFailure,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.24.0-rescue-develop-20261106T154004Z.916",
+  "version": "0.25.0-develop.933",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {