sevencop 0.5.1 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db281c40f5207dfc9759c52c60d4e2799d772e8d612364c4afa9b72eb935c90a
-  data.tar.gz: f84b8ca1a7d8cdb598c170b9c04b98b2358750ced29a8d3b3fb011b8a629449c
+  metadata.gz: 58fe0254e47cc59376f53d38640a163d0ba80dfba6f3f41998350b1b049010fd
+  data.tar.gz: 969301c9b0ef1af7ab3170c63fd72f5f8d6b095de2c84f2818917a003afe59cb
 SHA512:
-  metadata.gz: 4597d82f8b5dd02a575d4720c10f495e6ebf16bfe47ef88c9557e5cb474c83bc35505861e2a1e5fdbc3b1d062fac6a06d3cfe94a05593ee9638a0aecf1034125
-  data.tar.gz: 3e9c4206cec1f67a0ab9176814f4405fe6d7fbe83c8571d0759788206b67569a8ad144f0cb934dbc2ddf30df29d529906f7160c5b029fbc13a60464229f2a8d6
+  metadata.gz: be7586aef6236eab74b74494893ed7af665f01c4879ea36f63d6695250170174175859c439e7a6dd39775095b1ae210fa80a8d4e2e4f8a3c177e6ccc060c9c81
+  data.tar.gz: 7003edbafb17736c0e0bdfaaf65c1e1340fb45b684b31b1d573d6977f19e8c192700fb8bb4b57d0003f32753d2438110b6de937da42832a06641118c6e2b0c08

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## Unreleased
 
+## 0.6.0 - 2022-07-04
+
+### Added
+
+- Add `Sevencop/HashLiteralOrder` cop.
+
+### Removed
+
+- Remove unnecessary method names check on `Sevencop/OrderField`.
+
 ## 0.5.1 - 2022-06-28
 
 ### Fixed

data/Gemfile.lock

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

data/README.md

@@ -57,6 +57,26 @@ belongs_to :group, options
 
 This is useful for migration of `config.active_record.belongs_to_required_by_default`.
 
+### Sevencop/HashLiteralOrder
+
+Sort Hash literal by key.
+
+```ruby
+# bad
+{
+  b: 1,
+  a: 1,
+  c: 1
+}
+
+# good
+{
+  a: 1,
+  b: 1,
+  c: 1
+}
+```
+
 ### Sevencop/OrderField
 
 Identifies a String including "field" is passed to `order` or `reorder`.

data/config/default.yml

@@ -5,20 +5,29 @@ Sevencop/BelongsToOptional:
   Safe: false
   VersionAdded: '0.5'
 
+Sevencop/HashLiteralOrder:
+  Description: |
+    Sort Hash literal entries by key.
+  Enabled: false
+  VersionAdded: '0.6'
+
 Sevencop/OrderField:
-  Description: Wrap safe SQL String by `Arel.sql`.
+  Description: |
+    Wrap safe SQL String by `Arel.sql`.
   Enabled: false
   Safe: false
   VersionAdded: '0.4'
 
 Sevencop/RedundantExistenceCheck:
-  Description: Avoid redundant existent check before file operation.
+  Description: |
+    Avoid redundant existent check before file operation.
   Enabled: false
   Safe: false
   VersionAdded: '0.1'
 
 Sevencop/UniquenessValidatorExplicitCaseSensitivity:
-  Description: Specify :case_sensitivity option on use of UniquenessValidator.
+  Description: |
+    Specify :case_sensitivity option on use of UniquenessValidator.
   Enabled: false
   Safe: false
   VersionAdded: '0.3'

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

@@ -0,0 +1,105 @@
+# frozen_string_literal: true
+
+module RuboCop
+  module Cop
+    module Sevencop
+      # Sort Hash literal entries by key.
+      #
+      # @example
+      #
+      #   # bad
+      #   {
+      #     b: 1,
+      #     a: 1,
+      #     c: 1
+      #   }
+      #
+      #   # good
+      #   {
+      #     a: 1,
+      #     b: 1,
+      #     c: 1
+      #   }
+      #
+      class HashLiteralOrder < Base
+        extend AutoCorrector
+
+        MSG = 'Sort Hash literal entries by key.'
+
+        def_node_matcher :hash_literal?, <<~PATTERN
+          (hash
+            (pair
+              {sym | str}
+              _
+            )+
+          )
+        PATTERN
+
+        # @param [RuboCop::AST::HashNode] node
+        def on_hash(node)
+          return unless hash_literal?(node)
+
+          return if sorted?(node)
+
+          add_offense(node) do |corrector|
+            corrector.replace(
+              node,
+              autocorrect(node)
+            )
+          end
+        end
+
+        private
+
+        # @param [RuboCop::AST::HashNode] node
+        # @return [String]
+        def autocorrect(node)
+          whitespace = whitespace_leading(node)
+          [
+            '{',
+            whitespace,
+            sort(node.pairs).map(&:source).join(",#{whitespace}"),
+            whitespace_ending(node),
+            '}'
+          ].join
+        end
+
+        # @param [RuboCop::AST::HashNode] node
+        # @return [Boolean]
+        def multi_line?(node)
+          node.source.include?("\n")
+        end
+
+        # @param [Array<RuboCop::AST::PairNode>] pairs
+        # @return [Array<RuboCop::AST::PairNode>]
+        def sort(pairs)
+          pairs.sort_by do |pair|
+            pair.key.value
+          end
+        end
+
+        # @param [RuboCop::AST::HashNode] node
+        # @return [Boolean]
+        def sorted?(node)
+          node.pairs.map(&:source) == sort(node.pairs).map(&:source)
+        end
+
+        # @param [RuboCop::AST::HashNode] node
+        # @return [String]
+        #   {    a: 1,    b: 1  }
+        #                     ^^
+        def whitespace_ending(node)
+          node.source[node.pairs[-1].location.expression.end_pos...node.location.end.begin_pos]
+        end
+
+        # @param [RuboCop::AST::HashNode] node
+        # @return [String]
+        #   {    a: 1,    b: 1  }
+        #    ^^^^
+        def whitespace_leading(node)
+          node.source[node.location.begin.end_pos...node.pairs[0].location.expression.begin_pos]
+        end
+      end
+    end
+  end
+end

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

@@ -32,11 +32,9 @@ module RuboCop
           reorder
         ].freeze
 
-        ORDER_METHOD_NAMES = RESTRICT_ON_SEND.to_set.freeze
-
         def_node_matcher :order_with_field?, <<~PATTERN
           (send
-            _ ORDER_METHOD_NAMES
+            _ _
             {
               (str /field\(.+\)/) |
               (dstr <(str /field\(.+\)/) ...>)

data/lib/sevencop/version.rb

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

data/lib/sevencop.rb

@@ -7,6 +7,7 @@ require_relative 'sevencop/inject'
 require_relative 'sevencop/version'
 
 require_relative 'rubocop/cop/sevencop/belongs_to_optional'
+require_relative 'rubocop/cop/sevencop/hash_literal_order'
 require_relative 'rubocop/cop/sevencop/order_field'
 require_relative 'rubocop/cop/sevencop/redundant_existence_check'
 require_relative 'rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: sevencop
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.0
 platform: ruby
 authors:
 - Ryo Nakamura
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-06-27 00:00:00.000000000 Z
+date: 2022-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -42,6 +42,7 @@ files:
 - Rakefile
 - config/default.yml
 - lib/rubocop/cop/sevencop/belongs_to_optional.rb
+- lib/rubocop/cop/sevencop/hash_literal_order.rb
 - lib/rubocop/cop/sevencop/order_field.rb
 - lib/rubocop/cop/sevencop/redundant_existence_check.rb
 - lib/rubocop/cop/sevencop/uniqueness_validator_explicit_case_sensitivity.rb