@compilr-dev/sdk 0.10.34 → 0.10.35

package/dist/meta-tools/registry.js

@@ -489,6 +489,8 @@ function buildCompactSchema(tool) {
  * Handles common LLM mistakes when calling tools without guided decoding:
  * - Single object where array expected → wraps in array
  * - JSON string where array expected → parses it
+ * - JSON string where object expected → parses it (covers e.g. Gemini Flash
+ *   stringifying complex nested objects like build_interactive_flow.flow)
  */
 function coerceArgs(schema, args) {
     const properties = schema.properties;
@@ -497,27 +499,43 @@ function coerceArgs(schema, args) {
     let changed = false;
     const coerced = { ...args };
     for (const [name, prop] of Object.entries(properties)) {
-        if (prop.type === 'array' && name in coerced && !Array.isArray(coerced[name])) {
-            const value = coerced[name];
-            if (value !== null && value !== undefined) {
-                if (typeof value === 'object') {
-                    // Single object → wrap in array
-                    coerced[name] = [value];
-                    changed = true;
-                }
-                else if (typeof value === 'string') {
-                    // JSON string → try parsing
-                    try {
-                        const parsed = JSON.parse(value);
-                        if (Array.isArray(parsed)) {
-                            coerced[name] = parsed;
-                            changed = true;
-                        }
-                    }
-                    catch {
-                        // Leave as-is, will fail validation
+        if (!(name in coerced))
+            continue;
+        const value = coerced[name];
+        if (value === null || value === undefined)
+            continue;
+        if (prop.type === 'array' && !Array.isArray(value)) {
+            if (typeof value === 'object') {
+                // Single object → wrap in array
+                coerced[name] = [value];
+                changed = true;
+            }
+            else if (typeof value === 'string') {
+                // JSON string → try parsing
+                try {
+                    const parsed = JSON.parse(value);
+                    if (Array.isArray(parsed)) {
+                        coerced[name] = parsed;
+                        changed = true;
                     }
                 }
+                catch {
+                    // Leave as-is, will fail validation
+                }
+            }
+        }
+        else if (prop.type === 'object' && typeof value === 'string') {
+            // JSON-stringified object → try parsing. Some providers (Gemini Flash
+            // via OpenAI compat, etc.) ship deeply-nested object args as strings.
+            try {
+                const parsed = JSON.parse(value);
+                if (parsed !== null && typeof parsed === 'object' && !Array.isArray(parsed)) {
+                    coerced[name] = parsed;
+                    changed = true;
+                }
+            }
+            catch {
+                // Leave as-is, will fail validation
             }
         }
     }

package/dist/tools/interactive-flow-tool.js

@@ -706,29 +706,7 @@ export function createInteractiveFlowTool(handler) {
         inputSchema: INTERACTIVE_FLOW_INPUT_SCHEMA,
         execute: async (input) => {
             try {
-                // Defensive — some providers (Gemini Flash via OpenAI compat,
-                // etc.) ship complex object arguments as JSON-stringified strings
-                // rather than parsed objects. The agents library does the outer
-                // parse but doesn't recurse into nested values. Parse here so the
-                // validator sees the proper object shape. Same quirk hit by
-                // ask_user.questions and propose_alternatives.alternatives —
-                // fixed at the CLI display layer for those, but here it must
-                // happen pre-validation since the validator rejects strings.
-                let flow = input.flow;
-                if (typeof flow === 'string') {
-                    try {
-                        flow = JSON.parse(flow);
-                    }
-                    catch {
-                        return {
-                            success: false,
-                            error: 'Interactive flow validation failed:\n' +
-                                "[INVALID_NODE_TYPE] Flow must be an object — received a string that wasn't valid JSON. " +
-                                'Pass the flow as a parsed object, not as a JSON-stringified string.',
-                        };
-                    }
-                }
-                const validation = validateFlow(flow);
+                const validation = validateFlow(input.flow);
                 if (!validation.ok) {
                     const summary = validation.errors
                         .map((e) => `[${e.code}]${e.nodeId ? ` (node '${e.nodeId}')` : ''} ${e.message}`)
@@ -738,11 +716,7 @@ export function createInteractiveFlowTool(handler) {
                         error: `Interactive flow validation failed:\n${summary}`,
                     };
                 }
-                // Validation passed → flow is a well-formed Flow. Hand it to the
-                // handler with the (possibly-parsed) value so the host gets an
-                // object too.
-                const normalisedInput = { flow: flow };
-                const result = await handler(normalisedInput);
+                const result = await handler(input);
                 if (validation.warnings.length > 0) {
                     result.warnings = [...(result.warnings ?? []), ...validation.warnings];
                 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@compilr-dev/sdk",
-  "version": "0.10.34",
+  "version": "0.10.35",
   "description": "Universal agent runtime for building AI-powered applications",
   "type": "module",
   "main": "dist/index.js",