Skip to content
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

feat(copilot): sse mcp servers support #1899

Merged
merged 5 commits into from
Mar 20, 2025
Merged

feat(copilot): sse mcp servers support #1899

merged 5 commits into from
Mar 20, 2025

Conversation

ysfscream
Copy link
Member

@ysfscream ysfscream commented Mar 18, 2025

Based on the PR information provided, I'll complete the template with the relevant details:

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Currently, MQTTX only supports standard I/O transmission for MCP (MQTT Control Protocol).

Issue Number

(No specific issue number provided)

What is the new behavior?

Try to use this SSE Server: https://github.com/ysfscream/mqttx-mcp-sse-server

image image image

Added SSE (Server-Sent Events) support for MCP services. Now, users can enable MCP services simply by using a URL.

The screenshots show:

  1. The new interface for MCP connection configuration with SSE support
  2. The connection setup screen with the URL input option
  3. The working MCP connection using SSE transport

Does this PR introduce a breaking change?

  • No

Specific Instructions

To test this new feature:

  1. Enter an SSE-enabled MCP server URL in the connection settings
  2. The system will automatically use the SSE transport method for MCP services
  3. This enhancement maintains backward compatibility with the existing studio transport method

Other information

This PR extends the transport protocol support for MCP in MQTTX by adding SSE transport capability alongside the existing studio support. The implementation is designed to be straightforward for users - they must provide the URL to enable MCP services via SSE.

@ysfscream ysfscream marked this pull request as ready for review March 20, 2025 05:46
@ysfscream ysfscream requested a review from Red-Asuka March 20, 2025 05:46
@ysfscream ysfscream self-assigned this Mar 20, 2025
@ysfscream ysfscream added enhancement New feature or request feature This pr is a feature desktop MQTTX Desktop labels Mar 20, 2025
@ysfscream ysfscream added this to the v1.12.0 milestone Mar 20, 2025
@Red-Asuka Red-Asuka merged commit b04b0d0 into main Mar 20, 2025
4 checks passed
@Red-Asuka Red-Asuka deleted the ysf/desktop branch March 20, 2025 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
desktop MQTTX Desktop enhancement New feature or request feature This pr is a feature
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants