rubocop-callback_checker 0.1.0 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27a8d5e0a0b8d6c92ccabe6a928f106fc80a3d438ed07002686f07bf5c3c07c9
-  data.tar.gz: 1bad8a7bb9dcf33f5758655bbdfc208f9849dc75cde2b534dbaedef18e7e31ad
+  metadata.gz: b18d1882a23fe18befff3bb4fe0760099a3903fee3af4e960fe76ea4fdf8e099
+  data.tar.gz: 641a5f598be2759bb728a30e4cc409f747c932849327f4ea7b0be1f6439f5e3c
 SHA512:
-  metadata.gz: 222120c9a2df85dc84fd57709c17b52b494f9e05b21a9ae64fc7f94d058595f6aebb3a9a821daad78707de021ff796c7039c73710760fd1c410f43e04a07e04f
-  data.tar.gz: 29893d82a5f907f2d7cf88781864963e169f6c0e22b1bbc0f5615c0101773aa12f6a53a7e6dc5363c61a6849680090aa4edecd33e3edbe01af3130b47fc63bf6
+  metadata.gz: 762044ec4f25d593b2244bf72530c13f21841472a9ead75fd05a245f83755b580338edf96d10dae9497685d6a5595fe97deb5dab9f31c75f3ab173cbc910f621
+  data.tar.gz: bc79ee42e225d79eff6afbc277139b137f8c03231958dacb3d8921d4b59c3bcac26f3e43239b357dfc41f7e18b07aa54c468b90ed058cd5206b6514394612b96

data/.rubocop.yml

@@ -1,15 +1,61 @@
-require:
-  - rubocop-rspec
-
 plugins:
-  - rubocop-rake
+  - rubocop-callback_checker
 
 AllCops:
-  TargetRubyVersion: 3.1
   NewCops: enable
+  SuggestExtensions: false
+
+# The entry point file must be named rubocop-callback_checker.rb to match the gem name
+Naming/FileName:
+  Exclude:
+    - 'lib/rubocop-callback_checker.rb'
+
+# Allow longer blocks in spec files
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*_spec.rb'
+    - 'rubocop-callback_checker.gemspec'
+
+# Allow some cops to have longer methods due to RuboCop's pattern matching requirements
+Metrics/ClassLength:
+  Exclude:
+    - 'lib/rubocop/cop/**/*'
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Metrics/MethodLength:
+  Exclude:
+    - 'lib/rubocop/cop/**/*'
+    - 'lib/callback_checker/cli.rb'
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Metrics/AbcSize:
+  Exclude:
+    - 'lib/rubocop/cop/**/*'
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Metrics/CyclomaticComplexity:
+  Exclude:
+    - 'lib/rubocop/cop/**/*'
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Metrics/PerceivedComplexity:
+  Exclude:
+    - 'lib/rubocop/cop/**/*'
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Style/Documentation:
+  Enabled: false
+
+Lint/MissingSuper:
+  Exclude:
+    - 'lib/callback_checker/prism_analyzer.rb'
+
+Lint/UselessMethodDefinition:
+  Exclude:
+    - 'lib/callback_checker/prism_analyzer.rb'
 
-Style/StringLiterals:
-  EnforcedStyle: double_quotes
+Layout/LineLength:
+  Max: 160
 
-Style/StringLiteralsInInterpolation:
-  EnforcedStyle: double_quotes
+Gemspec/DevelopmentDependencies:
+  Enabled: false

data/README.md

@@ -12,14 +12,12 @@ Install the gem and add to the application's Gemfile by executing:
 
 ```bash
 bundle add rubocop-callback_checker
-
 ```
 
 If bundler is not being used to manage dependencies, install the gem by executing:
 
 ```bash
 gem install rubocop-callback_checker
-
 ```
 
 ---
