aroundhome_cops 8.1.0 → 9.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b37a7b7b0f18f2e07adc756a65d0d4b8fc1bdef36926e6d22f4530fde3873737
-  data.tar.gz: da2037e0fb471c000ea7e9f79a94e691c00b1c032603471c0f082f689cc54f49
+  metadata.gz: 90ef4f831026e674bae86f33196a592cece157d12f0858d551dc0c7626ac966b
+  data.tar.gz: 0b290909ba03bc58fb59985896fa8f31c703d2a1c2b90dcd2d00121061ed8a4f
 SHA512:
-  metadata.gz: 79ee0fca9b7e708f65e32c05c41213e56871fdf6ac6106f7ec2bb9b779f7a3f9a0ea94857127c409e54afc8bd742922a9866b6dc314920a75c04e051b952909e
-  data.tar.gz: 51a52882e0592ad66bc47b3226efb299e0f1a02138effcde06541de6133c3223cfed5cdd690680af3cd74050a2f910d7d1df2c09ae787084f354de1b336bdc20
+  metadata.gz: 2a712a3873aba9f2e6d696b44099d86e0c5cfb43cb9ad0fa633f96dfda32557a36e902aa236d48657b71b3b2ef2a3a5e453f67039d32519a3d15c4b24111d24d
+  data.tar.gz: 90bcaaf95de10536707be856b109101b5e0a673728ace2f2fe94622cc6310959d15660e248eec8e4abb1c9f268e767d72f997ea2897574951431d9d33610630e

data/.gitlab-ci.yml

@@ -17,6 +17,7 @@ rubocop:
   stage: test
   script:
     - bundle exec rubocop
+    - bundle exec rubocop --config .rubocop-rails-smoke.yml
 
 publish_gem:
   stage: publish

data/.rubocop-rails-smoke.yml

@@ -0,0 +1,12 @@
+# Loads rails.yml so a bad cop name fails CI. This gem is not a Rails app.
+plugins:
+  - rubocop-rails
+
+inherit_gem:
+  aroundhome_cops:
+    - default.yml
+    - rails.yml
+
+AllCops:
+  Exclude:
+    - "**/*"

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## Version 9.0.0 (2026-11-10)
+
+* Pin RuboCop and plugins to the current releases and enable the pending cops through those versions
+* Set `Metrics/ClassLength` to 175 so one cohesive class can stay in one file
+* Set `Metrics/ModuleLength` to 100 explicitly
+* Pin `Layout/LineLength` to 120
+* Add optional `rails.yml` for Rails API mistakes. `rubocop-rails` is a dependency and is not loaded from `default.yml`
+* Say in `rails.yml` why each Rails cop that stays off is off
+
 ## Version 8.1.0 (2026-10-27)
 
 * Raise `RSpec/MultipleExpectations` `Max` from 1 to 3

data/README.md

@@ -23,6 +23,19 @@ inherit_gem:
     - default.yml
 ```
 
+Rails apps also inherit `rails.yml`. `rubocop-rails` is already installed
+through this gem. Add the plugin so those cops load:
+
+```yml
+plugins:
+  - rubocop-rails
+
+inherit_gem:
+  aroundhome_cops:
+    - default.yml
+    - rails.yml
+```
+
 ## Customizing rules
 
 The snippet above is everything needed to use this configuration as default.
@@ -39,6 +52,6 @@ inherit_gem:
 inherit_from: .rubocop_todo.yml
 
 # This project agreed on a different line length than the rest of the company
-Metrics/LineLength:
-  Max: 120
+Layout/LineLength:
+  Max: 100
 ```

data/aroundhome_cops.gemspec

@@ -20,9 +20,10 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.7.0'
 
-  spec.add_dependency 'rubocop', '~> 1.72'
-  spec.add_dependency 'rubocop-factory_bot', '~> 2.27'
-  spec.add_dependency 'rubocop-performance', '~> 1.21'
-  spec.add_dependency 'rubocop-rake', '~> 0.6'
-  spec.add_dependency 'rubocop-rspec', '~> 3.5'
+  spec.add_dependency 'rubocop', '~> 1.92.2'
+  spec.add_dependency 'rubocop-factory_bot', '~> 2.28.0'
+  spec.add_dependency 'rubocop-performance', '~> 1.27.0'
+  spec.add_dependency 'rubocop-rails', '~> 2.38.0'
+  spec.add_dependency 'rubocop-rake', '~> 0.7.1'
+  spec.add_dependency 'rubocop-rspec', '~> 3.10.2'
 end

data/default.yml

@@ -24,7 +24,6 @@ AllCops:
     - "db/migrate/2023*.rb"
   SuggestExtensions:
     rubocop-rails: false
-    rubocop-factory_bot: false
 
 # While we like to write blocks like the following
 # expect { add_special_project }.not_to change { SpecialProject.count }
@@ -44,7 +43,9 @@ Metrics:
 
 # See above (Metrics). This cop was previously called Metrics/LineLength
 # and therefore disabled as well for migrations.
