test(nuxt): Cover root-level option resolution and sourcemaps.disable

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: nicohrubec

packages/nuxt/src/vite/sourceMaps.ts

@@ -163,8 +163,9 @@ export function getPluginOptions(
 }
 
 /**
- * Resolves the `filesToDeleteAfterUpload` value: the user-specified globs if provided, otherwise the
- * client/server fallback globs when Sentry enabled source maps itself. Logs the fallback in debug mode.
+ * Users can set `filesToDeleteAfterUpload` themselves. If they don't, we fall back to deleting the
+ * client/server source maps — but only the ones Sentry generated itself (i.e. when the user didn't
+ * configure source maps at all). If the user explicitly set source maps, we leave their files alone.
  */
 function resolveFilesToDeleteAfterUpload(
   moduleOptions: SentryNuxtModuleOptions,

packages/nuxt/test/vite/sourceMaps-nuxtHooks.test.ts

@@ -109,6 +109,17 @@ describe('setupSourceMaps hooks', () => {
       expect(mockAddVitePlugin).not.toHaveBeenCalled();
     });
 
+    it('does not add plugins when source maps are disabled via `sourcemaps.disable`', async () => {
+      const { setupSourceMaps } = await import('../../src/vite/sourceMaps');
+      const mockNuxt = createMockNuxt({});
+      const { mockAddVitePlugin } = createMockAddVitePlugin();
+
+      setupSourceMaps({ sourcemaps: { disable: true } }, mockNuxt as unknown as Nuxt, mockAddVitePlugin);
+      await mockNuxt.triggerHook('modules:done');
+
+      expect(mockAddVitePlugin).not.toHaveBeenCalled();
+    });
+
     it.each([
       { label: 'server (SSR) build', buildConfig: { build: { ssr: true }, plugins: [] } },
       { label: 'client build', buildConfig: { build: { ssr: false }, plugins: [] } },

packages/nuxt/test/vite/sourceMaps.test.ts

@@ -113,6 +113,35 @@ describe('getPluginOptions', () => {
     );
   });
 
+  it('resolves root-level options into plugin options', () => {
+    const errorHandler = (err: Error): void => {
+      throw err;
+    };
+    const result = getPluginOptions({
+      org: 'my-org',
+      project: 'my-project',
+      authToken: 'my-token',
+      sentryUrl: 'https://my.sentry.io',
+      telemetry: false,
+      silent: true,
+      errorHandler,
+      headers: { 'X-Foo': 'bar' },
+      release: { name: '1.2.3' },
+    } as SentryNuxtModuleOptions);
+
+    expect(result).toMatchObject({
+      org: 'my-org',
+      project: 'my-project',
+      authToken: 'my-token',
+      url: 'https://my.sentry.io', // sentryUrl is resolved to the plugin's `url` option
+      telemetry: false,
+      silent: true,
+      errorHandler,
+      headers: { 'X-Foo': 'bar' },
+      release: { name: '1.2.3' },
+    });
+  });
+
   it('normalizes source paths via rewriteSources', () => {
     const options = getPluginOptions({} as SentryNuxtModuleOptions, undefined);
     const rewrite = options.sourcemaps?.rewriteSources as ((s: string) => string) | undefined;