Removes Walkthroug from Home on Cursor

(#3837, #4390)

Author: sergeibbb

src/container.ts

@@ -59,7 +59,7 @@ import { log } from './system/decorators/log';
 import { Logger } from './system/logger';
 import { TelemetryService } from './telemetry/telemetry';
 import { UsageTracker } from './telemetry/usageTracker';
-import { WalkthroughStateProvider } from './telemetry/walkthroughStateProvider';
+import { isWalkthroughSupported, WalkthroughStateProvider } from './telemetry/walkthroughStateProvider';
 import { GitTerminalLinkProvider } from './terminal/linkProvider';
 import { GitDocumentTracker } from './trackers/documentTracker';
 import { LineTracker } from './trackers/lineTracker';
@@ -205,7 +205,9 @@ export class Container {
 		);
 		this._disposables.push((this._uri = new UriService(this)));
 		this._disposables.push((this._subscription = new SubscriptionService(this, this._connection, previousVersion)));
-		this._disposables.push((this._walkthrough = new WalkthroughStateProvider(this)));
+		if (isWalkthroughSupported()) {
+			this._disposables.push((this._walkthrough = new WalkthroughStateProvider(this)));
+		}
 		this._disposables.push((this._organizations = new OrganizationService(this, this._connection)));
 
 		this._disposables.push((this._git = new GitProviderService(this)));
@@ -713,8 +715,8 @@ export class Container {
 		return this._usage;
 	}
 
-	private readonly _walkthrough: WalkthroughStateProvider;
-	get walkthrough(): WalkthroughStateProvider {
+	private readonly _walkthrough: WalkthroughStateProvider | undefined;
+	get walkthrough(): WalkthroughStateProvider | undefined {
 		return this._walkthrough;
 	}
 

src/system/-webview/cursor.ts

@@ -0,0 +1,9 @@
+import { env } from 'vscode';
+
+/**
+ * Checks if the current IDE is Cursor
+ * @returns true if the current IDE is Cursor, false otherwise
+ */
+export function isCursor(): boolean {
+	return env.appName === 'Cursor';
+}

src/telemetry/walkthroughStateProvider.ts

@@ -5,6 +5,7 @@ import type { TrackedUsageKeys } from '../constants.telemetry';
 import type { Container } from '../container';
 import type { SubscriptionChangeEvent } from '../plus/gk/subscriptionService';
 import { setContext } from '../system/-webview/context';
+import { isCursor } from '../system/-webview/cursor';
 import { wait } from '../system/promise';
 import type { UsageChangeEvent } from './usageTracker';
 
@@ -319,3 +320,7 @@ export class WalkthroughStateProvider implements Disposable {
 		return true;
 	}
 }
+
+export function isWalkthroughSupported(): boolean {
+	return !isCursor();
+}

src/webviews/home/homeWebview.ts

@@ -170,7 +170,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 			this.container.subscription.onDidChange(this.onSubscriptionChanged, this),
 			onDidChangeContext(this.onContextChanged, this),
 			this.container.integrations.onDidChange(this.onIntegrationsChanged, this),
-			this.container.walkthrough.onDidChangeProgress(this.onWalkthroughProgressChanged, this),
+			this.container.walkthrough?.onDidChangeProgress(this.onWalkthroughProgressChanged, this) ?? emptyDisposable,
 			configuration.onDidChange(this.onDidChangeConfig, this),
 			this.container.launchpad.onDidChange(this.onLaunchpadChanged, this),
 			this.container.ai.onDidChangeModel(this.onAIModelChanged, this),
@@ -729,7 +729,9 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 	}
 
 	private getWalkthroughDismissed() {
-		return this.container.storage.get('home:walkthrough:dismissed') ?? false;
+		return (
+			this.container.walkthrough == null || (this.container.storage.get('home:walkthrough:dismissed') ?? false)
+		);
 	}
 
 	private getPreviewCollapsed() {
@@ -806,13 +808,14 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 			integrations: integrations,
 			ai: ai,
 			hasAnyIntegrationConnected: anyConnected,
-			walkthroughProgress: !this.getWalkthroughDismissed()
-				? {
-						allCount: this.container.walkthrough.walkthroughSize,
-						doneCount: this.container.walkthrough.doneCount,
-						progress: this.container.walkthrough.progress,
-				  }
-				: undefined,
+			walkthroughProgress:
+				!this.getWalkthroughDismissed() && this.container.walkthrough != null
+					? {
+							allCount: this.container.walkthrough.walkthroughSize,
+							doneCount: this.container.walkthrough.doneCount,
+							progress: this.container.walkthrough.progress,
+					  }
+					: undefined,
 			previewEnabled: this.getPreviewEnabled(),
 			newInstall: getContext('gitlens:install:new', false),
 			amaBannerCollapsed: this.getAmaBannerCollapsed(),
@@ -1205,6 +1208,8 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 	}
 
 	private notifyDidChangeProgress() {
+		if (this.container.walkthrough == null) return;
+
 		void this.host.notify(DidChangeWalkthroughProgress, {
 			allCount: this.container.walkthrough.walkthroughSize,
 			doneCount: this.container.walkthrough.doneCount,