Swap JS minifier from uglify to terser

alphagov · Nov 29, 2022 · 2e750ce · 2e750ce
1 parent 6488b3c
commit 2e750ce
Show file tree

Hide file tree

Showing 3 changed files with 64 additions and 71 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -55,7 +55,7 @@
     "gulp-rename": "^2.0.0",
     "gulp-sass": "^5.1.0",
     "gulp-task-listing": "^1.1.1",
-    "gulp-uglify": "^3.0.2",
+    "gulp-terser": "^2.1.0",
     "js-yaml": "^4.1.0",
     "jsdoc": "^4.0.0",
     "jsdoc-tsimport-plugin": "^1.0.5",

diff --git a/tasks/gulp/compile-assets.mjs b/tasks/gulp/compile-assets.mjs
@@ -9,7 +9,7 @@ import plumber from 'gulp-plumber'
 import postcss from 'gulp-postcss'
 import rename from 'gulp-rename'
 import gulpSass from 'gulp-sass'
-import uglify from 'gulp-uglify'
+import terser from 'gulp-terser'
 import merge from 'merge-stream'
 import minimatch from 'minimatch'
 import nodeSass from 'node-sass'
@@ -180,9 +180,12 @@ function compileJavaScript (stream, moduleName) {
     }))
 
     // Minify
-    .pipe(gulpif(isMinified, uglify({
-      ie8: true
-    })))
+    .pipe(gulpif(isMinified, terser({
+      ecma: 5,
+      format: { comments: false },
+      ie8: true,
+      safari10: true
+    }))
 
     // Rename
     .pipe(rename((path, file) => {