Skip to content

Commit 4fda4bf

Browse files
committed
fix: don't show erros on mcp server list page
1 parent 4d776e0 commit 4fda4bf

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -205,8 +205,9 @@ export class McpEventHandler {
205205
const header = {
206206
title: 'MCP Servers',
207207
description: "Add MCP servers to extend Q's capabilities.",
208-
status: combinedErrors
209-
? { title: combinedErrors, icon: 'cancel-circle', status: 'error' as Status }
208+
// only show error on list mcp server page if unable to read mcp.json file
209+
status: configLoadErrors
210+
? { title: configLoadErrors, icon: 'cancel-circle', status: 'error' as Status }
210211
: undefined,
211212
}
212213

@@ -226,7 +227,6 @@ export class McpEventHandler {
226227
'save-mcp': () => this.#handleSaveMcp(params),
227228
'open-mcp-server': () => this.#handleOpenMcpServer(params),
228229
'edit-mcp': () => this.#handleEditMcpServer(params),
229-
230230
'mcp-permission-change': () => this.#handleMcpPermissionChange(params),
231231
'refresh-mcp-list': () => this.#handleRefreshMCPList(params),
232232
'mcp-enable-server': () => this.#handleEnableMcpServer(params),

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

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -733,11 +733,6 @@ export class McpManager {
733733
if (server) {
734734
this.setState(server, McpServerStatus.FAILED, 0, msg)
735735
this.emitToolsChanged(server)
736-
737-
// Store the error in the configLoadErrors map
738-
if (server !== undefined) {
739-
this.configLoadErrors.set(server, msg)
740-
}
741736
}
742737
}
743738

0 commit comments

Comments
 (0)