Skip to content

Commit 8584723

Browse files
authored
fix: fix for mcp tool resgistration on smus (#2524)
1 parent 1eb3910 commit 8584723

File tree

3 files changed

+58
-59
lines changed

3 files changed

+58
-59
lines changed

package-lock.json

Lines changed: 49 additions & 54 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

server/aws-lsp-codewhisperer/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
"@aws/chat-client-ui-types": "0.1.68",
4040
"@aws/language-server-runtimes": "0.3.6",
4141
"@aws/lsp-core": "^0.0.17",
42-
"@modelcontextprotocol/sdk": "^1.15.0",
42+
"@modelcontextprotocol/sdk": "1.19.1",
4343
"@smithy/node-http-handler": "^2.5.0",
4444
"adm-zip": "^0.5.10",
4545
"archiver": "^7.0.1",

server/aws-lsp-codewhisperer/src/language-server/agenticChat/tools/toolServer.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -382,10 +382,6 @@ export const McpToolsServer: Server = ({
382382
registerServerTools(server, defs)
383383
}
384384

385-
McpManager.instance.events.on(AGENT_TOOLS_CHANGED, (server: string, defs: McpToolDefinition[]) => {
386-
registerServerTools(server, defs)
387-
})
388-
389385
// Emit metrics after tools are registered
390386
McpManager.instance.emitMcpConfigMetrics()
391387
}
@@ -466,6 +462,14 @@ export const McpToolsServer: Server = ({
466462
// Wait for auth to be initialized before discovering servers
467463
const waitForAuthAndDiscover = async () => {
468464
try {
465+
// Set up event listener for tool changes during server initialization
466+
McpManager.instance.events.on(AGENT_TOOLS_CHANGED, (server: string, defs: McpToolDefinition[]) => {
467+
if (!ProfileStatusMonitor.getMcpState()) {
468+
return
469+
}
470+
registerServerTools(server, defs)
471+
})
472+
469473
const serviceManager = AmazonQTokenServiceManager.getInstance()
470474
const authState = serviceManager.getState()
471475

0 commit comments

Comments
 (0)