Skip to content

feat(mcp): add streamable http client - #2292

Merged
hayescode merged 33 commits into
Chainlit:mainfrom
tituslhy:main
Jul 30, 2025
Merged

feat(mcp): add streamable http client#2292
hayescode merged 33 commits into
Chainlit:mainfrom
tituslhy:main

Conversation

@tituslhy

@tituslhy tituslhy commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. backend Pertains to the Python backend. frontend Pertains to the frontend. labels Jul 15, 2025
@asvishnyakov asvishnyakov changed the title add streamable-http client feat(mcp): add streamable http client Jul 15, 2025
@hayescode

hayescode commented Jul 23, 2025

Copy link
Copy Markdown
Contributor

@tituslhy can you please fix the CI issues?

@tituslhy

tituslhy commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

Sorry my frontend skills are extremely limited so I apologize for not being able to resolve the CI issues. On the backend side I only edited a few scripts to include the streamable-http class but I was also unable to clear mypy for some reason. I did what I could and pushed to kickstart the inclusion of the integration of this feature

If it's possible can I trouble you for your support please. My profuse apologies.

@agnivabanerjee

agnivabanerjee commented Jul 24, 2025

Copy link
Copy Markdown
Contributor

@tituslhy there are some issues:

  1. on server.py: "url": payload.url if payload.clientType in ["sse", "streamable-http"] else None,
  2. on config.py: add a dataclass for StreamableHTTP, and add streamable http to McpFeature
  3. on api/index.ts: add sessionId as well

there are more, and I am trying to test these changes locally. can you please provide write access to me?

thank you :)

@tituslhy

tituslhy commented Jul 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@agnivabanerjee sorry how do I provide write access, please? All I did was raise a pull request.

But I've added you as a collaborator to my personal fork of chainlit just in case.

@agnivabanerjee

agnivabanerjee commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

@agnivabanerjee sorry how do I provide write access, please? All I did was raise a pull request.

But I've added you as a collaborator to my personal fork of chainlit just in case.

Thank you Titus! I have made the changes and merged.
@hayescode can you please review and help make a release if everything looks good?

@hayescode

hayescode commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

@asvishnyakov

@asvishnyakov
asvishnyakov self-requested a review July 25, 2025 21:48
@agnivabanerjee

agnivabanerjee commented Jul 25, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

@asvishnyakov thank you for the review! sorry for the overlook before, I have since fixed lint and CI errors in the latest commit 🙏
Please let us know if any further changes are required!

@agnivabanerjee
fix: ruff formatting and linting issues
ae20139 
- Fix quote style in server.py (single to double quotes)
- Apply ruff format to config.py, server.py, and types.py
- Ensure all code passes ruff check and format --check
@agnivabanerjee
fix: add null check for interception.response in header_auth test
881e85c 
Fixes TypeError: Cannot read properties of undefined (reading 'statusCode')
that occurs after page reload due to browser caching.

Minimal fix: just wrap statusCode access in null check.
@agnivabanerjee
Merge branch 'main' into main
c27217e
@agnivabanerjee
Merge branch 'main' into pr/2292
f0209df
@agnivabanerjee
Handle streamable-http connection objects
9e87faa 
Ignoring get_session_id for now
@agnivabanerjee

agnivabanerjee commented Jul 26, 2025

Copy link
Copy Markdown
Contributor

hi @hayescode @asvishnyakov @tituslhy !

with the latest push, I have fixed the remaining errors when connecting to streamable-http.
PyTests and CI issues pass locally.
Let me know please if there are any more issues 🙏

having this feature in the next release would be great!

@agnivabanerjee agnivabanerjee mentioned this pull request Jul 27, 2025
Open
@agnivabanerjee
fix: ruff formatting and linting issues
06f81d3 
- Fix quote style in server.py (single to double quotes)
- Apply ruff format to config.py, server.py, and types.py
- Ensure all code passes ruff check and format --check
@agnivabanerjee
fix: add null check for interception.response in header_auth test
9fa3ad2 
Fixes TypeError: Cannot read properties of undefined (reading 'statusCode')
that occurs after page reload due to browser caching.

Minimal fix: just wrap statusCode access in null check.
@agnivabanerjee
Handle streamable-http connection objects
49a2cfe 
Ignoring get_session_id for now
@agnivabanerjee
Handle bug when passing quoted url variables
7905abd
@agnivabanerjee
Minor: streamable-http placeholder URL changed to end with /mcp
05e9ecb
@agnivabanerjee
Add optional header for SSE and Streamable-HTTP
e8496e6
@agnivabanerjee
Fix default server type selection to respect allowHttp flag
f6f54cc
@agnivabanerjee
Cypress: skip login if already authenticated to avoid flakiness
91ce8df
@agnivabanerjee
Merge branch 'pr/2292' of https://github.com/tituslhy/chainlit into p…
7540ea0 
…r/2292
@agnivabanerjee
Cypress: login helper checks pathname instead of element presence
f3d31fa
@agnivabanerjee
CI Fix - Data Layer
17a90ee
@agnivabanerjee
Merge branch 'main' into main
838cfe3
@hayescode
hayescode added this pull request to the merge queue Jul 30, 2025
Merged via the queue into Chainlit:main with commit df3d4ff Jul 30, 2025
@asvishnyakov asvishnyakov mentioned this pull request Aug 7, 2025
Merged
asvishnyakov added a commit to Chainlit/docs that referenced this pull request Aug 7, 2025
@asvishnyakov
Update docs: 2.6.0...2.6.7 changes, update project and feature settin…
8fb27bd 
…gs (#254)

Included changes from:
- Chainlit/chainlit#2299
- Chainlit/chainlit#2292
- Chainlit/chainlit#2271
- Chainlit/chainlit#2347
- Chainlit/chainlit#2364

Closes #251, #252

Also fixed boolean field values in some files
@r0h1tb r0h1tb mentioned this pull request Aug 19, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hayescode hayescode hayescode approved these changes
@asvishnyakov asvishnyakov Awaiting requested review from asvishnyakov asvishnyakov is a code owner
+1 more reviewer
@agnivabanerjee agnivabanerjee agnivabanerjee approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backend Pertains to the Python backend. frontend Pertains to the frontend. size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tituslhy @hayescode @agnivabanerjee