fix(node): Vendor InstrumentationNodeModuleFile to fix Bun --bytecode crash (#21262)

`InstrumentationNodeModuleFile` from `@opentelemetry/instrumentation`
resolves `path.normalize`. When Bun's `--bytecode` compiler inlines the
module tree into a single flat bundle, it renames `normalize` to
`normalize4` and then loses the scope binding in bytecode compilation,
producing a `ReferenceError` at runtime.

Vendor a local `InstrumentationNodeModuleFile` class that calls
`path.normalize()` directly, replacing the import from
`@opentelemetry/instrumentation` in all vendored instrumentations within
`@sentry/node`.

Closes #21256

Author: s1gr1d

.github/workflows/build.yml

@@ -984,7 +984,7 @@ jobs:
           node-version-file: 'dev-packages/e2e-tests/test-applications/${{ matrix.test-application }}/package.json'
       - name: Set up Bun
         if:
-          contains(fromJSON('["node-exports-test-app","nextjs-16-bun", "elysia-bun", "hono-4"]'),
+          contains(fromJSON('["node-exports-test-app","nextjs-16-bun", "elysia-bun", "hono-4", "bun-bytecode"]'),
           matrix.test-application)
         uses: oven-sh/setup-bun@v2
         with:

dev-packages/e2e-tests/test-applications/bun-bytecode/package.json

@@ -0,0 +1,19 @@
+{
+  "name": "bun-bytecode-test",
+  "version": "1.0.0",
+  "private": true,
+  "scripts": {
+    "clean": "npx rimraf node_modules pnpm-lock.yaml dist",
+    "test:build": "pnpm install && bun build src/main.ts --compile --bytecode --format=esm --outfile dist/main",
+    "test:assert": "dist/main"
+  },
+  "dependencies": {
+    "@sentry/bun": "file:../../packed/sentry-bun-packed.tgz"
+  },
+  "devDependencies": {
+    "bun-types": "^1.2.9"
+  },
+  "volta": {
+    "extends": "../../package.json"
+  }
+}

dev-packages/e2e-tests/test-applications/bun-bytecode/src/main.ts

@@ -0,0 +1,11 @@
+/**
+ * Verifies that @sentry/bun can be compiled with `bun build --compile --bytecode` without crashing at runtime.
+ */
+import * as Sentry from '@sentry/bun';
+
+Sentry.init({
+  dsn: 'https://username@domain/123',
+  tracesSampleRate: 0,
+});
+
+console.log('Bun bytecode compilation: OK');

dev-packages/e2e-tests/test-applications/bun-bytecode/tsconfig.json

@@ -0,0 +1,10 @@
+{
+  "compilerOptions": {
+    "types": ["bun-types"],
+    "strict": true,
+    "module": "ESNext",
+    "moduleResolution": "bundler",
+    "target": "ESNext"
+  },
+  "include": ["src"]
+}

packages/node/src/integrations/tracing/InstrumentationNodeModuleFile.ts

@@ -0,0 +1,45 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * NOTICE from the Sentry authors:
+ * - Vendored locally from @opentelemetry/instrumentation to work around a Bun
+ *   --bytecode bug (https://github.com/getsentry/sentry-javascript/issues/21256).
+ * - The upstream class imports `normalize` via an indirect platform/index
+ *   re-export chain, which Bun's bytecode bundler renames and loses scope for.
+ * - This copy imports `normalize` directly from 'path' to break that chain.
+ */
+/* oxlint-disable */
+
+import { normalize } from 'path';
+import type { InstrumentationModuleFile } from '@opentelemetry/instrumentation';
+
+export class InstrumentationNodeModuleFile implements InstrumentationModuleFile {
+  public name: string;
+  public supportedVersions: string[];
+  public patch: (moduleExports: any, moduleVersion?: string) => any;
+  public unpatch: (moduleExports?: any, moduleVersion?: string) => void;
+
+  constructor(
+    name: string,
+    supportedVersions: string[],
+    patch: (moduleExports: any, moduleVersion?: string) => any,
+    unpatch: (moduleExports?: any, moduleVersion?: string) => void,
+  ) {
+    this.name = normalize(name);
+    this.supportedVersions = supportedVersions;
+    this.patch = patch;
+    this.unpatch = unpatch;
+  }
+}

packages/node/src/integrations/tracing/amqplib/vendored/amqplib.ts

@@ -35,12 +35,12 @@ import { timestampInSeconds } from '@sentry/core';
 import {
   InstrumentationBase,
   InstrumentationNodeModuleDefinition,
-  InstrumentationNodeModuleFile,
   isWrapped,
   safeExecuteInTheMiddle,
   SemconvStability,
   semconvStabilityFromStr,
 } from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleFile } from '../../InstrumentationNodeModuleFile';
 import { ATTR_MESSAGING_OPERATION } from './semconv';
 import {
   ATTR_MESSAGING_DESTINATION,

packages/node/src/integrations/tracing/firebase/otel/patches/firestore.ts

@@ -1,12 +1,8 @@
 import * as net from 'node:net';
 import type { Span, Tracer } from '@opentelemetry/api';
 import { context, diag, SpanKind, trace } from '@opentelemetry/api';
-import {
-  InstrumentationNodeModuleDefinition,
-  InstrumentationNodeModuleFile,
-  isWrapped,
-  safeExecuteInTheMiddle,
-} from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleDefinition, isWrapped, safeExecuteInTheMiddle } from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleFile } from '../../../InstrumentationNodeModuleFile';
 import {
   ATTR_DB_COLLECTION_NAME,
   ATTR_DB_NAMESPACE,

packages/node/src/integrations/tracing/firebase/otel/patches/functions.ts

@@ -1,12 +1,8 @@
 import type { Span, Tracer } from '@opentelemetry/api';
 import { context, diag, SpanKind, trace } from '@opentelemetry/api';
 import type { InstrumentationBase } from '@opentelemetry/instrumentation';
-import {
-  InstrumentationNodeModuleDefinition,
-  InstrumentationNodeModuleFile,
-  isWrapped,
-  safeExecuteInTheMiddle,
-} from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleDefinition, isWrapped, safeExecuteInTheMiddle } from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleFile } from '../../../InstrumentationNodeModuleFile';
 import type { SpanAttributes } from '@sentry/core';
 import type { FirebaseInstrumentation } from '../firebaseInstrumentation';
 import type {

packages/node/src/integrations/tracing/google-genai/instrumentation.ts

@@ -1,9 +1,6 @@
 import type { InstrumentationConfig, InstrumentationModuleDefinition } from '@opentelemetry/instrumentation';
-import {
-  InstrumentationBase,
-  InstrumentationNodeModuleDefinition,
-  InstrumentationNodeModuleFile,
-} from '@opentelemetry/instrumentation';
+import { InstrumentationBase, InstrumentationNodeModuleDefinition } from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleFile } from '../InstrumentationNodeModuleFile';
 import type { GoogleGenAIClient, GoogleGenAIOptions } from '@sentry/core';
 import {
   _INTERNAL_shouldSkipAiProviderWrapping,

packages/node/src/integrations/tracing/graphql/vendored/instrumentation.ts

@@ -26,9 +26,9 @@ import {
   isWrapped,
   InstrumentationBase,
   InstrumentationNodeModuleDefinition,
-  InstrumentationNodeModuleFile,
   safeExecuteInTheMiddle,
 } from '@opentelemetry/instrumentation';
+import { InstrumentationNodeModuleFile } from '../../InstrumentationNodeModuleFile';
 import type {
   DefinitionNode,
   DocumentNode,