[PR #20] [MERGED] fix: pass MCP tool schemas through to LangChain tools #45

New Issue

2026-02-16T08:17:23-05:00

yindo commented

2026-02-16 08:17:23 -05:00

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchainjs-mcp-adapters/pull/20
Author: @benjamincburns
Created: 3/18/2025
Status: ✅ Merged
Merged: 3/18/2025
Merged by: @benjamincburns

Base: main ← Head: ben/fix-tool-schemas

📝 Commits (1)

0141384 fix: pass inputSchema to LangChain tools & fail loudly

📊 Changes

13 files changed (+3518 additions, -4597 deletions)

View changed files

📝 __tests__/client.basic.test.ts (+120 -128)
📝 __tests__/client.comprehensive.test.ts (+164 -177)
📝 __tests__/logger.test.ts (+42 -36)
📝 __tests__/tools.test.ts (+70 -186)
➖ jest.config.js (+0 -27)
📝 package-lock.json (+2804 -3890)
📝 package.json (+7 -7)
📝 src/client.ts (+208 -54)
📝 src/index.ts (+1 -2)
📝 src/logger.ts (+2 -2)
📝 src/tools.ts (+76 -83)
➕ vitest.config.ts (+18 -0)
📝 vitest.setup.ts (+6 -5)

📄 Description

Required some fairly substantial changes.

Also migrates test suite to vitest for better support for mocked modules with ESMs, and cleaned up some other miscellaneous things.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langchainjs-mcp-adapters/pull/20 **Author:** [@benjamincburns](https://github.com/benjamincburns) **Created:** 3/18/2025 **Status:** ✅ Merged **Merged:** 3/18/2025 **Merged by:** [@benjamincburns](https://github.com/benjamincburns) **Base:** `main` ← **Head:** `ben/fix-tool-schemas` --- ### 📝 Commits (1) - [`0141384`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/0141384a3caaa4f07b41bf2731c56f10a04f0e3c) fix: pass inputSchema to LangChain tools & fail loudly ### 📊 Changes **13 files changed** (+3518 additions, -4597 deletions) <details> <summary>View changed files</summary> 📝 `__tests__/client.basic.test.ts` (+120 -128) 📝 `__tests__/client.comprehensive.test.ts` (+164 -177) 📝 `__tests__/logger.test.ts` (+42 -36) 📝 `__tests__/tools.test.ts` (+70 -186) ➖ `jest.config.js` (+0 -27) 📝 `package-lock.json` (+2804 -3890) 📝 `package.json` (+7 -7) 📝 `src/client.ts` (+208 -54) 📝 `src/index.ts` (+1 -2) 📝 `src/logger.ts` (+2 -2) 📝 `src/tools.ts` (+76 -83) ➕ `vitest.config.ts` (+18 -0) 📝 `vitest.setup.ts` (+6 -5) </details> ### 📄 Description Required some fairly substantial changes. Also migrates test suite to vitest for better support for mocked modules with ESMs, and cleaned up some other miscellaneous things. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-16 08:17:23 -05:00

yindo closed this issue

2026-02-16 08:17:23 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: langchain-ai/langchainjs-mcp-adapters#45