@blockrun/franklin 3.15.19 → 3.15.21

package/dist/commands/logs.js

@@ -3,17 +3,25 @@ import path from 'node:path';
 import chalk from 'chalk';
 import { BLOCKRUN_DIR } from '../config.js';
 const LOG_FILE = path.join(BLOCKRUN_DIR, 'franklin-debug.log');
+const ARCHIVE_LOG_FILE = path.join(BLOCKRUN_DIR, 'franklin-debug.log.1');
 const LEGACY_LOG_FILE = path.join(BLOCKRUN_DIR, 'runcode-debug.log');
-const MAX_LOG_SIZE = 10 * 1024 * 1024; // 10MB auto-rotate threshold
 export function logsCommand(options) {
     if (options.clear) {
+        let cleared = false;
         try {
             fs.unlinkSync(LOG_FILE);
-            console.log(chalk.green('Logs cleared.'));
+            cleared = true;
         }
-        catch {
-            console.log(chalk.dim('No log file to clear.'));
+        catch { /* may not exist */ }
+        try {
+            fs.unlinkSync(ARCHIVE_LOG_FILE);
+            cleared = true;
         }
+        catch { /* may not exist */ }
+        if (cleared)
+            console.log(chalk.green('Logs cleared.'));
+        else
+            console.log(chalk.dim('No log file to clear.'));
         return;
     }
     // Migrate legacy log file
@@ -23,23 +31,17 @@ export function logsCommand(options) {
         }
         catch { /* best effort */ }
     }
-    if (!fs.existsSync(LOG_FILE)) {
+    // Logger now self-rotates on write (in src/logger.ts). The previous
+    // in-place "slice off the first half" rotation here was destructive
+    // — every invocation that crossed 10 MB silently dropped half the
+    // history. With self-rotation in place this command no longer needs
+    // to mutate the file at all; it just stitches the archive + live
+    // log for display.
+    if (!fs.existsSync(LOG_FILE) && !fs.existsSync(ARCHIVE_LOG_FILE)) {
         console.log(chalk.dim('No logs yet. Start franklin with --debug to enable logging:'));
         console.log(chalk.bold('  franklin start --debug'));
         return;
     }
-    // Auto-rotate: if file is over threshold, keep only last half
-    try {
-        const stat = fs.statSync(LOG_FILE);
-        if (stat.size > MAX_LOG_SIZE) {
-            const content = fs.readFileSync(LOG_FILE, 'utf-8');
-            const lines = content.split('\n');
-            const half = lines.slice(Math.floor(lines.length / 2));
-            fs.writeFileSync(LOG_FILE, half.join('\n'));
-            console.log(chalk.dim(`(Rotated log — was ${(stat.size / 1024 / 1024).toFixed(1)}MB)`));
-        }
-    }
-    catch { /* ignore rotation errors */ }
     const parsed = parseInt(options.lines || '50', 10);
     const tailLines = isNaN(parsed) ? 50 : Math.max(1, Math.min(10000, parsed));
     if (options.follow) {
@@ -78,8 +80,15 @@ export function logsCommand(options) {
 }
 function printLastLines(n) {
     try {
-        const content = fs.readFileSync(LOG_FILE, 'utf-8');
-        const lines = content.split('\n').filter(Boolean);
+        // Logger self-rotates to franklin-debug.log.1 when the live log
+        // crosses 10MB. Stitch the archive on first so requests for "last N"
+        // can span the rotation boundary — without this, immediately after
+        // a rotation `franklin logs --lines 1000` would show only whatever
+        // lines have been written since rotation, even though the archive
+        // is sitting right next to it.
+        const archive = fs.existsSync(ARCHIVE_LOG_FILE) ? fs.readFileSync(ARCHIVE_LOG_FILE, 'utf-8') : '';
+        const live = fs.existsSync(LOG_FILE) ? fs.readFileSync(LOG_FILE, 'utf-8') : '';
+        const lines = (archive + live).split('\n').filter(Boolean);
         const start = Math.max(0, lines.length - n);
         const slice = lines.slice(start);
         if (start > 0) {

package/dist/logger.js

@@ -14,6 +14,20 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { BLOCKRUN_DIR } from './config.js';
 const LOG_FILE = path.join(BLOCKRUN_DIR, 'franklin-debug.log');
+const ARCHIVE_FILE = path.join(BLOCKRUN_DIR, 'franklin-debug.log.1');
+// Self-rotation threshold. When the live log crosses this size on a
+// write, rename it to franklin-debug.log.1 (overwriting any previous
+// archive) and start fresh. Non-destructive: one full archive of the
+// most recent ROTATE_AT_BYTES is always retained, so users can still
+// read history across the rotation. Earlier behavior (only triggered
+// by `franklin logs`, sliced the file in half in-place) lost history
+// outright and only ran if the user happened to invoke `franklin logs`.
+const ROTATE_AT_BYTES = 10 * 1024 * 1024; // 10 MB
+// Probe every N writes to amortize the stat() — average debug entry is
+// ~80 bytes, so 1000 writes (~80 KB worth) between checks keeps the
+// overhead negligible while still catching a runaway log within seconds.
+const ROTATE_PROBE_EVERY_N_WRITES = 1000;
+let writesSinceRotateProbe = 0;
 // Strip ANSI escapes + carriage returns so the log stays grep-able.
 const ANSI_RE = /\x1b\[[0-9;]*m|\x1b\][^\x07]*\x07|\r/g;
 let debugMode = false;
@@ -36,9 +50,42 @@ function ensureDir() {
     }
     catch { /* readonly mount / disk full — keep trying so a remount recovers */ }
 }
+function maybeRotate() {
+    try {
+        if (!fs.existsSync(LOG_FILE))
+            return;
+        const { size } = fs.statSync(LOG_FILE);
+        if (size < ROTATE_AT_BYTES)
+            return;
+        // renameSync overwrites an existing target on POSIX, which is what
+        // we want — single archive, always the most recent rotation. On
+        // Windows the rename can EEXIST; in that case unlink the archive
+        // first and retry. If even that fails, fall through silently rather
+        // than leaving the log file in a half-rotated state.
+        try {
+            fs.renameSync(LOG_FILE, ARCHIVE_FILE);
+        }
+        catch {
+            try {
+                fs.unlinkSync(ARCHIVE_FILE);
+            }
+            catch { /* may not exist */ }
+            try {
+                fs.renameSync(LOG_FILE, ARCHIVE_FILE);
+            }
+            catch { /* give up */ }
+        }
+    }
+    catch { /* best effort */ }
+}
 function writeFile(level, msg) {
     ensureDir();
     try {
+        writesSinceRotateProbe++;
+        if (writesSinceRotateProbe >= ROTATE_PROBE_EVERY_N_WRITES) {
+            writesSinceRotateProbe = 0;
+            maybeRotate();
+        }
         const clean = msg.replace(ANSI_RE, '');
         fs.appendFileSync(LOG_FILE, `[${new Date().toISOString()}] [${level.toUpperCase()}] ${clean}\n`);
     }

package/dist/proxy/fallback.js

@@ -2,19 +2,7 @@
  * Fallback chain for Franklin
  * Automatically switches to backup models when primary fails (429, 5xx, etc.)
  */
-import fs from 'node:fs';
-import os from 'node:os';
-import path from 'node:path';
-const LOG_FILE = path.join(os.homedir(), '.blockrun', 'franklin-debug.log');
-// eslint-disable-next-line no-control-regex
-const ANSI_RE = /\x1B\[[0-9;]*[A-Za-z]|\x1B\][^\x07]*\x07|\x1B[()][A-B]|\r/g;
-function appendLog(msg) {
-    try {
-        fs.mkdirSync(path.dirname(LOG_FILE), { recursive: true });
-        fs.appendFileSync(LOG_FILE, `[${new Date().toISOString()}] ${msg.replace(ANSI_RE, '')}\n`);
-    }
-    catch { /* ignore */ }
-}
+import { logger } from '../logger.js';
 export const DEFAULT_FALLBACK_CONFIG = {
     chain: [
         'deepseek/deepseek-chat', // Direct fallback — cheap & reliable
@@ -93,7 +81,7 @@ export async function fetchWithFallback(url, init, originalBody, config = DEFAUL
             if (nextModel && onFallback) {
                 const errMsg = err instanceof Error ? err.message : 'Network error';
                 onFallback(model, 0, nextModel);
-                appendLog(`[franklin] [fallback] ${model} network error: ${errMsg}`);
+                logger.warn(`[franklin] [fallback] ${model} network error: ${errMsg}`);
             }
             if (i < config.chain.length - 1) {
                 await sleep(config.retryDelayMs);

package/dist/proxy/server.js

@@ -1,7 +1,4 @@
 import http from 'node:http';
-import fs from 'node:fs';
-import path from 'node:path';
-import os from 'node:os';
 import { getOrCreateWallet, getOrCreateSolanaWallet, createPaymentPayload, createSolanaPaymentPayload, parsePaymentRequired, extractPaymentDetails, solanaKeyToBytes, SOLANA_NETWORK, } from '@blockrun/llm';
 import { recordUsage } from '../stats/tracker.js';
 import { appendAudit } from '../stats/audit.js';
@@ -12,34 +9,13 @@ import { VERSION } from '../config.js';
 // User-Agent for backend requests
 const USER_AGENT = `franklin/${VERSION}`;
 const X_FRANKLIN_VERSION = VERSION;
-const LOG_FILE = path.join(os.homedir(), '.blockrun', 'franklin-debug.log');
-// Strip ANSI escape codes so log file doesn't distort terminal on replay
-function stripAnsi(str) {
-    // eslint-disable-next-line no-control-regex
-    return str.replace(/\x1B\[[0-9;]*[A-Za-z]|\x1B\][^\x07]*\x07|\x1B[()][A-B]|\r/g, '');
-}
-function debug(options, ...args) {
-    if (!options.debug)
-        return;
-    const msg = `[${new Date().toISOString()}] ${stripAnsi(args.map(String).join(' '))}\n`;
-    try {
-        fs.mkdirSync(path.dirname(LOG_FILE), { recursive: true });
-        fs.appendFileSync(LOG_FILE, msg);
-    }
-    catch {
-        /* ignore */
-    }
-}
-function log(...args) {
-    const msg = `[franklin] ${args.map(String).join(' ')}`;
-    // Do NOT print to stdout — the terminal is owned by the parent process (stdio: inherit).
-    // Use `franklin logs` to read runtime messages.
-    try {
-        fs.mkdirSync(path.dirname(LOG_FILE), { recursive: true });
-        fs.appendFileSync(LOG_FILE, `[${new Date().toISOString()}] ${stripAnsi(msg)}\n`);
-    }
-    catch { /* ignore */ }
-}
+// Logging here goes through the unified logger introduced in 3.15.11
+// (timestamp + [LEVEL] tag, self-rotating at 10 MB, optional stderr
+// mirror in debug mode). The previous per-module debug()/log() helpers
+// duplicated the file path, ANSI strip regex (with a slightly different
+// pattern!), and timestamp format — bug fixes never propagated. They
+// were the last holdouts after the agent loop was migrated.
+import { logger, setDebugMode } from '../logger.js';
 const DEFAULT_MAX_TOKENS = 4096;
 // 180s budget for *time-to-headers* — reasoning-class models (zai/glm-*,
 // nemotron *-reasoning, deepseek-r*, gpt-5-codex, anthropic extended-thinking)
@@ -249,6 +225,10 @@ function withinRateLimit() {
     return true;
 }
 export function createProxy(options) {
+    // Wire stderr-mirroring of unified logger output to the proxy's debug
+    // flag — same pattern as interactiveSession in agent/loop. File writes
+    // happen regardless; only the live stderr mirror is gated.
+    setDebugMode(!!options.debug);
     const chain = options.chain || 'base';
     let currentModel = options.modelOverride || DEFAULT_MODEL;
     const fallbackEnabled = options.fallbackEnabled !== false; // Default true
@@ -307,19 +287,22 @@ export function createProxy(options) {
             let requestModel = currentModel || options.modelOverride || 'unknown';
             let usedFallback = false;
             try {
-                debug(options, `request: ${req.method} ${req.url} currentModel=${currentModel || 'none'}`);
+                if (options.debug)
+                    logger.debug(`[franklin] request: ${req.method} ${req.url} currentModel=${currentModel || 'none'}`);
                 if (body) {
                     try {
                         const parsed = JSON.parse(body);
                         // Intercept "use <model>" commands for in-session model switching
                         if (parsed.messages) {
                             const last = parsed.messages[parsed.messages.length - 1];
-                            debug(options, `last msg role=${last?.role} content-type=${typeof last?.content} content=${JSON.stringify(last?.content).slice(0, 200)}`);
+                            if (options.debug)
+                                logger.debug(`[franklin] last msg role=${last?.role} content-type=${typeof last?.content} content=${JSON.stringify(last?.content).slice(0, 200)}`);
                         }
                         const switchCmd = detectModelSwitch(parsed);
                         if (switchCmd) {
                             currentModel = switchCmd;
-                            debug(options, `model switched to: ${currentModel}`);
+                            if (options.debug)
+                                logger.debug(`[franklin] model switched to: ${currentModel}`);
                             const fakeResponse = {
                                 id: `msg_franklin_${Date.now()}`,
                                 type: 'message',
@@ -374,7 +357,7 @@ export function createProxy(options) {
                             const routing = routeRequest(promptText, routingProfile);
                             parsed.model = routing.model;
                             requestModel = routing.model;
-                            log(`🧠 Smart routing: ${routingProfile} → ${routing.tier} → ${routing.model} ` +
+                            logger.info(`[franklin] 🧠 Smart routing: ${routingProfile} → ${routing.tier} → ${routing.model} ` +
                                 `(${(routing.savings * 100).toFixed(0)}% savings) [${routing.signals.join(', ')}]`);
                         }
                         {
@@ -392,8 +375,8 @@ export function createProxy(options) {
                                 ? Math.max(lastOut * 2, DEFAULT_MAX_TOKENS)
                                 : DEFAULT_MAX_TOKENS;
                             parsed.max_tokens = Math.min(adaptive, modelCap);
-                            if (original !== parsed.max_tokens) {
-                                debug(options, `max_tokens: ${original || 'unset'} → ${parsed.max_tokens} (last output: ${lastOut || 'none'})`);
+                            if (original !== parsed.max_tokens && options.debug) {
+                                logger.debug(`[franklin] max_tokens: ${original || 'unset'} → ${parsed.max_tokens} (last output: ${lastOut || 'none'})`);
                             }
                         }
                         body = JSON.stringify(parsed);
@@ -430,7 +413,7 @@ export function createProxy(options) {
                             body = JSON.stringify(parsed);
                         }
                         catch { /* body not JSON, skip */ }
-                        log(`⚠️  Safety net: resolved unrouted ${virtualName} → ${requestModel}`);
+                        logger.warn(`[franklin] ⚠️  Safety net: resolved unrouted ${virtualName} → ${requestModel}`);
                     }
                 }
                 // Build request init
@@ -456,7 +439,7 @@ export function createProxy(options) {
                         solanaWallet,
                         timeoutMs: requestTimeoutMs,
                     }, (failedModel, status, nextModel) => {
-                        log(`⚠️  ${failedModel} returned ${status}, falling back to ${nextModel}`);
+                        logger.warn(`[franklin] ⚠️  ${failedModel} returned ${status}, falling back to ${nextModel}`);
                     });
                     response = result.response;
                     finalModel = result.modelUsed;
@@ -464,7 +447,7 @@ export function createProxy(options) {
                     body = result.bodyUsed;
                     usedFallback = result.fallbackUsed;
                     if (usedFallback) {
-                        log(`↺ Fallback successful: using ${finalModel}`);
+                        logger.info(`[franklin] ↺ Fallback successful: using ${finalModel}`);
                     }
                 }
                 else {
@@ -516,7 +499,7 @@ export function createProxy(options) {
                     }
                     res.writeHead(response.status, { 'Content-Type': 'application/json' });
                     res.end(errorBody);
-                    log(`⚠️  ${response.status} from backend for ${finalModel}`);
+                    logger.warn(`[franklin] ⚠️  ${response.status} from backend for ${finalModel}`);
                     return;
                 }
                 res.writeHead(response.status, responseHeaders);
@@ -531,7 +514,7 @@ export function createProxy(options) {
                     const pump = async () => {
                         while (true) {
                             if (Date.now() > streamDeadline) {
-                                log('⚠️  Stream timeout after 5 minutes');
+                                logger.warn('[franklin] ⚠️  Stream timeout after 5 minutes');
                                 try {
                                     reader.cancel();
                                 }
@@ -576,7 +559,8 @@ export function createProxy(options) {
                                             fallback: usedFallback,
                                             source: 'proxy',
                                         });
-                                        debug(options, `recorded: model=${finalModel} in=${inputTokens} out=${outputTokens} cost=$${cost.toFixed(4)} fallback=${usedFallback}`);
+                                        if (options.debug)
+                                            logger.debug(`[franklin] recorded: model=${finalModel} in=${inputTokens} out=${outputTokens} cost=$${cost.toFixed(4)} fallback=${usedFallback}`);
                                     }
                                 }
                                 res.end();
@@ -590,7 +574,7 @@ export function createProxy(options) {
                         }
                     };
                     pump().catch((err) => {
-                        log(`❌ Stream error: ${err instanceof Error ? err.message : String(err)}`);
+                        logger.error(`[franklin] ❌ Stream error: ${err instanceof Error ? err.message : String(err)}`);
                         res.end();
                     });
                 }
@@ -615,7 +599,8 @@ export function createProxy(options) {
                                 fallback: usedFallback,
                                 source: 'proxy',
                             });
-                            debug(options, `recorded: model=${finalModel} in=${inputTokens} out=${outputTokens} cost=$${cost.toFixed(4)} fallback=${usedFallback}`);
+                            if (options.debug)
+                                logger.debug(`[franklin] recorded: model=${finalModel} in=${inputTokens} out=${outputTokens} cost=$${cost.toFixed(4)} fallback=${usedFallback}`);
                         }
                     }
                     catch {
@@ -626,7 +611,7 @@ export function createProxy(options) {
             }
             catch (error) {
                 const msg = error instanceof Error ? error.message : 'Proxy error';
-                log(`❌ Error: ${msg}`);
+                logger.error(`[franklin] ❌ Error: ${msg}`);
                 res.writeHead(502, { 'Content-Type': 'application/json' });
                 res.end(JSON.stringify({
                     type: 'error',
@@ -693,7 +678,7 @@ async function fetchWithPaymentFallback(url, init, originalBody, config, payment
             if (nextModel && onFallback) {
                 onFallback(model, 0, nextModel);
             }
-            log(`[fallback] ${model} request error: ${err instanceof Error ? err.message : String(err)}`);
+            logger.warn(`[franklin] [fallback] ${model} request error: ${err instanceof Error ? err.message : String(err)}`);
             if (i < config.chain.length - 1) {
                 await sleep(config.retryDelayMs);
             }

package/package.json

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