chainlesschain 0.166.103 → 0.166.105

package/src/commands/agent.js

@@ -46,6 +46,10 @@ import {
   isFormalQualityHermeticRuntime,
 } from "../lib/formal-quality-eval-runtime.js";
 import { withQuietStdout } from "../runtime/quiet-stdout.js";
+import {
+  createSessionPersistenceFailure,
+  projectSessionPersistenceFailure,
+} from "../lib/session-persistence-failure.js";
 import { registerControlledHostCommand } from "./agent-controlled-host.js";
 import { registerProcessOwnershipCommands } from "./agent-process-ownership.js";
 import { resolveCredentialEnvironmentValue } from "../lib/process-execution-broker/credential-transport.js";
@@ -1167,29 +1171,61 @@ export function registerAgentCommand(program, dependencies = {}) {
         // initialize AppConfig/DatabaseManager. Their legacy console.info
         // diagnostics belong on stderr; stdout may already be reserved for a
         // JSON/NDJSON protocol selected by this command.
-        await withQuietStdout(async () => {
-          const { sessionHasPersistedEvidence } = await import(
-            "../harness/jsonl-session-store.js"
+        try {
+          await withQuietStdout(async () => {
+            const { sessionHasPersistedEvidence } = await import(
+              "../harness/jsonl-session-store.js"
+            );
+            if (!sessionHasPersistedEvidence(options.session)) {
+              const [{ bootstrap }, { ensureCanonicalSessionFromDatabase }] =
+                await Promise.all([
+                  import("../runtime/bootstrap.js"),
+                  import("../lib/session-transcript-migration.js"),
+                ]);
+              const context =
+                sessionBootstrapContext ||
+                (await bootstrap({ verbose: program.opts().verbose }));
+              sessionBootstrapContext = context;
+              const database =
+                context.db?.getDatabase?.() || context.db?.getDb?.() || null;
+              const migrated = ensureCanonicalSessionFromDatabase(
+                database,
+                options.session,
+              );
+              if (migrated?.sessionId) options.session = migrated.sessionId;
+            }
+          });
+        } catch (error) {
+          const persistence = projectSessionPersistenceFailure(
+            createSessionPersistenceFailure(error, {
+              sessionId: options.session,
+              operation: "session-start",
+            }),
+            { phase: "before-model" },
           );
-          if (!sessionHasPersistedEvidence(options.session)) {
-            const [{ bootstrap }, { ensureCanonicalSessionFromDatabase }] =
-              await Promise.all([
-                import("../runtime/bootstrap.js"),
-                import("../lib/session-transcript-migration.js"),
-              ]);
-            const context =
-              sessionBootstrapContext ||
-              (await bootstrap({ verbose: program.opts().verbose }));
-            sessionBootstrapContext = context;
-            const database =
-              context.db?.getDatabase?.() || context.db?.getDb?.() || null;
-            const migrated = ensureCanonicalSessionFromDatabase(
-              database,
-              options.session,
+          if (!persistence) throw error;
+          // Emit after withQuietStdout restores stdout, so a machine client
+          // receives the failure even when preparation never reaches a runner.
+          if (["json", "stream-json"].includes(options.outputFormat)) {
+            process.stdout.write(
+              `${JSON.stringify({
+                type: "result",
+                subtype: "error_persistence",
+                is_error: true,
+                code: persistence.code,
+                error: "Canonical session preparation failed",
+                session_id: options.session,
+                persistence,
+              })}\n`,
             );
-            if (migrated?.sessionId) options.session = migrated.sessionId;
           }
-        });
+          process.stderr.write(
+            `${persistence.code}: canonical session preparation was refused\n`,
+          );
+          await _finishWorktree();
+          process.exitCode = 1;
+          return;
+        }
       }
 
       // --fork-session: branch the resolved session into a NEW id so the

package/src/commands/batch.js

@@ -44,6 +44,15 @@ export const _processDeps = {
   execFileSync: executionBroker.execFileSync.bind(executionBroker),
 };
 
+function collectUtf8(stream) {
+  const decoder = new TextDecoder("utf-8");
+  let output = "";
+  stream?.on("data", (chunk) => {
+    output += decoder.decode(chunk, { stream: true });
+  });
+  return () => output + decoder.decode();
+}
+
 const BIN = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
   "..",
@@ -382,8 +391,7 @@ function spawnAgent(prompt, cwd, opts = {}) {
       },
       "batch:agent-run",
     );
-    let err = "";
-    child.stderr?.on("data", (d) => (err += d.toString("utf8")));
+    const readErr = collectUtf8(child.stderr);
     if (child.stdin) {
       child.stdin.on("error", () => {});
       child.stdin.end(String(prompt || ""));
@@ -392,7 +400,7 @@ function spawnAgent(prompt, cwd, opts = {}) {
     child.on("close", (code) =>
       code === 0
         ? resolve({ code })
-        : reject(new Error(err.trim() || `agent exited ${code}`)),
+        : reject(new Error(readErr().trim() || `agent exited ${code}`)),
     );
   });
 }
@@ -410,13 +418,12 @@ function runTestCommand(command, cwd) {
       },
       "batch:test-command",
     );
