uglifier 2.7.2 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 93cb2441ef4a95a1bb409fa2d9c47acf7e18117a
-  data.tar.gz: 6e6265967cb909387d5e621262a84246f49b44a2
+  metadata.gz: 965824e428764606cd1617e6e72b9ab0ad912b7b
+  data.tar.gz: 996b18eac0ad32e9c9ad8b31e7a8ab90c0208039
 SHA512:
-  metadata.gz: c2524b95360adb6557e70e2a6354b6d969ed04a8981374187b27a4930c5d04885d112d4e70891baf5d7c6279652aac890772dac6c8972290ffb3a62cf8838631
-  data.tar.gz: ada6b5d1036f83ef1a236a62056b49947c9c583deb5c81fc66ac38b656eac367f25abc2f964ed05b492313d235603b4b789d21c761f97504fcefe3f9e613a5f4
+  metadata.gz: d41ff9576c911132ce28f20b530d1c3b925e6caa5416884608e69cbbbe25b1fda4a4a27fb267b320633b6d9bb34c632c9e1c60c5be6b7663309aecb0a17b93df
+  data.tar.gz: 748763a30cd991eec4c52648c0af2a40ae199646cb92b0dc481ff504b1ef388dbfd385817011ca5489dd4ad5f37d0dff0c6d0e52a39fc41b61be6d859946677a

data/.rubocop.yml

@@ -25,3 +25,9 @@ Style/SignalException:
 
 Style/StringLiterals:
   Enabled: false
+
+Style/Alias:
+  Enabled: false
+
+Style/MutableConstant:
+  Enabled: false

data/.travis.yml

@@ -2,12 +2,13 @@ sudo: false
 language: ruby
 cache: bundler
 rvm:
-  - 1.8.7
   - 1.9.3
   - 2.0.0
-  - 2.1.5
-  - 2.2.0
-  - jruby
+  - 2.1.7
+  - 2.2.3
+  - 2.3.0
+  - jruby-19mode
+  - jruby-9.0.4.0
   - rbx-2
 git:
   submodules: false
@@ -15,16 +16,16 @@ gemfile:
   - Gemfile
 matrix:
   include:
-    - rvm: 2.2.0
+    - rvm: 2.3.0
       gemfile: gemfiles/rubyracer
-    - rvm: 2.1.5
+    - rvm: 2.1.7
       gemfile: gemfiles/rubyracer
     - rvm: jruby-19mode
       gemfile: gemfiles/rubyrhino
-    - rvm: 2.2.0
+    - rvm: 2.3.0
       gemfile: gemfiles/alaska
       env: ALASKA=1
   allow_failures:
-    - rvm: 2.2.0
+    - rvm: 2.3.0
       gemfile: gemfiles/alaska
       env: ALASKA=1

data/.yardopts

@@ -0,0 +1,4 @@
+--title "Uglifier"
+-
+CHANGELOG.md
+LICENSE.txt

data/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 3.0.0 (22 March 2016)
+
+- drop support for Ruby 1.8
+- remove json as dependency
+- discard unused function arguments only in unsafe mode
+- add `keep_fnames` option to preserve function names in compressed code
+- add `collapse_vars` option to collapse single-use variables
+- backwards incompatible changes to source map options
+- support for inline base64 encoded source maps
+- mangle property names option (disabled by default)
+- update UglifyJS to 2.6.2
+
 ## 2.7.2 (26 August 2015)
 
 - update UglifyJS to 2.4.24

data/Gemfile

@@ -2,8 +2,4 @@ source "https://rubygems.org"
 
 gemspec
 
-if RUBY_VERSION >= '1.9'
-  gem 'rubocop', '~> 0.28.0', :group => [:development]
-else
-  gem 'execjs', '~> 2.0.2'
-end
+gem 'rubocop', '~> 0.37.2', :group => [:development]

data/README.md

