@blockrun/franklin 3.6.21 → 3.6.23

package/dist/agent/error-classifier.d.ts

@@ -6,10 +6,10 @@
  * - Auth errors (401) get special handling (token refresh, not retry)
  * - EPIPE/connection reset handled as network errors (retryable)
  */
-export type AgentErrorCategory = 'rate_limit' | 'payment' | 'network' | 'timeout' | 'context_limit' | 'overloaded' | 'server' | 'auth' | 'unknown';
+export type AgentErrorCategory = 'rate_limit' | 'payment' | 'network' | 'timeout' | 'context_limit' | 'overloaded' | 'server' | 'auth' | 'schema' | 'unknown';
 export interface AgentErrorInfo {
     category: AgentErrorCategory;
-    label: 'RateLimit' | 'Payment' | 'Network' | 'Timeout' | 'Context' | 'Overloaded' | 'Server' | 'Auth' | 'Unknown';
+    label: 'RateLimit' | 'Payment' | 'Network' | 'Timeout' | 'Context' | 'Overloaded' | 'Server' | 'Auth' | 'Schema' | 'Unknown';
     isTransient: boolean;
     /** Max retries for this error type (overrides default). undefined = use default. */
     maxRetries?: number;

package/dist/agent/error-classifier.js

@@ -103,6 +103,24 @@ export function classifyAgentError(message) {
             suggestion: 'The model is overloaded. Try /model to switch, or wait and /retry.',
         };
     }
