fix(node): Use normal require call to import Undici (#10388)

Author: lforst

dev-packages/e2e-tests/test-applications/nextjs-14/app/request-instrumentation/page.tsx

@@ -3,7 +3,7 @@ import http from 'http';
 export const dynamic = 'force-dynamic';
 
 export default async function Page() {
-  await fetch('http://example.com/');
+  await fetch('http://example.com/', { cache: 'no-cache' });
   await new Promise<void>(resolve => {
     http.get('http://example.com/', () => {
       resolve();

packages/node/package.json

@@ -63,7 +63,7 @@
     "test:express": "node test/manual/express-scope-separation/start.js",
     "test:jest": "jest",
     "test:release-health": "node test/manual/release-health/runner.js",
-    "test:webpack": "cd test/manual/webpack-async-context/ && yarn --silent && node npm-build.js",
+    "test:webpack": "cd test/manual/webpack-async-context/ && yarn --silent --ignore-engines && node npm-build.js",
     "test:watch": "jest --watch",
     "yalc:publish": "ts-node ../../scripts/prepack.ts && yalc publish build --push --sig"
   },

packages/node/src/integrations/undici/index.ts

@@ -15,7 +15,6 @@ import {
 import type { EventProcessor, Integration, IntegrationFn, IntegrationFnResult, Span } from '@sentry/types';
 import {
   LRUMap,
-  dynamicRequire,
   dynamicSamplingContextToSentryBaggageHeader,
   generateSentryTraceHeader,
   getSanitizedUrlString,
@@ -127,7 +126,7 @@ export class Undici implements Integration {
     let ds: DiagnosticsChannel | undefined;
     try {
       // eslint-disable-next-line @typescript-eslint/no-var-requires
-      ds = dynamicRequire(module, 'diagnostics_channel') as DiagnosticsChannel;
+      ds = require('diagnostics_channel') as DiagnosticsChannel;
     } catch (e) {
       // no-op
     }

packages/node/test/manual/webpack-async-context/npm-build.js

@@ -7,6 +7,11 @@ if (Number(process.versions.node.split('.')[0]) >= 18) {
   process.exit(0);
 }
 
+// Webpack test does not work in Node 8 and below.
+if (Number(process.versions.node.split('.')[0]) <= 8) {
+  process.exit(0);
+}
+
 // biome-ignore format: Follow-up for prettier
 webpack(
   {

packages/node/test/manual/webpack-async-context/package.json

@@ -4,9 +4,9 @@
   "main": "index.js",
   "license": "MIT",
   "dependencies": {
-    "webpack": "^4.42.1"
+    "webpack": "^5.90.0"
   },
-  "devDependencies": {
-    "webpack-cli": "^3.3.11"
+  "volta": {
+    "extends": "../../../../../package.json"
   }
 }