@tryarcanist/cli 0.1.81 → 0.1.83

package/dist/index.js

@@ -388,19 +388,31 @@ function sleep(ms) {
   return new Promise((resolve) => setTimeout(resolve, ms));
 }
 
-// src/constants/watch.ts
-var MIN_WATCH_POLL_INTERVAL_MS = 1;
-var DEFAULT_WATCH_POLL_INTERVAL_MS = 1e3;
-var WATCH_REPLAY_PAGE_SIZE = 200;
-var WATCH_TERMINAL_STATUSES = /* @__PURE__ */ new Set([
-  "idle",
+// ../../shared/session/phase.ts
+var NON_REPO_SESSION_KINDS = /* @__PURE__ */ new Set(["terminal_bench", "swe_bench_pro", "swe_polybench"]);
+function isRepoSession(sessionKind) {
+  return !sessionKind || !NON_REPO_SESSION_KINDS.has(sessionKind);
+}
+var TERMINAL_PHASES_ARRAY = [
   "completed",
   "blocked",
   "failed",
   "stopped",
-  "stopped_resumable",
   "archived"
-]);
+];
+var TERMINAL_PHASES = new Set(TERMINAL_PHASES_ARRAY);
+function isTerminalPhase(phase, sessionKind) {
+  if (TERMINAL_PHASES.has(phase)) return true;
+  return phase === "idle" && !isRepoSession(sessionKind);
+}
+
+// src/constants/watch.ts
+var MIN_WATCH_POLL_INTERVAL_MS = 1;
+var DEFAULT_WATCH_POLL_INTERVAL_MS = 1e3;
+var WATCH_REPLAY_PAGE_SIZE = 200;
+function isWatchTerminal(phase, sessionKind) {
+  return isTerminalPhase(phase, sessionKind);
+}
 
 // src/uploads.ts
 import { readFile } from "fs/promises";
@@ -961,6 +973,15 @@ function mergeMemoryRefs(ids, rawRefs) {
       const ref = { id };
       if (typeof record.path === "string" && record.path.trim()) ref.path = record.path.trim();
       if (typeof record.title === "string" && record.title.trim()) ref.title = record.title.trim();
+      if (typeof record.selectionRank === "number") ref.selectionRank = record.selectionRank;
+      if (typeof record.selectionScore === "number") ref.selectionScore = record.selectionScore;
+      if (typeof record.reason === "string" && record.reason.trim()) ref.reason = record.reason.trim();
+      if (typeof record.expectedEffect === "string" && record.expectedEffect.trim()) {
+        ref.expectedEffect = record.expectedEffect.trim();
+      }
+      if (typeof record.observedEffect === "string" && record.observedEffect.trim()) {
+        ref.observedEffect = record.observedEffect.trim();
+      }
       byId.set(id, ref);
     }
   }
