synvert-core 1.21.6 → 1.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be03ea2206a20768b84cfc41ae93146510c561b1ae271ae4f1d1e6edc1a5c0c8
-  data.tar.gz: fa219393c0c4db9e4b2080d9e6a53245e525a29f12416e013c0afaaaa2c3629c
+  metadata.gz: 90f6acc3b15b0d25a8df1320393862f574dffbfc1743fd5bbb03f7323d994ae1
+  data.tar.gz: 7fa5d3c80ed1c86fc8c67abc2add07e374681c3bb54bc53e590195ffb51c3fb2
 SHA512:
-  metadata.gz: e9700c9b08ea11158f741634883e1cec3a69c5b18b15dd80424e42d6541f605aa16ac494dfd4bc25b4d2aa88b61b976e76d1a2e8b9f09c4a4e7a870bb9d5b43a
-  data.tar.gz: e382f8066407aa95c8079ced3e08bc453574e8329779c1d80c425441fbb72882cca60b217a482c4c691dd0ed02d1d7a3c88a712ff5b53c8f64a6dda922a8ec59
+  metadata.gz: 755e6b198dc2ce1c652ea6bced3f9ee002599e6ab51dee5ea1a9dd8d103b0aafa86fa495808db548a675e0a6bf0356533687f7cd652e73dd519bec9e50cb55dd
+  data.tar.gz: f2364e3359310b864a813d6132e9475ea4f19f95219e849bcbd33cdfe362f849c5a11ddc9220ca6b7689058ca795e70be75a63043fc9498224d336f2fd10a567

data/.github/workflows/main.yml

@@ -11,10 +11,10 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby-version: ['2.6', '2.7', '3.0', '3.1']
+        ruby-version: ['2.7', '3.0', '3.1', '3.2']
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:
@@ -28,10 +28,11 @@ jobs:
     needs: test
     name: Update gh-pages to docs
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-ruby@v1
+      - uses: actions/checkout@v3
+      - uses: ruby/setup-ruby@v1
         with:
-          ruby-version: '3.1'
+          ruby-version: '3.2'
+          bundler-cache: true
       - name: Install required gem dependencies
         run: gem install yard redcarpet github-markup --no-document
       - name: Build YARD Ruby Documentation

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # CHANGELOG
 
+## 1.22.0 (2023-03-24)
+
+* Add `and_comma` option to `insert`, `insert_before`, `insert_after` api
+
+## 1.21.7 (2023-03-23)
+
+* Polish `erb` engine
+* Update `node_mutation` to 1.12.1
+
 ## 1.21.6 (2023-02-16)
 
 * `RAILS_VIEW_FILES` do not check `haml` and `slim` files until we support them

data/Gemfile

@@ -5,7 +5,6 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in synvert.gemspec
 gemspec
 
-require 'pp' # https://github.com/defunkt/fakefs/issues/99
 gem "fakefs", require: "fakefs/safe"
 gem "guard"
 gem "guard-rspec"

data/Gemfile.lock

@@ -1,10 +1,9 @@
 PATH
   remote: .
   specs:
-    synvert-core (1.21.6)
+    synvert-core (1.22.0)
       activesupport (< 7.0.0)
-      erubis
-      node_mutation (>= 1.9.2)
+      node_mutation (>= 1.12.1)
       node_query (>= 1.12.0)
       parallel
       parser