-    let err = "";
-    child.stderr?.on("data", (d) => (err += d.toString("utf8")));
+    const readErr = collectUtf8(child.stderr);
     child.on("error", (e) => reject(new Error(e.message)));
     child.on("close", (code) =>
       code === 0
         ? resolve({ code })
-        : reject(new Error(err.trim() || `test exited ${code}`)),
+        : reject(new Error(readErr().trim() || `test exited ${code}`)),
     );
   });
 }
@@ -494,18 +501,16 @@ async function decomposeGoal(goal, parts, opts = {}) {
       },
       "batch:decompose",
     );
-    let out = "";
-    let err = "";
-    child.stdout?.on("data", (d) => (out += d.toString("utf8")));
-    child.stderr?.on("data", (d) => (err += d.toString("utf8")));
+    const readOut = collectUtf8(child.stdout);
+    const readErr = collectUtf8(child.stderr);
     child.on("error", (e) => reject(new Error(e.message)));
     child.on("close", (code) => {
       if (code !== 0) {
-        reject(new Error(err.trim() || `decompose agent exited ${code}`));
+        reject(new Error(readErr().trim() || `decompose agent exited ${code}`));
         return;
       }
       try {
-        const parsed = JSON.parse(out.trim());
+        const parsed = JSON.parse(readOut().trim());
         resolve(parsed.units);
       } catch (e) {
         reject(new Error(`could not parse decomposition output: ${e.message}`));

package/src/commands/eval.js

@@ -206,6 +206,16 @@ function makeHeadlessRunAgent(opts = {}) {
       });
       let out = "";
       let err = "";
+      const stdoutDecoder = new TextDecoder("utf-8");
+      const stderrDecoder = new TextDecoder("utf-8");
+      const readOutput = () => {
+        out += stdoutDecoder.decode();
+        return out;
+      };
+      const readError = () => {
+        err += stderrDecoder.decode();
+        return err;
+      };
       let settled = false;
       let timedOut = false;
       let graceTimer = null;
@@ -235,25 +245,31 @@ function makeHeadlessRunAgent(opts = {}) {
         graceTimer = setTimeout(() => {
           done({
             ok: false,
-            output: out,
+            output: readOutput(),
             error: `timed out after ${timeoutMs || 120000}ms (process did not exit after kill)`,
           });
         }, 10000);
         if (typeof graceTimer.unref === "function") graceTimer.unref();
       }, timeoutMs || 120000);
       if (typeof timer.unref === "function") timer.unref();
-      child.stdout?.on("data", (d) => (out += d.toString("utf8")));
-      child.stderr?.on("data", (d) => (err += d.toString("utf8")));
+      child.stdout?.on("data", (d) => {
+        out += stdoutDecoder.decode(d, { stream: true });
+      });
+      child.stderr?.on("data", (d) => {
+        err += stderrDecoder.decode(d, { stream: true });
+      });
       child.on("error", (e) => {
         clearTimeout(timer);
-        done({ ok: false, output: out, error: e.message });
+        done({ ok: false, output: readOutput(), error: e.message });
       });
       child.on("close", (code, signal) => {
         clearTimeout(timer);
+        const output = readOutput();
+        const errorOutput = readError();
         if (timedOut) {
           done({
             ok: false,
-            output: out,
+            output,
             exitCode: code,
             signal: signal || null,
             error: `timed out after ${timeoutMs || 120000}ms`,
@@ -268,11 +284,11 @@ function makeHeadlessRunAgent(opts = {}) {
                 usage: null,
                 totalCostUsd: null,
               }
-            : verifyAgentTerminal(out);
+            : verifyAgentTerminal(output);
         const ok = code === 0 && !signal && (opts._argv || terminalVerified);
         done({
           ok: Boolean(ok),
-          output: out,
+          output,
           exitCode: code,
           signal: signal || null,
           terminalVerified,
@@ -283,7 +299,7 @@ function makeHeadlessRunAgent(opts = {}) {
             ? null
             : code === 0 && !signal
               ? "agent stream lacks a valid success terminal"
-              : err || `agent process exited with ${signal || code}`,
+              : errorOutput || `agent process exited with ${signal || code}`,
         });
       });
     });

package/src/data/changelog.json

@@ -1,6 +1,32 @@
 {
   "source": "CHANGELOG.md",
   "releases": [
+    {
+      "productVersion": null,
+      "cliVersion": "0.166.105",
+      "cliVersions": ["0.166.105"],
+      "date": "2026-10-18",
+      "title": "",
+      "sections": [
+        {
+          "heading": "Fixed - cc CLI 0.166.105: task delivery quality and safe diagnostics (candidate, 2026-10-18)",
+          "body": "- Add general coding guidance and static relative-import feedback without executing candidate code or injecting task-specific answers.\n- Do not report completion while deferred writes remain; distinguish provider transport aborts from owned cancellation.\n- Preserve bounded allowlisted transport cause details without changing original errors, exits, fetch, retries or replay.\n- Pair candidate VS Code 0.37.150 and JetBrains 0.4.167 with CLI 0.166.105 after exact-commit matrices, OIDC publishing and public readback. JetBrains restores saved Markdown consistently with live rendering.\n- Campaign: 25/45 cumulative candidate acceptances; latest 22/25 collected, 4 end-to-end passes, 3 Windows JetBrains samples ongoing. New versions are not yet published or retested."
+        }
+      ]
+    },
+    {
+      "productVersion": null,
+      "cliVersion": "0.166.104",
+      "cliVersions": ["0.166.104"],
+      "date": null,
+      "title": "",
+      "sections": [
+        {
+          "heading": "Fixed - cc CLI 0.166.104: recover governed sessions safely",
+          "body": "- Verify Jev model usage starts and settlements before resuming a durable\n  session; stop further provider calls when usage or persistence is unresolved.\n- Bind session budget roots to verified JSONL for single-prompt, stream-json and\n  REPL entry points, retaining the original limits across resume.\n- Preserve UTF-8 in background, batch and SDK process streams; reject malformed\n  background frames and stale MCP tool catalogs before dispatch.\n- Publish Agent SDK 0.2.15 before this CLI release, then pair VS Code 0.37.149\n  and JetBrains 0.4.166 with CLI 0.166.104 after exact-commit gates and public\n  npm readback. Formal paid evaluation remains NOT_RUN."
+        }
+      ]
+    },
     {
       "productVersion": null,
       "cliVersion": "0.166.103",