microsoft/vscode
Pull Request Overview
- Opened on September 18, 2026
- Status Merged
- Commit count 1 with first commit September 18, 2026
Total Delta
Open Days
Test Delta
How long has this pull request spent in each phase of its lifecycle?
| Fraction of total time | Business days | Phase |
|---|---|---|
| N/A | 2.2 days | Issue creation to work beginning |
|
|
0.0 days | Authoring 1 commit before pull request opened for review |
|
|
0.0 days | Awaiting first review |
|
|
0.5 days | Merge to deploy (still awaiting deployment) |
Total time for pull request
0.0 business days
from first commit authored to merge
Still awaiting deployment
build: update telemetry extractor to 1.20.5
Depends on #336746. Kept as a draft until that declaration-alignment prerequisite lands.
Fixes #336369
Refs microsoft/vscode-telemetry-extractor#80.
Summary
- Update
@vscode/telemetry-extractorfrom 1.20.4 to the published 1.20.5 in the root manifest and lockfile. - Pick up the upstream sequential-worker memory optimization.
- Keep the existing
--max-old-space-size=4096setting and all unrelated dependency resolutions unchanged.
Version 1.20.5 was published on 2026-09-18. Its npm tarball SHA-512 matches the updated lockfile, and the compiled worker is included.
Prerequisite
A dependency-only pickup on main currently fails because the workbench and Agent Host declare conflicting metadata for the same chat.toolApproval event. The new release detects that conflict. #336746 shares the existing classification between both emitters without changing their runtime JavaScript.
This PR deliberately contains only the two dependency files; it must not merge before #336746. No telemetry verification is disabled.
Validation
Using the actual published tarball, Node 24.18.0 / Windows x64, and the existing 4096 MiB setting:
-
With #336746 applied: core extraction succeeds with 485 events and 34 common properties. - The seven-entry extension/helper config succeeds with 72 events and 14 common properties.
- Exact manifest/lockfile checks confirm there are no unrelated dependency changes; published tarball integrity matches.
- Focused extraction-wrapper compilation, wrapper hygiene and
git diff --checkpass. The root JSON manifests are excluded from repository hygiene and were validated structurally.
Existing dependencies were reused read-only; this is not a full CI-agent/filesystem replay. Real macOS memory constraints and CI validation remain part of #336369. No pipeline was queued or rerun.
Comments Threads Pending Resolution
## Copilot review overview
### π΅ Needs a closer look
Prerequisite PR #336746 remains unmerged, so this dependency bump cannot yet pass telemetry extraction independently.
**Review effort:** Balanced
**Findings:** None
What changed in this PR
Updates the telemetry extractor to 1.20.5, incorporating upstream memory optimizations.
**Changes:**
- Bumps the root dependency declaration.
- Updates the locked package version, URL, and integrity hash.
| File | Description |
| ---- | ----------- |
| `package.json` | Declares telemetry extractor 1.20.5. |
| `package-lock.json` | Locks the matching published package. |
---
π‘ Add a `code-review` agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Resolved Comment Threads
No resolved comments have been left on this PR.