rubocop-petal 1.5.0 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cdc12661793d2fbaa5d5634b86c23eb3e249a884ec4902406d66d969565fccd7
-  data.tar.gz: 8b1b59059300e38b056fab5b4c1d342ae3bf61dcd0db1204f140586c15dc889a
+  metadata.gz: b251a6179c5a736bae1a6cf26c2dbf68057acd2640d4fcf8c34c8dd258ee1da6
+  data.tar.gz: 234f960f5ea66302e1d32dc265a7cb19c9eda1757c5417fae2f9dec81c8cc047
 SHA512:
-  metadata.gz: 5c1494de93b6053ecac07431001234072212e9f62f9130b3dcc6ccefd3218cb3b454e5ace6de2c1e0d6d6aa91adf651d5454ebcfbca346b707579d45e855e77f
-  data.tar.gz: adb3d03f9419b9629ae7db9b7b1b4e811ce9d27a6a3b209f6ad6feaa4ca7edce47da7c698d3e9074cf25c3d452f022a2abebf0c80bb9b1b9764178614a571133
+  metadata.gz: ae6a1f7cd5f6389bdf766715b5802ebdaf9268da780e78863dc3d040cac9845308f76aa4a875eefde4ea5be7923cd0e5e5ec02b6c3ad9421ce35c7865144675b
+  data.tar.gz: 4cb53383bc8a1fa6b062395d0d5f6665320d470d11b7890ff2bee96d912c15329435ad87cd2e409b7db76663ef364150754859765a2991dab15d88e94e4f4ac2

data/config/default.yml

@@ -157,3 +157,4 @@ Sidekiq/PerformInline:
 Sidekiq/SymbolArgument:
   Description: "Prevent passing keywords arguments in worker's perform method"
   Enabled: true
+  SafeAutoCorrect: false

data/lib/rubocop/cop/rails/risky_activerecord_invocation.rb

@@ -5,7 +5,7 @@ module RuboCop
     module Rails
       # Disallow ActiveRecord calls that pass interpolated or added strings as an argument.
       # https://github.com/airbnb/ruby/blob/12435e8136d2adf710de999bc0f6bef01215df2c/rubocop-airbnb/lib/rubocop/cop/airbnb/risky_activerecord_invocation.rb
-      class RiskyActiverecordInvocation < Cop
+      class RiskyActiverecordInvocation < Base
         VULNERABLE_AR_METHODS = %i[
           delete_all
           destroy_all

data/lib/rubocop/cop/rspec/aggregate_examples/language.rb

@@ -6,7 +6,7 @@ module RuboCop
   module Cop
     module RSpec
       # RSpec public API methods that are commonly used in cops
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         module Language
           RSPEC = '{(const {nil? cbase} :RSpec) nil?}'
 

data/lib/rubocop/cop/rspec/aggregate_examples/line_range_helpers.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         # @internal Support methods for keeping newlines around examples.
         module LineRangeHelpers
           include RangeHelp

data/lib/rubocop/cop/rspec/aggregate_examples/matchers_with_side_effects.rb

@@ -5,7 +5,7 @@ require_relative 'language'
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         # When aggregated, the expectations will fail when not supposed to or
         # have a risk of not failing when expected to. One example is
         # `validate_presence_of :comment` as it leaves an empty comment after

data/lib/rubocop/cop/rspec/aggregate_examples/metadata_helpers.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         # @internal
         #   Support methods for example metadata.
         #   Examples with similar metadata are grouped.

data/lib/rubocop/cop/rspec/aggregate_examples/node_matchers.rb

@@ -5,7 +5,7 @@ require_relative 'language'
 module RuboCop
   module Cop
     module RSpec
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         # @internal
         #   Node matchers and searchers.
         module NodeMatchers

data/lib/rubocop/cop/rspec/aggregate_examples.rb

@@ -109,10 +109,11 @@ module RuboCop
       #     expect(number).to be_odd
       #   end
       #
-      class AggregateExamples < ::RuboCop::Cop::Cop
+      class AggregateExamples < Base
         include LineRangeHelpers
         include MetadataHelpers
         include NodeMatchers
+        extend AutoCorrector
 
         # Methods from the following modules override and extend methods of this
         # class, extracting specific behavior.
@@ -124,25 +125,21 @@ module RuboCop
           example_group_with_several_examples(node) do |all_examples|
             example_clusters(all_examples).each_value do |examples|
               examples[1..].each do |example|
-                add_offense(example,
-                            location: :expression,
-                            message: message_for(example, examples[0]))
+                add_offense(example, message: message_for(example, examples[0])) do |corrector|
+                  clusters = example_clusters_for_autocorrect(example)
+                  autocorrect(corrector, clusters)
+                end
               end
             end
           end
         end
 
-        def autocorrect(example_node)
-          clusters = example_clusters_for_autocorrect(example_node)
-          return if clusters.empty?
-
-          lambda do |corrector|
-            clusters.each do |metadata, examples|
-              range = range_for_replace(examples)
-              replacement = aggregated_example(examples, metadata)
-              corrector.replace(range, replacement)
-              examples[1..].map { |example| drop_example(corrector, example) }
-            end
+        def autocorrect(corrector, clusters)
+          clusters.each do |metadata, examples|
+            range = range_for_replace(examples)
+            replacement = aggregated_example(examples, metadata)
+            corrector.replace(range, replacement)
+            examples[1..].map { |example| drop_example(corrector, example) }
           end
         end
 

data/lib/rubocop/cop/sidekiq/const_argument.rb

@@ -25,8 +25,8 @@ module RuboCop
 
         include Helpers
 
-        MSG = 'Objects are not Sidekiq-serializable.'
-        MSG_SELF = '`self` is not Sidekiq-serializable.'
+        MSG = 'Objects are not native JSON types.'
+        MSG_SELF = '`self` is not a native JSON type.'
 
         def_node_matcher :initializer?, <<~PATTERN
           (send const :new)

data/lib/rubocop/cop/sidekiq/date_time_argument.rb

@@ -34,7 +34,7 @@ module RuboCop
       #   # good
       #   MyWorker.perform_async(Time.now.mday)
       #
-      class DateTimeArgument < ::RuboCop::Cop::Cop
+      class DateTimeArgument < Base
         DURATION_METHODS = %i[
           second
           seconds
@@ -61,8 +61,8 @@ module RuboCop
 
         include Helpers
 
-        DURATION_MSG = 'Durations are not Sidekiq-serializable; use the integer instead.'
-        MSG = 'Date/Time objects are not Sidekiq-serializable; convert to integers or strings instead.'
+        DURATION_MSG = 'Durations are not native JSON types; use the integer instead.'
+        MSG = 'Date/Time objects are not native JSON types; convert to integers or strings instead.'
         ALLOWED_METHODS = %i[to_i to_s].freeze
 
         def_node_matcher :rational_literal?, <<~PATTERN

data/lib/rubocop/cop/sidekiq/helpers.rb

@@ -50,20 +50,49 @@ module RuboCop
           node.each_ancestor(:class, :block).detect { |anc| sidekiq_worker?(anc) }
         end
 
-        def sidekiq_arguments(node)
+        def sidekiq_arguments(node, &block)
           return [] unless node.send_type? && (method_name = sidekiq_perform?(node))
+          return enum_for(:sidekiq_arguments, node) unless block
 
           # Drop the first argument for perform_at and perform_in
-          expand_arguments(method_name == :perform_async ? node.arguments : node.arguments[1..])
+          expand_nodes(method_name == :perform_async ? node.arguments : node.arguments[1..], &block)
         end
 
-        def expand_arguments(arguments)
-          arguments.flat_map do |argument|
-            if argument.array_type? || argument.hash_type?
-              expand_arguments(argument.values)
-            else
-              argument
-            end
+        def expand_nodes(nodes, &block)
+          return enum_for(:expand_nodes, nodes) unless block
+
+          nodes.each { |node| expand_node(node, &block) }
+        end
+
+        def expand_node(node, &block)
+          return enum_for(:expand_node, node) unless block
+
+          expand_hash_array_node(node, &block) || yield(node)
+        end
+
+        def expand_hash_array_node(node, &block)
+          expand_hash_node(node, &block) || expand_array_node(node, &block)
+        end
+
+        def expand_hash_node(node, &block)
+          return unless node.hash_type?
+          return enum_for(:expand_hash_node, node) unless block
+
+          node.pairs.each do |pair_node|
+            expand_hash_array_node(pair_node.key, &block) ||
+              expand_hash_array_node(pair_node.value, &block) ||
+              yield(pair_node)
+          end
+        end
+
+        def expand_array_node(node, &block)
+          return unless node.array_type?
+          return enum_for(:expand_array_node, node) unless block
+
+          if node.square_brackets?
+            expand_nodes(node.values, &block)
+          else
+            yield node
           end
         end
 

data/lib/rubocop/cop/sidekiq/symbol_argument.rb

@@ -14,17 +14,129 @@ module RuboCop
       #   MyWorker.perform_async(:foo)
       #
       #   # good
-      #   MyWorker.perform_async('foo')
+      #   MyWorker.perform_async("foo")
+      #
+      #   # bad
+      #   MyWorker.perform_async(%i(foo))
+      #
+      #   # good
+      #   MyWorker.perform_async(%w(foo))
+      #
+      #   # bad
+      #   MyWorker.perform_async([:foo])
+      #
+      #   # good
+      #   MyWorker.perform_async(["foo"]))
+      #
+      #   # bad
+      #   MyWorker.perform_async(foo: 1)
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => 1)
+      #
+      #   # bad
+      #   MyWorker.perform_async('foo' => :baz)
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => "baz")
+      #
+      #   # bad
+      #   MyWorker.perform_async('foo' => [:bar]
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => ["baz"])
+      #
+      #   # bad
+      #   MyWorker.perform_async('foo' => %i(baz)
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => %w(baz))
+      #
+      #   # bad
+      #   MyWorker.perform_async('foo' => { bar: %i(baz) })
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => { bar: %w(baz) })
+      #
+      #   # bad
+      #   MyWorker.perform_async('foo' => { bar: [:baz]) })
+      #
+      #   # good
+      #   MyWorker.perform_async('foo' => { bar: ["baz"] })
+      #
       class SymbolArgument < Base
+        extend AutoCorrector
+
         include Helpers
 
-        MSG = 'Symbols are not Sidekiq-serializable; use strings instead.'
+        MSG = 'Symbols are not native JSON types; use strings instead.'
 
         def on_send(node)
-          sidekiq_arguments(node).select(&:sym_type?).each do |argument|
-            add_offense(argument)
+          sidekiq_arguments(node)
+            .lazy
+            .select { |n| node_contains_symbol?(n) }
+            .each do |selected_node|
+            offense_data(selected_node) do |offense_node, replace_node, replace_value|
+              add_offense(offense_node) do |corrector|
+                corrector.replace(replace_node, replace_value)
+              end
+            end
           end
         end
+
+        private
+
+        def node_contains_symbol?(node)
+          node.sym_type? || symbol_percent_literal?(node) || pair_with_symbol?(node)
+        end
+
+        def symbol_percent_literal?(node)
+          node.array_type? && node.percent_literal?(:symbol)
+        end
+
+        def pair_with_symbol?(node)
+          node.pair_type? && (node.key.sym_type? || node.value.sym_type?)
+        end
+
+        def offense_data(node)
+          yield sym_replace_value(node) || array_replace_value(node) || pair_replace_value(node)
+        end
+
+        def sym_replace_value(node)
+          return unless node.sym_type?
+
+          [node, node, %("#{node.value}")]
+        end
+
+        def array_replace_value(node)
+          return unless node.array_type?
+
+          [node, node, "%w(#{node.values.map(&:value).join(' ')})"]
+        end
+
+        def pair_replace_value(node)
+          return unless node.pair_type?
+
+          pair_both_symbol(node) || pair_only_key_symbol(node) || pair_only_value_symbol(node)
+        end
+
+        def pair_both_symbol(node)
+          return unless node.key.sym_type? && node.value.sym_type?
+
+          [node, node, %("#{node.key.value}" => "#{node.value.value}")]
+        end
+
+        def pair_only_key_symbol(node)
+          return unless node.key.sym_type?
+
+          [node.key, node, %("#{node.key.value}" => #{node.value.source})]
+        end
+
+        def pair_only_value_symbol(node)
+          return unless node.value.sym_type?
+
+          [node.value, node.value, %("#{node.value.value}")]
+        end
       end
     end
   end

data/lib/rubocop/petal/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Petal
-    VERSION = '1.5.0'
+    VERSION = '1.7.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-petal
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.7.0
 platform: ruby
 authors:
 - Jean-Francis Bastien
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-11 00:00:00.000000000 Z
+date: 2025-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -16,70 +16,84 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.59'
+        version: '1.70'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.59'
+        version: '1.70'
 - !ruby/object:Gem::Dependency
   name: rubocop-factory_bot
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.24'
+        version: '2.26'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.24'
+        version: '2.26'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.20'
+        version: '1.23'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.20'
+        version: '1.23'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.23'
+        version: '2.28'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.23'
+        version: '2.28'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.25'
+        version: '3.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.25'
+        version: '3.3'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec_rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.30'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.30'
 description:
 email:
 - jfbastien@petalmd.com