Skip to content

Add packageExtensions workaround for @vscode/extension-telemetry missing deps#10453

Open
Copilot wants to merge 1 commit intomainfrom
copilot/fix-vscode-telemetry-dependency
Open

Add packageExtensions workaround for @vscode/extension-telemetry missing deps#10453
Copilot wants to merge 1 commit intomainfrom
copilot/fix-vscode-telemetry-dependency

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 22, 2026

@vscode/extension-telemetry@1.5.1 implicitly depends on @microsoft/applicationinsights-common via transitive dep through @microsoft/applicationinsights-channel-js. Version 3.4.x of that package dropped the dep, breaking bundling with:

Could not resolve "@microsoft/applicationinsights-common"

Fixed upstream in vscode-extension-telemetry#245 (v1.5.2) but not yet released to npm.

  • Added packageExtensions in pnpm-workspace.yaml to explicitly declare @microsoft/applicationinsights-common and @microsoft/applicationinsights-core-js as dependencies of @vscode/extension-telemetry@<=1.5.1
  • Scoped to <=1.5.1 so it auto-stops applying once the fixed version is available and adopted

Remove the packageExtensions block once @vscode/extension-telemetry@>=1.5.2 is published and the catalog is bumped.

@vscode/extension-telemetry@1.5.1 implicitly depends on
@microsoft/applicationinsights-common and @microsoft/applicationinsights-core-js
via transitive deps, but @microsoft/applicationinsights-channel-js@3.4.x dropped
the @microsoft/applicationinsights-common dependency.
Fixed in vscode-extension-telemetry#245 (v1.5.2) but not yet released.

See: microsoft/vscode-extension-telemetry#244

Agent-Logs-Url: https://github.com/microsoft/typespec/sessions/be902a59-d20d-4c4b-bf6b-c5b2df9e44a4

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@timotheeguerin timotheeguerin marked this pull request as ready for review April 22, 2026 12:42
@azure-sdk
Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants