terser 5.23.0 → 5.25.0

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 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
+
 ## v5.23.0
  - When top_retain will keep a variable assignment around, inline the assignee when it's shorter than the name (#1434)
  - Remove empty class `static {}` blocks.

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

@@ -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';
 
 /***********************************************************************
 
@@ -8928,7 +8928,6 @@ function left_is_object(node) {
 
  ***********************************************************************/
 
-const EXPECT_DIRECTIVE = /^$|[;{][\s\n]*$/;
 const CODE_LINE_BREAK = 10;
 const CODE_SPACE = 32;
 
@@ -8942,6 +8941,7 @@ function is_some_comments(comment) {
     );
 }
 
+const ROPE_COMMIT_WHEN = 8 * 1000;
 class Rope {
     constructor() {
         this.committed = "";
@@ -8949,7 +8949,13 @@ class Rope {
     }
 
     append(str) {
-        this.current += str;
+        /** When `this.current` is too long, commit it. */
+        if (this.current.length > ROPE_COMMIT_WHEN) {
+            this.committed += this.current + str;
+            this.current = "";
+        } else {
+            this.current += str;
+        }
     }
 
     insertAt(char, index) {
@@ -8971,14 +8977,45 @@ class Rope {
         return this.current[index - committed.length];
     }
 
-    curLength() {
-        return this.current.length;
+    charCodeAt(index) {
+        const { committed } = this;
+        if (index < committed.length) return committed.charCodeAt(index);
+        return this.current.charCodeAt(index - committed.length);
     }
 
     length() {
         return this.committed.length + this.current.length;
     }
 
+    expectDirective() {
+        // /^$|[;{][\s\n]*$/
+
+        let ch, n = this.length();
+
+        if (n <= 0) return true;
+
+        // Skip N whitespace from the end
+        while (
+            (ch = this.charCodeAt(--n))
+            && (ch == CODE_SPACE || ch == CODE_LINE_BREAK)
+        );
+
+        // either ";", or "{", or the string ended
+        return !ch || ch === 59 || ch === 123;
+    }
+
+    hasNLB() {
+        let n = this.length() - 1;
+        while (n >= 0) {
+            const code = this.charCodeAt(n--);
+
+            if (code === CODE_LINE_BREAK) return true;
+            if (code !== CODE_SPACE) return false;
+        }
+        return true;
+    }
+
+
     toString() {
         return this.committed + this.current;
     }
@@ -9184,9 +9221,9 @@ function OutputStream(options) {
         if (current_col > options.max_line_len) {
             if (might_add_newline) {
                 OUTPUT.insertAt("\n", might_add_newline);
-                const curLength = OUTPUT.curLength();
+                const len_after_newline = OUTPUT.length() - might_add_newline - 1;
                 if (mappings) {
-                    var delta = curLength - current_col;
+                    var delta = len_after_newline - current_col;
                     mappings.forEach(function(mapping) {
                         mapping.line++;
                         mapping.col += delta;
@@ -9194,7 +9231,7 @@ function OutputStream(options) {
                 }
                 current_line++;
                 current_pos++;
-                current_col = curLength;
+                current_col = len_after_newline;
             }
         }
         if (might_add_newline) {
@@ -9393,23 +9430,6 @@ function OutputStream(options) {
         return OUTPUT.toString();
     }
 
-    function has_nlb() {
-        const output = OUTPUT.toString();
-        let n = output.length - 1;
-        while (n >= 0) {
-            const code = output.charCodeAt(n);
-            if (code === CODE_LINE_BREAK) {
-                return true;
-            }
-
-            if (code !== CODE_SPACE) {
-                return false;
-            }
-            n--;
-        }
-        return true;
-    }
-
     function filter_comment(comment) {
         if (!options.preserve_annotations) {
             comment = comment.replace(r_annotation, " ");
@@ -9490,7 +9510,7 @@ function OutputStream(options) {
 
         comments = comments.filter(comment_filter, node).filter(c => !printed_comments.has(c));
         if (comments.length == 0) return;
-        var last_nlb = has_nlb();
+        var last_nlb = OUTPUT.hasNLB();
         comments.forEach(function(c, i) {
             printed_comments.add(c);
             if (!last_nlb) {
@@ -9548,7 +9568,7 @@ function OutputStream(options) {
                 print("\n");
                 indent();
                 need_newline_indented = false;
-            } else if (c.nlb && (i > 0 || !has_nlb())) {
+            } else if (c.nlb && (i > 0 || !OUTPUT.hasNLB())) {
                 print("\n");
                 indent();
             } else if (i > 0 || !tail) {
@@ -9610,7 +9630,7 @@ function OutputStream(options) {
             var encoded = encode_string(str, quote);
             if (escape_directive === true && !encoded.includes("\\")) {
                 // Insert semicolons to break directive prologue
-                if (!EXPECT_DIRECTIVE.test(OUTPUT.toString())) {
+                if (!OUTPUT.expectDirective()) {
                     force_semicolon();
                 }
                 force_semicolon();
@@ -14150,6 +14170,9 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
             return true;
         }
     }
+    if ((this instanceof AST_New) && compressor.option("pure_new")) {
+        return true;
+    }
     return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
 });
 
@@ -18370,6 +18393,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,
@@ -26821,6 +26845,7 @@ var domprops = [
     "gridTemplateRows",
     "gripSpace",
     "group",
+    "groups",
     "groupCollapsed",
     "groupEnd",
     "groupId",
@@ -26928,6 +26953,7 @@ var domprops = [
     "indexOf",
     "indexedDB",
     "indicate",
+    "indices",
     "inert",
     "inertiaDestinationX",
     "inertiaDestinationY",
@@ -31295,4 +31321,4 @@ exports._default_options = _default_options;
 exports._run_cli = run_cli;
 exports.minify = minify;
 
-}));
+})));

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,6 +828,9 @@ AST_Call.DEFMETHOD("is_callee_pure", function(compressor) {
             return true;
         }
     }
+    if ((this instanceof AST_New) && compressor.option("pure_new")) {
+        return true;
+    }
     return !!has_annotation(this, _PURE) || !compressor.pure_funcs(this);
 });
 

package/lib/output.js

@@ -166,7 +166,6 @@ import {
     ALL_RESERVED_WORDS,
 } from "./parse.js";
 
-const EXPECT_DIRECTIVE = /^$|[;{][\s\n]*$/;
 const CODE_LINE_BREAK = 10;
 const CODE_SPACE = 32;
 
@@ -180,6 +179,7 @@ function is_some_comments(comment) {
     );
 }
 
+const ROPE_COMMIT_WHEN = 8 * 1000;
 class Rope {
     constructor() {
         this.committed = "";
@@ -187,7 +187,13 @@ class Rope {
     }
 
     append(str) {
-        this.current += str;
+        /** When `this.current` is too long, commit it. */
+        if (this.current.length > ROPE_COMMIT_WHEN) {
+            this.committed += this.current + str;
+            this.current = "";
+        } else {
+            this.current += str;
+        }
     }
 
     insertAt(char, index) {
@@ -209,14 +215,45 @@ class Rope {
         return this.current[index - committed.length];
     }
 
-    curLength() {
-        return this.current.length;
+    charCodeAt(index) {
+        const { committed } = this;
+        if (index < committed.length) return committed.charCodeAt(index);
+        return this.current.charCodeAt(index - committed.length);
     }
 
     length() {
         return this.committed.length + this.current.length;
     }
 
+    expectDirective() {
+        // /^$|[;{][\s\n]*$/
+
+        let ch, n = this.length();
+
+        if (n <= 0) return true;
+
+        // Skip N whitespace from the end
+        while (
+            (ch = this.charCodeAt(--n))
+            && (ch == CODE_SPACE || ch == CODE_LINE_BREAK)
+        );
+
+        // either ";", or "{", or the string ended
+        return !ch || ch === 59 || ch === 123;
+    }
+
+    hasNLB() {
+        let n = this.length() - 1;
+        while (n >= 0) {
+            const code = this.charCodeAt(n--);
+
+            if (code === CODE_LINE_BREAK) return true;
+            if (code !== CODE_SPACE) return false;
+        }
+        return true;
+    }
+
+
     toString() {
         return this.committed + this.current;
     }
@@ -422,9 +459,9 @@ function OutputStream(options) {
         if (current_col > options.max_line_len) {
             if (might_add_newline) {
                 OUTPUT.insertAt("\n", might_add_newline);
-                const curLength = OUTPUT.curLength();
+                const len_after_newline = OUTPUT.length() - might_add_newline - 1;
                 if (mappings) {
-                    var delta = curLength - current_col;
+                    var delta = len_after_newline - current_col;
                     mappings.forEach(function(mapping) {
                         mapping.line++;
                         mapping.col += delta;
@@ -432,7 +469,7 @@ function OutputStream(options) {
                 }
                 current_line++;
                 current_pos++;
-                current_col = curLength;
+                current_col = len_after_newline;
             }
         }
         if (might_add_newline) {
@@ -631,23 +668,6 @@ function OutputStream(options) {
         return OUTPUT.toString();
     }
 
-    function has_nlb() {
-        const output = OUTPUT.toString();
-        let n = output.length - 1;
-        while (n >= 0) {
-            const code = output.charCodeAt(n);
-            if (code === CODE_LINE_BREAK) {
-                return true;
-            }
-
-            if (code !== CODE_SPACE) {
-                return false;
-            }
-            n--;
-        }
-        return true;
-    }
-
     function filter_comment(comment) {
         if (!options.preserve_annotations) {
             comment = comment.replace(r_annotation, " ");
@@ -728,7 +748,7 @@ function OutputStream(options) {
 
         comments = comments.filter(comment_filter, node).filter(c => !printed_comments.has(c));
         if (comments.length == 0) return;
-        var last_nlb = has_nlb();
+        var last_nlb = OUTPUT.hasNLB();
         comments.forEach(function(c, i) {
             printed_comments.add(c);
             if (!last_nlb) {
@@ -786,7 +806,7 @@ function OutputStream(options) {
                 print("\n");
                 indent();
                 need_newline_indented = false;
-            } else if (c.nlb && (i > 0 || !has_nlb())) {
+            } else if (c.nlb && (i > 0 || !OUTPUT.hasNLB())) {
                 print("\n");
                 indent();
             } else if (i > 0 || !tail) {
@@ -848,7 +868,7 @@ function OutputStream(options) {
             var encoded = encode_string(str, quote);
             if (escape_directive === true && !encoded.includes("\\")) {
                 // Insert semicolons to break directive prologue
-                if (!EXPECT_DIRECTIVE.test(OUTPUT.toString())) {
+                if (!OUTPUT.expectDirective()) {
                     force_semicolon();
                 }
                 force_semicolon();

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.23.0",
+  "version": "5.25.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;