Pause telemetry on metered connections

[Dmitriy Vasyura (dmitrivMS)]

• propagate the renderer's initial metered-connection state to the main and shared processes
• pause workbench, main-process, shared-process, and custom-endpoint telemetry while the connection is metered
• suspend queued 1DS transport work while metered and resume it when the connection becomes unmetered
• skip shared-process and custom-endpoint shutdown flushes while metered
• prevent experiment-buffered telemetry from flushing after the connection becomes metered
• add regression coverage for initial-state propagation, telemetry IPC, transport pause/resume, and metered shutdown

Fixes #297362

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

Robo (@deepak1556)

Matched files:

• src/vs/code/electron-main/app.ts
• src/vs/code/electron-utility/sharedProcess/sharedProcessMain.ts

[Copilot]

Pull request overview

Pauses telemetry event submission when VS Code detects a metered connection.

Changes:

• Propagates initial metered state across Electron processes.
• Gates standard and custom-endpoint telemetry.
• Drops experiment-buffered events when flushing while metered.

Show a summary per file

File	Description
src/vs/workbench/services/telemetry/electron-browser/telemetryService.ts	Passes metered state to workbench telemetry.
src/vs/platform/telemetry/test/browser/telemetryService.test.ts	Tests dropping direct and buffered events.
src/vs/platform/telemetry/node/customEndpointTelemetryService.ts	Gates custom-endpoint telemetry.
src/vs/platform/telemetry/common/telemetryService.ts	Prevents buffered-event flushing while metered.
src/vs/platform/meteredConnection/test/electron-browser/meteredConnectionService.test.ts	Tests initial state propagation.
src/vs/platform/meteredConnection/electron-browser/meteredConnectionService.ts	Sends initial browser state to the main process.
src/vs/code/electron-utility/sharedProcess/sharedProcessMain.ts	Wires metered state into shared-process telemetry.
src/vs/code/electron-main/app.ts	Wires metered state into main-process telemetry.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

• Files reviewed: 8/8 changed files
• Comments generated: 3
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 14/14 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[Mohammad javad Dianat (dianatofficial)]

LGTM! Code changes are clean and well-structured.

[Dmitriy Vasyura (dmitrivMS)]

Superseded by #331897, which now includes these desktop telemetry fixes in 2f9e41f and includes Fixes #297362.

The integration adapts the fixes to native whenInitialized readiness, covers real 1DS batching/retries and metered shutdown, forwards state to debugger telemetry children without waking idle processes, and fixes the known-metered experiment-buffer edge case. The old renderer detector changes are superseded by native detection.

Local validation passed: 81 focused tests, 15 telemetry compatibility tests, client type-check, and hygiene/ESLint. Additional Agent Host, CLI, and browser collectors are tracked separately in #336679.

Closing this PR in favor of the consolidated implementation; the source branch is retained.

[Dmitriy Vasyura (dmitrivMS)]

The combined work has now been split for review. The replacement for this PR is #336922, stacked directly on the native metered service PR #331897. #336922 owns the telemetry implementation and the closing link for #297362; #331897 intentionally has no telemetry-metered coupling until that child is applied. This superseded PR remains closed and its branch is retained.