@@ -29,16 +27,38 @@ gem install rubocop-callback_checker
 In your `.rubocop.yml`, add the following:
 
 ```yaml
-require:
+# Modern method (RuboCop 1.72+)
+plugins:
   - rubocop-callback_checker
 
+# Alternative method (older RuboCop versions)
+require:
+  - rubocop-callback_checker
 ```
 
 You can then run RuboCop as usual:
 
 ```bash
 bundle exec rubocop
+```
+
+The gem will automatically load all callback checker cops with their default configurations.
+
+---
 
+## Configuration
+
+All cops are enabled by default with reasonable settings. You can customize them in your `.rubocop.yml`:
+
+```yaml
+plugins:
+  - rubocop-callback_checker
+
+CallbackChecker/CallbackMethodLength:
+  Max: 10  # Default is 5
+
+CallbackChecker/NoSideEffectsInCallbacks:
+  Enabled: false  # Disable if needed
 ```
 
 ---
@@ -54,13 +74,53 @@ If a side effect (like sending an email) triggers in an `after_save` but the tra
 * **Bad:** Calling `UserMailer.welcome.deliver_now` in `after_create`.
 * **Good:** Use `after_commit` or `after_create_commit`.
 
-### 2. `CallbackChecker/NoSelfPersistence`
+**Example:**
+
+```ruby
+# bad
+class User < ApplicationRecord
+  after_create { UserMailer.welcome(self).deliver_now }
+end
+
+# good
+class User < ApplicationRecord
+  after_create_commit { UserMailer.welcome(self).deliver_now }
+end
+```
+
+---
+
+### 2. `CallbackChecker/AvoidSelfPersistence`
 
 **Goal:** Prevent infinite recursion and "Stack Level Too Deep" errors.
 
 * **Bad:** Calling `self.save` or `update(status: 'active')` inside a `before_save`.
 * **Good:** Assign attributes directly: `self.status = 'active'`.
 
+**Example:**
+
+```ruby
+# bad
+class User < ApplicationRecord
+  before_save :activate
+  
+  def activate
+    self.update(status: 'active')  # triggers before_save again!
+  end
+end
+
+# good
+class User < ApplicationRecord
+  before_save :activate
+  
+  def activate
+    self.status = 'active'  # will be saved automatically
+  end
+end
+```
+
+---
+
 ### 3. `CallbackChecker/AttributeAssignmentOnly`
 
 **Goal:** Reduce unnecessary database I/O.
@@ -70,21 +130,94 @@ Callbacks that run "before" persistence should only modify the object's memory s
 * **Bad:** `before_validation { update(attr: 'val') }`
 * **Good:** `before_validation { self.attr = 'val' }`
 
+**Example:**
+
+```ruby
+# bad
+class User < ApplicationRecord
+  before_save :normalize_email
+  
+  def normalize_email
+    update(email: email.downcase)  # unnecessary extra query
+  end
+end
+
+# good
+class User < ApplicationRecord
+  before_save :normalize_email
+  
+  def normalize_email
+    self.email = email.downcase  # just modifies in memory
+  end
+end
+```
+
+---
+
 ### 4. `CallbackChecker/CallbackMethodLength`
 
 **Goal:** Prevent "Fat Models" and maintain testability.
 
 Callbacks should be "post-it notes," not "instruction manuals." If a callback method is too long, it should be moved to a Service Object.
 
-* **Default Max:** 10 lines.
+* **Default Max:** 5 lines (configurable).
+
+**Example:**
+
+```ruby
+# bad
+class User < ApplicationRecord
+  after_create :setup_account
+  
+  def setup_account
+    # 20 lines of complex logic...
+    create_default_settings
+    send_welcome_email
+    notify_admin
+    create_billing_account
+    # ...
+  end
+end
+
+# good
+class User < ApplicationRecord
+  after_create_commit :setup_account
+  
+  def setup_account
+    AccountSetupService.new(self).call
+  end
+end
+```
+
+---
 
-### 5. `CallbackChecker/SymbolicConditionals`
+### 5. `CallbackChecker/ConditionalStyle`
 
 **Goal:** Improve readability and allow for easier debugging.
 
 * **Bad:** `before_save :do_thing, if: -> { status == 'active' && !deleted? }`
 * **Good:** `before_save :do_thing, if: :active_and_present?`
 
+**Example:**
+
+```ruby
+# bad
+class User < ApplicationRecord
+  before_save :do_thing, if: -> { status == 'active' && !deleted? }
+end
+
+# good
+class User < ApplicationRecord
+  before_save :do_thing, if: :active_and_present?
+  
+  private
+  
+  def active_and_present?
+    status == 'active' && !deleted?
+  end
+end
+```
+
 ---
 
 ## Development
@@ -95,11 +228,10 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at [https://github.com/](https://github.com/)[USERNAME]/rubocop-callback_checker.
+Bug reports and pull requests are welcome on GitHub at https://github.com/rahsheen/rubocop-callback_checker.
 
 ## License
 
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).
 
 ---
-

data/Rakefile

@@ -1,11 +1,11 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-require "rubocop/rake_task"
+require 'rubocop/rake_task'
 
 RuboCop::RakeTask.new
 

data/lib/rubocop/callback_checker/plugin.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'lint_roller'
+
+module RuboCop
+  module CallbackChecker
+    # A plugin that integrates RuboCop CallbackChecker with RuboCop's plugin system.
+    class Plugin < LintRoller::Plugin
+      def about
+        LintRoller::About.new(
+          name: 'rubocop-callback_checker',
+          version: VERSION,
+          homepage: 'https://github.com/rahsheen/rubocop-callback_checker',
+          description: 'A RuboCop extension focused on avoiding callback hell in Rails.'
+        )
+      end
+
+      def supported?(context)
+        context.engine == :rubocop
+      end
+
+      def rules(_context)
+        LintRoller::Rules.new(
+          type: :path,
+          config_format: :rubocop,
+          value: Pathname.new(__dir__).join('../../../config/default.yml')
+        )
+      end
+    end
+  end
+end

data/lib/rubocop/callback_checker/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-module Rubocop
+module RuboCop
   module CallbackChecker
-    VERSION = "0.1.0"
+    VERSION = '0.1.2'
   end
 end

data/lib/rubocop/callback_checker.rb

@@ -1,21 +1,17 @@
 # frozen_string_literal: true
 
-require_relative "callback_checker/version"
-require "pathname"
-require "yaml"
+require_relative 'callback_checker/version'
+require_relative 'callback_checker/plugin'
+require 'pathname'
 
-# Load all the cops
-Dir[Pathname.new(__dir__).join("cop", "callback_checker", "**", "*.rb")].each { |file| require file }
-
-module Rubocop
+module RuboCop
   module CallbackChecker
     class Error < StandardError; end
 
     PROJECT_ROOT = Pathname.new(__dir__).parent.parent.freeze
-    CONFIG_DEFAULT = PROJECT_ROOT.join("config", "default.yml").freeze
+    CONFIG_DEFAULT = PROJECT_ROOT.join('config', 'default.yml').freeze
 
-    def self.version
-      VERSION
-    end
+    # Inject the plugin's default configuration into RuboCop
+    ::RuboCop::ConfigLoader.inject_defaults!(CONFIG_DEFAULT.to_s)
   end
 end

data/lib/rubocop/cop/callback_checker/attribute_assignment_only.rb

@@ -69,13 +69,13 @@ module RuboCop
 
         def on_class(node)
           @current_callbacks = {}
-          
+
           node.each_descendant(:send) do |send_node|
             next unless callback_method?(send_node)
-            
+
             callback_name = send_node.method_name
             callback_args = send_node.arguments
-            
+
             callback_args.each do |arg|
               if arg.sym_type?
                 method_name = arg.value
@@ -84,18 +84,16 @@ module RuboCop
                 check_block_for_persistence(arg, callback_name)
               end
             end
-            
-            if send_node.block_node
-              check_block_for_persistence(send_node.block_node, callback_name)
-            end
+
+            check_block_for_persistence(send_node.block_node, callback_name) if send_node.block_node
           end
