@tryarcanist/cli 0.1.221 → 0.1.223

package/dist/index.js

@@ -69,6 +69,15 @@ function parseApiErrorBody(body) {
     return null;
   }
 }
+function parseConflictBody(err) {
+  if (!(err instanceof ApiError) || err.status !== 409) return null;
+  try {
+    const parsed = JSON.parse(err.body);
+    return parsed && typeof parsed === "object" ? parsed : null;
+  } catch {
+    return null;
+  }
+}
 
 // src/errors.ts
 var EXIT_CODE_INTERRUPTED = 130;
@@ -967,58 +976,97 @@ async function agentSubscriptionLogoutCommand(config, options, command) {
   );
 }
 
-// src/commands/review.ts
-var PR_REVIEW_POLL_INTERVAL_MS = 1e4;
-var PR_REVIEW_WAIT_TIMEOUT_MS = 30 * 60 * 1e3;
-async function reviewCommand(prUrl, options = {}, command) {
-  assertArcanistSessionMutationAllowed("review");
-  const { config } = resolveBusinessContext(command, options);
-  const trigger = await apiFetch(config, "/api/pr-reviews", {
-    method: "POST",
-    body: JSON.stringify({
-      prUrl,
-      ...options.focus ? { focus: options.focus } : {},
-      // Send an explicitly passed model even when empty (e.g. --model "$VAR" with an
-      // unset VAR) so the server rejects it as invalid instead of silently routing.
-      ...options.model !== void 0 ? { model: options.model } : {}
-    })
-  });
-  if (!options.wait || trigger.cached) {
-    emit(command, options, trigger, (payload) => {
-      if (payload.cached) console.log(`Zeus review already completed: ${payload.verdict ?? "unknown"}.`);
-      else console.log(`Started Zeus review${payload.sessionId ? ` (${payload.sessionId})` : ""}.`);
-    });
-    return;
+// ../../shared/utils/timing.ts
+function sleep(ms) {
+  return new Promise((resolve2) => setTimeout(resolve2, ms));
+}
+
+// src/status-poll.ts
+async function pollUntilSettled(opts) {
+  const deadline = Date.now() + opts.timeoutMs;
+  while (Date.now() < deadline) {
+    const settled = await opts.fetchStatus();
+    if (settled !== null) return settled;
+    await sleep(opts.pollIntervalMs);
   }
-  const result = await waitForReview(config, prUrl, trigger.sessionId ?? null);
-  emit(command, options, result, (payload) => {
-    console.log(`Zeus review: ${sanitizeTerminalText(payload.verdict)}. ${payload.findings.length} finding(s).`);
-    for (const finding of payload.findings) {
-      console.log(
-        `${sanitizeTerminalText(finding.severity)} ${sanitizeTerminalText(finding.file)}:${finding.line} - ${sanitizeTerminalText(finding.title)}`
-      );
-    }
-  });
+  throw new CliError("server", opts.timeoutMessage);
 }
-function sanitizeTerminalText(value) {
-  return value.replace(/\u001b\][^\u0007]*(?:\u0007|\u001b\\)/g, "").replace(/\u001b\[[0-?]*[ -/]*[@-~]/g, "").replace(/[\u0000-\u001f\u007f]/g, "");
+function assertSameAttempt(triggeredSessionId, settledSessionId, label) {
+  if (triggeredSessionId && settledSessionId !== triggeredSessionId) {
+    throw new CliError("conflict", `${label} completed for a different attempt; retry the command.`);
+  }
 }
-async function waitForReview(config, prUrl, sessionId) {
-  const deadline = Date.now() + PR_REVIEW_WAIT_TIMEOUT_MS;
-  while (Date.now() < deadline) {
-    const status = await apiFetch(
-      config,
-      `/api/pr-reviews/status?prUrl=${encodeURIComponent(prUrl)}`
-    );
-    if (status.status === "completed") {
-      if (sessionId && status.sessionId !== sessionId) {
-        throw new CliError("conflict", "PR review completed for a different attempt; retry the command.");
-      }
-      return { ...status, sessionId: status.sessionId ?? sessionId };
-    }
-    await new Promise((resolve2) => setTimeout(resolve2, PR_REVIEW_POLL_INTERVAL_MS));
+
+// ../../shared/session/phase.ts
+var PHASES = [
+  "idle",
+  "running",
+  "waiting_for_input",
+  "finalizing",
+  "review_listening",
+  "completed",
+  "superseded",
+  "needs_you",
+  "blocked",
+  "failed",
+  "stopped",
+  "archived"
+];
+var TERMINAL_PHASES_ARRAY = [
+  "completed",
+  "superseded",
+  "needs_you",
+  "blocked",
+  "failed",
+  "stopped",
+  "archived"
+];
+var TERMINAL_PHASES = new Set(TERMINAL_PHASES_ARRAY);
+var TERMINAL_FOR_FALLBACK_POLLING_PHASES = new Set(
+  TERMINAL_PHASES_ARRAY.filter((phase) => phase !== "completed")
+);
+var CHILD_SLOT_RELEASE_PHASES = new Set(
+  TERMINAL_PHASES_ARRAY.filter((phase) => phase !== "stopped")
+);
+var ARCHIVABLE_STALE_TERMINAL_PHASES_ARRAY = TERMINAL_PHASES_ARRAY.filter(
+  (phase) => phase !== "archived" && phase !== "needs_you" && phase !== "blocked" && phase !== "stopped"
+);
+function isTerminalPhase(phase, _sessionKind) {
+  return TERMINAL_PHASES.has(phase);
+}
+
+// src/constants/watch.ts
+var MIN_WATCH_POLL_INTERVAL_MS = 250;
+var DEFAULT_WATCH_POLL_INTERVAL_MS = 1e3;
+var MAX_WATCH_POLL_INTERVAL_MS = 6e4;
+var WATCH_REPLAY_PAGE_SIZE = 200;
+function isWatchTerminal(phase) {
+  return isTerminalPhase(phase);
+}
+
+// src/utils/poll-interval.ts
+function clampPollInterval(ms, minimum) {
+  return Math.max(ms, minimum);
+}
+function parsePollInterval(raw, opts = {}) {
+  const defaultMs = opts.defaultMs ?? DEFAULT_WATCH_POLL_INTERVAL_MS;
+  const minMs = opts.minMs ?? MIN_WATCH_POLL_INTERVAL_MS;
+  const maxMs = opts.maxMs === void 0 ? MAX_WATCH_POLL_INTERVAL_MS : opts.maxMs;
+  if (!raw) return defaultMs;
+  if (!/^\d+$/.test(raw)) {
+    throw new CliError("user", "Polling interval must be a non-negative integer.");
+  }
+  const value = Number(raw);
+  if (value < minMs || maxMs !== null && value > maxMs) {
+    const range = maxMs === null ? `at least ${minMs}` : `between ${minMs} and ${maxMs}`;
+    throw new CliError("user", `Polling interval must be ${range} milliseconds.`);
   }
-  throw new CliError("server", "Timed out waiting for Zeus review results.");
+  return value;
+}
+
+// src/utils/terminal-text.ts
+function sanitizeTerminalText(value) {
+  return value.replace(/\u001b\][^\u0007]*(?:\u0007|\u001b\\)/g, "").replace(/\u001b\[[0-?]*[ -\/]*[@-~]/g, "").replace(/[\u0000-\u001f\u007f]/g, "");
 }
 
 // src/commands/anubis.ts
@@ -1026,6 +1074,7 @@ var ANUBIS_POLL_INTERVAL_MS = 15e3;
 var ANUBIS_WAIT_TIMEOUT_MS = 45 * 60 * 1e3;
 async function anubisCommand(prUrl, options = {}, command) {
   assertArcanistSessionMutationAllowed("anubis");
+  const pollIntervalMs = options.wait ? parsePollInterval(options.pollInterval, { defaultMs: ANUBIS_POLL_INTERVAL_MS, maxMs: null }) : null;
   const { config } = resolveBusinessContext(command, options);
   const trigger = await apiFetch(config, "/api/anubis-runs", {
     method: "POST",
@@ -1037,29 +1086,25 @@ async function anubisCommand(prUrl, options = {}, command) {
     });
     return;
   }
-  const result = await waitForAnubis(config, prUrl, trigger.sessionId);
+  const result = await pollUntilSettled({
+    pollIntervalMs: pollIntervalMs ?? ANUBIS_POLL_INTERVAL_MS,
+    timeoutMs: ANUBIS_WAIT_TIMEOUT_MS,
+    timeoutMessage: "Timed out waiting for Anubis results.",
+    fetchStatus: async () => {
+      const status = await apiFetch(
+        config,
+        `/api/anubis-runs/status?prUrl=${encodeURIComponent(prUrl)}`
+      );
+      if (status.sessionStatus === "active") return null;
+      assertSameAttempt(trigger.sessionId, status.sessionId, "Anubis");
+      return status;
+    }
+  });
   emit(command, options, result, (payload) => {
     console.log(`Anubis: ${sanitizeTerminalText(payload.verdict ?? "could-not-verify")}.`);
     if (payload.summary) console.log(sanitizeTerminalText(payload.summary));
   });
 }
-async function waitForAnubis(config, prUrl, sessionId) {
-  const deadline = Date.now() + ANUBIS_WAIT_TIMEOUT_MS;
-  while (Date.now() < deadline) {
-    const status = await apiFetch(
-      config,
-      `/api/anubis-runs/status?prUrl=${encodeURIComponent(prUrl)}`
-    );
-    if (status.sessionStatus !== "active") {
-      if (status.sessionId !== sessionId) {
-        throw new CliError("conflict", "Anubis completed for a different attempt; retry the command.");
-      }
-      return status;
-    }
-    await new Promise((resolve2) => setTimeout(resolve2, ANUBIS_POLL_INTERVAL_MS));
-  }
-  throw new CliError("server", "Timed out waiting for Anubis results.");
-}
 
 // src/commands/auth.ts
 async function whoamiCommand(options, command) {
@@ -1842,6 +1887,26 @@ var SESSION_START_MODEL_PROVIDER_GROUPS_BY_SUBSCRIPTIONS = {
   "true:true": buildSubscriptionAwareGroups({ codexSubscription: true, grokSubscription: true })
 };
 
+// src/utils/pagination.ts
+var MAX_ALL_PAGES = 1e3;
+async function fetchPages(label, all, initialCursor, fetchPage) {
+  const items = [];
+  let cursor2 = initialCursor;
+  let nextCursor = null;
+  let pageCount = 0;
+  do {
+    pageCount += 1;
+    if (all && pageCount > MAX_ALL_PAGES) {
+      throw new CliError("user", `${label} exceeded ${MAX_ALL_PAGES} pages without reaching the end.`);
+    }
+    const page = await fetchPage(cursor2);
+    items.push(...page.items);
+    nextCursor = page.nextCursor;
+    cursor2 = nextCursor ?? void 0;
+  } while (all && nextCursor);
+  return { items, nextCursor: all ? null : nextCursor };
+}
+
 // ../../shared/github/repo-url.ts
 function parseGithubRepoFullName(value) {
   const shorthand = value.match(/^([a-zA-Z0-9_.-]+)\/([a-zA-Z0-9_.-]+)$/);
@@ -1853,6 +1918,32 @@ function parseGithubRepoFullName(value) {
   return null;
 }
 
+// src/git.ts
+import { execFileSync as execFileSync2 } from "child_process";
+function git(args) {
+  try {
+    return execFileSync2("git", args, { encoding: "utf8", stdio: ["ignore", "pipe", "pipe"] }).trim();
+  } catch (err) {
+    throw new CliError("user", `git ${args.join(" ")} failed: ${stringifyError(err)}`);
+  }
+}
+function currentRepo() {
+  const parsed = parseGithubRepoFullName(git(["remote", "get-url", "origin"]));
+  if (!parsed) throw new CliError("user", "origin remote must point at a GitHub repository");
+  return { owner: parsed.owner, repo: parsed.repo };
+}
+
+// src/utils/repo-arg.ts
+function parseRepoArg(value, argName = "repo") {
+  const parsed = parseGithubRepoFullName(value);
+  if (!parsed) throw new CliError("user", `${argName} must be owner/name or a GitHub URL`);
+  return { owner: parsed.owner, repo: parsed.repo.replace(/\.git$/, "") };
+}
+function parseRepoArgOrCurrent(value, argName = "repo") {
+  if (value === void 0) return currentRepo();
+  return parseRepoArg(value, argName);
+}
+
 // src/commands/automations.ts
 var AUTOMATION_ERROR_HINTS = {
   invalid_cron: "Use a standard five-field cron expression, for example: */15 * * * *",
@@ -1869,9 +1960,8 @@ var AUTOMATION_ERROR_HINTS = {
   repo_skills_unavailable: "Arcanist could not verify the repo's skills right now. Retry once GitHub skill discovery is healthy.",
   unknown_skill: "That leading slash skill does not exist in the selected repository."
 };
-var MAX_ALL_PAGES = 1e3;
 async function createAutomationCommand(repoUrl, promptArg, options, command) {
-  const repo = parseAutomationRepo(repoUrl);
+  const repo = parseRepoArg(repoUrl, "repo-url");
   const prompt = await resolvePromptInput(promptArg, options);
   const slackTeamId = options.slackTeamId?.trim();
   const slackChannelId = options.slackChannelId?.trim();
@@ -1918,27 +2008,22 @@ async function createAutomationCommand(repoUrl, promptArg, options, command) {
 async function listAutomationsCommand(options, command) {
   const runtime = getRuntimeOptions(command, options);
   const config = requireConfig(runtime);
-  const items = [];
-  let cursor2 = options.cursor;
-  let nextCursor = null;
-  let pageCount = 0;
-  do {
-    pageCount += 1;
-    if (options.all === true && pageCount > MAX_ALL_PAGES) {
-      throw new CliError("user", `automations list --all exceeded ${MAX_ALL_PAGES} pages without reaching the end.`);
+  const { items, nextCursor } = await fetchPages(
+    "automations list --all",
+    options.all === true,
+    options.cursor,
+    async (cursor2) => {
+      const query = new URLSearchParams();
+      if (options.limit) query.set("limit", options.limit);
+      if (cursor2) query.set("cursor", cursor2);
+      const payload = await automationApiFetch(
+        config,
+        `/api/automation/schedules${query.size ? `?${query.toString()}` : ""}`
+      );
+      return { items: payload.data.items, nextCursor: payload.data.nextCursor };
     }
-    const query = new URLSearchParams();
-    if (options.limit) query.set("limit", options.limit);
-    if (cursor2) query.set("cursor", cursor2);
-    const payload = await automationApiFetch(
-      config,
-      `/api/automation/schedules${query.size ? `?${query.toString()}` : ""}`
-    );
-    items.push(...payload.data.items);
-    nextCursor = payload.data.nextCursor;
-    cursor2 = nextCursor ?? void 0;
-  } while (options.all === true && nextCursor);
-  const output = { items, nextCursor: options.all === true ? null : nextCursor };
+  );
+  const output = { items, nextCursor };
   if (isJson(command, options)) {
     writeJson(output);
     return;
@@ -1970,14 +2055,6 @@ async function deleteAutomationCommand(id, options, command) {
   }
   console.log(`Deleted automation ${id}.`);
 }
-function parseAutomationRepo(value) {
-  if (/^git@/i.test(value) && !/^git@github\.com:/i.test(value)) {
-    throw new CliError("user", "repo-url must be owner/name or a GitHub URL.");
-  }
-  const parsed = parseGithubRepoFullName(value);
-  if (!parsed) throw new CliError("user", "repo-url must be owner/name or a GitHub URL.");
-  return { owner: parsed.owner, repo: parsed.repo };
-}
 function resolveAutomationModelBackend(rawModel, normalizedModel) {
   if (normalizedModel) {
     const backend = getAgentRuntimeBackendForModel(normalizedModel);
@@ -2185,58 +2262,6 @@ async function codexLogoutCommand(options, command) {
   emit(command, options, payload, () => console.log("Codex subscription auth deactivated and cleared."));
 }
 
-// ../../shared/utils/timing.ts
-function sleep(ms) {
-  return new Promise((resolve2) => setTimeout(resolve2, ms));
-}
-
-// ../../shared/session/phase.ts
-var PHASES = [
-  "idle",
-  "running",
-  "waiting_for_input",
-  "finalizing",
-  "review_listening",
-  "completed",
-  "superseded",
-  "needs_you",
-  "blocked",
-  "failed",
-  "stopped",
-  "archived"
-];
-var TERMINAL_PHASES_ARRAY = [
-  "completed",
-  "superseded",
-  "needs_you",
-  "blocked",
-  "failed",
-  "stopped",
-  "archived"
-];
-var TERMINAL_PHASES = new Set(TERMINAL_PHASES_ARRAY);
-var TERMINAL_FOR_FALLBACK_POLLING_PHASES = new Set(
-  TERMINAL_PHASES_ARRAY.filter((phase) => phase !== "completed")
-);
-var CHILD_SLOT_RELEASE_PHASES = new Set(
-  TERMINAL_PHASES_ARRAY.filter((phase) => phase !== "stopped")
-);
-var ARCHIVABLE_STALE_TERMINAL_PHASES_ARRAY = TERMINAL_PHASES_ARRAY.filter(
-  (phase) => phase !== "archived" && phase !== "needs_you" && phase !== "blocked" && phase !== "stopped"
-);
-function isTerminalPhase(phase, _sessionKind) {
-  return TERMINAL_PHASES.has(phase);
-}
-
-// src/constants/watch.ts
-var MIN_WATCH_POLL_INTERVAL_MS = 250;
-var DEFAULT_WATCH_POLL_INTERVAL_MS = 1e3;
-var MAX_WATCH_POLL_INTERVAL_MS = 6e4;
-var WATCH_REPLAY_PAGE_SIZE = 200;
-function isWatchTerminal(phase) {
-  return isTerminalPhase(phase);
-}
-
 // src/uploads.ts
 import { readFile as readFile3 } from "fs/promises";
 import { basename, extname } from "path";
@@ -2382,11 +2407,6 @@ function normalizeUploadedFileOptions(files) {
   return Array.isArray(files) ? files : [files];
 }
 
-// src/utils/poll-interval.ts
-function clampPollInterval(ms, minimum) {
-  return Math.max(ms, minimum);
-}
-
 // ../../shared/session/no-change-outcome.ts
 function isNoChangesPromptResult(result) {
   return typeof result === "object" && result !== null && "noChanges" in result && result.noChanges === true;
@@ -3889,20 +3909,6 @@ function applyDisconnectMask(view, mask, now) {
 }
 
 // src/commands/watch.ts
-function parsePollInterval(raw) {
-  if (!raw) return DEFAULT_WATCH_POLL_INTERVAL_MS;
-  if (!/^\d+$/.test(raw)) {
-    throw new CliError("user", "Polling interval must be a non-negative integer.");
-  }
-  const value = Number(raw);
-  if (value < MIN_WATCH_POLL_INTERVAL_MS || value > MAX_WATCH_POLL_INTERVAL_MS) {
-    throw new CliError(
-      "user",
-      `Polling interval must be between ${MIN_WATCH_POLL_INTERVAL_MS} and ${MAX_WATCH_POLL_INTERVAL_MS} milliseconds.`
-    );
-  }
-  return value;
-}
 function parseNonNegativeInteger(raw, name, defaultValue) {
   if (!raw) return defaultValue;
   if (!/^\d+$/.test(raw)) {
@@ -4327,16 +4333,6 @@ function parseEgressAllowlistSourceFile(content, key = EGRESS_ALLOWLIST_SOURCE_P
 }
 
 // src/commands/egress.ts
-function parseRepoArg(value) {
-  const parsed = parseGithubRepoFullName(value);
-  if (!parsed) throw new CliError("user", "repo must be owner/name or a GitHub URL");
-  return {
-    ...parsed,
-    // Preserve the legacy shorthand contract while URLs and SSH remotes are
-    // normalized by the shared parser itself.
-    repo: /^[^/:]+\/[^/]+\.git$/.test(value.trim()) ? parsed.repo.slice(0, -4) : parsed.repo
-  };
-}
 async function egressSourceSetCommand(sourceRepoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
@@ -4569,13 +4565,13 @@ function parseCreateConflict(error) {
   let message = error.message;
   let sessionId;
   let sessionUrl;
-  try {
-    const parsed = JSON.parse(error.body);
-    const parsedMessage = typeof parsed.error === "string" ? parsed.error : typeof parsed.error?.message === "string" ? parsed.error.message : void 0;
+  const parsed = parseConflictBody(error);
+  if (parsed) {
+    const rawError = parsed.error;
+    const parsedMessage = typeof rawError === "string" ? rawError : rawError && typeof rawError === "object" && typeof rawError.message === "string" ? rawError.message : void 0;
     if (parsedMessage) message = parsedMessage;
     if (typeof parsed.sessionId === "string") sessionId = parsed.sessionId;
     if (typeof parsed.sessionUrl === "string") sessionUrl = parsed.sessionUrl;
-  } catch {
   }
   const location = sessionUrl ? ` (${sessionUrl})` : "";
   const existing = sessionId ? ` Existing verifier session: ${sessionId}${location}.` : "";
@@ -4667,28 +4663,7 @@ async function qaCommand(prUrl, options, command) {
   console.log(`Follow with: arcanist sessions events ${sessionId} --follow --json`);
 }
 
-// src/git.ts
-import { execFileSync as execFileSync2 } from "child_process";
-function git(args) {
-  try {
-    return execFileSync2("git", args, { encoding: "utf8", stdio: ["ignore", "pipe", "pipe"] }).trim();
-  } catch (err) {
-    throw new CliError("user", `git ${args.join(" ")} failed: ${stringifyError(err)}`);
-  }
-}
-function currentRepo() {
-  const parsed = parseGithubRepoFullName(git(["remote", "get-url", "origin"]));
-  if (!parsed) throw new CliError("user", "origin remote must point at a GitHub repository");
-  return { owner: parsed.owner, repo: parsed.repo };
-}
-
 // src/commands/repos.ts
-function parseRepoArg2(value) {
-  if (value === void 0) return currentRepo();
-  const parsed = parseGithubRepoFullName(value);
-  if (!parsed) throw new CliError("user", "repo must be owner/name or a GitHub URL");
-  return { owner: parsed.owner, repo: parsed.repo };
-}
 async function reposListCommand(options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const payload = await apiFetch(
@@ -4704,8 +4679,7 @@ async function reposListCommand(options = {}, command) {
 }
 async function repoBranchesCommand(repoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
-  const repo = parseRepoArg2(repoArg);
-  await apiFetch(config, "/api/repos");
+  const repo = parseRepoArgOrCurrent(repoArg);
   const payload = await apiFetch(
     config,
     `/api/repos/${encodeURIComponent(repo.owner)}/${encodeURIComponent(repo.repo)}/branches`
@@ -4716,7 +4690,7 @@ async function repoBranchesCommand(repoArg, options = {}, command) {
 }
 async function repoSkillsCommand(repoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
-  const repo = parseRepoArg2(repoArg);
+  const repo = parseRepoArgOrCurrent(repoArg);
   const payload = await apiFetch(
     config,
     `/api/repos/${encodeURIComponent(repo.owner)}/${encodeURIComponent(repo.repo)}/skills`
@@ -4764,17 +4738,62 @@ async function respondCommand(sessionId, answerArg, options = {}, command) {
   }
 }
 function parseRespondConflict(err) {
-  if (!(err instanceof ApiError) || err.status !== 409) return null;
-  try {
-    const body = JSON.parse(err.body);
-    if (body.error !== "session_not_respondable") return null;
-    return {
-      message: body.reason ? `Session cannot be answered from phase=${body.reason}.` : "Session does not have a pending question.",
-      reason: body.reason ?? "session_not_respondable"
-    };
-  } catch {
-    return null;
+  const body = parseConflictBody(err);
+  if (!body || body.error !== "session_not_respondable") return null;
+  const reason = typeof body.reason === "string" ? body.reason : void 0;
+  return {
+    message: reason ? `Session cannot be answered from phase=${reason}.` : "Session does not have a pending question.",
+    reason: reason ?? "session_not_respondable"
+  };
+}
+
+// src/commands/review.ts
+var PR_REVIEW_POLL_INTERVAL_MS = 1e4;
+var PR_REVIEW_WAIT_TIMEOUT_MS = 30 * 60 * 1e3;
+async function reviewCommand(prUrl, options = {}, command) {
+  assertArcanistSessionMutationAllowed("review");
+  const pollIntervalMs = options.wait ? parsePollInterval(options.pollInterval, { defaultMs: PR_REVIEW_POLL_INTERVAL_MS, maxMs: null }) : null;
+  const { config } = resolveBusinessContext(command, options);
+  const trigger = await apiFetch(config, "/api/pr-reviews", {
+    method: "POST",
+    body: JSON.stringify({
+      prUrl,
+      ...options.focus ? { focus: options.focus } : {},
+      // Send an explicitly passed model even when empty (e.g. --model "$VAR" with an
+      // unset VAR) so the server rejects it as invalid instead of silently routing.
+      ...options.model !== void 0 ? { model: options.model } : {}
+    })
+  });
+  if (!options.wait || trigger.cached) {
+    emit(command, options, trigger, (payload) => {
+      if (payload.cached) console.log(`Zeus review already completed: ${payload.verdict ?? "unknown"}.`);
+      else console.log(`Started Zeus review${payload.sessionId ? ` (${payload.sessionId})` : ""}.`);
+    });
+    return;
   }
+  const triggeredSessionId = trigger.sessionId ?? null;
+  const result = await pollUntilSettled({
+    pollIntervalMs: pollIntervalMs ?? PR_REVIEW_POLL_INTERVAL_MS,
+    timeoutMs: PR_REVIEW_WAIT_TIMEOUT_MS,
+    timeoutMessage: "Timed out waiting for Zeus review results.",
+    fetchStatus: async () => {
+      const status = await apiFetch(
+        config,
+        `/api/pr-reviews/status?prUrl=${encodeURIComponent(prUrl)}`
+      );
+      if (status.status !== "completed") return null;
+      assertSameAttempt(triggeredSessionId, status.sessionId, "PR review");
+      return { ...status, sessionId: status.sessionId ?? triggeredSessionId };
+    }
+  });
+  emit(command, options, result, (payload) => {
+    console.log(`Zeus review: ${sanitizeTerminalText(payload.verdict)}. ${payload.findings.length} finding(s).`);
+    for (const finding of payload.findings) {
+      console.log(
+        `${sanitizeTerminalText(finding.severity)} ${sanitizeTerminalText(finding.file)}:${finding.line} - ${sanitizeTerminalText(finding.title)}`
+      );
+    }
+  });
 }
 
 // src/commands/sandbox.ts
@@ -5327,23 +5346,9 @@ function assertCleanAndPushed() {
 function assertManifestExists(path) {
   if (!existsSync3(path)) throw new CliError("user", `Missing sandbox manifest: ${path}`);
 }
-function parseRepoArg3(value, fallback) {
-  if (!value) return fallback();
-  const parsed = parseGithubRepoFullName(value);
-  if (!parsed) throw new CliError("user", "repo must be owner/name or a GitHub URL");
-  return { owner: parsed.owner, repo: parsed.repo };
-}
 function repoPath(repo) {
   return `${repo.owner}/${repo.repo}`;
 }
-function parsePollInterval2(value) {
-  if (!value) return DEFAULT_POLL_INTERVAL_MS;
-  const parsed = Number(value);
-  if (!Number.isInteger(parsed) || parsed < MIN_POLL_INTERVAL_MS) {
-    throw new CliError("user", `--poll-interval must be an integer >= ${MIN_POLL_INTERVAL_MS}.`);
-  }
-  return parsed;
-}
 function printBuildRequest(buildRequest) {
   console.log(
     [
@@ -5615,8 +5620,8 @@ async function sandboxBuildCommand(sourceRepoArg, options = {}, command) {
   const manifestPath = options.manifest ?? DEFAULT_MANIFEST_PATH;
   assertManifestExists(manifestPath);
   if (!options.ref) assertCleanAndPushed();
-  const sourceRepo = parseRepoArg3(sourceRepoArg, currentRepo);
-  const targetRepo = options.targetRepo ? parseRepoArg3(options.targetRepo, currentRepo) : null;
+  const sourceRepo = parseRepoArgOrCurrent(sourceRepoArg);
+  const targetRepo = options.targetRepo ? parseRepoArgOrCurrent(options.targetRepo) : null;
   const businessId = await resolveBusinessId(config, options);
   const ref = options.ref ?? currentRef();
   let payload;
@@ -5654,7 +5659,11 @@ async function sandboxBuildCommand(sourceRepoArg, options = {}, command) {
   if (!options.wait && !options.follow) return;
   const finalBuild = await waitForBuild(config, businessId, payload.buildRequest.id, {
     follow: options.follow,
-    pollIntervalMs: parsePollInterval2(options.pollInterval),
+    pollIntervalMs: parsePollInterval(options.pollInterval, {
+      defaultMs: DEFAULT_POLL_INTERVAL_MS,
+      minMs: MIN_POLL_INTERVAL_MS,
+      maxMs: null
+    }),
     json
   });
   if (json)
@@ -5667,7 +5676,7 @@ async function sandboxBuildCommand(sourceRepoArg, options = {}, command) {
 async function sandboxStatusCommand(repoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
-  const repo = parseRepoArg3(repoArg, currentRepo);
+  const repo = parseRepoArgOrCurrent(repoArg);
   const payload = await apiFetch(
     config,
     `/api/businesses/${encodeURIComponent(businessId)}/repos/${encodeURIComponent(repo.owner)}/${encodeURIComponent(
@@ -5679,9 +5688,9 @@ async function sandboxStatusCommand(repoArg, options = {}, command) {
 async function sandboxHistoryCommand(sourceRepoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
-  const sourceRepo = parseRepoArg3(sourceRepoArg, currentRepo);
+  const sourceRepo = parseRepoArgOrCurrent(sourceRepoArg);
   const params = new URLSearchParams({ sourceRepo: repoPath(sourceRepo) });
-  if (options.targetRepo) params.set("targetRepo", repoPath(parseRepoArg3(options.targetRepo, currentRepo)));
+  if (options.targetRepo) params.set("targetRepo", repoPath(parseRepoArgOrCurrent(options.targetRepo)));
   if (options.status) params.set("status", options.status);
   if (options.limit) {
     const limit = Number(options.limit);
@@ -5776,13 +5785,19 @@ async function sandboxLogsCommand(buildId, options = {}, command) {
     }
     if (!options.follow) return;
     if (payload.logs.length > 0) afterSequence = payload.logs[payload.logs.length - 1].sequence;
-    await sleep(parsePollInterval2(options.pollInterval));
+    await sleep(
+      parsePollInterval(options.pollInterval, {
+        defaultMs: DEFAULT_POLL_INTERVAL_MS,
+        minMs: MIN_POLL_INTERVAL_MS,
+        maxMs: null
+      })
+    );
   }
 }
 async function sandboxAssignDefaultCommand(sourceRepoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
-  const sourceRepo = parseRepoArg3(sourceRepoArg, currentRepo);
+  const sourceRepo = parseRepoArgOrCurrent(sourceRepoArg);
   const manifestPath = options.manifest ?? DEFAULT_MANIFEST_PATH;
   const payload = await apiFetch(
     config,
@@ -5799,8 +5814,8 @@ async function sandboxAssignDefaultCommand(sourceRepoArg, options = {}, command)
 async function sandboxAssignRepoCommand(targetRepoArg, sourceRepoArg, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
-  const targetRepo = parseRepoArg3(targetRepoArg, currentRepo);
-  const sourceRepo = parseRepoArg3(sourceRepoArg, currentRepo);
+  const targetRepo = parseRepoArgOrCurrent(targetRepoArg);
+  const sourceRepo = parseRepoArgOrCurrent(sourceRepoArg);
   const manifestPath = options.manifest ?? DEFAULT_MANIFEST_PATH;
   const payload = await apiFetch(
     config,
@@ -5838,7 +5853,7 @@ async function sandboxUnassignRepoCommand(targetRepoArg, options = {}, command)
   }
   const { config } = resolveBusinessContext(command, options);
   const businessId = await resolveBusinessId(config, options);
-  const targetRepo = parseRepoArg3(targetRepoArg, currentRepo);
+  const targetRepo = parseRepoArgOrCurrent(targetRepoArg);
   if (!options.yes) await confirmOrThrow(`Clear sandbox layer assignment for ${repoPath(targetRepo)}?`);
   const payload = await apiFetch(
     config,
@@ -5851,35 +5866,29 @@ async function sandboxUnassignRepoCommand(targetRepoArg, options = {}, command)
 }
 
 // src/commands/sessions.ts
-var MAX_ALL_PAGES2 = 1e3;
 async function listSessionsCommand(options, command) {
   const runtime = getRuntimeOptions(command, options);
   const config = requireConfig(runtime);
-  const sessions2 = [];
-  let cursor2 = options.cursor;
-  let nextCursor = null;
-  let pageCount = 0;
-  do {
-    pageCount += 1;
-    if (options.all === true && pageCount > MAX_ALL_PAGES2) {
-      throw new CliError("user", `sessions list --all exceeded ${MAX_ALL_PAGES2} pages without reaching the end.`);
-    }
-    const query = new URLSearchParams();
-    if (options.status) query.set("status", options.status);
-    if (options.scope) query.set("scope", options.scope);
-    if (options.search) query.set("q", options.search);
-    if (options.repo) query.set("repo", options.repo);
-    if (options.limit) query.set("limit", options.limit);
-    if (cursor2) query.set("cursor", cursor2);
-    const payload2 = await apiFetch(
-      config,
-      `/api/sessions${query.size ? `?${query.toString()}` : ""}`
-    );
-    sessions2.push(...payload2.sessions);
-    nextCursor = payload2.nextCursor;
-    cursor2 = nextCursor ?? void 0;
-  } while (options.all === true && nextCursor);
-  const payload = { sessions: sessions2, nextCursor: options.all === true ? null : nextCursor };
+  const { items: sessions2, nextCursor } = await fetchPages(
+    "sessions list --all",
+    options.all === true,
+    options.cursor,
+    async (cursor2) => {
+      const query = new URLSearchParams();
+      if (options.status) query.set("status", options.status);
+      if (options.scope) query.set("scope", options.scope);
+      if (options.search) query.set("q", options.search);
+      if (options.repo) query.set("repo", options.repo);
+      if (options.limit) query.set("limit", options.limit);
+      if (cursor2) query.set("cursor", cursor2);
+      const payload2 = await apiFetch(
+        config,
+        `/api/sessions${query.size ? `?${query.toString()}` : ""}`
+      );
+      return { items: payload2.sessions, nextCursor: payload2.nextCursor };
+    }
+  );
+  const payload = { sessions: sessions2, nextCursor };
   if (isJson(command, options)) {
     writeJson(payload);
     return;
@@ -6021,25 +6030,15 @@ async function stopCommand(sessionId, options = {}, command) {
   });
 }
 function parseStopBlocked(err) {
-  if (!(err instanceof ApiError) || err.status !== 409) return null;
-  try {
-    const body = JSON.parse(err.body);
-    if (body.error !== "session_not_stoppable") return null;
-    return { reason: body.reason ?? "not_stoppable" };
-  } catch {
-    return null;
-  }
+  const body = parseConflictBody(err);
+  if (!body || body.error !== "session_not_stoppable") return null;
+  return { reason: typeof body.reason === "string" ? body.reason : "not_stoppable" };
 }
 
 // src/commands/test-creds.ts
 import { readFileSync as readFileSync4 } from "fs";
-function parseRepoArg4(value) {
-  const parsed = parseGithubRepoFullName(value);
-  if (!parsed) throw new CliError("user", "repo must be owner/name or a GitHub URL");
-  return { owner: parsed.owner, name: parsed.repo };
-}
 function basePath(businessId, repo) {
-  return `/api/businesses/${encodeURIComponent(businessId)}/repos/${encodeURIComponent(repo.owner)}/${encodeURIComponent(repo.name)}/test-credentials`;
+  return `/api/businesses/${encodeURIComponent(businessId)}/repos/${encodeURIComponent(repo.owner)}/${encodeURIComponent(repo.repo)}/test-credentials`;
 }
 function readValue(options) {
   const sources = [options.value, options.valueFile, options.valueStdin].filter((v) => v !== void 0 && v !== false);
@@ -6052,11 +6051,11 @@ function readValue(options) {
 }
 async function listTestCredentialsCommand(repo, options, command) {
   const { config } = resolveBusinessContext(command, options);
-  const repoArg = parseRepoArg4(repo);
+  const repoArg = parseRepoArg(repo);
   const payload = await apiFetch(config, basePath(options.business, repoArg));
   emit(command, options, payload, (listPayload) => {
     if (listPayload.credentials.length === 0) {
-      console.log(`No test credentials configured for ${repoArg.owner}/${repoArg.name}.`);
+      console.log(`No test credentials configured for ${repoArg.owner}/${repoArg.repo}.`);
       return;
     }
     for (const cred of listPayload.credentials) {
@@ -6067,7 +6066,7 @@ async function listTestCredentialsCommand(repo, options, command) {
 }
 async function setTestCredentialCommand(repo, name, options, command) {
   const { config } = resolveBusinessContext(command, options);
-  const repoArg = parseRepoArg4(repo);
+  const repoArg = parseRepoArg(repo);
   const value = readValue(options);
   const payload = await apiFetch(
     config,
@@ -6078,7 +6077,7 @@ async function setTestCredentialCommand(repo, name, options, command) {
     command,
     options,
     payload,
-    () => console.log(`Set test credential '${name}' for ${repoArg.owner}/${repoArg.name}.`)
+    () => console.log(`Set test credential '${name}' for ${repoArg.owner}/${repoArg.repo}.`)
   );
 }
 async function deleteTestCredentialCommand(repo, name, options, command) {
@@ -6086,11 +6085,11 @@ async function deleteTestCredentialCommand(repo, name, options, command) {
     throw new CliError("user", "`test-creds delete --json` requires --yes.");
   }
   if (options.yes !== true) {
-    const repoArg2 = parseRepoArg4(repo);
-    await confirmOrThrow(`Delete test credential '${name}' for ${repoArg2.owner}/${repoArg2.name}?`);
+    const repoArg2 = parseRepoArg(repo);
+    await confirmOrThrow(`Delete test credential '${name}' for ${repoArg2.owner}/${repoArg2.repo}?`);
   }
   const { config } = resolveBusinessContext(command, options);
-  const repoArg = parseRepoArg4(repo);
+  const repoArg = parseRepoArg(repo);
   const payload = await apiFetch(
     config,
     `${basePath(options.business, repoArg)}/${encodeURIComponent(name)}`,
@@ -6100,35 +6099,29 @@ async function deleteTestCredentialCommand(repo, name, options, command) {
     command,
     options,
     payload,
-    () => console.log(`Deleted test credential '${name}' for ${repoArg.owner}/${repoArg.name}.`)
+    () => console.log(`Deleted test credential '${name}' for ${repoArg.owner}/${repoArg.repo}.`)
   );
 }
 
 // src/commands/tokens.ts
-var MAX_ALL_PAGES3 = 1e3;
 async function listTokensCommand(options, command) {
   const { config } = resolveBusinessContext(command, options);
-  const data = [];
-  let cursor2 = options.cursor;
-  let nextCursor = null;
-  let pageCount = 0;
-  do {
-    pageCount += 1;
-    if (options.all === true && pageCount > MAX_ALL_PAGES3) {
-      throw new CliError("user", `tokens list --all exceeded ${MAX_ALL_PAGES3} pages without reaching the end.`);
+  const { items: data, nextCursor } = await fetchPages(
+    "tokens list --all",
+    options.all === true,
+    options.cursor,
+    async (cursor2) => {
+      const query = new URLSearchParams();
+      if (options.limit) query.set("limit", options.limit);
+      if (cursor2) query.set("cursor", cursor2);
+      const payload2 = await apiFetch(
+        config,
+        `/api/cli-tokens${query.size ? `?${query.toString()}` : ""}`
+      );
+      return { items: payload2.data, nextCursor: payload2.nextCursor };
     }
-    const query = new URLSearchParams();
-    if (options.limit) query.set("limit", options.limit);
-    if (cursor2) query.set("cursor", cursor2);
-    const payload2 = await apiFetch(
-      config,
-      `/api/cli-tokens${query.size ? `?${query.toString()}` : ""}`
-    );
-    data.push(...payload2.data);
-    nextCursor = payload2.nextCursor;
-    cursor2 = nextCursor ?? void 0;
-  } while (options.all === true && nextCursor);
-  const payload = { data, nextCursor: options.all === true ? null : nextCursor };
+  );
+  const payload = { data, nextCursor };
   emit(command, options, payload, (listPayload) => {
     if (listPayload.data.length === 0) {
       console.log("No CLI tokens found.");
@@ -6411,7 +6404,7 @@ cursor.command("use <state>").description("Turn using your saved Cursor subscrip
 cursor.command("status").description("Show whether your workspace is eligible and whether a Cursor subscription auth is saved").action((options, command) => agentSubscriptionStatusCommand(CURSOR_SUBSCRIPTION_CLI_CONFIG, options, command));
 cursor.command("logout").description("Deactivate the selector and remove the stored Cursor subscription auth for your user").action((options, command) => agentSubscriptionLogoutCommand(CURSOR_SUBSCRIPTION_CLI_CONFIG, options, command));
 var sessions = program.command("sessions").description("Session commands");
-program.command("review <pr-url>").description("Run a Zeus review for a GitHub pull request").option("--focus <text>", "Focus the review on a specific concern").option("--model <model>", "Pin the reviewer model (Arcanist members only); defaults to automatic routing").option("--wait", "Wait for the review and print its verdict and findings").addHelpText(
+program.command("review <pr-url>").description("Run a Zeus review for a GitHub pull request").option("--focus <text>", "Focus the review on a specific concern").option("--model <model>", "Pin the reviewer model (Arcanist members only); defaults to automatic routing").option("--wait", "Wait for the review and print its verdict and findings").option("--poll-interval <ms>", "Polling interval in milliseconds while waiting (default 10000)").addHelpText(
   "after",
   `
 Examples:
@@ -6425,7 +6418,7 @@ a specific reviewer model instead (members only).
 Zeus review is currently available to Arcanist members only.
 `
 ).action((prUrl, options, command) => reviewCommand(prUrl, options, command));
-program.command("anubis <pr-url>").description("Run an Anubis QA verification for a GitHub pull request").option("--wait", "Wait for the verdict").addHelpText(
+program.command("anubis <pr-url>").description("Run an Anubis QA verification for a GitHub pull request").option("--wait", "Wait for the verdict").option("--poll-interval <ms>", "Polling interval in milliseconds while waiting (default 15000)").addHelpText(
   "after",
   `
 Examples:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tryarcanist/cli",
-  "version": "0.1.221",
+  "version": "0.1.223",
   "description": "CLI for Arcanist — create and manage coding agent sessions",
   "type": "module",
   "bin": {