[PR #64] [MERGED] feat: support StreamableHTTPClientTransport #69

Closed
opened 2026-02-16 08:17:26 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchainjs-mcp-adapters/pull/64
Author: @leo237
Created: 4/22/2025
Status: Merged
Merged: 5/13/2025
Merged by: @benjamincburns

Base: mainHead: feature/streamable-http


📝 Commits (10+)

  • 0ca7648 Added support for basic streamable http adapter
  • b265abf added tests
  • 07ff66b updated example
  • e88ef0c Updated readme
  • 05f4b67 fixes formatting issues
  • ee26a0f Use Streamable HTTP for all URLs first, fallback to SSE
  • c193c20 partial test rework
  • b33a2de fix tests, add example (not working yet)
  • 0f42f28 fixes, add Streaming HTTP example w/ SSE fallback, update REAMDE
  • ce96b23 bump to latest MCP sdk

📊 Changes

13 files changed (+1453 additions, -416 deletions)

View changed files

📝 .gitignore (+2 -1)
📝 README.md (+18 -12)
📝 __tests__/client.basic.test.ts (+139 -0)
📝 __tests__/client.comprehensive.test.ts (+54 -1)
📝 __tests__/mocks.ts (+18 -0)
examples/calculator_server_shttp_sse.ts (+173 -0)
examples/calculator_sse_shttp_example.ts (+204 -0)
📝 langchain.config.js (+1 -0)
📝 package.json (+3 -1)
📝 src/client.ts (+388 -67)
📝 src/index.ts (+8 -1)
📝 src/tools.ts (+3 -4)
📝 yarn.lock (+442 -329)

📄 Description

Description

Adds support for Streamable HTTP. Related to https://github.com/langchain-ai/langchainjs-mcp-adapters/issues/59

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)

How Has This Been Tested?

  1. Tests are added
  2. Manually tested with a Streamable HTTP server

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

🔄 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/64 **Author:** [@leo237](https://github.com/leo237) **Created:** 4/22/2025 **Status:** ✅ Merged **Merged:** 5/13/2025 **Merged by:** [@benjamincburns](https://github.com/benjamincburns) **Base:** `main` ← **Head:** `feature/streamable-http` --- ### 📝 Commits (10+) - [`0ca7648`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/0ca764875e973a9048038c641fee55f00a9252ad) Added support for basic streamable http adapter - [`b265abf`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/b265abfc104719226ed36a37feea2106b57ae9d2) added tests - [`07ff66b`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/07ff66b7b17e50346cdbbadb60415ea96ea2835a) updated example - [`e88ef0c`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/e88ef0cdb686bf36af9ee20f96ebab86a733a324) Updated readme - [`05f4b67`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/05f4b67484f848976fe2a64685e8a67f616b35f3) fixes formatting issues - [`ee26a0f`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/ee26a0f016a0a08eba10314e30f7eab1a46b76ba) Use Streamable HTTP for all URLs first, fallback to SSE - [`c193c20`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/c193c209a4107020e2b62dd26f947bedcc01fd3e) partial test rework - [`b33a2de`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/b33a2de2aabbc7c617d99a8958c44863d53d4848) fix tests, add example (not working yet) - [`0f42f28`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/0f42f289ff3ede513a339203688370f1c47fae86) fixes, add Streaming HTTP example w/ SSE fallback, update REAMDE - [`ce96b23`](https://github.com/langchain-ai/langchainjs-mcp-adapters/commit/ce96b23933174054c989f66f50c23432972fd9db) bump to latest MCP sdk ### 📊 Changes **13 files changed** (+1453 additions, -416 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+2 -1) 📝 `README.md` (+18 -12) 📝 `__tests__/client.basic.test.ts` (+139 -0) 📝 `__tests__/client.comprehensive.test.ts` (+54 -1) 📝 `__tests__/mocks.ts` (+18 -0) ➕ `examples/calculator_server_shttp_sse.ts` (+173 -0) ➕ `examples/calculator_sse_shttp_example.ts` (+204 -0) 📝 `langchain.config.js` (+1 -0) 📝 `package.json` (+3 -1) 📝 `src/client.ts` (+388 -67) 📝 `src/index.ts` (+8 -1) 📝 `src/tools.ts` (+3 -4) 📝 `yarn.lock` (+442 -329) </details> ### 📄 Description ## Description Adds support for Streamable HTTP. Related to https://github.com/langchain-ai/langchainjs-mcp-adapters/issues/59 Fixes # (issue) ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [X] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Documentation update - [ ] Refactoring (no functional changes) ## How Has This Been Tested? 1. Tests are added 2. Manually tested with a Streamable HTTP server ## Checklist: - [X] My code follows the style guidelines of this project - [X] I have performed a self-review of my own code - [X] I have commented my code, particularly in hard-to-understand areas - [X] I have made corresponding changes to the documentation - [X] My changes generate no new warnings - [X] I have added tests that prove my fix is effective or that my feature works - [X] New and existing unit tests pass locally with my changes --- <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:26 -05:00
yindo closed this issue 2026-02-16 08:17:26 -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#69