terser 5.37.0 → 5.38.1

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## v5.38.1
+
+ - Fix inlining non-call expressions into an `optional_call?.()`
+
+## v5.38.0
+
+ - Remove `console` method-of-method calls (eg `console.log.apply()`) when `drop_console` option is used (#1585)
+ - Remove more object spreads, such as `{ ...void !0 }` (#1142)
+
 ## v5.37.0
 
  - Reserved object properties from chrome extensions (domprops)

package/README.md

@@ -647,7 +647,7 @@ console.log(result.map);  // source map
 Note that the source map is not saved in a file, it's just returned in
 `result.map`.  The value passed for `sourceMap.url` is only used to set
 `//# sourceMappingURL=out.js.map` in `result.code`. The value of
-`filename` is only used to set `file` attribute (see [the spec][sm-spec])
+`filename` is only used to set `file` attribute (see [the spec](https://docs.google.com/document/d/1U1RGAehQwRypUTovF1KRlpiOFze0b-_2gc6fAH0KY0k))
 in source map file.
 
 You can set option `sourceMap.url` to be `"inline"` and source map will
@@ -1233,7 +1233,7 @@ Terser has its own abstract syntax tree format; for
 we can't easily change to using the SpiderMonkey AST internally.  However,
 Terser now has a converter which can import a SpiderMonkey AST.
 
-For example [Acorn][acorn] is a super-fast parser that produces a
+For example [Acorn](https://github.com/acornjs/acorn) is a super-fast parser that produces a
 SpiderMonkey AST.  It has a small CLI utility that parses one file and dumps
 the AST in JSON on the standard output.  To use Terser to mangle and
 compress that:
@@ -1257,9 +1257,6 @@ Acorn is really fast (e.g. 250ms instead of 380ms on some 650K code), but
 converting the SpiderMonkey tree that Acorn produces takes another 150ms so
 in total it's a bit more than just using Terser's own parser.
 
-[acorn]: https://github.com/ternjs/acorn
-[sm-spec]: https://docs.google.com/document/d/1U1RGAehQwRypUTovF1KRlpiOFze0b-_2gc6fAH0KY0k
-
 ### Terser Fast Minify Mode
 
 It's not well known, but whitespace removal and symbol mangling accounts

package/dist/bundle.min.js

@@ -3286,6 +3286,7 @@ function parse($TEXT, options) {
             return subscripts(call, true, is_chain);
         }
 
+        // Optional chain
         if (is("punc", "?.")) {
             next();
 
@@ -14575,8 +14576,13 @@ AST_Node.DEFMETHOD("is_constant", function () {
         return !(this instanceof AST_RegExp);
     } else {
         return this instanceof AST_UnaryPrefix
-            && this.expression instanceof AST_Constant
-            && unaryPrefix.has(this.operator);
+            && unaryPrefix.has(this.operator)
+            && (
+                // `this.expression` may be an `AST_RegExp`,
+                // so not only `.is_constant()`.
+                this.expression instanceof AST_Constant
+                || this.expression.is_constant()
+            );
     }
 });
 
@@ -18982,17 +18988,19 @@ AST_Toplevel.DEFMETHOD("drop_console", function(options) {
             return;
         }
 
-        if (isArray && !options.includes(exp.property)) {
-            return;
-        }
-
         var name = exp.expression;
+        var property = exp.property;
         var depth = 2;
         while (name.expression) {
+            property = name.property;
             name = name.expression;
             depth++;
         }
 
+        if (isArray && !options.includes(property)) {
+            return;
+        }
+
         if (is_undeclared_ref(name) && name.name == "console") {
             if (
                 depth === 3
@@ -22130,7 +22138,15 @@ def_optimize(AST_Chain, function (self, compressor) {
         }
         return make_node(AST_Undefined, self);
     }
-    return self;
+    if (
+        self.expression instanceof AST_PropAccess
+        || self.expression instanceof AST_Call
+    ) {
+        return self;
+    } else {
+        // Keep the AST valid, in case the child swapped itself
+        return self.expression;
+    }
 });
 
 def_optimize(AST_Dot, function(self, compressor) {

package/lib/compress/evaluate.js

@@ -117,8 +117,13 @@ AST_Node.DEFMETHOD("is_constant", function () {
         return !(this instanceof AST_RegExp);
     } else {
         return this instanceof AST_UnaryPrefix
-            && this.expression instanceof AST_Constant
-            && unaryPrefix.has(this.operator);
+            && unaryPrefix.has(this.operator)
+            && (
+                // `this.expression` may be an `AST_RegExp`,
+                // so not only `.is_constant()`.
+                this.expression instanceof AST_Constant
+                || this.expression.is_constant()
+            );
     }
 });
 

package/lib/compress/index.js

@@ -524,17 +524,19 @@ AST_Toplevel.DEFMETHOD("drop_console", function(options) {
             return;
         }
 
-        if (isArray && !options.includes(exp.property)) {
-            return;
-        }
-
         var name = exp.expression;
+        var property = exp.property;
         var depth = 2;
         while (name.expression) {
+            property = name.property;
             name = name.expression;
             depth++;
         }
 
+        if (isArray && !options.includes(property)) {
+            return;
+        }
+
         if (is_undeclared_ref(name) && name.name == "console") {
             if (
                 depth === 3
@@ -3672,7 +3674,15 @@ def_optimize(AST_Chain, function (self, compressor) {
         }
         return make_node(AST_Undefined, self);
     }
-    return self;
+    if (
+        self.expression instanceof AST_PropAccess
+        || self.expression instanceof AST_Call
+    ) {
+        return self;
+    } else {
+        // Keep the AST valid, in case the child swapped itself
+        return self.expression;
+    }
 });
 
 def_optimize(AST_Dot, function(self, compressor) {

package/lib/parse.js

@@ -3141,6 +3141,7 @@ function parse($TEXT, options) {
             return subscripts(call, true, is_chain);
         }
 
+        // Optional chain
         if (is("punc", "?.")) {
             next();
 

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