From 9fbe555f25200476224536ce09e74b89efe9de9a Mon Sep 17 00:00:00 2001
From: Katie Byers <lobsterkatie@gmail.com>
Date: Mon, 28 Feb 2022 23:11:42 -0800
Subject: [PATCH] use function to generate base config for integrations

---
 packages/integrations/rollup.config.js | 73 +++++++++-----------------
 1 file changed, 24 insertions(+), 49 deletions(-)

diff --git a/packages/integrations/rollup.config.js b/packages/integrations/rollup.config.js
index 511d95588dc3..85672fccc6f9 100644
--- a/packages/integrations/rollup.config.js
+++ b/packages/integrations/rollup.config.js
@@ -1,39 +1,8 @@
 import * as fs from 'fs';
 
-import { terser } from 'rollup-plugin-terser';
 import commonjs from '@rollup/plugin-commonjs';
 
-import {
-  addOnBundleConfig,
-  baseBundleConfig,
-  markAsBrowserBuild,
-  nodeResolvePlugin,
-  typescriptPluginES5,
-} from '../../rollup.config';
-
-const terserInstance = terser({
-  mangle: {
-    // captureExceptions and captureMessage are public API methods and they don't need to be listed here
-    // as mangler doesn't touch user-facing thing, however sentryWrapped is not, and it would be mangled into a minified version.
-    // We need those full names to correctly detect our internal frames for stripping.
-    // I listed all of them here just for the clarity sake, as they are all used in the frames manipulation process.
-    reserved: ['captureException', 'captureMessage', 'sentryWrapped'],
-    properties: {
-      regex: /^_[^_]/,
-    },
-  },
-  output: {
-    comments: false,
-  },
-});
-
-const plugins = [
-  typescriptPluginES5,
-  // replace `__SENTRY_BROWSER_BUNDLE__` with `true` to enable treeshaking of non-browser code
-  markAsBrowserBuild,
-  nodeResolvePlugin,
-  commonjs(),
-];
+import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
 
 function allIntegrations() {
   return fs.readdirSync('./src').filter(file => file != 'index.ts');
@@ -41,29 +10,35 @@ function allIntegrations() {
 
 function loadAllIntegrations() {
   const builds = [];
-  [
-    {
-      extension: '.js',
-      plugins,
-    },
-    {
-      extension: '.min.js',
-      plugins: [...plugins, terserInstance],
-    },
-  ].forEach(build => {
-    builds.push(
-      ...allIntegrations().map(file => ({
+
+  allIntegrations().forEach(file => {
+    const baseBundleConfig = makeBaseBundleConfig({
+      input: `src/${file}`,
+      isAddOn: true,
+      outputFileBase: `build/${file.replace('.ts', '')}`,
+    });
+
+    [
+      {
+        extension: '.js',
+        plugins: [...baseBundleConfig.plugins, commonjs()],
+      },
+      {
+        extension: '.min.js',
+        plugins: [...baseBundleConfig.plugins, commonjs(), terserPlugin],
+      },
+    ].forEach(build => {
+      builds.push({
         ...baseBundleConfig,
-        input: `src/${file}`,
         output: {
           ...baseBundleConfig.output,
-          ...addOnBundleConfig.output,
-          file: `build/${file.replace('.ts', build.extension)}`,
+          file: `${baseBundleConfig.output.file}${build.extension}`,
         },
         plugins: build.plugins,
-      })),
-    );
+      });
+    });
   });
+
   return builds;
 }