@slashfi/agents-sdk 0.33.2 → 0.34.1

package/dist/validate.d.ts

@@ -14,16 +14,16 @@ export declare const SerializedToolSchema: z.ZodObject<{
     name: z.ZodString;
     description: z.ZodString;
     inputSchema: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
-    outputSchema: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+    outputSchema: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
 }, "strip", z.ZodTypeAny, {
     name: string;
     description: string;
     inputSchema: Record<string, unknown>;
-    outputSchema?: Record<string, unknown> | undefined;
+    outputSchema?: z.objectOutputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
 }, {
     name: string;
     description: string;
-    outputSchema?: Record<string, unknown> | undefined;
+    outputSchema?: z.objectInputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
     inputSchema?: Record<string, unknown> | undefined;
 }>;
 export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
@@ -32,7 +32,7 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
     description: z.ZodDefault<z.ZodString>;
     version: z.ZodDefault<z.ZodString>;
     visibility: z.ZodDefault<z.ZodEnum<["public", "private"]>>;
-    auth: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+    auth: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     serverSource: z.ZodOptional<z.ZodString>;
     serverInfo: z.ZodOptional<z.ZodObject<{
         name: z.ZodString;
@@ -48,16 +48,16 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
         name: z.ZodString;
         description: z.ZodString;
         inputSchema: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
-        outputSchema: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+        outputSchema: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     }, "strip", z.ZodTypeAny, {
         name: string;
         description: string;
         inputSchema: Record<string, unknown>;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectOutputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
     }, {
         name: string;
         description: string;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectInputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
         inputSchema?: Record<string, unknown> | undefined;
     }>, "many">;
     generatedAt: z.ZodOptional<z.ZodString>;
@@ -68,7 +68,7 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
     description: z.ZodDefault<z.ZodString>;
     version: z.ZodDefault<z.ZodString>;
     visibility: z.ZodDefault<z.ZodEnum<["public", "private"]>>;
-    auth: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+    auth: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     serverSource: z.ZodOptional<z.ZodString>;
     serverInfo: z.ZodOptional<z.ZodObject<{
         name: z.ZodString;
@@ -84,16 +84,16 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
         name: z.ZodString;
         description: z.ZodString;
         inputSchema: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
-        outputSchema: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+        outputSchema: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     }, "strip", z.ZodTypeAny, {
         name: string;
         description: string;
         inputSchema: Record<string, unknown>;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectOutputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
     }, {
         name: string;
         description: string;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectInputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
         inputSchema?: Record<string, unknown> | undefined;
     }>, "many">;
     generatedAt: z.ZodOptional<z.ZodString>;
@@ -104,7 +104,7 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
     description: z.ZodDefault<z.ZodString>;
     version: z.ZodDefault<z.ZodString>;
     visibility: z.ZodDefault<z.ZodEnum<["public", "private"]>>;
