terser 5.31.2 → 5.31.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v5.31.4
+ - drop_unused: drop classes which only have side effects in the `extends` part
+
+## 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)

package/dist/bundle.min.js

@@ -2,7 +2,7 @@
 typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('@jridgewell/source-map')) :
 typeof define === 'function' && define.amd ? define(['exports', '@jridgewell/source-map'], factory) :
 (global = typeof globalThis !== 'undefined' ? globalThis : global || self, factory(global.Terser = {}, global.sourceMap));
-}(this, (function (exports, sourceMap) { 'use strict';
+})(this, (function (exports, sourceMap) { 'use strict';
 
 /***********************************************************************
 
@@ -5905,6 +5905,9 @@ var AST_Class = DEFNODE("Class", "name extends properties", function AST_Class(p
         }));
         return found;
     },
+    body_has_side_effects(compressor) {
+        return this.properties.some(prop => prop.has_side_effects(compressor));
+    },
 }, AST_Scope /* TODO a class might have a scope but it's not a scope */);
 
 var AST_ClassProperty = DEFNODE("ClassProperty", "static quote", function AST_ClassProperty(props) {
@@ -14979,7 +14982,7 @@ def_drop_side_effect_free(AST_Arrow, return_null);
 def_drop_side_effect_free(AST_Class, function (compressor) {
     const with_effects = [];
 
-    if (this.is_self_referential() && this.has_side_effects(compressor)) {
+    if (this.is_self_referential() && this.body_has_side_effects(compressor)) {
         return this;
     }
 
@@ -15287,7 +15290,7 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
             });
         }
         if (node === self) return;
-        if (node instanceof AST_Class && node.has_side_effects(compressor)) {
+        if (node instanceof AST_Class && node.body_has_side_effects(compressor)) {
             if (node.is_self_referential()) {
                 descend();
             } else {
@@ -15391,7 +15394,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) {
@@ -32541,4 +32550,4 @@ exports._run_cli = run_cli;
 exports.minify = minify;
 exports.minify_sync = minify_sync;
 
-})));
+}));

package/lib/ast.js

@@ -2289,6 +2289,9 @@ var AST_Class = DEFNODE("Class", "name extends properties", function AST_Class(p
         }));
         return found;
     },
+    body_has_side_effects(compressor) {
+        return this.properties.some(prop => prop.has_side_effects(compressor));
+    },
 }, AST_Scope /* TODO a class might have a scope but it's not a scope */);
 
 var AST_ClassProperty = DEFNODE("ClassProperty", "static quote", function AST_ClassProperty(props) {

package/lib/compress/drop-side-effect-free.js

@@ -156,7 +156,7 @@ def_drop_side_effect_free(AST_Arrow, return_null);
 def_drop_side_effect_free(AST_Class, function (compressor) {
     const with_effects = [];
 
-    if (this.is_self_referential() && this.has_side_effects(compressor)) {
+    if (this.is_self_referential() && this.body_has_side_effects(compressor)) {
         return this;
     }
 

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,
@@ -152,7 +153,7 @@ AST_Scope.DEFMETHOD("drop_unused", function(compressor) {
             });
         }
         if (node === self) return;
-        if (node instanceof AST_Class && node.has_side_effects(compressor)) {
+        if (node instanceof AST_Class && node.body_has_side_effects(compressor)) {
             if (node.is_self_referential()) {
                 descend();
             } else {
@@ -256,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/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.2",
+  "version": "5.31.4",
   "engines": {
     "node": ">=10"
   },