synvert-core 0.31.0 → 0.32.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf20c3a501940025291f9e0ae6f306d1613fb4db8ee5d5cf649d0ad7da4d1c5a
-  data.tar.gz: 425f42f736ec76ca6ade1f881ab252bfa6659f0e2cc8f3b55dd31153a4d8853a
+  metadata.gz: eb6c6b0b57d35fa8d3f850685ccda616f56f19fe55fee0b766302574b2cc577e
+  data.tar.gz: fac77a466b7691c8b74c5a3bb4f2b00ef31be69dfdb0db93fb07593e8553702e
 SHA512:
-  metadata.gz: 52c43ad9b8986c68a4c77200e84b9cf7497fec7c46f0beb74fa695ed13d986f38519c167a3242d870dca79a0e09844fe6b564d6a54c258801801d4920be2f9da
-  data.tar.gz: 20aeb9c56536f335d8181e307bcf96ad8b3cbc7c02471077657a1ec0fa23db6c6f10269a4477aef3007ffc0b1dd07f3bb522289ecfed2c5f8f581abd49b3ee8b
+  metadata.gz: 6ce9c70f4892ed3e65cb4669e0e623b745d8edce13e06fbc758c33adce27870934b6e5ea78dcf0eca480c421b96b9afa6de666599094819fd6e5cb6f4ba3b056
+  data.tar.gz: '0800aedc7b404a04f4605e2586f123faed17152f7ebe81bd430f48834f760c8189efe7fb348f41d15f2285370bdd2d08c31badb6e99df064c5b50dd98801ea88'

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 0.32.0 (2021-05-07)
+
+* Remove `ArgumentsNode`
+
 ## 0.31.0 (2021-04-27)
 
 * Add `in` and `not_in` rules

data/lib/synvert/core/node_ext.rb

@@ -1,29 +1,6 @@
 # frozen_string_literal: true
 
 module Parser::AST
-  # ArgumentsNode allows to handle all args as one node or handle all args as an array.
-  class ArgumentsNode
-    # Initialize
-    #
-    # @param node [Parser::AST::Node] args node.
-    def initialize(node)
-      @node = node
-    end
-
-    # If args node responds method itself, call method on args node.
-    # If args children (array) responds method,  call method on args children.
-    # Otherwise raise method missing error.
-    def method_missing(meth, *args, &block)
-      if @node.respond_to?(meth)
-        @node.send meth, *args, &block
-      elsif @node.children.respond_to?(meth)
-        @node.children.send meth, *args, &block
-      else
-        super
-      end
-    end
-  end
-
   # Parser::AST::Node monkey patch.
   class Node
     # Get name node of :class, :module, :const, :mlhs, :def and :defs node.
@@ -101,9 +78,9 @@ module Parser::AST
     def arguments
       case type
       when :def, :block
-        ArgumentsNode.new(children[1])
+        children[1]
       when :defs
-        ArgumentsNode.new(children[2])
+        children[2]
       when :send
         children[2..-1]
       when :defined?
@@ -285,7 +262,9 @@ module Parser::AST
     # 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')
+      if :args == type && children.respond_to?(method_name)
+        return children.send(method_name, *args, &block)
+      elsif :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)
@@ -297,7 +276,9 @@ module Parser::AST
     end
 
     def respond_to_missing?(method_name, *args)
-      if :hash == type && method_name.to_s.include?('_value')
+      if :args == type && children.respond_to?(method_name)
+        return true
+      elsif :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
@@ -350,6 +331,8 @@ module Parser::AST
       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[block arguments], %i[def arguments], %i[defs arguments]
+        Parser::Source::Range.new('(string)', arguments.first.loc.expression.begin_pos, arguments.last.loc.expression.end_pos)
       when %i[class name]
         loc.name
       when %i[def name]
@@ -458,8 +441,12 @@ module Parser::AST
           evaluated = instance_eval old_code
           case evaluated
           when Parser::AST::Node
-            evaluated.loc.expression.source
-          when Array, ArgumentsNode
+            if evaluated.type == :args
+              evaluated.loc.expression.source[1...-1]
+            else
+              evaluated.loc.expression.source
+            end
+          when Array
             if evaluated.size > 0
               file_source = evaluated.first.loc.expression.source_buffer.source
               source = file_source[evaluated.first.loc.expression.begin_pos...evaluated.last.loc.expression.end_pos]

data/lib/synvert/core/version.rb

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

data/spec/synvert/core/node_ext_spec.rb

@@ -104,17 +104,17 @@ describe Parser::AST::Node do
   describe '#arguments' do
     it 'gets for def node' do
       node = parse('def test(foo, bar); foo + bar; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq %w[foo bar]
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for defs node' do
       node = parse('def self.test(foo, bar); foo + bar; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq %w[foo bar]
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for block node' do
       node = parse('RSpec.configure do |config|; end')
-      expect(node.arguments.map { |argument| argument.to_source }).to eq ['config']
+      expect(node.arguments.type).to eq :args
     end
 
     it 'gets for send node' do
@@ -408,13 +408,13 @@ describe Parser::AST::Node do
     it 'matches in' do
       source = 'FactoryBot.create(:user)'
       node = parse(source)
-      expect(node).to be_match(type: 'send', message: { in: [:create, :build] })
+      expect(node).to be_match(type: 'send', message: { in: %i[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] })
+      expect(node).not_to be_match(type: 'send', message: { not_in: %i[create build] })
     end
   end
 
@@ -566,7 +566,7 @@ describe Parser::AST::Node do
       expect(node.rewritten_source('{{name}}')).to eq 'Synvert'
     end
 
-    it 'rewrites for ArgumentsNode' do
+    it 'rewrites for arguments' do
       source = 'test { |a, b| }'
       node = parse(source)
       expect(node.rewritten_source('{{arguments}}')).to eq 'a, b'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.31.0
+  version: 0.32.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-27 00:00:00.000000000 Z
+date: 2021-05-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport