joliss / broccoli-uglify-js

UglifyJS filter for Broccoli

broccoli-uglify-js

This package is deprecated in favor of broccoli-uglify-sourcemap.

UglifyJS2 filter for Broccoli.

Installation

npm install --save-dev broccoli-uglify-js

Usage

var uglifyJavaScript = require('broccoli-uglify-js');
tree = uglifyJavaScript(tree, options);

Options

The following options are supported:

• mangle (passed through to UglifyJS): pass false to skip mangling names

• compress (passed through to UglifyJS): pass false to skip compressing; pass an object to specify compressor options

• output (passed through to UglifyJS): pass an object to specify additional output options

To Do

• Enable ascii_only by default

Source Maps

Source maps are not yet supported.

broccoli-uglify-js's Issues

Question: Incremental Builds

I have a huge application at work and we're trying to speed up our grunt build. One of the bottlenecks is our uglify task. Part of the reason is we're minifying vendor files like jQuery, Angular, lodash, d3, etc every time we run the build.

My question is, I don't see broccoli-caching-writer as a dependency so I'm wondering if using this plugin would give us incremental builds. Thanks!

depends on outdated broccoli-filter

this brings along broccoli and seems to cause some people grief.

Deprecate in favor of broccoli-uglify-sourcemap?

This package hasn't been updated to support broccoli@1 yet, and it seems that most consumers are likely using https://github.com/ef4/broccoli-uglify-sourcemap. Should we deprecate this one in favor of that?

NSP Vulnerabilities

Hi Guys

Ive been running nsp on my projects and im getting a vulnerability warning for broccoli-uglify

┌───────────────┬────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│               │ Regular Expression Denial of Service                                                                                               │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Name          │ uglify-js                                                                                                                          │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Installed     │ 2.4.24                                                                                                                             │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Vulnerable    │ <2.6.0                                                                                                                             │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Patched       │ >=2.6.0                                                                                                                            │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ Path          │ [email protected] > [email protected] > [email protected]                                                                │
├───────────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┤
│ More Info     │ https://nodesecurity.io/advisories/48                                                                                              │
└───────────────┴────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

preserveComments option support

There is no way to avoid comments removing during minification, it would be great if uglifier could support preserveComments option

Should show more detailed error info

See https://github.com/mishoo/UglifyJS/blob/master/lib/parse-js.js#L258-L268