starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-09-21 00:02:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: withdraw MCP servers when a listing fails after a workspace switch

Browse Source 
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JgzuxyafKNF2uaEeL35XQw
This commit is contained in:
Guilhem Lemouel
2026-09-10 21:27:54 +02:00
co-authored by Claude Opus 5
parent cc462bea3e
commit 5dc66dd794
1 changed files with 14 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/src/lib/components/copilot/chat/AIChatManager.svelte.ts
+14 -1
View File
@@ -2259,8 +2259,21 @@ export class AIChatManager {
} catch (e) {
// A listing that failed is not a workspace with nothing connected: reconciling
// against it would drop every registered tool and lift every refusal because a
// request happened to fail. Keep what the last good refresh settled on.
// request happened to fail. Keep what the last good refresh settled on — but
// only while the chat is still on the workspace that answer came from. The same
// path in another workspace is a different server, and one the user has not
// opted into, so a failure after a switch withdraws everything instead.
console.error('Failed to load MCP servers', e)
if (refreshId !== this.mcpServersRefreshId || this.mcpRegistryWorkspace === workspace) {
return
}
this.mcpServers = []
this.mcpServersSignature = ''
this.mcpRegistryWorkspace = workspace
forgetLoadedMcpTools(this.mcpOwnerId)
if (this.mode === AIMode.GLOBAL) {
this.configureGlobalMode()
}
return
}
if (refreshId !== this.mcpServersRefreshId) {