@evident-ai/cli 3.5.2-dev.8b315dd → 3.5.2-dev.8c3e0f7

package/README.md

@@ -117,9 +117,11 @@ Options:
   starting OpenCode. Requires `--litestream-config` for the replica location and
   `--session-db-no-replicate-marker` for give-up records. Omit it for a normal local
   run.
-- `--restore-runner-credentials` — Restore the hosted runner secret and persisted
-   credential stores before starting OpenCode. Intended for runner images; omit it
-   for a normal local run.
+- `--restore-runner-credentials` — Restore the persisted credential stores (Claude/OpenCode
+   model auth) before starting OpenCode. Intended for runner images; omit it for a normal
+   local run. The hosted runner secret's env vars (`GH_TOKEN` and the MCP tokens) are
+   fetched on every `evident run` boot regardless of this flag — it's in-memory,
+   per-process state, never persisted, so there's no store to restore over.
 - `--opencode-config-overlay <path>` — Apply a runner-provided OpenCode config
   before starting OpenCode. Relative paths are resolved from the working directory;
   an existing `opencode.jsonc` is replaced before `opencode.json`.

package/dist/index.js

@@ -1322,7 +1322,7 @@ import ora4 from "ora";
 import { select as select4 } from "@inquirer/prompts";
 
 // src/lib/telemetry.ts
-var CLI_VERSION = (true ? "3.5.2-dev.8b315dd" : void 0) ?? process.env.npm_package_version ?? "unknown";
+var CLI_VERSION = (true ? "3.5.2-dev.8c3e0f7" : void 0) ?? process.env.npm_package_version ?? "unknown";
 function getCliVersion() {
   return CLI_VERSION;
 }
@@ -1515,9 +1515,10 @@ function redactDiagnosticText(message) {
 }
 
 // src/lib/opencode/session-db-reclaim.ts
-var SESSION_DB_RECLAIM_CHUNK_PAGES = 2e3;
+var SESSION_DB_RECLAIM_CHUNK_PAGES = 100;
 var SESSION_DB_RECLAIM_TIME_BUDGET_MS = 500;
 var SESSION_DB_RECLAIM_PAGE_CEILING = 4e4;
