Merge pull request #870 from ssi02014/chore/build

chore: modify package.json and rollup.config

Author: cure53

package.json

@@ -7,11 +7,11 @@
     "commit-amend-build": "scripts/commit-amend-build.sh",
     "prebuild": "rimraf dist/**",
     "dev": "cross-env NODE_ENV=development BABEL_ENV=rollup rollup -w -c -o dist/purify.js",
-    "build": "cross-env run-p build:umd build:umd:min build:es build:cjs",
-    "build:umd": "cross-env NODE_ENV=development BABEL_ENV=rollup rollup -c -f umd -o dist/purify.js",
-    "build:umd:min": "cross-env NODE_ENV=production BABEL_ENV=rollup rollup -c -f umd -o dist/purify.min.js",
-    "build:es": "cross-env NODE_ENV=development BABEL_ENV=rollup rollup -c -f es -o dist/purify.es.js",
-    "build:cjs": "cross-env NODE_ENV=development BABEL_ENV=rollup rollup -c -f cjs -o dist/purify.cjs.js",
+    "build": "rollup -c",
+    "build:umd": "rollup -c -f umd -o dist/purify.js",
+    "build:umd:min": "rollup -c -f umd -o dist/purify.min.js -p terser",
+    "build:es": "rollup -c -f es -o dist/purify.es.js",
+    "build:cjs": "rollup -c -f cjs -o dist/purify.cjs.js",
     "test:jsdom": "cross-env NODE_ENV=test BABEL_ENV=rollup node test/jsdom-node-runner --dot",
     "test:karma": "cross-env NODE_ENV=test BABEL_ENV=rollup karma start test/karma.conf.js --log-level warn ",
     "test:ci": "cross-env NODE_ENV=test BABEL_ENV=rollup npm run test:jsdom && npm run test:karma -- --log-level error --reporters dots --single-run --shouldTestOnBrowserStack=\"${TEST_BROWSERSTACK}\" --shouldProbeOnly=\"${TEST_PROBE_ONLY}\"",
@@ -20,6 +20,7 @@
   "main": "dist/purify.cjs.js",
   "module": "dist/purify.es.js",
   "browser": "dist/purify.js",
+  "production": "dist/purify.min.js",
   "files": [
     "dist"
   ],

rollup.config.js

@@ -3,22 +3,48 @@ const babel = require('@rollup/plugin-babel').babel;
 const nodeResolve = require('@rollup/plugin-node-resolve').nodeResolve;
 const replace = require('@rollup/plugin-replace');
 const { terser } = require('rollup-plugin-terser');
+const pkg = require('./package.json');
 
 const env = process.env.NODE_ENV;
-const isProd = env === 'production';
+const event = process.env.npm_lifecycle_event; // build, build:umd, ...
 const version = process.env.npm_package_version;
-const license = fs.readFileSync('./src/license_header', 'utf8').replace(/VERSION/ig, version);
+const license = fs
+  .readFileSync('./src/license_header', 'utf8')
+  .replace(/VERSION/gi, version);
+
+const commonOutputConfig = {
+  name: 'DOMPurify',
+  sourcemap: true,
+  banner: license,
+  exports: 'default',
+};
 
 const config = {
   input: 'src/purify.js',
   external: [],
-  output: {
-    name: 'DOMPurify',
-    globals: {},
-    format: 'umd',
-    sourcemap: true,
-    banner: license,
-  },
+  output: [
+    {
+      ...commonOutputConfig,
+      file: pkg.browser,
+      format: 'umd',
+    },
+    {
+      ...commonOutputConfig,
+      file: pkg.production,
+      format: 'umd',
+      plugins: event === 'build' ? [terser()] : [],
+    },
+    {
+      ...commonOutputConfig,
+      file: pkg.module,
+      format: 'es',
+    },
+    {
+      ...commonOutputConfig,
+      file: pkg.main,
+      format: 'cjs',
+    },
+  ],
   plugins: [
     babel({
       // It is recommended to configure this option explicitly (even if with its default value) so an informed decision is taken on how those babel helpers are inserted into the code.
@@ -32,13 +58,9 @@ const config = {
       values: {
         'process.env.NODE_ENV': JSON.stringify(env),
         VERSION: `'${version}'`,
-      }
+      },
     }),
   ],
 };
 
-if (isProd) {
-  config.plugins.push(terser());
-}
-
 module.exports = config;