Skip to content

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)

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