fix: disable MCP auto start and instead request on first load and new connection attempts VSCODE-704 (#1132)

Author: himanshusinghs

.gitignore

@@ -11,3 +11,5 @@ constants.json
 .eslintcache
 .sbom
 src/test/ai-accuracy-tests/test-results.html
+.yalc
+yalc.lock

package-lock.json

@@ -1320,7 +1320,7 @@
         },
         "mdb.mcp.readOnly": {
           "type": "boolean",
-          "default": false,
+          "default": true,
           "description": "When set to true, MongoDB MCP server will only allow read, connect, and metadata operation types, disabling create/update/delete operations."
         },
         "mdb.mcp.indexCheck": {
@@ -1331,11 +1331,16 @@
         "mdb.mcp.server": {
           "type": "string",
           "enum": [
-            "ask",
-            "enabled",
-            "disabled"
+            "prompt",
+            "autoStartEnabled",
+            "autoStartDisabled"
+          ],
+          "enumItemLabels": [
+            "Ask",
+            "Auto Start Enabled",
+            "Auto Start Disabled"
           ],
-          "default": "ask",
+          "default": "prompt",
           "description": "Controls whether MongoDB MCP server starts automatically with the extension and connects to the active connection. If automatic startup is disabled, the server can still be started using the 'MongoDB: Start MCP Server' command."
         },
         "mdb.mcp.exportsPath": {
@@ -1418,7 +1423,7 @@
     "mongodb-connection-string-url": "^3.0.2",
     "mongodb-data-service": "^22.30.1",
     "mongodb-log-writer": "^2.4.1",
-    "mongodb-mcp-server": "^1.0.2-prerelease.1",
+    "mongodb-mcp-server": "^1.0.2-prerelease.2",
     "mongodb-query-parser": "^4.4.2",
     "mongodb-schema": "^12.6.2",
     "node-machine-id": "1.1.12",
@@ -1435,6 +1440,7 @@
   "devDependencies": {
     "@babel/preset-typescript": "^7.25.7",
     "@babel/types": "^7.25.8",
+    "@modelcontextprotocol/sdk": "^1.18.2",
     "@mongodb-js/oidc-mock-provider": "^0.11.4",
     "@mongodb-js/oidc-plugin": "^2.0.3",
     "@mongodb-js/prettier-config-devtools": "^1.0.2",

package.json

@@ -732,7 +732,9 @@ export default class ConnectionController {
       return false;
     }
 
-    const originalDisconnect = this._activeDataService.disconnect.bind(this);
+    const originalDisconnect = this._activeDataService.disconnect.bind(
+      this._activeDataService,
+    );
     this._activeDataService = null;
 
     try {

src/connectionController.ts

@@ -41,7 +41,7 @@ export class MCPConnectionManager extends ConnectionManager {
     this.getTelemetryAnonymousId = getTelemetryAnonymousId;
   }
 
-  connect(): Promise<AnyConnectionState> {
+  override connect(): Promise<AnyConnectionState> {
     return Promise.reject(
       new Error(
         // eslint-disable-next-line no-multi-str
@@ -83,7 +83,7 @@ To connect, choose a connection from MongoDB VSCode extensions's sidepanel - htt
     }
   }
 
-  async disconnect(): Promise<ConnectionStateDisconnected> {
+  override async disconnect(): Promise<ConnectionStateDisconnected> {
     try {
       await this.activeConnection?.provider?.close();
     } catch (error) {
@@ -100,6 +100,11 @@ To connect, choose a connection from MongoDB VSCode extensions's sidepanel - htt
     });
   }
 
+  override async close(): Promise<void> {
+    await this.disconnect();
+    this._events.emit('close', this.currentConnectionState);
+  }
+
   async updateConnection(
     connectParams: MCPConnectParams | undefined,
   ): Promise<void> {