Allows MCP banner to show on all hosts

Author: d13

src/plus/gk/utils/-webview/mcp.utils.ts

@@ -1,29 +1,16 @@
 import { lm, version } from 'vscode';
-import { getPlatform, isWeb } from '@env/platform';
+import { isWeb } from '@env/platform';
 import type { Container } from '../../../../container';
 import { configuration } from '../../../../system/-webview/configuration';
-import { getHostAppName } from '../../../../system/-webview/vscode';
 import { satisfies } from '../../../../system/version';
 
-export async function isMcpBannerEnabled(container: Container, showAutoRegistration = false): Promise<boolean> {
+export function isMcpBannerEnabled(container: Container, showAutoRegistration = false): boolean {
 	// Check if running on web or automatically registrable
 	if (isWeb || (!showAutoRegistration && mcpExtensionRegistrationAllowed())) {
 		return false;
 	}
 
-	// Check platform
-	const platform = getPlatform();
-	if (platform !== 'windows' && platform !== 'macOS' && platform !== 'linux') {
-		return false;
-	}
-
-	if (container.storage.get('mcp:banner:dismissed', false)) return false;
-
-	// Check host app
-	const hostAppName = await getHostAppName();
-	const supportedApps = ['code', 'code-insiders', 'cursor', 'windsurf'];
-
-	return hostAppName != null && supportedApps.includes(hostAppName);
+	return !container.storage.get('mcp:banner:dismissed', false);
 }
 
 export function supportsMcpExtensionRegistration(): boolean {

src/webviews/home/homeWebview.ts

@@ -246,7 +246,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 
 	private onStorageChanged(e: StorageChangeEvent) {
 		if (!e.workspace && e.keys.includes('mcp:banner:dismissed')) {
-			void this.onMcpBannerChanged();
+			this.onMcpBannerChanged();
 		}
 	}
 
@@ -802,8 +802,8 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 		return this.container.storage.get('home:sections:collapsed')?.includes('feb2025AmaBanner') ?? false;
 	}
 
-	private async getMcpBannerCollapsed() {
-		return !(await isMcpBannerEnabled(this.container, true));
+	private getMcpBannerCollapsed() {
+		return !isMcpBannerEnabled(this.container, true);
 	}
 
 	private getMcpCanAutoRegister() {
@@ -901,7 +901,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 			previewEnabled: this.getPreviewEnabled(),
 			newInstall: getContext('gitlens:install:new', false),
 			amaBannerCollapsed: this.getAmaBannerCollapsed(),
-			mcpBannerCollapsed: await this.getMcpBannerCollapsed(),
+			mcpBannerCollapsed: this.getMcpBannerCollapsed(),
 			mcpCanAutoRegister: this.getMcpCanAutoRegister(),
 		};
 	}
@@ -1136,11 +1136,11 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 		void this.host.notify(DidChangeAiAllAccessBanner, await this.getAiAllAccessBannerCollapsed());
 	}
 
-	private async onMcpBannerChanged() {
+	private onMcpBannerChanged() {
 		if (!this.host.visible) return;
 
 		void this.host.notify(DidChangeMcpBanner, {
-			mcpBannerCollapsed: await this.getMcpBannerCollapsed(),
+			mcpBannerCollapsed: this.getMcpBannerCollapsed(),
 			mcpCanAutoRegister: this.getMcpCanAutoRegister(),
 		});
 	}

src/webviews/plus/graph/graphWebview.ts

@@ -1040,18 +1040,18 @@ export class GraphWebviewProvider implements WebviewProvider<State, State, Graph
 
 	private onStorageChanged(e: StorageChangeEvent) {
 		if (!e.workspace && e.keys.includes('mcp:banner:dismissed')) {
-			void this.onMcpBannerChanged();
+			this.onMcpBannerChanged();
 		}
 	}
 
-	private async onMcpBannerChanged() {
+	private onMcpBannerChanged() {
 		if (!this.host.visible) return;
 
-		void this.host.notify(DidChangeMcpBanner, await this.getMcpBannerCollapsed());
+		void this.host.notify(DidChangeMcpBanner, this.getMcpBannerCollapsed());
 	}
 
-	private async getMcpBannerCollapsed() {
-		return !(await isMcpBannerEnabled(this.container));
+	private getMcpBannerCollapsed() {
+		return !isMcpBannerEnabled(this.container);
 	}
 
 	private onThemeChanged(theme: ColorTheme) {
@@ -2792,7 +2792,7 @@ export class GraphWebviewProvider implements WebviewProvider<State, State, Graph
 			useNaturalLanguageSearch: useNaturalLanguageSearch,
 			featurePreview: featurePreview,
 			orgSettings: this.getOrgSettings(),
-			mcpBannerCollapsed: await this.getMcpBannerCollapsed(),
+			mcpBannerCollapsed: this.getMcpBannerCollapsed(),
 		};
 	}