Skip to content

chore: bump @vscode/extension-telemetry from 0.9.7 to 1.5.2 - #1960

Open
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/npm_and_yarn/vscode/extension-telemetry-1.5.2
Open

chore: bump @vscode/extension-telemetry from 0.9.7 to 1.5.2#1960
dependabot[bot] wants to merge 1 commit into
masterfrom
dependabot/npm_and_yarn/vscode/extension-telemetry-1.5.2

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github May 19, 2026

Copy link
Copy Markdown
Contributor

Bumps @vscode/extension-telemetry from 0.9.7 to 1.5.2.

Release notes

Sourced from @​vscode/extension-telemetry's releases.

v1.5.2

Changes:

  • #245: Update and explicitly list direct dependencies
  • #243: npm update brace-expansion
  • #242: Bump flatted from 3.4.1 to 3.4.2
  • #241: chore: migrate to ESLint 9 with flat config
  • #240: Bump minimatch from 3.1.2 to 3.1.5

This list of changes was auto generated.

v1.5.1

Changes:

  • #239: setdefault content type to json

This list of changes was auto generated.

v1.5.0

Changes:

  • #238: Add API for sending exception data

This list of changes was auto generated.

v1.4.0

Changes:

  • #236: Update dist file
  • #235: Bump js-yaml from 4.1.0 to 4.1.1

This list of changes was auto generated.

v1.3.0

Changes:

Feature Requests:

... (truncated)

Commits
  • e33243f Merge pull request #245 from microsoft/updateAndAddExplicitDependencies
  • 5c7de26 Update version.
  • 2e31d47 Update and explicitly list all dependencies.
  • 31e45f2 npm update brace-expansion (#243)
  • 5d142e3 Bump flatted from 3.4.1 to 3.4.2 (#242)
  • 7d1752f Merge pull request #241 from microsoft/copilot/migrate-to-eslint-9
  • 8131abe chore: migrate to ESLint 9 with flat config
  • 71d0bac Initial plan
  • 9e42092 Merge pull request #240 from microsoft/dependabot/npm_and_yarn/minimatch-3.1.5
  • 66b07a3 Bump minimatch from 3.1.2 to 3.1.5
  • Additional commits viewable in compare view
Maintainer changes

This version was pushed to npm by microsoft1es, a new releaser for @​vscode/extension-telemetry since your current version.


Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

@ralphstodomingo

Copy link
Copy Markdown
Contributor

Verification: 🟡 needs a one-line source change before merge

The runtime API surface of @vscode/extension-telemetry@1.5.2 is fully backward-compatible with the 0.9.7 usage in src/telemetry/index.ts — the constructor (new TelemetryReporter(connectionString)), sendTelemetryEvent(name, props?, measurements?), sendTelemetryErrorEvent(name, props?, measurements?), and dispose() (now returns Promise<any> but we discard the return) all line up.

But v1.x dropped the default export. Only TelemetryReporter is exported by name. The current code does:

import TelemetryReporter from "@vscode/extension-telemetry";

…which against v1.5.2 yields:

src/telemetry/index.ts(6,30): error TS2709: Cannot use namespace 'TelemetryReporter' as a type.
src/telemetry/index.ts(6,54): error TS2351: This expression is not constructable.
  Type 'typeof import(".../telemetryReporter")' has no construct signatures.

rsbuild build then fails with the same error, so the extension won't bundle.

Fix is one line:

-import TelemetryReporter from "@vscode/extension-telemetry";
+import { TelemetryReporter } from "@vscode/extension-telemetry";

With that change applied on top of this PR:

  • tsc --noEmit → clean
  • rsbuild build → succeeds (2.7 MB extension.js + 91 KB chunk)
  • Bundle parses
  • npm test → 35/35 suites pass, 493 tests pass, including telemetryService.test.ts

Suggested action: push the import-style change as a follow-up commit on this branch, then merge. Alternatively, land the import conversion in a small standalone PR first, then @dependabot rebase and merge this one.

Do not merge as-is — the build will fail.

@ralphstodomingo

Copy link
Copy Markdown
Contributor

Correction — also fails on lockfile mismatch (in addition to the import-fix need)

This PR also fails CI at npm ci:

npm error code EUSAGE
npm error Invalid: lock file's @vscode/extension-telemetry@0.9.7 does not satisfy @vscode/extension-telemetry@1.5.2
npm error Invalid: lock file's @microsoft/1ds-core-js@4.3.3 does not satisfy 4.4.1
npm error Invalid: lock file's @microsoft/1ds-post-js@4.3.3 does not satisfy 4.4.1
npm error Invalid: lock file's @microsoft/applicationinsights-common@3.3.3 does not satisfy 3.4.1
...

The previously-identified one-line import fix in src/telemetry/index.ts still stands. To land this PR you need both:

  1. The import-style change (import TelemetryReporter from ...import { TelemetryReporter } from ...).
  2. A regenerated package-lock.json.

Note: these must land together — they can't be split. v0.9.7 exports TelemetryReporter as export default class only (no named export), while v1.5.2 exports it as export class only (no default). The two import forms are mutually exclusive across the version bump, so the import change has to ride with the bump in the same commit.

Suggested action: push both changes onto this dependabot branch directly (overwriting Dependabot's HEAD), then merge.

@dev-punia-altimate dev-punia-altimate left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dependency bump from @vscode/extension-telemetry@0.9.7 to 1.5.2 — no code changes, no breaking API shifts detected. Safe to merge.

Bumps [@vscode/extension-telemetry](https://github.com/Microsoft/vscode-extension-telemetry) from 0.9.7 to 1.5.2.
- [Release notes](https://github.com/Microsoft/vscode-extension-telemetry/releases)
- [Commits](microsoft/vscode-extension-telemetry@v0.9.7...v1.5.2)

---
updated-dependencies:
- dependency-name: "@vscode/extension-telemetry"
  dependency-version: 1.5.2
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot force-pushed the dependabot/npm_and_yarn/vscode/extension-telemetry-1.5.2 branch from 44461d9 to 4302eff Compare May 25, 2026 06:22
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