fix: improve error message for updating tool settings

daniel-lxs · daniel-lxs · commit 0d2947c93114 · 2025-06-20T16:53:12.000-05:00
diff --git a/src/services/mcp/McpHub.ts b/src/services/mcp/McpHub.ts
@@ -1604,7 +1604,7 @@ export class McpHub {
 			const addToolToDisabledList = !isEnabled
 			await this.updateServerToolList(serverName, source, toolName, "disabledTools", addToolToDisabledList)
 		} catch (error) {
-			this.showErrorMessage(`Failed to update disabledForPromptTool settings for tool ${toolName}`, error)
+			this.showErrorMessage(`Failed to update settings for tool ${toolName}`, error)
 			throw error // Re-throw to ensure the error is properly handled
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -1604,7 +1604,7 @@ export class McpHub {`
`1604`	`1604`	`const addToolToDisabledList = !isEnabled`
`1605`	`1605`	`await this.updateServerToolList(serverName, source, toolName, "disabledTools", addToolToDisabledList)`
`1606`	`1606`	`} catch (error) {`
`1607`		- this.showErrorMessage(`Failed to update disabledForPromptTool settings for tool ${toolName}`, error)
	`1607`	+ this.showErrorMessage(`Failed to update settings for tool ${toolName}`, error)
`1608`	`1608`	`throw error // Re-throw to ensure the error is properly handled`
`1609`	`1609`	`}`
`1610`	`1610`	`}`