@tryarcanist/cli 0.1.194 → 0.1.196

package/dist/index.js

@@ -17,6 +17,36 @@ function normalizeBaseUrl(url) {
   return url.replace(/\/+$/, "");
 }
 
+// src/utils/api-error.ts
+function parseApiErrorBody(body) {
+  try {
+    const parsed = JSON.parse(body);
+    if (!parsed || typeof parsed !== "object") return null;
+    const record = parsed;
+    const topLevelMessage = typeof record.message === "string" && record.message ? record.message : void 0;
+    if (typeof record.error === "string" && record.error) {
+      return {
+        serverCode: typeof record.code === "string" && record.code ? record.code : record.error,
+        rawError: record.error,
+        message: topLevelMessage ?? record.error
+      };
+    }
+    if (record.error && typeof record.error === "object") {
+      const error = record.error;
+      const message = typeof error.message === "string" && error.message ? error.message : topLevelMessage;
+      const serverCode = typeof error.code === "string" && error.code ? error.code : void 0;
+      return message || serverCode ? {
+        ...message ? { message } : {},
+        ...serverCode ? { serverCode } : {},
+        ...serverCode ? { rawError: serverCode } : {}
+      } : null;
+    }
+    return topLevelMessage ? { message: topLevelMessage } : null;
+  } catch {
+    return null;
+  }
+}
+
 // src/errors.ts
 var EXIT_CODE_INTERRUPTED = 130;
 var CliError = class extends Error {
@@ -82,33 +112,6 @@ function messageForApiError(status, body, parsed = parseApiErrorBody(body)) {
   if (parsed?.serverCode) return parsed.serverCode;
   return body ? `API error ${status}: ${body}` : `API error ${status}`;
 }
-function parseApiErrorBody(body) {
-  try {
-    const parsed = JSON.parse(body);
-    if (!parsed || typeof parsed !== "object") return null;
-    const error = parsed.error;
-    if (typeof error === "string" && error.length > 0) {
-      const message2 = parsed.message;
-      const code = parsed.code;
-      return {
-        serverCode: typeof code === "string" && code ? code : error,
-        ...typeof message2 === "string" && message2 ? { message: message2 } : { message: error }
-      };
-    }
-    if (error && typeof error === "object") {
-      const code = error.code;
-      const message2 = error.message;
-      return {
-        ...typeof message2 === "string" && message2 ? { message: message2 } : {},
-        ...typeof code === "string" && code ? { serverCode: code } : {}
-      };
-    }
-    const message = parsed.message;
-    return typeof message === "string" && message.length > 0 ? { message } : null;
-  } catch {
-    return null;
-  }
-}
 function hintForHttpStatus(status, resourceNoun) {
   if (status === 401 || status === 403) return "Run `arcanist auth login` or set `ARCANIST_TOKEN`.";
   if (status === 404) {
@@ -1062,7 +1065,7 @@ var CODEX_SUBSCRIPTION_SESSION_START_MODEL_PROVIDER_GROUPS = buildModelProviderG
 function parseGithubRepoFullName(value) {
   const shorthand = value.match(/^([a-zA-Z0-9_.-]+)\/([a-zA-Z0-9_.-]+)$/);
   if (shorthand) return { owner: shorthand[1], repo: shorthand[2] };
-  const ssh = value.match(/git@[^:]+:([^/]+)\/([^/]+?)(?:\.git)?$/);
+  const ssh = value.match(/^git@github\.com:([^/]+)\/([^/]+?)(?:\.git)?$/);
   if (ssh) return { owner: ssh[1], repo: ssh[2] };
   const https = value.match(/^https?:\/\/github\.com\/([^/]+)\/([^/]+?)(?:\.git)?\/?$/);
   if (https) return { owner: https[1], repo: https[2] };
@@ -1206,27 +1209,14 @@ function mapAutomationApiError(err) {
   if (!(err instanceof ApiError)) {
     return err instanceof CliError ? err : new CliError("server", stringifyError(err));
   }
-  const parsed = parseApiErrorBody2(err.body);
-  const serverCode = parsed?.error;
+  const parsed = parseApiErrorBody(err.body);
+  const serverCode = parsed?.rawError ?? parsed?.serverCode;
   return new CliError(codeForHttpStatus(err.status), parsed?.message || serverCode || err.message, {
     exitCode: err.exitCode,
     hint: serverCode ? AUTOMATION_ERROR_HINTS[serverCode] : void 0,
     requestId: err.requestId
   });
 }
-function parseApiErrorBody2(body) {
-  try {
-    const parsed = JSON.parse(body);
-    if (!parsed || typeof parsed !== "object") return null;
-    const record = parsed;
-    return {
-      error: typeof record.error === "string" ? record.error : void 0,
-      message: typeof record.message === "string" ? record.message : void 0
-    };
-  } catch {
-    return null;
-  }
-}
 function printAutomationRule(rule) {
   console.log(`ID: ${rule.id}`);
   console.log(`Repo: ${rule.repoOwner}/${rule.repoName}`);
@@ -1240,6 +1230,7 @@ function formatTime(value) {
 
 // src/commands/codex.ts
 import { spawn } from "child_process";
+import { rmSync } from "fs";
 import { mkdtemp, readFile, rm } from "fs/promises";
 import { tmpdir } from "os";
 import { join as join2 } from "path";
@@ -1288,8 +1279,19 @@ async function setCodexSubscriptionEnabled(config, enabled) {
 async function codexLoginCommand(options, command) {
   const { config } = resolveBusinessContext(command, options);
   const codexPath = resolveCodexPath(options.codexPath);
-  const codexHome = await mkdtemp(join2(tmpdir(), "arcanist-codex-"));
+  let codexHome;
+  const handleSigint = () => {
+    if (codexHome) {
+      try {
+        rmSync(codexHome, { recursive: true, force: true });
+      } catch {
+      }
+    }
+    process.exit(EXIT_CODE_INTERRUPTED);
+  };
+  process.on("SIGINT", handleSigint);
   try {
+    codexHome = await mkdtemp(join2(tmpdir(), "arcanist-codex-"));
     await runCodexDeviceLogin(codexPath, codexHome);
     let authJson;
     try {
@@ -1325,8 +1327,14 @@ async function codexLoginCommand(options, command) {
       }
     });
   } finally {
-    await rm(codexHome, { recursive: true, force: true }).catch(() => {
-    });
+    try {
+      if (codexHome) {
+        await rm(codexHome, { recursive: true, force: true }).catch(() => {
+        });
+      }
+    } finally {
+      process.off("SIGINT", handleSigint);
+    }
   }
 }
 async function codexUseCommand(state, options, command) {
@@ -1560,6 +1568,11 @@ 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;
@@ -2985,6 +2998,11 @@ function renderWatchEvent(event, state) {
   }
 }
 
+// src/utils/session-payload.ts
+function unwrapSessionState(payload) {
+  return payload.session && typeof payload.session === "object" ? payload.session : payload;
+}
+
 // src/commands/model-options.ts
 function resolveModelAndBackend(options) {
   let agentRuntimeBackend = CODEX_AGENT_RUNTIME_BACKEND;
@@ -3228,7 +3246,7 @@ async function fetchCreatedPromptStatus(config, sessionId, promptId) {
   return selectCreatedPrompt(promptList.prompts, promptId);
 }
 async function waitForCreatedPromptToSettle(config, sessionId, promptId, pollIntervalMs) {
-  const effectivePollIntervalMs = Math.max(pollIntervalMs, MIN_WATCH_POLL_INTERVAL_MS);
+  const effectivePollIntervalMs = clampPollInterval(pollIntervalMs, MIN_WATCH_POLL_INTERVAL_MS);
   while (true) {
     const createdPrompt = await fetchCreatedPromptStatus(config, sessionId, promptId);
     if (createdPrompt && PROMPT_TERMINAL_STATUSES.has(createdPrompt.status)) {
@@ -3270,15 +3288,12 @@ async function waitForCreatedPrompt(sessionId, promptId, sessionUrl, pollInterva
     );
   }
 }
-function unwrapSessionState(payload) {
-  return payload.session && typeof payload.session === "object" ? payload.session : payload;
-}
 async function fetchSessionResultFields(config, sessionId) {
   const sessionPayload = await apiFetch(config, `/api/sessions/${sessionId}`);
   return extractSessionResultFields(unwrapSessionState(sessionPayload));
 }
 async function fetchSettledSessionResultFields(config, sessionId, pollIntervalMs) {
-  const effectivePollIntervalMs = Math.max(pollIntervalMs, MIN_WATCH_POLL_INTERVAL_MS);
+  const effectivePollIntervalMs = clampPollInterval(pollIntervalMs, MIN_WATCH_POLL_INTERVAL_MS);
   let latest = {};
   for (let attempt = 0; attempt < RESULT_FETCH_ATTEMPTS; attempt++) {
     latest = await fetchSessionResultFields(config, sessionId);
@@ -3457,10 +3472,14 @@ function parseEgressAllowlistSourceFile(content, key = EGRESS_ALLOWLIST_SOURCE_P
 
 // src/commands/egress.ts
 function parseRepoArg(value) {
-  const trimmed = value.trim().replace(/^https:\/\/github\.com\//, "").replace(/\.git$/, "");
-  const [owner, repo, extra] = trimmed.split("/");
-  if (!owner || !repo || extra) throw new CliError("user", "repo must be owner/name or a GitHub URL");
-  return { owner, repo };
+  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);
@@ -3585,9 +3604,6 @@ async function loginCommand(options, command) {
 }
 
 // src/commands/message.ts
-function unwrapSessionState2(payload) {
-  return payload.session && typeof payload.session === "object" ? payload.session : payload;
-}
 async function messageCommand(sessionId, promptArg, options = {}, command) {
   const runtime = getRuntimeOptions(command, options);
   assertArcanistSessionMutationAllowed("send");
@@ -3619,7 +3635,7 @@ async function messageCommand(sessionId, promptArg, options = {}, command) {
     }
     try {
       const sessionPayload = await apiFetch(config, `/api/sessions/${sessionId}`);
-      const resultLine = formatSessionResult(unwrapSessionState2(sessionPayload));
+      const resultLine = formatSessionResult(unwrapSessionState(sessionPayload));
       if (resultLine) console.log(resultLine);
     } catch {
     }
@@ -3812,7 +3828,7 @@ function currentRepo() {
 
 // src/commands/repos.ts
 function parseRepoArg2(value) {
-  if (!value) return currentRepo();
+  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 };
@@ -5161,13 +5177,10 @@ function parseStopBlocked(err) {
 
 // src/commands/test-creds.ts
 import { readFileSync as readFileSync4 } from "fs";
-function parseRepoArg4(repo) {
-  const trimmed = repo.trim();
-  const slashIndex = trimmed.indexOf("/");
-  if (slashIndex <= 0 || slashIndex === trimmed.length - 1) {
-    throw new CliError("user", `repo must be in the form 'owner/name' (got: '${repo}')`);
-  }
-  return { owner: trimmed.slice(0, slashIndex), name: trimmed.slice(slashIndex + 1) };
+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`;

package/package.json

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