fix: remove the tool from the mapping after user set incase the conflict (#1609)

Randall-Jiang · web-flow · commit 48b996d1a325 · 2025-06-10T18:24:55.000-07:00
diff --git a/server/aws-lsp-codewhisperer/src/language-server/agenticChat/tools/toolServer.ts b/server/aws-lsp-codewhisperer/src/language-server/agenticChat/tools/toolServer.ts
@@ -91,6 +91,7 @@ export const McpToolsServer: Server = ({ credentialsProvider, workspace, logging
         // 1) remove old tools
         for (const name of registered[server] ?? []) {
             agent.removeTool(name)
+            allNamespacedTools.delete(name)
         }
         registered[server] = []
 

Original file line number	Diff line number	Diff line change
`@@ -91,6 +91,7 @@ export const McpToolsServer: Server = ({ credentialsProvider, workspace, logging`
`91`	`91`	`// 1) remove old tools`
`92`	`92`	`for (const name of registered[server] ?? []) {`
`93`	`93`	`agent.removeTool(name)`
	`94`	`+ allNamespacedTools.delete(name)`
`94`	`95`	`}`
`95`	`96`	`registered[server] = []`
`96`	`97`