rubocop-fourshark 0.6.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4b2d2e8d312ae14c7f582304f4b698783d89dcbaaabb97edbdbf8eb8637d782
-  data.tar.gz: 2c21cc3d147f2210f38dc5d694b72a3955da359518a037a9270d60c67bf352d3
+  metadata.gz: 582aea0059be3aad96e3b1017d419db000bb72f5a424684b4f90347de8359bc9
+  data.tar.gz: 685cb43ef7bbc16988a954b3821cc7a9a813bb98bf7777432da0960d4808d149
 SHA512:
-  metadata.gz: c314121bb42e4b79fe8e2ab9f6658c47f44042ef9cc0038bdd62a905ac8bbe986ff26f1ee67521e6d73ea2585ddaf0fd3d393cb2b54d853f16e115985cc769a6
-  data.tar.gz: 861c512174a3a30f7ecbe223467f41899ea8a418f631eb3e44c51c35715538a667be755e5e4bd7f87d7370827347fd75958cfb8cac2296c998ba2e4c1a82c058
+  metadata.gz: 1469cf2ff8d928ec8af532f2c09b0ca3fd35414c02105820ca519760d54d01ec177c172c79cf710a79e3314cafe4784c8c62f099bceb7b59753c51dc7ed5bb72
+  data.tar.gz: cf5dbfa5a4762d24715bb18b73224c92cb7251bced71e7003fa66d48d4d24be38bdcca4eb19e33e2d124e87dad74ec9a2cf53c890c69621dcefb2875aed33a07

data/CHANGELOG.md

@@ -1,3 +1,22 @@
+## [0.7.1] - 2026-08-05
+
+### Removed
+
+- `Style/ConditionalAssignment` configuration — the stock default governs assignment from a conditional again
+
+## [0.7.0] - 2026-08-05
+
+### Added
+
+- `Style/DisallowDelegate` — automatic delegation is flagged
+- `Style/DisallowTernary` — the ternary conditional is flagged
+
+### Changed
+
+- `Rails/Delegate` disabled — it asks for the macro `Style/DisallowDelegate` forbids
+- `Style/MultilineTernaryOperator`, `Style/NestedTernaryOperator` and `Style/TernaryParentheses` disabled — they shape a construct `Style/DisallowTernary` forbids outright
+- `Style/ConditionalAssignment` configured to `assign_inside_condition` — a conditional that decides a value assigns it in each branch, and ternaries are left to `Style/DisallowTernary`
+
 ## [0.6.0] - 2026-08-05
 
 ### Added

data/README.md

@@ -56,7 +56,7 @@ All cops are **enabled by default** the moment the plugin is activated. Cops sco
 
 Cop names follow RuboCop's empirical convention — a noun phrase describing the construct or smell, no `Disallow*`/`No*` prefixes — with two deliberate exceptions:
 
-- **`Style/DisallowSafeNavigation` / `Style/DisallowTry`** keep the `Disallow*` prefix. RuboCop has no idiom for "forbid a construct it otherwise permits" (it uses `EnforcedStyle` on one cop), and the natural noun name is already taken by a stock cop with the *opposite* intent — `Style/SafeNavigation` *converts to* `&.`. `Disallow*` is unambiguous and collision-free.
+- **`Style/DisallowDelegate` / `Style/DisallowSafeNavigation` / `Style/DisallowTernary` / `Style/DisallowTry`** keep the `Disallow*` prefix. RuboCop has no idiom for "forbid a construct it otherwise permits" (it uses `EnforcedStyle` on one cop), and the natural noun name is already taken by a stock cop with the *opposite* intent — `Style/SafeNavigation` *converts to* `&.`, `Rails/Delegate` *converts to* `delegate`. `Disallow*` is unambiguous and collision-free.
 - **`Rails/MandatoryInverseOf`** is named to distinguish it from stock `Rails/InverseOf`, which only flags associations where Active Record cannot auto-detect the inverse. Ours mandates `inverse_of` on *every* association — a strict superset — so the gem disables the stock cop (below).
 
 ### Stock cops disabled
@@ -65,14 +65,20 @@ Where a 4Shark cop supersedes or contradicts a stock cop, `config/default.yml` t
 
 | Disabled stock cop | Why |
 |---|---|
+| `Rails/Delegate` | contradicts `Style/DisallowDelegate` — it turns an explicit method into `delegate`, we forbid the macro |
 | `Rails/InverseOf` | superseded by `Rails/MandatoryInverseOf` (covers its cases and more) |
