Skip to content

Conversation

AhmedElhadidii
Copy link

Summary;
-Migrate server transport from SSE to Streamable HTTP.
-Consolidate all MCP traffic to a single /mcp endpoint with StreamableHTTPServerTransport.handleRequest.
-Keep stdio mode unchanged.
-Add Linux-safe guards to avoid xcrun calls on non-macOS.
-Bump MCP SDK to ^1.13.2.

@krismuhi
Copy link
Member

thanks for the PR!

we need to make this optional.
with defaults and easy way to switch

can you share your usecase?

@AhmedElhadidii
Copy link
Author

The Use case is that there are some MCP clients that only support the new Http Streamable method.
For example N8N Only allow integrations with Http Streamable

Copilot AI and others added 4 commits August 22, 2025 13:48
Co-authored-by: AhmedElhadidii <[email protected]>
…-36a8041e48ba

Support both Streamable HTTP and SSE transports for backward compatibility
@AhmedElhadidii AhmedElhadidii changed the title Server: migrate SSE to Streamable HTTP & Bump MCP SDK version Server: Added Streamable HTTP & Bump MCP SDK version Aug 22, 2025
@gmegidish
Copy link
Member

@AhmedElhadidii this PR is lit 🔥 🔥 !

I'll check it thoroughly with n8n and I'll merge.

@gmegidish
Copy link
Member

btw, why was the types.d file needed? these types are exported already in the mcp package itself.

thank you so much for this PR!!!! 🙏

@AhmedElhadidii
Copy link
Author

AhmedElhadidii commented Aug 24, 2025

btw, why was the types.d file needed? these types are exported already in the mcp package itself.

Yes You're absolutely Right!👌

Copy link
Author

@AhmedElhadidii AhmedElhadidii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants