Skip to content

ref(node): Replace @opentelemetry/api in lru-memoizer instrumentation with sentry APIs#21818

Open
nicohrubec wants to merge 1 commit into
developfrom
nh/lru-memoizer-sentry-context
Open

ref(node): Replace @opentelemetry/api in lru-memoizer instrumentation with sentry APIs#21818
nicohrubec wants to merge 1 commit into
developfrom
nh/lru-memoizer-sentry-context

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Jun 26, 2026

Copy link
Copy Markdown
Member

Drop the remaining @opentelemetry/api usage in the lru-memoizer instrumentation.

@nicohrubec nicohrubec changed the title ref(node): Remove remaining OTel API from lru-memoizer instrumentation ref(node): Replace @opentelemetry/api in lru-memoizer instrumentation with sentry APIs Jun 26, 2026
@nicohrubec nicohrubec marked this pull request as ready for review June 26, 2026 18:13
@nicohrubec nicohrubec requested a review from a team as a code owner June 26, 2026 18:13
@nicohrubec nicohrubec requested review from JPeer264, andreiborza and mydea and removed request for a team June 26, 2026 18:13
Replaces the last `@opentelemetry/api` usage (`context.bind(context.active(), …)`) with `withScope(getCurrentScope(), …)` from `@sentry/core`, matching the orchestrion lru-memoizer subscriber and mysql.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec force-pushed the nh/lru-memoizer-sentry-context branch from 07de195 to e7dad80 Compare June 26, 2026 19:10

@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 e7dad80. Configure here.

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.

2 participants