@tryarcanist/cli 0.1.220 → 0.1.222

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;
@@ -975,7 +984,13 @@ async function reviewCommand(prUrl, options = {}, command) {
   const { config } = resolveBusinessContext(command, options);
   const trigger = await apiFetch(config, "/api/pr-reviews", {
     method: "POST",
-    body: JSON.stringify({ prUrl, ...options.focus ? { focus: options.focus } : {} })
+    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) => {
@@ -1836,6 +1851,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_.-]+)$/);
@@ -1847,6 +1882,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 * * * *",
@@ -1863,9 +1924,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();
@@ -1912,27 +1972,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;
@@ -1964,14 +2019,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);
@@ -2380,6 +2427,21 @@ function normalizeUploadedFileOptions(files) {
 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.`);
+  }
+  return value;
+}
 
 // ../../shared/session/no-change-outcome.ts
 function isNoChangesPromptResult(result) {
@@ -3883,20 +3945,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)) {
@@ -4321,16 +4369,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);
@@ -4563,13 +4601,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}.` : "";
@@ -4661,28 +4699,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(
@@ -4698,8 +4715,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`
@@ -4710,7 +4726,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`
@@ -4758,17 +4774,13 @@ 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/sandbox.ts
@@ -5321,23 +5333,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(
     [
@@ -5609,8 +5607,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;
@@ -5648,7 +5646,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)
@@ -5661,7 +5663,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(
@@ -5673,9 +5675,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);
@@ -5770,13 +5772,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,
@@ -5793,8 +5801,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,
@@ -5832,7 +5840,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,
@@ -5845,35 +5853,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;
@@ -6015,25 +6017,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);
@@ -6046,11 +6038,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) {
@@ -6061,7 +6053,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,
@@ -6072,7 +6064,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) {
@@ -6080,11 +6072,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)}`,
@@ -6094,35 +6086,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.");
@@ -6405,14 +6391,17 @@ 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("--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").addHelpText(
   "after",
   `
 Examples:
   arcanist review https://github.com/org/repo/pull/123 --wait --json
   arcanist review https://github.com/org/repo/pull/123 --focus "Check auth" --wait
+  arcanist review https://github.com/org/repo/pull/123 --model gpt-5.5 --wait
 
-Zeus always runs the platform-managed Codex reviewer; --model and --backend do not apply to reviews.
+When the PR came from an Arcanist session, Zeus defaults to the runtime backend opposite
+the author's as a cross-check; otherwise it uses the default Codex reviewer. --model pins
+a specific reviewer model instead (members only).
 Zeus review is currently available to Arcanist members only.
 `
 ).action((prUrl, options, command) => reviewCommand(prUrl, options, command));

package/package.json

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