@turns/dev 0.0.29 → 0.0.30

package/dist/agent-anchor.js

@@ -15,6 +15,7 @@ import { recordLettaSessionContext } from "./letta-session-context.js";
 import { recoverAsyncQuestion } from "./codex-async-question-recovery.js";
 import { existingTranscriptPath } from "./compressed-transcripts.js";
 import { BackgroundTaskState } from "./background-task-state.js";
+import { harnessExitError, StderrTail } from "./harness-exit.js";
 const socketIndex = process.argv.indexOf('--socket');
 const socketPath = socketIndex >= 0 ? path.resolve(process.argv[socketIndex + 1] || '') : '';
 if (!socketPath)
@@ -73,7 +74,7 @@ function writeLine(socket, value) {
     if (!socket.destroyed && socket.writable)
         socket.write(`${JSON.stringify(value)}\n`);
 }
-function childResult(child, timeoutMs) {
+function childResult(child, timeoutMs, harness, stderrTail) {
     const result = deferred();
     let settled = false;
     const finish = (value) => {
@@ -84,7 +85,17 @@ function childResult(child, timeoutMs) {
         }
     };
     child.once('error', (error) => finish({ exitCode: null, signal: null, timedOut: false, error: { message: errorText(error) } }));
-    child.once('close', (code, signal) => finish({ exitCode: code, signal, timedOut: false, error: null }));
+    child.once('close', (code, signal) => {
+        if (!code)
+            return finish({ exitCode: code, signal, timedOut: false, error: null });
+        const error = harnessExitError(harness, code, signal, stderrTail.lines());
+        finish({
+            exitCode: code,
+            signal,
+            timedOut: false,
+            error: { message: error.message, harnessExit: error.harnessExit },
+        });
+    });
     const timer = setTimeout(() => {
         try {
             process.kill(-child.pid, 'SIGTERM');
@@ -108,6 +119,7 @@ class LineTransport {
     pending;
     lastEventAt;
     child;
+    stderrTail = new StderrTail();
     constructor(anchor, harness, command, args, cwd, env = {}) {
         this.anchor = anchor;
         this.harness = harness;
@@ -129,14 +141,17 @@ class LineTransport {
         });
         anchor.child = this.child;
         readline.createInterface({ input: this.child.stdout }).on('line', (line) => this.receive(line));
-        readline.createInterface({ input: this.child.stderr }).on('line', (line) => anchor.emitLive({
-            event: 'process.line',
-            source: `${harness}.stderr`,
-            raw: line,
-            phase: 'queued',
-        }));
+        readline.createInterface({ input: this.child.stderr }).on('line', (line) => {
+            this.stderrTail.observe(line);
+            anchor.emitLive({
+                event: 'process.line',
+                source: `${harness}.stderr`,
+                raw: line,
+                phase: 'queued',
+            });
+        });
         this.child.once('close', (code, signal) => {
-            const error = new Error(`${harness} transport exited with ${signal || code}`);
+            const error = harnessExitError(anchor.source, code, signal, this.stderrTail.lines());
             for (const waiter of this.pending.values())
                 waiter.reject(error);
             this.pending.clear();
@@ -1772,7 +1787,12 @@ class Anchor {
             result: null,
         };
         this.setStatus('running');
-        void this.run(Boolean(request.createSession), this.turnTimeoutMs).then((result) => this.finish(result), (error) => this.finish({ exitCode: 1, signal: null, timedOut: false, error: { message: errorText(error) } }));
+        void this.run(Boolean(request.createSession), this.turnTimeoutMs).then((result) => this.finish(result), (error) => this.finish({
+            exitCode: 1,
+            signal: null,
+            timedOut: false,
+            error: { message: errorText(error), ...(error?.harnessExit ? { harnessExit: error.harnessExit } : {}) },
+        }));
         return { accepted: true, messageId: this.currentTurn.messageId, pid: process.pid };
     }
     async run(createSession, timeoutMs) {
@@ -1871,6 +1891,7 @@ class Anchor {
             this.child = child;
             if (invocation.stdin)
                 child.stdin.end(`${invocation.stdin}\n`);
+            const stderrTail = new StderrTail();
             for (const [stream, source] of [
                 [child.stdout, `${this.source}.stdout`],
                 [child.stderr, `${this.source}.stderr`],
@@ -1884,9 +1905,11 @@ class Anchor {
                     const processSessionId = processEventSessionId(this.source, json);
                     if (processSessionId && processSessionId !== this.agentSessionId)
                         this.updateIdentity(processSessionId);
+                    if (stream === child.stderr)
+                        stderrTail.observe(line);
                     this.emitLive({ event: 'process.line', source, raw: line, json, phase: 'queued' });
                 });
-            return childResult(child, timeoutMs);
+            return childResult(child, timeoutMs, this.source, stderrTail);
         }
         if (ACP_PROFILES[this.source]) {
             if (!(this.transport instanceof AcpTransport) ||

package/dist/codex-update.js

@@ -1,6 +1,9 @@
 /** Machine-local Codex CLI self-update, callable by the Turns daemon. */
 import { spawn } from 'node:child_process';
+import fs from 'node:fs';
 import os from 'node:os';
+import path from 'node:path';
+import { executablePath } from "./executable-path.js";
 import { codexCommandIsAppManaged, resolveHarnessCommand } from "./harness-registry.js";
 const VERSION_TIMEOUT_MS = 15_000;
 const UPDATE_TIMEOUT_MS = 5 * 60_000;
@@ -11,7 +14,9 @@ function runCommand(command, args, timeoutMs) {
     return new Promise((resolve) => {
         const child = spawn(command, args, {
             cwd: os.homedir(),
-            env: process.env,
+            // npm and Codex's npm launcher are `#!/usr/bin/env node` scripts; a
+            // service manager's PATH may not include the daemon's own Node.
+            env: { ...process.env, PATH: executablePath() },
             stdio: ['ignore', 'pipe', 'pipe'],
         });
         let output = '';
@@ -56,10 +61,54 @@ function commandFailure(result, fallback) {
         .filter(Boolean);
     return new Error(lines.at(-1) || fallback);
 }
+function realpathOrSelf(file) {
+    try {
+        return fs.realpathSync(file);
+    }
+    catch {
+        return file;
+    }
+}
+function findExecutable(name, preferred) {
+    for (const directory of [...preferred, ...executablePath().split(path.delimiter)]) {
+        const candidate = path.join(directory, name);
+        try {
+            fs.accessSync(candidate, fs.constants.X_OK);
+            return candidate;
+        }
+        catch { }
+    }
+    return '';
+}
+export function codexUpgradeCommand(command, lookup = {}) {
+    const resolved = (lookup.realpath || realpathOrSelf)(command).replaceAll('\\', '/');
+    const find = lookup.findExecutable || findExecutable;
+    // `npm i -g @openai/codex` links <prefix>/bin/codex to the package's launcher.
+    const npm = resolved.match(/^(.*?)\/(?:lib\/)?node_modules\/@openai\/codex\//u);
+    // Windows npm is a .cmd shim, which cannot be spawned without a shell.
+    if (npm && process.platform !== 'win32') {
+        const prefix = npm[1];
+        const binary = find('npm', [path.join(prefix, 'bin')]);
+        if (binary)
+            return { manager: 'npm', command: binary, args: ['install', '-g', '--prefix', prefix, '@openai/codex@latest'] };
+    }
+    const brew = resolved.match(/^(.*?)\/(Cellar|Caskroom)\/codex\//u);
+    if (brew) {
+        const binary = find('brew', [path.join(brew[1], 'bin')]);
+        if (binary)
+            return {
+                manager: 'brew',
+                command: binary,
+                args: ['upgrade', ...(brew[2] === 'Caskroom' ? ['--cask'] : []), 'codex'],
+            };
+    }
+    return { manager: 'codex', command, args: ['update'] };
+}
 export function createCodexUpdateManager(dependencies = {}) {
     const resolveCommand = dependencies.resolveCommand || (() => resolveHarnessCommand('codex'));
     const isAppManaged = dependencies.isAppManaged || codexCommandIsAppManaged;
     const run = dependencies.run || runCommand;
+    const upgradeCommand = dependencies.upgradeCommand || ((command) => codexUpgradeCommand(command));
     let active = null;
     const updateCommand = () => {
         const command = resolveCommand();
@@ -74,9 +123,12 @@ export function createCodexUpdateManager(dependencies = {}) {
             throw commandFailure(beforeRun, 'Codex is not installed or its version could not be read.');
         }
         const previousVersion = parseCodexVersion(beforeRun.output);
-        const updateRun = await run(updateCommand(), ['update'], UPDATE_TIMEOUT_MS);
+        const upgrade = upgradeCommand(updateCommand());
+        const updateRun = await run(upgrade.command, upgrade.args, UPDATE_TIMEOUT_MS);
         if (updateRun.failed || updateRun.timedOut || updateRun.exitCode !== 0) {
-            throw commandFailure(updateRun, 'Codex could not update itself. Run `codex update` in a terminal on the machine.');
+            throw commandFailure(updateRun, upgrade.manager === 'codex'
+                ? 'Codex could not update itself. Run `codex update` in a terminal on the machine.'
+                : `${upgrade.manager} could not upgrade Codex. Run \`${[path.basename(upgrade.command), ...upgrade.args].join(' ')}\` in a terminal on the machine.`);
         }
         // Resolve again because the updater may replace a shim or install path.
         const afterRun = await run(updateCommand(), ['--version'], VERSION_TIMEOUT_MS);

package/dist/harness-exit.js

@@ -0,0 +1,44 @@
+/** Failure details for a harness process that exits underneath Turns. */
+const STDERR_TAIL_LINES = 12;
+const STDERR_TAIL_CHARS = 4_000;
+/** The last few stderr lines of a harness process. When the process exits on
+ * its own, they are usually the only explanation (for example, an outdated CLI
+ * rejecting an argument), and the turn's failure must carry them. */
+export class StderrTail {
+    recent = [];
+    observe(line) {
+        const text = String(line)
+            .replace(/\u001b\[[0-9;?]*[A-Za-z]/gu, '')
+            .trimEnd();
+        if (!text.trim())
+            return;
+        this.recent.push(text.slice(0, STDERR_TAIL_CHARS));
+        if (this.recent.length > STDERR_TAIL_LINES)
+            this.recent.shift();
+    }
+    lines() {
+        const kept = [];
+        let size = 0;
+        for (const line of [...this.recent].reverse()) {
+            size += line.length + 1;
+            if (size > STDERR_TAIL_CHARS)
+                break;
+            kept.unshift(line);
+        }
+        return kept;
+    }
+}
+/** A harness process that ended underneath Turns rather than reporting a
+ * failure through its own protocol. */
+export function harnessExitError(harness, code, signal, stderr) {
+    const label = harnessLabel(harness);
+    const how = signal ? `was stopped by ${signal}` : `exited with code ${code}`;
+    const message = [`${label} ${how} before the turn finished.`, ...(stderr.length ? ['', ...stderr] : [])].join('\n');
+    return Object.assign(new Error(message), {
+        harnessExit: { harness: String(harness), code: code ?? null, signal: signal ?? null, stderr },
+    });
+}
+function harnessLabel(harness) {
+    const name = String(harness || 'agent');
+    return name.charAt(0).toUpperCase() + name.slice(1);
+}

package/dist/index.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 import fs from 'node:fs';
+import os from 'node:os';
 import path from 'node:path';
 import process from 'node:process';
 import { fileURLToPath } from 'node:url';
@@ -225,6 +226,22 @@ async function runUpload(credentials, { paths, watch = false, all = false, inter
         provider?.close();
     }
 }
+/** First lines of `turns daemon run`, printed before authentication so a
+ * report pasted from a terminal or log always says which build and runtime
+ * produced it (`npx` can silently run an older cached release). */
+export function daemonStartupBanner({ version, nodeVersion = process.version, platform = process.platform, arch = process.arch, osName = `${os.type()} ${os.release()}`, hostname = os.hostname(), pid = process.pid, packageDirectory = CLI_DIRECTORY, paths, allowedDirectories = [], }) {
+    return [
+        `Turns daemon ${version}`,
+        `  Node ${nodeVersion} on ${platform}-${arch} (${osName}), host ${hostname}, pid ${pid}`,
+        `  Package ${packageDirectory}`,
+        `  Config ${paths.configDirectory}`,
+        `  State ${paths.stateDirectory}`,
+        allowedDirectories.length
+            ? `  Directories:\n${allowedDirectories.map((value) => `    ${value}`).join('\n')}`
+            : '  Directories: all',
+        '',
+    ].join('\n');
+}
 async function main(rawArguments = process.argv.slice(2)) {
     const parsed = parseGlobalArguments(rawArguments);
     const argv = parsed.argv;
@@ -274,6 +291,7 @@ async function main(rawArguments = process.argv.slice(2)) {
             throw new Error('--once cannot be combined with `turns daemon run`.');
         if (parsed.devUser && !parsed.origin)
             throw new Error('--dev-user requires an explicit --server URL.');
+        logInfo(daemonStartupBanner({ version: pkg.version, paths, allowedDirectories }));
         let credentials;
         if (parsed.devUser) {
             credentials = { origin: normalizeOrigin(parsed.origin) };
@@ -282,9 +300,6 @@ async function main(rawArguments = process.argv.slice(2)) {
             await migrateLegacyCredentials(paths, parsed.origin);
             credentials = await authenticate({ origin: parsed.origin, paths, interactive: false });
         }
-        if (allowedDirectories.length) {
-            logInfo(`Daemon directory allowlist:\n${allowedDirectories.map((value) => `  ${value}`).join('\n')}\n`);
-        }
         await runUpload(credentials, {
             paths,
             watch: true,

package/dist/queue-worker.js

@@ -301,6 +301,28 @@ export function runtimeEvent(anchorName, messageId, event, redactor = DEFAULT_RE
             value: redactor.redactJsonLine(JSON.stringify(event)),
         };
     }
+    // A harness that exits underneath its turn often reports the reason only on
+    // stderr, before any protocol frame the reducer could show. Record that
+    // ending explicitly so the transcript can explain the failed turn.
+    if (event?.event === 'turn.complete' && event.result?.error?.harnessExit && !event.result.interrupted) {
+        const exit = event.result.error.harnessExit;
+        return {
+            eventKey: key,
+            kind: 'session.harness_exit',
+            phase: 'queued',
+            ts,
+            source: 'agent-anchor',
+            value: redactor.redactJsonLine(JSON.stringify({
+                type: 'turns.session.harness_exit',
+                messageId: String(event.messageId || messageId || ''),
+                harness: String(exit.harness || event.agent || ''),
+                code: typeof exit.code === 'number' ? exit.code : null,
+                signal: exit.signal ? String(exit.signal) : null,
+                message: String(event.result.error.message || ''),
+                stderr: Array.isArray(exit.stderr) ? exit.stderr.map(String) : [],
+            })),
+        };
+    }
     if (event?.event === 'turn.steer') {
         return {
             eventKey: key,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@turns/dev",
-  "version": "0.0.29",
+  "version": "0.0.30",
   "description": "Upload local AI agent sessions to turns.dev",
   "license": "MIT",
   "type": "module",