@@ -1333,7 +1354,6 @@ var ERROR_CODES = [
   "rate_limit",
   "api_error",
   "config_error",
-  "doom_loop",
   "failed_edits",
   "empty_completion",
   "followup_not_started",
@@ -1366,7 +1386,6 @@ var ERROR_CODE_LABELS = {
   rate_limit: "Provider rate limit",
   api_error: "Provider API error",
   config_error: "Configuration error",
-  doom_loop: "Repeated failure loop",
   failed_edits: "Edit failure",
   empty_completion: "Empty completion",
   followup_not_started: "Follow-up did not start",
@@ -1402,6 +1421,20 @@ function formatSessionErrorMessage(error, code) {
 }
 
 // src/utils/session-output.ts
+var VALID_PHASES = /* @__PURE__ */ new Set([
+  "idle",
+  "running",
+  "waiting_for_input",
+  "finalizing",
+  "completed",
+  "blocked",
+  "failed",
+  "stopped",
+  "archived"
+]);
+var VALID_SANDBOX_SUBSTATES = /* @__PURE__ */ new Set(["creating", "reconnecting", "stopping", "none"]);
+var VALID_STOP_MODES = /* @__PURE__ */ new Set(["user", "resumable", "none"]);
+var VALID_FINALIZING_STEPS = /* @__PURE__ */ new Set(["post_execution", "publishing", "none"]);
 function formatDate(value) {
   const date = new Date(value);
   if (Number.isNaN(date.getTime())) return value;
@@ -1601,9 +1634,21 @@ function parseSsePayload(payload) {
   for (const message of messages) {
     const data = message.data ? parseJsonObject(message.data) : {};
     if (message.event === "status") {
+      const phaseRaw = data.phase;
+      const phase = typeof phaseRaw === "string" && VALID_PHASES.has(phaseRaw) ? phaseRaw : null;
       const entry = {
-        status: typeof data.status === "string" ? data.status : "unknown"
+        phase
       };
+      if (typeof data.sandboxSubstate === "string" && VALID_SANDBOX_SUBSTATES.has(data.sandboxSubstate)) {
+        entry.sandboxSubstate = data.sandboxSubstate;
+      }
+      if (typeof data.stopMode === "string" && VALID_STOP_MODES.has(data.stopMode)) {
+        entry.stopMode = data.stopMode;
+      }
+      if (typeof data.finalizingStep === "string" && VALID_FINALIZING_STEPS.has(data.finalizingStep)) {
+        entry.finalizingStep = data.finalizingStep;
+      }
+      if (typeof data.sessionKind === "string") entry.sessionKind = data.sessionKind;
       if (typeof data.title === "string") entry.title = data.title;
       if (typeof data.spawnDurationMs === "number" || data.spawnDurationMs === null) {
         entry.spawnDurationMs = data.spawnDurationMs;
@@ -1734,10 +1779,16 @@ function parseNonNegativeInteger(raw, name, defaultValue) {
   return Number(raw);
 }
 function formatStatusLine(status) {
+  const label = status.phase ?? "unknown";
+  const substate = [];
+  if (status.sandboxSubstate && status.sandboxSubstate !== "none") substate.push(status.sandboxSubstate);
+  if (status.stopMode && status.stopMode !== "none") substate.push(status.stopMode);
+  if (status.finalizingStep && status.finalizingStep !== "none") substate.push(status.finalizingStep);
+  const phaseLabel = substate.length > 0 ? `${label}/${substate.join("/")}` : label;
   const details = [];
   if (status.title) details.push(status.title);
   if (status.spawnDurationMs != null) details.push(`spawn ${(status.spawnDurationMs / 1e3).toFixed(1)}s`);
-  return details.length > 0 ? `[status] ${status.status} | ${details.join(" | ")}` : `[status] ${status.status}`;
+  return details.length > 0 ? `[status] ${phaseLabel} | ${details.join(" | ")}` : `[status] ${phaseLabel}`;
 }
 async function fetchPromptLabels(config, sessionId) {
   const data = await apiFetch(config, `/api/sessions/${sessionId}/prompts`);
@@ -1817,7 +1868,7 @@ async function watchCommand(sessionId, options, command) {
         console.log(rendered.line);
       }
       if (receivedFullPage) continue;
-      if (parsed.status?.status && WATCH_TERMINAL_STATUSES.has(parsed.status.status)) break;
+      if (parsed.status?.phase && isWatchTerminal(parsed.status.phase, parsed.status.sessionKind)) break;
       await sleep(effectivePollIntervalMs);
     }
   } finally {
@@ -1850,12 +1901,13 @@ function buildPromptFailureError(sessionId, prompt, sessionUrl) {
     hint: `Inspect with: arcanist sessions transcript ${sessionId}`
   });
 }
-function extractSessionStatus(payload) {
+function extractSessionLifecycle(payload) {
   const session = payload.session;
-  if (session && typeof session === "object" && typeof session.status === "string") {
-    return String(session.status);
-  }
-  return typeof payload.status === "string" ? payload.status : "unknown";
+  const root = session && typeof session === "object" ? session : payload;
+  const phaseRaw = root.phase;
+  const phase = typeof phaseRaw === "string" && VALID_PHASES.has(phaseRaw) ? phaseRaw : null;
+  const sessionKind = typeof root.sessionKind === "string" ? root.sessionKind : null;
+  return { phase, sessionKind };
 }
 async function fetchCreatedPromptStatus(config, sessionId, promptId) {
   const promptList = await apiFetch(config, `/api/sessions/${sessionId}/prompts`);
@@ -1869,7 +1921,8 @@ async function waitForCreatedPromptToSettle(config, sessionId, promptId, pollInt
       return createdPrompt;
     }
     const sessionPayload = await apiFetch(config, `/api/sessions/${sessionId}`);
-    if (WATCH_TERMINAL_STATUSES.has(extractSessionStatus(sessionPayload))) {
+    const lifecycle = extractSessionLifecycle(sessionPayload);
+    if (lifecycle.phase && isWatchTerminal(lifecycle.phase, lifecycle.sessionKind)) {
       return fetchCreatedPromptStatus(config, sessionId, promptId);
     }
     await sleep(effectivePollIntervalMs);
@@ -2092,9 +2145,11 @@ async function getSessionCommand(sessionId, options, command) {
   if (session.title) console.log(`Title: ${String(session.title)}`);
 }
 async function sessionEventsCommand(sessionId, options, command) {
+  const afterSequence = resolveSequenceOption(options.afterSequence, options.after, "--after-sequence", "--after");
+  const beforeSequence = resolveSequenceOption(options.beforeSequence, options.before, "--before-sequence", "--before");
   if (options.follow) {
-    if (options.beforeSequence) {
-      throw new CliError("user", "--before-sequence cannot be used with --follow.");
+    if (beforeSequence) {
+      throw new CliError("user", "--before-sequence/--before cannot be used with --follow.");
     }
     if (options.promptId) {
       throw new CliError(
@@ -2104,7 +2159,7 @@ async function sessionEventsCommand(sessionId, options, command) {
     }
     await watchCommand(
       sessionId,
-      { ...options, pollInterval: options.pollInterval, afterSequence: options.afterSequence, limit: options.limit },
+      { ...options, pollInterval: options.pollInterval, afterSequence, limit: options.limit },
       command
     );
     return;
@@ -2112,8 +2167,8 @@ async function sessionEventsCommand(sessionId, options, command) {
   const runtime = getRuntimeOptions(command, options);
   const config = requireConfig(runtime);
   const query = new URLSearchParams();
-  if (options.afterSequence) query.set("after_sequence", options.afterSequence);
-  if (options.beforeSequence) query.set("before_sequence", options.beforeSequence);
+  if (afterSequence) query.set("after_sequence", afterSequence);
+  if (beforeSequence) query.set("before_sequence", beforeSequence);
   if (options.promptId) query.set("prompt_id", options.promptId);
   if (options.limit) query.set("limit", options.limit);
   const payload = await apiFetch(config, `/api/sessions/${sessionId}/events/history${query.size ? `?${query.toString()}` : ""}`);
@@ -2146,6 +2201,12 @@ async function sessionEventsCommand(sessionId, options, command) {
   }
   if (textOpen) process.stdout.write("\n");
 }
+function resolveSequenceOption(canonicalValue, aliasValue, canonicalFlag, aliasFlag) {
+  if (canonicalValue !== void 0 && aliasValue !== void 0) {
+    throw new CliError("user", `${aliasFlag} cannot be combined with ${canonicalFlag}.`);
+  }
+  return canonicalValue ?? aliasValue;
+}
 async function usageCommand(sessionId, options, command) {
   const runtime = getRuntimeOptions(command, options);
   const config = requireConfig(runtime);
@@ -2165,17 +2226,38 @@ async function usageCommand(sessionId, options, command) {
 async function stopCommand(sessionId, options = {}, command) {
   const runtime = getRuntimeOptions(command, options);
   const config = requireConfig(runtime);
-  const response = await apiFetch(config, `/api/sessions/${sessionId}/stop`, {
-    method: "POST"
-  });
-  const status = response.status ?? "stopping";
+  let status;
+  try {
+    const response = await apiFetch(config, `/api/sessions/${sessionId}/stop`, {
+      method: "POST"
+    });
+    status = response.status ?? "stopping";
+  } catch (err) {
+    const parsed = parseStopBlocked(err);
+    if (!parsed) throw err;
+    status = parsed.reason;
+  }
   if (isJson(command, options)) {
     writeJson({ sessionId, status });
     return;
   }
-  console.log(
-    status === "already_stopped" ? `Session ${sessionId} is already stopped.` : `Stop requested for session ${sessionId}.`
-  );
+  if (status === "already_stopped") {
+    console.log(`Session ${sessionId} is already stopped.`);
+  } else if (status === "stopping" || status === "stopped") {
+    console.log(`Stop requested for session ${sessionId}.`);
+  } else {
+    console.log(`Session ${sessionId} cannot be stopped from phase=${status}.`);
+  }
+}
+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;
+  }
 }
 
 // src/commands/test-creds.ts
@@ -2455,11 +2537,13 @@ Examples:
   arcanist sessions search "mcp debugging" --tag codex --json
 `
 ).action((query, options, command) => searchSessionsCommand(query, options, command));
-sessions.command("events").description("Read or follow session replay events").argument("<session-id>", "Session ID").option("--after-sequence <n>", "Return events after this sequence").option("--before-sequence <n>", "Return events before this sequence").option("--prompt-id <id>", "Filter events by prompt ID").option("--limit <n>", "Maximum events to return").option("--follow", "Follow events until the session is idle").option("--poll-interval <ms>", "Polling interval in milliseconds", String(DEFAULT_WATCH_POLL_INTERVAL_MS)).addHelpText(
+sessions.command("events").description("Read or follow session replay events").argument("<session-id>", "Session ID").option("--after-sequence <n>", "Return events after this sequence").option("--after <n>", "Alias for --after-sequence").option("--before-sequence <n>", "Return events before this sequence").option("--before <n>", "Alias for --before-sequence").option("--prompt-id <id>", "Filter events by prompt ID").option("--limit <n>", "Maximum events to return").option("--follow", "Follow events until the session is idle").option("--poll-interval <ms>", "Polling interval in milliseconds", String(DEFAULT_WATCH_POLL_INTERVAL_MS)).addHelpText(
   "after",
   `
 Examples:
   arcanist sessions events <session-id> --json
+  arcanist sessions events <session-id> --after-sequence 540 --limit 250 --json
+  arcanist sessions events <session-id> --after 540 --limit 250 --json
   arcanist sessions events <session-id> --follow --json
 `
 ).action((sessionId, options, command) => sessionEventsCommand(sessionId, options, command));

package/package.json

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