fix: revert more changes

Author: logaretm

dev-packages/rollup-utils/npmHelpers.mjs

@@ -50,6 +50,10 @@ export function makeBaseNPMConfig(options = {}) {
       // Include __esModule property when there is a default prop
       esModule: 'if-default-prop',
 
+      // output individual files rather than one big bundle
+      preserveModules: true,
+
+      // output files relative to the src directory
       preserveModulesRoot: 'src',
 
       // Allow wrappers or helper functions generated by rollup to use any ES2015 features

dev-packages/test-utils/rollup.npm.config.mjs

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

packages/feedback/rollup.npm.config.mjs

@@ -7,10 +7,11 @@ export default makeNPMConfigVariants(
       output: {
         // set exports to 'named' or 'auto' so that rollup doesn't warn
         exports: 'named',
-        // set preserveModules to true because we don't want to bundle everything into one file.
+        // set preserveModules to false because for feedback we actually 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/node-core/rollup.npm.config.mjs

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

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

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

packages/node/rollup.npm.config.mjs

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

packages/opentelemetry/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/profiling-node/rollup.npm.config.mjs

@@ -10,7 +10,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/remix/rollup.npm.config.mjs

@@ -15,7 +15,6 @@ export default [
         output: {
           // make it so Rollup calms down about the fact that we're combining default and named exports
           exports: 'named',
-          preserveModules: true,
         },
       },
       sucrase: {

packages/replay-canvas/rollup.npm.config.mjs

@@ -7,10 +7,11 @@ export default makeNPMConfigVariants(
       output: {
         // set exports to 'named' or 'auto' so that rollup doesn't warn
         exports: 'named',
-        // set preserveModules to true because we don't want to bundle everything into one file.
+        // set preserveModules to false because for Replay we actually want
+        // to bundle everything into one file.
         preserveModules:
           process.env.SENTRY_BUILD_PRESERVE_MODULES === undefined
-            ? true
+            ? false
             : Boolean(process.env.SENTRY_BUILD_PRESERVE_MODULES),
       },
     },