terser 3.14.0 → 3.17.0

package/CHANGELOG.md

@@ -0,0 +1,28 @@
+# 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. (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
+
+ - No longer leaves names like Array or Object or window as a SimpleStatement (statement which is just a single expression).
+ - Add support for sections sourcemaps (IndexedSourceMapConsumer)
+ - Drops node.js v4 and starts using commonJS
+ - Is now built with rollup
+
+## v3.15.0
+
+ - Inlined spread syntax (`[...[1, 2, 3], 4, 5] => [1, 2, 3, 4, 5]`) in arrays and objects.
+ - Fixed typo in compressor warning
+ - Fixed inline source map input bug
+ - Fixed parsing of template literals with unnecessary escapes (Like `\\a`)

package/PATRONS.md

@@ -0,0 +1,5 @@
+ * 38elements
+ * CKEditor
+ * Philippe Léger
+ * Piotrek Koszuliński
+ * Viktor Hubert

package/README.md

@@ -5,13 +5,17 @@ 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_jscomp_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?
@@ -85,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.
@@ -237,11 +242,28 @@ to prevent the `require`, `exports` and `$` names from being changed.
 
 ### CLI mangling property names (`--mangle-props`)
 
-**Note:** THIS WILL PROBABLY 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
@@ -256,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=/_$/
 ```
@@ -367,6 +389,11 @@ like this:
 ```javascript
 var Terser = require("terser");
 ```
+Browser loading is also supported:
+```html
+<script src="node_modules/source-map/dist/source-map.min.js"></script>
+<script src="dist/bundle.min.js"></script>
+```
 
 There is a single high level function, **`minify(code, options)`**,
 which will perform all minification [phases](#minify-options) in a configurable
@@ -459,6 +486,7 @@ var options = {
         passes: 2
     },
     output: {
+        beautify: false,
         preamble: "/* minified */"
     }
 };
@@ -916,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.
@@ -932,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.
 
@@ -1263,8 +1298,8 @@ In the terser CLI we use [source-map-support](https://npmjs.com/source-map-suppo
 
 # README.md Patrons:
 
-*note*: You can support this project on patreon: <a target="_blank" rel="nofollow" href="https://www.patreon.com/terser_jscomp_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/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.
 
- * CKEditor ![CKEditor](https://c10.patreonusercontent.com/3/eyJoIjoxMDAsInciOjEwMH0%3D/patreon-media/p/user/15452278/f8548dcf48d740619071e8d614459280/1?token-time=2145916800&token-hash=SIQ54PhIPHv3M7CVz9LxS8_8v4sOw4H304HaXsXj8MM%3D)
 
+ * CKEditor ![](https://c10.patreonusercontent.com/3/eyJoIjoxMDAsInciOjEwMH0%3D/patreon-media/p/user/15452278/f8548dcf48d740619071e8d614459280/1?token-time=2145916800&token-hash=SIQ54PhIPHv3M7CVz9LxS8_8v4sOw4H304HaXsXj8MM%3D)
+ * 38elements ![](https://c10.patreonusercontent.com/3/eyJ3IjoyMDB9/patreon-media/p/user/12501844/88e7fc5dd62d45c6a5626533bbd48cfb/1?token-time=2145916800&token-hash=c3AsQ5T0IQWic0zKxFHu-bGGQJkXQFvafvJ4bPerFR4%3D)

package/bin/uglifyjs

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 // -*- js -*-
+/* eslint-env node */
 
 "use strict";
 
@@ -10,15 +11,13 @@ var info = require("../package.json");
 var path = require("path");
 var program = require("commander");
 
-var bundle_path = __dirname + "/../dist/bundle.js";
-if (!process.env.TERSER_NO_BUNDLE && fs.existsSync(bundle_path)) {
-    var UglifyJS = require(bundle_path)
-    try {
-        require("source-map-support").install();
-    } catch (err) {}
-} else {
-    var UglifyJS = require("../tools/node.js");
-}
+var bundle_path = __dirname + (process.env.TERSER_NO_BUNDLE ?
+    "/../dist/bundle.js" :
+    "/../dist/bundle.min.js");
+var UglifyJS = require(bundle_path);
+try {
+    require("source-map-support").install();
+} catch (err) {}
 
 var skip_keys = [ "cname", "inlined", "parent_scope", "scope", "uses_eval", "uses_with" ];
 var files = {};
@@ -29,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("");
     }
@@ -162,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();
@@ -299,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);
 }
@@ -312,7 +321,7 @@ function simple_glob(glob) {
     if (Array.isArray(glob)) {
         return [].concat.apply([], glob.map(simple_glob));
     }
-    if (glob && glob.match(/\*|\?/)) {
+    if (glob && glob.match(/[*?]/)) {
         var dir = path.dirname(glob);
         try {
             var entries = fs.readdirSync(dir);
@@ -392,7 +401,7 @@ function parse_js(flag) {
             }
         }
         return options;
-    }
+    };
 }
 
 function parse_source_map() {
@@ -401,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) {
@@ -444,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);
                 });
@@ -462,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");