@@ -1,7 +1,9 @@
-# Uglifier [![Build Status](https://travis-ci.org/lautis/uglifier.svg?branch=master)](https://travis-ci.org/lautis/uglifier) [![Dependency Status](https://gemnasium.com/lautis/uglifier.svg)](https://gemnasium.com/lautis/uglifier)
+# Uglifier
 
 Ruby wrapper for [UglifyJS](https://github.com/mishoo/UglifyJS2) JavaScript compressor.
 
+[![Build Status](https://travis-ci.org/lautis/uglifier.svg?branch=master)](https://travis-ci.org/lautis/uglifier)
+
 ## Installation
 
 Uglifier is available as a ruby gem.
@@ -65,6 +67,9 @@ Available options and their defaults are
     :sort => false,             # Assign shorter names to most frequently used variables. Often results in bigger output after gzip.
     :toplevel => false          # Mangle names declared in the toplevel scope
   },                            # Mangle variable and function names, set to false to skip mangling
+  :mangle_properties => {
+    :regex => nil               # A regular expression to filter property names to be mangled
+  }                             # Mangle property names, disabled by default
   :compress => {
     :sequences => true,         # Allow statements to be joined by commas
     :properties => true,        # Rewrite property access using the dot notation
@@ -82,22 +87,26 @@ Available options and their defaults are
     :if_return => true,         # Optimizations for if/return and if/continue
     :join_vars => true,         # Join consecutive var statements
     :cascade => true,           # Cascade sequences
+    :collapse_vars => false,    # Collapse single-use var and const definitions when possible.
     :negate_iife => true,       # Negate immediately invoked function expressions to avoid extra parens
     :pure_getters => false,     # Assume that object property access does not have any side-effects
     :pure_funcs => nil,         # List of functions without side-effects. Can safely discard function calls when the result value is not used
     :drop_console => false,     # Drop calls to console.* functions
-    :angular => false           # Process @ngInject annotations
-    :keep_fargs => false        # Preserve unused function arguments
+    :angular => false,          # Process @ngInject annotations
+    :keep_fargs => false,       # Preserve unused function arguments
+    :keep_fnames => false       # Preserve function names
   },                            # Apply transformations to code, set to false to skip
   :define => {},                # Define values for symbol replacement
   :enclose => false,            # Enclose in output function wrapper, define replacements as key-value pairs
-  :source_filename => nil,      # The filename of the input file
-  :source_root => nil,          # The URL of the directory which contains :source_filename
-  :output_filename => nil,      # The filename or URL where the minified output can be found
-  :input_source_map => nil,     # The contents of the source map describing the input
-  :screw_ie8 => false,          # Don't bother to generate safe code for IE8
-  :source_map_url => false,     # Url for source mapping to be appended in minified source
-  :source_url => false          # Url to original source to be appended in minified source
+  :source_map => {
+    :map_url => false,          # Url for source mapping to be appended in minified source
+    :url => false,              # Url for original source to be appended in minified source
+    :sources_content => false,  # Include original source content in map
+    :filename => nil,           # The filename of the input file
+    :root => nil,               # The URL of the directory which contains :filename
+    :output_filename => nil,    # The filename or URL where the minified output can be found
+    :input_source_map => nil    # The contents of the source map describing the input
+  }
 }
 ```
 

data/Rakefile

@@ -1,40 +1,15 @@
 # encoding: utf-8
 
-require 'rubygems'
-require 'bundler'
 require 'bundler/gem_tasks'
-
-begin
-  Bundler.setup(:default, :development)
-rescue Bundler::BundlerError => e
-  $stderr.puts e.message
-  $stderr.puts "Run `bundle install` to install missing gems"
-  exit e.status_code
-end
-
-require 'rake'
-
-require 'rspec/core'
 require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:spec) do |spec|
   spec.pattern = FileList['spec/**/*_spec.rb']
 end
 
-require 'rdoc/task'
-Rake::RDocTask.new do |rdoc|
-  version = File.exist?('VERSION') ? File.read('VERSION') : ""
-
-  rdoc.rdoc_dir = 'rdoc'
-  rdoc.title = "uglifier #{version}"
-  rdoc.rdoc_files.include('README*')
-  rdoc.rdoc_files.include('lib/**/*.rb')
-end
-
 desc "Rebuild lib/uglify.js"
 task :js do
   cd 'vendor/source-map/' do
     `npm install`
-    `node Makefile.dryice.js`
   end
 
   cd 'vendor/uglifyjs/' do
@@ -54,7 +29,7 @@ end
 begin
   require 'rubocop/rake_task'
   RuboCop::RakeTask.new(:rubocop)
-  task :default => [:spec]
+  task :default => [:rubocop, :spec]
 rescue LoadError
   task :default => [:spec]
 end

data/lib/uglifier.js

@@ -0,0 +1,111 @@
+function comments(option) {
+  if (Object.prototype.toString.call(option) === '[object Array]') {
+    return new RegExp(option[0], option[1]);
+  } else if (option == "jsdoc") {
+    return function(node, comment) {
+      if (comment.type == "comment2") {
+        return /@preserve|@license|@cc_on/i.test(comment.value);
+      } else {
+        return false;
+      }
+    };
+  } else {
+    return option;
+  }
+}
+
+function readNameCache(key) {
+  return UglifyJS.readNameCache(null, key);
+}
+
+function writeNameCache(key, cache) {
+  return UglifyJS.writeNameCache(null, key, cache);
+}
+
+function regexOption(options) {
+  if (typeof options === 'object' && options.regex) {
+    return new RegExp(options.regex[0], options.regex[1]);
+  } else {
+    return null;
+  }
+}
+
+function parse(source, options) {
+  var ast = UglifyJS.parse(source, options.parse_options);
+  ast.figure_out_scope();
+
+  if (options.compress) {
+    var compressor = UglifyJS.Compressor(options.compress);
+    ast = ast.transform(compressor);
+    ast.figure_out_scope();
+  }
+
+  if (options.mangle) {
+    ast.compute_char_frequency();
+    ast.mangle_names(options.mangle);
+  }
+
+  if (options.mangle_properties) {
+    var regex = regexOption(options.mangle_properties);
+    UglifyJS.mangle_properties(ast, {
+      reserved: [],
+      only_cache: false,
+      regex: regex
+    });
+  }
+
+  if (options.enclose) {
+    ast = ast.wrap_enclose(options.enclose);
+  }
+  return ast;
+}
+
+function copySourcesContent(sourceMap, options) {
+  sourceMap.get().setSourceContent(options.parse_options.filename, options.source);
+
+  var original = options.source_map_options.orig;
+
+  if (original && original.sources && original.sourcesContent) {
+    for(var i = 0; i < original.sources.length; i++) {
+      sourceMap.get().setSourceContent(original.sources[i], original.sourcesContent[i]);
+    }
+  }
+}
+
+function uglifier(options) {
+  var source = options.source;
+  var ast = parse(source, options);
+  var source_map;
+
+  var gen_code_options = options.output;
+  gen_code_options.comments = comments(options.output.comments);
+
+  if (options.generate_map) {
+    source_map = UglifyJS.SourceMap(options.source_map_options);
+    gen_code_options.source_map = source_map;
+
+    if (options.source_map_options.sources_content) {
+      copySourcesContent(source_map, options);
+    }
+  }
+
+  var stream = UglifyJS.OutputStream(gen_code_options);
+  ast.print(stream);
+
+  if (options.source_map_options.map_url) {
+    stream += "\n//# sourceMappingURL=" + options.source_map_options.map_url;
+  }
+
+  if (options.source_map_options.url) {
+    stream += "\n//# sourceURL=" + options.source_map_options.url;
+  }
+
+  if (options.generate_map) {
+    if (options.source_map_options.sources_content) {
+      source_map.get().setSourceContent(options.parse_options.filename, options.source);
+    }
+    return [stream.toString(), source_map.toString()];
+  } else {
+    return stream.toString();
+  }
+}

data/lib/uglifier.rb

@@ -1,78 +1,14 @@
 # encoding: UTF-8
 
-require "execjs"
 require "json"
+require "base64"
+require "execjs"
 require "uglifier/version"
 
 # A wrapper around the UglifyJS interface
 class Uglifier
   # Error class for compilation errors.
   Error = ExecJS::Error
-  # JavaScript code to call UglifyJS
-  JS = <<-JS
-    (function(options) {
-      function comments(option) {
-        if (Object.prototype.toString.call(option) === '[object Array]') {
-          return new RegExp(option[0], option[1]);
-        } else if (option == "jsdoc") {
-          return function(node, comment) {
-            if (comment.type == "comment2") {
-              return /@preserve|@license|@cc_on/i.test(comment.value);
-            } else {
-              return false;
-            }
-          }
-        } else {
-          return option;
-        }
-      }
-
-      var source = options.source;
-      var ast = UglifyJS.parse(source, options.parse_options);
-      ast.figure_out_scope();
-
-      if (options.compress) {
-        var compressor = UglifyJS.Compressor(options.compress);
-        ast = ast.transform(compressor);
-        ast.figure_out_scope();
-      }
-
-      if (options.mangle) {
-        ast.compute_char_frequency();
-        ast.mangle_names(options.mangle);
-      }
-
-      if (options.enclose) {
-        ast = ast.wrap_enclose(options.enclose);
-      }
-
-      var gen_code_options = options.output;
-      gen_code_options.comments = comments(options.output.comments);
-
-      if (options.generate_map) {
-          var source_map = UglifyJS.SourceMap(options.source_map_options);
-          gen_code_options.source_map = source_map;
-      }
-
-      var stream = UglifyJS.OutputStream(gen_code_options);
-
-      ast.print(stream);
-
-      if (options.source_map_options.map_url) {
-        stream += "\\n//# sourceMappingURL=" + options.source_map_options.map_url;
-      }
-
-      if (options.source_map_options.url) {
-        stream += "\\n//# sourceURL=" + options.source_map_options.url;
-      }
-
-      if (options.generate_map) {
-          return [stream.toString(), source_map.toString()];
-      } else {
-          return stream.toString();
-      }
-    })
-  JS
 
   # UglifyJS source path
   SourcePath = File.expand_path("../uglify.js", __FILE__)
@@ -80,6 +16,8 @@ class Uglifier
   ES5FallbackPath = File.expand_path("../es5.js", __FILE__)
   # String.split shim source path
   SplitFallbackPath = File.expand_path("../split.js", __FILE__)
+  # UglifyJS wrapper path
+  UglifyJSWrapperPath = File.expand_path("../uglifier.js", __FILE__)
 
   # Default options for compilation
   DEFAULTS = {
@@ -104,8 +42,10 @@ class Uglifier
       :eval => false, # Mangle names when eval of when is used in scope
       :except => ["$super"], # Argument names to be excluded from mangling
       :sort => false, # Assign shorter names to most frequently used variables. Often results in bigger output after gzip.
-      :toplevel => false # Mangle names declared in the toplevel scope
+      :toplevel => false, # Mangle names declared in the toplevel scope
+      :properties => false # Mangle property names
     }, # Mangle variable and function names, set to false to skip mangling
+    :mangle_properties => false, # Mangle property names
     :compress => {
       :sequences => true, # Allow statements to be joined by commas
       :properties => true, # Rewrite property access using the dot notation
@@ -123,23 +63,37 @@ class Uglifier
       :if_return => true, # Optimizations for if/return and if/continue
       :join_vars => true, # Join consecutive var statements
       :cascade => true, # Cascade sequences
+      :collapse_vars => false, # Collapse single-use var and const definitions when possible.
       :negate_iife => true, # Negate immediately invoked function expressions to avoid extra parens
       :pure_getters => false, # Assume that object property access does not have any side-effects
       :pure_funcs => nil, # List of functions without side-effects. Can safely discard function calls when the result value is not used
       :drop_console => false, # Drop calls to console.* functions
       :angular => false, # Process @ngInject annotations
-      :keep_fargs => false # Preserve unused function arguments
+      :keep_fargs => false, # Preserve unused function arguments
+      :keep_fnames => false # Preserve function names
     }, # Apply transformations to code, set to false to skip
     :define => {}, # Define values for symbol replacement
     :enclose => false, # Enclose in output function wrapper, define replacements as key-value pairs
-    :source_filename => nil, # The filename of the input file
-    :source_root => nil, # The URL of the directory which contains :source_filename
-    :output_filename => nil, # The filename or URL where the minified output can be found
-    :input_source_map => nil, # The contents of the source map describing the input
     :screw_ie8 => false, # Don't bother to generate safe code for IE8
-    :source_map_url => false, # Url for source mapping to be appended in minified source
-    :source_url => false # Url for original source to be appended in minified source
+    :source_map => false # Generate source map
+  }
+
+  LEGACY_OPTIONS = [:comments, :squeeze, :copyright, :mangle]
+
+  MANGLE_PROPERTIES_DEFAULTS = {
+    :regex => nil # A regular expression to filter property names to be mangled
   }
+
+  SOURCE_MAP_DEFAULTS = {
+    :map_url => false, # Url for source mapping to be appended in minified source
+    :url => false, # Url for original source to be appended in minified source
+    :sources_content => false, # Include original source content in map
+    :filename => nil, # The filename of the input file
+    :root => nil, # The URL of the directory which contains :filename
+    :output_filename => nil, # The filename or URL where the minified output can be found
+    :input_source_map => nil # The contents of the source map describing the input
+  }
+
   # rubocop:enable LineLength
 
   # Minifies JavaScript code using implicit context.
@@ -164,7 +118,7 @@ class Uglifier
   #
   # @param options [Hash] optional overrides to +Uglifier::DEFAULTS+
   def initialize(options = {})
-    (options.keys - DEFAULTS.keys - [:comments, :squeeze, :copyright])[0..1].each do |missing|
+    (options.keys - DEFAULTS.keys - LEGACY_OPTIONS)[0..1].each do |missing|
       raise ArgumentError, "Invalid option: #{missing}"
     end
     @options = options
@@ -176,7 +130,14 @@ class Uglifier
   # @param source [IO, String] valid JS source code.
   # @return [String] minified code.
   def compile(source)
-    run_uglifyjs(source, false)
+    if @options[:source_map]
+      compiled, source_map = run_uglifyjs(source, true)
+      source_map_uri = Base64.strict_encode64(source_map)
+      source_map_mime = "application/json;charset=utf-8;base64"
+      compiled + "\n//# sourceMappingURL=data:#{source_map_mime},#{source_map_uri}"
+    else
+      run_uglifyjs(source, false)
+    end
   end
   alias_method :compress, :compile
 
@@ -191,25 +152,28 @@ class Uglifier
   private
 
   def uglifyjs_source
-    [ES5FallbackPath, SplitFallbackPath, SourcePath].map do |file|
-      File.open(file, "r:UTF-8") { |f| f.read }
+    [ES5FallbackPath, SplitFallbackPath, SourcePath, UglifyJSWrapperPath].map do |file|
+      File.open(file, "r:UTF-8", &:read)
     end.join("\n")
   end
 
   # Run UglifyJS for given source code
-  def run_uglifyjs(source, generate_map)
+  def run_uglifyjs(input, generate_map)
+    source = read_source(input)
+    input_map = input_source_map(source, generate_map)
     options = {
-      :source => read_source(source),
+      :source => source,
       :output => output_options,
       :compress => compressor_options,
       :mangle => mangle_options,
+      :mangle_properties => mangle_properties_options,
       :parse_options => parse_options,
-      :source_map_options => source_map_options,
+      :source_map_options => source_map_options(input_map),
       :generate_map => generate_map,
       :enclose => enclose_options
     }
 
-    @context.call(Uglifier::JS, options)
+    @context.call("uglifier", options)
   end
 
   def read_source(source)
@@ -221,14 +185,25 @@ class Uglifier
   end
 
   def mangle_options
-    conditional_option(@options[:mangle], DEFAULTS[:mangle])
+    mangle_options = @options.fetch(:mangle, @options[:mangle])
+    conditional_option(mangle_options, DEFAULTS[:mangle])
+  end
+
+  def mangle_properties_options
+    mangle_options = @options.fetch(:mangle_properties, DEFAULTS[:mangle_properties])
+    options = conditional_option(mangle_options, MANGLE_PROPERTIES_DEFAULTS)
+    if options && options[:regex]
+      options.merge(:regex => encode_regexp(options[:regex]))
+    else
+      options
+    end
   end
 
   def compressor_options
     defaults = conditional_option(
       DEFAULTS[:compress],
       :global_defs => @options[:define] || {},
-      :screw_ie8 => @options[:screw_ie8] || DEFAULTS[:screw_ie8]
+      :screw_ie8 => screw_ie8?
     )
     conditional_option(@options[:compress] || @options[:squeeze], defaults)
   end
@@ -269,24 +244,31 @@ class Uglifier
 
   def screw_ie8?
     if (@options[:output] || {}).has_key?(:ie_proof)
-      false
+      !@options[:output][:ie_proof]
     else
-      @options[:screw_ie8] || DEFAULTS[:screw_ie8]
+      @options.fetch(:screw_ie8, DEFAULTS[:screw_ie8])
     end
   end
 
-  def source_map_options
+  def source_map_options(input_map)
+    options = conditional_option(@options[:source_map], SOURCE_MAP_DEFAULTS)
+
     {
-      :file => @options[:output_filename],
-      :root => @options[:source_root],
-      :orig => @options[:input_source_map],
-      :map_url => @options[:source_map_url],
-      :url => @options[:source_url]
+      :file => options[:output_filename],
+      :root => options.fetch(:root) { input_map ? input_map["sourceRoot"] : nil },
+      :orig => input_map,
+      :map_url => options[:map_url],
+      :url => options[:url],
+      :sources_content => options[:sources_content]
     }
   end
 
   def parse_options
-    { :filename => @options[:source_filename] }
+    if @options[:source_map].respond_to?(:[])
+      { :filename => @options[:source_map][:filename] }
+    else
+      {}
+    end
   end
 
   def enclose_options
@@ -318,4 +300,38 @@ class Uglifier
       false
     end
   end
+
+  def sanitize_map_root(map)
+    if map.nil?
+      nil
+    elsif map.is_a? String
+      sanitize_map_root(JSON.load(map))
+    elsif map["sourceRoot"] == ""
+      map.merge("sourceRoot" => nil)
+    else
+      map
+    end
+  end
+
+  def extract_source_mapping_url(source)
+    comment_start = %r{(?://|/\*\s*)}
+    comment_end = %r{\s*(?:\r?\n?\*/|$)?}
+    source_mapping_regex = /#{comment_start}[@#]\ssourceMappingURL=\s*(\S*?)#{comment_end}/
+    rest = /\s#{comment_start}[@#]\s[a-zA-Z]+=\s*(?:\S*?)#{comment_end}/
+    regex = /#{source_mapping_regex}(?:#{rest})*\Z/m
+    match = regex.match(source)
+    match && match[1]
+  end
+
+  def input_source_map(source, generate_map)
+    return nil unless generate_map
+    sanitize_map_root(@options.fetch(:source_map, {}).fetch(:input_source_map) do
+      url = extract_source_mapping_url(source)
+      if url && url.start_with?("data:")
+        Base64.strict_decode64(url.split(",", 2)[-1])
+      end
+    end)
+  rescue ArgumentError, JSON::ParserError
+    nil
+  end
 end