fix(mastra): omit outputSchema to avoid invalid output validation

.changeset/shaky-dogs-start.md

@@ -0,0 +1,5 @@
+---
+'@composio/mastra': patch
+---
+
+Omit `outputSchema` from Mastra `createTool` so third-party API responses with `null` or extra fields are not rejected by Mastra output validation (see ComposioHQ/composio#3047).

pnpm-workspace.yaml

@@ -37,7 +37,7 @@ catalog:
   openai: ^6.16.0
   pnpm: ^10.28.0
   publint: ^0.3.16
-  tsdown: ^0.18.4
+  tsdown: ^0.21.7
   tsx: ^4.20.5
   typescript: ^5.9.2
   uuid: ^13.0.0

ts/packages/cli/package.json

@@ -35,7 +35,7 @@
     "table:metrics": "bun run ./scripts/optimize-metrics-table.ts",
     "mock": "bun run ./scripts/copy-mocks-from-cache.ts",
     "prebuild": "pnpm run typecheck",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "bin": "./dist/composio",
     "test": "vitest run",
     "prepublishOnly": "pnpm run build && pnpm run build:binary",

ts/packages/cli/tsdown.config.ts

@@ -12,13 +12,19 @@ export default defineConfig({
   ],
   format: ['esm'],
   tsconfig: './tsconfig.src.json',
-  external: undefined,
-  noExternal: [
-    '@composio/core',
-    /^zod(?:\/.*)?$/,
-    /^@agentclientprotocol\/sdk(?:\/.*)?$/,
-    /^@modelcontextprotocol\/sdk(?:\/.*)?$/,
-  ],
+  /**
+   * Bundle these deps (overrides base `neverBundle` for @composio/core / zod).
+   * Keep `node:` built-ins external.
+   */
+  deps: {
+    neverBundle: [/^node:/],
+    alwaysBundle: [
+      '@composio/core',
+      /^zod(?:\/.*)?$/,
+      /^@agentclientprotocol\/sdk(?:\/.*)?$/,
+      /^@modelcontextprotocol\/sdk(?:\/.*)?$/,
+    ],
+  },
   publint: undefined,
   attw: undefined,
 });

ts/packages/core/package.json

@@ -131,7 +131,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run",
     "typecheck": "tsc --noEmit --skipLibCheck && tsc --noEmit --skipLibCheck -p tsconfig.type-tests.json",
     "generate:docs": "tsx scripts/generate-docs.ts"

ts/packages/core/tsdown.config.ts