+| `Style/MultilineTernaryOperator` | superseded by `Style/DisallowTernary` — it shapes a construct we forbid outright |
+| `Style/NestedTernaryOperator` | superseded by `Style/DisallowTernary` — same |
 | `Style/SafeNavigation` | contradicts `Style/DisallowSafeNavigation` — it pushes `&.`, we forbid it |
+| `Style/TernaryParentheses` | superseded by `Style/DisallowTernary` — same |
 
 ### Style
 
 | Cop | Intent |
 |---|---|
+| `Style/DisallowDelegate` | Flags automatic delegation (`delegate`, `delegate_missing_to`, `def_delegator(s)`, `DelegateClass`). A `delegate` line is a macro call, not a definition, so `grep 'def foo'` and jump-to-definition find nothing — the community name for the smell is Fowler's Middle Man. Write the method, or let the caller ask the object that knows. |
 | `Style/DisallowSafeNavigation` | Flags safe navigation (`&.`). 4Shark rejects it — a `&.` chain silently swallows a `nil` that usually signals a real bug. Use an explicit conditional so the `nil` case is handled on purpose. |
+| `Style/DisallowTernary` | Flags the ternary conditional (`cond ? a : b`). `?` and `:` are punctuation that already mean other things in Ruby (`valid?`, `:symbol`), and the ternary hides a branch inside an expression where skimming misses it. Use an explicit `if`/`else`. |
 | `Style/DisallowTry` | Flags `try` / `try!`. Same rationale — it hides the `nil`/missing-method case instead of handling it explicitly. |
 
 ### Layout
@@ -103,7 +109,7 @@ Where a 4Shark cop supersedes or contradicts a stock cop, `config/default.yml` t
 | `RSpec/Dialect` | Use `let`, never `subject` or `let!` — every `subject`/`subject!`/`let!` is flagged in favor of a lazy `let` (force creation in an explicit `before`, not with `let!`). The implicit subject behind `is_expected` is a different method and is left untouched. Stock cop, configured via `PreferredMethods`. |
 | `RSpec/InverseOfMatcher` | Root models must assert `.inverse_of` in association specs; subclasses must not (it belongs to the parent). Scoped to `spec/models`. |
 | `RSpec/OverwrittenLet` | A `let`/`let!` must not override one defined in an outer example group — shadowing makes it ambiguous which value applies. Scenario-specific `let`s, and the same name across sibling contexts, are fine. |
-| `RSpec/ConditionalInLet` | A `let` must not contain conditional logic (`if`/`case`) — branch with separate `context`s instead. Ternaries are allowed. |
+| `RSpec/ConditionalInLet` | A `let` must not contain conditional logic (`if`/`case`) — branch with separate `context`s instead. A ternary inside a `let` is flagged by `Style/DisallowTernary`, not by this cop. |
 | `RSpec/FactoryBotInBefore` | Object creation belongs in `let`, not `before`. `before` is for actions, not for building the subjects under test. |
 
 ### FactoryBot (factories)

data/config/default.yml

@@ -22,11 +22,21 @@ Layout/MultilineMethodCallIndentation:
 Naming/RescuedExceptionsVariableName:
   PreferredName: exception
 
+Style/DisallowDelegate:
+  Description: 'Do not use automatic delegation. Write the method explicitly.'
+  Enabled: true
+  VersionAdded: '0.7.0'
+
 Style/DisallowSafeNavigation:
   Description: 'Do not use safe navigation (`&.`). Use explicit conditionals instead.'
   Enabled: true
   VersionAdded: '0.1.0'
 
+Style/DisallowTernary:
+  Description: 'Do not use the ternary conditional. Use an explicit `if`/`else` instead.'
+  Enabled: true
+  VersionAdded: '0.7.0'
+
 Style/DisallowTry:
   Description: 'Do not use `try` or `try!`. Use explicit conditionals instead.'
   Enabled: true
@@ -113,11 +123,27 @@ FactoryBot/AssociationInFactory:
   VersionAdded: '0.2.0'
 
 # Stock cops disabled because a 4Shark cop supersedes or contradicts them.
+# `Rails/Delegate` flags `def bar; foo.bar; end` and asks for `delegate :bar, to: :foo`,
+# the opposite of `Style/DisallowDelegate`.
 # `Rails/InverseOf` only flags when AR can't auto-detect the inverse;
 # `Rails/MandatoryInverseOf` covers it and more.