+var SESSION_DB_RECLAIM_BUSY_TIMEOUT_MS = 5e3;
 function errorMessage(error2) {
   if (!(error2 instanceof Error)) return String(error2);
   return error2.cause instanceof Error ? error2.cause.message : error2.message;
@@ -1642,6 +1643,7 @@ function readFreelistCount(db) {
   return db.prepare("PRAGMA freelist_count").get().freelist_count;
 }
 function readCheckpointResult(db) {
+  db.exec("PRAGMA busy_timeout=0");
   const row = db.prepare("PRAGMA wal_checkpoint(TRUNCATE)").get();
   return { busy: row.busy !== 0, log: row.log, checkpointed: row.checkpointed };
 }
@@ -1659,7 +1661,7 @@ async function probeReclaimAvailability(input) {
   try {
     const db = new sqlite.DatabaseSync(dbPath, { readOnly: true });
     try {
-      db.exec("PRAGMA busy_timeout=5000");
+      db.exec(`PRAGMA busy_timeout=${SESSION_DB_RECLAIM_BUSY_TIMEOUT_MS}`);
       autoVacuum = db.prepare("PRAGMA auto_vacuum").get().auto_vacuum;
     } finally {
       db.close();
@@ -1679,7 +1681,8 @@ async function reclaimSessionDbSpace(input) {
     timeBudgetMs = SESSION_DB_RECLAIM_TIME_BUDGET_MS,
     chunkPages = SESSION_DB_RECLAIM_CHUNK_PAGES,
     now = () => performance.now(),
-    allowFullVacuum = true
+    hasLiveTurn = false,
+    hasProvenanceAnomaly = false
   } = input;
   let sqlite;
   try {
@@ -1695,17 +1698,19 @@ async function reclaimSessionDbSpace(input) {
   let db;
   try {
     db = new DatabaseSync(dbPath);
-    db.exec("PRAGMA busy_timeout=5000");
+    db.exec(`PRAGMA busy_timeout=${SESSION_DB_RECLAIM_BUSY_TIMEOUT_MS}`);
     const autoVacuum = db.prepare("PRAGMA auto_vacuum").get().auto_vacuum;
     if (autoVacuum === 0) {
-      if (!allowFullVacuum) {
-        console.warn(
-          `[reclaimSessionDbSpace] skipping VACUUM conversion of ${dbPath}: a session turn is live`
-        );
+      if (hasLiveTurn || hasProvenanceAnomaly) {
+        const detail = [
+          ...hasLiveTurn ? ["a session turn is live"] : [],
+          ...hasProvenanceAnomaly ? ["session-DB provenance is anomalous"] : []
+        ].join(" and ");
+        console.warn(`[reclaimSessionDbSpace] skipping VACUUM conversion of ${dbPath}: ${detail}`);
         return {
           ok: false,
           skipped: "full-vacuum-blocked",
-          detail: "a session turn is live"
+          detail
         };
       }
       const fileBytesForGuard = statSync2(dbPath).size;
@@ -1723,7 +1728,9 @@ async function reclaimSessionDbSpace(input) {
       db.exec("VACUUM");
       const afterBytes = readLogicalBytes(db);
       const freelistAfter = readFreelistCount(db);
+      const checkpointStartedAt = now();
       const checkpoint = readCheckpointResult(db);
+      const checkpointElapsedMs = Math.max(0, now() - checkpointStartedAt);
       return {
         ok: true,
         mode: "convert",
@@ -1734,10 +1741,18 @@ async function reclaimSessionDbSpace(input) {
         pagesReclaimed: Math.max(0, freelistBefore - freelistAfter),
         stoppedBy: "freelist-drained",
         elapsedMs: Math.max(0, now() - startedAt),
+        checkpointElapsedMs,
         checkpoint
       };
     }
     if (autoVacuum === 2) {
+      if (hasLiveTurn) {
+        const detail = "a session turn is live";
+        console.warn(
+          `[reclaimSessionDbSpace] skipping incremental reclaim of ${dbPath}: ${detail}`
+        );
+        return { ok: false, skipped: "live-turn", detail };
+      }
       const beforeBytes = readLogicalBytes(db);
       const freelistBefore = readFreelistCount(db);
       const startedAt = now();
@@ -1762,7 +1777,9 @@ async function reclaimSessionDbSpace(input) {
         freelistAfter = readFreelistCount(db);
       }
       const afterBytes = readLogicalBytes(db);
+      const checkpointStartedAt = now();
       const checkpoint = readCheckpointResult(db);
+      const checkpointElapsedMs = Math.max(0, now() - checkpointStartedAt);
       return {
         ok: true,
         mode: "incremental",
@@ -1773,6 +1790,7 @@ async function reclaimSessionDbSpace(input) {
         pagesReclaimed: Math.max(0, freelistBefore - freelistAfter),
         stoppedBy,
         elapsedMs: Math.max(0, now() - startedAt),
+        checkpointElapsedMs,
         checkpoint
       };
     }
@@ -3058,9 +3076,11 @@ function sumAssistantUsage(messages) {
   let cacheWriteSum = 0;
   let costSum = 0;
   let sawCost = false;
+  let toolCallCount = 0;
   let modelId = null;
   let providerId = null;
   for (const m of selected) {
+    toolCallCount += m.parts?.filter((part) => part.type === "tool").length ?? 0;
     const info = m.info;
     if (!info) continue;
     const tokens = info.tokens;
@@ -3097,7 +3117,8 @@ function sumAssistantUsage(messages) {
     usage_tokens_cache_write: cacheWriteSum,
     // NULL means OpenCode never reported a cost; it is distinct from a genuine
     // zero-cost message, which sets `sawCost` with `costSum === 0`.
-    usage_cost_usd: sawCost ? costSum : null
+    usage_cost_usd: sawCost ? costSum : null,
+    turn_tool_calls: toolCallCount
   };
 }
 function messageUsage(messages, userMessageId) {
@@ -12858,7 +12879,9 @@ async function installRunnerSecret({
 }) {
   const arn = env.RUNNER_SECRET_ARN?.trim();
   if (!arn) {
-    log3("runner secret is not configured; continuing without GitHub and MCP credentials");
+    if (!env.GH_TOKEN) {
+      log3("runner secret is not configured; continuing without GitHub and MCP credentials");
+    }
     return false;
   }
   const result = await (commandRunner ?? runCommand2)(
@@ -13978,21 +14001,31 @@ async function runSweep(state, driver, config) {
       sessions_remaining: remaining,
       ...enumerationReason !== null ? { session_enumeration_reason: enumerationReason } : {}
     };
+    const protectedForReclaim = driver.protectedSessionIds();
     const reclaimResult = await reclaimSessionDbSpace({
       dbPath: sessionDbPath(),
       chunkPages: SESSION_DB_RECLAIM_CHUNK_PAGES,
       pageCeiling: SESSION_DB_RECLAIM_PAGE_CEILING,
       timeBudgetMs: SESSION_DB_RECLAIM_TIME_BUDGET_MS,
-      allowFullVacuum: protectedNow.size === 0 && !state.sessionDbProvenanceAnomaly
+      hasLiveTurn: protectedForReclaim.size > 0,
+      hasProvenanceAnomaly: state.sessionDbProvenanceAnomaly
     });
     if (reclaimResult.ok) {
       state.sessionStoreHealth.observeWritable();
       const beforeMib = (reclaimResult.beforeBytes / 1024 / 1024).toFixed(1);
       const afterMib = (reclaimResult.afterBytes / 1024 / 1024).toFixed(1);
       const checkpointNote = reclaimResult.checkpoint.busy ? ` (on-disk file truncation deferred: checkpoint busy, ${reclaimResult.checkpoint.log} WAL frames pending)` : "";
+      const loopElapsedMs = Math.max(
+        0,
+        reclaimResult.elapsedMs - reclaimResult.checkpointElapsedMs
+      );
+      const reclaimIsSlow = reclaimResult.elapsedMs >= 4 * SESSION_DB_RECLAIM_TIME_BUDGET_MS;
+      const perPageNote = reclaimResult.pagesReclaimed > 0 ? `, per-page ${(loopElapsedMs / reclaimResult.pagesReclaimed).toFixed(2)} ms/page` : "";
+      const slowReclaimNote = reclaimIsSlow ? ` (slow reclaim: total elapsed ${reclaimResult.elapsedMs} ms, checkpoint ${reclaimResult.checkpointElapsedMs} ms, loop ${loopElapsedMs} ms, budget ${SESSION_DB_RECLAIM_TIME_BUDGET_MS} ms${perPageNote})` : "";
       logSessionCleanupActivity(state, "sweep", {
         type: "info",
-        message: `Session cleanup: reclaimed session-db space (${reclaimResult.mode}): ${beforeMib} MiB -> ${afterMib} MiB (stopped by ${reclaimResult.stoppedBy}; freelist ${reclaimResult.freelistBefore} -> ${reclaimResult.freelistAfter} pages)` + checkpointNote,
+        level: reclaimIsSlow ? "warn" : "info",
+        message: `Session cleanup: reclaimed session-db space (${reclaimResult.mode}): ${beforeMib} MiB -> ${afterMib} MiB (stopped by ${reclaimResult.stoppedBy}; freelist ${reclaimResult.freelistBefore} -> ${reclaimResult.freelistAfter} pages)` + checkpointNote + slowReclaimNote,
         metadata: {
           outcome: "reclaimed",
           mode: reclaimResult.mode,
@@ -14005,6 +14038,7 @@ async function runSweep(state, driver, config) {
           freelist_after: reclaimResult.freelistAfter,
           pages_reclaimed: reclaimResult.pagesReclaimed,
           reclaim_elapsed_ms: reclaimResult.elapsedMs,
+          checkpoint_elapsed_ms: reclaimResult.checkpointElapsedMs,
           ...sessionMetadata
         }
       });
@@ -14757,22 +14791,21 @@ async function run(options) {
         );
       }
     }
-    if (options.restoreRunnerCredentials) {
-      log2(state, "Restoring runner credentials before starting OpenCode");
-      const credentialContext = {
-        env: process.env,
-        log: (message, level = "info") => {
-          if (level === "error") {
-            logActivity(state, { type: "error", error: message });
-          } else {
-            logActivity(state, { type: "info", level, message });
-          }
+    const credentialContext = {
+      env: process.env,
+      log: (message, level = "info") => {
+        if (level === "error") {
+          logActivity(state, { type: "error", error: message });
+        } else {
+          logActivity(state, { type: "info", level, message });
         }
-      };
-      const githubTokenPopulated = await installRunnerSecret(credentialContext);
+      }
+    };
+    const githubTokenPopulated = await installRunnerSecret(credentialContext);
+    if (options.restoreRunnerCredentials) {
       await restoreCredentialStores(credentialContext);
-      if (githubTokenPopulated) await configureGitHubAccess(credentialContext);
     }
+    if (githubTokenPopulated) await configureGitHubAccess(credentialContext);
     state.credentialSync?.arm();
     const openCodeActivityForwarder = createOpenCodeActivityForwarder(
       () => ({