fix: avoid adding defne plugins if no treeshake opts specified

logaretm · logaretm · commit 66aa19c8b360 · 2025-12-04T15:43:35.000+01:00
diff --git a/packages/nextjs/src/config/webpack.ts b/packages/nextjs/src/config/webpack.ts
@@ -944,5 +944,8 @@ function setupTreeshakingFromConfig(
     defines.__SENTRY_EXCLUDE_REPLAY_WORKER__ = true;
   }
 
-  newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));
+  // Only add DefinePlugin if there are actual defines to set
+  if (Object.keys(defines).length > 0) {
+    newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));
+  }
 }

Original file line number	Diff line number	Diff line change
`@@ -944,5 +944,8 @@ function setupTreeshakingFromConfig(`
`944`	`944`	`defines.__SENTRY_EXCLUDE_REPLAY_WORKER__ = true;`
`945`	`945`	`}`
`946`	`946`
`947`		`- newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));`
	`947`	`+ // Only add DefinePlugin if there are actual defines to set`
	`948`	`+ if (Object.keys(defines).length > 0) {`
	`949`	`+ newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));`
	`950`	`+ }`
`948`	`951`	`}`