opal-optimizer 0.1.2 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d76a6684777438ef696216358695d66b0e2ddd212f5b9d28d482843c912a2ac4
-  data.tar.gz: 536b978277f2af33f5ac47ad8661629e151d54adf11d88f8e0a63487327956d2
+  metadata.gz: 6498304a0ea20dac60d150ea41df0876ade32476a48a489daca24396686de528
+  data.tar.gz: e453ead6b82eedb0fd1c03fc38cd2de64a5d1ceda31a55a541e83f50b9ed76d8
 SHA512:
-  metadata.gz: 3e87db8dbc1976b663dfc0fcbbc25b3317058e06c72f759b131e16caeaf589355f85e029e2307392ba386f8c7787128a70a39743570a2d854f327f8b054187fb
-  data.tar.gz: f29200eb79910e28c08acf486e3881468ec0b5cdc8718787a0e719193e6a2ac1b96c9b43acfc5fbcfe2158e710ce19371d67795521119fbd448db216f87512f4
+  metadata.gz: bcbf6fb6a5f209e3cea950be73637504ce830c16a9ef8e7402156b64d82e8905c6406a12e0acbc52feb2a1770a4f869ca1c52b78c6f08b590c6c64df94d7c9c7
+  data.tar.gz: 941e429601ce081347bad1179f152f0dd5aa0f3b20374cf683e021ca1c00e5e078cd63e44476af127fcc8d21eb0f34b35c605df5ae9a129c2e28c3917c970a59

data/.github/workflows/build.yml

