eager_eye 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9a37ac20117548282deb586bacdab0deab006b41581efa0810d3b64ffdcf1465
-  data.tar.gz: fa2dc1e3b2fa8582c0b48606ee88a75de9f7d6638593ea3404493b1383ad9512
+  metadata.gz: ecc47488d2632329805448f55dfafc1f6670efd02b04a3dd0464c22ef9a3a4ba
+  data.tar.gz: b37b1be91854abeea6a11d33f28df1ada2a3fb0a427b28a892cb1249c22a7e67
 SHA512:
-  metadata.gz: 8d4c41a777a9d79da47c9b0a5ce467acd951fdf1e72682e0ea235b5cc10329c403177c039ee7beb42a2f0b025796e6484d5e412f0e7697ad66902961f1348c2c
-  data.tar.gz: e293e6c338bc7413d922e3d0daad30e0033477b7819f5b95b551dd8aebe53d0efd350615ece1ff6cadabc03267fcce7aaa08814deb8fb16620f0974daa41a597
+  metadata.gz: 19adae5e3c000cfa658f5d2615da31c9a702ff031766240b86968d69165445bbbc7b630b5a1596822eaa9683c23db84379127d0c83ed4a59366475b3585972b7
+  data.tar.gz: a87d19f36dcaf9bd7a360b31051ed32da241e7c0b867815e8e17962f4b29bdd43d6983d84981e54a350ba4d415a076a84a8b1abc6852311277f25398c51d6a1a

data/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.0.3] - 2025-12-20
+
+### Fixed
+
+- Only flag queries in callbacks where the iteration variable is the receiver (reduces false positives)
+- Updated README callback query documentation to accurately reflect current behavior
+
+## [1.0.2] - 2025-12-19
+
+### Fixed
+
+- Only detect queries inside iterations in callback detector
+
 ## [1.0.1] - 2025-12-16
 
 ### Changed

data/README.md

@@ -1,7 +1,7 @@
 # EagerEye
 
 [![CI](https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml/badge.svg)](https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml)
-[![Gem Version](https://img.shields.io/badge/gem-v1.0.1-red.svg)](https://rubygems.org/gems/eager_eye)
+[![Gem Version](https://img.shields.io/badge/gem-v1.0.3-red.svg)](https://rubygems.org/gems/eager_eye)
 [![Coverage](https://img.shields.io/badge/coverage-95%25-brightgreen.svg)](https://github.com/hamzagedikkaya/eager_eye)
 [![Ruby](https://img.shields.io/badge/ruby-%3E%3D%203.1-ruby.svg)](https://www.ruby-lang.org/)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
@@ -209,42 +209,45 @@ user.posts_count  # Just reads the column
 
 ### 6. Callback Query Detection
 
-Detects database queries and iterations inside ActiveRecord callbacks. These become performance disasters during bulk operations.
+Detects N+1 patterns inside ActiveRecord callbacks - specifically iterations that execute queries on each loop.
 
 ```ruby
-# Bad - Queries in callbacks
-class Article < ApplicationRecord
-  after_save :recalculate_stats
+# Bad - N+1 in callback (DETECTED)
+class Order < ApplicationRecord
+  after_create :notify_subscribers
 
-  def recalculate_stats
-    author.articles.published.count  # Runs on EVERY save
-    category.update_article_count!   # Another query on EVERY save
+  def notify_subscribers
+    customer.followers.each do |follower|  # Error: Iteration in callback
+      follower.notifications.create!(...)  # Warning: Query on iteration variable
+    end
   end
 end
 
-# Disaster scenario
-Article.import(1000.times.map { |i| { title: "Post #{i}" } })
-# = 2000+ queries from callbacks!
+# OK - Single query in callback (NOT flagged - not N+1)
+class Article < ApplicationRecord
+  after_save :update_stats
 
-# Bad - N+1 in callback
-class Order < ApplicationRecord
-  after_create :notify_subscribers
+  def update_stats
+    author.articles.count  # Single query, acceptable
+  end
+end
 
-  def notify_subscribers
-    customer.followers.each do |follower|  # N+1!
-      NotificationMailer.new_order(follower).deliver_later
+# OK - Query not on iteration variable (NOT flagged)
+class Post < ApplicationRecord
+  after_save :process_items
+
+  def process_items
+    items.each do |item|
+      OtherModel.where(name: item.name).first  # OtherModel is receiver, not item
     end
   end
 end
 
-# Good - Use conditional callbacks
-after_save :recalculate_stats, if: :should_recalculate?
-
-# Good - Move to background job
-after_commit :schedule_stats_update, on: :create
+# Good - Move iterations to background job
+after_commit :schedule_notifications, on: :create
 
-def schedule_stats_update
-  RecalculateStatsJob.perform_later(id)
+def schedule_notifications
+  NotifySubscribersJob.perform_later(id)
 end
 ```
 

data/lib/eager_eye/detectors/callback_query.rb

@@ -106,8 +106,9 @@ module EagerEye
         return unless node.is_a?(Parser::AST::Node)
 
         if iteration_block?(node)
+          block_var = extract_block_variable(node)
           add_iteration_issue(node, method_name, callback_type)
-          find_query_calls_in_block(node, method_name, callback_type)
+          find_query_calls_in_block(node, method_name, callback_type, block_var) if block_var
         end
 
         node.children.each do |child|
@@ -115,13 +116,15 @@ module EagerEye
         end
       end
 
-      def find_query_calls_in_block(node, method_name, callback_type)
+      def find_query_calls_in_block(node, method_name, callback_type, block_var)
         return unless node.is_a?(Parser::AST::Node)
 
-        add_query_issue(node, method_name, callback_type) if query_call?(node)
+        if query_call?(node) && receiver_chain_starts_with?(node.children[0], block_var)
+          add_query_issue(node, method_name, callback_type)
+        end
 
         node.children.each do |child|
-          find_query_calls_in_block(child, method_name, callback_type)
+          find_query_calls_in_block(child, method_name, callback_type, block_var)
         end
       end
 
@@ -163,6 +166,29 @@ module EagerEye
           suggestion: "Avoid iterations in callbacks. Use background jobs for bulk operations"
         )
       end
+
+      def extract_block_variable(block_node)
+        args_node = block_node.children[1]
+        return nil unless args_node&.type == :args
+
+        first_arg = args_node.children[0]
+        return nil unless first_arg&.type == :arg
+
+        first_arg.children[0]
+      end
+
+      def receiver_chain_starts_with?(node, block_var)
+        return false unless node.is_a?(Parser::AST::Node)
+
+        case node.type
+        when :lvar
+          node.children[0] == block_var
+        when :send
+          receiver_chain_starts_with?(node.children[0], block_var)
+        else
+          false
+        end
+      end
     end
   end
 end

data/lib/eager_eye/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - hamzagedikkaya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-12-19 00:00:00.000000000 Z
+date: 2025-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast