fix: widenClientFileUpload should be a top-level

Author: logaretm

packages/nextjs/src/config/getBuildPluginOptions.ts

@@ -241,7 +241,8 @@ export function getBuildPluginOptions({
   const normalizedDistDirAbsPath = normalizePathForGlob(distDirAbsPath);
 
   const loggerPrefix = LOGGER_PREFIXES[buildTool];
-  const widenClientFileUpload = sentryBuildOptions.webpack?.widenClientFileUpload ?? false;
+  // widenClientFileUpload is used for both webpack and turbopack, so it's a top-level option
+  const widenClientFileUpload = sentryBuildOptions.widenClientFileUpload ?? false;
   const deleteSourcemapsAfterUpload = sentryBuildOptions.sourcemaps?.deleteSourcemapsAfterUpload ?? false;
 
   const sourcemapUploadAssets = createSourcemapUploadAssetPatterns(

packages/nextjs/src/config/types.ts

@@ -58,19 +58,6 @@ export type NextConfigObject = {
 };
 
 export type SentryBuildWebpackOptions = {
-  /**
-   * Include Next.js-internal code and code from dependencies when uploading source maps.
-   *
-   * Note: Enabling this option can lead to longer build times.
-   * Disabling this option will leave you without readable stacktraces for dependencies and Next.js-internal code.
-   *
-   * Defaults to `false`.
-   */
-  // Enabling this option may upload a lot of source maps and since the sourcemap upload endpoint in Sentry is super
-  // slow we don't enable it by default so that we don't opaquely increase build times for users.
-  // TODO: Add an alias to this function called "uploadSourceMapsForDependencies"
-  widenClientFileUpload?: boolean;
-
   /**
    * Automatically instrument Next.js data fetching methods and Next.js API routes with error and performance monitoring.
    * Defaults to `true`.
@@ -492,7 +479,8 @@ export type SentryBuildOptions = {
    * Disabling this option will leave you without readable stacktraces for dependencies and Next.js-internal code.
    *
    * Defaults to `false`.
-   * @deprecated Use `webpack.widenClientFileUpload` instead.
+   *
+   * This option applies to both webpack and turbopack builds.
    */
   // Enabling this option may upload a lot of source maps and since the sourcemap upload endpoint in Sentry is super
   // slow we don't enable it by default so that we don't opaquely increase build times for users.

packages/nextjs/src/config/withSentryConfig.ts

@@ -150,12 +150,6 @@ function migrateDeprecatedWebpackOptions(userSentryOptions: SentryBuildOptions):
     deprecatedMessage('excludeServerRoutes', 'webpack.excludeServerRoutes'),
   );
 
-  webpack.widenClientFileUpload = withDeprecatedFallback(
-    webpack.widenClientFileUpload,
-    userSentryOptions.widenClientFileUpload,
-    deprecatedMessage('widenClientFileUpload', 'webpack.widenClientFileUpload'),
-  );
-
   webpack.unstable_sentryWebpackPluginOptions = withDeprecatedFallback(
     webpack.unstable_sentryWebpackPluginOptions,
     userSentryOptions.unstable_sentryWebpackPluginOptions,

packages/nextjs/test/config/getBuildPluginOptions.test.ts

@@ -129,9 +129,7 @@ describe('getBuildPluginOptions', () => {
       const result = getBuildPluginOptions({
         sentryBuildOptions: {
           ...baseSentryOptions,
-          webpack: {
-            widenClientFileUpload: true,
-          },
+          widenClientFileUpload: true, // top-level option
         },
         releaseName: mockReleaseName,
         distDirAbsPath: mockDistDirAbsPath,

packages/nextjs/test/config/withSentryConfig.test.ts

@@ -387,12 +387,10 @@ describe('withSentryConfig', () => {
           expect.stringContaining('[@sentry/nextjs] DEPRECATION WARNING: disableLogger is deprecated'),
         );
         expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining('Use webpack.treeshake.debugLogs instead'));
-        expect(consoleWarnSpy).toHaveBeenCalledWith(
+        // widenClientFileUpload is not deprecated as it's used for both webpack and turbopack
+        expect(consoleWarnSpy).not.toHaveBeenCalledWith(
           expect.stringContaining('[@sentry/nextjs] DEPRECATION WARNING: widenClientFileUpload is deprecated'),
         );
-        expect(consoleWarnSpy).toHaveBeenCalledWith(
-          expect.stringContaining('Use webpack.widenClientFileUpload instead'),
-        );
       });
 
       it('does not warn when using new webpack path', () => {
@@ -403,8 +401,8 @@ describe('withSentryConfig', () => {
             treeshake: {
               debugLogs: true,
             },
-            widenClientFileUpload: true,
           },
+          widenClientFileUpload: true, // top-level option, not deprecated
         };
 
         materializeFinalNextConfig(exportedNextConfig, undefined, sentryOptions);