terser 5.38.0 → 5.38.1

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # 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)

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();
 
@@ -22137,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/index.js

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