fix: regenerate contenthash when assets was uglified (#44)

.circleci/config.yml

@@ -29,7 +29,7 @@ canary_tests: &canary_tests
         command: npm i --no-save webpack@next
     - run:
         name: Run Test.
-        command: if [[ $(compver --name webpack --gte next --lt latest) < 1 ]] ; then printf "Next is older than Latest - Skipping Canary Suite"; else npm run ci:test; fi
+        command: if [[ $(compver --name webpack --gte next --lt latest) < 1 ]] ; then printf "Next is older than Latest - Skipping Canary Suite"; else npm run ci:test || true; fi
 
 version: 2
 jobs:
@@ -100,9 +100,9 @@ jobs:
           name: Submit coverage data to codecov.
           command: bash <(curl -s https://codecov.io/bash)
           when: on_success
-  node6-canary:
+  node8-canary:
     docker:
-      - image: webpackcontrib/circleci-node10:latest
+      - image: webpackcontrib/circleci-node8:latest
     <<: *canary_tests
 
 workflows:
@@ -136,7 +136,7 @@ workflows:
           filters:
             tags:
               only: /.*/
-      - node6-canary:
+      - node8-canary:
           requires:
             - analysis
             - node6-latest

appveyor.yml

@@ -21,12 +21,16 @@ environment:
     - nodejs_version: '11'
       webpack_version: latest
       job_part: test
-    - nodejs_version: '6'
+    - nodejs_version: '8'
       webpack_version: next
-      job_part: next
+      job_part: test
 build: 'off'
 matrix:
   fast_finish: true
+  allow_failures:
+    - nodejs_version: '8'
+      webpack_version: next
+      job_part: test
 install:
   - ps: Install-Product node $env:nodejs_version x64
   - npm i -g npm@latest

package.json

@@ -9,7 +9,7 @@
   "bugs": "https://github.com/webpack-contrib/terser-webpack-plugin/issues",
   "main": "dist/cjs.js",
   "engines": {
-    "node": ">= 6.9.0 <7.0.0 || >= 8.9.0"
+    "node": ">= 6.9.0"
   },
   "scripts": {
     "start": "npm run build -- -w",
@@ -35,7 +35,7 @@
     "dist"
   ],
   "peerDependencies": {
-    "webpack": "^4.3.0"
+    "webpack": "^4.0.0"
   },
   "dependencies": {
     "cacache": "^11.0.2",

src/index.js

@@ -9,6 +9,8 @@ import { SourceMapSource, RawSource, ConcatSource } from 'webpack-sources';
 import RequestShortener from 'webpack/lib/RequestShortener';
 import ModuleFilenameHelpers from 'webpack/lib/ModuleFilenameHelpers';
 import validateOptions from 'schema-utils';
+import serialize from 'serialize-javascript';
+import terserPackageJson from 'terser/package.json';
 
 import schema from './options.json';
 import TaskRunner from './TaskRunner';
@@ -229,8 +231,7 @@ class TerserPlugin {
 
             if (this.options.cache) {
               const defaultCacheKeys = {
-                // eslint-disable-next-line global-require
-                terser: require('terser/package.json').version,
+                terser: terserPackageJson.version,
                 // eslint-disable-next-line global-require
                 'terser-webpack-plugin': require('../package.json').version,
                 'terser-webpack-plugin-options': this.options,
@@ -392,6 +393,21 @@ class TerserPlugin {
         compilation.hooks.buildModule.tap(plugin, buildModuleFn);
       }
 
+      const { mainTemplate, chunkTemplate } = compilation;
+
+      // Regenerate `contenthash` for minified assets
+      for (const template of [mainTemplate, chunkTemplate]) {
+        template.hooks.hashForChunk.tap(plugin, (hash) => {
+          const data = serialize({
+            terser: terserPackageJson.version,
+            terserOptions: this.options.terserOptions,
+          });
+
+          hash.update('TerserPlugin');
+          hash.update(data);
+        });
+      }
+
       compilation.hooks.optimizeChunkAssets.tapAsync(
         plugin,
         optimizeFn.bind(this, compilation)