betterlint 1.25.0 → 1.26.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 04c560ef56ed92b4393449c238d64d433b16caa1db01dbcb81c647c04f9931d6
-  data.tar.gz: bac5e05a6ec457e6c047eba8c1d709549167ceae3b408fa084c5aa83c9594676
+  metadata.gz: 521829c1a24b2013526ec739c7767473854130c2e57f7c29b184011f059264a4
+  data.tar.gz: a7cb5622266cc225688096b99811eae23fec5a1af226e410f205986c3d7414ac
 SHA512:
-  metadata.gz: 57bf6d2446f5ec19e463e3bad15bb22934b08104a7614f08b1f67291bd8243ef7c1fdbccd7d8259f8b7b69c10f995248561c2e2cecf032c2bac01df2fd074aba
-  data.tar.gz: f2cee92d7b16ba483562f7bb1ebf822ff186e106cbf1df9f3f026f013f801a2f4f9fb01737e50b3e8bbfe83f4ee26af1b44614e35a48cf65028e7d3f6afb1dd2
+  metadata.gz: cc3c3e063757b97e37257303a939e2d6d13bd8534646c5b0047f2fd58a229979bf2687702025a6d24971dac784b987c2393d67df79bc58875842500d67f55cb0
+  data.tar.gz: 0e1171c10b8c6b26f71cc13ddd77694487fce3749e40d84931ad82d90d0f57bf9eaea48bd16504c64695cdc95686932625de0dd4c56853c6bb2bf2807539a6ce

data/README.md

@@ -15,9 +15,8 @@ gem 'betterlint'
 .rubocop.yml:
 
 ```yml
-inherit_gem:
-  betterlint:
-    - config/default.yml
+plugins:
+  - betterlint
 ```
 
 ## Dependencies

data/config/default.yml

@@ -2,11 +2,16 @@
 
 require:
   - rubocop/cop/betterment.rb
+
+plugins:
+  - rubocop-capybara
+  - rubocop-factory_bot
   - rubocop-graphql
   - rubocop-performance
   - rubocop-rails
   - rubocop-rake
   - rubocop-rspec
+  - rubocop-rspec_rails
 
 AllCops:
   DisplayCopNames: true
@@ -42,10 +47,9 @@ Betterment/DynamicParams:
 
 Betterment/EnvironmentConfiguration:
   StyleGuide: '#bettermentenvironmentconfiguration'
-  Exclude:
-    - config/**/*.rb
-    - spec/**/*.rb
-    - test/**/*.rb
+  Include:
+    - app/**/*.rb
+    - lib/**/*.rb
 
 Betterment/HardcodedID:
   AutoCorrect: false
@@ -254,7 +258,7 @@ Metrics/PerceivedComplexity:
 Naming/HeredocDelimiterNaming:
   Enabled: false
 
-Naming/PredicateName:
+Naming/PredicatePrefix:
   ForbiddenPrefixes:
     - is_
   NamePrefix:
@@ -269,9 +273,6 @@ Performance/RedundantMatch:
 RSpec/BeEq:
   Enabled: false
 
-RSpec/Capybara/FeatureMethods:
-  Enabled: false
-
 RSpec/ContextWording:
   Enabled: false
 
@@ -303,7 +304,7 @@ RSpec/ExpectChange:
   EnforcedStyle: block
 
 RSpec/SpecFilePathSuffix:
-  Enabled: false
+  Enabled: true
 
 RSpec/SpecFilePathFormat:
   Enabled: false

data/lib/betterlint/version.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module Betterlint
+  VERSION = '1.26.0'
+end

data/lib/betterlint.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'lint_roller'
+require_relative 'betterlint/version'
+
+module Betterlint
+  class Plugin < LintRoller::Plugin
+    def about
+      LintRoller::About.new(
+        name: 'betterlint',
+        version: VERSION,
+        homepage: 'https://github.com/Betterment/betterlint',
+        description: 'Shared rubocop configuration for Betterment Rails apps/engines.',
+      )
+    end
+
+    def supported?(context)
+      context.engine == :rubocop
+    end
+
+    def rules(_context)
+      LintRoller::Rules.new(
+        type: :path,
+        config_format: :rubocop,
+        value: File.expand_path('../config/default.yml', __dir__),
+      )
+    end
+  end
+end

data/lib/rubocop/cop/betterment/authorization_in_controller.rb

