synvert-core 0.21.0 → 0.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bdaf84db78fb98d9d71b00025af06c099f821fc8a178002ecbc5b5e3ff68c740
-  data.tar.gz: 46ddc90eea269bb83475296081acfd0f73ed13ee94d7523bdd0e5e15434c760e
+  metadata.gz: b07d8def5ce35c251b7d1f7b04388654aadce0b6498c969b81457c9867d37dfb
+  data.tar.gz: d43a62b62ba6a4fa6bd51a97f7dfd6a4a805651d8ae6a595eac4724bba67f792
 SHA512:
-  metadata.gz: c84578b52bf4bf4f60136f764b0df276c97c4ee32be3595fd55b1303a823f282fc35f6dfee1a68166b85b0b592a2f239b22b796b0f10af861ea62bdb11e29641
-  data.tar.gz: dd47cdf203979bbd4b1cd8e164cffe6561894d25a3edcd98527983bbf46d7089f92e04d01db28893a8ac6fa1182ab846210fee4137b8b18fbbdf746f3fa0e144
+  metadata.gz: 98a9a81c409ca86ad13a58d00ef380ced1bc0a0fd3288c970e13f9a04ed4f13180b2b3e20ed87fc3521ee61531709507e7ce412ab7802c5d1091ef39fc63dca3
+  data.tar.gz: 05ecb25382bca6d4cc0e677b1ea3ccf19a0113bf2376fc1888135c1f069c4a4d7b2196f648b0433133761646d6dae8de4a43f43e28b506062ad97c236c9c38f3

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # CHANGELOG
 
+## 0.24.0 (2021-03-17)
+
+* Rename helper method `has_key?` to `key?`
+
+## 0.23.0 (2021-03-14)
+
+* Accept a node as goto_node argument
+
+## 0.22.0 (2021-03-13)
+
+* Track `affected_files` for rewriter
+* Fix `find_matching_nodes` for `current_node`
+
 ## 0.21.0 (2021-02-25)
 
 * Set `env['BUNDLE_GEMFILE']` before parsing `Gemfile.lock`

data/lib/synvert/core.rb

@@ -7,6 +7,7 @@ require 'parser/current'
 require 'ast'
 require 'active_support/core_ext/object'
 require 'erubis'
+require 'set'
 require 'synvert/core/node_ext'
 
 module Synvert

data/lib/synvert/core/node_ext.rb

@@ -187,11 +187,11 @@ module Parser::AST
     # @param [Object] key value.
     # @return [Boolean] true if specified key exists.
     # @raise [Synvert::Core::MethodNotSupported] if calls on other node.
-    def has_key?(key)
+    def key?(key)
       if :hash == type
         children.any? { |pair_node| pair_node.key.to_value == key }
       else
-        raise Synvert::Core::MethodNotSupported, "has_key? is not handled for #{debug_info}"
+        raise Synvert::Core::MethodNotSupported, "key? is not handled for #{debug_info}"
       end
     end
 
@@ -205,7 +205,7 @@ module Parser::AST
         value_node = children.find { |pair_node| pair_node.key.to_value == key }
         value_node ? value_node.value : nil
       else
-        raise Synvert::Core::MethodNotSupported, "has_key? is not handled for #{debug_info}"
+        raise Synvert::Core::MethodNotSupported, "has_value is not handled for #{debug_info}"
       end
     end
 
@@ -335,24 +335,22 @@ module Parser::AST
     # @param rules [Hash] rules to match.
     # @return true if matches.
     def match?(rules)
-      flat_hash(rules)
-        .keys
-        .all? do |multi_keys|
-          case multi_keys.last
-          when :any
-            actual_values = actual_value(self, multi_keys[0...-1])
-            expected = expected_value(rules, multi_keys)
-            actual_values.any? { |actual| match_value?(actual, expected) }
-          when :not
-            actual = actual_value(self, multi_keys[0...-1])
-            expected = expected_value(rules, multi_keys)
-            !match_value?(actual, expected)
-          else
-            actual = actual_value(self, multi_keys)
-            expected = expected_value(rules, multi_keys)
-            match_value?(actual, expected)
-          end
+      flat_hash(rules).keys.all? do |multi_keys|
+        case multi_keys.last
+        when :any
+          actual_values = actual_value(self, multi_keys[0...-1])
+          expected = expected_value(rules, multi_keys)
+          actual_values.any? { |actual| match_value?(actual, expected) }
+        when :not
+          actual = actual_value(self, multi_keys[0...-1])
+          expected = expected_value(rules, multi_keys)
+          !match_value?(actual, expected)
+        else
+          actual = actual_value(self, multi_keys)
+          expected = expected_value(rules, multi_keys)
+          match_value?(actual, expected)
         end
+      end
     end
 
     # Get rewritten source code.

data/lib/synvert/core/rewriter.rb

@@ -145,7 +145,9 @@ module Synvert::Core
     #   @return [Array] helper methods.
     # @!attribute [r] warnings
     #   @return [Array<Synvert::Core::Rewriter::Warning>] warning messages.
-    attr_reader :group, :name, :sub_snippets, :helpers, :warnings
+    # @!attribute [r] affected_files
+    #   @return [Set] affected fileds
+    attr_reader :group, :name, :sub_snippets, :helpers, :warnings, :affected_files
 
     # Initialize a rewriter.
     # When a rewriter is initialized, it is also registered.
@@ -161,6 +163,7 @@ module Synvert::Core
       @helpers = []
       @sub_snippets = []
       @warnings = []
+      @affected_files = Set.new
       self.class.register(@group, @name, self)
     end
 
@@ -188,6 +191,13 @@ module Synvert::Core
       @warnings << warning
     end
 
+    # Add an affected file.
+    #
+    # @param file_path [String]
+    def add_affected_file(file_path)
+      @affected_files.add(file_path)
+    end
+
     #######
     # DSL #
     #######

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

@@ -23,11 +23,9 @@ module Synvert::Core
     def rewritten_code
       if rewritten_source.split("\n").length > 1
         new_code = []
-        rewritten_source
-          .split("\n")
-          .each_with_index { |line, index|
-            new_code << (index == 0 || !@options[:autoindent] ? line : indent(@node) + line)
-          }
+        rewritten_source.split("\n").each_with_index { |line, index|
+          new_code << (index == 0 || !@options[:autoindent] ? line : indent(@node) + line)
+        }
         new_code.join("\n")
       else
         rewritten_source

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

@@ -87,43 +87,41 @@ module Synvert::Core
     # and rewrite source code back to original file.
     def process
       file_pattern = File.join(Configuration.path, @file_pattern)
-      Dir
-        .glob(file_pattern)
-        .each do |file_path|
-          next if Configuration.skip_files.include? file_path
-
-          begin
-            conflict_actions = []
-            source = +self.class.file_source(file_path)
-            ast = self.class.file_ast(file_path)
-
-            @current_file = file_path
-
-            process_with_node ast do
-              begin
-                instance_eval(&@block)
-              rescue NoMethodError
-                puts @current_node.debug_info
-                raise
-              end
+      Dir.glob(file_pattern).each do |file_path|
+        next if Configuration.skip_files.include? file_path
+
+        begin
+          conflict_actions = []
+          source = +self.class.file_source(file_path)
+          ast = self.class.file_ast(file_path)
+
+          @current_file = file_path
+
+          process_with_node ast do
+            begin
+              instance_eval(&@block)
+            rescue NoMethodError
+              puts @current_node.debug_info
+              raise
             end
+          end
 
-            if @actions.length > 0
-              @actions.sort_by! { |action| action.send(@options[:sort_by]) }
-              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 = []
-
-              self.class.write_file(file_path, source)
+          if @actions.length > 0
+            @actions.sort_by! { |action| action.send(@options[:sort_by]) }
+            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
-          rescue Parser::SyntaxError
-            puts "[Warn] file #{file_path} was not parsed correctly."
-            # do nothing, iterate next file
-          end while !conflict_actions.empty?
-        end
+            @actions = []
+
+            update_file(file_path, source)
+          end
+        rescue Parser::SyntaxError
+          puts "[Warn] file #{file_path} was not parsed correctly."
+          # do nothing, iterate next file
+        end while !conflict_actions.empty?
+      end
     end
 
     # Gets current node, it allows to get current node in block code.
@@ -312,5 +310,14 @@ module Synvert::Core
         source
       end
     end
+
+    # It updates a file with new source code.
+    #
+    # @param file_path [String] the file path
+    # @param source [String] the new source code
+    def update_file(file_path, source)
+      self.class.write_file(file_path, source)
+      @rewriter.add_affected_file(file_path)
+    end
   end
 end

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

@@ -19,7 +19,7 @@ module Synvert::Core
       current_node = @instance.current_node
       return unless current_node
 
-      child_node = current_node.send @child_node_name
+      child_node = @child_node_name.is_a?(Parser::AST::Node) ? @child_node_name : current_node.send(@child_node_name)
       @instance.process_with_other_node child_node do
         @instance.instance_eval(&@block)
       end

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

@@ -41,6 +41,14 @@ module Synvert::Core
         current_node.recursive_children do |child_node|
           matching_nodes << child_node if child_node.match? @rules
         end
+      elsif current_node.is_a?(Parser::AST::Node)
+        if current_node.type == :begin
+          current_node.children.each do |child_node|
+            matching_nodes << child_node if child_node.match? @rules
+          end
+        elsif current_node.match? @rules
+          matching_nodes << current_node
+        end
       else
         current_node.each do |child_node|
           matching_nodes << child_node if child_node.match? @rules

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.21.0'
+    VERSION = '0.24.0'
   end
 end

data/spec/synvert/core/node_ext_spec.rb

@@ -186,15 +186,15 @@ describe Parser::AST::Node do
     end
   end
 
-  describe '#has_key?' do
+  describe '#key?' do
     it 'gets true if key exists' do
       node = parse('{:foo => :bar}')
-      expect(node.has_key?(:foo)).to be_truthy
+      expect(node.key?(:foo)).to be_truthy
     end
 
     it 'gets false if key does not exist' do
       node = parse('{:foo => :bar}')
-      expect(node.has_key?('foo')).to be_falsey
+      expect(node.key?('foo')).to be_falsey
     end
   end
 

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

@@ -243,6 +243,7 @@ end
         expect(File).to receive(:read).with('spec/models/post_spec.rb').and_return(output)
         instance.process
         instance.process
+        expect(rewriter.affected_files).to be_include('spec/models/post_spec.rb')
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.24.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-25 00:00:00.000000000 Z
+date: 2021-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport