eager_eye 1.1.0 → 1.1.2

data/lib/eager_eye/fixers/pluck_to_select.rb

@@ -3,27 +3,20 @@
 module EagerEye
   module Fixers
     class PluckToSelect < Base
-      # This fixer only works for single-line pluck + where patterns
-      # Two-line patterns are too complex to fix automatically
-
       def fixable?
-        issue.detector == :pluck_to_array &&
-          single_line_pattern?
+        issue.detector == :pluck_to_array && single_line_pattern?
       end
 
       protected
 
       def fixed_content
-        # Model.where(col: OtherModel.pluck(:id)) -> Model.where(col: OtherModel.select(:id))
         line_content.gsub(/\.pluck\((:\w+)\)/, '.select(\1)')
       end
 
       private
 
       def single_line_pattern?
-        return false unless line_content
-
-        line_content.include?(".pluck(") && line_content.include?(".where(")
+        line_content&.include?(".pluck(") && line_content.include?(".where(")
       end
     end
   end

data/lib/eager_eye/issue.rb

@@ -40,20 +40,13 @@ module EagerEye
     end
 
     def ==(other)
-      return false unless other.is_a?(Issue)
-
-      detector == other.detector &&
-        file_path == other.file_path &&
-        line_number == other.line_number &&
-        message == other.message &&
-        severity == other.severity &&
-        suggestion == other.suggestion
+      other.is_a?(Issue) && to_h == other.to_h
     end
 
     alias eql? ==
 
     def hash
-      [detector, file_path, line_number, message, severity, suggestion].hash
+      to_h.hash
     end
 
     private

data/lib/eager_eye/railtie.rb

@@ -10,32 +10,20 @@ module EagerEye
       namespace :eager_eye do
         desc "Analyze Rails application for N+1 query issues"
         task analyze: :environment do
-          require "eager_eye"
-
-          load_config_file
-
-          analyzer = EagerEye::Analyzer.new
-          issues = analyzer.run
-
-          reporter = EagerEye::Reporters::Console.new(issues)
-          puts reporter.report
-
-          exit 1 if issues.any? && EagerEye.configuration.fail_on_issues
+          puts run_analysis(EagerEye::Reporters::Console)
         end
 
         desc "Analyze and output results as JSON"
         task json: :environment do
-          require "eager_eye"
+          puts run_analysis(EagerEye::Reporters::Json, pretty: true)
+        end
 
+        def run_analysis(reporter_class, **opts)
+          require "eager_eye"
           load_config_file
-
-          analyzer = EagerEye::Analyzer.new
-          issues = analyzer.run
-
-          reporter = EagerEye::Reporters::Json.new(issues, pretty: true)
-          puts reporter.report
-
+          issues = EagerEye::Analyzer.new.run
           exit 1 if issues.any? && EagerEye.configuration.fail_on_issues
+          reporter_class.new(issues, **opts).report
         end
 
         def load_config_file
@@ -55,7 +43,6 @@ module EagerEye
       end
     end
 
-    # Generate initializer for configuration
     generators do
       require_relative "generators/install_generator"
     end

data/lib/eager_eye/reporters/console.rb

@@ -48,15 +48,7 @@ module EagerEye
       end
 
       def file_section(file_path, file_issues)
-        lines = []
-        lines << colorize(file_path, :cyan)
-
-        file_issues.each do |issue|
-          lines << format_issue(issue)
-        end
-
-        lines << ""
-        lines.join("\n")
+        [colorize(file_path, :cyan), *file_issues.map { |i| format_issue(i) }, ""].join("\n")
       end
 
       def format_issue(issue)
@@ -81,15 +73,12 @@ module EagerEye
       end
 
       def summary
-        total = issues.size
-        errors = error_count
-        warnings = warning_count
-        infos = info_count
-
-        "Total: #{total} issue#{"s" unless total == 1} " \
-          "(#{errors} error#{"s" unless errors == 1}, " \
-          "#{warnings} warning#{"s" unless warnings == 1}, " \
-          "#{infos} info)"
+        t = issues.size
+        e = error_count
+        w = warning_count
+        i = info_count
+        "Total: #{t} issue#{"s" unless t == 1} " \
+          "(#{e} error#{"s" unless e == 1}, #{w} warning#{"s" unless w == 1}, #{i} info)"
       end
 
       def colorize(text, color)

data/lib/eager_eye/rspec/matchers.rb

@@ -19,8 +19,7 @@ module EagerEye
         def matches?(path)
           @path = path
           configure_eager_eye
-          analyzer = build_analyzer
-          @issues = analyzer.run
+          @issues = EagerEye::Analyzer.new(paths: [@path]).run
           @issues.count <= @max_issues
         end
 
@@ -61,10 +60,6 @@ module EagerEye
             config.fail_on_issues = false
           end
         end
-
-        def build_analyzer
-          EagerEye::Analyzer.new(paths: [@path])
-        end
       end
     end
   end

data/lib/eager_eye/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module EagerEye
-  VERSION = "1.1.0"
+  VERSION = "1.1.2"
 end

data/lib/eager_eye.rb

@@ -41,5 +41,4 @@ module EagerEye
   end
 end
 
-# Load Railtie only if Rails is defined
 require_relative "eager_eye/railtie" if defined?(Rails::Railtie)

data/sig/eager_eye.rbs

@@ -1,4 +1,3 @@
 module EagerEye
   VERSION: String
-  # See the writing guide of rbs: https://github.com/ruby/rbs#guides
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.2
 platform: ruby
 authors:
 - hamzagedikkaya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-12-28 00:00:00.000000000 Z
+date: 2026-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast