fix: avoid adding defne plugins if no treeshake opts specified

logaretm · logaretm · commit 547fccf2f660 · 2025-12-05T16:23:13.000+01:00
diff --git a/packages/nextjs/src/config/webpack.ts b/packages/nextjs/src/config/webpack.ts
@@ -941,5 +941,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
`@@ -941,5 +941,8 @@ function setupTreeshakingFromConfig(`
`941`	`941`	`defines.__SENTRY_EXCLUDE_REPLAY_WORKER__ = true;`
`942`	`942`	`}`
`943`	`943`
`944`		`- newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));`
	`944`	`+ // Only add DefinePlugin if there are actual defines to set`
	`945`	`+ if (Object.keys(defines).length > 0) {`
	`946`	`+ newConfig.plugins.push(new buildContext.webpack.DefinePlugin(defines));`
	`947`	`+ }`
`945`	`948`	`}`