fix(mcp): Revert changes causing startup issues and temporarily disable notifications

- Reverted PR #6779 which prevented unnecessary MCP server refreshes but caused startup failures
- Temporarily disabled MCP notification popups as a stopgap solution
- Added TODO comments explaining the temporary nature of disabled notifications
- This allows MCP servers to function properly while a more robust solution is developed

Author: hannesrudolph

src/services/mcp/McpHub.ts

@@ -1212,7 +1212,10 @@ export class McpHub {
 
 	public async refreshAllConnections(): Promise<void> {
 		if (this.isConnecting) {
-			vscode.window.showInformationMessage(t("mcp:info.already_refreshing"))
+			// TODO: This notification is temporarily disabled as a stopgap until we can separate
+			// MCP settings from regular settings, allowing them to be saved independently without
+			// triggering unnecessary MCP server refreshes
+			// Disabled notification: vscode.window.showInformationMessage(t("mcp:info.already_refreshing"))
 			return
 		}
 
@@ -1234,7 +1237,10 @@ export class McpHub {
 		}
 
 		this.isConnecting = true
-		vscode.window.showInformationMessage(t("mcp:info.refreshing_all"))
+		// TODO: This notification is temporarily disabled as a stopgap until we can separate
+		// MCP settings from regular settings, allowing them to be saved independently without
+		// triggering unnecessary MCP server refreshes
+		// Disabled notification: vscode.window.showInformationMessage(t("mcp:info.refreshing_all"))
 
 		try {
 			const globalPath = await this.getMcpSettingsFilePath()
@@ -1244,11 +1250,14 @@ export class McpHub {
 				const globalConfig = JSON.parse(globalContent)
 				globalServers = globalConfig.mcpServers || {}
 				const globalServerNames = Object.keys(globalServers)
-				vscode.window.showInformationMessage(
-					t("mcp:info.global_servers_active", {
-						mcpServers: `${globalServerNames.join(", ") || "none"}`,
-					}),
-				)
+				// TODO: This notification is temporarily disabled as a stopgap until we can separate
+				// MCP settings from regular settings, allowing them to be saved independently without
+				// triggering unnecessary MCP server refreshes
+				// Disabled notification: vscode.window.showInformationMessage(
+				// 	t("mcp:info.global_servers_active", {
+				// 		mcpServers: `${globalServerNames.join(", ") || "none"}`,
+				// 	}),
+				// )
 			} catch (error) {
 				console.log("Error reading global MCP config:", error)
 			}
@@ -1261,11 +1270,14 @@ export class McpHub {
 					const projectConfig = JSON.parse(projectContent)
 					projectServers = projectConfig.mcpServers || {}
 					const projectServerNames = Object.keys(projectServers)
-					vscode.window.showInformationMessage(
-						t("mcp:info.project_servers_active", {
-							mcpServers: `${projectServerNames.join(", ") || "none"}`,
-						}),
-					)
+					// TODO: This notification is temporarily disabled as a stopgap until we can separate
+					// MCP settings from regular settings, allowing them to be saved independently without
+					// triggering unnecessary MCP server refreshes
+					// Disabled notification: vscode.window.showInformationMessage(
+					// 	t("mcp:info.project_servers_active", {
+					// 		mcpServers: `${projectServerNames.join(", ") || "none"}`,
+					// 	}),
+					// )
 				} catch (error) {
 					console.log("Error reading project MCP config:", error)
 				}
@@ -1286,7 +1298,10 @@ export class McpHub {
 
 			await this.notifyWebviewOfServerChanges()
 
-			vscode.window.showInformationMessage(t("mcp:info.all_refreshed"))
+			// TODO: This notification is temporarily disabled as a stopgap until we can separate
+			// MCP settings from regular settings, allowing them to be saved independently without
+			// triggering unnecessary MCP server refreshes
+			// Disabled notification: vscode.window.showInformationMessage(t("mcp:info.all_refreshed"))
 		} catch (error) {
 			this.showErrorMessage("Failed to refresh MCP servers", error)
 		} finally {