synvert-core 0.25.1 → 0.27.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1aab544b0a434d8d34f445906448203f8ebf670f2e3fc14ac791163c1c059bbb
-  data.tar.gz: c913b856a8281cbd07ac6ca59fea393a633b3d920b0849b9a9f16dd897989b0f
+  metadata.gz: afa25ca6412906c13399aa27967676033670bcbab36910a7bd528e3a9948e61d
+  data.tar.gz: f5844fee19d2aec45fd24b50af95decb53b7b33c90e2c83fad4fb62d565d0782
 SHA512:
-  metadata.gz: cc0c4ea16085329d3385227885af3fc10fc925be5abe0bf408afa89ee95d5aba048225be488b8624dd0cdb7c6c0b4ed994f9d8c66aa8891c252eaf5784e3f06c
-  data.tar.gz: d3a6ac011d673c981cc59e34ad0bbcba5c5042a5bc20ea15a46d6c77b775d8f682d212951ec8503432473044e2f58c80342810fe9d4521cf405bf719e845bdd3
+  metadata.gz: 2d188889dec4144d5508fc2ff3f9c6356703354cc1d37464a4d59fa406c56095fd1be796c5ffa474263bcb9a232f2ebbf48796de778fe80bbfe0a18137b7d048
+  data.tar.gz: 85a83b35f0b9c60682d75647a63eff485eff55cf41e46e12b7849b00d098c1268e615eee9d363e3ce068c8c4239e05bc499a86cb73d4ba70d1c986366e2f96a0

data/CHANGELOG.md

@@ -1,8 +1,22 @@
 # CHANGELOG
 
-## 0.25.1 (2021-03-24)
+## 0.27.2 (2021-03-31)
+
+* Fix selector with operator in `child_node_range`
+
+## 0.27.1 (2021-03-31)
+
+* Fix `delete` action arguments
+
+## 0.27.0 (2021-03-31)
+
+* Support `:class` in `child_node_range`
+* Add `delete` action
+
+## 0.26.0 (2021-03-30)
 
 * attr_reader ruby_version and gem_spec
+* Add `replace` action
 
 ## 0.25.0 (2021-03-23)
 

data/lib/synvert/core/node_ext.rb

@@ -317,6 +317,35 @@ module Parser::AST
       loc.expression.line
     end
 
+    # Get the source range of child node.
+    #
+    # @param [String] name of child node.
+    # @return [Parser::Source::Range] source range of child node.
+    def child_node_range(child_name)
+      case type
+      when :class
+        case child_name
+        when :name
+          loc.name
+        when :parent_class
+          parent_class&.loc&.expression
+        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?
+        end
+      else
+        raise Synvert::Core::MethodNotSupported, "child_node_range is not handled for #{evaluated.inspect}, child_name: #{child_name}"
+      end
+    end
+
     # Recursively iterate all child nodes of current node.
     #
     # @yield [child] Gives a child node.

data/lib/synvert/core/rewriter.rb

@@ -21,8 +21,10 @@ module Synvert::Core
     autoload :InsertAction, 'synvert/core/rewriter/action/insert_action'
     autoload :InsertAfterAction, 'synvert/core/rewriter/action/insert_after_action'
     autoload :ReplaceWithAction, 'synvert/core/rewriter/action/replace_with_action'
+    autoload :ReplaceAction, 'synvert/core/rewriter/action/replace_action'
     autoload :ReplaceErbStmtWithExprAction, 'synvert/core/rewriter/action/replace_erb_stmt_with_expr_action'
     autoload :RemoveAction, 'synvert/core/rewriter/action/remove_action'
+    autoload :DeleteAction, 'synvert/core/rewriter/action/delete_action'
 
     autoload :Warning, 'synvert/core/rewriter/warning'
 

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

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Synvert::Core
+  # RemoveAction to delete code.
+  class Rewriter::DeleteAction < Rewriter::Action
+    def initialize(instance, *selectors)
+      super(instance, nil)
+      @selectors = selectors
+    end
+
+    # Begin position of code to replace.
+    #
+    # @return [Integer] begin position.
+    def 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
+      @selectors.map { |selector| @node.child_node_range(selector).end_pos }.max
+    end
+
+    # The rewritten code, always empty string.
+    def rewritten_code
+      ''
+    end
+  end
+end

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

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module Synvert::Core
+  # ReplaceAction to replace child node with code.
+  class Rewriter::ReplaceAction < Rewriter::Action
+    def initialize(instance, selector, with:)
+      @instance = instance
+      @selector = selector
+      @code = with
+      @node = @instance.current_node
+    end
+
+    # Begin position of code to replace.
+    #
+    # @return [Integer] begin position.
+    def begin_pos
+      @node.child_node_range(@selector).begin_pos
+    end
+
+    # End position of code to replace.
+    #
+    # @return [Integer] end position.
+    def end_pos
+      @node.child_node_range(@selector).end_pos
+    end
+
+    # The rewritten source code.
+    #
+    # @return [String] rewritten code.
+    def rewritten_code
+      rewritten_source
+    end
+  end
+end

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

@@ -251,17 +251,33 @@ module Synvert::Core
       @actions << Rewriter::ReplaceWithAction.new(self, code, options)
     end
 
+    # Parse replace with dsl, it creates a [Synvert::Core::Rewriter::ReplaceAction] to
+    # replace child node with code.
+    #
+    # @param selector [Symbol] selector name of child node.
+    # @param with [String] code need to be replaced with.
+    def replace(selector, with:)
+      @actions << Rewriter::ReplaceAction.new(self, selector, with: with)
+    end
+
     # Parse replace_erb_stmt_with_expr dsl, it creates a [Synvert::Core::Rewriter::ReplaceErbStmtWithExprAction] to
     # replace erb stmt code to expr code.
     def replace_erb_stmt_with_expr
       @actions << Rewriter::ReplaceErbStmtWithExprAction.new(self)
     end
 
-    # Parse remove dsl, it creates a [Synvert::Core::Rewriter::RemoveAction] to current node.
+    # Parse remove dsl, it creates a [Synvert::Core::Rewriter::RemoveAction] to remove current node.
     def remove
       @actions << Rewriter::RemoveAction.new(self)
     end
 
+    # Parse delete dsl, it creates a [Synvert::Core::Rewriter::DeleteAction] to delete child nodes.
+    #
+    # @param selectors [Array<Symbol>] selector names of child node.
+    def delete(*selectors)
+      @actions << Rewriter::DeleteAction.new(self, *selectors)
+    end
+
     # Parse warn dsl, it creates a [Synvert::Core::Rewriter::Warning] to save warning message.
     #
     # @param message [String] warning message.

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '0.25.1'
+    VERSION = '0.27.2'
   end
 end

data/spec/synvert/core/node_ext_spec.rb

@@ -394,6 +394,76 @@ describe Parser::AST::Node do
     end
   end
 
