terser 5.5.1 → 5.7.0

package/lib/propmangle.js

@@ -52,10 +52,13 @@ import { base54 } from "./scope.js";
 import {
     AST_Binary,
     AST_Call,
+    AST_ClassPrivateProperty,
     AST_Conditional,
     AST_Dot,
+    AST_DotHash,
     AST_ObjectKeyVal,
     AST_ObjectProperty,
+    AST_PrivateMethod,
     AST_Sequence,
     AST_String,
     AST_Sub,
@@ -155,7 +158,10 @@ function mangle_properties(ast, options) {
     if (!options.builtins) find_builtins(reserved);
 
     var cname = -1;
+    var cprivate = -1;
+
     var cache;
+    var private_cache = new Map();
     if (options.cache) {
         cache = options.cache.props;
         cache.forEach(function(mangled_name) {
@@ -178,12 +184,20 @@ function mangle_properties(ast, options) {
 
     var names_to_mangle = new Set();
     var unmangleable = new Set();
+    var private_properties = new Set();
 
     var keep_quoted_strict = options.keep_quoted === "strict";
 
     // step 1: find candidates to mangle
     ast.walk(new TreeWalker(function(node) {
-        if (node instanceof AST_ObjectKeyVal) {
+        if (
+            node instanceof AST_ClassPrivateProperty
+            || node instanceof AST_PrivateMethod
+        ) {
+            private_properties.add(node.key.name);
+        } else if (node instanceof AST_DotHash) {
+            private_properties.add(node.property);
+        } else if (node instanceof AST_ObjectKeyVal) {
             if (typeof node.key == "string" &&
                 (!keep_quoted_strict || !node.quote)) {
                 add(node.key);
@@ -220,7 +234,14 @@ function mangle_properties(ast, options) {
 
     // step 2: transform the tree, renaming properties
     return ast.transform(new TreeTransformer(function(node) {
-        if (node instanceof AST_ObjectKeyVal) {
+        if (
+            node instanceof AST_ClassPrivateProperty
+            || node instanceof AST_PrivateMethod
+        ) {
+            node.key.name = mangle_private(node.key.name);
+        } else if (node instanceof AST_DotHash) {
+            node.property = mangle_private(node.property);
+        } else if (node instanceof AST_ObjectKeyVal) {
             if (typeof node.key == "string" &&
                 (!keep_quoted_strict || !node.quote)) {
                 node.key = mangle(node.key);
@@ -300,6 +321,16 @@ function mangle_properties(ast, options) {
         return mangled;
     }
 
+    function mangle_private(name) {
+        let mangled = private_cache.get(name);
+        if (!mangled) {
+            mangled = base54(++cprivate);
+            private_cache.set(name, mangled);
+        }
+
+        return mangled;
+    }
+
     function mangleStrings(node) {
         return node.transform(new TreeTransformer(function(node) {
             if (node instanceof AST_Sequence) {

package/lib/scope.js

@@ -65,6 +65,7 @@ import {
     AST_Defun,
     AST_Destructuring,
     AST_Dot,
+    AST_DotHash,
     AST_Export,
     AST_For,
     AST_ForIn,
@@ -919,7 +920,9 @@ AST_Toplevel.DEFMETHOD("compute_char_frequency", function(options) {
             if (this instanceof AST_Symbol && !this.unmangleable(options)) {
                 base54.consider(this.name, -1);
             } else if (options.properties) {
-                if (this instanceof AST_Dot) {
+                if (this instanceof AST_DotHash) {
+                    base54.consider("#" + this.property, -1);
+                } else if (this instanceof AST_Dot) {
                     base54.consider(this.property, -1);
                 } else if (this instanceof AST_Sub) {
                     skip_string(this.property);

package/lib/size.js

@@ -10,6 +10,7 @@ import {
     AST_Call,
     AST_Case,
     AST_Class,
+    AST_ClassPrivateProperty,
     AST_ClassProperty,
     AST_ConciseMethod,
     AST_Conditional,
@@ -22,6 +23,7 @@ import {
     AST_Directive,
     AST_Do,
     AST_Dot,
+    AST_DotHash,
     AST_EmptyStatement,
     AST_Expansion,
     AST_Export,
@@ -47,6 +49,9 @@ import {
     AST_ObjectKeyVal,
     AST_ObjectGetter,
     AST_ObjectSetter,
+    AST_PrivateGetter,
+    AST_PrivateMethod,
+    AST_PrivateSetter,
     AST_RegExp,
     AST_Return,
     AST_Sequence,
@@ -87,6 +92,12 @@ AST_Node.prototype.size = function (compressor, stack) {
     let size = 0;
     walk_parent(this, (node, info) => {
         size += node._size(info);
+
+        // Braceless arrow functions have fake "return" statements
+        if (node instanceof AST_Arrow && node.is_braceless()) {
+            size += node.body[0].value._size(info);
+            return true;
+        }
     }, stack || (compressor && compressor.stack));
 
     // just to save a bit of memory
@@ -131,7 +142,6 @@ AST_With.prototype._size = () => 6;
 
 AST_Expansion.prototype._size = () => 3;
 
-/*#__INLINE__*/
 const lambda_modifiers = func =>
     (func.is_generator ? 1 : 0) + (func.async ? 6 : 0);
 
@@ -160,7 +170,9 @@ AST_Arrow.prototype._size = function () {
         args_and_arrow += 2;
     }
 
-    return lambda_modifiers(this) + args_and_arrow + (Array.isArray(this.body) ? list_overhead(this.body) : this.body._size());
+    const body_overhead = this.is_braceless() ? 0 : list_overhead(this.body) + 2;
+
+    return lambda_modifiers(this) + args_and_arrow + body_overhead;
 };
 
 AST_Destructuring.prototype._size = () => 2;
@@ -302,6 +314,13 @@ AST_Dot.prototype._size = function () {
     return this.property.length + 1;
 };
 
+AST_DotHash.prototype._size = function () {
+    if (this.optional) {
+        return this.property.length + 3;
+    }
+    return this.property.length + 2;
+};
+
 AST_Sub.prototype._size = function () {
     return this.optional ? 4 : 2;
 };
@@ -369,6 +388,14 @@ AST_ConciseMethod.prototype._size = function () {
     return static_size(this.static) + key_size(this.key) + lambda_modifiers(this);
 };
 
+AST_PrivateMethod.prototype._size = function () {
+    return AST_ConciseMethod.prototype._size.call(this) + 1;
+};
+
+AST_PrivateGetter.prototype._size = AST_PrivateSetter.prototype._size = function () {
+    return AST_ConciseMethod.prototype._size.call(this) + 4;
+};
+
 AST_Class.prototype._size = function () {
     return (
         (this.name ? 8 : 7)
@@ -384,6 +411,10 @@ AST_ClassProperty.prototype._size = function () {
     );
 };
 
+AST_ClassPrivateProperty.prototype._size = function () {
+    return AST_ClassProperty.prototype._size.call(this) + 1;
+};
+
 AST_Symbol.prototype._size = function () {
     return !mangle_options || this.definition().unmangleable(mangle_options)
         ? this.name.length

package/lib/utils/index.js

@@ -196,7 +196,7 @@ function mergeSort(array, cmp) {
 function makePredicate(words) {
     if (!Array.isArray(words)) words = words.split(" ");
 
-    return new Set(words);
+    return new Set(words.sort());
 }
 
 function map_add(map, key, value) {
@@ -235,6 +235,7 @@ function keep_name(keep_setting, name) {
 }
 
 var lineTerminatorEscape = {
+    "\0": "0",
     "\n": "n",
     "\r": "r",
     "\u2028": "u2028",
@@ -242,7 +243,8 @@ var lineTerminatorEscape = {
 };
 function regexp_source_fix(source) {
     // V8 does not escape line terminators in regexp patterns in node 12
-    return source.replace(/[\n\r\u2028\u2029]/g, function (match, offset) {
+    // We'll also remove literal \0
+    return source.replace(/[\0\n\r\u2028\u2029]/g, function (match, offset) {
         var escaped = source[offset - 1] == "\\"
             && (source[offset - 2] != "\\"
             || /(?:^|[^\\])(?:\\{2})*$/.test(source.slice(0, offset - 1)));

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.5.1",
+  "version": "5.7.0",
   "engines": {
     "node": ">=10"
   },
@@ -24,7 +24,8 @@
       "./dist/bundle.min.js"
     ],
     "./package": "./package.json",
-    "./package.json": "./package.json"
+    "./package.json": "./package.json",
+    "./bin/terser": "./bin/terser"
   },
   "types": "tools/terser.d.ts",
   "bin": {
@@ -48,16 +49,16 @@
   },
   "devDependencies": {
     "@ls-lint/ls-lint": "^1.9.2",
-    "acorn": "^7.4.0",
-    "astring": "^1.4.1",
-    "eslint": "^7.0.0",
+    "acorn": "^8.0.5",
+    "astring": "^1.6.2",
+    "eslint": "^7.19.0",
     "eslump": "^2.0.0",
     "esm": "^3.2.25",
-    "mocha": "^8.0.0",
+    "mocha": "^8.2.1",
     "pre-commit": "^1.2.2",
-    "rimraf": "^3.0.0",
-    "rollup": "2.0.6",
-    "semver": "^7.1.3"
+    "rimraf": "^3.0.2",
+    "rollup": "2.38.4",
+    "semver": "^7.3.4"
   },
   "scripts": {
     "test": "node test/compress.js && mocha test/mocha",
@@ -124,7 +125,7 @@
       "no-unused-vars": [
         "error",
         {
-          "varsIgnorePattern": "^_$"
+          "varsIgnorePattern": "^_"
         }
       ],
       "no-tabs": "error",

package/tools/domprops.js

@@ -4120,6 +4120,7 @@ export var domprops = [
     "exponent",
     "exponentialRampToValueAtTime",
     "exportKey",
+    "exports",
     "extend",
     "extensions",
     "extentNode",

package/tools/terser.d.ts

@@ -96,6 +96,7 @@ export interface ManglePropertiesOptions {
 
 export interface FormatOptions {
     ascii_only?: boolean;
+    /** @deprecated Not implemented anymore */
     beautify?: boolean;
     braces?: boolean;
     comments?: boolean | 'all' | 'some' | RegExp | ( (node: any, comment: {