@@ -13,7 +12,7 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.1.7.2)
+    activesupport (6.1.7.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -21,10 +20,10 @@ GEM
       zeitwerk (~> 2.3)
     ast (2.4.2)
     coderay (1.1.3)
-    concurrent-ruby (1.2.0)
+    concurrent-ruby (1.2.2)
     diff-lcs (1.5.0)
     erubis (2.7.0)
-    fakefs (1.8.0)
+    fakefs (2.4.0)
     ffi (1.15.5)
     formatador (1.1.0)
     guard (2.18.0)
@@ -48,16 +47,16 @@ GEM
       rb-inotify (~> 0.9, >= 0.9.10)
     lumberjack (1.2.8)
     method_source (1.0.0)
-    minitest (5.17.0)
+    minitest (5.18.0)
     nenv (0.3.0)
-    node_mutation (1.9.3)
+    node_mutation (1.12.1)
       erubis
     node_query (1.12.0)
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
     parallel (1.22.1)
-    parser (3.2.1.0)
+    parser (3.2.1.1)
       ast (~> 2.4.1)
     parser_node_ext (1.0.0)
       parser

data/lib/synvert/core/engine/erb.rb

@@ -1,19 +1,17 @@
 # frozen_string_literal: true
 
-require 'erubis'
-
 module Synvert::Core
   module Engine
     class Erb
       class << self
-        # convert erb to ruby code.
+        # Encode erb string, leave only ruby code, replace other erb code with whitespace.
         #
-        # @param source [String] erb source code.
-        # @return [String] ruby source code.
+        # @param source [String] erb code.
+        # @return [String] encoded ruby code.
         def encode(source)
-          source.gsub(/^.*?<%=?/m) { |str| ' ' * str.size }
-                .gsub(/%>.*?$/m) { |str| ' ' * str.size }
-                .gsub(/%>.*?<%=?/m) { |str| ' ' * str.size }
+          source.gsub(/%>.*?<%=?/m) { |str| ' ' * str.size }
+                .sub(/^.*?<%=?/m) { |str| ' ' * str.size }
+                .sub(/%>.*?$/m) { |str| ' ' * str.size }
         end
       end
     end

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

@@ -20,7 +20,7 @@ module Synvert::Core
       @block = block
       strategy = NodeMutation::Strategy::KEEP_RUNNING
       if rewriter.options[:strategy] == Strategy::ALLOW_INSERT_AT_SAME_POSITION
-        strategy |=  NodeMutation::Strategy::ALLOW_INSERT_AT_SAME_POSITION
+        strategy |= NodeMutation::Strategy::ALLOW_INSERT_AT_SAME_POSITION
       end
       NodeMutation.configure({ strategy: strategy, tab_width: Configuration.tab_width })
       rewriter.helpers.each { |helper| singleton_class.send(:define_method, helper[:name], &helper[:block]) }
@@ -90,7 +90,6 @@ module Synvert::Core
       end
     end
 
-
     # Gets current node, it allows to get current node in block code.
     #
     # @return [Parser::AST::Node]
@@ -251,8 +250,9 @@ module Synvert::Core
     # @param code [String] code need to be inserted.
     # @param at [String] insert position, beginning or end
     # @param to [String] where to insert, if it is nil, will insert to current node.
-    def insert(code, at: 'end', to: nil)
-      @current_mutation.insert(@current_node, code, at: at, to: to)
+    # @param and_comma [Boolean] insert extra comma.
+    def insert(code, at: 'end', to: nil, and_comma: false)
+      @current_mutation.insert(@current_node, code, at: at, to: to, and_comma: and_comma)
     end
 
     # It inserts the code next to the current node.
@@ -265,9 +265,11 @@ module Synvert::Core
     #     insert_after "{{receiver}}.secret_key_base = \"#{SecureRandom.hex(64)}\""
     #   end
     # @param code [String] code need to be inserted.
-    def insert_after(code, to: nil)
+    # @param to [String] where to insert, if it is nil, will insert to current node.
+    # @param and_comma [Boolean] insert extra comma.
+    def insert_after(code, to: nil, and_comma: false)
       column = ' ' * NodeMutation.adapter.get_start_loc(@current_node).column
-      @current_mutation.insert(@current_node, "\n#{column}#{code}", at: 'end', to: to)
+      @current_mutation.insert(@current_node, "\n#{column}#{code}", at: 'end', to: to, and_comma: and_comma)
     end
 
     # It inserts the code previous to the current node.
@@ -280,9 +282,11 @@ module Synvert::Core
     #     insert_before "{{receiver}}.secret_key_base = \"#{SecureRandom.hex(64)}\""
     #   end
     # @param code [String] code need to be inserted.
-    def insert_before(code, to: nil)
+    # @param to [String] where to insert, if it is nil, will insert to current node.
+    # @param and_comma [Boolean] insert extra comma.
+    def insert_before(code, to: nil, and_comma: false)
       column = ' ' * NodeMutation.adapter.get_start_loc(@current_node).column
-      @current_mutation.insert(@current_node, "#{code}\n#{column}", at: 'beginning', to: to)
+      @current_mutation.insert(@current_node, "#{code}\n#{column}", at: 'beginning', to: to, and_comma: and_comma)
     end
 
     # It replaces erb stmt code to expr code.
@@ -333,10 +337,9 @@ module Synvert::Core
     #   with_node type: 'send', message: { in: %w[puts p] } do
     #     remove
     #   end
-    # @param options [Hash] options.
     # @option and_comma [Boolean] delete extra comma.
-    def remove(**options)
-      @current_mutation.remove(@current_node, **options)
+    def remove(and_comma: false)
+      @current_mutation.remove(@current_node, and_comma: and_comma)
     end
 
     # It deletes child nodes.
@@ -348,10 +351,9 @@ module Synvert::Core
     #     delete :receiver, :dot
     #   end
     # @param selectors [Array<Symbol>] selector names of child node.
-    # @param options [Hash]
     # @option and_comma [Boolean] delete extra comma.
-    def delete(*selectors, **options)
-      @current_mutation.delete(@current_node, *selectors, **options)
+    def delete(*selectors, and_comma: false)
+      @current_mutation.delete(@current_node, *selectors, and_comma: and_comma)
     end
 
     # It wraps current node with code.
@@ -404,7 +406,8 @@ module Synvert::Core
       if str.include?(quote) && !str.include?(another_quote)
         return "#{another_quote}#{str}#{another_quote}"
       end
-      escaped_str = str.gsub(quote) { |char| '\\' + quote }
+
+      escaped_str = str.gsub(quote) { |_char| '\\' + quote }
       quote + escaped_str + quote
     end
 
@@ -413,7 +416,7 @@ module Synvert::Core
     # @param tab_size [Integer] tab size
     # @return [String]
     def add_leading_spaces(str, tab_size: 1)
-      " " * Configuration.tab_width * tab_size + str;
+      (" " * Configuration.tab_width * tab_size) + str;
     end
 
     private

data/lib/synvert/core/rewriter.rb

@@ -94,7 +94,15 @@ module Synvert::Core
     #   @return [Array<Object>] the test results
     # @!attribute [rw] options
     #   @return [Hash] the rewriter options
-    attr_reader :group, :name, :sub_snippets, :helpers, :warnings, :affected_files, :ruby_version, :gem_spec, :test_results
+    attr_reader :group,
+                :name,
+                :sub_snippets,
+                :helpers,
+                :warnings,
+                :affected_files,
+                :ruby_version,
+                :gem_spec,
+                :test_results
     attr_accessor :options
 
     # Initialize a Rewriter.
@@ -230,10 +238,11 @@ module Synvert::Core
           instance.process
         end
       else
-        results = handle_one_file(Array(file_patterns)) do |file_path|
-          instance = Rewriter::Instance.new(self, file_path, &block)
-          instance.test
-        end
+        results =
+          handle_one_file(Array(file_patterns)) do |file_path|
+            instance = Rewriter::Instance.new(self, file_path, &block)
+            instance.test
+          end
         merge_test_results(results)
       end
     end

data/lib/synvert/core/snippet_not_found_error.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 
 class SnippetNotFoundError < StandardError
-end
+end

data/lib/synvert/core/strategy.rb

@@ -4,4 +4,4 @@ module Synvert::Core
   class Strategy
     ALLOW_INSERT_AT_SAME_POSITION = "allow_insert_at_same_position"
   end
-end
+end

data/lib/synvert/core/utils.rb

@@ -15,14 +15,17 @@ module Synvert::Core
         if is_valid_url?(snippet_name)
           uri = URI.parse(format_url(snippet_name))
           return uri.open.read if remote_snippet_exists?(uri)
+
           raise SnippetNotFoundError.new("#{snippet_name} nout found")
         elsif is_valid_file?(snippet_name)
           return File.read(snippet_name)
         else
           snippet_path = snippet_expand_path(snippet_name)
           return File.read(snippet_path) if File.exist?(snippet_path)
+
           snippet_uri = URI.parse(format_url(remote_snippet_url(snippet_name)))
           return snippet_uri.open.read if remote_snippet_exists?(snippet_uri)
+
           raise SnippetNotFoundError.new("#{snippet_name} nout found")
         end
       end
@@ -32,9 +35,10 @@ module Synvert::Core
       # @return [Array<String>] file paths
       def glob(file_patterns)
         Dir.chdir(Configuration.root_path) do
-          all_files = file_patterns.flat_map do |file_pattern|
-            Dir.glob(file_pattern)
-          end
+          all_files =
+            file_patterns.flat_map do |file_pattern|
+              Dir.glob(file_pattern)
+            end
           filter_only_paths(all_files) - get_skip_files
         end
       end
@@ -105,4 +109,4 @@ module Synvert::Core
       end
     end
   end
-end
+end

data/lib/synvert/core/version.rb

@@ -2,6 +2,6 @@
 
 module Synvert
   module Core
-    VERSION = '1.21.6'
+    VERSION = '1.22.0'
   end
 end

data/lib/synvert/core.rb

@@ -9,7 +9,6 @@ require_relative './core/node_ext'
 require 'active_support'
 require 'active_support/core_ext'
 require 'erubis'
-require 'set'
 require 'node_query'
 require 'node_mutation'
 

data/spec/synvert/core/engine/erb_spec.rb

@@ -34,11 +34,14 @@ module Synvert::Core
           <% end %>
         <% end %>
       EOF
-      encoded_lines = Engine::Erb.encode(source).split("\n")
-      expect(encoded_lines[0]).to eq '   content_for :head do   '
-      expect(encoded_lines[1]).to eq "                                                     asset_path('bg.png')     "
-      expect(encoded_lines[-2]).to eq '     end   '
-      expect(encoded_lines[-1]).to eq '   end   '
+      encoded_source = Engine::Erb.encode(source)
+      expect(encoded_source).to be_include 'content_for :head do'
+      expect(encoded_source).to be_include " asset_path('bg.png')"
+      expect(encoded_source).to be_include 'post = Post.find(:first)'
+      expect(encoded_source).to be_include "link_to_function 'test', \"confirm('test');\""
+      expect(encoded_source).to be_include 'end'
+      expect(encoded_source).not_to be_include 'style'
+      expect(encoded_source).not_to be_include 'div'
     end
   end
 end

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

@@ -12,7 +12,12 @@ module Synvert::Core
     it 'parses find_node' do
       scope = double
       block = proc {}
-      expect(Rewriter::WithinScope).to receive(:new).with(instance, '.send[message=create]', {}, &block).and_return(scope)
+      expect(Rewriter::WithinScope).to receive(:new).with(
+        instance,
+        '.send[message=create]',
+        {},
+        &block
+      ).and_return(scope)
       expect(scope).to receive(:process)
       instance.find_node('.send[message=create]', &block)
     end
@@ -99,28 +104,46 @@ module Synvert::Core
     it 'parses append' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:append).with(instance.current_node, 'Foobar')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:append).with(
+        instance.current_node,
+        'Foobar'
+      )
       instance.append 'Foobar'
     end
 
     it 'parses prepend' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:prepend).with(instance.current_node, 'Foobar')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:prepend).with(
+        instance.current_node,
+        'Foobar'
+      )
       instance.prepend 'Foobar'
     end
 
     it 'parses insert at end' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(instance.current_node, 'Foobar', at: 'end', to: 'receiver')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(
+        instance.current_node,
+        'Foobar',
+        at: 'end',
+        to: 'receiver',
+        and_comma: false
+      )
       instance.insert 'Foobar', to: 'receiver'
     end
 
     it 'parses insert at beginning' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(instance.current_node, 'Foobar', at: 'beginning', to: nil)
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(
+        instance.current_node,
+        'Foobar',
+        at: 'beginning',
+        to: nil,
+        and_comma: false
+      )
       instance.insert 'Foobar', at: 'beginning'
     end
 
@@ -128,7 +151,13 @@ module Synvert::Core
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
       expect(NodeMutation).to receive_message_chain(:adapter, :get_start_loc, :column).and_return(2)
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(instance.current_node, "\n  Foobar", at: 'end', to: nil)
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(
+        instance.current_node,
+        "\n  Foobar",
+        at: 'end',
+        to: nil,
+        and_comma: false
+      )
       instance.insert_after 'Foobar'
     end
 
@@ -136,7 +165,13 @@ module Synvert::Core
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
       expect(NodeMutation).to receive_message_chain(:adapter, :get_start_loc, :column).and_return(2)
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(instance.current_node, "Foobar\n  ", at: 'beginning', to: nil)
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:insert).with(
+        instance.current_node,
+        "Foobar\n  ",
+        at: 'beginning',
+        to: nil,
+        and_comma: false
+      )
       instance.insert_before 'Foobar'
     end
 
@@ -153,35 +188,53 @@ module Synvert::Core
     it 'parses replace_with' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:replace_with).with(instance.current_node, 'Foobar')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:replace_with).with(
+        instance.current_node,
+        'Foobar'
+      )
       instance.replace_with 'Foobar'
     end
 
     it 'parses replace with' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:replace).with(instance.current_node, :message, with: 'Foobar')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:replace).with(
+        instance.current_node,
+        :message,
+        with: 'Foobar'
+      )
       instance.replace :message, with: 'Foobar'
     end
 
     it 'parses remove' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:remove).with(instance.current_node, and_comma: true)
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:remove).with(
+        instance.current_node,
+        and_comma: true
+      )
       instance.remove and_comma: true
     end
 
     it 'parses delete' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:delete).with(instance.current_node, :dot, :message, and_comma: true)
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:delete).with(
+        instance.current_node,
+        :dot,
+        :message,
+        and_comma: true
+      )
       instance.delete :dot, :message, and_comma: true
     end
 
     it 'parses wrap with' do
       instance.instance_variable_set(:@current_mutation, double)
       instance.current_node = double
-      expect(instance.instance_variable_get(:@current_mutation)).to receive(:wrap).with(instance.current_node, with: 'module Foobar')
+      expect(instance.instance_variable_get(:@current_mutation)).to receive(:wrap).with(
+        instance.current_node,
+        with: 'module Foobar'
+      )
       instance.wrap with: 'module Foobar'
     end
 
@@ -330,8 +383,8 @@ module Synvert::Core
         results = instance.test
         expect(results.file_path).to eq 'spec/models/post_spec.rb'
         expect(results.actions).to eq [
-          OpenStruct.new(start: 35, end: 59, new_code: 'create :user'),
-          OpenStruct.new(start: 69, end: 105, new_code: 'create :post, user: user')
+          NodeMutation::Struct::Action.new(35, 59, 'create :user'),
+          NodeMutation::Struct::Action.new(69, 105, 'create :post, user: user')
         ]
       end
 
@@ -369,7 +422,7 @@ module Synvert::Core
         expect(File).to receive(:read).with('./app/views/posts/_form.html.erb', encoding: 'UTF-8').and_return(input)
         result = instance.test
         expect(result.file_path).to eq 'app/views/posts/_form.html.erb'
-        expect(result.actions).to eq [OpenStruct.new(start: 2, end: 2, new_code: '=')]
+        expect(result.actions).to eq [NodeMutation::Struct::Action.new(2, 2, '=')]
       end
     end
 

data/spec/synvert/core/rewriter_spec.rb

@@ -33,13 +33,14 @@ module Synvert::Core
 
     describe '#process' do
       it 'rewrites the file' do
-        rewriter = Rewriter.new('group', 'name') do
-          within_files '**/*.rb' do
-            with_node node_type: 'class', name: 'Foobar' do
-              replace :name, with: 'Synvert'
+        rewriter =
+          Rewriter.new('group', 'name') do
+            within_files '**/*.rb' do
+              with_node node_type: 'class', name: 'Foobar' do
+                replace :name, with: 'Synvert'
+              end
             end
           end
-        end
         input = "class Foobar\nend"
         output = "class Synvert\nend"
         FakeFS do
@@ -52,13 +53,14 @@ module Synvert::Core
 
     describe '#test' do
       it 'gets test results' do
-        rewriter = Rewriter.new('group', 'name') do
-          within_files '**/*.rb' do
-            with_node node_type: 'class', name: 'Foobar' do
-              replace :name, with: 'Synvert'
+        rewriter =
+          Rewriter.new('group', 'name') do
+            within_files '**/*.rb' do
+              with_node node_type: 'class', name: 'Foobar' do
+                replace :name, with: 'Synvert'
+              end
             end
           end
-        end
         input = "class Foobar\nend"
         FakeFS do
           File.write("code.rb", input)
@@ -66,7 +68,7 @@ module Synvert::Core
           expect(results[0].file_path).to eq '/code.rb'
           expect(results[0].affected?).to be_truthy
           expect(results[0].conflicted?).to be_falsey
-          expect(results[0].actions).to eq [OpenStruct.new(start: 6, end: 12, new_code: 'Synvert')]
+          expect(results[0].actions).to eq [NodeMutation::Struct::Action.new(6, 12, 'Synvert')]
         end
       end
     end
@@ -226,9 +228,10 @@ module Synvert::Core
       it 'adds snippet by http url' do
         expect(Utils).to receive(:remote_snippet_exists?).with(URI.parse('http://synvert.net/foo/bar.rb')).and_return(true)
         expect_any_instance_of(URI::HTTP).to receive(:open).and_return(StringIO.new("Rewriter.new 'group', 'sub_rewriter' do\nend"))
-        rewriter = Rewriter.new 'group', 'rewriter' do
-          add_snippet 'http://synvert.net/foo/bar.rb'
-        end
+        rewriter =
+          Rewriter.new 'group', 'rewriter' do
+            add_snippet 'http://synvert.net/foo/bar.rb'
+          end
         rewriter.process
         expect(Rewriter.fetch('group', 'sub_rewriter')).not_to be_nil
       end
@@ -236,9 +239,10 @@ module Synvert::Core
       it 'adds snippet by file path' do
         expect(File).to receive(:exist?).and_return(true)
         expect(File).to receive(:read).and_return("Rewriter.new 'group', 'sub_rewriter' do\nend")
-        rewriter = Rewriter.new 'group', 'rewriter' do
-          add_snippet '/home/richard/foo/bar.rb'
-        end
+        rewriter =
+          Rewriter.new 'group', 'rewriter' do
+            add_snippet '/home/richard/foo/bar.rb'
+          end
         rewriter.process
         expect(Rewriter.fetch('group', 'sub_rewriter')).not_to be_nil
       end

data/spec/synvert/core/utils_spec.rb

@@ -61,22 +61,37 @@ module Synvert::Core
       end
 
       it 'gets all files' do
-        expect(Dir).to receive(:glob).with('**/*.rb').and_return(['app/models/post.rb', 'app/controllers/posts_controller.rb'])
+        expect(Dir).to receive(:glob).with('**/*.rb').and_return(
+          [
+            'app/models/post.rb',
+            'app/controllers/posts_controller.rb'
+          ]
+        )
         expect(described_class.glob(['**/*.rb'])).to eq(['app/models/post.rb', 'app/controllers/posts_controller.rb'])
       end
 
       it 'filters only paths' do
         Configuration.only_paths = ['app/models']
-        expect(Dir).to receive(:glob).with('**/*.rb').and_return(['app/models/post.rb', 'app/controllers/posts_controller.rb'])
+        expect(Dir).to receive(:glob).with('**/*.rb').and_return(
+          [
+            'app/models/post.rb',
+            'app/controllers/posts_controller.rb'
+          ]
+        )
         expect(described_class.glob(['**/*.rb'])).to eq(['app/models/post.rb'])
       end
 
       it 'skip files' do
         Configuration.skip_paths = ['app/controllers/**/*']
-        expect(Dir).to receive(:glob).with('**/*.rb').and_return(['app/models/post.rb', 'app/controllers/posts_controller.rb'])
+        expect(Dir).to receive(:glob).with('**/*.rb').and_return(
+          [
+            'app/models/post.rb',
+            'app/controllers/posts_controller.rb'
+          ]
+        )
         expect(Dir).to receive(:glob).with('app/controllers/**/*').and_return(['app/controllers/posts_controller.rb'])
         expect(described_class.glob(['**/*.rb'])).to eq(['app/models/post.rb'])
       end
     end
   end
-end
+end

data/synvert-core-ruby.gemspec

@@ -20,9 +20,8 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_runtime_dependency "activesupport", "< 7.0.0"
-  spec.add_runtime_dependency "erubis"
   spec.add_runtime_dependency "node_query", ">= 1.12.0"
-  spec.add_runtime_dependency "node_mutation", ">= 1.9.2"
+  spec.add_runtime_dependency "node_mutation", ">= 1.12.1"
   spec.add_runtime_dependency "parser"
   spec.add_runtime_dependency "parser_node_ext", ">= 1.0.0"
   spec.add_runtime_dependency "parallel"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synvert-core
 version: !ruby/object:Gem::Version
-  version: 1.21.6
+  version: 1.22.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-02-16 00:00:00.000000000 Z
+date: 2023-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -24,20 +24,6 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: 7.0.0
-- !ruby/object:Gem::Dependency
-  name: erubis
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: node_query
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.9.2
+        version: 1.12.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.9.2
+        version: 1.12.1
 - !ruby/object:Gem::Dependency
   name: parser
   requirement: !ruby/object:Gem::Requirement
@@ -188,7 +174,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.1
+rubygems_version: 3.4.6
 signing_key:
 specification_version: 4
 summary: convert ruby code to better syntax.