move license plugin to main rollup config

Author: lobsterkatie

packages/browser/rollup.config.js

@@ -1,14 +1,13 @@
-import { makeBaseBundleConfig, makeLicensePlugin, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
 
 const builds = [];
 
-const licensePlugin = makeLicensePlugin();
-
 ['es5', 'es6'].forEach(jsVersion => {
   const baseBundleConfig = makeBaseBundleConfig({
     input: 'src/index.ts',
     isAddOn: false,
     jsVersion,
+    licenseTitle: '@sentry/browser',
     outputFileBase: `build/bundle${jsVersion === 'es6' ? '.es6' : ''}`,
   });
 
@@ -20,15 +19,15 @@ const licensePlugin = makeLicensePlugin();
           ...baseBundleConfig.output,
           file: `${baseBundleConfig.output.file}.js`,
         },
-        plugins: [...baseBundleConfig.plugins, licensePlugin],
+        plugins: [...baseBundleConfig.plugins],
       },
       {
         ...baseBundleConfig,
         output: {
           ...baseBundleConfig.output,
           file: `${baseBundleConfig.output.file}.min.js`,
         },
-        plugins: [...baseBundleConfig.plugins, terserPlugin, licensePlugin],
+        plugins: [...baseBundleConfig.plugins, terserPlugin],
       },
     ],
   );

packages/integrations/rollup.config.js

@@ -16,6 +16,7 @@ function loadAllIntegrations() {
       input: `src/${file}`,
       isAddOn: true,
       jsVersion: 'es5',
+      licenseTitle: '@sentry/integrations',
       outputFileBase: `build/${file.replace('.ts', '')}`,
     });
 

packages/tracing/rollup.config.js

@@ -1,14 +1,13 @@
-import { makeBaseBundleConfig, makeLicensePlugin, terserPlugin } from '../../rollup.config';
+import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
 
 const builds = [];
 
-const licensePlugin = makeLicensePlugin('@sentry/tracing & @sentry/browser');
-
 ['es5', 'es6'].forEach(jsVersion => {
   const baseBundleConfig = makeBaseBundleConfig({
     input: 'src/index.bundle.ts',
     isAddOn: false,
     jsVersion,
+    licenseTitle: '@sentry/tracing & @sentry/browser',
     outputFileBase: `build/bundle.tracing${jsVersion === 'es6' ? '.es6' : ''}`,
   });
 
@@ -20,15 +19,15 @@ const licensePlugin = makeLicensePlugin('@sentry/tracing & @sentry/browser');
           ...baseBundleConfig.output,
           file: `${baseBundleConfig.output.file}.js`,
         },
-        plugins: [...baseBundleConfig.plugins, licensePlugin],
+        plugins: baseBundleConfig.plugins,
       },
       {
         ...baseBundleConfig,
         output: {
           ...baseBundleConfig.output,
           file: `${baseBundleConfig.output.file}.min.js`,
         },
-        plugins: [...baseBundleConfig.plugins, terserPlugin, licensePlugin],
+        plugins: [...baseBundleConfig.plugins, terserPlugin],
       },
     ],
   );

packages/vue/rollup.config.js

@@ -1,11 +1,10 @@
-import { makeBaseBundleConfig, makeLicensePlugin, terserPlugin } from '../../rollup.config';
-
-const licensePlugin = makeLicensePlugin();
+import { makeBaseBundleConfig, terserPlugin } from '../../rollup.config';
 
 const baseBundleConfig = makeBaseBundleConfig({
   input: 'src/index.bundle.ts',
   isAddOn: false,
   jsVersion: 'es5',
+  licenseTitle: '@sentry/vue',
   outputFileBase: 'build/bundle.vue',
 });
 
@@ -16,14 +15,14 @@ export default [
       ...baseBundleConfig.output,
       file: `${baseBundleConfig.output.file}.js`,
     },
-    plugins: [...baseBundleConfig.plugins, licensePlugin],
+    plugins: baseBundleConfig.plugins,
   },
   {
     ...baseBundleConfig,
     output: {
       ...baseBundleConfig.output,
       file: `${baseBundleConfig.output.file}.min.js`,
     },
-    plugins: [...baseBundleConfig.plugins, terserPlugin, licensePlugin],
+    plugins: [...baseBundleConfig.plugins, terserPlugin],
   },
 ];

packages/wasm/rollup.config.js

@@ -4,6 +4,7 @@ const baseBundleConfig = makeBaseBundleConfig({
   input: 'src/index.ts',
   isAddOn: true,
   jsVersion: 'es5',
+  licenseTitle: '@sentry/wasm',
   outputFileBase: 'build/wasm',
 });
 

rollup.config.js

@@ -21,12 +21,12 @@ import typescript from 'rollup-plugin-typescript2';
  * @param title The title to use for the SDK, if not the package name
  * @returns An instance of the `rollup-plugin-license` plugin
  */
-export function makeLicensePlugin(title) {
+function makeLicensePlugin(title) {
   const commitHash = require('child_process').execSync('git rev-parse --short HEAD', { encoding: 'utf-8' }).trim();
 
   return license({
     banner: {
-      content: `/*! <%= data.title || pkg.name %> <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/sentry-javascript */`,
+      content: `/*! <%= data.title %> <%= pkg.version %> (${commitHash}) | https://github.com/getsentry/sentry-javascript */`,
       data: { title },
     },
   });
@@ -56,7 +56,7 @@ export const terserPlugin = terser({
 });
 
 export function makeBaseBundleConfig(options) {
-  const { input, isAddOn, jsVersion, outputFileBase } = options;
+  const { input, isAddOn, jsVersion, licenseTitle, outputFileBase } = options;
 
   const baseTSPluginOptions = {
     tsconfig: 'tsconfig.esm.json',
@@ -111,6 +111,8 @@ export function makeBaseBundleConfig(options) {
     },
   });
 
+  const licensePlugin = makeLicensePlugin(licenseTitle);
+
   const standAloneBundleConfig = {
     output: {
       format: 'iife',
@@ -159,7 +161,12 @@ export function makeBaseBundleConfig(options) {
       strict: false,
       esModule: false,
     },
-    plugins: [jsVersion === 'es5' ? typescriptPluginES5 : typescriptPluginES6, markAsBrowserBuild, nodeResolvePlugin],
+    plugins: [
+      jsVersion === 'es5' ? typescriptPluginES5 : typescriptPluginES6,
+      markAsBrowserBuild,
+      nodeResolvePlugin,
+      licensePlugin,
+    ],
     treeshake: 'smallest',
   };