fix: prevent duplicate MCP resources from appearing

- Remove redundant resource fetching in toggleServerDisabled method
- Resources are already fetched during initial connection in connectToServer
- This fixes issue #8301 where resources appeared twice in the UI

The duplicate fetching was happening when toggling server disabled state
while the server was already connected. Since toggling disabled state
doesn't affect the available resources/tools, we don't need to refresh
them.
This commit is contained in:
Roo Code 2025-09-25 11:14:41 +00:00
parent 2f1b94f430
commit 2bfce691f5
2 changed files with 3 additions and 8 deletions

1
.tmp/Roo-Code Submodule

@ -0,0 +1 @@
Subproject commit 86debeef43acbea9bdc1aa4b38d514541e164c91

View file

@ -1376,15 +1376,9 @@ export class McpHub {
await this.deleteConnection(serverName, serverSource)
// When re-enabling, file watchers will be set up in connectToServer
await this.connectToServer(serverName, config, serverSource)
} else if (connection.server.status === "connected") {
// Only refresh capabilities if connected
connection.server.tools = await this.fetchToolsList(serverName, serverSource)
connection.server.resources = await this.fetchResourcesList(serverName, serverSource)
connection.server.resourceTemplates = await this.fetchResourceTemplatesList(
serverName,
serverSource,
)
}
// Note: We don't need to refresh resources when the server is already connected
// because toggling disabled state doesn't affect the resources/tools available
} catch (error) {
console.error(`Failed to refresh capabilities for ${serverName}:`, error)
}