+# Pinned at RuboCop's 120 so a later default cannot move the company value.
 Layout/LineLength:
+  Max: 120
   Exclude:
     - "db/migrate/*.rb"
 
@@ -76,9 +77,28 @@ Metrics/BlockLength:
 Metrics/MethodLength:
   Max: 16
 
+# 175 is about nine methods, which a reviewer can hold in one pass.
+# CountAsOne keeps hashes and heredocs from spending that budget.
+Metrics/ClassLength:
+  Max: 175
+  CountAsOne:
+    - array
+    - hash
+    - heredoc
+
+# A mixin is read against a host class that is often outside the diff.
+Metrics/ModuleLength:
+  Max: 100
+  CountAsOne:
+    - array
+    - hash
+    - heredoc
+
+# Names and German words in comments are not ASCII. Forcing ASCII makes people strip them.
 Style/AsciiComments:
   Enabled: false
 
+# YARD on every class and module flags application code that is already named clearly.
 Style/Documentation:
   Enabled: false
 
@@ -411,6 +431,126 @@ Style/SendWithLiteralMethodName: # new in 1.64
   Enabled: true
 Style/SuperArguments: # new in 1.64
   Enabled: true
+Gemspec/AddRuntimeDependency: # new in 1.65
+  Enabled: true
+Gemspec/AttributeAssignment: # new in 1.77
+  Enabled: true
+Layout/EmptyLinesAfterModuleInclusion: # new in 1.79
+  Enabled: true
+Lint/ArgumentMismatch: # new in 1.90
+  Enabled: true
+Lint/ArrayLiteralInRegexp: # new in 1.71
+  Enabled: true
+Lint/ConstantReassignment: # new in 1.70
+  Enabled: true
+Lint/DataDefineOverride: # new in 1.85
+  Enabled: true
+Lint/DeprecatedReference: # new in 1.89
+  Enabled: true
+Lint/DuplicateSetElement: # new in 1.67
+  Enabled: true
+Lint/HashNewWithKeywordArgumentsAsDefault: # new in 1.69
+  Enabled: true
+Lint/MisplacedMagicComment: # new in 1.91
+  Enabled: true
+Lint/NameTypo: # new in 1.89
+  Enabled: true
+Lint/NumericOperationWithConstantResult: # new in 1.69
+  Enabled: true
+Lint/RedundantTypeConversion: # new in 1.72
+  Enabled: true
+Lint/SharedMutableDefault: # new in 1.70
+  Enabled: true
+Lint/SuperArgumentMismatch: # new in 1.90
+  Enabled: true
+Lint/SuppressedExceptionInNumberConversion: # new in 1.72
+  Enabled: true
+Lint/UnescapedBracketInRegexp: # new in 1.68
+  Enabled: true
+Lint/UnreachablePatternBranch: # new in 1.85
+  Enabled: true
+Lint/UselessConstantScoping: # new in 1.72
+  Enabled: true
+Lint/UselessDefaultValueArgument: # new in 1.76
+  Enabled: true
+Lint/UselessDefined: # new in 1.69
+  Enabled: true
+Lint/UselessNumericOperation: # new in 1.66
+  Enabled: true
+Lint/UselessOr: # new in 1.76
+  Enabled: true
+Naming/PredicateMethod: # new in 1.76
+  Enabled: true
+Style/AmbiguousEndlessMethodDefinition: # new in 1.68
+  Enabled: true
+Style/ArrayIntersectWithSingleElement: # new in 1.81
+  Enabled: true
+Style/BitwisePredicate: # new in 1.68
+  Enabled: true
+Style/CollectionQuerying: # new in 1.77
+  Enabled: true
+Style/CombinableDefined: # new in 1.68
+  Enabled: true
+Style/ComparableBetween: # new in 1.74
+  Enabled: true
+Style/DigChain: # new in 1.69
+  Enabled: true
+Style/DirectiveScope: # new in 1.90
+  Enabled: true
+Style/EmptyClassDefinition: # new in 1.84
+  Enabled: true
+Style/EmptyStringInsideInterpolation: # new in 1.76
+  Enabled: true
+Style/FileNull: # new in 1.69
+  Enabled: true
+Style/FileOpen: # new in 1.85
+  Enabled: true
+Style/FileTouch: # new in 1.69
+  Enabled: true
+Style/HashFetchChain: # new in 1.75
+  Enabled: true
+Style/HashSlice: # new in 1.71
+  Enabled: true
+Style/ItAssignment: # new in 1.70
+  Enabled: true
+Style/ItBlockParameter: # new in 1.75
+  Enabled: true
+Style/KeywordArgumentsMerging: # new in 1.68
+  Enabled: true
+Style/MapJoin: # new in 1.85
+  Enabled: true
+Style/ModuleMemberExistenceCheck: # new in 1.82
+  Enabled: true
+Style/NegativeArrayIndex: # new in 1.84
+  Enabled: true
+Style/OneClassPerFile: # new in 1.85
+  Enabled: true
+Style/PartitionInsteadOfDoubleSelect: # new in 1.85
+  Enabled: true
+Style/PredicateWithKind: # new in 1.85
+  Enabled: true
+Style/ReduceToHash: # new in 1.85
+  Enabled: true
+Style/RedundantArrayFlatten: # new in 1.76
+  Enabled: true
+Style/RedundantFormat: # new in 1.72
+  Enabled: true
+Style/RedundantInterpolationUnfreeze: # new in 1.66
+  Enabled: true
+Style/RedundantMinMaxBy: # new in 1.85
+  Enabled: true
+Style/ReverseFind: # new in 1.84
+  Enabled: true
+Style/SafeNavigationChainLength: # new in 1.68
+  Enabled: true
+Style/SelectByKind: # new in 1.85
+  Enabled: true
+Style/SelectByRange: # new in 1.85
+  Enabled: true
+Style/TallyMethod: # new in 1.85
+  Enabled: true
+Style/TimeNow: # new in 1.90
+  Enabled: true
 
 # ----------------- Rubocop-Rspec forced enablements --------------
 RSpec/ExcessiveDocstringSpacing: # (new in 2.5)
@@ -473,6 +613,16 @@ RSpec/RedundantAround: # new in 2.19
   Enabled: true
 RSpec/ExpectInLet: # new in 2.30
   Enabled: true
+RSpec/DiscardedMatcher: # new in 3.10
+  Enabled: true
+RSpec/IncludeExamples: # new in 3.6
+  Enabled: true
+RSpec/LeakyLocalVariable: # new in 3.8
+  Enabled: true
+RSpec/MatchWithSimpleRegex: # new in 3.10
+  Enabled: true
+RSpec/Output: # new in 3.9
+  Enabled: true
 
 # ----------------- Rubocop-Performance forced enablements --------------
 Performance/AncestorsInclude: # new in 1.7
@@ -511,8 +661,12 @@ Performance/StringIdentifierArgument: # new in 1.13
   Enabled: true
 Performance/StringInclude: # new in 1.7
   Enabled: true
+Performance/StringBytesize: # new in 1.23
+  Enabled: true
 Performance/Sum: # new in 1.8
   Enabled: true
+Performance/ZipWithoutBlock: # new in 1.24
+  Enabled: true
 
 # ----------------- FactoryBot forced enablements --------------
 FactoryBot/SyntaxMethods: # new in 2.7

data/lib/aroundhome_cops.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AroundhomeCops
-  VERSION = '8.1.0'
+  VERSION = '9.0.0'
 end

data/rails.yml