+  describe '#child_node_range' do
+    context 'send node' do
+      it 'checks receiver' do
+        node = parse('foo.bar(test)')
+        range = node.child_node_range(:receiver)
+        expect(range.to_range).to eq(0...3)
+
+        node = parse('foobar(test)')
+        range = node.child_node_range(:receiver)
+        expect(range).to be_nil
+      end
+
+      it 'checks dot' do
+        node = parse('foo.bar(test)')
+        range = node.child_node_range(:dot)
+        expect(range.to_range).to eq(3...4)
+
+        node = parse('foobar(test)')
+        range = node.child_node_range(:dot)
+        expect(range).to be_nil
+      end
+
+      it 'checks message' do
+        node = parse('foo.bar(test)')
+        range = node.child_node_range(:message)
+        expect(range.to_range).to eq(4...7)
+
+        node = parse('foo.bar = test')
+        range = node.child_node_range(:message)
+        expect(range.to_range).to eq(4...9)
+
+        node = parse('foobar(test)')
+        range = node.child_node_range(:message)
+        expect(range.to_range).to eq(0...6)
+      end
+
+      it 'checks arguments' do
+        node = parse('foo.bar(test)')
+        range = node.child_node_range(:arguments)
+        expect(range.to_range).to eq(8...12)
+
+        node = parse('foobar(test)')
+        range = node.child_node_range(:arguments)
+        expect(range.to_range).to eq(7...11)
+
+        node = parse('foo.bar')
+        range = node.child_node_range(:arguments)
+        expect(range).to be_nil
+      end
+    end
+
+    context 'class node' do
+      it 'checks name' do
+        node = parse('class Post < ActiveRecord::Base; end')
+        range = node.child_node_range(:name)
+        expect(range.to_range).to eq(6...10)
+      end
+
+      it 'checks parent_class' do
+        node = parse('class Post < ActiveRecord::Base; end')
+        range = node.child_node_range(:parent_class)
+        expect(range.to_range).to eq(13...31)
+
+        node = parse('class Post; end')
+        range = node.child_node_range(:parent_class)
+        expect(range).to be_nil
+      end
+    end
+  end
+
   describe '#rewritten_source' do
     let(:instance) {
       rewriter = Synvert::Rewriter.new('foo', 'bar')

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

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+module Synvert::Core
+  describe Rewriter::DeleteAction do
+    subject {
+      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
+    end
+
+    it 'gets end_pos' do
+      expect(subject.end_pos).to eq "arr.map {}.flatten".length
+    end
+
+    it 'gets rewritten_code' do
+      expect(subject.rewritten_code).to eq ''
+    end
+  end
+end

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

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+module Synvert::Core
+  describe Rewriter::ReplaceAction do
+    context 'replace with single line' do
+      subject {
+        source = "'slug from title'.gsub(' ', '_')"
+        node = Parser::CurrentRuby.parse(source)
+        instance = double(current_node: node)
+        Rewriter::ReplaceAction.new(instance, :message, with: 'tr')
+      }
+
+      it 'gets begin_pos' do
+        expect(subject.begin_pos).to eq "'slug from title'.".length
+      end
+
+      it 'gets end_pos' do
+        expect(subject.end_pos).to eq "'slug from title'.gsub".length
+      end
+
+      it 'gets rewritten_code' do
+        expect(subject.rewritten_code).to eq 'tr'
+      end
+    end
+  end
+end

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

@@ -117,11 +117,21 @@ module Synvert::Core
       instance.replace_with 'create {{arguments}}'
     end
 
+    it 'parses replace with' do
+      expect(Rewriter::ReplaceAction).to receive(:new).with(instance, :message, with: 'test')
+      instance.replace :message, with: 'test'
+    end
+
     it 'parses remove' do
       expect(Rewriter::RemoveAction).to receive(:new).with(instance)
       instance.remove
     end
 
+    it 'parses remove' do
+      expect(Rewriter::DeleteAction).to receive(:new).with(instance, :dot, :message)
+      instance.delete :dot, :message
+    end
+
     it 'parses warn' do
       expect(Rewriter::Warning).to receive(:new).with(instance, 'foobar')
       instance.warn 'foobar'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 0.25.1
+  version: 0.27.2
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-24 00:00:00.000000000 Z
+date: 2021-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -147,9 +147,11 @@ files:
 - lib/synvert/core/rewriter.rb
 - lib/synvert/core/rewriter/action.rb
 - lib/synvert/core/rewriter/action/append_action.rb
+- lib/synvert/core/rewriter/action/delete_action.rb
 - lib/synvert/core/rewriter/action/insert_action.rb
 - lib/synvert/core/rewriter/action/insert_after_action.rb
 - lib/synvert/core/rewriter/action/remove_action.rb
+- 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/condition.rb
@@ -170,9 +172,11 @@ files:
 - spec/synvert/core/engine/erb_spec.rb
 - spec/synvert/core/node_ext_spec.rb
 - spec/synvert/core/rewriter/action/append_action_spec.rb
+- spec/synvert/core/rewriter/action/delete_action_spec.rb
 - spec/synvert/core/rewriter/action/insert_action_spec.rb
 - spec/synvert/core/rewriter/action/insert_after_action_spec.rb
 - spec/synvert/core/rewriter/action/remove_action_spec.rb
+- spec/synvert/core/rewriter/action/replace_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_with_action_spec.rb
 - spec/synvert/core/rewriter/action_spec.rb
@@ -219,9 +223,11 @@ test_files:
 - spec/synvert/core/engine/erb_spec.rb
 - spec/synvert/core/node_ext_spec.rb
 - spec/synvert/core/rewriter/action/append_action_spec.rb
+- spec/synvert/core/rewriter/action/delete_action_spec.rb
 - spec/synvert/core/rewriter/action/insert_action_spec.rb
 - spec/synvert/core/rewriter/action/insert_after_action_spec.rb
 - spec/synvert/core/rewriter/action/remove_action_spec.rb
+- spec/synvert/core/rewriter/action/replace_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_erb_stmt_with_expr_action_spec.rb
 - spec/synvert/core/rewriter/action/replace_with_action_spec.rb
 - spec/synvert/core/rewriter/action_spec.rb