Track all tab switches in posthog

Author: mrubens

packages/telemetry/src/TelemetryService.ts

@@ -153,10 +153,11 @@ export class TelemetryService {
 	}
 
 	/**
-	 * Captures when the ModesView settings UI is shown
+	 * Captures when a tab is shown due to user action
+	 * @param tab The tab that was shown
 	 */
-	public captureModesViewShown(): void {
-		this.captureEvent(TelemetryEventName.MODE_SETTINGS_SHOWN)
+	public captureTabShown(tab: string): void {
+		this.captureEvent(TelemetryEventName.TAB_SHOWN, { tab })
 	}
 
 	/**

packages/types/src/telemetry.ts

@@ -31,7 +31,7 @@ export enum TelemetryEventName {
 	CHECKPOINT_RESTORED = "Checkpoint Restored",
 	CHECKPOINT_DIFFED = "Checkpoint Diffed",
 
-	MODE_SETTINGS_SHOWN = "Mode Settings Shown",
+	TAB_SHOWN = "Tab Shown",
 	MODE_SETTINGS_CHANGED = "Mode Setting Changed",
 	CUSTOM_MODE_CREATED = "Custom Mode Created",
 
@@ -123,7 +123,7 @@ export const rooCodeTelemetryEventSchema = z.discriminatedUnion("type", [
 			TelemetryEventName.CONSECUTIVE_MISTAKE_ERROR,
 			TelemetryEventName.CONTEXT_CONDENSED,
 			TelemetryEventName.SLIDING_WINDOW_TRUNCATION,
-			TelemetryEventName.MODE_SETTINGS_SHOWN,
+			TelemetryEventName.TAB_SHOWN,
 			TelemetryEventName.MODE_SETTINGS_CHANGED,
 			TelemetryEventName.CUSTOM_MODE_CREATED,
 		]),

src/core/webview/webviewMessageHandler.ts

@@ -1703,9 +1703,9 @@ export const webviewMessageHandler = async (
 
 		case "switchTab": {
 			if (message.tab) {
-				// This could be more generic, but keeping it specific for now
-				if (message.tab === "modes" && TelemetryService.hasInstance()) {
-					TelemetryService.instance.captureModesViewShown()
+				// Capture tab shown event for all switchTab messages (which are user-initiated)
+				if (TelemetryService.hasInstance()) {
+					TelemetryService.instance.captureTabShown(message.tab)
 				}
 
 				await provider.postMessageToWebview({ type: "action", action: "switchTab", tab: message.tab })