Skip to content

fix(bun) Export pinoIntegration from @sentry/node#17990

Open
LudvigHz wants to merge 2 commits intogetsentry:developfrom
LudvigHz:pinointegration-bun
Open

fix(bun) Export pinoIntegration from @sentry/node#17990
LudvigHz wants to merge 2 commits intogetsentry:developfrom
LudvigHz:pinointegration-bun

Conversation

@LudvigHz
Copy link

@LudvigHz LudvigHz commented Oct 21, 2025

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

Resolves #17989

@chargome chargome requested a review from timfish October 22, 2025 07:40
@chargome
Copy link
Member

@timfish not sure if the pino integration works properly on bun due to TracingChannel – anyway we need to either merge this or update the docs for it.

@timfish
Copy link
Collaborator

timfish commented Oct 22, 2025

I think Bun supports TracingChannel so it could work with the latest versions of Pino that include a TracingChannel.

What won't work is the code injection via ESM loader hook because Bun doesn't support them. If loader hook registration is simply a no-op in Bun, it might just work with the latest versions of Pino without causing runtime errors?

@LudvigHz have you tested this working with Bun?

@s1gr1d
Copy link
Member

s1gr1d commented Oct 22, 2025

Bun supports the tracing channel, but they do not publish to the channels we use for the instrumentation.

More about that here: #17779 (comment)

@s1gr1d
Copy link
Member

s1gr1d commented Nov 13, 2025

Follow-up on the above comment: Bun has a PR for supporting the channels oven-sh/bun#24375

@github-actions
Copy link
Contributor

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you apply the label PR: no-auto-close I will leave it alone ... forever!

@timfish
Copy link
Collaborator

timfish commented Feb 17, 2026

I think this can probably be merged!

The Pino integration will work with Bun for any Pino version that publishes tracing channel events (I forget the exact version!)

@timfish timfish removed the PR: stale label Feb 17, 2026
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.

Pino integration not exported from @sentry/bun

4 participants