fix(nuxt): Allow overwriting server-side defaultIntegrations (#18717)

There is no reason for spreading the user-provided options before adding
the `defaultIntegrations`. This changes the order to allow overriding
and allowing passing `defaultIntegrations: false`. Also added tests for
that

Came up in this review comment of another PR:
#18671 (comment)


Closes #18718 (added automatically)

Author: s1gr1d

packages/nuxt/src/server/sdk.ts

@@ -27,8 +27,8 @@ import type { SentryNuxtServerOptions } from '../common/types';
 export function init(options: SentryNuxtServerOptions): Client | undefined {
   const sentryOptions = {
     environment: !isCjs() && import.meta.dev ? DEV_ENVIRONMENT : DEFAULT_ENVIRONMENT,
-    ...options,
     defaultIntegrations: getNuxtDefaultIntegrations(options),
+    ...options,
   };
 
   applySdkMetadata(sentryOptions, 'nuxt', ['nuxt', 'node']);

packages/nuxt/test/client/sdk.test.ts

@@ -41,5 +41,41 @@ describe('Nuxt Client SDK', () => {
     it('returns client from init', () => {
       expect(init({})).not.toBeUndefined();
     });
+
+    it('uses default integrations when not provided in options', () => {
+      init({ dsn: 'https://[email protected]/1337' });
+
+      expect(browserInit).toHaveBeenCalledTimes(1);
+      const callArgs = browserInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBeDefined();
+      expect(Array.isArray(callArgs?.defaultIntegrations)).toBe(true);
+    });
+
+    it('allows options.defaultIntegrations to override default integrations', () => {
+      const customIntegrations = [{ name: 'CustomIntegration' }];
+
+      init({
+        dsn: 'https://[email protected]/1337',
+        defaultIntegrations: customIntegrations as any,
+      });
+
+      expect(browserInit).toHaveBeenCalledTimes(1);
+      const callArgs = browserInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBe(customIntegrations);
+    });
+
+    it('allows options.defaultIntegrations to be set to false', () => {
+      init({
+        dsn: 'https://[email protected]/1337',
+        defaultIntegrations: false,
+      });
+
+      expect(browserInit).toHaveBeenCalledTimes(1);
+      const callArgs = browserInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBe(false);
+    });
   });
 });

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

@@ -41,6 +41,42 @@ describe('Nuxt Server SDK', () => {
       expect(init({})).not.toBeUndefined();
     });
 
+    it('uses default integrations when not provided in options', () => {
+      init({ dsn: 'https://[email protected]/1337' });
+
+      expect(nodeInit).toHaveBeenCalledTimes(1);
+      const callArgs = nodeInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBeDefined();
+      expect(Array.isArray(callArgs?.defaultIntegrations)).toBe(true);
+    });
+
+    it('allows options.defaultIntegrations to override default integrations', () => {
+      const customIntegrations = [{ name: 'CustomIntegration' }];
+
+      init({
+        dsn: 'https://[email protected]/1337',
+        defaultIntegrations: customIntegrations as any,
+      });
+
+      expect(nodeInit).toHaveBeenCalledTimes(1);
+      const callArgs = nodeInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBe(customIntegrations);
+    });
+
+    it('allows options.defaultIntegrations to be set to false', () => {
+      init({
+        dsn: 'https://[email protected]/1337',
+        defaultIntegrations: false,
+      });
+
+      expect(nodeInit).toHaveBeenCalledTimes(1);
+      const callArgs = nodeInit.mock.calls[0]?.[0];
+      expect(callArgs).toBeDefined();
+      expect(callArgs?.defaultIntegrations).toBe(false);
+    });
+
     describe('lowQualityTransactionsFilter', () => {
       const options = { debug: false };
       const filter = lowQualityTransactionsFilter(options);