-          
+
           node.each_descendant(:def) do |def_node|
             method_name = def_node.method_name
             callback_name = @current_callbacks[method_name]
-            
+
             next unless callback_name
-            
+
             check_method_for_persistence(def_node, callback_name)
           end
         end
@@ -105,7 +103,7 @@ module RuboCop
         def check_block_for_persistence(block_node, callback_name)
           block_node.each_descendant(:send) do |send_node|
             next unless persistence_call?(send_node)
-            
+
             add_offense_for_node(send_node, callback_name)
           end
         end
@@ -113,7 +111,7 @@ module RuboCop
         def check_method_for_persistence(method_node, callback_name)
           method_node.each_descendant(:send) do |send_node|
             next unless persistence_call?(send_node)
-            
+
             add_offense_for_node(send_node, callback_name)
           end
         end
@@ -121,14 +119,14 @@ module RuboCop
         def add_offense_for_node(node, callback_name)
           method_name = node.method_name
           attribute = extract_attribute_name(node)
-          
+
           message = format(
             MSG,
             attribute: attribute,
             method: method_name,
             callback: callback_name
           )
-          
+
           add_offense(node, message: message)
         end
 
@@ -138,11 +136,11 @@ module RuboCop
             key = first_hash_key(node)
             return key.to_s if key
           end
-          
+
           # Try to extract from symbol argument (e.g., update_column(:name, 'foo'))
           symbol_arg = first_symbol_arg(node)
           return symbol_arg.to_s if symbol_arg
-          
+
           'attribute'
         end
       end

data/lib/rubocop/cop/callback_checker/avoid_self_persistence.rb

@@ -23,9 +23,9 @@ module RuboCop
       #     self.status = 'active'
       #   end
       class AvoidSelfPersistence < Base
