synvert-core 0.27.4 → 0.31.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a239badccf8d5b2774e75423755123fe55393f951cb3064030800ce351e014bf
-  data.tar.gz: df9c575e61f833873bafaa672b95977390ce5eef62ace7898f209f684fef595d
+  metadata.gz: bf20c3a501940025291f9e0ae6f306d1613fb4db8ee5d5cf649d0ad7da4d1c5a
+  data.tar.gz: 425f42f736ec76ca6ade1f881ab252bfa6659f0e2cc8f3b55dd31153a4d8853a
 SHA512:
-  metadata.gz: fc153d13c26d917155db767031b5371df250014e1c86cfbf58e89693775e5488a02276e272a62e6c41489a744056846ff2237f74433c283dbfc5b9664c989372
-  data.tar.gz: 4c6e8df70fc14379a871b1b187ef9ce4b3b32ec964aedf096d6db1f69ae911e005933576b1adbc01478a61b1c694a1d7002bd1d4f0647f1e421bcbe894f191f5
+  metadata.gz: 52c43ad9b8986c68a4c77200e84b9cf7497fec7c46f0beb74fa695ed13d986f38519c167a3242d870dca79a0e09844fe6b564d6a54c258801801d4920be2f9da
+  data.tar.gz: 20aeb9c56536f335d8181e307bcf96ad8b3cbc7c02471077657a1ec0fa23db6c6f10269a4477aef3007ffc0b1dd07f3bb522289ecfed2c5f8f581abd49b3ee8b

data/CHANGELOG.md

@@ -1,20 +1,23 @@
 # CHANGELOG
 
-## 0.27.4 (2021-04-01)
+## 0.31.0 (2021-04-27)
 
-* Support `:block` in `child_node_range`
+* Add `in` and `not_in` rules
 
-## 0.27.3 (2021-03-31)
+## 0.30.0 (2021-04-26)
 
-* Support `:def` in `child_node_range`
-* Support `:defs` in `child_node_range`
+* `goto_node` accepts multiple child node names
+* Match any_value
 
-## 0.27.2 (2021-03-31)
+## 0.29.0 (2021-04-25)
 
-* Fix selector with operator in `child_node_range`
+* Make `child_name_range` support [:block, :pipe]
+* Get key value for hash node
 
-## 0.27.1 (2021-03-31)
+## 0.28.0 (2021-04-07)
 
+* Make `child_name_range` support all dsl nodes
+* Make `replace` action support multi child names
 * Fix `delete` action arguments
 
 ## 0.27.0 (2021-03-31)

data/lib/synvert/core/node_ext.rb

@@ -101,9 +101,9 @@ module Parser::AST
     def arguments
       case type
       when :def, :block
-        ArgumentsNode.new children[1]
+        ArgumentsNode.new(children[1])
       when :defs
-        ArgumentsNode.new children[2]
+        ArgumentsNode.new(children[2])
       when :send
         children[2..-1]
       when :defined?
@@ -203,7 +203,7 @@ module Parser::AST
     def hash_value(key)
       if :hash == type
         value_node = children.find { |pair_node| pair_node.key.to_value == key }
-        value_node ? value_node.value : nil
+        value_node&.value
       else
         raise Synvert::Core::MethodNotSupported, "hash_value is not handled for #{debug_info}"
       end
@@ -280,6 +280,31 @@ module Parser::AST
       end
     end
 
+    # Respond key value for hash node, e.g.
+    #
+    # Current node is s(:hash, s(:pair, s(:sym, :number), s(:int, 10)))
+    # node.number_value is 10
+    def method_missing(method_name, *args, &block)
+      if :hash == type && method_name.to_s.include?('_value')
+        key = method_name.to_s.sub('_value', '')
+        return hash_value(key.to_sym)&.to_value if key?(key.to_sym)
+        return hash_value(key.to_s)&.to_value if key?(key.to_s)
+
+        return nil
+      end
+
+      super
+    end
+
+    def respond_to_missing?(method_name, *args)
+      if :hash == type && method_name.to_s.include?('_value')
+        key = method_name.to_s.sub('_value', '')
+        return true if key?(key.to_sym) || key?(key.to_s)
+      end
+
+      super
+    end
+
     def to_s
       if :mlhs == type
         "(#{children.map(&:name).join(', ')})"
@@ -322,52 +347,56 @@ module Parser::AST
     # @param [String] name of child node.
     # @return [Parser::Source::Range] source range of child node.
     def child_node_range(child_name)
-      case type
-      when :block
-        case child_name
-        when :caller
-          caller&.loc&.expression
-        when :arguments
-          arguments&.loc&.expression
-        end
-      when :class
-        case child_name
-        when :name
-          loc.name
-        when :parent_class
-          parent_class&.loc&.expression
-        end
-      when :def
-        case child_name
-        when :name
-          loc.name
-        when :arguments
-          Parser::Source::Range.new('(string)', arguments.first.loc.expression.begin_pos, arguments.last.loc.expression.end_pos) unless arguments.empty?
-        end
-      when :defs
-        case child_name
-        when :dot
-          loc.operator
-        when :self
-          Parser::Source::Range.new('(string)', loc.operator.begin_pos - 4, loc.operator.begin_pos)
-        when :name
-          loc.name
-        when :arguments
-          Parser::Source::Range.new('(string)', arguments.first.loc.expression.begin_pos, arguments.last.loc.expression.end_pos) unless arguments.empty?
-        end
-      when :send
-        case child_name
-        when :receiver
-          receiver&.loc&.expression
-        when :dot
-          loc.dot
-        when :message
-          loc.operator ? Parser::Source::Range.new('(string)', loc.selector.begin_pos, loc.operator.end_pos) : loc.selector
-        when :arguments
-          Parser::Source::Range.new('(string)', arguments.first.loc.expression.begin_pos, arguments.last.loc.expression.end_pos) unless arguments.empty?
+      case [type, child_name]
+      when %i[block pipe]
+        Parser::Source::Range.new('(string)', arguments.loc.expression.begin_pos, arguments.loc.expression.end_pos)
+      when %i[class name]
+        loc.name
+      when %i[def name]
+        loc.name
+      when %i[defs name]
+        loc.name
+      when %i[defs dot]
+        loc.operator
+      when %i[defs self]
+        Parser::Source::Range.new('(string)', loc.operator.begin_pos - 4, loc.operator.begin_pos)
+      when %i[send dot]
+        loc.dot
+      when %i[send message]
+        if loc.operator
+          Parser::Source::Range.new('(string)', loc.selector.begin_pos, loc.operator.end_pos)
+        else
+          loc.selector
         end
       else
-        raise Synvert::Core::MethodNotSupported, "child_node_range is not handled for #{evaluated.inspect}, child_name: #{child_name}"
+        if respond_to?(child_name)
+          child_node = send(child_name)
+          return nil if child_node.nil?
+
+          if child_node.is_a?(Parser::AST::Node)
+            return(
+              Parser::Source::Range.new(
+                '(string)',
+                child_node.loc.expression.begin_pos,
+                child_node.loc.expression.end_pos
+              )
+            )
+          end
+
+          # arguments
+          return nil if child_node.empty?
+
+          return(
+            Parser::Source::Range.new(
+              '(string)',
+              child_node.first.loc.expression.begin_pos,
+              child_node.last.loc.expression.end_pos
+            )
+          )
+        end
+
+        raise Synvert::Core::MethodNotSupported,
+              "child_node_range is not handled for #{evaluated.inspect}, child_name: #{child_name}"
       end
     end
 
@@ -399,6 +428,14 @@ module Parser::AST
           actual = actual_value(self, multi_keys[0...-1])
           expected = expected_value(rules, multi_keys)
           !match_value?(actual, expected)
+        when :in
+          actual = actual_value(self, multi_keys[0...-1])
+          expected_values = expected_value(rules, multi_keys)
+          expected_values.any? { |expected| match_value?(actual, expected) }
+        when :not_in
+          actual = actual_value(self, multi_keys[0...-1])
+          expected_values = expected_value(rules, multi_keys)
+          expected_values.all? { |expected| !match_value?(actual, expected) }
         else
           actual = actual_value(self, multi_keys)
           expected = expected_value(rules, multi_keys)
@@ -438,7 +475,7 @@ module Parser::AST
                 source
               end
             end
-          when String, Symbol
+          when String, Symbol, Integer, Float
             evaluated
           when NilClass
             'nil'
@@ -498,6 +535,8 @@ module Parser::AST
         :false == actual.type
       when Parser::AST::Node
         actual == expected
+      when Synvert::Core::Rewriter::AnyValue
+        !actual.nil?
       else
         raise Synvert::Core::MethodNotSupported, "#{expected.class} is not handled for match_value?"
       end

data/lib/synvert/core/rewriter.rb

@@ -44,6 +44,8 @@ module Synvert::Core
     autoload :RubyVersion, 'synvert/core/rewriter/ruby_version'
     autoload :GemSpec, 'synvert/core/rewriter/gem_spec'
 
+    autoload :AnyValue, 'synvert/core/rewriter/any_value'
+
     class << self
       # Execute the temporary rewriter without group and name.
       #

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

@@ -3,9 +3,9 @@
 module Synvert::Core
   # ReplaceAction to replace child node with code.
   class Rewriter::ReplaceAction < Rewriter::Action
-    def initialize(instance, selector, with:)
+    def initialize(instance, *selectors, with:)
       @instance = instance
-      @selector = selector
+      @selectors = selectors
       @code = with
       @node = @instance.current_node
     end
@@ -14,14 +14,14 @@ module Synvert::Core
     #
     # @return [Integer] begin position.
     def begin_pos
-      @node.child_node_range(@selector).begin_pos
+      @selectors.map { |selector| @node.child_node_range(selector).begin_pos }.min
     end
 
     # End position of code to replace.
     #
     # @return [Integer] end position.
     def end_pos
-      @node.child_node_range(@selector).end_pos
+      @selectors.map { |selector| @node.child_node_range(selector).end_pos }.max
     end
 
     # The rewritten source code.

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

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module Synvert::Core
+  class Rewriter::AnyValue
+  end
+end

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

@@ -181,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_name [String] the name of the child node.
+    # @param *child_node_names [Array] the name of the child nodes.
     # @param block [Block] block code to continue operating on the matching nodes.
-    def goto_node(child_node_name, &block)
-      Rewriter::GotoScope.new(self, child_node_name, &block).process
+    def goto_node(*child_node_names, &block)
+      Rewriter::GotoScope.new(self, *child_node_names, &block).process
     end
 
     # Parse if_exist_node dsl, it creates a [Synvert::Core::Rewriter::IfExistCondition] to check
@@ -252,12 +252,12 @@ module Synvert::Core
     end
 
     # Parse replace with dsl, it creates a [Synvert::Core::Rewriter::ReplaceAction] to
-    # replace child node with code.
+    # replace child nodes with code.
     #
-    # @param selector [Symbol] selector name of child node.
+    # @param selectors [Array<Symbol>] selector names of child node.
     # @param with [String] code need to be replaced with.
-    def replace(selector, with:)
-      @actions << Rewriter::ReplaceAction.new(self, selector, with: with)
+    def replace(*selectors, with:)
+      @actions << Rewriter::ReplaceAction.new(self, *selectors, with: with)
     end
 
     # Parse replace_erb_stmt_with_expr dsl, it creates a [Synvert::Core::Rewriter::ReplaceErbStmtWithExprAction] to
@@ -285,6 +285,11 @@ module Synvert::Core
       @rewriter.add_warning Rewriter::Warning.new(self, message)
     end
 
+    # Any value but nil.
+    def any_value
+      Rewriter::AnyValue.new
+    end
+
     private
 
     # It changes source code from bottom to top, and it can change source code twice at the same time,

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_name [String]
+    # @param *child_node_names [Array]
     # @param block [Block]
-    def initialize(instance, child_node_name, &block)
+    def initialize(instance, *child_node_names, &block)
       @instance = instance
-      @child_node_name = child_node_name
+      @child_node_names = child_node_names
       @block = block
     end
 
@@ -19,7 +19,10 @@ module Synvert::Core
       current_node = @instance.current_node
       return unless current_node
 
-      child_node = @child_node_name.is_a?(Parser::AST::Node) ? @child_node_name : current_node.send(@child_node_name)
+      child_node = current_node
+      @child_node_names.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
         @instance.instance_eval(&@block)
       end

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.27.4'
+    VERSION = '0.31.0'
   end
 end

data/spec/synvert/core/node_ext_spec.rb

@@ -333,6 +333,18 @@ describe Parser::AST::Node do
     end
   end
 
+  describe 'key value by method_missing' do
+    it 'gets for key value' do
+      node = parse('{:foo => :bar}')
+      expect(node.foo_value).to eq :bar
+
+      node = parse("{'foo' => 'bar'}")
+      expect(node.foo_value).to eq 'bar'
+
+      expect(node.bar_value).to be_nil
+    end
+  end
+
   describe '#recursive_children' do
     it 'iterates all children recursively' do
       node = parse('class Synvert; def current_node; @node; end; end')
@@ -392,6 +404,18 @@ describe Parser::AST::Node do
       node = parse(source)
       expect(node).not_to be_match(type: 'class', name: { not: 'Synvert' })
     end
+
+    it 'matches in' do
+      source = 'FactoryBot.create(:user)'
+      node = parse(source)
+      expect(node).to be_match(type: 'send', message: { in: [:create, :build] })
+    end
+
+    it 'matches not_in' do
+      source = 'FactoryBot.create(:user)'
+      node = parse(source)
+      expect(node).not_to be_match(type: 'send', message: { not_in: [:create, :build] })
+    end
   end
 
   describe '#child_node_range' do
@@ -405,6 +429,12 @@ describe Parser::AST::Node do
       it 'checks arguments' do
         node = parse('Factory.define :user do |user|; end')
         range = node.child_node_range(:arguments)
+        expect(range.to_range).to eq(25...29)
+      end
+
+      it 'checks pipe' do
+        node = parse('Factory.define :user do |user|; end')
+        range = node.child_node_range(:pipe)
         expect(range.to_range).to eq(24...30)
       end
     end

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

@@ -5,18 +5,18 @@ require 'spec_helper'
 module Synvert::Core
   describe Rewriter::DeleteAction do
     subject {
-      source = "arr.map {}.flatten"
+      source = 'arr.map {}.flatten'
       node = Parser::CurrentRuby.parse(source)
       instance = double(current_node: node)
       Rewriter::DeleteAction.new(instance, :dot, :message)
     }
 
     it 'gets begin_pos' do
-      expect(subject.begin_pos).to eq "arr.map {}".length
+      expect(subject.begin_pos).to eq 'arr.map {}'.length
     end
 
     it 'gets end_pos' do
-      expect(subject.end_pos).to eq "arr.map {}.flatten".length
+      expect(subject.end_pos).to eq 'arr.map {}.flatten'.length
     end
 
     it 'gets rewritten_code' do

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

@@ -6,22 +6,22 @@ module Synvert::Core
   describe Rewriter::ReplaceAction do
     context 'replace with single line' do
       subject {
-        source = "'slug from title'.gsub(' ', '_')"
+        source = 'FactoryBot.create(:user)'
         node = Parser::CurrentRuby.parse(source)
         instance = double(current_node: node)
-        Rewriter::ReplaceAction.new(instance, :message, with: 'tr')
+        Rewriter::ReplaceAction.new(instance, :receiver, :dot, :message, with: 'create')
       }
 
       it 'gets begin_pos' do
-        expect(subject.begin_pos).to eq "'slug from title'.".length
+        expect(subject.begin_pos).to eq 0
       end
 
       it 'gets end_pos' do
-        expect(subject.end_pos).to eq "'slug from title'.gsub".length
+        expect(subject.end_pos).to eq 'FactoryBot.create'.length
       end
 
       it 'gets rewritten_code' do
-        expect(subject.rewritten_code).to eq 'tr'
+        expect(subject.rewritten_code).to eq 'create'
       end
     end
   end

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, &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, &block)
+      instance.goto_node(:caller, :receiver, &block)
     end
 
     it 'parses if_exist_node' do

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

@@ -8,12 +8,11 @@ module Synvert::Core
       rewriter = Rewriter.new('foo', 'bar')
       Rewriter::Instance.new(rewriter, 'file pattern')
     }
-    let(:source) {
-      '
-Factory.define :user do |user|
-end
-    '
-    }
+    let(:source) { <<~EOS }
+      Factory.define :user do |user|
+      end
+    EOS
+
     let(:node) { Parser::CurrentRuby.parse(source) }
     before do
       Rewriter::Instance.reset
@@ -25,13 +24,13 @@ end
         run = false
         type_in_scope = nil
         scope =
-          Rewriter::GotoScope.new instance, :caller do
+          Rewriter::GotoScope.new instance, :caller, :receiver do
             run = true
             type_in_scope = node.type
           end
         scope.process
         expect(run).to be_truthy
-        expect(type_in_scope).to eq :send
+        expect(type_in_scope).to eq :const
         expect(instance.current_node.type).to eq :block
       end
     end

data/synvert-core.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency "activesupport"
   spec.add_runtime_dependency "erubis"
-  spec.add_runtime_dependency "parser", "~> 3.0.0"
+  spec.add_runtime_dependency "parser"
 
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "guard"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.27.4
+  version: 0.31.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-01 00:00:00.000000000 Z
+date: 2021-04-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -42,16 +42,16 @@ dependencies:
   name: parser
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -154,6 +154,7 @@ files:
 - lib/synvert/core/rewriter/action/replace_action.rb
 - lib/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action.rb
 - lib/synvert/core/rewriter/action/replace_with_action.rb
+- lib/synvert/core/rewriter/any_value.rb
 - lib/synvert/core/rewriter/condition.rb
 - lib/synvert/core/rewriter/condition/if_exist_condition.rb
 - lib/synvert/core/rewriter/condition/if_only_exist_condition.rb
@@ -213,7 +214,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: convert ruby code to better syntax.