pull replacement of browser bundle flag into central config

Author: lobsterkatie

packages/browser/rollup.config.js

@@ -1,9 +1,8 @@
 import { terser } from 'rollup-plugin-terser';
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
-import replace from '@rollup/plugin-replace';
 
-import { makeLicensePlugin, paths } from '../../rollup.config';
+import { makeLicensePlugin, markAsBrowserBuild, paths } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin();
 
@@ -49,16 +48,8 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
-  replace({
-    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
-    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
-    // syntax error)
-    preventAssignment: true,
-    // the replacements to make
-    values: {
-      __SENTRY_BROWSER_BUNDLE__: true,
-    },
-  }),
+  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
+  markAsBrowserBuild,
   resolve({
     mainFields: ['module'],
   }),

packages/integrations/rollup.config.js

@@ -4,9 +4,8 @@ import { terser } from 'rollup-plugin-terser';
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
 import commonjs from '@rollup/plugin-commonjs';
-import replace from '@rollup/plugin-replace';
 
-import { paths } from '../../rollup.config';
+import { markAsBrowserBuild, paths } from '../../rollup.config';
 
 const terserInstance = terser({
   mangle: {
@@ -35,16 +34,8 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
-  replace({
-    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
-    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
-    // syntax error)
-    preventAssignment: true,
-    // the replacements to make
-    values: {
-      __SENTRY_BROWSER_BUNDLE__: true,
-    },
-  }),
+  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
+  markAsBrowserBuild,
   resolve({
     mainFields: ['module'],
   }),

packages/tracing/rollup.config.js

@@ -1,8 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
-import replace from '@rollup/plugin-replace';
 
-import { makeLicensePlugin, paths, terserPlugin } from '../../rollup.config';
+import { makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin('@sentry/tracing & @sentry/browser');
 
@@ -19,16 +18,8 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
-  replace({
-    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
-    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
-    // syntax error)
-    preventAssignment: true,
-    // the replacements to make
-    values: {
-      __SENTRY_BROWSER_BUNDLE__: true,
-    },
-  }),
+  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
+  markAsBrowserBuild,
   resolve({
     mainFields: ['module'],
   }),

packages/vue/rollup.config.js

@@ -1,8 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
-import replace from '@rollup/plugin-replace';
 
-import { makeLicensePlugin, paths, terserPlugin } from '../../rollup.config';
+import { makeLicensePlugin, markAsBrowserBuild, paths, terserPlugin } from '../../rollup.config';
 
 const licensePlugin = makeLicensePlugin();
 
@@ -19,16 +18,8 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
-  replace({
-    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
-    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
-    // syntax error)
-    preventAssignment: true,
-    // the replacements to make
-    values: {
-      __SENTRY_BROWSER_BUNDLE__: true,
-    },
-  }),
+  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
+  markAsBrowserBuild,
   resolve({
     mainFields: ['module'],
   }),

packages/wasm/rollup.config.js

@@ -1,8 +1,7 @@
 import typescript from 'rollup-plugin-typescript2';
 import resolve from '@rollup/plugin-node-resolve';
-import replace from '@rollup/plugin-replace';
 
-import { paths, terserPlugin } from '../../rollup.config';
+import { paths, markAsBrowserBuild, terserPlugin } from '../../rollup.config';
 
 const plugins = [
   typescript({
@@ -17,16 +16,8 @@ const plugins = [
     },
     include: ['*.ts+(|x)', '**/*.ts+(|x)', '../**/*.ts+(|x)'],
   }),
-  replace({
-    // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
-    // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
-    // syntax error)
-    preventAssignment: true,
-    // the replacements to make
-    values: {
-      __SENTRY_BROWSER_BUNDLE__: true,
-    },
-  }),
+  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
+  markAsBrowserBuild,
   resolve({
     mainFields: ['module'],
   }),

rollup.config.js

@@ -3,6 +3,7 @@
  */
 
 import license from 'rollup-plugin-license';
+import replace from '@rollup/plugin-replace';
 import { terser } from 'rollup-plugin-terser';
 
 export const paths = {
@@ -46,3 +47,14 @@ export const terserPlugin = terser({
     comments: false,
   },
 });
+
+export const markAsBrowserBuild = replace({
+  // don't replace `__placeholder__` where it's followed immediately by a single `=` (to prevent ending up
+  // with something of the form `let "replacementValue" = "some assigned value"`, which would cause a
+  // syntax error)
+  preventAssignment: true,
+  // the replacement to make
+  values: {
+    __SENTRY_BROWSER_BUNDLE__: true,
+  },
+});