Skip to content

ref(opentelemetry): Remove wrapContextManager export#22586

Open
mydea wants to merge 2 commits into
developfrom
fn/wrap-context-manager
Open

ref(opentelemetry): Remove wrapContextManager export#22586
mydea wants to merge 2 commits into
developfrom
fn/wrap-context-manager

Conversation

@mydea

@mydea mydea commented Jul 24, 2026

Copy link
Copy Markdown
Member

Instead, the context manager accepts the async local storage instance now, so we can also use this in vercel-edge. this will eventually go away/move to server-utils likely so we can revisit this in a follow up.

This also changes the manager slightly to not check for event emitter instance, as this does not work in vercel-edge, but look at the shape instead.

Instead, the context manager accepts the async local storage instance now, so we can also use this in vercel-edge. this will eventually go away/move to server-utils likely so we can revisit this in a follow up.
Comment thread packages/vercel-edge/src/sdk.ts

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3f7d8fa. Configure here.


export { setupEventContextTrace } from './setupEventContextTrace';

// eslint-disable-next-line typescript/no-deprecated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Public API removed without migration docs

Medium Severity

wrapContextManagerClass is removed from the public @sentry/opentelemetry surface. That is a breaking public API removal. It was marked deprecated, but MIGRATION.md does not list this removal under @sentry/opentelemetry, and the package README still tells users they can use wrapContextManagerClass for a custom base. Flagged because the PR review rules call out public API removals and breaking changes without proper notices.

Fix in Cursor Fix in Web

Triggered by project rule: PR Review Guidelines for Cursor Bot

Reviewed by Cursor Bugbot for commit 3f7d8fa. Configure here.

@mydea
mydea marked this pull request as ready for review July 24, 2026 11:29
@mydea
mydea requested a review from a team as a code owner July 24, 2026 11:29
@mydea
mydea requested review from JPeer264 and isaacs and removed request for a team July 24, 2026 11:29
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.

1 participant