Fix #48

Author: Tim Etchells

dev/src/microclimate/connection/Connection.ts

@@ -17,7 +17,6 @@ import { MCEndpoints, EndpointUtil } from "../../constants/Endpoints";
 import MCSocket from "./MCSocket";
 import ConnectionManager, { OnChangeCallbackArgs } from "./ConnectionManager";
 import Log from "../../Logger";
-import DebugUtils from "../project/DebugUtils";
 import Translator from "../../constants/strings/translator";
 import StringNamespaces from "../../constants/strings/StringNamespaces";
 import MCEnvironment from "./MCEnvironment";
@@ -56,7 +55,6 @@ export default class Connection implements vscode.QuickPickItem, vscode.Disposab
         this.label = Translator.t(StringNamespaces.TREEVIEW, "connectionLabel", { uri: this.mcUri });
         // this.description = this.workspacePath.fsPath.toString();
         Log.i(`Created new Connection @ ${this}, workspace ${this.workspacePath}`);
-        DebugUtils.cleanDebugLaunchConfigsFor(this);
     }
 
     public async dispose(): Promise<void> {

dev/src/microclimate/project/DebugUtils.ts

@@ -118,27 +118,27 @@ export default class DebugUtils {
         }
     }
 
-    public static async cleanDebugLaunchConfigsFor(connection: Connection): Promise<void> {
-        Log.d("Clean launch configs from " + connection.workspacePath);
-
-        // The potential names of the projects' debug configurations, whether or not they exist
-        const projectDebugNames: string[] = connection.projects.map( (project) => this.getDebugName(project));
-
-        const workspaceConfig = this.getWorkspaceConfigFor(connection);
-        const launchConfigs = this.getLaunchConfigurationsFrom(workspaceConfig);
-
-        // Loop backwards so we can remove elements
-        for (let i = launchConfigs.length - 1; i >= 0; i--) {
-            const existingLaunch: vscode.DebugConfiguration = launchConfigs[i];
-            if (!projectDebugNames.includes(existingLaunch.name)) {
-                // This launch config does not map to an existing project, so we delete it.
-                Log.i(`Delete launch config: ${existingLaunch.name}`);
-                launchConfigs.splice(i, 1);
-            }
-        }
-
-        await this.updateWorkspaceLaunchConfigs(workspaceConfig, launchConfigs);
-    }
+    // public static async cleanDebugLaunchConfigsFor(connection: Connection): Promise<void> {
+    //     Log.d("Clean launch configs from " + connection.workspacePath);
+
+    //     // The potential names of the projects' debug configurations, whether or not they exist
+    //     const projectDebugNames: string[] = connection.projects.map( (project) => this.getDebugName(project));
+
+    //     const workspaceConfig = this.getWorkspaceConfigFor(connection);
+    //     const launchConfigs = this.getLaunchConfigurationsFrom(workspaceConfig);
+
+    //     // Loop backwards so we can remove elements
+    //     for (let i = launchConfigs.length - 1; i >= 0; i--) {
+    //         const existingLaunch: vscode.DebugConfiguration = launchConfigs[i];
+    //         if (!projectDebugNames.includes(existingLaunch.name)) {
+    //             // This launch config does not map to an existing project, so we delete it.
+    //             Log.i(`Delete launch config: ${existingLaunch.name}`);
+    //             launchConfigs.splice(i, 1);
+    //         }
+    //     }
+
+    //     await this.updateWorkspaceLaunchConfigs(workspaceConfig, launchConfigs);
+    // }
 
     // keys for launch.json
     private static readonly LAUNCH: string = "launch";                      // non-nls