Uninstalls duplicate GK MCP servers

(#4626)

Author: sergeibbb

src/env/node/gk/mcp/integration.ts

@@ -109,7 +109,7 @@ export class GkMcpProvider implements McpServerDefinitionProvider, Disposable {
 				throw new Error(`Invalid MCP configuration: missing required properties (${output})`);
 			}
 
-			this.onRegistrationCompleted(cliInstall.version);
+			this.onRegistrationCompleted(cliPath, appName, cliInstall.version);
 
 			return {
 				name: config.name ?? 'GitKraken',
@@ -126,10 +126,12 @@ export class GkMcpProvider implements McpServerDefinitionProvider, Disposable {
 		return undefined;
 	}
 
-	private onRegistrationCompleted(_cliVersion?: string | undefined) {
-		if (!this.container.telemetry.enabled) return;
+	private async onRegistrationCompleted(cliPath: string, appName: string, _cliVersion?: string) {
+		if (this.container.telemetry.enabled) {
+			this.container.telemetry.setGlobalAttribute('gk.mcp.registrationCompleted', true);
+		}
 
-		this.container.telemetry.setGlobalAttribute('gk.mcp.registrationCompleted', true);
+		await this.uninstallExternalMcp(cliPath, appName);
 	}
 
 	private onRegistrationFailed(reason: string, message?: string | undefined, cliVersion?: string | undefined) {
@@ -142,4 +144,17 @@ export class GkMcpProvider implements McpServerDefinitionProvider, Disposable {
 			'cli.version': cliVersion,
 		});
 	}
+
+	@debug()
+	private async uninstallExternalMcp(cliPath: string, appName: string): Promise<void> {
+		try {
+			Logger.log(`Uninstalling external MCP configuration for ${appName}...`);
+			const output = await runCLICommand(['mcp', 'uninstall', appName], {
+				cwd: cliPath,
+			});
+			Logger.log(`External MCP configuration uninstalled successfully: ${output}`);
+		} catch (ex) {
+			Logger.warn(`Failed to uninstall external MCP configuration: ${ex}`);
+		}
+	}
 }