terser 5.30.1 → 5.30.2

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v5.30.2
+ - Avoid optimizations inside computed keys, because they can cause js-engine-specific bugs.
+
 ## v5.30.1
  - Removed useless `\` escapes for non-ascii characters
  - Make modern identifier characters quoted for older environments (#1512)

package/dist/bundle.min.js

@@ -16560,6 +16560,7 @@ function tighten_body(statements, compressor) {
                 || node instanceof AST_SymbolRef
                     && parent instanceof AST_Call
                     && has_annotation(parent, _NOINLINE)
+                || node instanceof AST_ObjectProperty && node.key instanceof AST_Node
             ) {
                 abort = true;
                 return node;
@@ -17842,6 +17843,8 @@ function is_const_symbol_short_than_init_value(def, fixed_value) {
 }
 
 function inline_into_symbolref(self, compressor) {
+    if (compressor.in_computed_key()) return self;
+
     const parent = compressor.parent();
     const def = self.definition();
     const nearest_scope = compressor.find_scope();
@@ -17991,6 +17994,8 @@ function inline_into_symbolref(self, compressor) {
 }
 
 function inline_into_call(self, compressor) {
+    if (compressor.in_computed_key()) return self;
+
     var exp = self.expression;
     var fn = exp;
     var simple_args = self.args.every((arg) => !(arg instanceof AST_Expansion));
@@ -18646,6 +18651,17 @@ class Compressor extends TreeWalker {
         }
     }
 
+    in_computed_key() {
+        if (!this.option("evaluate")) return false;
+        var self = this.self();
+        for (var i = 0, p; p = this.parent(i); i++) {
+            if (p instanceof AST_ObjectProperty && p.key === self) {
+                return true;
+            }
+        }
+        return false;
+    }
+
     get_toplevel() {
         return this._toplevel;
     }

package/lib/compress/index.js

@@ -401,6 +401,17 @@ class Compressor extends TreeWalker {
         }
     }
 
+    in_computed_key() {
+        if (!this.option("evaluate")) return false;
+        var self = this.self();
+        for (var i = 0, p; p = this.parent(i); i++) {
+            if (p instanceof AST_ObjectProperty && p.key === self) {
+                return true;
+            }
+        }
+        return false;
+    }
+
     get_toplevel() {
         return this._toplevel;
     }

package/lib/compress/inline.js

@@ -167,6 +167,8 @@ function is_const_symbol_short_than_init_value(def, fixed_value) {
 }
 
 export function inline_into_symbolref(self, compressor) {
+    if (compressor.in_computed_key()) return self;
+
     const parent = compressor.parent();
     const def = self.definition();
     const nearest_scope = compressor.find_scope();
@@ -316,6 +318,8 @@ export function inline_into_symbolref(self, compressor) {
 }
 
 export function inline_into_call(self, compressor) {
+    if (compressor.in_computed_key()) return self;
+
     var exp = self.expression;
     var fn = exp;
     var simple_args = self.args.every((arg) => !(arg instanceof AST_Expansion));

package/lib/compress/tighten-body.js

@@ -82,6 +82,7 @@ import {
     AST_Number,
     AST_Object,
     AST_ObjectKeyVal,
+    AST_ObjectProperty,
     AST_PropAccess,
     AST_RegExp,
     AST_Return,
@@ -116,7 +117,7 @@ import {
     walk,
     walk_abort,
 
-    _NOINLINE
+    _NOINLINE,
 } from "../ast.js";
 import {
     make_node,
@@ -328,6 +329,7 @@ export function tighten_body(statements, compressor) {
                 || node instanceof AST_SymbolRef
                     && parent instanceof AST_Call
                     && has_annotation(parent, _NOINLINE)
+                || node instanceof AST_ObjectProperty && node.key instanceof AST_Node
             ) {
                 abort = true;
                 return node;

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