@@ -0,0 +1,63 @@
+name: build
+
+on:
+  push:
+    branches:
+      - master
+      - "*-stable"
+      - "*/ci-check"
+  pull_request: {}
+
+jobs:
+  rake:
+    name: ${{ matrix.combo.name }}
+    strategy:
+      fail-fast: false
+      matrix:
+        combo:
+          - name: Opal-master
+            opal: master
+          - name: Opal-1.4
+            opal: 1.4
+          - name: Opal-1.3
+            opal: 1.3
+          - name: Opal-1.2
+            opal: 1.2
+          - name: Opal-1.1
+            opal: 1.1
+          - name: Opal-1.0
+            opal: '1.0'
+          - name: Ruby-head
+            ruby: head
+          - name: Windows
+            os: windows-latest
+          - name: macOS
+            os: macos-latest
+
+    runs-on: ${{ matrix.combo.os || 'ubuntu-latest' }}
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: set environment variables
+        run: |
+          echo "OPAL_VERSION=${{ matrix.combo.opal || '1.3' }}" >> $GITHUB_ENV
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.combo.ruby || '3.0' }}
+      - run: bundle lock
+      - uses: actions/cache@v2
+        with:
+          path: ./vendor/bundle
+          key: ${{ runner.os }}-${{ matrix.combo.ruby || '3.0' }}-gems-${{ matrix.combo.opal || '1.3' }}-${{ github.ref }}-${{ hashFiles('**/Gemfile.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-${{ matrix.combo.ruby || '3.0' }}-gems-${{ matrix.combo.opal || '1.3' }}-${{ github.ref }}
+            ${{ runner.os }}-${{ matrix.combo.ruby || '3.0' }}-gems-${{ matrix.combo.opal || '1.3' }}-master
+            ${{ runner.os }}-${{ matrix.combo.ruby || '3.0' }}-gems-${{ matrix.combo.opal || '1.3' }}-
+            ${{ runner.os }}-${{ matrix.combo.ruby || '3.0' }}-gems-
+      - name: bundle install
+        run: |
+          bundle config path $PWD/vendor/bundle
+          bundle install --jobs 4 --retry 3
+          bundle clean
+      - run: bundle exec rake
+

data/.gitignore

@@ -7,5 +7,7 @@
 /spec/reports/
 /tmp/
 
+/Gemfile.lock
+
 # rspec failure tracking
 .rspec_status

data/CHANGELOG.md

@@ -0,0 +1,7 @@
+## v0.1.6
+
+* Further Opal 1.4 compatibility
+
+## v0.1.5
+
+* Opal 1.4 compatibility

data/Gemfile

@@ -3,14 +3,20 @@ source "https://rubygems.org"
 # Specify your gem's dependencies in opal-optimizer.gemspec
 gemspec
 
-gem "rake", "~> 12.0"
-gem "rspec", "~> 3.0"
+gem "rake"
+gem "rspec"
 
-gem "opal", path: "../opal"
-gem "opal-sprockets", path: "../opal-sprockets"
-gem "opal-browser", path: "../opal-browser"
+case ENV['OPAL_VERSION']
+when nil
+when /\A[0-9.]+\z/
+  gem 'opal', "~> #{ENV['OPAL_VERSION']}.0a"
+else
+  gem 'opal', github: 'opal/opal', ref: ENV['OPAL_VERSION']
+end
+gem "opal-sprockets"
+gem "opal-browser"
 
-gem "rkelly-turbo", path: "../rkelly-turbo"
+gem "rkelly-turbo"
 
 gem "ruby-prof"
 gem "pry"

data/README.md

@@ -1,8 +1,11 @@
 # Opal::Optimizer
 
-Welcome to your new gem! In this directory, you'll find the files you need to be able to package up your Ruby library into a gem. Put your Ruby code in the file `lib/opal/optimizer`. To experiment with that code, run `bin/console` for an interactive prompt.
+A utility to optimize the JavaScript output from Opal with a help of RKelly-Turbo JavaScript parser.
 
-TODO: Delete this and the text above, and describe your gem
+As of now it contains two steps:
+
+* Tree shaking for Opal methods - if a named method is not called anywhere in the code, it's removed
+* Collapsing stubs
 
 ## Installation
 
@@ -22,7 +25,7 @@ Or install it yourself as:
 
 ## Usage
 
-TODO: Write usage instructions here
+For Sprockets environments, all you need is to `require "opal/optimizer/sprockets"` in your pipeline.
 
 ## Development
 
@@ -32,7 +35,7 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/opal-optimizer.
+Bug reports and pull requests are welcome on GitHub at https://github.com/hmdne/opal-optimizer.
 
 
 ## License

data/bin/opal-optimizer

@@ -1,6 +1,2 @@
 #!/usr/bin/env ruby
-
-require "bundler/setup"
-require "opal/optimizer"
-
-print Opal::Optimizer.new(ARGV[0] ? File.read(ARGV[0]) : STDIN.read).optimize
+load __dir__+"/../exe/opal-optimizer"

data/exe/opal-optimizer

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+
+require "bundler/setup"
+require "opal/optimizer"
+
+print Opal::Optimizer.new(ARGF.read).optimize

data/lib/opal/optimizer/step/collapse_stubs.rb

@@ -11,8 +11,12 @@ class CollapseStubs < Step
 
     nodes = corelib_calls["add_stubs"] || []
     nodes.each do |node|
-      stubs += node.arguments.value.first.value.map do |i|
-        i.value.value.gsub(/['"]/, '')
+      if opal_version >= 1.4
+        stubs += node.arguments.value.first.value[1..-2].split(',').map(&"$".method(:+))
+      else
+        stubs += node.arguments.value.first.value.map do |i|
+          i.value.value.gsub(/['"]/, '')
+        end
       end
 
       node.destroy! "CollapseStubs"
@@ -25,12 +29,13 @@ class CollapseStubs < Step
 
     stubs -= ["$respond_to_missing?"] if opal_version >= 1.1
 
-    new_stub_code = <<~end
-      var stubs = '#{stubs.to_a.join('#')}'.split('#'), stubs_obj = {};
+
+    new_stub_code = <<~JAVASCRIPT
+      var stubs = '#{stubs.to_a.join(',')}'.split(','), stubs_obj = {};
       for (var i = 0; i < stubs.length; i++)
         stubs_obj[stubs[i]] = {value: Opal.stub_for(stubs[i]), enumerable: false, configurable: true, writable: true};
       Object.defineProperties(Opal.BasicObject.$$prototype, stubs_obj);
-    end
+    JAVASCRIPT
 
     new_stub_code = parse_js(new_stub_code)
 

data/lib/opal/optimizer/step/tree_shaking.rb

@@ -12,9 +12,10 @@ class TreeShaking < Step
     method_defs = corelib_calls["def"] +
                   corelib_calls["defs"] +
                   corelib_calls["defn"] +
+                  corelib_calls["udef"] +
                   aliases.keys
 
-    method_calls = Set.new
+    method_calls = Set.new(['$method_missing'])
 
     method_calls += ( function_calls + exports.function_calls ).map do |i|
       out = if i.value_path?(DotAccessorNode) && i.value.accessor.start_with?("$")
@@ -23,8 +24,12 @@ class TreeShaking < Step
             StringNode === i.value.accessor &&
             i.value.accessor.value[1] == '$'
         i.value.accessor.value[1..-2]
-      elsif i.value_path?(ResolveNode, "$send")
-        old = i.arguments.value[1]
+      elsif i.value_path?(ResolveNode, ->(i) { %w[$send $send2 $refined_send].include? i })
+        if i.value.value == '$send'
+          old = i.arguments.value[1]
+        else
+          old = i.arguments.value[2]
+        end
         "$" + old.value[1..-2] if StringNode === old
       end
 
@@ -44,16 +49,30 @@ class TreeShaking < Step
     method_calls += aliases.values
 
     # Protected methods
-    method_calls += ["$register", "$negative?"]
+    method_calls += []
 
     removed = Set.new
 
     method_defs.each do |m|
       name = m.arguments.value[1]
-      next unless StringNode === name
-
-      name = name.value[1..-2]
-      name = "$" + name if m.value.accessor == "alias"
+      case name
+      when AddNode
+        if StringNode === name.left && StringNode === name.value
+          name = name.left.value[1..-2] + name.value.value[1..-2]
+        else
+          next
+        end
+      when StringNode
+        name = name.value[1..-2]
+        case m.value
+        when ResolveNode
+          name = "$" + name if m.value.value == "$alias"
+        when DotAccessorNode
+          name = "$" + name if m.value.accessor == "alias"
+        end
+      else
+        next
+      end
 
       next if method_calls.include? name
 
@@ -62,8 +81,17 @@ class TreeShaking < Step
     end
 
     corelib_calls["add_stubs"].each do |stubcall|
-      stubcall.arguments.value.first.value.reject! do |i|
-        removed.include? i.value.value[1..-2]
+      if opal_version >= 1.4
+        stubs = stubcall.arguments.value.first.value
+        new_stubs = stubs[1..-2].split(",").reject do |i|
+          removed.include? i[1..-1]
+        end.join(',')
+        new_stubs = "'#{new_stubs}'"
+        stubs.replace(new_stubs)
+      else
+        stubcall.arguments.value.first.value.reject! do |i|
+          removed.include? i.value.value[1..-2]
+        end
       end
     end
 
@@ -73,7 +101,7 @@ class TreeShaking < Step
   def run
     loop do
       removed = shake_methods
-      #$stdout.puts removed.inspect
+      # $stdout.puts removed.inspect
       reload
       break if removed.length == 0
     end

data/lib/opal/optimizer/version.rb

@@ -1,5 +1,5 @@
 module Opal
   class Optimizer
-    VERSION = "0.1.2"
+    VERSION = "0.1.6"
   end
 end

data/lib/opal/optimizer.rb

@@ -26,7 +26,11 @@ module Opal
         elsif es.start_with?("(function(global_object) {\n  \"use strict\";\n\n  // @note\n  "\
                              "//   A few conventions for the documentation of this file:") &&
               es.end_with?("TypeError.$$super = Error;\n}).call(this);")
-          @opal_version = 1.1
+          if es.include? 'function $prop(object,'
+            @opal_version = 1.4
+          else
+            @opal_version = 1.1
+          end
         end
       end
 
@@ -46,8 +50,9 @@ module Opal
     def reload
       @function_calls = ast.pointcut(FunctionCallNode).matches
       @corelib_calls = @function_calls.select do |i|
-        i.value_path?(DotAccessorNode, ResolveNode, "Opal")
-      end.group_by { |i| i.value.accessor }
+        i.value_path?(DotAccessorNode, ResolveNode, "Opal") ||
+        i.value_path?(ResolveNode, /\A\$/)
+      end.group_by { |i| (i.value.value.is_a?(String) ? i.value.value : i.value.accessor).gsub('$', '') }
       @corelib_calls = Hash.new { [] }.merge(@corelib_calls)
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opal-optimizer
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.6
 platform: ruby
 authors:
 - hmdne
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-05-26 00:00:00.000000000 Z
+date: 2021-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rkelly-turbo
@@ -40,21 +40,24 @@ dependencies:
         version: 1.0.0
 description:
 email: []
-executables: []
+executables:
+- opal-optimizer
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/build.yml"
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
+- CHANGELOG.md
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile
 - bin/console
 - bin/opal-optimizer
 - bin/setup
+- exe/opal-optimizer
 - lib/opal/optimizer.rb
 - lib/opal/optimizer/helpers.rb
 - lib/opal/optimizer/sprockets.rb
@@ -85,7 +88,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: Optimize Opal's resulting javascript code

data/Gemfile.lock

@@ -1,85 +0,0 @@
-PATH
-  remote: ../opal-browser
-  specs:
-    opal-browser (0.2.0)
-      opal (>= 1.0, < 2.0)
-      paggio
-
-PATH
-  remote: ../opal-sprockets
-  specs:
-    opal-sprockets (1.0.0)
-      opal (>= 1.0, < 1.2)
-      sprockets (~> 4.0)
-      tilt (>= 1.4)
-
-PATH
-  remote: ../opal
-  specs:
-    opal (1.1.1)
-      ast (>= 2.3.0)
-      parser (~> 3.0)
-
-PATH
-  remote: ../rkelly-turbo
-  specs:
-    rkelly-turbo (0.1.1.20210526224136)
-
-PATH
-  remote: .
-  specs:
-    opal-optimizer (0.1.2)
-      opal (>= 1.0.0)
-      rkelly-turbo
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    ast (2.4.2)
-    coderay (1.1.3)
-    concurrent-ruby (1.1.8)
-    diff-lcs (1.4.4)
-    method_source (1.0.0)
-    paggio (0.2.6)
-    parser (3.0.1.1)
-      ast (~> 2.4.1)
-    pry (0.13.1)
-      coderay (~> 1.1)
-      method_source (~> 1.0)
-    rack (2.2.3)
-    rake (12.3.3)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.2)
-      rspec-support (~> 3.9.3)
-    rspec-expectations (3.9.2)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.3)
-    ruby-prof (1.4.3)
-    sprockets (4.0.2)
-      concurrent-ruby (~> 1.0)
-      rack (> 1, < 3)
-    tilt (2.0.10)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  opal!
-  opal-browser!
-  opal-optimizer!
-  opal-sprockets!
-  pry
-  rake (~> 12.0)
-  rkelly-turbo!
-  rspec (~> 3.0)
-  ruby-prof
-
-BUNDLED WITH
-   2.1.4