Skip to content

Commit 7e7e28f

Browse files
committed
chore: remove applicationinsights-web-basic override after upstream fix (fixes #614)
@microsoft/applicationinsights-web-basic was pinned to ~3.3.4 to work around @vscode/extension-telemetry not declaring applicationinsights-common as a direct dependency. The fix shipped in extension-telemetry@1.5.2 (microsoft/vscode-extension-telemetry#245). vscode-azext-utils@4.1.x now depends on extension-telemetry@^1.5.2, so the override is no longer needed. Regenerated package-lock.json to collapse the previously nested duplicate AI packages, recovering ~100-120 KB from the extension bundle.
1 parent 61f43a6 commit 7e7e28f

2 files changed

Lines changed: 25 additions & 58 deletions

File tree

package-lock.json

Lines changed: 23 additions & 55 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,11 +206,10 @@
206206
"vscode-uri": "~3.1.0",
207207
"zod": "~4.3.6"
208208
},
209-
"//overrides": "Pin transitive dependencies: glob 12 for latest, test-exclude 7 for compatibility, applicationinsights-web-basic 3.3.x to retain applicationinsights-common needed by @vscode/extension-telemetry (see https://github.com/microsoft/vscode-extension-telemetry/issues/244, fixed in 1.5.2 but not yet released)",
209+
"//overrides": "Pin transitive dependencies: glob 12.0.x for latest, test-exclude 7 for compatibility.",
210210
"overrides": {
211211
"glob": "~12.0.0",
212-
"test-exclude": "~7.0.1",
213-
"@microsoft/applicationinsights-web-basic": "~3.3.4"
212+
"test-exclude": "~7.0.1"
214213
},
215214
"extensionDependencies": [],
216215
"contributes": {

0 commit comments

Comments
 (0)