-        MSG = "Avoid calling `%<method>s` on self within `%<callback>s`. " \
-              "This can trigger infinite loops or run callbacks multiple times. " \
-              "Assign attributes directly instead: `self.attribute = value`."
+        MSG = 'Avoid calling `%<method>s` on self within `%<callback>s`. ' \
+              'This can trigger infinite loops or run callbacks multiple times. ' \
+              'Assign attributes directly instead: `self.attribute = value`.'
 
         CALLBACK_METHODS = %i[
           before_validation after_validation
@@ -84,7 +84,7 @@ module RuboCop
           elsif arg.block_pass_type?
             # Handle block pass like: after_save &:method_name
             # We can't easily analyze these, so skip
-            return
+            nil
           elsif arg.lambda_or_proc?
             check_proc_callback(arg, node.method_name)
           end

data/lib/rubocop/cop/callback_checker/callback_method_length.rb

@@ -38,8 +38,8 @@ module RuboCop
       #     # No callback, call UserRegistrationService.new(user).call from controller
       #   end
       class CallbackMethodLength < Base
-        MSG = "Callback method `%<method>s` is too long (%<length>d lines). " \
-              "Max allowed: %<max>d lines. Extract complex logic to a service object."
+        MSG = 'Callback method `%<method>s` is too long (%<length>d lines). ' \
+              'Max allowed: %<max>d lines. Extract complex logic to a service object.'
 
         CALLBACK_METHODS = %i[
           before_validation after_validation
@@ -54,7 +54,7 @@ module RuboCop
 
         def on_send(node)
           return unless callback_method?(node)
-          
+
           # Only check symbol arguments (method name references)
           node.arguments.each do |arg|
             check_callback_argument(node, arg) if arg.sym_type?
@@ -100,11 +100,11 @@ module RuboCop
           return 0 unless method_node.body
 
           body = method_node.body
-          
+
           # Calculate line count
           first_line = body.first_line
           last_line = body.last_line
-          
+
           # Count non-empty lines
           (first_line..last_line).count do |line_number|
             line = processed_source.lines[line_number - 1]

data/lib/rubocop/cop/callback_checker/conditional_style.rb

@@ -26,8 +26,8 @@ module RuboCop
       #     status == 'active' && !deleted?
       #   end
       class ConditionalStyle < Base
-        MSG = "Use a named method instead of a %<type>s for callback conditionals. " \
-              "Extract the logic to a private method with a descriptive name."
+        MSG = 'Use a named method instead of a %<type>s for callback conditionals. ' \
+              'Extract the logic to a private method with a descriptive name.'
 
         CALLBACK_METHODS = %i[
           before_validation after_validation
@@ -44,7 +44,7 @@ module RuboCop
 
         def on_send(node)
           return unless callback_method?(node)
-          
+
           check_callback_conditionals(node)
         end
 
@@ -79,7 +79,7 @@ module RuboCop
 
         def conditional_key?(key)
           return false unless key.sym_type?
-          
+
           CONDITIONAL_KEYS.include?(key.value)
         end
       end

data/lib/rubocop/cop/callback_checker/no_side_effects_in_callbacks.rb

@@ -14,8 +14,8 @@ module RuboCop
       #   # good
       #   after_commit :notify_external_api, on: :create
       class NoSideEffectsInCallbacks < Base
-        MSG = "Avoid side effects (API calls, mailers, background jobs, or modifying other records) " \
-              "in %<callback>s. Use `after_commit` instead."
+        MSG = 'Avoid side effects (API calls, mailers, background jobs, or modifying other records) ' \
+              'in %<callback>s. Use `after_commit` instead.'
 
         SIDE_EFFECT_SENSITIVE_CALLBACKS = %i[
           before_validation before_save after_save
@@ -123,12 +123,12 @@ module RuboCop
         def part_of_reported_chain?(send_node)
           parent = send_node.parent
           return false unless parent&.send_type?
-          
+
           # If parent is also a side effect, we'll report the parent instead
           # This handles cases like UserMailer.welcome(self).deliver_now
           # We want to report the .deliver_now, not the .welcome
           return false if async_delivery?(send_node) # Always report delivery methods
-          
+
           # If this is a receiver of a delivery method, don't report it
           parent.receiver == send_node && async_delivery?(parent)
         end

data/lib/rubocop/cop/callback_checker_cops.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+require_relative 'callback_checker/no_side_effects_in_callbacks'
+require_relative 'callback_checker/avoid_self_persistence'
+require_relative 'callback_checker/attribute_assignment_only'
+require_relative 'callback_checker/callback_method_length'
+require_relative 'callback_checker/conditional_style'

data/lib/rubocop/cop/cops.rb

@@ -1,5 +1,7 @@
-require_relative "callback_checker/no_side_effects_in_callbacks"
-require_relative "callback_checker/avoid_self_persistence"
-require_relative "callback_checker/attribute_assignment_only"
-require_relative "callback_checker/callback_method_length"
-require_relative "callback_checker/conditional_style"
+# frozen_string_literal: true
+
+require_relative 'callback_checker/no_side_effects_in_callbacks'
+require_relative 'callback_checker/avoid_self_persistence'
+require_relative 'callback_checker/attribute_assignment_only'
+require_relative 'callback_checker/callback_method_length'
+require_relative 'callback_checker/conditional_style'

data/lib/rubocop-callback_checker.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+require_relative 'rubocop/callback_checker'
+require_relative 'rubocop/cop/callback_checker_cops'

metadata

@@ -1,29 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-callback_checker
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.2
 platform: ruby
 authors:
 - rahsheen
 autorequire:
-bindir: exe
+bindir: bin
 cert_chain: []
-date: 2026-03-13 00:00:00.000000000 Z
+date: 2026-03-15 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: lint_roller
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.72.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: 1.72.0
 - !ruby/object:Gem::Dependency
   name: rubocop-ast
   requirement: !ruby/object:Gem::Requirement
@@ -81,38 +95,37 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-rspec
+  name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
-  name: rubocop-rake
+  name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.7'
+        version: '2.0'
 description: A RuboCop extension focused on avoiding callback hell in Rails.
 email:
 - rahsheen.porter@gmail.com
-executables:
-- rubocop-callback-checker
+executables: []
 extensions: []
 extra_rdoc_files: []
 files:
@@ -124,16 +137,16 @@ files:
 - README.md
 - Rakefile
 - config/default.yml
-- exe/rubocop-callback-checker
-- lib/callback_checker/cli.rb
-- lib/callback_checker/prism_analyzer.rb
+- lib/rubocop-callback_checker.rb
 - lib/rubocop/callback_checker.rb
+- lib/rubocop/callback_checker/plugin.rb
 - lib/rubocop/callback_checker/version.rb
 - lib/rubocop/cop/callback_checker/attribute_assignment_only.rb
 - lib/rubocop/cop/callback_checker/avoid_self_persistence.rb
 - lib/rubocop/cop/callback_checker/callback_method_length.rb
 - lib/rubocop/cop/callback_checker/conditional_style.rb
 - lib/rubocop/cop/callback_checker/no_side_effects_in_callbacks.rb
+- lib/rubocop/cop/callback_checker_cops.rb
 - lib/rubocop/cop/cops.rb
 - sig/rubocop/callback_checker.rbs
 homepage: https://github.com/rahsheen/rubocop-callback_checker
@@ -144,6 +157,8 @@ metadata:
   homepage_uri: https://github.com/rahsheen/rubocop-callback_checker
   source_code_uri: https://github.com/rahsheen/rubocop-callback_checker
   changelog_uri: https://github.com/rahsheen/rubocop-callback_checker/blob/main/CHANGELOG.md
+  default_lint_roller_plugin: RuboCop::CallbackChecker::Plugin
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -162,5 +177,5 @@ requirements: []
 rubygems_version: 3.5.9
 signing_key:
 specification_version: 4
-summary: rubocop
+summary: RuboCop extension for checking ActiveRecord callbacks
 test_files: []

data/exe/rubocop-callback-checker

@@ -1,6 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-require_relative '../lib/callback_checker/cli'
-
-exit CallbackChecker::CLI.run(ARGV)

data/lib/callback_checker/cli.rb

@@ -1,104 +0,0 @@
-# frozen_string_literal: true
-
-require_relative 'prism_analyzer'
-require_relative '../rubocop/callback_checker/version'
-require 'optparse'
-
-module CallbackChecker
-  VERSION = Rubocop::CallbackChecker::VERSION
-
-  class CLI
-    def self.run(argv)
-      new(argv).run
-    end
-
-    def initialize(argv)
-      @argv = argv
-      @paths = []
-      @options = {}
-    end
-
-    def run
-      parse_options
-
-      if @paths.empty?
-        puts "Usage: rubocop-callback-checker [options] FILE..."
-        puts "Try 'rubocop-callback-checker --help' for more information."
-        return 1
-      end
-
-      files = collect_files(@paths)
-      
-      if files.empty?
-        puts "No Ruby files found to analyze."
-        return 1
-      end
-
-      total_offenses = 0
-
-      files.each do |file|
-        offenses = PrismAnalyzer.analyze_file(file)
-        
-        if offenses.any?
-          total_offenses += offenses.size
-          print_offenses(file, offenses)
-        end
-      end
-
-      print_summary(files.size, total_offenses)
-
-      total_offenses > 0 ? 1 : 0
-    end
-
-    private
-
-    def parse_options
-      OptionParser.new do |opts|
-        opts.banner = "Usage: rubocop-callback-checker [options] FILE..."
-
-        opts.on("-h", "--help", "Print this help") do
-          puts opts
-          exit 0
-        end
-
-        opts.on("-v", "--version", "Print version") do
-          puts "rubocop-callback-checker version #{VERSION}"
-          exit 0
-        end
-      end.parse!(@argv)
-
-      @paths = @argv
-    end
-
-    def collect_files(paths)
-      files = []
-
-      paths.each do |path|
-        if File.file?(path)
-          files << path if path.end_with?('.rb')
-        elsif File.directory?(path)
-          files.concat(Dir.glob(File.join(path, '**', '*.rb')))
-        else
-          warn "Warning: #{path} is not a file or directory"
-        end
-      end
-
-      files
-    end
-
-    def print_offenses(file, offenses)
-      puts "\n#{file}"
-      
-      offenses.each do |offense|
-        location = offense[:location]
-        puts "  #{location[:start_line]}:#{location[:start_column]}: #{offense[:message]}"
-        puts "    #{offense[:code]}"
-      end
-    end
-
-    def print_summary(file_count, offense_count)
-      puts "\n" + "=" * 80
-      puts "#{file_count} file(s) inspected, #{offense_count} offense(s) detected"
-    end
-  end
-end

data/lib/callback_checker/prism_analyzer.rb

@@ -1,252 +0,0 @@
-# frozen_string_literal: true
-
-require 'prism'
-
-module CallbackChecker
-  class PrismAnalyzer < Prism::Visitor
-    attr_reader :offenses
-
-    SIDE_EFFECT_SENSITIVE_CALLBACKS = %i[
-      before_validation before_save after_save
-      before_create after_create
-      before_update after_update
-      before_destroy after_destroy
-      around_save around_create around_update around_destroy
-    ].freeze
-
-    SAFE_CALLBACKS = %i[
-      after_commit after_create_commit after_update_commit
-      after_destroy_commit after_save_commit after_rollback
-    ].freeze
-
-    SUSPICIOUS_CONSTANTS = %w[
-      RestClient Faraday HTTParty Net Sidekiq ActionCable
-    ].freeze
-
-    SIDE_EFFECT_METHODS = %i[
-      deliver_later deliver_now perform_later broadcast_later
-      save save! update update! destroy destroy! create create!
-      delete delete_all destroy_all update_all update_columns touch
-    ].freeze
-
-    def initialize(source)
-      @source = source
-      @offenses = []
-      @current_class = nil
-      @callback_methods = {}
-      @current_callback_type = nil
-    end
-
-    def visit_class_node(node)
-      previous_class = @current_class
-      previous_methods = @callback_methods.dup
-
-      @current_class = node
-      @callback_methods = {}
-
-      # First pass: collect all method definitions
-      collect_methods(node)
-
-      # Second pass: check callbacks
-      super
-
-      @current_class = previous_class
-      @callback_methods = previous_methods
-    end
-
-    def visit_call_node(node)
-      if callback_declaration?(node)
-        check_callback(node)
-      elsif @current_callback_type && side_effect_call?(node)
-        add_offense(node, @current_callback_type)
-      end
-
-      super
-    end
-
-    def visit_if_node(node)
-      # Make sure we visit all branches of conditionals
-      super
-    end
-
-    def visit_unless_node(node)
-      # Make sure we visit all branches of unless statements
-      super
-    end
-
-    private
-
-    def collect_methods(class_node)
-      class_node.body&.body&.each do |statement|
-        if statement.is_a?(Prism::DefNode)
-          @callback_methods[statement.name] = statement
-        end
-      end
-    end
-
-    def callback_declaration?(node)
-      return false unless node.receiver.nil?
-      return false unless node.name
-
-      callback_name = node.name
-      SIDE_EFFECT_SENSITIVE_CALLBACKS.include?(callback_name) ||
-        SAFE_CALLBACKS.include?(callback_name)
-    end
-
-    def check_callback(node)
-      callback_name = node.name
-
-      # Skip safe callbacks
-      return if SAFE_CALLBACKS.include?(callback_name)
-
-      if node.block
-        # Block form: before_save do ... end
-        check_block_callback(node, callback_name)
-      elsif node.arguments
-        # Symbol form: before_save :method_name
-        check_symbol_callback(node, callback_name)
-      end
-    end
-
-    def check_block_callback(node, callback_name)
-      previous_callback = @current_callback_type
-      @current_callback_type = callback_name
-
-      visit(node.block)
-
-      @current_callback_type = previous_callback
-    end
-
-    def check_symbol_callback(node, callback_name)
-      return unless node.arguments&.arguments
-
-      node.arguments.arguments.each do |arg|
-        next unless arg.is_a?(Prism::SymbolNode)
-
-        method_name = arg.value
-        method_def = @callback_methods[method_name.to_sym]
-
-        next unless method_def
-
-        check_method_for_side_effects(method_def, callback_name)
-      end
-    end
-
-    def check_method_for_side_effects(method_node, callback_name)
-      previous_callback = @current_callback_type
-      @current_callback_type = callback_name
-
-      visit(method_node.body) if method_node.body
-
-      @current_callback_type = previous_callback
-    end
-
-    def side_effect_call?(node)
-      return false unless node.is_a?(Prism::CallNode)
-
-      # Check for suspicious constant calls (RestClient.get, etc.)
-      if node.receiver.is_a?(Prism::ConstantReadNode)
-        constant_name = node.receiver.name.to_s
-        return true if SUSPICIOUS_CONSTANTS.include?(constant_name)
-        
-        # Check for any constant that isn't a known safe pattern
-        # This catches things like NewsletterSDK, CustomAPI, etc.
-        return true if constant_appears_to_be_external_service?(constant_name)
-      end
-
-      # Check for side effect methods
-      method_name = node.name
-      return true if SIDE_EFFECT_METHODS.include?(method_name)
-
-      # Check for mailer patterns (anything ending with Mailer)
-      if node.receiver.is_a?(Prism::ConstantReadNode)
-        constant_name = node.receiver.name.to_s
-        return true if constant_name.end_with?('Mailer')
-      end
-
-      # Check for method chains that end with deliver_now
-      if method_name == :deliver_now && node.receiver.is_a?(Prism::CallNode)
-        return true
-      end
-
-      # Check for calls on associations or other objects (not self)
-      if node.receiver && !self_reference?(node.receiver)
-        return true if persistence_method?(method_name)
-      end
-
-      # Check for save/update on self or implicit self
-      if node.receiver.nil? || self_reference?(node.receiver)
-        return true if %i[save save! update update!].include?(method_name)
-      end
-
-      false
-    end
-
-    def constant_appears_to_be_external_service?(constant_name)
-      # Heuristic: if it's all caps or ends with SDK, API, Client, Service
-      # it's probably an external service
-      return true if constant_name.end_with?('SDK', 'API', 'Client', 'Service')
-      return true if constant_name == constant_name.upcase && constant_name.length > 1
-      
-      false
-    end
-
-    def self_reference?(node)
-      node.is_a?(Prism::SelfNode)
-    end
-
-    def persistence_method?(method_name)
-      %i[
-        save save! update update! destroy destroy! create create!
-        delete delete_all destroy_all update_all update_columns touch
-      ].include?(method_name)
-    end
-
-    def add_offense(node, callback_type)
-      location = node.location
-      start_line = location.start_line
-      start_column = location.start_column
-      end_line = location.end_line
-      end_column = location.end_column
-
-      # Extract the source code for this node
-      source_range = location.start_offset...location.end_offset
-      code = @source[source_range]
-
-      @offenses << {
-        message: "Avoid side effects (API calls, mailers, background jobs, or modifying other records) in #{callback_type}. Use `after_commit` instead.",
-        location: {
-          start_line: start_line,
-          start_column: start_column,
-          end_line: end_line,
-          end_column: end_column
-        },
-        code: code,
-        callback_type: callback_type
-      }
-    end
-
-    class << self
-      def analyze_file(path)
-        source = File.read(path)
-        analyze_source(source, path)
-      end
-
-      def analyze_source(source, path = nil)
-        result = Prism.parse(source)
-
-        if result.errors.any?
-          warn "Parse errors in #{path}:" if path
-          result.errors.each do |error|
-            warn "  #{error.message}"
-          end
-          return []
-        end
-
-        analyzer = new(source)
-        analyzer.visit(result.value)
-        analyzer.offenses
-      end
-    end
-  end
-end