synvert-core 0.21.1 → 0.25.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 945aecfd794891da0a5fc8d5a1ac1f03946090c7bb01ff458545292a91296160
-  data.tar.gz: d0e863e0fdeda70cba509a84a44a3665de4edabb178bdb38aa3649095d0b4c99
+  metadata.gz: 8365e77c33d2effc81fe2cd19d522785282db9c06aa8e399c6469f9cbdc32cc7
+  data.tar.gz: c60c8a032106b681abdac98b31a87cfbac6649959eb21fe1bbd0e6be1486c53c
 SHA512:
-  metadata.gz: b328ae8b13f6c056d18f59fdd643bab5e97aba335f34b9b0d50ef558f5b74a4ed14549c19675495bd213d852db09a3bbf2da8fc95abfa2cb3f833b06e2aca50a
-  data.tar.gz: 92050432ab95a2cec0cbbfa3b54f12101880da2f856935537e85fef43e196c8f560d78b2d06f6ed26e866c64bf889ded1b4d1b4e88c04ac4b4f0df6f6c16cb31
+  metadata.gz: 2017b6bb33289a7a2c2cba047f966079ae07fa6035acf819b1898c32ee7fc38417dc424c9ba263a33989e3eb0e241bdef32b816135ae1b86e0aae9f176be14d0
+  data.tar.gz: a71a80f5a7c177950a52eebad506778fd7a8993577d7c7ec22e57467a7a80a0610edf5a08bc23d9d831547ec4560da077bae5e358f9f4e3c34c089a6674bbd7f

data/CHANGELOG.md

@@ -1,8 +1,21 @@
 # CHANGELOG
 
-## 0.21.1 (2021-02-26)
+## 0.25.0 (2021-03-23)
 
-* Fix `find_matching_nodes` if `current_node` is a `Parser::AST::Node`
+* Use `Gem::Dependency#match?` to check gem version
+
+## 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)
 

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, "hash_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/gem_spec.rb

@@ -3,22 +3,13 @@
 module Synvert::Core
   # GemSpec checks and compares gem version.
   class Rewriter::GemSpec
-    OPERATORS = { eq: '==', lt: '<', gt: '>', lte: '<=', gte: '>=', ne: '!=' }.freeze
-
     # Initialize a gem_spec.
     #
     # @param name [String] gem name
-    # @param comparator [Hash] comparator to gem version, e.g. {eq: '2.0.0'},
-    #   comparator key can be eq, lt, gt, lte, gte or ne.
-    def initialize(name, comparator)
+    # @param version [String] gem version, e.g. '~> 2.0.0',
+    def initialize(name, version)
       @name = name
-      if comparator.is_a?(Hash)
-        @operator = comparator.keys.first
-        @version = Gem::Version.new comparator.values.first
-      else
-        @operator = :eq
-        @version = Gem::Version.new comparator
-      end
+      @version = version
     end
 
     # Check if the specified gem version in Gemfile.lock matches gem_spec comparator.
@@ -33,11 +24,7 @@ module Synvert::Core
 
       ENV['BUNDLE_GEMFILE'] = Configuration.path # make sure bundler reads Gemfile.lock in the correct path
       parser = Bundler::LockfileParser.new(File.read(gemfile_lock_path))
-      if spec = parser.specs.find { |spec| spec.name == @name }
-        Gem::Version.new(spec.version).send(OPERATORS[@operator], @version)
-      else
-        false
-      end
+      parser.specs.any? { |spec| Gem::Dependency.new(@name, @version).match?(spec) }
     end
   end
 end

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

@@ -42,7 +42,13 @@ module Synvert::Core
           matching_nodes << child_node if child_node.match? @rules
         end
       elsif current_node.is_a?(Parser::AST::Node)
-        matching_nodes << current_node if current_node.match? @rules
+        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.1'
+    VERSION = '0.25.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/gem_spec_spec.rb

@@ -21,7 +21,7 @@ GEM
     it 'returns true if version in Gemfile.lock is greater than definition' do
       expect(File).to receive(:exist?).with('./Gemfile.lock').and_return(true)
       expect(File).to receive(:read).with('./Gemfile.lock').and_return(gemfile_lock_content)
-      gem_spec = Rewriter::GemSpec.new('ast', { gte: '1.0.0' })
+      gem_spec = Rewriter::GemSpec.new('ast', '~> 1.1')
       expect(gem_spec).to be_match
     end
 
@@ -35,7 +35,7 @@ GEM
     it 'returns false if version in Gemfile.lock is less than definition' do
       expect(File).to receive(:exist?).with('./Gemfile.lock').and_return(true)
       expect(File).to receive(:read).with('./Gemfile.lock').and_return(gemfile_lock_content)
-      gem_spec = Rewriter::GemSpec.new('ast', { gt: '1.2.0' })
+      gem_spec = Rewriter::GemSpec.new('ast', '> 1.2.0')
       expect(gem_spec).not_to be_match
     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.1
+  version: 0.25.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-26 00:00:00.000000000 Z
+date: 2021-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport