chore: should switch connection if a server is running

Author: himanshusinghs

src/mcp/mcpController.ts

@@ -336,26 +336,25 @@ ${jsonConfig}`,
   private async onActiveConnectionChanged(): Promise<void> {
     const autoStartConfig = this.getMCPAutoStartConfig();
 
+    const shouldSendMCPServerNotification =
+      this.connectionController.getActiveConnectionId() !== null &&
+      (autoStartConfig === 'Ask' ||
+        autoStartConfig === null ||
+        autoStartConfig === undefined);
+
     logger.info('Active connection changed, should update MCP server', {
       'mdb.mcp.server': autoStartConfig,
       serverStarted: !!this.server,
+      shouldSendMCPServerNotification,
     });
 
-    if (
-      this.connectionController.getActiveConnectionId() &&
-      (autoStartConfig === 'Ask' ||
-        autoStartConfig === null ||
-        autoStartConfig === undefined)
-    ) {
+    if (shouldSendMCPServerNotification) {
       await this.requestAutoStartPermission();
-      return;
     }
 
-    if (!this.server) {
-      return;
+    if (this.server) {
+      await this.switchConnectionManagerToCurrentConnection();
     }
-
-    await this.switchConnectionManagerToCurrentConnection();
   }
 
   private async switchConnectionManagerToCurrentConnection(): Promise<void> {