+# The three ternary-shaping cops are superseded by `Style/DisallowTernary`, which bans
+# every ternary — they could only ever fire on a line that is already an offense.
 # `Style/SafeNavigation` converts to `&.`, the opposite of `Style/DisallowSafeNavigation`.
+Rails/Delegate:
+  Enabled: false
+
 Rails/InverseOf:
   Enabled: false
 
+Style/MultilineTernaryOperator:
+  Enabled: false
+
+Style/NestedTernaryOperator:
+  Enabled: false
+
 Style/SafeNavigation:
   Enabled: false
+
+Style/TernaryParentheses:
+  Enabled: false

data/lib/rubocop/cop/factory_bot/association_in_factory.rb

@@ -35,7 +35,11 @@ module RuboCop
           body = node.body
           return unless body
 
-          statements = body.begin_type? ? body.children : [body]
+          statements = if body.begin_type?
+                         body.children
+                       else
+                         [body]
+                       end
 
           statements.each do |statement|
             add_offense(statement.loc.selector) if association_call?(statement)

data/lib/rubocop/cop/fourshark_cops.rb

@@ -10,5 +10,7 @@ require_relative 'rspec/conditional_in_let'
 require_relative 'rspec/factory_bot_in_before'
 require_relative 'rspec/inverse_of_matcher'
 require_relative 'rspec/overwritten_let'
+require_relative 'style/disallow_delegate'
 require_relative 'style/disallow_safe_navigation'
+require_relative 'style/disallow_ternary'
 require_relative 'style/disallow_try'

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

@@ -47,24 +47,22 @@ module RuboCop
 
             opposite_content = File.read(opposite_model_path)
 
-            expected_inverse =
-              if inverse_name
-                inverse_name.to_s
-              else
-                camel_to_snake(model_name).split('/').last.pluralize
-              end
+            expected_inverse = if inverse_name
+                                 inverse_name.to_s
+                               else
+                                 camel_to_snake(model_name).split('/').last.pluralize
+                               end
 
             unless /(belongs_to|has_many|has_one)\s+:#{expected_inverse}/.match?(opposite_content)
               add_global_offense("#{target_class} is missing opposite association for #{model_name}")
             end
           end
         rescue StandardError => exception
-          source_name =
-            if processed_source && processed_source.buffer
-              processed_source.file_path
-            else
-              'unknown'
-            end
+          source_name = if processed_source && processed_source.buffer
+                          processed_source.file_path
+                        else
+                          'unknown'
+                        end
 
           warn "Rails/BidirectionalAssociation failed on #{source_name}: #{exception.message}"
         end

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

@@ -42,12 +42,11 @@ module RuboCop
 
           add_offense(node.loc.selector) if missing_inverse_of
         rescue StandardError => exception
-          source_name =
-            if processed_source && processed_source.buffer
-              processed_source.file_path
-            else
-              'unknown'
-            end
+          source_name = if processed_source && processed_source.buffer
+                          processed_source.file_path
+                        else
+                          'unknown'
+                        end
 
           warn "Rails/MandatoryInverseOf failed on #{source_name}: #{exception.message}"
         end

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

@@ -39,7 +39,11 @@ module RuboCop
           body = node.body
           return unless body
 
-          statements = body.begin_type? ? body.children : [body]
+          statements = if body.begin_type?
+                         body.children
+                       else
+                         [body]
+                       end
 
           MACROS.each do |macro|
             calls = statements.select { |statement| macro_call?(statement, macro) }

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

@@ -7,7 +7,7 @@ module RuboCop
     module RSpec
       # Forbids conditional logic (`if`/`case`) inside a `let`/`let!` block.
       # Branching object setup belongs in separate `context` blocks, not in a
-      # single `let`. Ternaries are allowed.
+      # single `let`. A ternary is flagged by `Style/DisallowTernary`, not here.
       #
       # @example
       #   # bad

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

@@ -47,12 +47,11 @@ module RuboCop
             add_offense(node.loc.selector, message: MSG_FORBIDDEN_INVERSE) if chain_has_inverse_of?(node)
           end
         rescue StandardError => exception
-          source_name =
-            if processed_source && processed_source.buffer
-              processed_source.file_path
-            else
-              'unknown'
-            end
+          source_name = if processed_source && processed_source.buffer
+                          processed_source.file_path
+                        else
+                          'unknown'
+                        end
 
           warn "RSpec/InverseOfMatcher failed on #{source_name}: #{exception.message}"
         end
