terser 5.24.0 → 5.26.0

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## v5.26.0
+ - Do not take the `/*#__PURE__*/` annotation into account when the `side_effects` compress option is off.
+ - The `preserve_annotations` option now automatically opts annotation comments in, instead of requiring the `comments` option to be configured for this.
+ - Refuse to parse empty parenthesized expressions (`()`)
+
+## v5.25.0
+ - Regex properties added to reserved property mangler (#1471)
+ - `pure_new` option added to drop unused `new` expressions.
+
 ## v5.24.0
  - Improve formatting performance in V8 by keeping a small work string and a large output string
 

package/README.md

@@ -820,6 +820,9 @@ If you happen to need the source map as a raw object, set `sourceMap.asObject` t
   Specify `"strict"` to treat `foo.bar` as side-effect-free only when
   `foo` is certain to not throw, i.e. not `null` or `undefined`.
 
+- `pure_new` (default: `false`) -- Set to `true` to assume `new X()` never has
+  side effects.
+
 - `reduce_vars` (default: `true`) -- Improve optimization on variables assigned with and
   used as constant values.
 

package/dist/bundle.min.js

@@ -2475,9 +2475,7 @@ function parse($TEXT, options) {
                 var ex = async ? new AST_Call({
                     expression: async,
                     args: exprs
-                }) : exprs.length == 1 ? exprs[0] : new AST_Sequence({
-                    expressions: exprs
-                });
+                }) : to_expr_or_sequence(start, exprs);
                 if (ex.start) {
                     const outer_comments_before = start.comments_before.length;
                     outer_comments_before_counts.set(start, outer_comments_before);
@@ -3574,6 +3572,16 @@ function parse($TEXT, options) {
         return left;
     };
 
+    var to_expr_or_sequence = function(start, exprs) {
+        if (exprs.length === 1) {
+            return exprs[0];
+        } else if (exprs.length > 1) {
+            return new AST_Sequence({ start, expressions: exprs, end: peek() });
+        } else {
+            croak("Invalid parenthesized expression");
+        }
+    };
+
     var expression = function(commas, no_in) {
         var start = S.token;
         var exprs = [];
@@ -3583,11 +3591,7 @@ function parse($TEXT, options) {
             next();
             commas = true;
         }
-        return exprs.length == 1 ? exprs[0] : new AST_Sequence({
-            start       : start,
-            expressions : exprs,
-            end         : peek()
-        });
+        return to_expr_or_sequence(start, exprs);
     };
 
     function in_loop(cont) {
@@ -8931,7 +8935,7 @@ function left_is_object(node) {
 const CODE_LINE_BREAK = 10;
 const CODE_SPACE = 32;
 
-const r_annotation = /[@#]__(PURE|INLINE|NOINLINE)__/g;
+const r_annotation = /[@#]__(PURE|INLINE|NOINLINE)__/;
 
 function is_some_comments(comment) {
     // multiline comment
@@ -9083,6 +9087,13 @@ function OutputStream(options) {
         }
     }
 
+    if (options.preserve_annotations) {
+        let prev_comment_filter = comment_filter;
+        comment_filter = function (comment) {
+            return r_annotation.test(comment.value) || prev_comment_filter.apply(this, arguments);
+        };
+    }
+
     var indentation = 0;
     var current_col = 0;
     var current_line = 1;
@@ -14170,7 +14181,13 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
             return true;
         }
     }
-    return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
+    if ((this instanceof AST_New) && compressor.option("pure_new")) {
+        return true;
+    }
+    if (compressor.option("side_effects") && has_annotation(this, _PURE)) {
+        return true;
+    }
+    return !compressor.pure_funcs(this);
 });
 
 // If I call this, is it a pure function?
@@ -18390,6 +18407,7 @@ class Compressor extends TreeWalker {
             properties    : !false_by_default,
             pure_getters  : !false_by_default && "strict",
             pure_funcs    : null,
+            pure_new      : false,
             reduce_funcs  : !false_by_default,
             reduce_vars   : !false_by_default,
             sequences     : !false_by_default,
@@ -26841,6 +26859,7 @@ var domprops = [
     "gridTemplateRows",
     "gripSpace",
     "group",
+    "groups",
     "groupCollapsed",
     "groupEnd",
     "groupId",
@@ -26948,6 +26967,7 @@ var domprops = [
     "indexOf",
     "indexedDB",
     "indicate",
+    "indices",
     "inert",
     "inertiaDestinationX",
     "inertiaDestinationY",

package/lib/compress/index.js

@@ -251,6 +251,7 @@ class Compressor extends TreeWalker {
             properties    : !false_by_default,
             pure_getters  : !false_by_default && "strict",
             pure_funcs    : null,
+            pure_new      : false,
             reduce_funcs  : !false_by_default,
             reduce_vars   : !false_by_default,
             sequences     : !false_by_default,

package/lib/compress/inference.js

@@ -828,7 +828,13 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
             return true;
         }
     }
-    return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
+    if ((this instanceof AST_New) && compressor.option("pure_new")) {
+        return true;
+    }
+    if (compressor.option("side_effects") && has_annotation(this, _PURE)) {
+        return true;
+    }
+    return !compressor.pure_funcs(this);
 });
 
 // If I call this, is it a pure function?

package/lib/output.js

@@ -169,7 +169,7 @@ import {
 const CODE_LINE_BREAK = 10;
 const CODE_SPACE = 32;
 
-const r_annotation = /[@#]__(PURE|INLINE|NOINLINE)__/g;
+const r_annotation = /[@#]__(PURE|INLINE|NOINLINE)__/;
 
 function is_some_comments(comment) {
     // multiline comment
@@ -321,6 +321,13 @@ function OutputStream(options) {
         }
     }
 
+    if (options.preserve_annotations) {
+        let prev_comment_filter = comment_filter;
+        comment_filter = function (comment) {
+            return r_annotation.test(comment.value) || prev_comment_filter.apply(this, arguments);
+        };
+    }
+
     var indentation = 0;
     var current_col = 0;
     var current_line = 1;

package/lib/parse.js

@@ -2330,9 +2330,7 @@ function parse($TEXT, options) {
                 var ex = async ? new AST_Call({
                     expression: async,
                     args: exprs
-                }) : exprs.length == 1 ? exprs[0] : new AST_Sequence({
-                    expressions: exprs
-                });
+                }) : to_expr_or_sequence(start, exprs);
                 if (ex.start) {
                     const outer_comments_before = start.comments_before.length;
                     outer_comments_before_counts.set(start, outer_comments_before);
@@ -3429,6 +3427,16 @@ function parse($TEXT, options) {
         return left;
     };
 
+    var to_expr_or_sequence = function(start, exprs) {
+        if (exprs.length === 1) {
+            return exprs[0];
+        } else if (exprs.length > 1) {
+            return new AST_Sequence({ start, expressions: exprs, end: peek() });
+        } else {
+            croak("Invalid parenthesized expression");
+        }
+    };
+
     var expression = function(commas, no_in) {
         var start = S.token;
         var exprs = [];
@@ -3438,11 +3446,7 @@ function parse($TEXT, options) {
             next();
             commas = true;
         }
-        return exprs.length == 1 ? exprs[0] : new AST_Sequence({
-            start       : start,
-            expressions : exprs,
-            end         : peek()
-        });
+        return to_expr_or_sequence(start, exprs);
     };
 
     function in_loop(cont) {

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

package/tools/domprops.js

@@ -4771,6 +4771,7 @@ export var domprops = [
     "gridTemplateRows",
     "gripSpace",
     "group",
+    "groups",
     "groupCollapsed",
     "groupEnd",
     "groupId",
@@ -4878,6 +4879,7 @@ export var domprops = [
     "indexOf",
     "indexedDB",
     "indicate",
+    "indices",
     "inert",
     "inertiaDestinationX",
     "inertiaDestinationY",

package/tools/terser.d.ts

@@ -50,6 +50,7 @@ export interface CompressOptions {
     passes?: number;
     properties?: boolean;
     pure_funcs?: string[];
+    pure_new?: boolean;
     pure_getters?: boolean | 'strict';
     reduce_funcs?: boolean;
     reduce_vars?: boolean;