@@ -0,0 +1,648 @@
+# The Rails department stays off. The cops further down catch a wrong API.
+# Every other cop is listed with the reason it stays off. A Rails cop added
+# upstream later stays off until this file explains it.
+Rails:
+  Enabled: false
+
+Rails/SaveBang:
+  Enabled: true
+Rails/SkipsModelValidations:
+  Enabled: true
+Rails/TransactionExitStatement:
+  Enabled: true
+Rails/FindBy:
+  Enabled: true
+Rails/FindEach:
+  Enabled: true
+Rails/WhereExists:
+  Enabled: true
+Rails/InverseOf:
+  Enabled: true
+Rails/HasManyOrHasOneDependent:
+  Enabled: true
+Rails/UniqueValidationWithoutIndex:
+  Enabled: true
+Rails/TimeZone:
+  Enabled: true
+Rails/Date:
+  Enabled: true
+Rails/OutputSafety:
+  Enabled: true
+Rails/ReversibleMigration:
+  Enabled: true
+Rails/BulkChangeTable:
+  Enabled: true
+Rails/NotNullColumn:
+  Enabled: true
+Rails/AddColumnIndex:
+  Enabled: true
+
+# `flash` before `render` is correct when the next request must show it.
+# Agents switch it to `flash.now` and the message is gone after this response.
+Rails/ActionControllerFlashBeforeRender:
+  Enabled: false
+
+# A functional controller test can be the test that belongs on the action.
+# Agents rewrite it as an integration test and break assertions that passed.
+Rails/ActionControllerTestCase:
+  Enabled: false
+
+# `before_filter` and `before_action` are the same callback.
+# Agents rename a working filter and drop `only` or `except` while editing it.
+Rails/ActionFilter:
+  Enabled: false
+
+# The order of actions in the file does not change which action runs.
+# Agents reshuffle the controller and the diff hides the real edit.
+Rails/ActionOrder:
+  Enabled: false
+
+# `update_attributes` and `update` are the same write.
+# Agents spend the change on the alias and leave a failed save unchecked.
+Rails/ActiveRecordAliases:
+  Enabled: false
+
+# Same-type callbacks run in source order. Autocorrect swaps declarations.
+# Two `before_save` hooks can change which one runs first.
+Rails/ActiveRecordCallbacksOrder:
+  Enabled: false
+
+# Overriding `save` is sometimes the method that has to wrap `super`.
+# Agents move that body into a callback and the `super` path is gone.
+Rails/ActiveRecordOverride:
+  Enabled: false
+
+# `starts_with?` and `start_with?` do the same check.
+# Agents rename a call that already says what it checks.
+Rails/ActiveSupportAliases:
+  Enabled: false
+
+# An initializer that reopens a loaded Rails class is already on time.
+# Agents wrap it in `on_load` and the patch waits for a hook that already fired.
+Rails/ActiveSupportOnLoad:
+  Enabled: false
+
+# A second `after_*_commit :log_action` replaces the first. Collapsing both into
+# `after_save_commit` also runs on saves that are neither create nor update.
+Rails/AfterCommitOverride:
+  Enabled: false
+
+# A controller can subclass an API or engine base, not `ApplicationController`.
+# Agents retarget the superclass and it inherits the wrong filters.
+Rails/ApplicationController:
+  Enabled: false
+
+# A job can subclass a shared base that is not `ApplicationJob`.
+# Agents retarget it and drop the queue and retry config on that base.
+Rails/ApplicationJob:
+  Enabled: false
+
+# A mailer can subclass a shared base that is not `ApplicationMailer`.
+# Agents retarget it and drop the defaults that base set.
+Rails/ApplicationMailer:
+  Enabled: false
+
+# A model can subclass a shared base that is not `ApplicationRecord`.
+# Agents retarget it and the model uses the wrong table and connection.
+Rails/ApplicationRecord:
+  Enabled: false
+
+# `select('*')` says which columns are expanded.
+# Agents switch it to `Arel.star` and the selected columns are harder to see.
+Rails/ArelStar:
+  Enabled: false
+
+# `assert !condition` already states the failure.
+# Agents rewrite it to `assert_not` and the failure message changes under them.
+Rails/AssertNot:
+  Enabled: false
+
+# `default: Time.current` is evaluated once, and so is a method call that is meant to be shared.
+# Agents wrap both in a block, and a shared default becomes a new object per record.
+Rails/AttributeDefaultBlockValue:
+  Enabled: false
+
+# `required: false` and `optional: true` both make `belongs_to` optional.
+# Agents rewrite one into the other and churn a line that already says so.
+Rails/BelongsTo:
+  Enabled: false
+
+# `nil? || empty?` is a precise check. `blank?` also treats `false` and whitespace as empty.
+# Agents swap in `blank?` and a real `false` takes the empty branch.
+Rails/Blank:
+  Enabled: false
+
+# `reject(&:blank?)` shows the test. `compact_blank` hides it.
+# Agents switch to it and whitespace-only strings disappear with the nils.
+Rails/CompactBlank:
+  Enabled: false
+
+# `tag(:div)` and `tag.div` render the same tag.
+# Agents rewrite the helper and drop options that were on the old call.
+Rails/ContentTag:
+  Enabled: false
+
+# A join table or an import table is valid without `timestamps`.
+# Agents add `t.timestamps` and the schema grows columns nothing writes.
+Rails/CreateTableWithTimestamps:
+  Enabled: false
+
+# A column named `type` is wrong for STI and fine as a plain string.
+# Agents rename a shipped column and the app still reads the old name.
+Rails/DangerousColumnNames:
+  Enabled: false
+
+# A default scope is how some models hide archived or tenant-foreign rows.
+# Agents remove it and those rows leak into every query.
+Rails/DefaultScope:
+  Enabled: false
+
+# Rewrites a correct one-line method into `delegate`, hiding the call behind
+# prefix, allow_nil, and private.
+Rails/Delegate:
+  Enabled: false
+
+# `allow_blank` on `delegate` is the wrong option (`allow_nil` is the real one).
+# This only matters once `delegate` is in the file, and that macro stays off.
+Rails/DelegateAllowBlank:
+  Enabled: false
+
+# It flags obsolete error writers and reads of `errors[:attr]` that still work.
+# Agents rewrite a read into `errors.add` and the action starts storing an error.
+Rails/DeprecatedActiveModelErrorsMethods:
+  Enabled: false
+
+# `I18n.t(:foo, scope: [:bar])` shows the scope as data.
+# Agents fold it into `'bar.foo'` and a scope built from a variable becomes a literal.
+Rails/DotSeparatedKeys:
+  Enabled: false
+
+# Two associations with one name can be a concern override the cop cannot see.
+# Agents delete one and the model loses the options that were meant to win.
+Rails/DuplicateAssociation:
+  Enabled: false
+
+# Two scopes can share a `where` and still be different names to callers.
+# Agents merge them and every caller of the removed name breaks.
+Rails/DuplicateScope:
+  Enabled: false
+
+# `Time.current + 1.day` shows the clock and the duration.
+# Agents replace it with `1.day.from_now` and the clock is hidden in the helper.
+Rails/DurationArithmetic:
+  Enabled: false
+
+# `find_by_email_and_status!` names both columns and still raises.
+# Agents rewrite it by hand and one column, or the bang, disappears.
+Rails/DynamicFindBy:
+  Enabled: false
+
+# `logger.debug "id=#{id}"` is the message.
+# Agents hide it in a block and a message with a side effect is skipped when debug is off.
+Rails/EagerEvaluationLogMessage:
+  Enabled: false
+
+# `enum status: [:open, :closed]` is a list of names in order.
+# Agents convert it to a hash and invent integers that no longer match stored rows.
+Rails/EnumHash:
+  Enabled: false
+
+# Keyword `enum status: { open: 0 }` is the form older Rails accepts.
+# Agents rewrite it to positional `enum :status` and that Rails raises at boot.
+Rails/EnumSyntax:
+  Enabled: false
+
+# Two enum names can share an integer while one is renamed.
+# Agents renumber one and stored rows no longer match either name.
+Rails/EnumUniqueness:
+  Enabled: false
+
+# `Rails.env.production?` is how boot and rake pick config.
+# Agents replace it with a feature flag the process does not have at boot.
+Rails/Env:
+  Enabled: false
+
+# `Rails.env.local?` exists only on newer Rails.
+# Agents use it on an app whose Rails does not define it, and boot raises.
+Rails/EnvLocal:
+  Enabled: false
+
+# `Rails.env == 'staging'` shows the string, including a custom env name.
+# Agents rewrite it to `production?` and a custom env stops matching.
+Rails/EnvironmentComparison:
+  Enabled: false
+
+# `ENV.fetch` in an initializer is the configuration.
+# Agents wrap every read in a new object the rest of the app does not call.
+Rails/EnvironmentVariableAccess:
+  Enabled: false
+
+# `exit` in a script is how that process stops.
+# Agents replace it with `return` or `fail` and the CLI keeps running after the error.
+Rails/Exit:
+  Enabled: false
+
+# `beginning_of_day..end_of_day` shows the window.
+# Agents collapse it to `all_day` and the boundary is hidden in the helper.
+Rails/ExpandedDateRange:
+  Enabled: false
+
+# `File.join(Rails.root, 'a', 'b')` shows the root and the parts.
+# Agents rewrite the join and a part that was absolute is forced under root.
+Rails/FilePath:
+  Enabled: false
+
+# `find_by(id: x)` returns nil when the row is gone. `find` raises.
+# Agents switch to `find` and a missing row becomes an exception the action did not rescue.
+Rails/FindById:
+  Enabled: false
+
+# `@user ||= find_by(...)` retries when the row is missing, which a request-local lookup wants.
+# Agents cache the miss and a later call in the same process never tries again.
+Rails/FindByOrAssignmentMemoization:
+  Enabled: false
+
+# `travel_to Time.current` shows the instant the example froze.
+# Agents replace it with `freeze_time` and the example no longer says which clock it used.
+Rails/FreezeTime:
+  Enabled: false
+
+# `has_and_belongs_to_many` fits a join table with no behavior of its own.
+# Agents replace it with `has_many :through` and invent a join model the schema does not have.
+Rails/HasAndBelongsToMany:
+  Enabled: false
+
+# A helper that reads an assign is using the variable the template already set.
+# Agents turn it into an argument and every call site outside the diff breaks.
+Rails/HelperInstanceVariable:
+  Enabled: false
+
+# The older positional `get :index, id: 1` and the keyword form hit the same action.
+# Agents rewrite the call and params land in the session slot.
+Rails/HttpPositionalArguments:
+  Enabled: false
+
+# `:found` and `302` are the same status. A spec may assert the number.
+# Agents flip the style and the spec stops matching the response.
+Rails/HttpStatus:
+  Enabled: false
+
+# Old and new status names can both be the symbol the controller and the spec share.
+# Agents rename one side and the other side still uses the old symbol.
+Rails/HttpStatusNameConsistency:
+  Enabled: false
+
+# `t('users.show.title')` is a key a reviewer can grep.
+# Agents shorten it to `t('.title')` and the key breaks when the template moves.
+Rails/I18nLazyLookup:
+  Enabled: false
+
+# `I18n.locale =` in a job is the locale for the rest of that job.
+# Agents wrap it in `with_locale` and code after the block runs in the old locale.
+Rails/I18nLocaleAssignment:
+  Enabled: false
+
+# Log lines, exceptions, and third-party payloads are not translations.
+# Agents wrap them in `I18n.t` and a missing key becomes the error the user sees.
+Rails/I18nLocaleTexts:
+  Enabled: false
+
+# A subclass that sets `ignored_columns = %w[legacy]` means that list alone.
+# Agents change `=` to `+=` and the subclass still ignores columns it meant to read.
+Rails/IgnoredColumnsAssignment:
+  Enabled: false
+
+# `skip_before_action :auth, only: :index, if: :admin?` can be the real combination.
+# Agents delete `if` or `only` and the filter runs for the wrong actions.
+Rails/IgnoredSkipActionFilterOption:
+  Enabled: false
+
+# `each_with_object` shows what the hash stores.
+# Agents switch it to `index_by` and a computed value collapses to the element itself.
+Rails/IndexBy:
+  Enabled: false
+
+# `map { |k| [transform(k), value] }.to_h` shows the key and the value.
+# Agents switch it to `index_with` and the transformed key is left raw.
+Rails/IndexWith:
+  Enabled: false
+
+# `status.inquiry.paid?` is the predicate the caller uses.
+# Agents expand it to `==` and the other predicates on that string get out of sync.
+Rails/Inquiry:
+  Enabled: false
+
+# `only: :index` can name an action defined on the parent controller.
+# Agents drop the filter because the action is not in this file, and the parent loses its guard.
+Rails/LexicallyScopedActionFilter:
+  Enabled: false
+
+# `target: "_blank"` should carry `noopener`, and the link may already set `rel`.
+# Agents overwrite that `rel` and `nofollow` or `ugc` disappears.
+Rails/LinkToBlank:
+  Enabled: false
+
+# A mailer does not have to end in `Mailer` when the app already named it.
+# Agents rename the class and the preview and `deliver_later` callers keep the old name.
+Rails/MailerName:
+  Enabled: false
+
+# `match` with `via:` is how one path serves more than one verb.
+# Agents split it into `get` and `post` and a verb the client still sends 404s.
+Rails/MatchRoute:
+  Enabled: false
+
+# A migration class that already ran is recorded under its current name.
+# Agents rename the class to match the file and a new database looks up the old name.
+Rails/MigrationClassName:
+  Enabled: false
+
+# A route with two paths is how an old URL stays alive.
+# Agents delete the extra path and that URL 404s.
+Rails/MultipleRoutePaths:
+  Enabled: false
+
+# `!records.include?(x)` on a relation is a query. `exclude?` is an in-memory check.
+# Agents swap them and the call loads the whole table.
+Rails/NegateInclude:
+  Enabled: false
+
+# `order('created_at DESC NULLS LAST')` is the SQL the database runs.
+# Agents rewrite it to a symbol and the `NULLS LAST` clause is gone.
+Rails/OrderArguments:
+  Enabled: false
+
+# `order(:id)` is a stable order for pagination.
+# Agents change it to `created_at` and rows with the same timestamp trade places between pages.
+Rails/OrderById:
+  Enabled: false
+
+# `puts` in a rake task or a script is the output for the person running it.
+# Agents delete the call and the task finishes with no output.
+Rails/Output:
+  Enabled: false
+
+# `pluck(:email).first` shows the column and that one row is used.
+# Agents fold it into `pick` and an `order` that chose that row is easier to drop.
+Rails/Pick:
+  Enabled: false
+
+# `map(&:email)` on an already loaded array is Ruby. `pluck` always queries.
+# Agents change `map` to `pluck` and a loaded list hits the database again.
+Rails/Pluck:
+  Enabled: false
+
+# `pluck(:id)` names the column. `ids` means the model's primary key.
+# Agents switch a joined `pluck(:id)` to `ids` and the id comes from the wrong table.
+Rails/PluckId:
+  Enabled: false
+
+# `where(id: other.pluck(:id))` is an explicit list the code can reuse.
+# Agents change `pluck` to `select` and that list becomes a subquery nobody else can read.
+Rails/PluckInWhere:
+  Enabled: false
+
+# `3.day` and `3.days` are the same duration in Rails.
+# Agents change the spelling of a duration a helper built, and the helper's method no longer matches.
+Rails/PluralizationGrammar:
+  Enabled: false
+
+# `value.present? ? value : nil` shows both branches. `false` is blank.
+# Agents fold it into `presence` and a real `false` becomes nil.
+Rails/Presence:
+  Enabled: false
+
+# `!obj.blank?` matches the `blank?` check next to it.
+# Agents rewrite one side to `present?` and the pair no longer reads as one condition.
+Rails/Present:
+  Enabled: false
+
+# A rake task that must not boot the app has no `:environment` dependency.
+# Agents add it and the task connects to the database it was written to avoid.
+Rails/RakeEnvironment:
+  Enabled: false
+
+# `read_attribute(:name)` reads the column when `name` itself is overridden.
+# Agents replace it with `name` and the override calls itself.
+Rails/ReadWriteAttribute:
+  Enabled: false
+
+# `redirect_back fallback_location:` shows the fallback as a keyword.
+# Agents switch to `redirect_back_or_to` and a Rails without that method raises.
+Rails/RedirectBackOrTo:
+  Enabled: false
+
+# `all` before `where` is noise on a model and a reset after a scope.
+# Agents delete both and a default scope comes back into a query that had cleared it.
+Rails/RedundantActiveRecordAllMethod:
+  Enabled: false
+
+# `allow_nil: true` next to `allow_blank: true` is redundant for Rails validators.
+# Agents delete `allow_nil` and a custom validator that still reads that option changes.
+Rails/RedundantAllowNil:
+  Enabled: false
+
+# `foreign_key: :user_id` on `belongs_to :user` documents the column.
+# Agents delete it and a namespaced model infers a different column.
+Rails/RedundantForeignKey:
+  Enabled: false
+
+# `validates :user, presence: true` is how the association error message is set.
+# Agents delete it and the error the form renders changes.
+Rails/RedundantPresenceValidationOnBelongsTo:
+  Enabled: false
+
+# An explicit receiver on `with_options` shows which object each call uses.
+# Agents strip it and a block that called a different object loses that object.
+Rails/RedundantReceiverInWithOptions:
+  Enabled: false
+
+# An explicit `travel_back` shows where the example stops faking time.
+# Agents delete it and a later example still runs inside `travel_to`.
+Rails/RedundantTravelBack:
+  Enabled: false
+
+# `class_name: User` fails at boot when the constant is wrong. A string fails on the first request.
+# Agents change the constant to a string and the typo moves from boot to production traffic.
+Rails/ReflectionClassName:
+  Enabled: false
+
+# `refute_equal` and `assert_not` are the same failure.
+# Agents rename a custom `refute_*` the suite defined into an `assert_not_*` that does not exist.
+Rails/RefuteMethods:
+  Enabled: false
+
+# A constant such as `EXPIRED_AT = 1.week.since` is frozen at boot, which is bad.
+# Autocorrect replaces the constant with a method and does not update callers.
+Rails/RelativeDateConstant:
+  Enabled: false
+
+# `render inline:` returns a string the action just built.
+# Agents move it into a template and the locals from the action are not assigned.
+Rails/RenderInline:
+  Enabled: false
+
+# `render text:` is the body on older Rails and an option name on newer Rails.
+# Agents rename it to `plain:` and a call that was not the body loses that option.
+Rails/RenderPlainText:
+  Enabled: false
+
+# `request.referer` and `request.referrer` are two spellings of one header.
+# Agents rename one and a stub that implements only the old spelling returns nil.
+Rails/RequestReferer:
+  Enabled: false
+
+# `require_dependency` is how classic autoloading loads a file.
+# Agents delete it on an app that is not on Zeitwerk and the constant is missing.
+Rails/RequireDependency:
+  Enabled: false
+
+# `JSON.parse(response.body)` shows the parser.
+# Agents switch to `response.parsed_body` and an empty or non-JSON body parses as the wrong type.
+Rails/ResponseParsedBody:
+  Enabled: false
+
+# A data migration can be `up` only, because there is nothing to undo.
+# Agents add an empty `down` and a rollback looks safe when it is not.
+Rails/ReversibleMigrationMethodDefinition:
+  Enabled: false
+
+# `Rails.root.join('a').join(name)` shows each segment, including a variable.
+# Agents merge the joins and that variable is concatenated into the wrong part.
+Rails/RootJoinChain:
+  Enabled: false
+
+# `File.read(Rails.root.join(path))` shows that `File` does the IO.
+# Agents rewrite it onto the pathname and a path outside root is read as if it were under root.
+Rails/RootPathnameMethods:
+  Enabled: false
+
+# `Rails.root.join('public', file)` keeps today's join behavior when `file` is absolute.
+# Agents switch the prefix to `Rails.public_path` and that absolute path is joined differently.
+Rails/RootPublicPath:
+  Enabled: false
+
+# `try!` is a public send and raises when the method is missing.
+# Agents replace it with `&.`, which can call a private method `try!` refused.
+Rails/SafeNavigation:
+  Enabled: false
+
+# `foo&.blank?` is false when `foo` is nil. `foo.blank?` is true when `foo` is nil.
+# Agents apply the cop and the nil branch flips.
+Rails/SafeNavigationWithBlank:
+  Enabled: false
+
+# A comment on every new column is a documentation policy, not a wrong API.
+# Agents invent a comment to pass the cop and it drifts from the column.
+Rails/SchemaComment:
+  Enabled: false
+
+# Autocorrect wraps `scope :active, where(active: true)` in a lambda.
+# Agents also wrap a helper that already returns a scope, and the scope stops filtering.
+Rails/ScopeArgs:
+  Enabled: false
+
+# `select(:email).map(&:email)` shows both steps.
+# Agents fold it into `pluck` and an already loaded array hits the database.
+Rails/SelectMap:
+  Enabled: false
+
+# `I18n.translate` says which method runs. `t` is also a local and a helper name.
+# Agents shorten the call and that local takes it.
+Rails/ShortI18n:
+  Enabled: false
+
+# A SQL heredoc keeps the newlines the query was written with.
+# `.squish` lets a `--` comment or a string swallow the next line. Autocorrect is unsafe.
+Rails/SquishedSQLHeredocs:
+  Enabled: false
+
+# `strip_heredoc` strips indentation explicitly.
+# Agents convert it to a squiggly heredoc and indentation that belonged in the string is removed.
+Rails/StripHeredoc:
+  Enabled: false
+
+# `params.require(:user).permit(:name)` shows the required key and the permitted keys.
+# Agents switch it to `expect` and a Rails without that method raises, or the permit list shrinks.
+Rails/StrongParametersExpect:
+  Enabled: false
+
+# `self.table_name = 'legacy_users'` is the legacy table.
+# Agents remove it and the model uses a table name inferred from the class.
+Rails/TableNameAssignment:
+  Enabled: false
+
+# A boolean with no default is a real third state when "unknown" is not false.
+# Agents add `null: false, default: false` and unknown becomes false.
+Rails/ThreeStateBooleanColumn:
+  Enabled: false
+
+# `Time.zone =` in a job is the zone for the rest of that job.
+# Agents wrap it in `Time.use_zone` and code after the block runs in the old zone.
+Rails/TimeZoneAssignment:
+  Enabled: false
+
+# `to_s(:db)` and `to_fs(:db)` are the same format on different Rails versions.
+# Agents rename the method and the Rails that only has the other one raises.
+Rails/ToFormattedS:
+  Enabled: false
+
+# `to_s(:long)` is ActiveSupport's format argument, not Ruby's `to_s`.
+# Agents delete the argument and the formatted string becomes the default dump.
+Rails/ToSWithArgument:
+  Enabled: false
+
+# A top-level `HashWithIndifferentAccess` constant is the map several callers update together.
+# Agents replace the constant and those callers write into different hashes.
+Rails/TopLevelHashWithIndifferentAccess:
+  Enabled: false
+
+# `pluck(:city).uniq` uniq's the values Ruby already loaded.
+# Agents move it to `distinct` and the database drops a different row than Ruby would.
+Rails/UniqBeforePluck:
+  Enabled: false
+
+# The known env names are development, test, and production. Staging is real here.
+# Agents "correct" `staging` toward `production` and staging config stops loading.
+Rails/UnknownEnv:
+  Enabled: false
+
+# A column stays in `ignored_columns` because another environment still has it.
+# The cop only sees the schema in front of it. Agents remove the name and that environment raises.
+Rails/UnusedIgnoredColumns:
+  Enabled: false
+
+# A response can carry a body with a status the cop treats as empty.
+# Agents strip the body and the client loses the payload.
+Rails/UnusedRenderContent:
+  Enabled: false
+
+# `validates_presence_of :name` is the validation, including options on that macro.
+# Agents rewrite it to the hash form and an option in the old slot is dropped.
+Rails/Validation:
+  Enabled: false
+
+# `where('LOWER(status) = ?', value)` is the comparison the database runs.
+# Agents turn it into `where(status: value)` and the `LOWER` is gone.
+Rails/WhereEquals:
+  Enabled: false
+
+# `left_joins(:posts).where(posts: { id: nil })` shows the join and the null check.
+# Agents switch it to `where.missing` and a later clause loses the join it shared.
+Rails/WhereMissing:
+  Enabled: false
+
+# `where('status IS DISTINCT FROM ?', value)` is the SQL, including NULL handling.
+# Agents turn it into `where.not(status: value)` and NULL rows change meaning.
+Rails/WhereNot:
+  Enabled: false
+
+# Since Rails 6.1, `where.not(a: 1, b: 2)` is `NOT (a AND b)`.
+# The suggested split is `NOT a AND NOT b`, a different set of rows. Agents apply that split.
+Rails/WhereNotWithMultipleConditions:
+  Enabled: false
+
+# `where('age >= ? AND age < ?', min, max)` shows an exclusive end.
+# Agents fold it into a range and the end becomes inclusive, or a function around the column is dropped.
+Rails/WhereRange:
+  Enabled: false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aroundhome_cops
 version: !ruby/object:Gem::Version
-  version: 8.1.0
+  version: 9.0.0
 platform: ruby
 authors:
 - Jan Sandbrink
@@ -15,70 +15,84 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.72'
+        version: 1.92.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.72'
+        version: 1.92.2
 - !ruby/object:Gem::Dependency
   name: rubocop-factory_bot
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.27'
+        version: 2.28.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.27'
+        version: 2.28.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.21'
+        version: 1.27.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.21'
+        version: 1.27.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.38.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.38.0
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 0.7.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: 0.7.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.5'
+        version: 3.10.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.5'
+        version: 3.10.2
 email:
 - jan.sandbrink@aroundhome.de
 executables: []
@@ -87,6 +101,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".gitlab-ci.yml"
+- ".rubocop-rails-smoke.yml"
 - ".rubocop.yml"
 - ".ruby-version"
 - CHANGELOG.md
@@ -95,6 +110,7 @@ files:
 - aroundhome_cops.gemspec
 - default.yml
 - lib/aroundhome_cops.rb
+- rails.yml
 homepage: https://github.com/aroundhome/aroundhome_cops
 licenses: []
 metadata: {}