terser 5.19.1 → 5.19.3

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## v5.19.3
+ - Fix side effect detection of `optional?.chains`.
+ - Add roundRect to domprops.js (#1426)
+
+## v5.19.2
+ - fix performance hit from avoiding HTML comments in the output
+
 ## v5.19.1
  - Better avoid outputting `</script>` and HTML comments.
  - Fix unused variables in class static blocks not being dropped correctly.

package/LICENSE

@@ -1,5 +1,3 @@
-Terser is released under the BSD license:
-
 Copyright 2012-2018 (c) Mihai Bazon <mihai.bazon@gmail.com>
 
 Redistribution and use in source and binary forms, with or without

package/bin/terser.mjs

@@ -0,0 +1,18 @@
+#!/usr/bin/env node
+
+"use strict";
+
+import "../tools/exit.cjs";
+
+import fs from "fs";
+import path from "path";
+import program from "commander";
+
+import { _run_cli as run_cli } from "../main.js";
+
+const packageJson= JSON.parse( fs.readFileSync( new URL("./package.json", import.meta.url)));
+
+run_cli({ program, packageJson, fs, path }).catch((error) => {
+    console.error(error);
+    process.exitCode = 1;
+});

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';
 
 /***********************************************************************
 
@@ -8954,18 +8954,6 @@ class Rope {
         this.current += str;
     }
 
-    endsWith(str) {
-        const { committed, current } = this;
-        const len = str.length;
-        if (committed.length >= len) {
-            return committed.slice(committed.length - str.length) === str;
-        } else {
-            // `str` is small and this is a rare case so keep it simple
-            const last_bit = committed.slice(-len) + current;
-            return last_bit.endsWith(str);
-        }
-    }
-
     insertAt(char, index) {
         const { committed, current } = this;
         if (index < committed.length) {
@@ -9347,10 +9335,6 @@ function OutputStream(options) {
         might_need_semicolon = true;
     };
 
-    function ends_with(str) {
-        return OUTPUT.endsWith(str);
-    }
-
     function force_semicolon() {
         might_need_semicolon = false;
         print(";");
@@ -9620,7 +9604,6 @@ function OutputStream(options) {
         comma           : comma,
         colon           : colon,
         last            : function() { return last; },
-        ends_with       : ends_with,
         semicolon       : semicolon,
         force_semicolon : force_semicolon,
         to_utf8         : to_utf8,
@@ -10744,7 +10727,8 @@ function OutputStream(options) {
     });
     DEFPRINT(AST_UnaryPrefix, function(self, output) {
         var op = self.operator;
-        if (op === "--" && output.ends_with("<!")) {
+        if (op === "--" && output.last().endsWith("!")) {
+            // avoid printing "<!--"
             output.print(" ");
         }
         output.print(op);
@@ -10764,7 +10748,7 @@ function OutputStream(options) {
         var op = self.operator;
         self.left.print(output);
         if (op[0] == ">" /* ">>" ">>>" ">" ">=" */
-            && output.ends_with("--")) {
+            && output.last().endsWith("--")) {
             // space is mandatory to avoid outputting -->
             output.print(" ");
         } else {
@@ -11083,7 +11067,7 @@ function OutputStream(options) {
 
         // Avoid outputting end of script tag
         source = source.replace(r_slash_script, slash_script_replace);
-        if (r_starts_with_script.test(source) && output.ends_with("<")) {
+        if (r_starts_with_script.test(source) && output.last().endsWith("<")) {
             output.print(" ");
         }
 
@@ -13706,16 +13690,27 @@ function is_nullish(node, compressor) {
         return any(this.elements, compressor);
     });
     def_has_side_effects(AST_Dot, function(compressor) {
-        if (is_nullish(this, compressor)) return false;
-        return !this.optional && this.expression.may_throw_on_access(compressor)
-            || this.expression.has_side_effects(compressor);
+        if (is_nullish(this, compressor)) {
+            return this.expression.has_side_effects(compressor);
+        }
+        if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+            return true;
+        }
+
+        return this.expression.has_side_effects(compressor);
     });
     def_has_side_effects(AST_Sub, function(compressor) {
-        if (is_nullish(this, compressor)) return false;
+        if (is_nullish(this, compressor)) {
+            return this.expression.has_side_effects(compressor);
+        }
+        if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+            return true;
+        }
 
-        return !this.optional && this.expression.may_throw_on_access(compressor)
-            || this.expression.has_side_effects(compressor)
-            || this.property.has_side_effects(compressor);
+        var property = this.property.has_side_effects(compressor);
+        if (property && this.optional) return true; // "?." is a condition
+
+        return property || this.expression.has_side_effects(compressor);
     });
     def_has_side_effects(AST_Chain, function (compressor) {
         return this.expression.has_side_effects(compressor);
@@ -15055,7 +15050,9 @@ def_drop_side_effect_free(AST_Dot, function (compressor, first_in_statement) {
     if (is_nullish_shortcircuited(this, compressor)) {
         return this.expression.drop_side_effect_free(compressor, first_in_statement);
     }
-    if (this.expression.may_throw_on_access(compressor)) return this;
+    if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+        return this;
+    }
 
     return this.expression.drop_side_effect_free(compressor, first_in_statement);
 });
@@ -15064,15 +15061,17 @@ def_drop_side_effect_free(AST_Sub, function (compressor, first_in_statement) {
     if (is_nullish_shortcircuited(this, compressor)) {
         return this.expression.drop_side_effect_free(compressor, first_in_statement);
     }
-    if (this.expression.may_throw_on_access(compressor)) return this;
+    if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+        return this;
+    }
 
-    var expression = this.expression.drop_side_effect_free(compressor, first_in_statement);
-    if (!expression)
-        return this.property.drop_side_effect_free(compressor, first_in_statement);
     var property = this.property.drop_side_effect_free(compressor);
-    if (!property)
-        return expression;
-    return make_sequence(this, [expression, property]);
+    if (property && this.optional) return this;
+
+    var expression = this.expression.drop_side_effect_free(compressor, first_in_statement);
+
+    if (expression && property) return make_sequence(this, [expression, property]);
+    return expression || property;
 });
 
 def_drop_side_effect_free(AST_Chain, function (compressor, first_in_statement) {
@@ -28544,6 +28543,7 @@ var domprops = [
     "rotationAngle",
     "rotationRate",
     "round",
+    "roundRect",
     "row-gap",
     "rowGap",
     "rowIndex",
@@ -30985,4 +30985,4 @@ exports._default_options = _default_options;
 exports._run_cli = run_cli;
 exports.minify = minify;
 
-}));
+})));

package/lib/compress/drop-side-effect-free.js

@@ -321,7 +321,9 @@ def_drop_side_effect_free(AST_Dot, function (compressor, first_in_statement) {
     if (is_nullish_shortcircuited(this, compressor)) {
         return this.expression.drop_side_effect_free(compressor, first_in_statement);
     }
-    if (this.expression.may_throw_on_access(compressor)) return this;
+    if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+        return this;
+    }
 
     return this.expression.drop_side_effect_free(compressor, first_in_statement);
 });
@@ -330,15 +332,17 @@ def_drop_side_effect_free(AST_Sub, function (compressor, first_in_statement) {
     if (is_nullish_shortcircuited(this, compressor)) {
         return this.expression.drop_side_effect_free(compressor, first_in_statement);
     }
-    if (this.expression.may_throw_on_access(compressor)) return this;
+    if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+        return this;
+    }
 
-    var expression = this.expression.drop_side_effect_free(compressor, first_in_statement);
-    if (!expression)
-        return this.property.drop_side_effect_free(compressor, first_in_statement);
     var property = this.property.drop_side_effect_free(compressor);
-    if (!property)
-        return expression;
-    return make_sequence(this, [expression, property]);
+    if (property && this.optional) return this;
+
+    var expression = this.expression.drop_side_effect_free(compressor, first_in_statement);
+
+    if (expression && property) return make_sequence(this, [expression, property]);
+    return expression || property;
 });
 
 def_drop_side_effect_free(AST_Chain, function (compressor, first_in_statement) {

package/lib/compress/drop-unused.js

@@ -77,10 +77,6 @@ import {
     TreeTransformer,
     TreeWalker,
     walk,
-
-    _INLINE,
-    _NOINLINE,
-    _PURE
 } from "../ast.js";
 import {
     keep_name,

package/lib/compress/index.js

@@ -137,9 +137,7 @@ import {
     walk,
     walk_abort,
 
-    _INLINE,
     _NOINLINE,
-    _PURE
 } from "../ast.js";
 import {
     defaults,

package/lib/compress/inference.js

@@ -377,16 +377,27 @@ export function is_nullish(node, compressor) {
         return any(this.elements, compressor);
     });
     def_has_side_effects(AST_Dot, function(compressor) {
-        if (is_nullish(this, compressor)) return false;
-        return !this.optional && this.expression.may_throw_on_access(compressor)
-            || this.expression.has_side_effects(compressor);
+        if (is_nullish(this, compressor)) {
+            return this.expression.has_side_effects(compressor);
+        }
+        if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+            return true;
+        }
+
+        return this.expression.has_side_effects(compressor);
     });
     def_has_side_effects(AST_Sub, function(compressor) {
-        if (is_nullish(this, compressor)) return false;
+        if (is_nullish(this, compressor)) {
+            return this.expression.has_side_effects(compressor);
+        }
+        if (!this.optional && this.expression.may_throw_on_access(compressor)) {
+            return true;
+        }
 
-        return !this.optional && this.expression.may_throw_on_access(compressor)
-            || this.expression.has_side_effects(compressor)
-            || this.property.has_side_effects(compressor);
+        var property = this.property.has_side_effects(compressor);
+        if (property && this.optional) return true; // "?." is a condition
+
+        return property || this.expression.has_side_effects(compressor);
     });
     def_has_side_effects(AST_Chain, function (compressor) {
         return this.expression.has_side_effects(compressor);

package/lib/compress/reduce-vars.js

@@ -96,10 +96,6 @@ import {
     walk_body,
 
     TreeWalker,
-
-    _INLINE,
-    _NOINLINE,
-    _PURE
 } from "../ast.js";
 import { HOP, make_node, noop } from "../utils/index.js";
 

package/lib/output.js

@@ -190,18 +190,6 @@ class Rope {
         this.current += str;
     }
 
-    endsWith(str) {
-        const { committed, current } = this;
-        const len = str.length;
-        if (committed.length >= len) {
-            return committed.slice(committed.length - str.length) === str;
-        } else {
-            // `str` is small and this is a rare case so keep it simple
-            const last_bit = committed.slice(-len) + current;
-            return last_bit.endsWith(str);
-        }
-    }
-
     insertAt(char, index) {
         const { committed, current } = this;
         if (index < committed.length) {
@@ -583,10 +571,6 @@ function OutputStream(options) {
         might_need_semicolon = true;
     };
 
-    function ends_with(str) {
-        return OUTPUT.endsWith(str);
-    }
-
     function force_semicolon() {
         might_need_semicolon = false;
         print(";");
@@ -856,7 +840,6 @@ function OutputStream(options) {
         comma           : comma,
         colon           : colon,
         last            : function() { return last; },
-        ends_with       : ends_with,
         semicolon       : semicolon,
         force_semicolon : force_semicolon,
         to_utf8         : to_utf8,
@@ -1980,7 +1963,8 @@ function OutputStream(options) {
     });
     DEFPRINT(AST_UnaryPrefix, function(self, output) {
         var op = self.operator;
-        if (op === "--" && output.ends_with("<!")) {
+        if (op === "--" && output.last().endsWith("!")) {
+            // avoid printing "<!--"
             output.print(" ");
         }
         output.print(op);
@@ -2000,7 +1984,7 @@ function OutputStream(options) {
         var op = self.operator;
         self.left.print(output);
         if (op[0] == ">" /* ">>" ">>>" ">" ">=" */
-            && output.ends_with("--")) {
+            && output.last().endsWith("--")) {
             // space is mandatory to avoid outputting -->
             output.print(" ");
         } else {
@@ -2319,7 +2303,7 @@ function OutputStream(options) {
 
         // Avoid outputting end of script tag
         source = source.replace(r_slash_script, slash_script_replace);
-        if (r_starts_with_script.test(source) && output.ends_with("<")) {
+        if (r_starts_with_script.test(source) && output.last().endsWith("<")) {
             output.print(" ");
         }
 

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

package/tools/domprops.js

@@ -6570,6 +6570,7 @@ export var domprops = [
     "rotationAngle",
     "rotationRate",
     "round",
+    "roundRect",
     "row-gap",
     "rowGap",
     "rowIndex",