synvert-core 0.21.2 → 0.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2cb208eddfa8a9d304c434950ab7054f0836f52860bb6edaeebe0c421cf7647c
-  data.tar.gz: 3609ebb014a30b489761bdd27ce2a0236695636b4a51196eb7f7c122d59ff692
+  metadata.gz: 8c112197bf98b5adfe2b5826ea3301f06f66511c111906bfd0c72a8c54a10d33
+  data.tar.gz: adb56b1ae52dd1b87d84d2861b7433586c96ddc6718fafcd79852775f84b38c4
 SHA512:
-  metadata.gz: b503e1bfe14c3e489465f71eb3edba0e5cc6ebaab0a860b14c6766208468951832eb61dd070353313b2bba3f0d3001ade5c9683e7de799a8a380db06ae7997eb
-  data.tar.gz: 80923462e716f67f4f278627be2ab765534c302cd2d572f78ee0e03a6cb7c8569bdc8182ee0550f05e41d3184184dca378e9a262da1eb8cf386e5c5f5e5c76f3
+  metadata.gz: e2f79639b669fc7b054ba8e57df70925f7f9bfdf6e40144069d41848bc0d0b3df0d60e73fc078c8e78eb29718bc8f3a181ee7964c63a3dcadd9a8981ce31544b
+  data.tar.gz: 758e0896064a0c15dbe4e3d3d128bc3f42517c2f2802067cccab20f3b366165d7ef5f3203975e261903573f2989f82f3918b09db925a2d896b9bee5897299a58

data/CHANGELOG.md

@@ -1,12 +1,9 @@
 # CHANGELOG
 
-## 0.21.2 (2021-02-26)
+## 0.22.0 (2021-03-13)
 
-* Fix `find_matching_nodes` if `current_node` is a `:begin` node
-
-## 0.21.1 (2021-02-26)
-
-* Fix `find_matching_nodes` if `current_node` is a `Parser::AST::Node`
+* Track `affected_files` for rewriter
+* Fix `find_matching_nodes` for `current_node`
 
 ## 0.21.0 (2021-02-25)
 

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

@@ -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/within_scope.rb

@@ -46,8 +46,8 @@ module Synvert::Core
           current_node.children.each do |child_node|
             matching_nodes << child_node if child_node.match? @rules
           end
-        else
-          matching_nodes << current_node if current_node.match? @rules
+        elsif current_node.match? @rules
+          matching_nodes << current_node
         end
       else
         current_node.each do |child_node|

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.21.2'
+    VERSION = '0.22.0'
   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.2
+  version: 0.22.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-26 00:00:00.000000000 Z
+date: 2021-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -209,7 +209,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.0.3
 signing_key:
 specification_version: 4
 summary: convert ruby code to better syntax.