feat(sveltekit)!: Remove deprecated sourceMapsUploadOptions Vite plugin option

packages/sveltekit/src/vite/sentryVitePlugins.ts

@@ -187,80 +187,42 @@ export function generateVitePluginOptions(
     };
   }
 
-  // todo(v11): remove deprecated options (Also from options type)
-
   // Source Maps
   if (svelteKitPluginOptions.autoUploadSourceMaps && process.env.NODE_ENV !== 'development') {
     const {
-      // eslint-disable-next-line typescript/no-deprecated
-      unstable_sentryVitePluginOptions: deprecated_unstableSourceMapUploadOptions,
-      ...deprecatedSourceMapUploadOptions
-      // eslint-disable-next-line typescript/no-deprecated
-    } = svelteKitPluginOptions.sourceMapsUploadOptions || {};
-
-    const {
-      // eslint-disable-next-line @typescript-eslint/no-unused-vars,typescript/no-deprecated
-      sourceMapsUploadOptions: _filtered1,
       // eslint-disable-next-line @typescript-eslint/no-unused-vars
-      unstable_sentryVitePluginOptions: _filtered2,
+      autoUploadSourceMaps: _filtered1,
       // eslint-disable-next-line @typescript-eslint/no-unused-vars
-      autoUploadSourceMaps: _filtered3,
-      // eslint-disable-next-line @typescript-eslint/no-unused-vars
-      autoInstrument: _filtered4,
+      autoInstrument: _filtered2,
+      unstable_sentryVitePluginOptions,
       sentryUrl,
       ...newSvelteKitPluginOptions
     } = svelteKitPluginOptions;
 
-    const { unstable_sentryVitePluginOptions } = svelteKitPluginOptions;
-
     sentryVitePluginsOptions = {
       ...(sentryVitePluginsOptions ? sentryVitePluginsOptions : {}),
 
-      ...deprecatedSourceMapUploadOptions,
       ...newSvelteKitPluginOptions,
 
       url: sentryUrl,
 
-      ...deprecated_unstableSourceMapUploadOptions,
       ...unstable_sentryVitePluginOptions,
 
       adapter: svelteKitPluginOptions.adapter,
       // override the plugin's debug flag with the one from the top-level options
       debug: svelteKitPluginOptions.debug,
     };
 
-    // Handle sourcemaps options - merge deprecated and new, with new taking precedence
-    if (
-      // eslint-disable-next-line typescript/no-deprecated
-      deprecatedSourceMapUploadOptions.sourcemaps ||
-      svelteKitPluginOptions.sourcemaps ||
-      deprecated_unstableSourceMapUploadOptions?.sourcemaps ||
-      unstable_sentryVitePluginOptions?.sourcemaps
-    ) {
+    if (svelteKitPluginOptions.sourcemaps || unstable_sentryVitePluginOptions?.sourcemaps) {
       sentryVitePluginsOptions.sourcemaps = {
-        // eslint-disable-next-line typescript/no-deprecated
-        ...deprecatedSourceMapUploadOptions.sourcemaps,
         ...svelteKitPluginOptions.sourcemaps,
-        // Also handle nested deprecated options from unstable plugin options
-        ...deprecated_unstableSourceMapUploadOptions?.sourcemaps,
         ...unstable_sentryVitePluginOptions?.sourcemaps,
       };
     }
 
-    // Handle release options - merge deprecated and new, with new taking precedence
-    if (
-      // eslint-disable-next-line typescript/no-deprecated
-      deprecatedSourceMapUploadOptions.release ||
-      svelteKitPluginOptions.release ||
-      deprecated_unstableSourceMapUploadOptions?.release ||
-      unstable_sentryVitePluginOptions?.release
-    ) {
+    if (svelteKitPluginOptions.release || unstable_sentryVitePluginOptions?.release) {
       sentryVitePluginsOptions.release = {
-        // eslint-disable-next-line typescript/no-deprecated
-        ...deprecatedSourceMapUploadOptions.release,
         ...svelteKitPluginOptions.release,
-        // Also handle nested deprecated options from unstable plugin options
-        ...deprecated_unstableSourceMapUploadOptions?.release,
         ...unstable_sentryVitePluginOptions?.release,
       };
     }

packages/sveltekit/src/vite/sourceMaps.ts

@@ -140,7 +140,7 @@ export async function makeCustomSentryVitePlugins(
         debug &&
           // eslint-disable-next-line no-console
           console.info(
-            `[Sentry] Automatically setting \`sourceMapsUploadOptions.sourcemaps.filesToDeleteAfterUpload: [${defaultFileDeletionGlob
+            `[Sentry] Automatically setting \`sourcemaps.filesToDeleteAfterUpload: [${defaultFileDeletionGlob
               .map(file => `"${file}"`)
               .join(', ')}]\` to delete generated source maps after they were uploaded to Sentry.`,
           );

packages/sveltekit/src/vite/types.ts

@@ -8,149 +8,6 @@ export type CustomSentryVitePluginOptions = SentryVitePluginOptions & {
   adapter?: SupportedSvelteKitAdapters;
 };
 
-/**
- * Options for the Sentry Vite plugin to customize and override the release creation and source maps upload process.
- * See [Sentry Vite Plugin Options](https://github.com/getsentry/sentry-javascript-bundler-plugins/tree/main/packages/vite-plugin#configuration) for a detailed description.
- */
-type SourceMapsUploadOptions = {
-  /**
-   * The auth token to use when uploading source maps to Sentry.
-   *
-   * Instead of specifying this option, you can also set the `SENTRY_AUTH_TOKEN` environment variable.
-   *
-   * To create an auth token, follow this guide:
-   * @see https://docs.sentry.io/product/accounts/auth-tokens/#organization-auth-tokens
-   *
-   * @deprecated Use option `authToken` instead of `sourceMapsUploadOptions.authToken`
-   */
-  authToken?: string;
-
-  /**
-   * The organization slug of your Sentry organization.
-   * Instead of specifying this option, you can also set the `SENTRY_ORG` environment variable.
-   *
-   * @deprecated Use option `org` instead of `sourceMapsUploadOptions.org`
-   */
-  org?: string;
-
-  /**
-   * The project slug of your Sentry project.
-   * Instead of specifying this option, you can also set the `SENTRY_PROJECT` environment variable.
-   *
-   * @deprecated Use option `project` instead of `sourceMapsUploadOptions.project`
-   */
-  project?: string;
-
-  /**
-   * If this flag is `true`, the Sentry plugin will collect some telemetry data and send it to Sentry.
-   * It will not collect any sensitive or user-specific data.
-   *
-   * @default true
-   * @deprecated Use option `telemetry` instead of `sourceMapsUploadOptions.telemetry`
-   */
-  telemetry?: boolean;
-
-  /**
-   * Options related to sourcemaps
-   * @deprecated Use `sourcemaps` instead of `sourceMapsUploadOptions.sourcemaps`
-   */
-  sourcemaps?: {
-    /**
-     * A glob or an array of globs that specify the build artifacts and source maps that will be uploaded to Sentry.
-     *
-     * If this option is not specified, sensible defaults based on your adapter and svelte.config.js
-     * setup will be used. Use this option to override these defaults, for instance if you have a
-     * customized build setup that diverges from SvelteKit's defaults.
-     *
-     * The globbing patterns must follow the implementation of the `glob` package.
-     * @see https://www.npmjs.com/package/glob#glob-primer
-     * @deprecated Use `sourcemaps.assets` instead of `sourceMapsUploadOptions.sourcemaps.assets`
-     */
-    assets?: string | Array<string>;
-
-    /**
-     * A glob or an array of globs that specifies which build artifacts should not be uploaded to Sentry.
-     *
-     * @default [] - By default no files are ignored. Thus, all files matching the `assets` glob
-     * or the default value for `assets` are uploaded.
-     *
-     * The globbing patterns follow the implementation of the glob package. (https://www.npmjs.com/package/glob)
-     *
-     * @deprecated Use `sourcemaps.ignore` instead of `sourceMapsUploadOptions.sourcemaps.ignore`
-     */
-    ignore?: string | Array<string>;
-
-    /**
-     * A glob or an array of globs that specifies the build artifacts that should be deleted after the artifact
-     * upload to Sentry has been completed.
-     *
-     * @default [] - By default no files are deleted.
-     *
-     * The globbing patterns follow the implementation of the glob package. (https://www.npmjs.com/package/glob)
-     *
-     * @deprecated Use `sourcemaps.filesToDeleteAfterUpload` instead of `sourceMapsUploadOptions.sourcemaps.filesToDeleteAfterUpload`
-     */
-    filesToDeleteAfterUpload?: string | Array<string>;
-  };
-
-  /**
-   * Options related to managing the Sentry releases for a build.
-   *
-   * Note: Managing releases is optional and not required for uploading source maps.
-   *
-   * @deprecated Use `release` instead of `sourceMapsUploadOptions.release`
-   */
-  release?: {
-    /**
-     * Unique identifier for the release you want to create.
-     * This value can also be specified via the SENTRY_RELEASE environment variable.
-     *
-     * Defaults to automatically detecting a value for your environment. This includes values for Cordova, Heroku,
-     * AWS CodeBuild, CircleCI, Xcode, and Gradle, and otherwise uses the git HEAD's commit SHA (the latter requires
-     * access to git CLI and for the root directory to be a valid repository).
-     *
-     * If you didn't provide a value and the plugin can't automatically detect one, no release will be created.
-     *
-     * @deprecated Use `release.name` instead of `sourceMapsUploadOptions.release.name`
-     */
-    name?: string;
-
-    /**
-     * Whether the plugin should inject release information into the build for the SDK to pick it up when
-     * sending events.
-     *
-     * Defaults to `true`.
-     *
-     * @deprecated Use `release.inject` instead of `sourceMapsUploadOptions.release.inject`
-     */
-    inject?: boolean;
-  };
-
-  /**
-   * The URL of the Sentry instance to upload the source maps to.
-   *
-   * @deprecated Use `sentryUrl` instead of `sourceMapsUploadOptions.url`
-   */
-  url?: string;
-
-  /**
-   * Options to further customize the Sentry Vite Plugin (@sentry/bundler-plugins/vite) behavior directly.
-   * Options specified in this object take precedence over the options specified in
-   * the `sourcemaps` and `release` objects.
-   *
-   * @see https://www.npmjs.com/package/@sentry/vite-plugin/v/2.14.2#options which lists all available options.
-   *
-   * Warning: Options within this object are subject to change at any time.
-   * We DO NOT guarantee semantic versioning for these options, meaning breaking
-   * changes can occur at any time within a major SDK version.
-   *
-   * Furthermore, some options are untested with SvelteKit specifically. Use with caution.
-   *
-   * @deprecated Use `unstable_sentryVitePluginOptions` instead of `sourceMapsUploadOptions.unstable_sentryVitePluginOptions`
-   */
-  unstable_sentryVitePluginOptions?: Partial<SentryVitePluginOptions>;
-};
-
 /** Options for the Sentry SvelteKit plugin */
 export type SentrySvelteKitPluginOptions = BuildTimeOptionsBase &
   UnstableVitePluginOptions<Partial<SentryVitePluginOptions>> & {
@@ -189,10 +46,4 @@ export type SentrySvelteKitPluginOptions = BuildTimeOptionsBase &
      * @default true`.
      */
     autoUploadSourceMaps?: boolean;
-
-    /** * Options related to source maps upload to Sentry
-     *
-     * @deprecated This option was deprecated as it adds unnecessary nesting. Put the options one level higher to the root-level of the Sentry Svelte plugin options.
-     */
-    sourceMapsUploadOptions?: SourceMapsUploadOptions;
   };