Skip to content

Commit ebeb796

Browse files
committed
fix: prevent disabled MCP servers from starting processes
- Add disabled checks in updateServerConnections() before calling connectToServer() - Ensures disabled servers do not consume system resources - Follows existing pattern used in readResource() and callTool() methods - Fixes issue #2797
1 parent a163053 commit ebeb796

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/services/mcp/McpHub.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -976,7 +976,9 @@ export class McpHub {
976976
// New server
977977
try {
978978
this.setupFileWatcher(name, validatedConfig, source)
979-
await this.connectToServer(name, validatedConfig, source)
979+
if (!validatedConfig.disabled) {
980+
await this.connectToServer(name, validatedConfig, source)
981+
}
980982
} catch (error) {
981983
this.showErrorMessage(`Failed to connect to new MCP server ${name}`, error)
982984
}
@@ -985,7 +987,9 @@ export class McpHub {
985987
try {
986988
this.setupFileWatcher(name, validatedConfig, source)
987989
await this.deleteConnection(name, source)
988-
await this.connectToServer(name, validatedConfig, source)
990+
if (!validatedConfig.disabled) {
991+
await this.connectToServer(name, validatedConfig, source)
992+
}
989993
} catch (error) {
990994
this.showErrorMessage(`Failed to reconnect MCP server ${name}`, error)
991995
}

0 commit comments

Comments
 (0)