fix: prevent disabled MCP servers from restarting due to config file watcher

- Added check in handleConfigFileChange to ignore file change events for servers with active toggle operations
- This prevents the race condition where disabling a server triggers a config write, which triggers the file watcher, which would restart the server
- Fixes the issue where disabled servers would immediately restart after being shut down

Author: hannesrudolph

src/services/mcp/McpHub.ts

@@ -315,7 +315,19 @@ export class McpHub {
 				return
 			}
 
-			await this.updateServerConnections(result.data.mcpServers || {}, source)
+			// Check if any servers have toggle operations in progress
+			const servers = result.data.mcpServers || {}
+			const hasActiveToggleOperations = Object.keys(servers).some((serverName) => {
+				const operationKey = `${serverName}-${source}`
+				return this.toggleOperations.has(operationKey)
+			})
+
+			if (hasActiveToggleOperations) {
+				console.log(`Skipping config file change for ${source} - toggle operations in progress`)
+				return
+			}
+
+			await this.updateServerConnections(servers, source)
 		} catch (error) {
 			// Check if the error is because the file doesn't exist
 			if (error.code === "ENOENT" && source === "project") {