fix(codecov): cover telemetry guard branch, exempt CT-only fixtures

Author: edelauna

codecov.yml

@@ -42,6 +42,14 @@ flag_management:
         - packages/core/src/
       carryforward: true
 
+ignore:
+  # Playwright CT-only fixtures/helpers: exercised by the webview-ui-ct flag's browser run, not
+  # Vitest, and excluded from that flag's own lcov by playwright-ct.config.ts's sourceFilter (same
+  # ".visual." match). Without this, patch coverage sees 0% for these paths on any PR that adds or
+  # touches one, since no flag's lcov contains them.
+  - "webview-ui/src/**/*.visual.fixture.tsx"
+  - "webview-ui/src/**/*.visual.i18n.ts"
+
 component_management:
   individual_components:
     - component_id: webview_components

src/core/webview/__tests__/webviewMessageHandler.spec.ts

@@ -1929,4 +1929,34 @@ describe("webviewMessageHandler - telemetrySetting", () => {
 		const calls = vi.mocked(TelemetryService.instance.updateTelemetryState).mock.calls
 		expect(calls.at(-1)).toEqual([false])
 	})
+
+	// CodeRabbit finding: webviewDidLaunch's queued telemetry update called
+	// TelemetryService.instance directly, unlike the "telemetrySetting" case a few lines
+	// below which checks hasInstance() first. If webviewDidLaunch fires before the service
+	// is created (e.g. during activation), TelemetryService.instance throws -- and since
+	// this whole chain isn't awaited by the "webviewDidLaunch" case, that throw becomes an
+	// unhandled promise rejection instead of a no-op.
+	it("does not throw or update telemetry state when webviewDidLaunch fires before TelemetryService exists", async () => {
+		const { TelemetryService } = await import("@roo-code/telemetry")
+		vi.mocked(TelemetryService.hasInstance).mockReturnValue(false)
+
+		vi.mocked(mockClineProvider.contextProxy.getValue).mockReturnValue("unset")
+		vi.mocked(mockClineProvider.customModesManager.getCustomModes).mockResolvedValue([])
+		const providerForLaunch = mockClineProvider as unknown as {
+			getMcpHub: ReturnType<typeof vi.fn>
+			providerSettingsManager: { listConfig: ReturnType<typeof vi.fn> }
+			getStateToPostToWebview: ReturnType<typeof vi.fn>
+		}
+		providerForLaunch.getMcpHub = vi.fn().mockReturnValue(undefined)
+		providerForLaunch.providerSettingsManager = { listConfig: vi.fn().mockResolvedValue(undefined) }
+		providerForLaunch.getStateToPostToWebview = vi.fn().mockResolvedValue({ telemetrySetting: "unset" })
+
+		await expect(webviewMessageHandler(mockClineProvider, { type: "webviewDidLaunch" })).resolves.not.toThrow()
+
+		// The queued telemetry update is fire-and-forget from the handler's own point of
+		// view -- flush a microtask turn so its .then() callback runs before asserting.
+		await Promise.resolve()
+
+		expect(TelemetryService.instance.updateTelemetryState).not.toHaveBeenCalled()
+	})
 })