unify buildPluginOptions

chargome · chargome · commit ad0529934270 · 2025-09-15T18:22:49.000+02:00
diff --git a/packages/nextjs/src/config/getBuildPluginOptions.ts b/packages/nextjs/src/config/getBuildPluginOptions.ts
@@ -1,40 +1,103 @@
 import type { Options as SentryBuildPluginOptions } from '@sentry/bundler-plugin-core';
 import * as path from 'path';
-import type { SentryBuildOptions } from './types';
+import type { BuildContext, NextConfigObject, SentryBuildOptions } from './types';
 
 /**
- * Get Sentry Build Plugin options for the runAfterProductionCompile hook.
+ * Get Sentry Build Plugin options for both webpack and turbopack builds.
+ * These options can be used in two ways:
+ * 1. The build can be done in a single operation after the production build completes
+ * 2. The build can be done in multiple operations, one for each webpack build
  */
 export function getBuildPluginOptions({
   sentryBuildOptions,
   releaseName,
   distDirAbsPath,
+  buildTool,
+  useRunAfterProductionCompileHook,
 }: {
   sentryBuildOptions: SentryBuildOptions;
   releaseName: string | undefined;
   distDirAbsPath: string;
+  buildTool: 'webpack-client' | 'webpack-nodejs' | 'webpack-edge' | 'after-production-compile';
+  useRunAfterProductionCompileHook?: boolean; // Whether the user has opted into using the experimental hook
 }): SentryBuildPluginOptions {
   const sourcemapUploadAssets: string[] = [];
   const sourcemapUploadIgnore: string[] = [];
-
   const filesToDeleteAfterUpload: string[] = [];
 
   // We need to convert paths to posix because Glob patterns use `\` to escape
   // glob characters. This clashes with Windows path separators.
   // See: https://www.npmjs.com/package/glob
   const normalizedDistDirAbsPath = distDirAbsPath.replace(/\\/g, '/');
 
-  sourcemapUploadAssets.push(
-    path.posix.join(normalizedDistDirAbsPath, '**'), // Next.js build output
-  );
-  if (sentryBuildOptions.sourcemaps?.deleteSourcemapsAfterUpload) {
-    filesToDeleteAfterUpload.push(
-      path.posix.join(normalizedDistDirAbsPath, '**', '*.js.map'),
-      path.posix.join(normalizedDistDirAbsPath, '**', '*.mjs.map'),
-      path.posix.join(normalizedDistDirAbsPath, '**', '*.cjs.map'),
+  const loggerPrefix = {
+    'webpack-nodejs': '[@sentry/nextjs - Node.js]',
+    'webpack-edge': '[@sentry/nextjs - Edge]',
+    'webpack-client': '[@sentry/nextjs - Client]',
+    'after-production-compile': '[@sentry/nextjs - After Production Compile]',
+  }[buildTool];
+
+  if (buildTool === 'after-production-compile') {
+    // Turbopack builds
+    sourcemapUploadAssets.push(
+      path.posix.join(normalizedDistDirAbsPath, '**'), // Next.js build output
     );
+
+    if (sentryBuildOptions.sourcemaps?.deleteSourcemapsAfterUpload) {
+      filesToDeleteAfterUpload.push(
+        path.posix.join(normalizedDistDirAbsPath, '**', '*.js.map'),
+        path.posix.join(normalizedDistDirAbsPath, '**', '*.mjs.map'),
+        path.posix.join(normalizedDistDirAbsPath, '**', '*.cjs.map'),
+      );
+    }
+  } else {
+    if (buildTool === 'webpack-nodejs' || buildTool === 'webpack-edge') {
+      sourcemapUploadAssets.push(
+        path.posix.join(distDirAbsPath, 'server', '**'), // Standard output location for server builds
+        path.posix.join(distDirAbsPath, 'serverless', '**'), // Legacy output location for serverless Next.js
+      );
+    } else {
+      // Client builds
+      if (sentryBuildOptions.widenClientFileUpload) {
+        sourcemapUploadAssets.push(path.posix.join(distDirAbsPath, 'static', 'chunks', '**'));
+      } else {
+        sourcemapUploadAssets.push(
+          path.posix.join(distDirAbsPath, 'static', 'chunks', 'pages', '**'),
+          path.posix.join(distDirAbsPath, 'static', 'chunks', 'app', '**'),
+        );
+      }
+
+      // We want to include main-* files if widenClientFileUpload is true as they have proven to be useful
+      if (!sentryBuildOptions.widenClientFileUpload) {
+        sourcemapUploadIgnore.push(path.posix.join(distDirAbsPath, 'static', 'chunks', 'main-*'));
+      }
+
+      // Always ignore framework, polyfills, and webpack files
+      sourcemapUploadIgnore.push(
+        path.posix.join(distDirAbsPath, 'static', 'chunks', 'framework-*'),
+        path.posix.join(distDirAbsPath, 'static', 'chunks', 'framework.*'),
+        path.posix.join(distDirAbsPath, 'static', 'chunks', 'polyfills-*'),
+        path.posix.join(distDirAbsPath, 'static', 'chunks', 'webpack-*'),
+      );
+
+      // File deletion for webpack client builds
+      if (sentryBuildOptions.sourcemaps?.deleteSourcemapsAfterUpload) {
+        filesToDeleteAfterUpload.push(
+          // 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(distDirAbsPath, 'static', '**', '*.js.map'),
+          path.posix.join(distDirAbsPath, 'static', '**', '*.mjs.map'),
+          path.posix.join(distDirAbsPath, 'static', '**', '*.cjs.map'),
+        );
+      }
+    }
   }
 
+  // If the user has opted into using the experimental hook, we skip sourcemaps and release management in the plugin
+  // to avoid double sourcemap uploads.
+  const shouldSkipSourcemapsUpload = useRunAfterProductionCompileHook && buildTool.startsWith('webpack');
+
   return {
     authToken: sentryBuildOptions.authToken,
     headers: sentryBuildOptions.headers,
@@ -43,14 +106,17 @@ export function getBuildPluginOptions({
     telemetry: sentryBuildOptions.telemetry,
     debug: sentryBuildOptions.debug,
     errorHandler: sentryBuildOptions.errorHandler,
-    reactComponentAnnotation: {
-      ...sentryBuildOptions.reactComponentAnnotation,
-      ...sentryBuildOptions.unstable_sentryWebpackPluginOptions?.reactComponentAnnotation,
-    },
+    reactComponentAnnotation:
+      buildTool === 'after-production-compile'
+        ? undefined
+        : {
+            ...sentryBuildOptions.reactComponentAnnotation,
+            ...sentryBuildOptions.unstable_sentryWebpackPluginOptions?.reactComponentAnnotation,
+          },
     silent: sentryBuildOptions.silent,
     url: sentryBuildOptions.sentryUrl,
     sourcemaps: {
-      disable: sentryBuildOptions.sourcemaps?.disable,
+      disable: sentryBuildOptions.sourcemaps?.disable || shouldSkipSourcemapsUpload,
       rewriteSources(source) {
         if (source.startsWith('webpack://_N_E/')) {
           return source.replace('webpack://_N_E/', '');
@@ -62,7 +128,7 @@ export function getBuildPluginOptions({
       },
       assets: sentryBuildOptions.sourcemaps?.assets ?? sourcemapUploadAssets,
       ignore: sentryBuildOptions.sourcemaps?.ignore ?? sourcemapUploadIgnore,
-      filesToDeleteAfterUpload,
+      filesToDeleteAfterUpload: filesToDeleteAfterUpload.length > 0 ? filesToDeleteAfterUpload : undefined,
       ...sentryBuildOptions.unstable_sentryWebpackPluginOptions?.sourcemaps,
     },
     release:
@@ -87,11 +153,35 @@ export function getBuildPluginOptions({
       ...sentryBuildOptions.bundleSizeOptimizations,
     },
     _metaOptions: {
-      loggerPrefixOverride: '[@sentry/nextjs]',
+      loggerPrefixOverride: loggerPrefix,
       telemetry: {
         metaFramework: 'nextjs',
       },
     },
     ...sentryBuildOptions.unstable_sentryWebpackPluginOptions,
   };
 }
+
+/**
+ * Legacy function for webpack builds. Now calls the unified getBuildPluginOptions function.
+ * @deprecated Use getBuildPluginOptions instead
+ */
+export function getWebpackPluginOptions(
+  buildContext: BuildContext,
+  sentryBuildOptions: SentryBuildOptions,
+  releaseName: string | undefined,
+): SentryBuildPluginOptions {
+  const { isServer, config: userNextConfig, dir, nextRuntime } = buildContext;
+  const buildTool = isServer ? (nextRuntime === 'edge' ? 'webpack-edge' : 'webpack-nodejs') : 'webpack-client';
+
+  const projectDir = dir.replace(/\\/g, '/');
+  const distDir = (userNextConfig as NextConfigObject).distDir?.replace(/\\/g, '/') ?? '.next';
+  const distDirAbsPath = path.posix.join(projectDir, distDir);
+
+  return getBuildPluginOptions({
+    sentryBuildOptions,
+    releaseName,
+    distDirAbsPath,
+    buildTool,
+  });
+}
diff --git a/packages/nextjs/src/config/handleRunAfterProductionCompile.ts b/packages/nextjs/src/config/handleRunAfterProductionCompile.ts
@@ -11,12 +11,6 @@ export async function handleRunAfterProductionCompile(
   { releaseName, distDir, buildTool }: { releaseName?: string; distDir: string; buildTool: 'webpack' | 'turbopack' },
   sentryBuildOptions: SentryBuildOptions,
 ): Promise<void> {
-  // We don't want to do anything for webpack at this point because the plugin already handles this
-  // TODO: Actually implement this for webpack as well
-  if (buildTool === 'webpack') {
-    return;
-  }
-
   if (sentryBuildOptions.debug) {
     // eslint-disable-next-line no-console
     console.debug('[@sentry/nextjs] Running runAfterProductionCompile logic.');
@@ -36,17 +30,17 @@ export async function handleRunAfterProductionCompile(
     return;
   }
 
-  const sentryBuildPluginManager = createSentryBuildPluginManager(
-    getBuildPluginOptions({
-      sentryBuildOptions,
-      releaseName,
-      distDirAbsPath: distDir,
-    }),
-    {
-      buildTool,
-      loggerPrefix: '[@sentry/nextjs]',
-    },
-  );
+  const options = getBuildPluginOptions({
+    sentryBuildOptions,
+    releaseName,
+    distDirAbsPath: distDir,
+    buildTool: 'after-production-compile',
+  });
+
+  const sentryBuildPluginManager = createSentryBuildPluginManager(options, {
+    buildTool,
+    loggerPrefix: '[@sentry/nextjs - After Production Compile]',
+  });
 
   await sentryBuildPluginManager.telemetry.emitBundlerPluginExecutionSignal();
   await sentryBuildPluginManager.createRelease();
diff --git a/packages/nextjs/src/config/webpack.ts b/packages/nextjs/src/config/webpack.ts
@@ -7,6 +7,7 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { sync as resolveSync } from 'resolve';
 import type { VercelCronsConfig } from '../common/types';
+import { getBuildPluginOptions } from './getBuildPluginOptions';
 import type { RouteManifest } from './manifest/types';
 // Note: If you need to import a type from Webpack, do it in `types.ts` and export it from there. Otherwise, our
 // circular dependency check thinks this file is importing from itself. See https://github.com/pahen/madge/issues/306.
@@ -22,7 +23,6 @@ import type {
   WebpackEntryProperty,
 } from './types';
 import { getNextjsVersion } from './util';
-import { getWebpackPluginOptions } from './webpackPluginOptions';
 
 // Next.js runs webpack 3 times, once for the client, the server, and for edge. Because we don't want to print certain
 // warnings 3 times, we keep track of them here.
@@ -408,9 +408,21 @@ export function constructWebpackConfigFunction(
         }
 
         newConfig.plugins = newConfig.plugins || [];
+        const { config: userNextConfig, dir, nextRuntime } = buildContext;
+        const buildTool = isServer ? (nextRuntime === 'edge' ? 'webpack-edge' : 'webpack-nodejs') : 'webpack-client';
+        const projectDir = dir.replace(/\\/g, '/');
+        const distDir = (userNextConfig as NextConfigObject).distDir?.replace(/\\/g, '/') ?? '.next';
+        const distDirAbsPath = path.posix.join(projectDir, distDir);
+
         const sentryWebpackPluginInstance = sentryWebpackPlugin(
-          getWebpackPluginOptions(buildContext, userSentryOptions, releaseName),
+          getBuildPluginOptions({
+            sentryBuildOptions: userSentryOptions,
+            releaseName,
+            distDirAbsPath,
+            buildTool,
+          }),
         );
+
         // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
         sentryWebpackPluginInstance._name = 'sentry-webpack-plugin'; // For tests and debugging. Serves no other purpose.
         newConfig.plugins.push(sentryWebpackPluginInstance);
diff --git a/packages/nextjs/src/config/webpackPluginOptions.ts b/packages/nextjs/src/config/webpackPluginOptions.ts