@@ -118,7 +118,7 @@ module RuboCop
             #   params[:user_id]
             # end
             if ret.send_type? && ret.method?(:[])
-              internal_params = ret.arguments.select { |x| x.sym_type? || x.str_type? }.map(&:value)
+              internal_params = ret.arguments.select { |x| x.type?(:sym, :str) }.map(&:value)
             else
               internal_returns = get_method_returns(Utils::Parser.get_root_token(ret))
               internal_params = internal_returns.flat_map { |x| Utils::Parser.get_extracted_parameters(x, param_aliases: @param_wrappers) }

data/lib/rubocop/cop/betterment/non_standard_actions.rb

@@ -52,6 +52,10 @@ module RuboCop
           route = route_to(node)
           if route
             (path, param, value) = route
+
+            # Newer Rubocop versions return param as an array
+            param = param.first if param.is_a?(Array)
+
             action = case param
             when :to then value.first.split('#').last
             when :action then value.first

data/lib/rubocop/cop/betterment/utils/parser.rb

@@ -67,16 +67,16 @@ module RuboCop
             end
           end
 
-          def self.params_from_arguments(arguments) # rubocop:disable Metrics/PerceivedComplexity
+          def self.params_from_arguments(arguments)
             parameter_names = []
 
             arguments.each do |arg|
               if arg.hash_type?
                 arg.children.each do |pair|
                   value = pair.value
-                  parameter_names << value.value if value.sym_type? || value.str_type?
+                  parameter_names << value.value if value.type?(:sym, :str)
                 end
-              elsif arg.sym_type? || arg.str_type?
+              elsif arg.type?(:sym, :str)
                 parameter_names << arg.value
               end
             end
@@ -92,7 +92,7 @@ module RuboCop
 
             if node.method?(:[]) && aliases.include?(get_root_token(node))
               return node.arguments.select { |x|
-                x.sym_type? || x.str_type?
+                x.type?(:sym, :str)
               }.map(&:value)
             end
 

metadata

@@ -1,28 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: betterlint
 version: !ruby/object:Gem::Version
-  version: 1.25.0
+  version: 1.26.0
 platform: ruby
 authors:
 - Development
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2025-12-17 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: lint_roller
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.71'
+        version: '1.82'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.82'
+- !ruby/object:Gem::Dependency
+  name: rubocop-capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.22'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.22'
+- !ruby/object:Gem::Dependency
+  name: rubocop-factory_bot
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.28'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.71'
+        version: '2.28'
 - !ruby/object:Gem::Dependency
   name: rubocop-graphql
   requirement: !ruby/object:Gem::Requirement
@@ -30,6 +73,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.5.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -37,62 +83,79 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.5.6
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.23'
+        version: '1.26'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.23'
+        version: '1.26'
 - !ruby/object:Gem::Dependency
   name: rubocop-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.29'
+        version: '2.34'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.29'
+        version: '2.34'
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '0.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.29'
+        version: '3.8'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.8'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec_rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.32'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.29'
+        version: '2.32'
 description: Betterment rubocop configuration
 email:
 - development@betterment.com
@@ -103,6 +166,8 @@ files:
 - README.md
 - STYLEGUIDE.md
 - config/default.yml
+- lib/betterlint.rb
+- lib/betterlint/version.rb
 - lib/rubocop/cop/betterment.rb
 - lib/rubocop/cop/betterment/active_job_performable.rb
 - lib/rubocop/cop/betterment/allowlist_blocklist.rb
@@ -137,11 +202,13 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/Betterment/betterlint
-  source_code_uri: https://github.com/Betterment/betterlint/tree/v1.25.0
-  changelog_uri: https://github.com/Betterment/betterlint/blob/v1.25.0/CHANGELOG.md
+  source_code_uri: https://github.com/Betterment/betterlint/tree/v1.26.0
+  changelog_uri: https://github.com/Betterment/betterlint/blob/v1.26.0/CHANGELOG.md
   bug_tracker_uri: https://github.com/Betterment/betterlint/issues
-  documentation_uri: https://www.rubydoc.info/gems/betterlint/1.25.0
+  documentation_uri: https://www.rubydoc.info/gems/betterlint/1.26.0
   rubygems_mfa_required: 'true'
+  default_lint_roller_plugin: Betterlint::Plugin
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -156,7 +223,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.8
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: Betterment rubocop configuration
 test_files: []