Skip to content

[Feature Request]: BasicMCPClient Support streamablehttp #18755

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
JosephWuMTK opened this issue May 16, 2025 · 3 comments
Closed

[Feature Request]: BasicMCPClient Support streamablehttp #18755

JosephWuMTK opened this issue May 16, 2025 · 3 comments
Labels
enhancement New feature or request triage Issue needs to be triaged/prioritized

Comments

@JosephWuMTK
Copy link

Feature Description

Hi team,

I noticed that the MCP Python SDK now supports the new Streamable HTTP transport from protocol version 2025-03-26 (see release notes: https://github.com/modelcontextprotocol/python-sdk/releases/tag/v1.8.0). There are also relevant examples provided.

It would be great if BasicMCPClient could also support streamablehttp_client from mcp.client.streamable_http.
This feature would be very useful for our applications.

Reason

No response

Value of Feature

No response

@JosephWuMTK JosephWuMTK added enhancement New feature or request triage Issue needs to be triaged/prioritized labels May 16, 2025
@logan-markewich
Copy link
Collaborator

logan-markewich commented May 16, 2025

PRs certainly welcome. I can look into it eventually

@ayowu1981
Copy link

Hi! Just a suggestion for improving the design of _run_session() in BasicMCPClient.

Currently, the transport type is inferred from command_or_url (e.g., checking if the scheme is "http"/"https"), which makes it harder to support additional transport types like streamable_http.

Consider making the transport type explicit—e.g., passing a transport: Literal["stdio", "sse", "streamable_http"] argument—and dispatching via a transport handler registry or factory function. This would improve clarity, modularity, and future extensibility.

@logan-markewich
Copy link
Collaborator

This has been added, fyi (and the connection type checking has also been greatly improved)

Feel free to open another PR if the current iteration is not enough @ayowu1981

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triage Issue needs to be triaged/prioritized
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants