Skip to content

Commit aee21c7

Browse files
JPeer264claude
andauthored
chore(bun): Fix stale comment in getPerformanceIntegrations (#24533)
closes #23888 The orchestrion gate that the comment described was removed on purpose when the bun auto-instrumentation handling was streamlined. The tracing integrations are now always added when spans are enabled, and the channel-based ones do nothing without the bun plugin. got stale with #23392 Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
1 parent ec43b68 commit aee21c7

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

‎packages/bun/src/sdk.ts‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,12 +28,9 @@ import { bunHttpServerIntegration } from './integrations/bunHttpServer';
2828
import { getErrorIntegrations, getTracingIntegrations } from '@sentry/server-utils';
2929

3030
/**
31-
* The performance integrations for bun: the OTel auto-performance set, but with
32-
* the orchestrion diagnostics-channel subscribers swapped in for their OTel
33-
* equivalents *only* when the orchestrion channels were actually injected (i.e.
34-
* the app was built with `@sentry/bun/plugin`). Without that, the channels
35-
* never fire — and the OTel versions rely on a runtime require-hook bun doesn't
36-
* support — so leave the auto-performance set alone.
31+
* The tracing integrations for bun, added whenever spans are enabled. Most of them listen on
32+
* the orchestrion diagnostics channels, which only exist when the app is built with
33+
* `@sentry/bun/plugin`. Without the plugin, those integrations stay installed but create no spans.
3734
*/
3835
function getPerformanceIntegrations(options: Options): Integration[] {
3936
if (!hasSpansEnabled(options)) {

0 commit comments

Comments
 (0)