feat(build): preserveModules: false

Author: timfish

packages/browser-utils/rollup.npm.config.mjs

@@ -9,7 +9,7 @@ export default makeNPMConfigVariants(
         // set preserveModules to true because we don't want to bundle everything into one file.
         preserveModules:
           process.env.SENTRY_BUILD_PRESERVE_MODULES === undefined
-            ? true
+            ? false
             : Boolean(process.env.SENTRY_BUILD_PRESERVE_MODULES),
       },
     },

packages/browser/rollup.npm.config.mjs

@@ -11,7 +11,7 @@ export default makeNPMConfigVariants(
         // set preserveModules to true because we don't want to bundle everything into one file.
         preserveModules:
           process.env.SENTRY_BUILD_PRESERVE_MODULES === undefined
-            ? true
+            ? false
             : Boolean(process.env.SENTRY_BUILD_PRESERVE_MODULES),
       },
     },

packages/bun/rollup.npm.config.mjs

@@ -1,3 +1,9 @@
 import { makeBaseNPMConfig, makeNPMConfigVariants } from '@sentry-internal/rollup-utils';
 
-export default makeNPMConfigVariants(makeBaseNPMConfig());
+export default makeNPMConfigVariants(makeBaseNPMConfig({
+  packageSpecificConfig: {
+    output: {
+      preserveModules: false
+    }
+  }
+}));

packages/core/rollup.npm.config.mjs

@@ -23,7 +23,7 @@ export default makeNPMConfigVariants(
         // set preserveModules to true because we don't want to bundle everything into one file.
         preserveModules:
           process.env.SENTRY_BUILD_PRESERVE_MODULES === undefined
-            ? true
+            ? false
             : Boolean(process.env.SENTRY_BUILD_PRESERVE_MODULES),
       },
       plugins: [

packages/deno/rollup.npm.config.mjs

@@ -1,3 +1,9 @@
 import { makeBaseNPMConfig, makeNPMConfigVariants } from '@sentry-internal/rollup-utils';
 
-export default makeNPMConfigVariants(makeBaseNPMConfig(), { emitCjs: false });
+export default makeNPMConfigVariants(makeBaseNPMConfig({
+  packageSpecificConfig: {
+    output: {
+      preserveModules: false
+    }
+  }
+}), { emitCjs: false });

packages/nestjs/rollup.npm.config.mjs

@@ -3,5 +3,8 @@ import { makeBaseNPMConfig, makeNPMConfigVariants } from '@sentry-internal/rollu
 export default makeNPMConfigVariants(
   makeBaseNPMConfig({
     entrypoints: ['src/index.ts', 'src/setup.ts'],
+    output: {
+      preserveModules: false
+    }
   }),
 );

packages/node-core/rollup.npm.config.mjs

@@ -24,7 +24,7 @@ export default [
         output: {
           // set exports to 'named' or 'auto' so that rollup doesn't warn
           exports: 'named',
-          preserveModules: true,
+          preserveModules: false,
         },
         plugins: [
           replace({

packages/node-native/rollup.npm.config.mjs

@@ -8,7 +8,7 @@ export default makeNPMConfigVariants(
         dir: 'build',
         // set exports to 'named' or 'auto' so that rollup doesn't warn
         exports: 'named',
-        preserveModules: true,
+        preserveModules: false,
       },
     },
   }),

packages/node/rollup.npm.config.mjs

@@ -9,7 +9,7 @@ export default [
         output: {
           // set exports to 'named' or 'auto' so that rollup doesn't warn
           exports: 'named',
-          preserveModules: true,
+          preserveModules: false,
         },
       },
     }),

packages/react/rollup.npm.config.mjs

@@ -3,6 +3,9 @@ import { makeBaseNPMConfig, makeNPMConfigVariants } from '@sentry-internal/rollu
 export default makeNPMConfigVariants(
   makeBaseNPMConfig({
     packageSpecificConfig: {
+      output: {
+        preserveModules: true,
+      },
       external: ['react', 'react/jsx-runtime'],
     },
     sucrase: {