terser 5.22.0 → 5.23.0

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v5.23.0
+ - When top_retain will keep a variable assignment around, inline the assignee when it's shorter than the name (#1434)
+ - Remove empty class `static {}` blocks.
+
 ## v5.22.0
  - Do not `unsafe`ly shorten expressions like a?.toString() when they're conditional.
  - Avoid running drop_unused in nodes that aren't scopes. Fixes a rare crash.

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';
 
 /***********************************************************************
 
@@ -17696,18 +17696,43 @@ function scope_encloses_variables_in_this_scope(scope, pulled_scope) {
     return false;
 }
 
+/**
+ * An extra check function for `top_retain` option, compare the length of const identifier
+ * and init value length and return true if init value is longer than identifier. for example:
+ * ```
+ * // top_retain: ["example"]
+ * const example = 100
+ * ```
+ * it will return false because length of "100" is short than identifier "example".
+ */
+function is_const_symbol_short_than_init_value(def, fixed_value) {
+    if (def.orig.length === 1 && fixed_value) {
+        const init_value_length = fixed_value.size();
+        const identifer_length = def.name.length;
+        return init_value_length > identifer_length;
+    }
+    return true;
+}
+
 function inline_into_symbolref(self, compressor) {
     const parent = compressor.parent();
-
     const def = self.definition();
     const nearest_scope = compressor.find_scope();
-    if (compressor.top_retain && def.global && compressor.top_retain(def)) {
+    let fixed = self.fixed_value();
+    if (
+        compressor.top_retain &&
+        def.global && 
+        compressor.top_retain(def) && 
+        // when identifier is in top_retain option dose not mean we can always inline it.
+        // if identifier name is longer then init value, we can replace it.
+        is_const_symbol_short_than_init_value(def, fixed)
+    ) {
+        // keep it
         def.fixed = false;
         def.single_use = false;
         return self;
     }
 
-    let fixed = self.fixed_value();
     let single_use = def.single_use
         && !(parent instanceof AST_Call
             && (parent.is_callee_pure(compressor))
@@ -21643,8 +21668,14 @@ def_optimize(AST_Function, function(self, compressor) {
 });
 
 def_optimize(AST_Class, function(self) {
-    // HACK to avoid compress failure.
-    // AST_Class is not really an AST_Scope/AST_Block as it lacks a body.
+    for (let i = 0; i < self.properties.length; i++) {
+        const prop = self.properties[i];
+        if (prop instanceof AST_ClassStaticBlock && prop.body.length == 0) {
+            self.properties.splice(i, 1);
+            i--;
+        }
+    }
+
     return self;
 });
 
@@ -31264,4 +31295,4 @@ exports._default_options = _default_options;
 exports._run_cli = run_cli;
 exports.minify = minify;
 
-})));
+}));

package/lib/compress/index.js

@@ -3549,8 +3549,14 @@ def_optimize(AST_Function, function(self, compressor) {
 });
 
 def_optimize(AST_Class, function(self) {
-    // HACK to avoid compress failure.
-    // AST_Class is not really an AST_Scope/AST_Block as it lacks a body.
+    for (let i = 0; i < self.properties.length; i++) {
+        const prop = self.properties[i];
+        if (prop instanceof AST_ClassStaticBlock && prop.body.length == 0) {
+            self.properties.splice(i, 1);
+            i--;
+        }
+    }
+
     return self;
 });
 

package/lib/compress/inline.js

@@ -84,7 +84,7 @@ import {
 
     _INLINE,
     _NOINLINE,
-    _PURE
+    _PURE,
 } from "../ast.js";
 import { make_node, has_annotation } from "../utils/index.js";
 import "../size.js";
@@ -148,18 +148,43 @@ function scope_encloses_variables_in_this_scope(scope, pulled_scope) {
     return false;
 }
 
+/**
+ * An extra check function for `top_retain` option, compare the length of const identifier
+ * and init value length and return true if init value is longer than identifier. for example:
+ * ```
+ * // top_retain: ["example"]
+ * const example = 100
+ * ```
+ * it will return false because length of "100" is short than identifier "example".
+ */
+function is_const_symbol_short_than_init_value(def, fixed_value) {
+    if (def.orig.length === 1 && fixed_value) {
+        const init_value_length = fixed_value.size();
+        const identifer_length = def.name.length;
+        return init_value_length > identifer_length;
+    }
+    return true;
+}
+
 export function inline_into_symbolref(self, compressor) {
     const parent = compressor.parent();
-
     const def = self.definition();
     const nearest_scope = compressor.find_scope();
-    if (compressor.top_retain && def.global && compressor.top_retain(def)) {
+    let fixed = self.fixed_value();
+    if (
+        compressor.top_retain &&
+        def.global && 
+        compressor.top_retain(def) && 
+        // when identifier is in top_retain option dose not mean we can always inline it.
+        // if identifier name is longer then init value, we can replace it.
+        is_const_symbol_short_than_init_value(def, fixed)
+    ) {
+        // keep it
         def.fixed = false;
         def.single_use = false;
         return self;
     }
 
-    let fixed = self.fixed_value();
     let single_use = def.single_use
         && !(parent instanceof AST_Call
             && (parent.is_callee_pure(compressor))

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.22.0",
+  "version": "5.23.0",
   "engines": {
     "node": ">=10"
   },