eager_eye 1.2.12 → 1.2.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d6d4e056fa0d51ee629c031675f0e1b2ae2de62a1ebde0161f9d8d3d6f860f90
-  data.tar.gz: e8c8239cc5017c68b1d4b91279a212086b1741493327eea0bc0059622de83116
+  metadata.gz: c9b4195fa833057ee04ef87d148e78de8463fdbca97e234ba9d5b08ecaab9afc
+  data.tar.gz: f5c45a109aba0f563866669c4b6a32ce0823990fe5de467d65f6baea8d405197
 SHA512:
-  metadata.gz: deb89e2a0cedaa0d0fc41874a124725c1403634a105f157e66c52b6f80c63d02bf120bd4aa43765b452951b648972ad442deea3ca970d5355c4f79c34a3b5def
-  data.tar.gz: dcdbff4d567c605f0525c56d1e205e33e9383098eb30428bd336913917367afa00ff3b3bae893ba3b961d7bcb069528b28d44dd4a9613904aff29344bdfd8865
+  metadata.gz: 8312903e2476381ad7d9fe8a5a22feb74eb97937c5d488efc96448df9b0f5973f9d612c35cccd5235900e2eb25011694c862bb1a45a5bdeb70b2a8e17be155e2
+  data.tar.gz: cc7b4336007a72b0d4131b311a2897d4dae9cf0b4ba18ceb4826f25931875f418067bd5a16e6d1fec472cbb29abd7d1f2d2f0b58cc943b871fdea4e9991fa623

data/README.md

@@ -57,7 +57,8 @@
 
 🔧 **Developer-friendly:**
 - Inline suppression (like RuboCop)
-- Auto-fix support (experimental)
+- Auto-fix support (3 fixers: PluckToSelect, CountToSize, AddIncludes)
+- `.jbuilder` file support (`json.array!` iteration detection)
 - JSON/Console output formats
 - RSpec integration
 
@@ -524,6 +525,8 @@ eager_eye --fix --force
 | Issue | Fix |
 |-------|-----|
 | `.pluck(:id)` inline | → `.select(:id)` |
+| `.count` in iteration | → `.size` |
+| Missing `includes` before loop | → `.includes(:assoc)` inserted |
 
 ### Example
 
@@ -535,6 +538,15 @@ app/services/user_service.rb:
     - Post.where(user_id: User.active.pluck(:id))
     + Post.where(user_id: User.active.select(:id))
 
+app/controllers/posts_controller.rb:
+  Line 8:
+    - user.posts.count
+    + user.posts.size
+
+  Line 5:
+    - @posts.each do |post|
+    + @posts.includes(:author).each do |post|
+
 $ eager_eye --fix
 app/services/user_service.rb:12
   - Post.where(user_id: User.active.pluck(:id))
@@ -739,7 +751,7 @@ EagerEye uses static analysis, which means:
 - **No runtime context** - Cannot know if associations are already eager loaded elsewhere
 - **Heuristic-based** - Uses naming conventions to identify associations (may have false positives)
 - **Ruby code only** - Does not analyze SQL queries or ActiveRecord internals
-- **Cross-file scope** - Cross-file analysis is limited to model-defined methods; controller-to-view or service-to-service patterns are not yet tracked
+- **Cross-file scope** - Cross-file analysis covers model-defined query methods; controller-to-view or service-to-service patterns are not yet tracked
 
 For best results, use EagerEye alongside runtime tools like Bullet for comprehensive N+1 detection.
 

data/lib/eager_eye/fixers/add_includes.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+module EagerEye
+  module Fixers
+    class AddIncludes < Base
+      ITERATION_METHODS_RE = %w[
+        each map collect select find_all reject filter filter_map
+        flat_map find_each find_in_batches in_batches
+      ].join("|")
+      ITERATION_PATTERN = /\.(#{ITERATION_METHODS_RE})\b/
+
+      def fixable?
+        issue.detector == :loop_association &&
+          !association_name.nil? &&
+          !iteration_line_index.nil?
+      end
+
+      def diff
+        return nil unless fixable?
+
+        idx = iteration_line_index
+        original_line = @source_lines[idx]
+        fixed_line = insert_includes(original_line)
+        return nil if original_line == fixed_line
+
+        {
+          file: issue.file_path,
+          line: idx + 1,
+          original: original_line.chomp,
+          fixed: fixed_line.chomp
+        }
+      end
+
+      protected
+
+      def fixed_content
+        raise NotImplementedError
+      end
+
+      private
+
+      def association_name
+        return nil unless issue.suggestion
+
+        match = issue.suggestion.match(/includes\(:(\w+)\)/)
+        match && match[1]
+      end
+
+      def iteration_line_index
+        start = issue.line_number - 2
+        start.downto([start - 10, 0].max) do |i|
+          return i if @source_lines[i]&.match?(ITERATION_PATTERN)
+        end
+        nil
+      end
+
+      def insert_includes(line)
+        line.sub(ITERATION_PATTERN, ".includes(:#{association_name})\\0")
+      end
+    end
+  end
+end

data/lib/eager_eye/fixers/count_to_size.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module EagerEye
+  module Fixers
+    class CountToSize < Base
+      def fixable?
+        issue.detector == :count_in_iteration &&
+          single_line_count?
+      end
+
+      protected
+
+      def fixed_content
+        line_content.sub(/\.count\b/, ".size")
+      end
+
+      private
+
+      def single_line_count?
+        line_content&.match?(/\.count\b/)
+      end
+    end
+  end
+end

data/lib/eager_eye/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.2.12
+  version: 1.2.13
 platform: ruby
 authors:
 - hamzagedikkaya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-19 00:00:00.000000000 Z
+date: 2026-03-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast
@@ -80,7 +80,9 @@ files:
 - lib/eager_eye/detectors/serializer_nesting.rb
 - lib/eager_eye/detectors/validation_n_plus_one.rb
 - lib/eager_eye/fixer_registry.rb
+- lib/eager_eye/fixers/add_includes.rb
 - lib/eager_eye/fixers/base.rb
+- lib/eager_eye/fixers/count_to_size.rb
 - lib/eager_eye/fixers/pluck_to_select.rb
 - lib/eager_eye/generators/install_generator.rb
 - lib/eager_eye/issue.rb