terser 5.31.1 → 5.31.3

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## v5.31.3
+ - drop_unused: drop unused parameters from IIFEs in some more situations.
+
+## v5.31.2
+ - drop_unused: scan variables in self-referential class declarations that contain side effects.
+ - Don't add parens to arrow function when it's the default for an argument (#1540)
+ - Update domprops (#1538)
+
 ## v5.31.1
  - Allow drop-unused to drop the whole assignment (not just the assigned name) in more situations, in order to avoid duplication of long strings.
 

package/dist/bundle.min.js

@@ -10276,7 +10276,9 @@ function OutputStream(options) {
     AST_Arrow.DEFMETHOD("_do_print", function(output) {
         var self = this;
         var parent = output.parent();
-        var needs_parens = (parent instanceof AST_Binary && !(parent instanceof AST_Assign)) ||
+        var needs_parens = (parent instanceof AST_Binary &&
+                !(parent instanceof AST_Assign) &&
+                !(parent instanceof AST_DefaultAssign)) ||
             parent instanceof AST_Unary ||
             (parent instanceof AST_Call && self === parent.expression);
         if (needs_parens) { output.print("("); }
@@ -15272,7 +15274,6 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
     }
     var var_defs_by_id = new Map();
     var initializations = new Map();
-    var self_referential_classes = new Set();
 
     // pass 1: find out which symbols are directly used in
     // this scope (not in nested scopes).
@@ -15287,8 +15288,11 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
         }
         if (node === self) return;
         if (node instanceof AST_Class && node.has_side_effects(compressor)) {
-            if (node.is_self_referential()) self_referential_classes.add(node);
-            node.visit_nondeferred_class_parts(tw);
+            if (node.is_self_referential()) {
+                descend();
+            } else {
+                node.visit_nondeferred_class_parts(tw);
+            }
         }
         if (node instanceof AST_Defun || node instanceof AST_DefClass) {
             var node_def = node.name.definition();
@@ -15352,9 +15356,6 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
             init.walk(tw);
         });
     });
-    self_referential_classes.forEach(function (cls) {
-        cls.walk(tw);
-    });
     // pass 3: we should drop declarations not in_use
     var tt = new TreeTransformer(
         function before(node, descend, in_list) {
@@ -15390,7 +15391,13 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
                 if (!in_use_ids.has(def.id) || def.orig.length > 1) node.name = null;
             }
             if (node instanceof AST_Lambda && !(node instanceof AST_Accessor)) {
-                var trim = !compressor.option("keep_fargs");
+                var trim =
+                    !compressor.option("keep_fargs")
+                    // Is this an IIFE that won't refer to its name?
+                    || parent instanceof AST_Call
+                        && parent.expression === node
+                        && !node.pinned()
+                        && (!node.name || node.name.unreferenced());
                 for (var a = node.argnames, i = a.length; --i >= 0;) {
                     var sym = a[i];
                     if (sym instanceof AST_Expansion) {
@@ -30779,6 +30786,7 @@ var domprops = [
     "uint32",
     "uint8",
     "uint8Clamped",
+    "unadjustedMovement",
     "unclippedDepth",
     "unconfigure",
     "undefined",

package/lib/compress/drop-unused.js

@@ -45,6 +45,7 @@ import {
     AST_Accessor,
     AST_Assign,
     AST_BlockStatement,
+    AST_Call,
     AST_Class,
     AST_ClassExpression,
     AST_ClassStaticBlock,
@@ -139,7 +140,6 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
     }
     var var_defs_by_id = new Map();
     var initializations = new Map();
-    var self_referential_classes = new Set();
 
     // pass 1: find out which symbols are directly used in
     // this scope (not in nested scopes).
@@ -154,8 +154,11 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
         }
         if (node === self) return;
         if (node instanceof AST_Class && node.has_side_effects(compressor)) {
-            if (node.is_self_referential()) self_referential_classes.add(node);
-            node.visit_nondeferred_class_parts(tw);
+            if (node.is_self_referential()) {
+                descend();
+            } else {
+                node.visit_nondeferred_class_parts(tw);
+            }
         }
         if (node instanceof AST_Defun || node instanceof AST_DefClass) {
             var node_def = node.name.definition();
@@ -219,9 +222,6 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
             init.walk(tw);
         });
     });
-    self_referential_classes.forEach(function (cls) {
-        cls.walk(tw);
-    });
     // pass 3: we should drop declarations not in_use
     var tt = new TreeTransformer(
         function before(node, descend, in_list) {
@@ -257,7 +257,13 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
                 if (!in_use_ids.has(def.id) || def.orig.length > 1) node.name = null;
             }
             if (node instanceof AST_Lambda && !(node instanceof AST_Accessor)) {
-                var trim = !compressor.option("keep_fargs");
+                var trim =
+                    !compressor.option("keep_fargs")
+                    // Is this an IIFE that won't refer to its name?
+                    || parent instanceof AST_Call
+                        && parent.expression === node
+                        && !node.pinned()
+                        && (!node.name || node.name.unreferenced());
                 for (var a = node.argnames, i = a.length; --i >= 0;) {
                     var sym = a[i];
                     if (sym instanceof AST_Expansion) {

package/lib/output.js

@@ -1501,7 +1501,9 @@ function OutputStream(options) {
     AST_Arrow.DEFMETHOD("_do_print", function(output) {
         var self = this;
         var parent = output.parent();
-        var needs_parens = (parent instanceof AST_Binary && !(parent instanceof AST_Assign)) ||
+        var needs_parens = (parent instanceof AST_Binary &&
+                !(parent instanceof AST_Assign) &&
+                !(parent instanceof AST_DefaultAssign)) ||
             parent instanceof AST_Unary ||
             (parent instanceof AST_Call && self === parent.expression);
         if (needs_parens) { output.print("("); }

package/package.json

@@ -4,7 +4,7 @@
   "homepage": "https://terser.org",
   "author": "Mihai Bazon <mihai.bazon@gmail.com> (http://lisperator.net/)",
   "license": "BSD-2-Clause",
-  "version": "5.31.1",
+  "version": "5.31.3",
   "engines": {
     "node": ">=10"
   },

package/tools/domprops.js

@@ -8225,6 +8225,7 @@ export var domprops = [
     "uint32",
     "uint8",
     "uint8Clamped",
+    "unadjustedMovement",
     "unclippedDepth",
     "unconfigure",
     "undefined",