Better defaultIntegrations

Author: timfish

packages/node/src/sdk.ts

@@ -7,7 +7,7 @@ import {
   Integrations as CoreIntegrations,
   setHubOnCarrier,
 } from '@sentry/core';
-import { SessionStatus, StackParser } from '@sentry/types';
+import { Integration, SessionStatus, StackParser } from '@sentry/types';
 import {
   createStackParser,
   dynamicRequire,
@@ -109,29 +109,11 @@ export const defaultIntegrations = [
  *
  * @see {@link NodeOptions} for documentation on configuration options.
  */
-// eslint-disable-next-line complexity
 export function init(options: NodeOptions = {}): void {
   const carrier = getMainCarrier();
   const autoloadedIntegrations = carrier.__SENTRY__?.integrations || [];
 
-  options.defaultIntegrations =
-    options.defaultIntegrations === false
-      ? []
-      : [
-          ...(Array.isArray(options.defaultIntegrations) ? options.defaultIntegrations : defaultIntegrations),
-          ...autoloadedIntegrations,
-        ];
-
-  const nodeVersion = parseSemver(process.versions.node);
-
-  if (options.includeStackLocals && (nodeVersion.major || 0) >= 14) {
-    const { LocalVariables } = dynamicRequire(
-      module,
-      './integrations/localvariables',
-    ) as typeof import('./integrations/localvariables');
-
-    options.defaultIntegrations.push(new LocalVariables());
-  }
+  options.defaultIntegrations = getDefaultIntegrations(options, autoloadedIntegrations);
 
   if (options.dsn === undefined && process.env.SENTRY_DSN) {
     options.dsn = process.env.SENTRY_DSN;
@@ -186,6 +168,29 @@ export function init(options: NodeOptions = {}): void {
   }
 }
 
+function getDefaultIntegrations(options: NodeOptions, autoloadedIntegrations: Integration[]): Integration[] {
+  const integrations =
+    options.defaultIntegrations === false
+      ? []
+      : [
+          ...(Array.isArray(options.defaultIntegrations) ? options.defaultIntegrations : defaultIntegrations),
+          ...autoloadedIntegrations,
+        ];
+
+  const nodeVersion = parseSemver(process.versions.node);
+
+  if (options.includeStackLocals && (nodeVersion.major || 0) >= 14) {
+    const { LocalVariables } = dynamicRequire(
+      module,
+      './integrations/localvariables',
+    ) as typeof import('./integrations/localvariables');
+
+    integrations.push(new LocalVariables());
+  }
+
+  return integrations;
+}
+
 /**
  * This is the getter for lastEventId.
  *