synvert-core 0.52.1 → 0.54.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0ab6b6fbee0d7b58203ccde9af31179caa3f6cd0b52bdceb8262683daa31e1e
-  data.tar.gz: 0670bf038c8e7074893e68eafc4a60e4a27c94d4287dda21a7ab6a03322ce044
+  metadata.gz: d76620f06a4691e9d494c0b8cdcca0906652f1bdf9e4c977085c9d649a272c3e
+  data.tar.gz: 7f6ef2fdc13be57264052f8a231fce3f33682f14234a13e520548190895ecb5c
 SHA512:
-  metadata.gz: ceae8c9049ae4488f4831594ba2527c970ba88a8479d0eb1477e832597de59f7ca2e9e45389eab4bfa609f8eb2de9498d54340163323cc1329278ad90de1db70
-  data.tar.gz: 5a74ad849852d919bc274726201a03f00aec7741ddd8a8a3abd4a38e3f1584a358a7ca65c2238d885686274d1cfe9157dd4b5b74ac971bb05efd0af411366ae0
+  metadata.gz: 592a9feb14d4f773c4e474af198c42e31418343fd84eab1851e41231c99ccc288efc968a7af082060546448069f6bd92058e81a621abd192f3f6fbedbfa84be7
+  data.tar.gz: a06e8ec70349dbf60026488700bdbd6acc2df2bcfe5cc795a208b6c30078cb18c2c78435f34c71593e9bc8623ab555d2c2caf97047a75435e820ca81b9121c37

data/CHANGELOG.md

@@ -1,8 +1,21 @@
 # CHANGELOG
 
-## 0.52.1 (2021-08-21)
+## 0.54.2 (2021-09-09)
+
+* Fix `nil` match
+
+## 0.54.1 (2021-09-08)
+
+* Rewrite `remove` action
+
+## 0.54.0 (2021-08-28)
+
+* Change `goto_scope` param from array to string
+
+## 0.53.0 (2021-08-22)
 
 * Fix nested child in Node#child_node_range
+* Rename synvert-core to synvert-core-ruby
 
 ## 0.52.0 (2021-08-21)
 

data/README.md

@@ -1,12 +1,11 @@
-# Synvert::Core
+# synvert-core-ruby
 
-<img src="https://xinminlabs.github.io/synvert/img/logo_96.png" alt="logo" width="32" height="32" />
+<img src="https://synvert.xinminlabs.com/img/logo_96.png" alt="logo" width="32" height="32" />
 
-![Main workflow](https://github.com/xinminlabs/synvert-core/actions/workflows/main.yml/badge.svg)
-[![Coverage Status](https://coveralls.io/repos/xinminlabs/synvert-core/badge.png?branch=master)](https://coveralls.io/r/xinminlabs/synvert-core)
+![Main workflow](https://github.com/xinminlabs/synvert-core-ruby/actions/workflows/main.yml/badge.svg)
 [![Gem Version](https://badge.fury.io/rb/synvert-core.png)](http://badge.fury.io/rb/synvert-core)
 
-synvert-core provides a dsl to convert ruby source code.
+synvert-core-ruby provides a dsl to convert ruby source code.
 
 ## Installation
 
@@ -31,11 +30,11 @@ Or install it yourself as:
 
 ## Contributing
 
-1. Fork it ( https://github.com/[my-github-username]/synvert-core/fork )
+1. Fork it ( https://github.com/[my-github-username]/synvert-core-ruby/fork )
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create a new Pull Request
 
-[1]: https://xinminlabs.github.io/synvert/
-[2]: https://rubydoc.info/github/xinminlabs/synvert-core/master/frames
+[1]: https://synvert.xinminlabs.com
+[2]: https://rubydoc.info/github/xinminlabs/synvert-core-ruby/master/frames

data/lib/synvert/core/node_ext.rb

@@ -576,7 +576,11 @@ module Parser::AST
 
         actual.zip(expected).all? { |a, e| match_value?(a, e) }
       when NilClass
-        actual.nil?
+        if actual.is_a?(Parser::AST::Node)
+          :nil == actual.type
+        else
+          actual.nil?
+        end
       when Numeric
         if actual.is_a?(Parser::AST::Node)
           actual.children[0] == expected

data/lib/synvert/core/rewriter/action/delete_action.rb

@@ -1,21 +1,21 @@
 # frozen_string_literal: true
 
 module Synvert::Core
-  # RemoveAction to delete code.
+  # DeleteAction to delete child nodes.
   class Rewriter::DeleteAction < Rewriter::Action
     def initialize(instance, *selectors)
       super(instance, nil)
       @selectors = selectors
     end
 
-    # Begin position of code to replace.
+    # Begin position of code to delete.
     #
     # @return [Integer] begin position.
     def begin_pos
       @selectors.map { |selector| @node.child_node_range(selector) }.compact.map(&:begin_pos).min
     end
 
-    # End position of code to replace.
+    # End position of code to delete.
     #
     # @return [Integer] end position.
     def end_pos

data/lib/synvert/core/rewriter/action/insert_action.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Synvert::Core
-  # AddAction to add code to the node.
+  # InsertAction to add code to the node.
   class Rewriter::InsertAction < Rewriter::Action
     def initialize(instance, code, at:)
       super(instance, code)

data/lib/synvert/core/rewriter/action/remove_action.rb

@@ -1,29 +1,55 @@
 # frozen_string_literal: true
 
 module Synvert::Core
-  # RemoveAction to remove code.
+  # RemoveAction to remove current node.
   class Rewriter::RemoveAction < Rewriter::Action
-    def initialize(instance, code = nil)
-      super
+    def initialize(instance)
+      super(instance, nil)
     end
 
     # Begin position of code to replace.
     #
     # @return [Integer] begin position.
     def begin_pos
-      @node.loc.expression.begin_pos
+      if take_whole_line?
+        start_index
+      else
+        @node.loc.expression.begin_pos
+      end
     end
 
     # End position of code to replace.
     #
     # @return [Integer] end position.
     def end_pos
-      @node.loc.expression.end_pos
+      if take_whole_line?
+        end_index
+      else
+        @node.loc.expression.end_pos
+      end
     end
 
     # The rewritten code, always empty string.
     def rewritten_code
       ''
     end
+
+    private
+
+    def take_whole_line?
+      @node.to_source == file_source[start_index...end_index].strip
+    end
+
+    def start_index
+      file_source[0..@node.loc.expression.begin_pos].rindex("\n") + "\n".length
+    end
+
+    def end_index
+      file_source[@node.loc.expression.end_pos..-1].index("\n") + @node.loc.expression.end_pos + "\n".length
+    end
+
+    def file_source
+      @file_source ||= @instance.file_source
+    end
   end
 end

data/lib/synvert/core/rewriter/instance.rb

@@ -64,6 +64,11 @@ module Synvert::Core
     #   @return current filename
     attr_accessor :current_node, :current_file
 
+    # Current file source
+    def file_source
+      self.class.file_source(current_file)
+    end
+
     # Initialize an instance.
     #
     # @param rewriter [Synvert::Core::Rewriter]
@@ -107,7 +112,6 @@ module Synvert::Core
             conflict_actions = get_conflict_actions
             @actions.reverse_each do |action|
               source[action.begin_pos...action.end_pos] = action.rewritten_code
-              source = remove_code_or_whole_line(source, action.line)
             end
             @actions = []
 
@@ -139,7 +143,7 @@ module Synvert::Core
 
     # Set current_node properly, process and set current_node back to original current_node.
     #
-    # @param node [Parser::AST::Node] node set to current_node
+    # @param node [Parser::AST::Node] node set to other_node
     # @yield process
     def process_with_other_node(node)
       original_node = current_node
@@ -177,10 +181,10 @@ module Synvert::Core
     # Parse goto_node dsl, it creates a [Synvert::Core::Rewriter::GotoScope] to go to a child node,
     # then continue operating on the child node.
     #
-    # @param *child_node_names [Array] the name of the child nodes.
+    # @param child_node_name [Symbol|String] the name of the child nodes.
     # @param block [Block] block code to continue operating on the matching nodes.
-    def goto_node(*child_node_names, &block)
-      Rewriter::GotoScope.new(self, *child_node_names, &block).process
+    def goto_node(child_node_name, &block)
+      Rewriter::GotoScope.new(self, child_node_name, &block).process
     end
 
     # Parse if_exist_node dsl, it creates a [Synvert::Core::Rewriter::IfExistCondition] to check
@@ -323,25 +327,6 @@ module Synvert::Core
       conflict_actions
     end
 
-    # It checks if code is removed and that line is empty.
-    #
-    # @param source [String] source code of file
-    # @param line [String] the line number
-    def remove_code_or_whole_line(source, line)
-      newline_at_end_of_line = source[-1] == "\n"
-      source_arr = source.split("\n")
-      if source_arr[line - 1] && source_arr[line - 1].strip.empty?
-        source_arr.delete_at(line - 1)
-        if source_arr[line - 2] && source_arr[line - 2].strip.empty? && source_arr[line - 1] &&
-             source_arr[line - 1].strip.empty?
-          source_arr.delete_at(line - 1)
-        end
-        source_arr.join("\n") + (newline_at_end_of_line ? "\n" : '')
-      else
-        source
-      end
-    end
-
     # It updates a file with new source code.
     #
     # @param file_path [String] the file path

data/lib/synvert/core/rewriter/scope/goto_scope.rb

@@ -6,11 +6,11 @@ module Synvert::Core
     # Initialize a scope
     #
     # @param instance [Synvert::Core::Rewriter::Instance]
-    # @param *child_node_names [Array]
+    # @param child_node_name [Symbol|string]
     # @param block [Block]
-    def initialize(instance, *child_node_names, &block)
+    def initialize(instance, child_node_name, &block)
       @instance = instance
-      @child_node_names = child_node_names
+      @child_node_name = child_node_name
       @block = block
     end
 
@@ -20,7 +20,7 @@ module Synvert::Core
       return unless current_node
 
       child_node = current_node
-      @child_node_names.each do |child_node_name|
+      @child_node_name.to_s.split('.').each do |child_node_name|
         child_node = child_node_name.is_a?(Parser::AST::Node) ? child_node_name : child_node.send(child_node_name)
       end
       @instance.process_with_other_node child_node do

data/lib/synvert/core/rewriter.rb

@@ -81,7 +81,7 @@ module Synvert::Core
       def fetch(group, name)
         group = group.to_s
         name = name.to_s
-        if exist? group, name
+        if rewriters[group] && rewriters[group][name]
           rewriters[group][name]
         else
           raise RewriterNotFound, "Rewriter #{group} #{name} not found"
@@ -96,34 +96,13 @@ module Synvert::Core
       # @return [Synvert::Core::Rewriter] the registered rewriter.
       # @raise [Synvert::Core::RewriterNotFound] if the registered rewriter is not found.
       def call(group, name, sandbox = false)
-        group = group.to_s
-        name = name.to_s
-        if exist? group, name
-          rewriter = rewriters[group][name]
-          if sandbox
-            rewriter.process_with_sandbox
-          else
-            rewriter.process
-          end
-          rewriter
-        else
-          raise RewriterNotFound, "Rewriter #{group}/#{name} not found"
-        end
-      end
-
-      # Check if one rewriter exist.
-      #
-      # @param group [String] the rewriter group.
-      # @param name [String] the rewriter name.
-      # @return [Boolean] true if the rewriter exist.
-      def exist?(group, name)
-        group = group.to_s
-        name = name.to_s
-        if rewriters[group] && rewriters[group][name]
-          true
+        rewriter = fetch(group, name)
+        if sandbox
+          rewriter.process_with_sandbox
         else
-          false
+          rewriter.process
         end
+        rewriter
       end
 
       # Get all available rewriters

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.52.1'
+    VERSION = '0.54.2'
   end
 end

data/spec/synvert/core/rewriter/action/remove_action_spec.rb

@@ -7,7 +7,7 @@ module Synvert::Core
     subject {
       source = "user = User.new params[:user]\nuser.save\nrender\n"
       send_node = Parser::CurrentRuby.parse(source).children[1]
-      instance = double(current_node: send_node)
+      instance = double(current_node: send_node, file_source: source)
       Rewriter::RemoveAction.new(instance)
     }
 
@@ -16,7 +16,7 @@ module Synvert::Core
     end
 
     it 'gets end_pos' do
-      expect(subject.end_pos).to eq "user = User.new params[:user]\nuser.save".length
+      expect(subject.end_pos).to eq "user = User.new params[:user]\nuser.save\n".length
     end
 
     it 'gets rewritten_code' do

data/spec/synvert/core/rewriter/instance_spec.rb

@@ -54,9 +54,9 @@ module Synvert::Core
     it 'parses goto_node' do
       scope = double
       block = proc {}
-      expect(Rewriter::GotoScope).to receive(:new).with(instance, :caller, :receiver, &block).and_return(scope)
+      expect(Rewriter::GotoScope).to receive(:new).with(instance, 'caller.receiver', &block).and_return(scope)
       expect(scope).to receive(:process)
-      instance.goto_node(:caller, :receiver, &block)
+      instance.goto_node('caller.receiver', &block)
     end
 
     it 'parses if_exist_node' do

data/spec/synvert/core/rewriter/scope/goto_scope_spec.rb

@@ -24,7 +24,7 @@ module Synvert::Core
         run = false
         type_in_scope = nil
         scope =
-          Rewriter::GotoScope.new instance, :caller, :receiver do
+          Rewriter::GotoScope.new instance, 'caller.receiver' do
             run = true
             type_in_scope = node.type
           end

data/spec/synvert/core/rewriter_spec.rb

@@ -249,17 +249,6 @@ module Synvert::Core
         expect { Rewriter.call 'group', 'rewriter' }.to raise_error(RewriterNotFound)
       end
 
-      context 'exist?' do
-        it 'returns true if rewriter exists' do
-          Rewriter.new 'group', 'rewriter'
-          expect(Rewriter.exist?('group', 'rewriter')).to be_truthy
-        end
-
-        it 'returns false if rewriter does not exist' do
-          expect(Rewriter.exist?('group', 'rewriter')).to be_falsey
-        end
-      end
-
       context 'available' do
         it 'lists empty rewriters' do
           expect(Rewriter.availables).to eq({})

data/{synvert-core.gemspec → synvert-core-ruby.gemspec}

@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
   spec.email         = ["flyerhzm@gmail.com"]
   spec.summary       = 'convert ruby code to better syntax.'
   spec.description   = 'convert ruby code to better syntax automatically.'
-  spec.homepage      = "https://github.com/xinminlabs/synvert-core"
+  spec.homepage      = "https://github.com/xinminlabs/synvert-core-ruby"
   spec.license       = "MIT"
 
   spec.files         = `git ls-files -z`.split("\x0")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.52.1
+  version: 0.54.2
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-21 00:00:00.000000000 Z
+date: 2021-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -198,8 +198,8 @@ files:
 - spec/synvert/core/rewriter/scope_spec.rb
 - spec/synvert/core/rewriter/warning_spec.rb
 - spec/synvert/core/rewriter_spec.rb
-- synvert-core.gemspec
-homepage: https://github.com/xinminlabs/synvert-core
+- synvert-core-ruby.gemspec
+homepage: https://github.com/xinminlabs/synvert-core-ruby
 licenses:
 - MIT
 metadata: {}