@@ -1,4 +1,4 @@
-import { defineConfig, UserConfig } from 'tsdown';
+import { defineConfig } from 'tsdown';
 import { baseConfig } from '../../../tsdown.config.base';
 
 export default defineConfig({
@@ -46,11 +46,17 @@ export default defineConfig({
    * We don't want to accidentally bundle `node:*` packages (e.g., `node:module`)
    * as not all of them are available in Cloudflare Workers / Vercel Edge runtimes.
    */
-  external: [
-    ...(baseConfig.external ?? []),
-    '#platform',
-    '#files',
-    '#file_tool_modifier',
-    '#config_defaults',
-  ],
+  deps: {
+    neverBundle: [
+      ...(Array.isArray(baseConfig.deps?.neverBundle)
+        ? baseConfig.deps.neverBundle
+        : baseConfig.deps?.neverBundle != null
+          ? [baseConfig.deps.neverBundle]
+          : []),
+      '#platform',
+      '#files',
+      '#file_tool_modifier',
+      '#config_defaults',
+    ],
+  },
 });

ts/packages/json-schema-to-zod/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run",
     "typecheck:src": "tsc --noEmit -p ./tsconfig.src.json",
     "typecheck:test": "tsc --noEmit -p ./tsconfig.test.json",

ts/packages/providers/anthropic/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [

ts/packages/providers/claude-agent-sdk/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [

ts/packages/providers/cloudflare/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [],

ts/packages/providers/google/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [

ts/packages/providers/langchain/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [],

ts/packages/providers/llamaindex/package.json

@@ -29,7 +29,7 @@
     "dist"
   ],
   "scripts": {
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [

ts/packages/providers/mastra/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run",
     "typecheck": "tsc --noEmit --skipLibCheck"
   },

ts/packages/providers/mastra/src/index.ts

@@ -96,19 +96,10 @@ export class MastraProvider extends BaseAgenticProvider<
       mode: 'jsonSchema',
     });
 
-    const outputSchema = applyCompatLayer({
-      schema: tool.outputParameters ?? {},
-      compatLayers: [],
-      mode: 'jsonSchema',
-    });
-
     const mastraTool = createTool({
       id: tool.slug,
       description: tool.description ?? '',
-      // @ts-ignore
       inputSchema,
-      // @ts-ignore
-      outputSchema,
       execute: async (inputData, _context) => {
         const result = await executeTool(tool.slug, inputData as Record<string, unknown>);
         return result;

ts/packages/providers/mastra/test/mastra.test.ts

@@ -113,7 +113,6 @@ describe('MastraProvider', () => {
         id: mockTool.slug,
         description: mockTool.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.inputParameters },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
 
@@ -135,7 +134,6 @@ describe('MastraProvider', () => {
         id: toolWithoutInputParams.slug,
         description: toolWithoutInputParams.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: {} },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
 
@@ -157,7 +155,6 @@ describe('MastraProvider', () => {
         id: toolWithoutOutputParams.slug,
         description: toolWithoutOutputParams.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.inputParameters },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: {} },
         execute: expect.any(Function),
       });
 
@@ -179,7 +176,6 @@ describe('MastraProvider', () => {
         id: toolWithoutDescription.slug,
         description: '',
         inputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.inputParameters },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
 
@@ -289,14 +285,12 @@ describe('MastraProvider', () => {
         id: mockTool.slug,
         description: mockTool.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.inputParameters },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
       expect(createTool).toHaveBeenCalledWith({
         id: anotherTool.slug,
         description: anotherTool.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: anotherTool.inputParameters },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: anotherTool.outputParameters },
         execute: expect.any(Function),
       });
     });
@@ -400,7 +394,7 @@ describe('MastraProvider', () => {
       expect(wrapped).toHaveProperty('id');
       expect(wrapped).toHaveProperty('description');
       expect(wrapped).toHaveProperty('inputSchema');
-      expect(wrapped).toHaveProperty('outputSchema');
+      expect(wrapped.outputSchema).toBeUndefined();
       expect(wrapped).toHaveProperty('execute');
 
       // The execute property should be a function
@@ -434,7 +428,6 @@ describe('MastraProvider', () => {
         id: 'minimal-tool',
         description: 'A minimal tool',
         inputSchema: { type: 'mock-zod-schema', originalSchema: {} },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: {} },
         execute: expect.any(Function),
       });
 
@@ -540,7 +533,6 @@ describe('MastraProvider', () => {
             additionalProperties: false,
           },
         },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
     });
@@ -589,7 +581,6 @@ describe('MastraProvider', () => {
             required: ['required_field'],
           },
         },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
     });
@@ -618,7 +609,6 @@ describe('MastraProvider', () => {
             description: 'A string parameter',
           },
         },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
     });
@@ -638,7 +628,6 @@ describe('MastraProvider', () => {
         id: toolWithoutInputParams.slug,
         description: toolWithoutInputParams.description,
         inputSchema: { type: 'mock-zod-schema', originalSchema: {} },
-        outputSchema: { type: 'mock-zod-schema', originalSchema: mockTool.outputParameters },
         execute: expect.any(Function),
       });
     });

ts/packages/providers/openai-agents/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [],

ts/packages/providers/openai/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [],

ts/packages/providers/vercel/package.json

@@ -30,7 +30,7 @@
   ],
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "keywords": [],

ts/packages/ts-builders/package.json

@@ -35,7 +35,7 @@
   },
   "scripts": {
     "clean": "git clean -xdf node_modules",
-    "build": "pnpm exec tsdown",
+    "build": "pnpm exec tsdown --config-loader unrun",
     "test": "vitest run"
   },
   "files": [

tsdown.config.base.ts

@@ -63,9 +63,11 @@ export const baseConfig = {
   },
 
   /**
-   * External dependencies that should not be bundled, but provided by the consumer.
+   * Dependencies that should not be bundled (provided by the consumer at runtime).
    */
-  external: ['zod', '@composio/core', /^node:/],
+  deps: {
+    neverBundle: ['zod', '@composio/core', /^node:/],
+  },
 
   /**
    * Control how Node.js built-in module imports are handled.