rm reference in nuxt

Author: Luca Forstner

packages/nuxt/src/server/sdk.ts

@@ -18,7 +18,6 @@ import type { SentryNuxtServerOptions } from '../common/types';
 export function init(options: SentryNuxtServerOptions): Client | undefined {
   const sentryOptions = {
     ...options,
-    registerEsmLoaderHooks: mergeRegisterEsmLoaderHooks(options),
     defaultIntegrations: getNuxtDefaultIntegrations(options),
   };
 
@@ -92,28 +91,6 @@ function getNuxtDefaultIntegrations(options: NodeOptions): Integration[] {
   ];
 }
 
-/**
- * Adds /vue/ to the registerEsmLoaderHooks options and merges it with the old values in the array if one is defined.
- * If the registerEsmLoaderHooks option is already a boolean, nothing is changed.
- *
- * Only exported for Testing purposes.
- */
-export function mergeRegisterEsmLoaderHooks(
-  options: SentryNuxtServerOptions,
-): SentryNuxtServerOptions['registerEsmLoaderHooks'] {
-  if (typeof options.registerEsmLoaderHooks === 'object' && options.registerEsmLoaderHooks !== null) {
-    return {
-      // eslint-disable-next-line deprecation/deprecation
-      exclude: Array.isArray(options.registerEsmLoaderHooks.exclude)
-        ? // eslint-disable-next-line deprecation/deprecation
-          [...options.registerEsmLoaderHooks.exclude, /vue/]
-        : // eslint-disable-next-line deprecation/deprecation
-          options.registerEsmLoaderHooks.exclude ?? [/vue/],
-    };
-  }
-  return options.registerEsmLoaderHooks ?? { exclude: [/vue/] };
-}
-
 /**
  * Flushes pending Sentry events with a 2-second timeout and in a way that cannot create unhandled promise rejections.
  */

packages/nuxt/test/server/sdk.test.ts

@@ -5,9 +5,8 @@ import { Scope } from '@sentry/node';
 import { getGlobalScope } from '@sentry/node';
 import { SDK_VERSION } from '@sentry/node';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import type { SentryNuxtServerOptions } from '../../src/common/types';
 import { init } from '../../src/server';
-import { clientSourceMapErrorFilter, mergeRegisterEsmLoaderHooks } from '../../src/server/sdk';
+import { clientSourceMapErrorFilter } from '../../src/server/sdk';
 
 const nodeInit = vi.spyOn(SentryNode, 'init');
 
@@ -163,42 +162,4 @@ describe('Nuxt Server SDK', () => {
       });
     });
   });
-
-  describe('mergeRegisterEsmLoaderHooks', () => {
-    it('merges exclude array when registerEsmLoaderHooks is an object with an exclude array', () => {
-      const options: SentryNuxtServerOptions = {
-        registerEsmLoaderHooks: { exclude: [/test/] },
-      };
-      const result = mergeRegisterEsmLoaderHooks(options);
-      expect(result).toEqual({ exclude: [/test/, /vue/] });
-    });
-
-    it('sets exclude array when registerEsmLoaderHooks is an object without an exclude array', () => {
-      const options: SentryNuxtServerOptions = {
-        registerEsmLoaderHooks: {},
-      };
-      const result = mergeRegisterEsmLoaderHooks(options);
-      expect(result).toEqual({ exclude: [/vue/] });
-    });
-
-    it('returns boolean when registerEsmLoaderHooks is a boolean', () => {
-      const options1: SentryNuxtServerOptions = {
-        registerEsmLoaderHooks: true,
-      };
-      const result1 = mergeRegisterEsmLoaderHooks(options1);
-      expect(result1).toBe(true);
-
-      const options2: SentryNuxtServerOptions = {
-        registerEsmLoaderHooks: false,
-      };
-      const result2 = mergeRegisterEsmLoaderHooks(options2);
-      expect(result2).toBe(false);
-    });
-
-    it('sets exclude array when registerEsmLoaderHooks is undefined', () => {
-      const options: SentryNuxtServerOptions = {};
-      const result = mergeRegisterEsmLoaderHooks(options);
-      expect(result).toEqual({ exclude: [/vue/] });
-    });
-  });
 });