sevencop 0.12.1 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: adcb3382c32455ca63651bf5593974e773cf00dfd0b3d14955541c6ce219c6c7
-  data.tar.gz: d6f50b292bb6f56c0f6ec36fc4271c931af4c1c46cef9b8c14a2505cf984f8ed
+  metadata.gz: 5b95d5ba5dafb534e8e3a9e5c49bd9724db92840e1ca00543251c48a6b0f74f5
+  data.tar.gz: c6f7822017c94416417238387b5e16b76b00e401a8819a27c0d6df40ac1373c2
 SHA512:
-  metadata.gz: 552d6582fc4aa3baa550afd6d27477c822e4f098b470c0c038a36380c6ab74f9962a27e8f04be05cc60586be43376de1ec20a0a5b56587d71b24d3fa36e8e4cb
-  data.tar.gz: 05df622c8de515879b816077ba5fc40cd47858c1666390beac8d2ac29e86c1983921d5d8964b8b7de64106e789f3217032b6338a9988442207dbe5aeb62a8031
+  metadata.gz: 131bb0495d50ebd37c0c03aa81b22d79d3488d8428697cd3af4263342f884b1857363920c40fa6794ba99e1de3e118bf94cf205cee96cc0aad77525f040072ee
+  data.tar.gz: f49be306dde9d6fda9db8d79a653c7814a4b43059265ceaf42fb4bbad2eee61fe112cb0ad56cdb152f626c26da9b54302f89f51f98652aa5ef37574ffcb06d45

data/.rubocop.yml

@@ -3,6 +3,7 @@ require:
   - rubocop-performance
   - rubocop-rake
   - rubocop-rspec
+  - sevencop
 
 AllCops:
   NewCops: enable
@@ -25,5 +26,11 @@ RSpec/ExampleLength:
 RSpec/MultipleExpectations:
   Enabled: false
 
+Sevencop/MethodDefinitionOrdered:
+  Enabled: true
+
 Style/Documentation:
   Enabled: false
+
+Style/MultilineBlockChain:
+  Enabled: false

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    sevencop (0.12.1)
+    sevencop (0.14.0)
       rubocop
 
 GEM

data/README.md

@@ -1,7 +1,6 @@
 # sevencop
 
 [![test](https://github.com/r7kamura/sevencop/actions/workflows/test.yml/badge.svg)](https://github.com/r7kamura/sevencop/actions/workflows/test.yml)
-[![Gem Version](https://badge.fury.io/rb/sevencop.svg)](https://rubygems.org/gems/sevencop)
 
 Custom cops for [RuboCop](https://github.com/rubocop/rubocop).
 
@@ -45,193 +44,16 @@ Sevencop/BelongsToOptional:
 
 ## Cops
 
-All cops are `Enabled: false` by default.
+See YARD comments in each cop class for details:
 
-### Sevencop/AutoloadOrdered
+- [Sevencop/BelongsToOptional](lib/rubocop/cop/sevencop/belongs_to_optional.rb)
+- [Sevencop/HashElementOrdered](lib/rubocop/cop/sevencop/hash_element_ordered.rb)
+- [Sevencop/InferredSpecType](lib/rubocop/cop/sevencop/inferred_spec_type.rb)
+- [Sevencop/MethodDefinitionArgumentsMultiline](lib/rubocop/cop/sevencop/method_definition_arguments_multiline.rb)
+- [Sevencop/MethodDefinitionKeywordArgumentsOrdered](lib/rubocop/cop/sevencop/method_definition_keyword_arguments_ordered.rb)
+- [Sevencop/MethodDefinitionOrdered](lib/rubocop/cop/sevencop/method_definition_ordered.rb)
+- [Sevencop/OrderField](lib/rubocop/cop/sevencop/order_field.rb)
+- [Sevencop/UniquenessValidatorExplicitCaseSensitivity](lib/rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity.rb)
+- [Sevencop/WhereNot](lib/rubocop/cop/sevencop/where_not.rb)
 
-Sort `autoload` in alphabetical order within their section.
-
-```ruby
-# bad
-autoload :B, 'b'
-autoload :A, 'a'
-
-# good
-autoload :A, 'a'
-autoload :B, 'b'
-
-# good
-autoload :B, 'b'
-autoload :D, 'd'
-
-autoload :A, 'a'
-autoload :C, 'a'
-```
-
-### Sevencop/BelongsToOptional
-
-Force `belongs_to` with `optional: true` option.
-
-```ruby
-# bad
-belongs_to :group
-
-# good
-belongs_to :group, optional: true
-
-# good
-belongs_to :group, optional: false
-
-# good
-belongs_to :group, options
-```
-
-This is useful for migration of `config.active_record.belongs_to_required_by_default`.
-
-### Sevencop/FactoryBotCreateList
-
-Finds possible substitutions for `FactoryBot.create_list`.
-
-```ruby
-# bad
-Array.new(2) do
-  create(:user)
-end
-
-# good
-create_list(:user, 2)
-
-# good
-Array.new(2) do |i|
-  create(:user, order: i)
-end
-```
-
-### Sevencop/HashLiteralOrder
-
-Sort Hash literal entries by key.
-
-```ruby
-# bad
-{
-  b: 1,
-  a: 1,
-  c: 1
-}
-
-# good
-{
-  a: 1,
-  b: 1,
-  c: 1
-}
-```
-
-### Sevencop/InferredSpecType
-
-Identifies redundant spec type.
-
-```ruby
-# bad
-# spec/models/user_spec.rb
-RSpec.describe User, type: :model
-
-# good
-# spec/models/user_spec.rb
-RSpec.describe User
-
-# good
-# spec/models/user_spec.rb
-RSpec.describe User, type: :request
-```
-
-### Sevencop/MethodDefinitionMultilineArguments
-
-Inserts new lines between method definition parameters.
-
-```ruby
-# bad
-def foo(a, b)
-end
-
-# good
-def foo(
-  a,
-  b
-)
-
-# good
-def foo(a)
-end
-```
-
-### Sevencop/OrderField
-
-Identifies a String including "field" is passed to `order` or `reorder`.
-
-```ruby
-# bad
-articles.order('field(id, ?)', a)
-
-# good
-articles.order(Arel.sql('field(id, ?)'), a)
-
-# bad
-reorder('field(id, ?)', a)
-
-# good
-reorder(Arel.sql('field(id, ?)'), a)
-```
-
-### Sevencop/RedundantExistenceCheck
-
-Identifies redundant existent check before file operation.
-
-```ruby
-# bad
-FileUtils.mkdir(a) unless FileTest.exist?(a)
-
-# good
-FileUtils.mkdir_p(a)
-
-# bad
-FileUtils.rm(a) if FileTest.exist?(a)
-
-# good
-FileUtils.rm_f(a)
-```
-
-### Sevencop/UniquenessValidatorExplicitCaseSensitivity
-
-Identifies use of UniquenessValidator without :case_sensitive option.
-
-```ruby
-# bad
-validates :name, uniqueness: true
-
-# good
-validates :name, uniqueness: { case_sensitive: true }
-
-# good
-validates :name, uniqueness: { case_sensitive: false }
-
-# bad
-validates :name, uniqueness: { allow_nil: true, scope: :user_id }
-
-# good
-validates :name, uniqueness: { allow_nil: true, scope: :user_id, case_sensitive: true }
-```
-
-Useful to keep the same behavior between Rails 6.0 and 6.1 where case insensitive collation is used in MySQL.
-
-### Sevencop/WhereNot
-
-Identifies passing multi-elements Hash literal to `where.not`.
-
-```ruby
-# bad
-where.not(key1: value1, key2: value2)
-
-# good
-where.not(key1: value1).where.not(key2: value2)
-```
+Note that all cops are `Enabled: false` by default.

data/config/default.yml

@@ -2,7 +2,7 @@ Sevencop/AutoloadOrdered:
   Description: |
     Sort `autoload` in alphabetical order.
   Enabled: false
-  VersionAdded: '0.12.0'
+  VersionAdded: '0.12'
 
 Sevencop/BelongsToOptional:
   Description: |
@@ -11,15 +11,9 @@ Sevencop/BelongsToOptional:
   Safe: false
   VersionAdded: '0.5'
 
-Sevencop/FactoryBotCreateList:
+Sevencop/HashElementOrdered:
   Description: |
-    Finds possible substitutions for `FactoryBot.create_list`.
-  Enabled: false
-  VersionAdded: '0.10'
-
-Sevencop/HashLiteralOrder:
-  Description: |
-    Sort Hash literal entries by key.
+    Sort Hash elements by key.
   Enabled: false
   VersionAdded: '0.6'
 
@@ -29,32 +23,30 @@ Sevencop/InferredSpecType:
   Enabled: false
   VersionAdded: '0.9'
 
-Sevencop/MethodDefinitionMultilineArguments:
+Sevencop/MethodDefinitionArgumentsMultiline:
   Description: |
-    Inserts new lines between method definition parameters.
+    Inserts new lines between method definition arguments.
   Enabled: false
   VersionAdded: '0.11'
 
-Sevencop/OrderField:
+Sevencop/MethodDefinitionOrdered:
   Description: |
-    Wrap safe SQL String by `Arel.sql`.
+    Sort method definitions in alphabetical order.
   Enabled: false
-  Safe: false
-  VersionAdded: '0.4'
+  VersionAdded: '0.14'
 
-Sevencop/RedundantExistenceCheck:
+Sevencop/MethodDefinitionKeywordArgumentsOrdered:
   Description: |
-    Avoid redundant existent check before file operation.
+    Sort method definition keyword arguments in alphabetical order.
   Enabled: false
-  Safe: false
-  VersionAdded: '0.1'
+  VersionAdded: '0.13'
 
-Sevencop/ToSWithArgument:
+Sevencop/OrderField:
   Description: |
-    Identifies passing any argument to `#to_s`.
+    Wrap safe SQL String by `Arel.sql`.
   Enabled: false
   Safe: false
-  VersionAdded: '0.8'
+  VersionAdded: '0.4'
 
 Sevencop/UniquenessValidatorExplicitCaseSensitivity:
   Description: |

data/lib/rubocop/cop/sevencop/autoload_ordered.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module Sevencop
-      # Sort `autoload` in alphabetical order within their section.
+      # Sort `autoload` in alphabetical order.
       #
       # @example
       #   # bad
@@ -25,7 +25,7 @@ module RuboCop
 
         include RangeHelp
 
-        MSG = 'Sort `autoload` in alphabetical order within their section.'
+        MSG = 'Sort `autoload` in alphabetical order.'
 
         RESTRICT_ON_SEND = %i[
           autoload

data/lib/rubocop/cop/sevencop/{hash_literal_order.rb → hash_element_ordered.rb}

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module Sevencop
-      # Sort Hash literal entries by key.
+      # Sort Hash elements by key.
       #
       # @example
       #
@@ -21,10 +21,10 @@ module RuboCop
       #     c: 1
       #   }
       #
-      class HashLiteralOrder < Base
+      class HashElementOrdered < Base
         extend AutoCorrector
 
-        MSG = 'Sort Hash literal entries by key.'
+        MSG = 'Sort Hash elements by key.'
 
         # @!method hash_literal?(node)
         def_node_matcher :hash_literal?, <<~PATTERN
@@ -64,6 +64,16 @@ module RuboCop
           parts.join
         end
 
+        # @param [RuboCop::AST::HashNode] node
+        # @return [Integer]
+        def offset_for(node)
+          if node.braces?
+            1
+          else
+            0
+          end
+        end
+
         # @param [Array<RuboCop::AST::PairNode>] pairs
         # @return [Array<RuboCop::AST::PairNode>]
         def sort(pairs)
@@ -95,32 +105,22 @@ module RuboCop
         # @param [RuboCop::AST::HashNode] node
         # @return [String]
         #   {    a: 1,   b: 1  }
-        #                    ^^
-        def whitespace_trailing(node)
+        #    ^^^^
+        def whitespace_leading(node)
           processed_source.raw_source[
-            node.pairs[-1].location.expression.end_pos...node.location.expression.end.begin_pos - offset_for(node)
+            node.location.expression.begin.end_pos + offset_for(node)...node.pairs[0].location.expression.begin_pos
           ]
         end
 
         # @param [RuboCop::AST::HashNode] node
         # @return [String]
         #   {    a: 1,   b: 1  }
-        #    ^^^^
-        def whitespace_leading(node)
+        #                    ^^
+        def whitespace_trailing(node)
           processed_source.raw_source[
-            node.location.expression.begin.end_pos + offset_for(node)...node.pairs[0].location.expression.begin_pos
+            node.pairs[-1].location.expression.end_pos...node.location.expression.end.begin_pos - offset_for(node)
           ]
         end
-
-        # @param [RuboCop::AST::HashNode] node
-        # @return [Integer]
-        def offset_for(node)
-          if node.braces?
-            1
-          else
-            0
-          end
-        end
       end
     end
   end

data/lib/rubocop/cop/sevencop/{method_definition_multiline_arguments.rb → method_definition_arguments_multiline.rb}

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module Sevencop
-      # Inserts new lines between method definition parameters.
+      # Inserts new lines between method definition arguments.
       #
       # @example
       #   # bad
@@ -19,10 +19,10 @@ module RuboCop
       #   # good
       #   def foo(a)
       #   end
-      class MethodDefinitionMultilineArguments < Base
+      class MethodDefinitionArgumentsMultiline < Base
         extend AutoCorrector
 
-        MSG = 'Insert new lines between method definition parameters.'
+        MSG = 'Insert new lines between method definition arguments.'
 
         # @param node [RuboCop::AST::ArgsNode]
         def on_args(node)
@@ -54,6 +54,11 @@ module RuboCop
           )
         end
 
+        # @return [Boolean]
+        def method_args?(node)
+          !node.parent.nil? && node.parent.def_type?
+        end
+
         # @param node [RuboCop::AST::ArgsNode]
         # @return [Boolean]
         def multilined?(node)
@@ -61,11 +66,6 @@ module RuboCop
             child.location.expression.line
           end.uniq.length == node.children.length
         end
-
-        # @return [Boolean]
-        def method_args?(node)
-          !node.parent.nil? && node.parent.def_type?
-        end
       end
     end
   end

data/lib/rubocop/cop/sevencop/method_definition_keyword_arguments_ordered.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Sevencop
+      # Sort method definition keyword arguments in alphabetical order.
+      #
+      # @example
+      #   # bad
+      #   def foo(b:, a:); end
+      #
+      #   # good
+      #   def foo(a:, b:); end
+      #
+      #   # bad
+      #   def foo(c:, d:, b: 1, a: 2); end
+      #
+      #   # good
+      #   def foo(c:, d:, a: 2, b: 1); end
+      class MethodDefinitionKeywordArgumentsOrdered < Base
+        extend AutoCorrector
+
+        MSG = 'Sort method definition keyword arguments in alphabetical order.'
+
+        # @param node [RuboCop::AST::ArgNode]
+        def on_kwarg(node)
+          previous_older_kwarg = find_previous_older_sibling(node)
+          return unless previous_older_kwarg
+
+          add_offense(node) do |corrector|
+            corrector.insert_before(
+              previous_older_kwarg,
+              "#{node.source}, "
+            )
+            corrector.remove(
+              node.location.expression.with(
+                begin_pos: node.left_sibling.location.expression.end_pos
+              )
+            )
+          end
+        end
+        alias on_kwoptarg on_kwarg
+
+        private
+
+        # @param node [RuboCop::AST::ArgNode]
+        # @return [RuboCop::AST::ArgNode]
+        def find_previous_older_sibling(node)
+          node.left_siblings.find do |sibling|
+            next if sibling.type != node.type
+
+            sibling.name > node.name
+          end
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sevencop/method_definition_ordered.rb

@@ -0,0 +1,111 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Sevencop
+      # Sort method definition in alphabetical order.
+      #
+      # @example
+      #   # bad
+      #   def b; end
+      #   def a; end
+      #
+      #   # good
+      #   def a; end
+      #   def b; end
+      #
+      #   # good
+      #   def b; end
+      #   def c; end
+      #   private
+      #   def a; end
+      #   def d; end
+      #
+      #   # good
+      #   def initialize; end
+      #   def a; end
+      class MethodDefinitionOrdered < Base
+        extend AutoCorrector
+
+        include RangeHelp
+        include VisibilityHelp
+
+        MSG = 'Sort method definition in alphabetical order.'
+
+        # @param node [RuboCop::AST::DefNode]
+        def on_def(node)
+          previous_older_sibling = find_previous_older_sibling(node)
+          return unless previous_older_sibling
+
+          add_offense(node) do |corrector|
+            swap(
+              range_by_whole_lines(
+                range_with_comments(previous_older_sibling),
+                include_final_newline: true
+              ),
+              range_by_whole_lines(
+                range_with_comments(node),
+                include_final_newline: true
+              ),
+              corrector: corrector
+            )
+          end
+        end
+        alias on_defs on_def
+
+        private
+
+        # @param node [RuboCop::AST::DefNode]
+        # @return [RuboCop::AST::DefNode]
+        def find_previous_older_sibling(node)
+          previous_siblings_in_same_section_of(node).find do |sibling|
+            next if sibling.type != node.type
+
+            (sort_key_of(sibling) <=> sort_key_of(node)) == 1
+          end
+        end
+
+        # @param node [RuboCop::AST::Node]
+        # @return [Array<RuboCop::AST::Node>]
+        def previous_siblings_in_same_section_of(node)
+          return node.left_siblings if node.defs_type?
+
+          node.left_siblings.reverse.take_while do |sibling|
+            !visibility_block?(sibling)
+          end.reverse
+        end
+
+        # @param node [RuboCop::AST::Node]
+        # @return [Paresr::Source::Range]
+        def range_with_comments(node)
+          comment = processed_source.ast_with_comments[node].first
+          if comment
+            node.location.expression.with(begin_pos: comment.location.expression.begin_pos)
+          else
+            node.location.expression
+          end
+        end
+
+        # @param node [RuboCop::AST::DefNode]
+        # @return [#<=>]
+        def sort_key_of(node)
+          [
+            node.method?(:initialize) ? 0 : 1,
+            node.method_name
+          ]
+        end
+
+        # @param range1 [Paresr::Source::Range]
+        # @param range2 [Paresr::Source::Range]
+        # @param corrector [RuboCop::AST::Corrector]
+        def swap(range1, range2, corrector:)
+          corrector.insert_before(
+            range1,
+            "#{range2.source}\n"
+          )
+          corrector.remove(range2)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity.rb

@@ -65,6 +65,11 @@ module RuboCop
           )
         PATTERN
 
+        # @param [RuboCop::AST::SendNode] node
+        def find_uniqueness_value(node)
+          node.arguments[1].pairs.find { |pair| pair.key.value == :uniqueness }.value
+        end
+
         # @param [RuboCop::AST::SendNode] node
         def on_send(node)
           return unless validates_uniqueness?(node) && !validates_uniqueness_with_case_sensitivity?(node)
@@ -84,11 +89,6 @@ module RuboCop
             end
           end
         end
-
-        # @param [RuboCop::AST::SendNode] node
-        def find_uniqueness_value(node)
-          node.arguments[1].pairs.find { |pair| pair.key.value == :uniqueness }.value
-        end
       end
     end
   end

data/lib/sevencop/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Sevencop
-  VERSION = '0.12.1'
+  VERSION = '0.14.0'
 end

data/lib/sevencop.rb

@@ -5,12 +5,11 @@ require_relative 'sevencop/version'
 
 require_relative 'rubocop/cop/sevencop/autoload_ordered'
 require_relative 'rubocop/cop/sevencop/belongs_to_optional'
-require_relative 'rubocop/cop/sevencop/factory_bot_create_list'
-require_relative 'rubocop/cop/sevencop/hash_literal_order'
+require_relative 'rubocop/cop/sevencop/hash_element_ordered'
 require_relative 'rubocop/cop/sevencop/inferred_spec_type'
-require_relative 'rubocop/cop/sevencop/method_definition_multiline_arguments'
+require_relative 'rubocop/cop/sevencop/method_definition_arguments_multiline'
+require_relative 'rubocop/cop/sevencop/method_definition_keyword_arguments_ordered'
+require_relative 'rubocop/cop/sevencop/method_definition_ordered'
 require_relative 'rubocop/cop/sevencop/order_field'
-require_relative 'rubocop/cop/sevencop/redundant_existence_check'
-require_relative 'rubocop/cop/sevencop/to_s_with_argument'
 require_relative 'rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity'
 require_relative 'rubocop/cop/sevencop/where_not'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sevencop
 version: !ruby/object:Gem::Version
-  version: 0.12.1
+  version: 0.14.0
 platform: ruby
 authors:
 - Ryo Nakamura
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-09-23 00:00:00.000000000 Z
+date: 2022-09-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -43,13 +43,12 @@ files:
 - config/default.yml
 - lib/rubocop/cop/sevencop/autoload_ordered.rb
 - lib/rubocop/cop/sevencop/belongs_to_optional.rb
-- lib/rubocop/cop/sevencop/factory_bot_create_list.rb
-- lib/rubocop/cop/sevencop/hash_literal_order.rb
+- lib/rubocop/cop/sevencop/hash_element_ordered.rb
 - lib/rubocop/cop/sevencop/inferred_spec_type.rb
-- lib/rubocop/cop/sevencop/method_definition_multiline_arguments.rb
+- lib/rubocop/cop/sevencop/method_definition_arguments_multiline.rb
+- lib/rubocop/cop/sevencop/method_definition_keyword_arguments_ordered.rb
+- lib/rubocop/cop/sevencop/method_definition_ordered.rb
 - lib/rubocop/cop/sevencop/order_field.rb
-- lib/rubocop/cop/sevencop/redundant_existence_check.rb
-- lib/rubocop/cop/sevencop/to_s_with_argument.rb
 - lib/rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity.rb
 - lib/rubocop/cop/sevencop/where_not.rb
 - lib/sevencop.rb

data/lib/rubocop/cop/sevencop/factory_bot_create_list.rb

@@ -1,95 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module Sevencop
-      # Finds possible substitutions for `FactoryBot.create_list`.
-      #
-      # @example
-      #
-      #   # bad
-      #   Array.new(2) do
-      #     create(:user)
-      #   end
-      #
-      #   # good
-      #   create_list(:user, 2)
-      #
-      #   # good
-      #   Array.new(2) do |i|
-      #     create(:user, order: i)
-      #   end
-      #
-      # @note
-      #   This cop does not support `Integer#times` intentionally because it
-      #   should be treated by `Performance/TimesMap` cop.
-      #
-      class FactoryBotCreateList < Base
-        extend AutoCorrector
-
-        MSG = 'Use `create_list` instead.'
-
-        # @param node [RuboCop::AST::BlockNode]
-        # @return [void]
-        def on_block(node)
-          count_node, factory_name_node, extra_argument_nodes = extract(node)
-          return unless count_node
-
-          add_offense(node) do |corrector|
-            corrector.replace(
-              node,
-              format(
-                'create_list(%<arguments>s)',
-                arguments: [
-                  factory_name_node,
-                  count_node,
-                  *extra_argument_nodes
-                ].map(&:source).join(', ')
-              )
-            )
-          end
-        end
-        alias on_numblock on_block
-
-        private
-
-        # @!method extract(node)
-        #   @param node [RuboCop::AST::BlockNode]
-        #   @return [Array(Integer, RuboCop::AST::SendNode)]
-        def_node_matcher :extract, <<~PATTERN
-          (block
-            (send
-              (const {nil? | cbase} :Array)
-              :new
-              $(int _)
-            )
-            (args)
-            (send
-              nil?
-              :create
-              $(sym _) $(...)?
-            )
-          )
-        PATTERN
-
-        # @param corrector [RuboCop::Cop::Corrector]
-        # @param node [RuboCop::AST::BlockNode]
-        # @return [void]
-        def autocorrect(corrector, node)
-          count_node, factory_name_node, extra_argument_nodes = extract(node)
-          corrector.replace(
-            node,
-            format(
-              'create_list(%<arguments>s)',
-              arguments: [
-                factory_name_node,
-                count_node,
-                *extra_argument_nodes
-              ].map(&:source).join(', ')
-            )
-          )
-        end
-      end
-    end
-  end
-end

data/lib/rubocop/cop/sevencop/redundant_existence_check.rb

@@ -1,188 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module Sevencop
-      # Identifies redundant existent check before file operation.
-      #
-      # @safety
-      #   This cop is unsafe because it can register a false positive where the check is truly needed.
-      #
-      # @example
-      #
-      #   # bad
-      #   FileUtils.mkdir(a) unless FileTest.exist?(a)
-      #
-      #   # good
-      #   FileUtils.mkdir_p(a)
-      #
-      #   # bad
-      #   FileUtils.rm(a) if File.exist?(a)
-      #
-      #   # good
-      #   FileUtils.rm_f(a)
-      #
-      class RedundantExistenceCheck < Base
-        extend AutoCorrector
-
-        CLASS_NAMES_FOR_EXIST = ::Set[
-          :File,
-          :FileTest,
-        ]
-
-        CLASS_NAMES_FOR_OPERATION = ::Set[
-          :File,
-          :FileUtils,
-        ]
-
-        METHOD_NAMES_FOR_MAKE = ::Set[
-          :makedirs,
-          :mkdir,
-          :mkdir_p,
-          :mkpath,
-          :touch,
-        ]
-
-        METHOD_NAMES_FOR_REMOVE = ::Set[
-          :delete,
-          :remove,
-          :remove_dir,
-          :remove_entry,
-          :remove_entry_secure,
-          :remove_file,
-          :rm,
-          :rm_f,
-          :rm_r,
-          :rm_rf,
-          :rmdir,
-          :rmtree,
-          :safe_unlink,
-          :unlink,
-        ]
-
-        METHOD_NAMES_FOR_EXIST = ::Set[
-          :exist?,
-          :exists?,
-        ]
-
-        METHOD_NAMES_FOR_FORCE_OPERATION = ::Set[
-          :makedirs,
-          :mkdir_p,
-          :mkpath,
-          :rm_f,
-          :rm_rf,
-          :rm_tree,
-          :safe_unlink,
-          :touch,
-        ]
-
-        METHOD_MAPPING_FOR_FORCE_REPLACEMENT = {
-          'FileUtils.mkdir' => 'FileUtils.mkdir_p',
-          'File.delete' => 'FileUtils.rm_f',
-          'File.unlink' => 'FileUtils.rm_f'
-        }.freeze
-
-        MSG = 'Avoid redundant existent check before file operation.'
-
-        # @!method make_unless_exist?(node)
-        def_node_matcher :make_unless_exist?, <<~PATTERN
-          (if
-            (send (const nil? CLASS_NAMES_FOR_EXIST) METHOD_NAMES_FOR_EXIST _)
-            nil?
-            (send (const nil? CLASS_NAMES_FOR_OPERATION) METHOD_NAMES_FOR_MAKE ...)
-          )
-        PATTERN
-
-        # @!method remove_if_exist?(node)
-        def_node_matcher :remove_if_exist?, <<~PATTERN
-          (if
-            (send (const nil? CLASS_NAMES_FOR_EXIST) METHOD_NAMES_FOR_EXIST _)
-            (send (const nil? CLASS_NAMES_FOR_OPERATION) METHOD_NAMES_FOR_REMOVE ...)
-            nil?
-          )
-        PATTERN
-
-        def on_if(node)
-          return unless redundant_on_if(node) || redundant_on_unless(node)
-
-          add_offense(node) do |corrector|
-            corrector.replace(
-              node.location.expression,
-              enforce(node)
-            )
-          end
-        end
-
-        private
-
-        def enforce(node)
-          if force_operation?(node)
-            node.if_branch.source
-          elsif force_replaceable_method?(node)
-            enforce_by_replacement(node)
-          else
-            enforce_by_force_option(node)
-          end
-        end
-
-        def enforce_by_force_option(node)
-          arguments = node.if_branch.arguments.map(&:source)
-          arguments << 'force: true' unless force_operation?(node)
-          format(
-            '%<receiver>s.%<method_name>s(%<arguments>s)',
-            arguments: arguments.join(', '),
-            method_name: node.if_branch.method_name,
-            receiver: node.if_branch.receiver.source
-          )
-        end
-
-        def enforce_by_replacement(node)
-          format(
-            '%<signature>s(%<arguments>s)',
-            arguments: node.if_branch.arguments.map(&:source).join(', '),
-            signature: METHOD_MAPPING_FOR_FORCE_REPLACEMENT[operation_method_signature(node)]
-          )
-        end
-
-        def force_operation?(node)
-          force_operation_method_name?(node) || force_operation_argument?(node)
-        end
-
-        def force_operation_argument?(node)
-          node.if_branch.last_argument.hash_type? &&
-            node.if_branch.last_argument.pairs.any? do |pair|
-              pair.key.value == :force && pair.value.true_type?
-            end
-        end
-
-        def force_operation_method_name?(node)
-          METHOD_NAMES_FOR_FORCE_OPERATION.include?(node.if_branch.method_name)
-        end
-
-        def redundant_on_if(node)
-          remove_if_exist?(node) && same_argument?(node)
-        end
-
-        def redundant_on_unless(node)
-          make_unless_exist?(node) && same_argument?(node)
-        end
-
-        def force_replaceable_method?(node)
-          METHOD_MAPPING_FOR_FORCE_REPLACEMENT.key?(operation_method_signature(node))
-        end
-
-        def operation_method_signature(node)
-          format(
-            '%<receiver>s.%<method_name>s',
-            method_name: node.if_branch.method_name,
-            receiver: node.if_branch.receiver.source
-          )
-        end
-
-        def same_argument?(node)
-          node.condition.first_argument == node.if_branch.first_argument
-        end
-      end
-    end
-  end
-end

data/lib/rubocop/cop/sevencop/to_s_with_argument.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-module RuboCop
-  module Cop
-    module Sevencop
-      # Identifies passing any argument to `#to_s`.
-      #
-      # @example
-      #
-      #   # bad
-      #   to_s(:delimited)
-      #
-      #   # good
-      #   to_formatted_s(:delimited)
-      #
-      class ToSWithArgument < Base
-        extend AutoCorrector
-
-        MSG = 'Use `to_formatted_s(...)` instead of `to_s(...)`.'
-
-        RESTRICT_ON_SEND = %i[to_s].freeze
-
-        # @!method to_s_with_any_argument?(node)
-        def_node_matcher :to_s_with_any_argument?, <<~PATTERN
-          (call _ :to_s _+)
-        PATTERN
-
-        def on_send(node)
-          return unless to_s_with_any_argument?(node)
-
-          add_offense(node) do |rewriter|
-            rewriter.replace(
-              node.loc.selector,
-              'to_formatted_s'
-            )
-          end
-        end
-        alias on_csend on_send
-      end
-    end
-  end
-end