pull common output options into central config

Author: lobsterkatie

packages/browser/rollup.config.js

@@ -2,7 +2,7 @@ import { terser } from 'rollup-plugin-terser';
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 
-import { makeLicensePlugin, markAsBrowserBuild, paths } from '../../rollup.config';
+import { baseBundleConfig, makeLicensePlugin, markAsBrowserBuild, paths } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin();
 
@@ -58,11 +58,9 @@ const plugins = [
 const bundleConfig = {
   input: 'src/index.ts',
   output: {
+    ...baseBundleConfig.output,
     format: 'iife',
     name: 'Sentry',
-    sourcemap: true,
-    strict: false,
-    esModule: false,
   },
   context: 'window',
   plugins: [...plugins, licensePlugin],

packages/integrations/rollup.config.js

@@ -5,7 +5,7 @@ import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 import commonjs from '@rollup/plugin-commonjs';
 
-import { markAsBrowserBuild, paths } from '../../rollup.config';
+import { baseBundleConfig, markAsBrowserBuild, paths } from '../../rollup.config';
 
 const terserInstance = terser({
   mangle: {
@@ -78,11 +78,9 @@ function loadAllIntegrations() {
           intro: 'var exports = {};',
           outro: mergeIntoSentry(),
           footer: '}(window));',
+          ...baseBundleConfig.output,
           file: `build/${file.replace('.ts', build.extension)}`,
           format: 'cjs',
-          sourcemap: true,
-          strict: false,
-          esModule: false,
         },
         plugins: build.plugins,
         treeshake: 'smallest',

packages/tracing/rollup.config.js

@@ -1,7 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 
-import { makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
+import { baseBundleConfig, makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin('@sentry/tracing & @sentry/browser');
 
@@ -28,11 +28,9 @@ const plugins = [
 const bundleConfig = {
   input: 'src/index.ts',
   output: {
+    ...baseBundleConfig.output,
     format: 'iife',
     name: 'Sentry',
-    sourcemap: true,
-    strict: false,
-    esModule: false,
   },
   context: 'window',
   plugins: [...plugins, licensePlugin],

packages/vue/rollup.config.js

@@ -1,7 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 
-import { makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
+import { baseBundleConfig, makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin();
 
@@ -28,11 +28,9 @@ const plugins = [
 const bundleConfig = {
   input: 'src/index.ts',
   output: {
+    ...baseBundleConfig.output,
     format: 'iife',
     name: 'Sentry',
-    sourcemap: true,
-    strict: false,
-    esModule: false,
   },
   context: 'window',
   plugins: [...plugins, licensePlugin],

packages/wasm/rollup.config.js

@@ -1,7 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 
-import { paths, markAsBrowserBuild, terserPlugin } from '../../rollup.config';
+import { baseBundleConfig, paths, markAsBrowserBuild, terserPlugin } from '../../rollup.config';
 
 const plugins = [
   typescript({
@@ -54,11 +54,9 @@ function loadAllIntegrations() {
         intro: 'var exports = {};',
         outro: mergeIntoSentry(),
         footer: '}(window));',
+        ...baseBundleConfig.output,
         file: `build/wasm${build.extension}`,
         format: 'cjs',
-        sourcemap: true,
-        strict: false,
-        esModule: false,
       },
       plugins: build.plugins,
       treeshake: 'smallest',

rollup.config.js

@@ -58,3 +58,11 @@ export const markAsBrowserBuild = replace({
     __SENTRY_BROWSER_BUNDLE__: true,
   },
 });
+
+export const baseBundleConfig = {
+  output: {
+    sourcemap: true,
+    strict: false,
+    esModule: false,
+  },
+};