@@ -114,7 +113,9 @@ module RuboCop
 
         def model_source(model_name)
           path = File.join(Dir.pwd, 'app/models', "#{camel_to_snake(model_name)}.rb")
-          File.exist?(path) ? File.read(path) : nil
+          return File.read(path) if File.exist?(path)
+
+          nil
         end
 
         # The declared superclass of the specific class named by the spec — not the

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

@@ -65,7 +65,11 @@ module RuboCop
           body = group.body
           return false unless body
 
-          statements = body.begin_type? ? body.children : [body]
+          statements = if body.begin_type?
+                         body.children
+                       else
+                         [body]
+                       end
 
           statements.any? { |statement| let_named?(statement, name) }
         end
@@ -73,7 +77,12 @@ module RuboCop
         def let_named?(statement, name)
           return false unless statement.is_a?(::RuboCop::AST::Node)
 
-          send = statement.respond_to?(:send_node) ? statement.send_node : statement
+          send = if statement.respond_to?(:send_node)
+                   statement.send_node
+                 else
+                   statement
+                 end
+
           return false unless send.send_type?
           return false unless LET_METHODS.include?(send.method_name) && send.receiver.nil?
 
@@ -83,7 +92,9 @@ module RuboCop
 
         def let_name(node)
           first = node.first_argument
-          first && first.sym_type? ? first.value : nil
+          return first.value if first && first.sym_type?
+
+          nil
         end
       end
     end

data/lib/rubocop/cop/style/disallow_delegate.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+module RuboCop
+  module Cop
+    module Style
+      # Forbids automatic delegation — `delegate`, `delegate_missing_to`,
+      # `def_delegator`/`def_delegators` (Forwardable) and `DelegateClass`.
+      # A delegation macro is a call, not a definition, so `grep 'def foo'` and
+      # jump-to-definition find nothing; the community name for the smell is
+      # Fowler's Middle Man. Write the method, or delete the wrapper and let the
+      # caller ask the object that knows.
+      #
+      # @example
+      #   # bad
+      #   delegate :commission, to: :user_commission
+      #
+      #   # good
+      #   def commission
+      #     user_commission.commission
+      #   end
+      #
+      class DisallowDelegate < ::RuboCop::Cop::Base
+        MSG = 'Do not use automatic delegation. Write the method explicitly instead.'
+
+        RESTRICT_ON_SEND = %i[delegate delegate_missing_to def_delegator def_delegators DelegateClass].freeze
+
+        def on_send(node)
+          return unless node.receiver.nil?
+
+          add_offense(node)
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/cop/style/disallow_ternary.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+module RuboCop
+  module Cop
+    module Style
+      # Forbids the ternary conditional. `?` and `:` are punctuation that already
+      # mean other things in Ruby (`valid?`, `:symbol`), so the reader
+      # disambiguates characters before seeing the branch — and the ternary hides
+      # a branch inside an expression, where skimming misses it. Inside a hash or
+      # an argument list, extract a named local and branch on it.
+      #
+      # @example
+      #   # bad
+      #   status = saved ? :applied : :failed
+      #
+      #   # good
+      #   if saved
+      #     status = :applied
+      #   else
+      #     status = :failed
+      #   end
+      #
+      class DisallowTernary < ::RuboCop::Cop::Base
+        MSG = 'Do not use the ternary conditional. Use an explicit `if`/`else` instead.'
+
+        def on_if(node)
+          add_offense(node) if node.ternary?
+        end
+      end
+    end
+  end
+end

data/lib/rubocop/fourshark/version.rb

@@ -2,6 +2,6 @@
 
 module RuboCop
   module Fourshark
-    VERSION = '0.6.0'
+    VERSION = '0.7.1'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-fourshark
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Paulo Ribeiro
@@ -147,7 +147,9 @@ files:
 - lib/rubocop/cop/rspec/factory_bot_in_before.rb
 - lib/rubocop/cop/rspec/inverse_of_matcher.rb
 - lib/rubocop/cop/rspec/overwritten_let.rb
+- lib/rubocop/cop/style/disallow_delegate.rb
 - lib/rubocop/cop/style/disallow_safe_navigation.rb
+- lib/rubocop/cop/style/disallow_ternary.rb
 - lib/rubocop/cop/style/disallow_try.rb
 - lib/rubocop/fourshark.rb
 - lib/rubocop/fourshark/plugin.rb