Fixes #4701 guards against missing config props

Author: eamodio

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

@@ -95,26 +95,32 @@ export class GkMcpProvider implements McpServerDefinitionProvider, Disposable {
 			return undefined;
 		}
 
-		let output = await runCLICommand(['mcp', 'config', appName, '--source=gitlens', `--scheme=${env.uriScheme}`], {
-			cwd: cliPath,
-		});
-		output = output.replace(CLIProxyMCPConfigOutputs.checkingForUpdates, '').trim();
-
 		try {
+			let output = await runCLICommand(
+				['mcp', 'config', appName, '--source=gitlens', `--scheme=${env.uriScheme}`],
+				{
+					cwd: cliPath,
+				},
+			);
+			output = output.replace(CLIProxyMCPConfigOutputs.checkingForUpdates, '').trim();
+
 			const config: McpConfiguration = JSON.parse(output);
+			if (!config.type || !config.command || !Array.isArray(config.args)) {
+				throw new Error(`Invalid MCP configuration: missing required properties (${output})`);
+			}
 
 			this.onRegistrationCompleted(cliInstall.version);
 
 			return {
-				name: config.name,
+				name: config.name ?? 'GitKraken',
 				type: config.type,
 				command: config.command,
 				args: config.args,
 				version: cliInstall.version,
 			};
 		} catch (ex) {
 			Logger.error(`Error getting MCP configuration: ${ex}`);
-			this.onRegistrationFailed('Error getting MCP configuration', undefined, cliInstall.version);
+			this.onRegistrationFailed('Error getting MCP configuration', String(ex), cliInstall.version);
 		}
 
 		return undefined;