terser 5.25.0 → 5.26.0

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # 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.

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';
 
 /***********************************************************************
 
@@ -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;
@@ -14173,7 +14184,10 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
     if ((this instanceof AST_New) && compressor.option("pure_new")) {
         return true;
     }
-    return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
+    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?
@@ -31321,4 +31335,4 @@ exports._default_options = _default_options;
 exports._run_cli = run_cli;
 exports.minify = minify;
 
-})));
+}));

package/lib/compress/inference.js

@@ -831,7 +831,10 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
     if ((this instanceof AST_New) && compressor.option("pure_new")) {
         return true;
     }
-    return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
+    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.25.0",
+  "version": "5.26.0",
   "engines": {
     "node": ">=10"
   },