Skip to content

Remove language server telemetry - #20205

Merged
Shenglong Li (shenglol) merged 1 commit into
mainfrom
shenglol/clean-up-lang-server-telemetry
Aug 20, 2026
Merged

Remove language server telemetry#20205
Shenglong Li (shenglol) merged 1 commit into
mainfrom
shenglol/clean-up-lang-server-telemetry

Conversation

@shenglol

@shenglol Shenglong Li (shenglol) commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove telemetry events, providers, handlers, and dependency injection from Bicep.LangServer
  • retain user-facing command error handling through a telemetry-free helper
  • remove telemetry-only protocol fields, tests, mocks, assertions, and completion baseline data

This follows the removal of telemetry from vscode-bicep and eliminates the now-unused server-side telemetry path.

Validation

  • dotnet build src\Bicep.LangServer\Bicep.LangServer.csproj --no-restore
  • dotnet build src\Bicep.LangServer.UnitTests\Bicep.LangServer.UnitTests.csproj --no-restore
  • 358 targeted language server unit/integration tests
  • 183 completion integration tests
  • npm run build in src\vscode-bicep
Microsoft Reviewers: Open in CodeFlow

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: eb4fa8c4-db14-4bb8-8f01-9e644cd17357
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Test this change out locally with the following install scripts (Action run 32319173715)

VSCode
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 32319173715
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 32319173715"
Azure CLI
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 32319173715
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 32319173715"

@shenglol
Shenglong Li (shenglol) merged commit f8197cc into main Aug 20, 2026
45 of 46 checks passed
@shenglol
Shenglong Li (shenglol) deleted the shenglol/clean-up-lang-server-telemetry branch August 20, 2026 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants