Skip to content

Handle native telemetry through the standard LSP channel - #14777

Merged
Colen Garoutte-Carson (Colengms) merged 1 commit into
mainfrom
dev/coleng/lsp_telemetry_msg
Sep 17, 2026
Merged

Colen Garoutte-Carson (Colengms) merged 1 commit into
mainfrom
dev/coleng/lsp_telemetry_msg

Conversation

@Colengms

Copy link
Copy Markdown
Contributor

Updates the extension for the native server's migration from the custom cpptools/logTelemetry notification to the standard LSP telemetry/event notification.

The client now subscribes through LanguageClient.onTelemetry and continues routing each payload through the existing DefaultClient.logTelemetry path. Configuration-provider enrichment, large-workspace warnings, event naming, and telemetry upload behavior remain unchanged.

This change must ship with the corresponding native update.

Validation:

  • Ran npm run compile.

@github-project-automation github-project-automation Bot moved this to Pull Request in cpptools Sep 17, 2026
@Colengms
Colen Garoutte-Carson (Colengms) marked this pull request as ready for review September 17, 2026 02:22
@Colengms
Colen Garoutte-Carson (Colengms) merged commit 874d326 into main Sep 17, 2026
6 checks passed
@Colengms
Colen Garoutte-Carson (Colengms) deleted the dev/coleng/lsp_telemetry_msg branch September 17, 2026 18:08
@github-project-automation github-project-automation Bot moved this from Pull Request to Done in cpptools Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants