Skip to content

fix(nextjs): Shim pinoIntegration on edge runtime - #21347

Merged
chargome merged 2 commits into
developfrom
charlygomez/js-2679-pinointegration-not-available-via-sentrynextjs
Jun 5, 2026
Merged

fix(nextjs): Shim pinoIntegration on edge runtime#21347
chargome merged 2 commits into
developfrom
charlygomez/js-2679-pinointegration-not-available-via-sentrynextjs

Conversation

@chargome

Copy link
Copy Markdown
Member

Next.js compiles instrumentation modules for the edge runtime too, so we shim this integration on edge.

closes#21317

@chargomechargome self-assigned this Jun 5, 2026
@linear-code

Copy link
Copy Markdown

JS-2679

@chargome

Copy link
Copy Markdown
MemberAuthor

bugbot run

@cursorcursorBot 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.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit a3e99ee. Configure here.

@chargome
chargome marked this pull request as ready for review June 5, 2026 11:57
@chargome
chargome requested a review from a team as a code ownerJune 5, 2026 11:57
@chargome
chargome requested review from nicohrubec and s1gr1d and removed request for a teamJune 5, 2026 11:57
@chargome
chargome merged commit 7a2f01e into developJun 5, 2026
80 checks passed
@chargome
chargome deleted the charlygomez/js-2679-pinointegration-not-available-via-sentrynextjs branch June 5, 2026 14:33
Sign up for freeto 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.

pinoIntegration not available via @sentry/nextjs

2 participants

@chargome@nicohrubec