+    // Schema / tool-definition errors — NOT transient, retrying won't help.
+    // These can be wrapped in 5xx responses (e.g. '503: 400 Invalid schema'),
+    // so classify them BEFORE the generic server-error branch below.
+    if (includesAny(err, [
+        'invalid schema',
+        'array schema missing items',
+        'schema missing',
+        'invalid tool_use',
+        'invalid function',
+        'tool_use_id',
+        'unsupported parameter',
+        'invalid request',
+    ])) {
+        return {
+            category: 'schema', label: 'Schema', isTransient: false, maxRetries: 0,
+            suggestion: 'Tool schema rejected by this model. Try /model to switch to a more permissive model (e.g. sonnet), or upgrade Franklin.',
+        };
+    }
     if (includesAny(err, [
         '500',
         '502',

package/dist/mcp/client.js

@@ -7,6 +7,40 @@ import { Client } from '@modelcontextprotocol/sdk/client/index.js';
 import { StdioClientTransport } from '@modelcontextprotocol/sdk/client/stdio.js';
 // ─── Connection Management ────────────────────────────────────────────────
 const connections = new Map();
+/**
+ * Sanitize a JSON schema for strict LLM providers (OpenAI o3, etc.).
+ * Walks the schema tree and adds `items` to any array missing it.
+ * Without this, models like o3 reject the tool with:
+ *   "Invalid schema: In context=(...), array schema missing items."
+ */
+function sanitizeSchema(schema) {
+    if (!schema || typeof schema !== 'object') {
+        return { type: 'object', properties: {} };
+    }
+    const s = schema;
+    // If array type without items, add a permissive default
+    if (s.type === 'array' && !s.items) {
+        s.items = {};
+    }
+    // Recurse into properties
+    if (s.properties && typeof s.properties === 'object') {
+        const props = s.properties;
+        for (const key of Object.keys(props)) {
+            props[key] = sanitizeSchema(props[key]);
+        }
+    }
+    // Recurse into items (nested arrays)
+    if (s.items && typeof s.items === 'object') {
+        s.items = sanitizeSchema(s.items);
+    }
+    // Recurse into anyOf / oneOf / allOf
+    for (const key of ['anyOf', 'oneOf', 'allOf']) {
+        if (Array.isArray(s[key])) {
+            s[key] = s[key].map(sanitizeSchema);
+        }
+    }
+    return s;
+}
 /**
  * Connect to an MCP server via stdio transport.
  * Discovers tools and returns them as CapabilityHandlers.
@@ -47,10 +81,7 @@ async function connectStdio(name, config) {
             spec: {
                 name: toolName,
                 description: toolDescription || `MCP tool from ${name}`,
-                input_schema: tool.inputSchema || {
-                    type: 'object',
-                    properties: {},
-                },
+                input_schema: sanitizeSchema(tool.inputSchema),
             },
             execute: async (input, _ctx) => {
                 const MCP_TOOL_TIMEOUT = 30_000;

package/dist/tools/task.js

@@ -130,10 +130,12 @@ export const taskCapability = {
                 },
                 addBlocks: {
                     type: 'array',
+                    items: { type: 'number' },
                     description: 'Task IDs that cannot start until this task completes (for update)',
                 },
                 addBlockedBy: {
                     type: 'array',
+                    items: { type: 'number' },
                     description: 'Task IDs that must complete before this task can start (for update)',
                 },
             },

package/dist/ui/app.js

@@ -657,7 +657,7 @@ function RunCodeApp({ initialModel, workDir, walletAddress, walletBalance, chain
                 const elapsed = Math.round((Date.now() - tool.startTime) / 1000);
                 const elapsedStr = elapsed > 0 ? ` ${elapsed}s` : '';
                 return (_jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Text, { children: [_jsx(Text, { color: "cyan", children: _jsx(Spinner, { type: "dots" }) }), ' ', _jsx(Text, { bold: true, color: "cyan", children: tool.name }), tool.preview ? _jsxs(Text, { dimColor: true, children: ["(", tool.preview.slice(0, 70), ")"] }) : null, _jsx(Text, { dimColor: true, children: elapsedStr })] }), tool.liveLines.length > 0 && (_jsx(Box, { flexDirection: "column", marginLeft: 2, children: tool.liveLines.map((line, i) => (_jsxs(Text, { dimColor: true, wrap: "truncate-end", children: ['⎿  ', line.slice(0, 120)] }, i))) }))] }, id));
-            }), thinking && (_jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Text, { color: "magenta", children: [_jsx(Spinner, { type: "dots" }), ' ', _jsx(Text, { bold: true, children: "thinking" }), completedTools.length > 0 ? _jsxs(Text, { dimColor: true, children: [' ', "\u00B7 step ", completedTools.length + 1] }) : null] }), thinkingText && (() => {
+            }), thinking && (_jsxs(Box, { flexDirection: "column", marginLeft: 2, children: [_jsxs(Text, { color: "magenta", children: [_jsx(Spinner, { type: "dots" }), ' ', _jsx(Text, { bold: true, children: "thinking" }), completedTools.length > 0 ? _jsxs(Text, { dimColor: true, children: [' ', "\u00B7 step ", completedTools.length + 1] }) : null] }), process.env.FRANKLIN_SHOW_THINKING === '1' && thinkingText && (() => {
                         const lines = thinkingText.split('\n').filter(Boolean).slice(-3);
                         return (_jsx(Box, { flexDirection: "column", marginLeft: 2, children: lines.map((line, i) => (_jsxs(Text, { dimColor: true, wrap: "truncate-end", children: ['⎿  ', line.slice(0, 120)] }, i))) }));
                     })()] })), waiting && !thinking && tools.size === 0 && (_jsx(Box, { marginLeft: 2, children: _jsxs(Text, { color: "yellow", children: [_jsx(Spinner, { type: "dots" }), ' ', _jsxs(Text, { dimColor: true, children: [shortModelName(currentModel), completedTools.length > 0 ? ` · step ${completedTools.length + 1}` : ''] })] }) })), streamText && (_jsx(Box, { marginTop: 0, marginBottom: 0, children: _jsx(Text, { wrap: "wrap", children: renderMarkdown(streamText) }) })), responsePreview && !streamText && (_jsx(Box, { flexDirection: "column", marginBottom: 0, children: _jsx(Text, { wrap: "wrap", children: renderMarkdown(responsePreview) }) })), inPicker && (() => {

package/package.json

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