don't delete server source maps

Author: chargome

packages/nextjs/src/config/getBuildPluginOptions.ts

@@ -53,17 +53,24 @@ export function getBuildPluginOptions({
       sourcemapUploadAssets.push(path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', '**'));
     } else {
       // Webpack client builds
-      sourcemapUploadAssets.push(
-        path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', 'pages', '**'),
-        path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', 'app', '**'),
-      );
+      if (sentryBuildOptions.widenClientFileUpload) {
+        sourcemapUploadAssets.push(path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', '**'));
+      } else {
+        sourcemapUploadAssets.push(
+          path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', 'pages', '**'),
+          path.posix.join(normalizedDistDirAbsPath, 'static', 'chunks', 'app', '**'),
+        );
+      }
     }
 
     if (sentryBuildOptions.sourcemaps?.deleteSourcemapsAfterUpload) {
       filesToDeleteAfterUpload.push(
-        path.posix.join(normalizedDistDirAbsPath, '**', '*.js.map'),
-        path.posix.join(normalizedDistDirAbsPath, '**', '*.mjs.map'),
-        path.posix.join(normalizedDistDirAbsPath, '**', '*.cjs.map'),
+        // We only care to delete client bundle source maps because they would be the ones being served.
+        // Removing the server source maps crashes Vercel builds for (thus far) unknown reasons:
+        // https://github.com/getsentry/sentry-javascript/issues/13099
+        path.posix.join(normalizedDistDirAbsPath, 'static', '**', '*.js.map'),
+        path.posix.join(normalizedDistDirAbsPath, 'static', '**', '*.mjs.map'),
+        path.posix.join(normalizedDistDirAbsPath, 'static', '**', '*.cjs.map'),
       );
     }
   } else {

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

@@ -313,9 +313,9 @@ describe('getBuildPluginOptions', () => {
       });
 
       expect(result.sourcemaps?.filesToDeleteAfterUpload).toEqual([
-        '/path/to/.next/**/*.js.map',
-        '/path/to/.next/**/*.mjs.map',
-        '/path/to/.next/**/*.cjs.map',
+        '/path/to/.next/static/**/*.js.map',
+        '/path/to/.next/static/**/*.mjs.map',
+        '/path/to/.next/static/**/*.cjs.map',
       ]);
     });
 
@@ -336,9 +336,9 @@ describe('getBuildPluginOptions', () => {
       });
 
       expect(result.sourcemaps?.filesToDeleteAfterUpload).toEqual([
-        '/path/to/.next/**/*.js.map',
-        '/path/to/.next/**/*.mjs.map',
-        '/path/to/.next/**/*.cjs.map',
+        '/path/to/.next/static/**/*.js.map',
+        '/path/to/.next/static/**/*.mjs.map',
+        '/path/to/.next/static/**/*.cjs.map',
       ]);
     });
 
@@ -802,9 +802,9 @@ describe('getBuildPluginOptions', () => {
         `${complexPath}/static/chunks/**`,
       ]);
       expect(result.sourcemaps?.filesToDeleteAfterUpload).toEqual([
-        `${complexPath}/**/*.js.map`,
-        `${complexPath}/**/*.mjs.map`,
-        `${complexPath}/**/*.cjs.map`,
+        `${complexPath}/static/**/*.js.map`,
+        `${complexPath}/static/**/*.mjs.map`,
+        `${complexPath}/static/**/*.cjs.map`,
       ]);
     });
   });