use minification variant function in individual packages

Author: lobsterkatie

packages/browser/rollup.config.js

@@ -1,4 +1,4 @@
-import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, makeMinificationVariants } from '../../rollup.config';
 
 const builds = [];
 
@@ -11,26 +11,7 @@ const builds = [];
     outputFileBase: `build/bundle${jsVersion === 'es6' ? '.es6' : ''}`,
   });
 
-  builds.push(
-    ...[
-      {
-        ...baseBundleConfig,
-        output: {
-          ...baseBundleConfig.output,
-          file: `${baseBundleConfig.output.file}.js`,
-        },
-        plugins: [...baseBundleConfig.plugins],
-      },
-      {
-        ...baseBundleConfig,
-        output: {
-          ...baseBundleConfig.output,
-          file: `${baseBundleConfig.output.file}.min.js`,
-        },
-        plugins: [...baseBundleConfig.plugins, terserPlugin],
-      },
-    ],
-  );
+  builds.push(...makeMinificationVariants(baseBundleConfig));
 });
 
 export default builds;

packages/integrations/rollup.config.js

@@ -2,46 +2,25 @@ import * as fs from 'fs';
 
 import commonjs from '@rollup/plugin-commonjs';
 
-import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
-
-function allIntegrations() {
-  return fs.readdirSync('./src').filter(file => file != 'index.ts');
-}
-
-function loadAllIntegrations() {
-  const builds = [];
-
-  allIntegrations().forEach(file => {
-    const baseBundleConfig = makeBaseBundleConfig({
-      input: `src/${file}`,
-      isAddOn: true,
-      jsVersion: 'es5',
-      licenseTitle: '@sentry/integrations',
-      outputFileBase: `build/${file.replace('.ts', '')}`,
-    });
-
-    [
-      {
-        extension: '.js',
-        plugins: [...baseBundleConfig.plugins, commonjs()],
-      },
-      {
-        extension: '.min.js',
-        plugins: [...baseBundleConfig.plugins, commonjs(), terserPlugin],
-      },
-    ].forEach(build => {
-      builds.push({
-        ...baseBundleConfig,
-        output: {
-          ...baseBundleConfig.output,
-          file: `${baseBundleConfig.output.file}${build.extension}`,
-        },
-        plugins: build.plugins,
-      });
-    });
+import { insertAt, makeBaseBundleConfig, makeMinificationVariants } from '../../rollup.config';
+
+const builds = [];
+
+const integrationSourceFiles = fs.readdirSync('./src').filter(file => file != 'index.ts');
+
+integrationSourceFiles.forEach(file => {
+  const baseBundleConfig = makeBaseBundleConfig({
+    input: `src/${file}`,
+    isAddOn: true,
+    jsVersion: 'es5',
+    licenseTitle: '@sentry/integrations',
+    outputFileBase: `build/${file.replace('.ts', '')}`,
   });
 
-  return builds;
-}
+  // TODO We only need `commonjs` for localforage (used in the offline plugin). Once that's fixed, this can come out.
+  baseBundleConfig.plugins = insertAt(baseBundleConfig.plugins, -2, commonjs());
+
+  builds.push(...makeMinificationVariants(baseBundleConfig));
+});
 
-export default loadAllIntegrations();
+export default builds;

packages/tracing/rollup.config.js

@@ -1,4 +1,4 @@
-import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, makeMinificationVariants } from '../../rollup.config';
 
 const builds = [];
 
@@ -11,26 +11,7 @@ const builds = [];
     outputFileBase: `build/bundle.tracing${jsVersion === 'es6' ? '.es6' : ''}`,
   });
 
-  builds.push(
-    ...[
-      {
-        ...baseBundleConfig,
-        output: {
-          ...baseBundleConfig.output,
-          file: `${baseBundleConfig.output.file}.js`,
-        },
-        plugins: baseBundleConfig.plugins,
-      },
-      {
-        ...baseBundleConfig,
-        output: {
-          ...baseBundleConfig.output,
-          file: `${baseBundleConfig.output.file}.min.js`,
-        },
-        plugins: [...baseBundleConfig.plugins, terserPlugin],
-      },
-    ],
-  );
+  builds.push(...makeMinificationVariants(baseBundleConfig));
 });
 
 export default builds;

packages/vue/rollup.config.js

@@ -1,4 +1,4 @@
-import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, makeMinificationVariants } from '../../rollup.config';
 
 const baseBundleConfig = makeBaseBundleConfig({
   input: 'src/index.bundle.ts',
@@ -8,21 +8,4 @@ const baseBundleConfig = makeBaseBundleConfig({
   outputFileBase: 'build/bundle.vue',
 });
 
-export default [
-  {
-    ...baseBundleConfig,
-    output: {
-      ...baseBundleConfig.output,
-      file: `${baseBundleConfig.output.file}.js`,
-    },
-    plugins: baseBundleConfig.plugins,
-  },
-  {
-    ...baseBundleConfig,
-    output: {
-      ...baseBundleConfig.output,
-      file: `${baseBundleConfig.output.file}.min.js`,
-    },
-    plugins: [...baseBundleConfig.plugins, terserPlugin],
-  },
-];
+export default makeMinificationVariants(baseBundleConfig);

packages/wasm/rollup.config.js

@@ -1,4 +1,4 @@
-import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, makeMinificationVariants } from '../../rollup.config';
 
 const baseBundleConfig = makeBaseBundleConfig({
   input: 'src/index.ts',
@@ -8,28 +8,4 @@ const baseBundleConfig = makeBaseBundleConfig({
   outputFileBase: 'build/wasm',
 });
 
-function loadAllIntegrations() {
-  const builds = [];
-  [
-    {
-      extension: '.js',
-      plugins: baseBundleConfig.plugins,
-    },
-    {
-      extension: '.min.js',
-      plugins: [...baseBundleConfig.plugins, terserPlugin],
-    },
-  ].forEach(build => {
-    builds.push({
-      ...baseBundleConfig,
-      output: {
-        ...baseBundleConfig.output,
-        file: `${baseBundleConfig.output.file}${build.extension}`,
-      },
-      plugins: build.plugins,
-    });
-  });
-  return builds;
-}
-
-export default loadAllIntegrations();
+export default makeMinificationVariants(baseBundleConfig);