Skip to content

Bump @opentelemetry/core and applicationinsights (with v3 telemetry compatibility fix) - #2660

Open
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/npm_and_yarn/multi-2c236f6ce7
Open

Bump @opentelemetry/core and applicationinsights (with v3 telemetry compatibility fix)#2660
dependabot[bot] wants to merge 3 commits into
mainfrom
dependabot/npm_and_yarn/multi-2c236f6ce7

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jun 17, 2026 •
edited by Copilot AI
Loading

Copy link
Copy Markdown
Contributor

Bumps @opentelemetry/core to 2.8.0 and updates ancestor dependency applicationinsights. These dependencies need to be updated together.

Updates @opentelemetry/core from 2.0.1 to 2.8.0


Updates applicationinsights from 2.9.8 to 3.15.0


Also includes a compatibility update in src/common/Telemetry.ts required for applicationinsights 3.x (TelemetryClient import/type updates) so the project builds successfully, and an ChangeLog.md update under Upcoming Release to document these changes.

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

@dependabot
Bump @opentelemetry/core and applicationinsights
7320938 
Bumps [@opentelemetry/core](https://github.com/open-telemetry/opentelemetry-js) to 2.8.0 and updates ancestor dependency [applicationinsights](https://github.com/microsoft/ApplicationInsights-node.js). These dependencies need to be updated together.


Updates `@opentelemetry/core` from 2.0.1 to 2.8.0
- [Release notes](https://github.com/open-telemetry/opentelemetry-js/releases)
- [Changelog](https://github.com/open-telemetry/opentelemetry-js/blob/main/CHANGELOG.md)
- [Commits](open-telemetry/opentelemetry-js@v2.0.1...v2.8.0)

Updates `applicationinsights` from 2.9.8 to 3.15.0
- [Release notes](https://github.com/microsoft/ApplicationInsights-node.js/releases)
- [Changelog](https://github.com/microsoft/ApplicationInsights-node.js/blob/main/CHANGELOG.md)
- [Commits](https://github.com/microsoft/ApplicationInsights-node.js/commits/3.15.0)

---
updated-dependencies:
- dependency-name: "@opentelemetry/core"
  dependency-version: 2.8.0
  dependency-type: indirect
- dependency-name: applicationinsights
  dependency-version: 3.15.0
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Jun 17, 2026
Copilot AI review requested due to automatic review settings June 17, 2026 02:40
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Jun 17, 2026
Copilot AI reviewed Jun 17, 2026

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@jainakanksha-msft

jainakanksha-msft commented Jun 17, 2026

Copy link
Copy Markdown
Member

@copilot, review and fix the whole PR. this PR is hvaing build issues.

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@copilot, review and fix the whole PR. this PR is hvaing build issues.

Addressed the build break in commit 7025d65. I updated src/common/Telemetry.ts for applicationinsights@3.x compatibility (new TelemetryClient import and request payload typing updates), and npm run build now succeeds locally. I also ran npm run test:exe; it fails due to missing prebuilt Windows binary (npm run build:exe) rather than this change.

Copilot AI requested a review from jainakanksha-msft June 17, 2026 08:21
@jainakanksha-msft

jainakanksha-msft commented Jun 17, 2026

Copy link
Copy Markdown
Member

@copilot please update Changelog file as per changes.

Copilot AI review requested due to automatic review settings June 17, 2026 08:26
@dependabot
dependabot Bot removed the request for review from Copilot June 17, 2026 08:26

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@copilot please update Changelog file as per changes.

Updated in 7c64a44: ChangeLog.md now includes Upcoming Release notes for the applicationinsights/@opentelemetry/core bump and the telemetry compatibility update.

Copilot AI changed the title Bump @opentelemetry/core and applicationinsights Bump @opentelemetry/core and applicationinsights (with v3 telemetry compatibility fix) Jun 17, 2026
@md84419

md84419 commented Jul 24, 2026

Copy link
Copy Markdown

@jainakanksha-msft what else is need to get this merged in, please?

Copilot AI reviewed Jul 27, 2026
View reviewed changes

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

Comment thread src/common/Telemetry.ts
}

private static removeRoleInstance ( envelope: Contracts.EnvelopeTelemetry) : boolean {
private static removeRoleInstance(envelope: any): boolean {
@jainakanksha-msft

jainakanksha-msft commented Jul 27, 2026

Copy link
Copy Markdown
Member

@md84419, thanks for checking in.
We were primarily focused on resolving release-related issues and completing the recent Azurite release, which delayed our review of this PR.
We are now revisiting the change and will validate it against the latest codebase before making a merge decision.
Thanks for your patience and contribution.

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

Reviewers

Copilot code review Copilot Copilot left review comments
@jainakanksha-msft jainakanksha-msft Awaiting requested review from jainakanksha-msft

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code v3.37.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jainakanksha-msft @md84419