terser 3.16.1 → 3.17.0

package/CHANGELOG.md

@@ -1,8 +1,17 @@
 # Changelog
 
+## v3.17.0
+
+ - More DOM properties added to --mangle-properties's DOM property list
+ - Closed issue where if 2 functions had the same argument name, Terser would not inline them together properly
+ - Fixed issue with `hasOwnProperty.call`
+ - You can now list files to minify in a Terser config file
+ - Started replacing `new Array(<number>)` with an array literal
+ - Started using ES6 capabilities like `Set` and the `includes` method for strings and arrays
+
 ## v3.16.1
 
- - Fixed issue where Terser being imported with `import` would cause it not to work due to the `__esModule` property.
+ - Fixed issue where Terser being imported with `import` would cause it not to work due to the `__esModule` property. (PR #254 was submitted, which was nice, but since it wasn't a pure commonJS approach I decided to go with my own solution)
 
 ## v3.16.0
 

package/README.md

@@ -5,16 +5,18 @@ terser
 
 A JavaScript parser and mangler/compressor toolkit for ES6+.
 
-*note*: You can support this project on patreon: <a target="_blank" rel="nofollow" href="https://www.patreon.com/terser_ecmacomp_maintainer"><img src="https://c5.patreon.com/external/logo/become_a_patron_button@2x.png" alt="patron" width="100px" height="auto"></a>. Check out PATRONS.md for our first-tier patrons.
+*note*: You can support this project on patreon: <a target="_blank" rel="nofollow" href="https://www.patreon.com/fabiosantoscode"><img src="https://c5.patreon.com/external/logo/become_a_patron_button@2x.png" alt="patron" width="100px" height="auto"></a>. Check out PATRONS.md for our first-tier patrons.
 
 Terser recommends you use RollupJS to bundle your modules, as that produces smaller code overall.
 
 *Beautification* has been undocumented and is *being removed* from terser, we recommend you use [prettier](https://npmjs.com/package/prettier).
 
-[![Build Status](https://travis-ci.org/terser-js/terser.svg?branch=master)](https://travis-ci.org/terser-js/terser) [![Coverage Status](https://coveralls.io/repos/github/terser-js/terser/badge.svg?branch=master)](https://coveralls.io/github/terser-js/terser?branch=master)
+[![Build Status](https://travis-ci.org/terser-js/terser.svg?branch=master)](https://travis-ci.org/terser-js/terser)
 
 Find the changelog in [CHANGELOG.md](https://github.com/terser-js/terser/blob/master/CHANGELOG.md)
 
+A JavaScript parser, mangler/compressor and beautifier toolkit for ES6+.
+
 
 Why choose terser?
 ------------------
@@ -87,6 +89,7 @@ a double dash to prevent input files being used as option arguments:
                                 `keep_quoted`  Only mangle unquoted properties.
                                 `regex`  Only mangle matched property names.
                                 `reserved`  List of names that should not be mangled.
+    -b, --beautify [options]     Specify output options:
                                 `preamble`  Preamble to prepend to the output. You
                                             can use this to insert a comment, for
                                             example for licensing information.
@@ -239,11 +242,28 @@ to prevent the `require`, `exports` and `$` names from being changed.
 
 ### CLI mangling property names (`--mangle-props`)
 
-**Note:** THIS *MIGHT* BREAK YOUR CODE.  Mangling property names
-is a separate step, different from variable name mangling.  Pass
-`--mangle-props` to enable it.  It will mangle all properties in the
+**Note:** THIS **WILL** BREAK YOUR CODE. A good rule of thumb is not to use this unless you know exactly what you're doing and how this works and read this section until the end.
+
+Mangling property names is a separate step, different from variable name mangling.  Pass
+`--mangle-props` to enable it. The least dangerous
+way to use this is to use the `regex` option like so:
+
+```
+terser example.js -c -m --mangle-props regex=/_$/
+```
+
+This will mangle all properties that start with an 
+underscore. So you can use it to mangle internal methods.
+
+By default, it will mangle all properties in the
 input code with the exception of built in DOM properties and properties
-in core JavaScript classes.  For example:
+in core JavaScript classes, which is what will break your code if you don't:
+
+1. Control all the code you're mangling
+2. Avoid using a module bundler, as they usually will call Terser on each file individually, making it impossible to pass mangled objects between modules.
+3. Avoid calling functions like `defineProperty` or `hasOwnProperty`, because they refer to object properties using strings and will break your code if you don't know what you are doing.
+
+An example:
 
 ```javascript
 // example.js
@@ -258,21 +278,21 @@ x.bar_ = 2;
 x["baz_"] = 3;
 console.log(x.calc());
 ```
-Mangle all properties (except for JavaScript `builtins`):
+Mangle all properties (except for JavaScript `builtins`) (**very** unsafe):
 ```bash
 $ terser example.js -c -m --mangle-props
 ```
 ```javascript
 var x={o:0,_:1,l:function(){return this._+this.o}};x.t=2,x.o=3,console.log(x.l());
 ```
-Mangle all properties except for `reserved` properties:
+Mangle all properties except for `reserved` properties (still very unsafe):
 ```bash
 $ terser example.js -c -m --mangle-props reserved=[foo_,bar_]
 ```
 ```javascript
 var x={o:0,foo_:1,_:function(){return this.foo_+this.o}};x.bar_=2,x.o=3,console.log(x._());
 ```
-Mangle all properties matching a `regex`:
+Mangle all properties matching a `regex` (not as unsafe but still unsafe):
 ```bash
 $ terser example.js -c -m --mangle-props regex=/_$/
 ```
@@ -466,6 +486,7 @@ var options = {
         passes: 2
     },
     output: {
+        beautify: false,
         preamble: "/* minified */"
     }
 };
@@ -923,12 +944,18 @@ Terser.minify(code, { mangle: { toplevel: true } }).code;
 
 ## Output options
 
-The code generator tries to output shortest code possible. Optionally you
+The code generator tries to output shortest code possible by default.  In
+case you want beautified output, pass `--beautify` (`-b`).  Optionally you
 can pass additional arguments that control the code output:
 
 - `ascii_only` (default `false`) -- escape Unicode characters in strings and
   regexps (affects directives with non-ascii characters becoming invalid)
 
+- `beautify` (default `true`) -- whether to actually beautify the output.
+  Passing `-b` will set this to true, but you might need to pass `-b` even
+  when you want to generate minified code, in order to specify additional
+  arguments, so you can use `-b beautify=false` to override it.
+
 - `braces` (default `false`) -- always insert braces in `if`, `for`,
   `do`, `while` or `with` statements, even if their body is a single
   statement.
@@ -939,7 +966,8 @@ can pass additional arguments that control the code output:
 
 - `ecma` (default `5`) -- set output printing mode. Set `ecma` to `6` or
   greater to emit shorthand object properties - i.e.: `{a}` instead of `{a: a}`.
-  Non-compatible features in the abstract syntax tree will still
+  The `ecma` option will only change the output in direct control of the
+  beautifier. Non-compatible features in the abstract syntax tree will still
   be output as is. For example: an `ecma` setting of `5` will **not** convert
   ES6+ code to ES5.
 

package/bin/uglifyjs

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 // -*- js -*-
+/* eslint-env node */
 
 "use strict";
 
@@ -27,12 +28,12 @@ var options = {
 program.version(info.name + " " + info.version);
 program.parseArgv = program.parse;
 program.parse = undefined;
-if (process.argv.indexOf("ast") >= 0) program.helpInformation = describe_ast;
-else if (process.argv.indexOf("options") >= 0) program.helpInformation = function() {
+if (process.argv.includes("ast")) program.helpInformation = describe_ast;
+else if (process.argv.includes("options")) program.helpInformation = function() {
     var text = [];
     var options = UglifyJS.default_options();
     for (var option in options) {
-        text.push("--" + (option == "output" ? "beautify" : option == "sourceMap" ? "source-map" : option) + " options:");
+        text.push("--" + (option === "output" ? "beautify" : option === "sourceMap" ? "source-map" : option) + " options:");
         text.push(format_object(options[option]));
         text.push("");
     }
@@ -160,8 +161,18 @@ if (program.verbose) {
 } else if (program.warn) {
     options.warnings = true;
 }
-if (program.args.length) {
-    simple_glob(program.args).forEach(function(name) {
+
+let filesList;
+if (options.files && options.files.length) {
+    filesList = options.files;
+
+    delete options.files;
+} else if (program.args.length) {
+    filesList = program.args;
+}
+
+if (filesList) {
+    simple_glob(filesList).forEach(function(name) {
         files[convert_path(name)] = read_file(name);
     });
     run();
@@ -297,7 +308,7 @@ function run() {
 }
 
 function fatal(message) {
-    if (message instanceof Error) message = message.stack.replace(/^\S*?Error:/, "ERROR:")
+    if (message instanceof Error) message = message.stack.replace(/^\S*?Error:/, "ERROR:");
     print_error(message);
     process.exit(1);
 }
@@ -390,7 +401,7 @@ function parse_js(flag) {
             }
         }
         return options;
-    }
+    };
 }
 
 function parse_source_map() {
@@ -399,15 +410,14 @@ function parse_source_map() {
         var hasContent = options && "content" in options;
         var settings = parse(value, options);
         if (!hasContent && settings.content && settings.content != "inline") {
-            print_error("INFO: Using input source map: " + settings.content);
             settings.content = read_file(settings.content, settings.content);
         }
         return settings;
-    }
+    };
 }
 
 function skip_key(key) {
-    return skip_keys.indexOf(key) >= 0;
+    return skip_keys.includes(key);
 }
 
 function symdef(def) {
@@ -442,13 +452,13 @@ function describe_ast() {
     var out = UglifyJS.OutputStream({ beautify: true });
     function doitem(ctor) {
         out.print("AST_" + ctor.TYPE);
-        var props = ctor.SELF_PROPS.filter(function(prop){
+        var props = ctor.SELF_PROPS.filter(function(prop) {
             return !/^\$/.test(prop);
         });
         if (props.length > 0) {
             out.space();
-            out.with_parens(function(){
-                props.forEach(function(prop, i){
+            out.with_parens(function() {
+                props.forEach(function(prop, i) {
                     if (i) out.space();
                     out.print(prop);
                 });
@@ -460,15 +470,15 @@ function describe_ast() {
         }
         if (ctor.SUBCLASSES.length > 0) {
             out.space();
-            out.with_block(function(){
-                ctor.SUBCLASSES.forEach(function(ctor, i){
+            out.with_block(function() {
+                ctor.SUBCLASSES.forEach(function(ctor, i) {
                     out.indent();
                     doitem(ctor);
                     out.newline();
                 });
             });
         }
-    };
+    }
     doitem(UglifyJS.AST_Node);
     return out + "\n";
 }

package/bin/uglifyjsnobundle

@@ -1,3 +1,4 @@
 #!/usr/bin/env node
-process.env.TERSER_NO_BUNDLE = '1'
-require('./uglifyjs')
+/* eslint-env node */
+process.env.TERSER_NO_BUNDLE = "1";
+require("./uglifyjs");