ref(nitro): Use BuildTimeOptionsBase for SentryNitroOptions type

Derive `SentryNitroOptions` from `BuildTimeOptionsBase` (from `@sentry/core`)
instead of picking from the bundler plugin's internal `Options` type. This
provides a stable public API that follows this repo's semver, rather than
exposing bundler-plugin-specific fields like `_metaOptions` and top-level
`disable`.

Key changes:
- `url` option renamed to `sentryUrl` (consistent with other SDKs)
- Top-level `disable` removed (use `sourcemaps.disable` instead)
- `_metaOptions` no longer user-configurable

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

Author: logaretm

packages/nitro/src/config.ts

@@ -1,25 +1,9 @@
-import type { Options as SentryBundlerPluginOptions } from '@sentry/bundler-plugin-core';
+import type { BuildTimeOptionsBase } from '@sentry/core';
 import type { NitroConfig } from 'nitro/types';
 import { createNitroModule } from './module';
 import { configureSourcemapSettings } from './sourceMaps';
 
-export type SentryNitroOptions = Pick<
-  SentryBundlerPluginOptions,
-  | 'org'
-  | 'project'
-  | 'authToken'
-  | 'url'
-  | 'headers'
-  | 'debug'
-  | 'silent'
-  | 'errorHandler'
-  | 'telemetry'
-  | 'disable'
-  | 'sourcemaps'
-  | 'release'
-  | 'bundleSizeOptimizations'
-  | '_metaOptions'
->;
+export type SentryNitroOptions = BuildTimeOptionsBase;
 
 /**
  * Modifies the passed in Nitro configuration with automatic build-time instrumentation.

packages/nitro/src/sourceMaps.ts

@@ -1,4 +1,4 @@
-import type { Options } from '@sentry/bundler-plugin-core';
+import type { Options as BundlerPluginOptions } from '@sentry/bundler-plugin-core';
 import { createSentryBuildPluginManager } from '@sentry/bundler-plugin-core';
 import { debug } from '@sentry/core';
 import type { Nitro, NitroConfig } from 'nitro/types';
@@ -15,7 +15,7 @@ export function setupSourceMaps(nitro: Nitro, options?: SentryNitroOptions): voi
   }
 
   // Respect user's explicit disable
-  if (options?.sourcemaps?.disable === true || options?.disable === true) {
+  if (options?.sourcemaps?.disable === true) {
     return;
   }
 
@@ -62,12 +62,12 @@ function normalizePath(path: string): string {
  *
  * Only exported for testing purposes.
  */
-export function getPluginOptions(options?: SentryNitroOptions): Options {
+export function getPluginOptions(options?: SentryNitroOptions): BundlerPluginOptions {
   return {
     org: options?.org ?? process.env.SENTRY_ORG,
     project: options?.project ?? process.env.SENTRY_PROJECT,
     authToken: options?.authToken ?? process.env.SENTRY_AUTH_TOKEN,
-    url: options?.url ?? process.env.SENTRY_URL,
+    url: options?.sentryUrl ?? process.env.SENTRY_URL,
     headers: options?.headers,
     telemetry: options?.telemetry ?? true,
     debug: options?.debug ?? false,
@@ -86,7 +86,6 @@ export function getPluginOptions(options?: SentryNitroOptions): Options {
       telemetry: {
         metaFramework: 'nitro',
       },
-      ...options?._metaOptions,
     },
   };
 }
@@ -95,7 +94,7 @@ export function getPluginOptions(options?: SentryNitroOptions): Options {
  * Configures the Nitro config to enable source map generation.
  */
 export function configureSourcemapSettings(config: NitroConfig, moduleOptions?: SentryNitroOptions): void {
-  const sourcemapUploadDisabled = moduleOptions?.sourcemaps?.disable === true || moduleOptions?.disable === true;
+  const sourcemapUploadDisabled = moduleOptions?.sourcemaps?.disable === true;
   if (sourcemapUploadDisabled) {
     return;
   }

packages/nitro/test/sourceMaps.test.ts

@@ -56,28 +56,31 @@ describe('getPluginOptions', () => {
     expect(options.org).toBe('env-org');
     expect(options.project).toBe('env-project');
     expect(options.authToken).toBe('env-token');
-    expect(options.url).toBe('https://custom.sentry.io');
+    expect(options.url).toBe('https://custom.sentry.io'); // sentryUrl maps to url
   });
 
   it('prefers direct options over environment variables', () => {
     process.env.SENTRY_ORG = 'env-org';
     process.env.SENTRY_AUTH_TOKEN = 'env-token';
+    process.env.SENTRY_URL = 'https://env.sentry.io';
 
     const options = getPluginOptions({
       org: 'direct-org',
       authToken: 'direct-token',
+      sentryUrl: 'https://direct.sentry.io',
     });
 
     expect(options.org).toBe('direct-org');
     expect(options.authToken).toBe('direct-token');
+    expect(options.url).toBe('https://direct.sentry.io');
   });
 
   it('passes through all user options', () => {
     const sentryOptions: SentryNitroOptions = {
       org: 'my-org',
       project: 'my-project',
       authToken: 'my-token',
-      url: 'https://my-sentry.io',
+      sentryUrl: 'https://my-sentry.io',
       headers: { 'X-Custom': 'header' },
       debug: true,
       silent: true,
@@ -119,10 +122,9 @@ describe('getPluginOptions', () => {
   });
 
   it('always sets metaFramework to nitro', () => {
-    const options = getPluginOptions({ _metaOptions: { loggerPrefixOverride: '[custom]' } });
+    const options = getPluginOptions();
 
     expect(options._metaOptions?.telemetry?.metaFramework).toBe('nitro');
-    expect(options._metaOptions?.loggerPrefixOverride).toBe('[custom]');
   });
 
   it('passes through sourcemaps.disable', () => {
@@ -182,11 +184,11 @@ describe('configureSourcemapSettings', () => {
     expect(config.sourcemap).toBe(false);
   });
 
-  it('skips sourcemap config when disable is true', () => {
-    const config: NitroConfig = { sourcemap: false };
-    configureSourcemapSettings(config, { disable: true });
+  it('still configures sourcemaps when sourcemaps.disable is disable-upload', () => {
+    const config: NitroConfig = {};
+    configureSourcemapSettings(config, { sourcemaps: { disable: 'disable-upload' } });
 
-    expect(config.sourcemap).toBe(false);
+    expect(config.sourcemap).toBe(true);
   });
 });
 
@@ -240,18 +242,6 @@ describe('setupSourceMaps', () => {
     expect(hookFn).not.toHaveBeenCalled();
   });
 
-  it('does not register hook when disable is true', () => {
-    const hookFn = vi.fn();
-    const nitro = {
-      options: { dev: false, output: { serverDir: '/output/server' } },
-      hooks: { hook: hookFn },
-    } as any;
-
-    setupSourceMaps(nitro, { disable: true });
-
-    expect(hookFn).not.toHaveBeenCalled();
-  });
-
   it('registers compiled hook in production mode', () => {
     const hookFn = vi.fn();
     const nitro = {