carwow_rubocop 6.9.0 → 6.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e2a4da2858ed109b02d01111b35e24184159d2d0c592855a4bc57bbd8d0d2f04
-  data.tar.gz: 8bf60cd532eae6c3a401eecb64f2460ba377c7573ba67657db2f792a1ac6b212
+  metadata.gz: 978ba9d52dc3a0f7c586dab02666847b3e7ac2bcc62990909b4ecab964e794d8
+  data.tar.gz: 35c9f5b58b396765fef73fa76e622ce28029843a96fc25f6f434897a9333cdcd
 SHA512:
-  metadata.gz: 57c7b0a76743efcafe2da17ad73254f484e83ae6d7ccd7b855f8b3f60648ff64fb83de0a63d091153b69b07e72f8df8121e2736ac88bb1d64319a5f64b6f9a23
-  data.tar.gz: 157af413e04a76aa595103680d403d846d3d9ef9018cf4b41c123a6fff97feb73e88fc55b3ebf057bd68b5cd2cc13254ecfcb6ace01d6d06e7b40f19b1166b44
+  metadata.gz: 51ca0f3638fd002ef701414af166ff97f63ab65f28031e385130e3813721b534ef88e75ecb3b36988d303c040a0df710552d2e210b4b242387df97a10dae2f43
+  data.tar.gz: d4a8e1e9941469ac1a1a3f88a803ddcf38c2f8992f1aefd14a0ffdb97ceda74af045de2717c49e196c072fbedc162779014457ac14ecd0eb8374c04f575d924e

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    carwow_rubocop (6.9.0)
+    carwow_rubocop (6.11.0)
       rubocop (>= 1.75)
       rubocop-factory_bot
       rubocop-performance

data/config/rubocop-carwow.yml

@@ -32,6 +32,10 @@ Carwow/NoStubbingBusinessEvent:
     - "**/*_spec.rb"
     - "**/spec/**/*"
 
+Carwow/NoStubbingHoneycombExtraData:
+  Enabled: true
+  Include: *spec_only
+
 Carwow/AddColumnWithComment:
   Enabled: true
   Include: &migrations_only

data/lib/rubocop/carwow/version.rb

@@ -1,5 +1,5 @@
 module RuboCop
   module Carwow
-    VERSION = '6.9.0'.freeze
+    VERSION = '6.11.0'.freeze
   end
 end

data/lib/rubocop/cop/carwow/jobs_queue_name_style.rb

@@ -24,6 +24,11 @@ module RuboCop
       #     queue_as :with_symbol
       #   end
       #
+      #   # bad - literals returned from a block are checked too
+      #   class Bad < ApplicationJob
+      #     queue_as { urgent? ? :with_symbol : "with_string" }
+      #   end
+      #
       #
       class JobsQueueNameStyle < ::RuboCop::Cop::Base
         extend AutoCorrector
@@ -42,27 +47,49 @@ module RuboCop
         # optimization: don't call `on_send` unless the method name is in this list
         RESTRICT_ON_SEND = %i[queue_as].freeze
         def on_send(node)
-          _, _queue_as, queue_name = *node
+          queue_names(node).each do |queue_name|
+            next if preferred_style?(queue_name)
 
-          return if prefer_symbol? && queue_name.sym_type?
-          return if prefer_string? && queue_name.str_type?
+            add_offense(queue_name, message: (prefer_symbol? ? MSG : MSG_STRING)) do |corrector|
+              content, _other = *queue_name
+              fix = prefer_symbol? ? ":#{content}" : content.to_s.inspect
 
-          add_offense(queue_name, message: (prefer_symbol? ? MSG : MSG_STRING)) do |corrector|
-            content, _other = *queue_name
-            fix = prefer_symbol? ? ":#{content}" : content.to_s.inspect
-
-            corrector.replace(queue_name, fix)
+              corrector.replace(queue_name, fix)
+            end
           end
         end
 
         private
 
-        def prefer_symbol?
-          !cop_config.fetch('QueueNameAs', 'Symbol').casecmp('string').zero?
+        def queue_names(node)
+          return [node.first_argument].compact unless node.block_node
+
+          # The queue is computed at runtime, so only the literals the block can return are checked.
+          possible_return_values(node.block_node.body).select { |value| value&.type?(:sym, :str) }
         end
 
-        def prefer_string?
-          !prefer_symbol?
+        def possible_return_values(node)
+          return [node] unless node
+
+          case node.type
+          when :begin then possible_return_values(node.children.last)
+          when :if then [node.if_branch, node.else_branch].flat_map { |branch| possible_return_values(branch) }
+          when :case then possible_case_return_values(node)
+          else [node]
+          end
+        end
+
+        def possible_case_return_values(node)
+          branches = node.when_branches.map(&:body) + [node.else_branch]
+          branches.flat_map { |branch| possible_return_values(branch) }
+        end
+
+        def preferred_style?(queue_name)
+          prefer_symbol? ? queue_name.sym_type? : queue_name.str_type?
+        end
+
+        def prefer_symbol?
+          !cop_config.fetch('QueueNameAs', 'Symbol').casecmp('string').zero?
         end
       end
     end

data/lib/rubocop/cop/carwow/no_stubbing_honeycomb_extra_data.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+module RuboCop
+  module Cop
+    module Carwow
+      class NoStubbingHoneycombExtraData < ::RuboCop::Cop::Base
+        MSG = "Don't stub or spy on `append_extra_honeycomb_data`. Other code (ActiveSupport::Notifications " \
+              'subscribers, product area tagging, Faraday middleware) appends to the same store, so expectations ' \
+              'on individual calls break when calls are split, merged or reordered. Do the work, then assert ' \
+              "on the data instead: `expect('some.key' => value).to have_been_appended_to_honeycomb_event`, " \
+              "or `expect { perform }.to append_to_honeycomb_event('some.key' => value)` to only check what " \
+              'the block appended. Both come from `CarwowCore::Spec::HoneycombHelper` in `carwow_core/spec`.'
+
+        RESTRICT_ON_SEND = %i[receive have_received receive_messages].freeze
+
+        def_node_matcher :stubs_append?, <<~PATTERN
+          {
+            (send nil? {:receive :have_received} {(sym :append_extra_honeycomb_data) (str "append_extra_honeycomb_data")})
+            (send nil? :receive_messages (hash <(pair (sym :append_extra_honeycomb_data) _) ...>))
+          }
+        PATTERN
+
+        def on_send(node)
+          add_offense(node) if stubs_append?(node)
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: carwow_rubocop
 version: !ruby/object:Gem::Version
-  version: 6.9.0
+  version: 6.11.0
 platform: ruby
 authors:
 - carwow Developers
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-13 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -165,6 +164,7 @@ files:
 - lib/rubocop/cop/carwow/jobs_queue_name_style.rb
 - lib/rubocop/cop/carwow/no_change_matcher_with_browser_action.rb
 - lib/rubocop/cop/carwow/no_stubbing_business_event.rb
+- lib/rubocop/cop/carwow/no_stubbing_honeycomb_extra_data.rb
 - lib/rubocop/cop/carwow/no_vehicle_brand_model_sideload.rb
 - lib/rubocop/cop/carwow/product_area_required.rb
 homepage: https://github.com/carwow/carwow_rubocop
@@ -173,7 +173,6 @@ licenses:
 metadata:
   allowed_push_host: https://rubygems.org
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -188,8 +187,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.20
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: carwow's rubocop configuration
 test_files: []