Skip to content

Fix options handling #85

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
kumavis wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix options handling #85

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 19 additions & 12 deletions index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,16 +33,11 @@ function uglifyify(file, opts) {
return through()
}

// remove exts before passing opts to uglify
delete opts.global
delete opts.exts
delete opts.x

return through(function write(chunk) {
buffer += chunk
}, capture(function ready() {
debug = opts.sourceMap !== false && debug
opts = extend({}, {
var _opts = extend({}, {
compress: true,
mangle: true,
sourceMap: {
Expand All @@ -53,21 +48,33 @@ function uglifyify(file, opts) {
// map out command line options to uglify compatible ones
mapArgv(opts)

if (typeof opts.compress === 'object') {
delete opts.compress._
// remove exts before passing opts to uglify
delete _opts.global
delete _opts.exts
delete _opts.x
delete _opts._
delete _opts._flags

if (typeof _opts.compress === 'object') {
delete _opts.compress._
}

if (debug) opts.sourceMap.url = 'out.js.map'
if (debug) _opts.sourceMap.url = 'out.js.map'

// Check if incoming source code already has source map comment.
// If so, send it in to ujs.minify as the inSourceMap parameter
if (debug) {
opts.sourceMap.content = 'inline'
_opts.sourceMap.content = 'inline'
}

var min = ujs.minify(buffer, opts)
var min = ujs.minify(buffer, _opts)

// we should catcch the min error if it comes back and end the stream
if (min.error) return this.emit('error', min.error)
if (min.error) {
const err = min.error instanceof Error ? min.error : new Error(min.error.message)
// will be emitted by `capture()`
throw err
}

// Uglify leaves a source map comment pointing back to "out.js.map",
// which we want to get rid of because it confuses browserify.
Expand Down