Skip to content

Cannot configure MCP to use SSE mode in YAML #5359

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.

Sign up for GitHub

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

Jump to bottom
Open
2 tasks done
jasongwq opened this issue Apr 25, 2025 · 1 comment · May be fixed by #5517
Open
2 tasks done

Cannot configure MCP to use SSE mode in YAML #5359

jasongwq opened this issue Apr 25, 2025 · 1 comment · May be fixed by #5517
Assignees
@sestinj
Labels
area:configuration Relates to configuration options good-first-issue Suggested issue for new contributors kind:enhancement Indicates a new feature request, imrovement, or extension priority:medium Indicates medium priority

Comments

@jasongwq
Copy link

Validations

  • I believe this is a way to improve. I'll try to join the Continue Discord for questions
  • I'm not able to find an open issue that requests the same enhancement

Problem

I have reviewed the Discord chat logs and the code, and I found that when using YAML configuration for MCP, only the stdio mode can be configured. It would be great if support could be added for configuring SSE and WebSocket modes.

https://discord.com/channels/1108621136150929458/1357364237219139615

continue/core/config/yaml/loadYaml.ts

Lines 65 to 76 in fb1935d

function convertYamlMcpToContinueMcp(
server: MCPServer,
): ExperimentalMCPOptions {
return {
transport: {
type: "stdio",
command: server.command,
args: server.args ?? [],
env: server.env,
},
};
}

continue/core/config/types.ts

Lines 1009 to 1029 in fb1935d

interface StdioOptions {
type: "stdio";
command: string;
args: string[];
}
interface WebSocketOptions {
type: "websocket";
url: string;
}
interface SSEOptions {
type: "sse";
url: string;
}
type TransportOptions = StdioOptions | WebSocketOptions | SSEOptions;
export interface MCPOptions {
transport: TransportOptions;
}

Solution

No response
@sestinj sestinj self-assigned this Apr 25, 2025
@dosubot dosubot bot added area:configuration Relates to configuration options kind:enhancement Indicates a new feature request, imrovement, or extension priority:medium Indicates medium priority labels Apr 25, 2025
@sestinj
Copy link
Contributor

sestinj commented May 2, 2025

Yes this is definitely an addition we want to make! I'm going to mark this as good-first-issue in case anyone would like to make an attempt at adjusting the configuration. I think it's a feasible thing for someone new to accomplish since we already have some of the code existing

@sestinj sestinj added good-first-issue Suggested issue for new contributors and removed "needs-triage" labels May 2, 2025
@nejch nejch mentioned this issue May 4, 2025
Closed
3 tasks
@04cfb1ed 04cfb1ed linked a pull request May 5, 2025 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sestinj sestinj

Labels
area:configuration Relates to configuration options good-first-issue Suggested issue for new contributors kind:enhancement Indicates a new feature request, imrovement, or extension priority:medium Indicates medium priority
Projects
None yet
Milestone
No milestone
2 participants
@jasongwq @sestinj