-    auth: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+    auth: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     serverSource: z.ZodOptional<z.ZodString>;
     serverInfo: z.ZodOptional<z.ZodObject<{
         name: z.ZodString;
@@ -120,16 +120,16 @@ export declare const SerializedAgentDefinitionSchema: z.ZodObject<{
         name: z.ZodString;
         description: z.ZodString;
         inputSchema: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
-        outputSchema: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
+        outputSchema: z.ZodOptional<z.ZodObject<{}, "passthrough", z.ZodTypeAny, z.objectOutputType<{}, z.ZodTypeAny, "passthrough">, z.objectInputType<{}, z.ZodTypeAny, "passthrough">>>;
     }, "strip", z.ZodTypeAny, {
         name: string;
         description: string;
         inputSchema: Record<string, unknown>;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectOutputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
     }, {
         name: string;
         description: string;
-        outputSchema?: Record<string, unknown> | undefined;
+        outputSchema?: z.objectInputType<{}, z.ZodTypeAny, "passthrough"> | undefined;
         inputSchema?: Record<string, unknown> | undefined;
     }>, "many">;
     generatedAt: z.ZodOptional<z.ZodString>;

package/dist/validate.js

@@ -19,7 +19,7 @@ export const SerializedToolSchema = z.object({
     inputSchema: z
         .record(z.unknown())
         .default({ type: "object", properties: {} }),
-    outputSchema: z.record(z.unknown()).optional(),
+    outputSchema: z.object({}).passthrough().optional(),
 });
 // ============================================
 // Agent Definition Schema
@@ -31,7 +31,7 @@ export const SerializedAgentDefinitionSchema = z
     description: z.string().default(""),
     version: z.string().default("1.0.0"),
     visibility: z.enum(["public", "private"]).default("public"),
-    auth: z.record(z.unknown()).optional(),
+    auth: z.object({}).passthrough().optional(),
     serverSource: z.string().optional(),
     serverInfo: z
         .object({

package/dist/validate.js.map

@@ -1 +1 @@
-{"version":3,"file":"validate.js","sourceRoot":"","sources":["../src/validate.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,+CAA+C;AAC/C,cAAc;AACd,+CAA+C;AAE/C,MAAM,CAAC,MAAM,oBAAoB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC3C,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,uBAAuB,CAAC;IAChD,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE;IACvB,WAAW,EAAE,CAAC;SACX,MAAM,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;SACnB,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,UAAU,EAAE,EAAE,EAAE,CAAC;IAC9C,YAAY,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,QAAQ,EAAE;CAC/C,CAAC,CAAC;AAEH,+CAA+C;AAC/C,0BAA0B;AAC1B,+CAA+C;AAE/C,MAAM,CAAC,MAAM,+BAA+B,GAAG,CAAC;KAC7C,MAAM,CAAC;IACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,wBAAwB,CAAC;IACjD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,wBAAwB,CAAC;IACjD,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACnC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC;IACpC,UAAU,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC;IAC3D,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,QAAQ,EAAE;IACtC,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,UAAU,EAAE,CAAC;SACV,MAAM,CAAC;QACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE;KACpB,CAAC;SACD,QAAQ,EAAE;IACb,KAAK,EAAE,CAAC;SACL,KAAK,CAAC,oBAAoB,CAAC;SAC3B,GAAG,CAAC,CAAC,EAAE,+BAA+B,CAAC;IAC1C,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACjC,+DAA+D;CAChE,CAAC;KACD,WAAW,EAAE,CAAC;AAUjB;;;;GAIG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAc;IAC/C,MAAM,MAAM,GAAG,+BAA+B,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAChE,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACnB,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,CAAC,IAAI,EAAE,CAAC;IAC/C,CAAC;IACD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE;QAC/C,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC;QACnE,OAAO,GAAG,IAAI,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC;IACrC,CAAC,CAAC,CAAC;IACH,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;AAC/B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,qBAAqB,CAAC,KAAc,EAAE,OAAgB;IACpE,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC;IACzC,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;QACf,MAAM,MAAM,GAAG,OAAO;YACpB,CAAC,CAAC,uBAAuB,OAAO,GAAG;YACnC,CAAC,CAAC,oBAAoB,CAAC;QACzB,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,QAAQ,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IACjE,CAAC;AACH,CAAC"}
+{"version":3,"file":"validate.js","sourceRoot":"","sources":["../src/validate.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,+CAA+C;AAC/C,cAAc;AACd,+CAA+C;AAE/C,MAAM,CAAC,MAAM,oBAAoB,GAAG,CAAC,CAAC,MAAM,CAAC;IAC3C,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,uBAAuB,CAAC;IAChD,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE;IACvB,WAAW,EAAE,CAAC;SACX,MAAM,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;SACnB,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,UAAU,EAAE,EAAE,EAAE,CAAC;IAC9C,YAAY,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,QAAQ,EAAE;CACpD,CAAC,CAAC;AAEH,+CAA+C;AAC/C,0BAA0B;AAC1B,+CAA+C;AAE/C,MAAM,CAAC,MAAM,+BAA+B,GAAG,CAAC;KAC7C,MAAM,CAAC;IACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,wBAAwB,CAAC;IACjD,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,EAAE,wBAAwB,CAAC;IACjD,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC;IACnC,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,OAAO,CAAC,OAAO,CAAC;IACpC,UAAU,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,SAAS,CAAC,CAAC,CAAC,OAAO,CAAC,QAAQ,CAAC;IAC3D,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,QAAQ,EAAE;IAC3C,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACnC,UAAU,EAAE,CAAC;SACV,MAAM,CAAC;QACN,IAAI,EAAE,CAAC,CAAC,MAAM,EAAE;QAChB,OAAO,EAAE,CAAC,CAAC,MAAM,EAAE;KACpB,CAAC;SACD,QAAQ,EAAE;IACb,KAAK,EAAE,CAAC;SACL,KAAK,CAAC,oBAAoB,CAAC;SAC3B,GAAG,CAAC,CAAC,EAAE,+BAA+B,CAAC;IAC1C,WAAW,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAClC,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IACjC,+DAA+D;CAChE,CAAC;KACD,WAAW,EAAE,CAAC;AAUjB;;;;GAIG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAc;IAC/C,MAAM,MAAM,GAAG,+BAA+B,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;IAChE,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QACnB,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,CAAC,IAAI,EAAE,CAAC;IAC/C,CAAC;IACD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE;QAC/C,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC;QACnE,OAAO,GAAG,IAAI,KAAK,KAAK,CAAC,OAAO,EAAE,CAAC;IACrC,CAAC,CAAC,CAAC;IACH,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;AAC/B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,qBAAqB,CAAC,KAAc,EAAE,OAAgB;IACpE,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC;IACzC,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;QACf,MAAM,MAAM,GAAG,OAAO;YACpB,CAAC,CAAC,uBAAuB,OAAO,GAAG;YACnC,CAAC,CAAC,oBAAoB,CAAC;QACzB,MAAM,IAAI,KAAK,CAAC,GAAG,MAAM,QAAQ,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC;IACjE,CAAC;AACH,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@slashfi/agents-sdk",
-  "version": "0.33.2",
+  "version": "0.34.1",
   "author": "Slash Financial",
   "repository": {
     "type": "git",

package/src/call-agent-schema.test.ts

@@ -0,0 +1,306 @@
+import { describe, expect, test } from "bun:test";
+import { z } from "zod";
+import {
+  stripNulls,
+  nullTolerant,
+  zodToOpenAiJsonSchema,
+  callAgentValidationSchema,
+  callAgentRequestSchema,
+  listAgentsValidationSchema,
+} from "./call-agent-schema";
+
+// ─────────────────────────────────────────────────────────────────────────────
+// stripNulls
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe("stripNulls", () => {
+  test("converts null to undefined", () => {
+    expect(stripNulls(null)).toBe(undefined);
+  });
+
+  test("passes through primitives", () => {
+    expect(stripNulls("hello")).toBe("hello");
+    expect(stripNulls(42)).toBe(42);
+    expect(stripNulls(true)).toBe(true);
+    expect(stripNulls(undefined)).toBe(undefined);
+  });
+
+  test("strips null from flat objects", () => {
+    expect(stripNulls({ a: null, b: "hello" })).toEqual({ b: "hello" });
+  });
+
+  test("strips null from nested objects", () => {
+    expect(stripNulls({ a: { b: null, c: "ok" }, d: null })).toEqual({
+      a: { c: "ok" },
+    });
+  });
+
+  test("strips null from arrays", () => {
+    expect(stripNulls([1, null, 3])).toEqual([1, undefined, 3]);
+  });
+
+  test("handles deeply nested structures", () => {
+    const input = {
+      action: "execute_tool",
+      path: "/agents/@test",
+      tool: "my_tool",
+      params: null,
+      callerId: null,
+      callerType: null,
+      metadata: { nested: null, keep: "this" },
+    };
+    const result = stripNulls(input) as Record<string, unknown>;
+    expect(result.action).toBe("execute_tool");
+    expect(result.path).toBe("/agents/@test");
+    expect(result.params).toBe(undefined);
+    expect(result.callerId).toBe(undefined);
+    expect(result.callerType).toBe(undefined);
+    expect(result.metadata).toEqual({ keep: "this" });
+  });
+
+  test("handles empty objects and arrays", () => {
+    expect(stripNulls({})).toEqual({});
+    expect(stripNulls([])).toEqual([]);
+  });
+});
+
+// ─────────────────────────────────────────────────────────────────────────────
+// nullTolerant
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe("nullTolerant", () => {
+  const schema = z.object({
+    name: z.string().optional(),
+    count: z.number().optional(),
+    tags: z.array(z.string()).optional(),
+    nested: z
+      .object({
+        value: z.string().optional(),
+      })
+      .optional(),
+  });
+
+  test("original schema rejects null", () => {
+    expect(() => schema.parse({ name: null })).toThrow();
+    expect(() => schema.parse({ tags: null })).toThrow();
+    expect(() => schema.parse({ count: null })).toThrow();
+  });
+
+  test("tolerant schema accepts null for optional string", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({ name: null });
+    expect(result).toEqual({});
+  });
+
+  test("tolerant schema accepts null for optional array", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({ tags: null });
+    expect(result).toEqual({});
+  });
+
+  test("tolerant schema accepts null for optional number", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({ count: null });
+    expect(result).toEqual({});
+  });
+
+  test("tolerant schema accepts null for nested optional", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({ nested: { value: null } });
+    expect(result).toEqual({ nested: {} });
+  });
+
+  test("tolerant schema still validates real values", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({
+      name: "test",
+      count: 5,
+      tags: ["a", "b"],
+    });
+    expect(result).toEqual({ name: "test", count: 5, tags: ["a", "b"] });
+  });
+
+  test("tolerant schema still rejects invalid types", () => {
+    const tolerant = nullTolerant(schema);
+    expect(() => tolerant.parse({ name: 123 })).toThrow();
+    expect(() => tolerant.parse({ count: "not a number" })).toThrow();
+    expect(() => tolerant.parse({ tags: "not an array" })).toThrow();
+  });
+
+  test("all nulls at once", () => {
+    const tolerant = nullTolerant(schema);
+    const result = tolerant.parse({
+      name: null,
+      count: null,
+      tags: null,
+      nested: null,
+    });
+    expect(result).toEqual({});
+  });
+});
+
+// ─────────────────────────────────────────────────────────────────────────────
+// callAgentValidationSchema — real-world scenarios
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe("callAgentValidationSchema", () => {
+  test("accepts execute_tool with null optional fields (the original bug)", () => {
+    const input = {
+      request: {
+        action: "execute_tool",
+        path: "/agents/@librarian",
+        tool: "search_skill",
+        params: { query: "agent registry" },
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("accepts describe_tools with tools: null (the original bug)", () => {
+    const input = {
+      request: {
+        action: "describe_tools",
+        path: "/agents/@librarian",
+        tools: null,
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("accepts invoke with null sessionId and branchAttributes", () => {
+    const input = {
+      request: {
+        action: "invoke",
+        path: "/agents/@compactor",
+        prompt: "Hello",
+        sessionId: null,
+        branchAttributes: null,
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("accepts ask with null optional fields", () => {
+    const input = {
+      request: {
+        action: "ask",
+        path: "/agents/@worker",
+        prompt: "What time is it?",
+        sessionId: null,
+        branchAttributes: null,
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("accepts list_resources with null optional fields", () => {
+    const input = {
+      request: {
+        action: "list_resources",
+        path: "/agents/@agent-fs",
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("accepts read_resources with null optional fields", () => {
+    const input = {
+      request: {
+        action: "read_resources",
+        path: "/agents/@agent-fs",
+        uris: ["AUTH.md"],
+        callerId: null,
+        callerType: null,
+        metadata: null,
+      },
+    };
+    const result = callAgentValidationSchema.safeParse(input);
+    expect(result.success).toBe(true);
+  });
+
+  test("still rejects truly invalid input", () => {
+    const result = callAgentValidationSchema.safeParse({
+      request: {
+        action: "execute_tool",
+        // missing required 'path' and 'tool'
+      },
+    });
+    expect(result.success).toBe(false);
+  });
+
+  test("still rejects unknown action", () => {
+    const result = callAgentValidationSchema.safeParse({
+      request: {
+        action: "unknown_action",
+        path: "/agents/@test",
+      },
+    });
+    expect(result.success).toBe(false);
+  });
+});
+
+// ─────────────────────────────────────────────────────────────────────────────
+// listAgentsValidationSchema
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe("listAgentsValidationSchema", () => {
+  test("accepts all-null optional fields", () => {
+    const result = listAgentsValidationSchema.parse({
+      query: null,
+      limit: null,
+      cursor: null,
+    });
+    expect(result).toEqual({});
+  });
+
+  test("accepts mix of null and real values", () => {
+    const result = listAgentsValidationSchema.parse({
+      query: "notion",
+      limit: null,
+      cursor: null,
+    });
+    expect(result).toEqual({ query: "notion" });
+  });
+});
+
+// ─────────────────────────────────────────────────────────────────────────────
+// zodToOpenAiJsonSchema
+// ─────────────────────────────────────────────────────────────────────────────
+
+describe("zodToOpenAiJsonSchema", () => {
+  test("produces JSON schema with nullable optional fields", () => {
+    const schema = z.object({
+      required: z.string(),
+      optional: z.string().optional(),
+    });
+    const jsonSchema = zodToOpenAiJsonSchema(schema) as any;
+
+    // Required field should be in required array
+    expect(jsonSchema.required).toContain("required");
+    // Optional field should also be required (openAi convention)
+    expect(jsonSchema.required).toContain("optional");
+    // Optional field should allow null
+    const optProp = jsonSchema.properties.optional;
+    expect(optProp.anyOf || optProp.type).toBeTruthy();
+  });
+});

package/src/call-agent-schema.ts

@@ -12,7 +12,102 @@
  * Response types live in types.ts (they're output shapes, not validated input).
  */
 
-import { z } from "zod";
+import { z, type ZodTypeAny } from "zod";
+import { zodToJsonSchema } from "zod-to-json-schema";
+
+// ─────────────────────────────────────────────────────────────────────────────
+// OpenAI null-tolerance transform
+// ─────────────────────────────────────────────────────────────────────────────
+
+/**
+ * Recursively strip null values from an object, converting them to undefined.
+ * This is the inverse of zod-to-json-schema's openAi target behavior, which
+ * converts .optional() fields to nullable+required in JSON Schema.
+ *
+ * Used as a z.preprocess() step so that Zod's .optional() (which accepts
+ * undefined but not null) works correctly with LLM outputs that send null
+ * for "no value" per the OpenAI function calling convention.
+ */
+export function stripNulls(obj: unknown): unknown {
+  if (obj === null) return undefined;
+  if (Array.isArray(obj)) return obj.map(stripNulls);
+  if (typeof obj === "object") {
+    return Object.fromEntries(
+      Object.entries(obj as Record<string, unknown>).map(([k, v]) => [
+        k,
+        stripNulls(v),
+      ]),
+    );
+  }
+  return obj;
+}
+
+/**
+ * Wrap a Zod schema with a preprocess step that converts null → undefined.
+ * This makes the schema "null-tolerant" — matching what the OpenAI JSON Schema
+ * target promises to LLMs (nullable fields) while keeping Zod's .optional()
+ * semantics internally.
+ *
+ * @example
+ * ```ts
+ * const schema = z.object({ name: z.string().optional() });
+ * const tolerant = nullTolerant(schema);
+ * tolerant.parse({ name: null }); // { name: undefined } — no error
+ * ```
+ */
+export function nullTolerant<T extends ZodTypeAny>(schema: T) {
+  return z.preprocess(stripNulls, schema) as unknown as T;
+}
+
+/**
+ * Convert a Zod schema to JSON Schema using the OpenAI target,
+ * which makes all optional fields nullable+required.
+ *
+ * This is the standard way to generate input schemas for MCP tools
+ * that will be called by LLMs.
+ */
+/**
+ * Recursively normalize a JSON schema for Anthropic compatibility.
+ * Replaces `additionalProperties: {}` (empty schema = any) with `true`,
+ * which is semantically equivalent but accepted by Anthropic's validator.
+ */
+function normalizeJsonSchema(schema: Record<string, unknown>): Record<string, unknown> {
+  if (typeof schema !== "object" || schema === null) return schema;
+
+  const result: Record<string, unknown> = {};
+  for (const [key, value] of Object.entries(schema)) {
+    if (
+      key === "additionalProperties" &&
+      typeof value === "object" &&
+      value !== null &&
+      Object.keys(value).length === 0
+    ) {
+      result[key] = true;
+    } else if (Array.isArray(value)) {
+      result[key] = value.map((item) =>
+        typeof item === "object" && item !== null
+          ? normalizeJsonSchema(item as Record<string, unknown>)
+          : item,
+      );
+    } else if (typeof value === "object" && value !== null) {
+      result[key] = normalizeJsonSchema(value as Record<string, unknown>);
+    } else {
+      result[key] = value;
+    }
+  }
+  return result;
+}
+
+export function zodToOpenAiJsonSchema(
+  schema: ZodTypeAny,
+): Record<string, unknown> {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  const raw = zodToJsonSchema(schema as any, { target: "openAi" }) as Record<
+    string,
+    unknown
+  >;
+  return normalizeJsonSchema(raw);
+}
 
 // ─────────────────────────────────────────────────────────────────────────────
 // Base schemas
@@ -24,7 +119,7 @@ const callAgentBaseSchema = z.object({
   path: z.string().describe("Agent path (e.g., '@my-agent')"),
   callerId: z.string().optional().describe("Caller ID for access control"),
   callerType: callerTypeSchema.optional().describe("Caller type"),
-  metadata: z.record(z.unknown()).optional().describe("Additional metadata"),
+  metadata: z.object({}).passthrough().optional().describe("Additional metadata"),
 });
 
 // ─────────────────────────────────────────────────────────────────────────────
@@ -64,7 +159,7 @@ export const executeToolActionSchema = callAgentBaseSchema.extend({
   action: z.literal("execute_tool"),
   tool: z.string().describe("Tool name to call"),
   params: z
-    .record(z.unknown())
+    .object({}).passthrough()
     .optional()
     .describe("Parameters for the tool"),
 });
@@ -85,12 +180,20 @@ export const loadActionSchema = callAgentBaseSchema.extend({
 
 /** List resources: discover available resources on an agent */
 export const listResourcesActionSchema = callAgentBaseSchema.extend({
-  action: z.literal("list_resources").describe("List all resources available on an agent — docs, auth instructions, config schemas, etc."),
+  action: z
+    .literal("list_resources")
+    .describe(
+      "List all resources available on an agent — docs, auth instructions, config schemas, etc.",
+    ),
 });
 
 /** Read resources: fetch one or more resources by URI */
 export const readResourcesActionSchema = callAgentBaseSchema.extend({
-  action: z.literal("read_resources").describe("Fetch one or more resources by URI. Use list_resources first to discover available URIs."),
+  action: z
+    .literal("read_resources")
+    .describe(
+      "Fetch one or more resources by URI. Use list_resources first to discover available URIs.",
+    ),
   uris: z
     .array(z.string())
     .describe("Resource URIs to read (e.g., ['AUTH.md'])"),
@@ -140,15 +243,13 @@ export type CallerType = z.infer<typeof callerTypeSchema>;
 /** All supported action strings as a const array. */
 export const CALL_AGENT_ACTIONS: AgentAction[] =
   callAgentRequestSchema.options.map(
-    (s) => (s.shape as { action: z.ZodLiteral<AgentAction> }).action.value
+    (s) => (s.shape as { action: z.ZodLiteral<AgentAction> }).action.value,
   );
 
 // ─────────────────────────────────────────────────────────────────────────────
 // JSON Schema for MCP (derived from zod)
 // ─────────────────────────────────────────────────────────────────────────────
 
-import { zodToJsonSchema } from "zod-to-json-schema";
-
 /**
  * Zod schema for the full MCP tool input (wraps request in an outer object).
  * This is the schema that gets converted to JSON Schema for the LLM.
@@ -163,10 +264,17 @@ export const callAgentToolInputSchema = z.object({
  *
  * Fully derived from the zod schemas — no hand-written JSON Schema.
  */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export const callAgentInputSchema = zodToJsonSchema(
-  callAgentToolInputSchema as any,
-  { target: "openAi" }
+export const callAgentInputSchema = zodToOpenAiJsonSchema(
+  callAgentToolInputSchema,
+);
+
+/**
+ * Null-tolerant validation schema for `call_agent`.
+ * Accepts null values where the JSON Schema promises nullable,
+ * converting them to undefined before Zod validation.
+ */
+export const callAgentValidationSchema = nullTolerant(
+  callAgentToolInputSchema,
 );
 
 // ─────────────────────────────────────────────────────────────────────────────
@@ -183,9 +291,7 @@ export const listAgentsToolInputSchema = z.object({
   limit: z
     .number()
     .optional()
-    .describe(
-      "Maximum number of results per page (default: 20)",
-    ),
+    .describe("Maximum number of results per page (default: 20)"),
   cursor: z
     .string()
     .optional()
@@ -196,8 +302,10 @@ export const listAgentsToolInputSchema = z.object({
 
 export type ListAgentsInput = z.infer<typeof listAgentsToolInputSchema>;
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-export const listAgentsInputSchema = zodToJsonSchema(
-  listAgentsToolInputSchema as any,
-  { target: "openAi" }
+export const listAgentsInputSchema = zodToOpenAiJsonSchema(
+  listAgentsToolInputSchema,
+);
+
+export const listAgentsValidationSchema = nullTolerant(
+  listAgentsToolInputSchema,
 );