Skip to content

feat(config-yaml): support configuring MCP server transport #5511

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
wants to merge 1 commit into from

Conversation

nejch
Copy link
Contributor

@nejch nejch commented May 4, 2025

Description

Adds support for transport type (enabling SSE use via YAML).

Closes #5359

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Copy link

netlify bot commented May 4, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit c645517
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/681797e6535556000812dc33

@nejch nejch force-pushed the feat/mcp-sse-yaml branch from d3d8870 to c645517 Compare May 4, 2025 16:37
@enzoescipy
Copy link

Hi @nejch, thank you for initiating this important feature to support SSE/WebSocket transports for MCP servers! We were also really looking forward to this capability for our own setup.
Since the PR is currently in draft and we were quite eager to use this feature, we took the liberty of building upon your work to see if we could get a version running quickly.
We've managed to get a working version (at least for our use case) implemented in our fork, which you can find here:
https://github.com/enzoescipy/continue/tree/sse-mcp-support
It mainly involved completing the URL handling in loadYaml.ts and ensuring the schema definition in schemas/index.ts was correct. The core connection logic in MCPConnection.ts was already mostly there!
Please feel free to use this as a reference, or even integrate parts of it if it helps you finalize your PR. No pressure at all, just wanted to share what we quickly put together in case it's useful! Thanks again for getting this started.

@nejch
Copy link
Contributor Author

nejch commented May 5, 2025

Thanks @enzoescipy for taking the time for this - I'll admit I just pushed a quick draft yesterday without properly going through it. It seems like #5517 is more complete so feel free to add your feedback there, I'll close this one 🙇

@nejch nejch closed this May 5, 2025
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs May 5, 2025
@nejch nejch deleted the feat/mcp-sse-yaml branch May 5, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Cannot configure MCP to use SSE mode in YAML
2 participants