@blockrun/franklin 3.15.31 → 3.15.33

package/dist/agent/streaming-executor.js

@@ -7,6 +7,7 @@ import { mkdirSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { recordFailure } from '../stats/failures.js';
 import { BLOCKRUN_DIR } from '../config.js';
+import { logger } from '../logger.js';
 /** Persist a large tool result to disk and return a preview string. */
 const PERSIST_THRESHOLD = 50_000;
 const PREVIEW_SIZE = 2_000;
@@ -205,8 +206,22 @@ export class StreamingExecutor {
                     }
                 }
             }
+            // Track elapsed for slow-tool forensics. Verified 2026-05-04
+            // from a real session: a 337.6s Bash error left no trace in
+            // franklin-debug.log — the user could see the ✗ in the UI but
+            // had no post-hoc way to ask "which Bash took 5+ minutes
+            // yesterday and what did it run". 30s threshold is conservative
+            // (Read/Glob/Grep finish in <1s; only network or shell work
+            // crosses).
+            const execStart = Date.now();
             let result = await handler.execute(invocation.input, progressScope);
             this.guard?.afterExecute(invocation, result);
+            const execElapsed = Date.now() - execStart;
+            if (execElapsed >= 30_000) {
+                const status = result.isError ? 'error' : 'ok';
+                const preview = this.inputPreview(invocation) || '';
+                logger.info(`[franklin] Slow tool: ${invocation.name} ${status} after ${(execElapsed / 1000).toFixed(1)}s${preview ? ` — ${preview.slice(0, 80)}` : ''}`);
+            }
             // Persist large results to disk with preview.
             // Instead of just truncating, save the full result to disk so it can be re-read later.
             if (result.output.length > PERSIST_THRESHOLD) {
@@ -219,15 +234,22 @@ export class StreamingExecutor {
         }
         catch (err) {
             this.guard?.cancelInvocation(invocation.id);
+            const errMsg = err.message;
             recordFailure({
                 timestamp: Date.now(),
                 model: '', // not available at tool level
                 failureType: 'tool_error',
                 toolName: invocation.name,
-                errorMessage: err.message,
+                errorMessage: errMsg,
             });
+            // Always log thrown tool errors. Pre-3.15.32 these went only to
+            // failures.jsonl (which the user rarely opens) and were absent
+            // from franklin-debug.log entirely. Now `franklin logs` shows
+            // them alongside everything else.
+            const preview = this.inputPreview(invocation) || '';
+            logger.warn(`[franklin] Tool error: ${invocation.name} threw "${errMsg.slice(0, 120)}"${preview ? ` — ${preview.slice(0, 80)}` : ''}`);
             return {
-                output: `Error executing ${invocation.name}: ${err.message}`,
+                output: `Error executing ${invocation.name}: ${errMsg}`,
                 isError: true,
             };
         }

package/dist/tasks/spawn.d.ts

@@ -16,8 +16,13 @@
  *
  * CLI path resolution (in priority order):
  *   1. process.env.FRANKLIN_CLI_PATH — escape hatch for tests / dev.
- *   2. process.argv[1] — the script Node is currently executing, i.e. the
- *      running franklin bundle. Works regardless of the user's cwd.
+ *   2. STARTUP_CLI_PATH (captured at module load) — absolute path of
+ *      the script Node is currently executing. Captured early so it
+ *      survives any later chdir; resolved to absolute so it survives
+ *      the spawn's `cwd:` override (the bug it fixes — verified
+ *      2026-05-04 from a real session: dev-mode `node dist/index.js`
+ *      run, then Detach with workingDir=other-repo, child fails with
+ *      MODULE_NOT_FOUND on `<other-repo>/dist/index.js`).
  */
 export interface StartDetachedTaskInput {
     label: string;

package/dist/tasks/spawn.js

@@ -16,22 +16,46 @@
  *
  * CLI path resolution (in priority order):
  *   1. process.env.FRANKLIN_CLI_PATH — escape hatch for tests / dev.
- *   2. process.argv[1] — the script Node is currently executing, i.e. the
- *      running franklin bundle. Works regardless of the user's cwd.
+ *   2. STARTUP_CLI_PATH (captured at module load) — absolute path of
+ *      the script Node is currently executing. Captured early so it
+ *      survives any later chdir; resolved to absolute so it survives
+ *      the spawn's `cwd:` override (the bug it fixes — verified
+ *      2026-05-04 from a real session: dev-mode `node dist/index.js`
+ *      run, then Detach with workingDir=other-repo, child fails with
+ *      MODULE_NOT_FOUND on `<other-repo>/dist/index.js`).
  */
 import { spawn } from 'node:child_process';
 import fs from 'node:fs';
+import path from 'node:path';
 import { randomUUID } from 'node:crypto';
 import { writeTaskMeta } from './store.js';
 import { taskLogPath, ensureTaskDir } from './paths.js';
+// Captured at module load so it survives later chdir / argv mutation.
+// `process.argv[1]` may be relative (`dist/index.js` in dev mode); we
+// resolve against process.cwd() at startup which is when the user's
+// shell exec'd the bundle. Doing this at call time would be wrong if
+// any code chdir'd between startup and the Detach call.
+const STARTUP_CLI_PATH = (() => {
+    const argv1 = process.argv[1];
+    if (!argv1)
+        return undefined;
+    try {
+        return path.resolve(argv1);
+    }
+    catch {
+        return argv1;
+    }
+})();
 function resolveCliPath() {
     const fromEnv = process.env.FRANKLIN_CLI_PATH;
     if (fromEnv && fromEnv.length > 0)
         return fromEnv;
-    // Resolving from process.cwd() breaks whenever Franklin is launched outside
-    // the source tree (npm global install, brew, or just `cd /elsewhere &&
-    // franklin`). process.argv[1] is the actual entry script Node loaded.
-    return process.argv[1];
+    // STARTUP_CLI_PATH is the absolute path resolved at module load —
+    // safe to use after `cwd: input.workingDir` redirects the child.
+    // npm global installs already give an absolute path; this only
+    // matters in dev mode where `node dist/index.js` puts a relative
+    // path into argv[1].
+    return STARTUP_CLI_PATH || process.argv[1];
 }
 function generateRunId() {
     return `t_${Date.now().toString(36)}_${randomUUID().slice(0, 8)}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blockrun/franklin",
-  "version": "3.15.31",
+  "version": "3.15.33",
   "description": "Franklin — The AI agent with a wallet. Spends USDC autonomously to get real work done. Pay per action, no subscriptions.",
   "type": "module",
   "exports": {