@tryarcanist/cli 0.1.222 → 0.1.223

package/dist/index.js

@@ -976,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.");
   }
-  throw new CliError("server", "Timed out waiting for Zeus review results.");
+  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;
+}
+
+// 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
@@ -1035,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",
@@ -1046,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) {
@@ -2226,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";
@@ -2423,26 +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);
-}
-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) {
   return typeof result === "object" && result !== null && "noChanges" in result && result.noChanges === true;
@@ -4783,6 +4747,55 @@ function parseRespondConflict(err) {
   };
 }
 
+// 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
 import { existsSync as existsSync3, mkdirSync as mkdirSync2, readFileSync as readFileSync3, writeFileSync as writeFileSync2 } from "fs";
 import { dirname as dirname2 } from "path";
@@ -6391,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:
@@ -6405,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.222",
+  "version": "0.1.223",
   "description": "CLI for Arcanist — create and manage coding agent sessions",
   "type": "module",
   "bin": {