eager_eye 1.2.6 → 1.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f996747b6c12e539180e66cc74c11c70acfab4a6e4b84aed4039cc7b58dafe53
-  data.tar.gz: f23d5a21ceecc39c71b0303a5748472a8ea5e0f5f3256ffd5a10084a5af8a39c
+  metadata.gz: dd5177a12d64b54cc6cd426440f79018da9ccc7dd931db70c491c9154bb385a1
+  data.tar.gz: dec16deb3b55ab821225d5d0e06a6fa308c41fd472dad405bf125c7e5839f2e0
 SHA512:
-  metadata.gz: a75d25669ab2d7e96dbccb107ae30f817e5fd6a291651b68bc56dbea21f50bd5c8b24b0633dd21d6faeeb5a8abd999abc6baf57ac9c7e8c3c06aa4542dc570d3
-  data.tar.gz: 7d2e5af151c02923bf4e64f05bc277711228729fa08ddee3dd79699623efc33f3689290c1739c025d7851db0b59d90feeac99e3a65c9a13862fac8f36ddf46b6
+  metadata.gz: 4336421ff71269732f594163dc3dbe29ca70d1b10c19dd333d972ea63bff9da36a4e83a041263fa6f61d758c8b0551912e075e00ce8f05978ab59c7db698af41
+  data.tar.gz: 51de528f0dca6b3279c0cb1fe6f21e5c35592dda81220419c209d8d43faf35b41375cbfffd33a463546f5e81e4807f8c48268f618d8383c82a570f81da823efa

data/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.2.7] - 2026-03-10
+
+### Added
+
+- **New Detector: `ScopeChainNPlusOne`** - Detects scope calls on associations inside iterations
+  - Catches `post.comments.recent`, `post.comments.approved.count` patterns
+  - Parses model files for `scope :name, -> { ... }` declarations
+  - Flags known scope names called on association chains inside loops
+  - Each scope call executes a new query per iteration — suggests preloading or joined queries
+- **New Parser: `ScopeParser`** - Collects scope definitions from model files for cross-file detection
+
 ## [1.2.6] - 2026-02-25
 
 ### Changed

data/README.md

@@ -10,7 +10,7 @@
 
 <p align="center">
   <a href="https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml"><img src="https://github.com/hamzagedikkaya/eager_eye/actions/workflows/main.yml/badge.svg" alt="CI"></a>
-  <a href="https://rubygems.org/gems/eager_eye"><img src="https://img.shields.io/badge/gem-v1.2.6-red.svg" alt="Gem Version"></a>
+  <a href="https://rubygems.org/gems/eager_eye"><img src="https://img.shields.io/badge/gem-v1.2.7-red.svg" alt="Gem Version"></a>
   <a href="https://github.com/hamzagedikkaya/eager_eye"><img src="https://img.shields.io/badge/coverage-95%25-brightgreen.svg" alt="Coverage"></a>
   <a href="https://www.ruby-lang.org/"><img src="https://img.shields.io/badge/ruby-%3E%3D%203.1-ruby.svg" alt="Ruby"></a>
   <a href="https://opensource.org/licenses/MIT"><img src="https://img.shields.io/badge/License-MIT-yellow.svg" alt="License: MIT"></a>
@@ -42,7 +42,7 @@
 
 ## Features
 
-✨ **Detects 9 types of N+1 problems:**
+✨ **Detects 10 types of N+1 problems:**
 - Loop associations (queries in iterations)
 - Serializer nesting issues
 - Missing counter caches
@@ -52,6 +52,7 @@
 - Pluck to array misuse
 - Delegation N+1s (hidden via `delegate :method, to: :association`)
 - Decorator N+1s (Draper, SimpleDelegator, Presenter, ViewObject)
+- Scope chain N+1s (named scopes on associations in loops)
 
 🔧 **Developer-friendly:**
 - Inline suppression (like RuboCop)
@@ -398,6 +399,33 @@ Supports the following object references inside decorators:
 - `__getobj__` — SimpleDelegator standard
 - `source`, `model` — alternative Draper aliases
 
+### 10. Scope Chain N+1
+
+Detects named scope calls on associations inside iterations. Unlike explicit query methods (`.where`, `.find_by`) caught by `CustomMethodQuery`, named scopes (`.recent`, `.active`, `.published`) are invisible query triggers.
+
+```ruby
+# Model
+class Comment < ApplicationRecord
+  scope :recent, -> { where("created_at > ?", 1.week.ago) }
+  scope :approved, -> { where(approved: true) }
+end
+
+# Bad - scope call per iteration
+posts.each do |post|
+  post.comments.recent          # Query for each post!
+  post.comments.approved.count  # Query for each post!
+end
+
+# Good - preload and filter in Ruby
+posts.includes(:comments).each do |post|
+  post.comments.select { |c| c.created_at > 1.week.ago }
+end
+```
+
+EagerEye detects these by:
+1. Scanning model files for `scope :name, -> { ... }` declarations
+2. Flagging known scope names called on association chains inside iteration blocks
+
 ## Inline Suppression
 
 Suppress false positives using inline comments (RuboCop-style):
@@ -442,6 +470,7 @@ Both CamelCase and snake_case formats are accepted:
 | Pluck to Array | `PluckToArray` | `pluck_to_array` |
 | Delegation N+1 | `DelegationNPlusOne` | `delegation_n_plus_one` |
 | Decorator N+1 | `DecoratorNPlusOne` | `decorator_n_plus_one` |
+| Scope Chain N+1 | `ScopeChainNPlusOne` | `scope_chain_n_plus_one` |
 | All Detectors | `all` | `all` |
 
 ## Auto-fix (Experimental)
@@ -546,18 +575,20 @@ enabled_detectors:
   - pluck_to_array
   - delegation_n_plus_one
   - decorator_n_plus_one
+  - scope_chain_n_plus_one
 
 # Severity levels per detector (error, warning, info)
 severity_levels:
-  loop_association: error        # Definite N+1
+  loop_association: error          # Definite N+1
   serializer_nesting: warning
   custom_method_query: warning
   count_in_iteration: warning
   callback_query: warning
-  pluck_to_array: warning        # Optimization
-  delegation_n_plus_one: warning # Hidden delegation N+1
-  decorator_n_plus_one: warning  # Decorator/Presenter N+1
-  missing_counter_cache: info    # Suggestion
+  pluck_to_array: warning          # Optimization
+  delegation_n_plus_one: warning   # Hidden delegation N+1
+  decorator_n_plus_one: warning    # Decorator/Presenter N+1
+  scope_chain_n_plus_one: warning  # Scope chain on association
+  missing_counter_cache: info      # Suggestion
 
 # Minimum severity to report (default: info)
 min_severity: warning

data/lib/eager_eye/analyzer.rb

@@ -13,16 +13,18 @@ module EagerEye
       callback_query: Detectors::CallbackQuery,
       pluck_to_array: Detectors::PluckToArray,
       delegation_n_plus_one: Detectors::DelegationNPlusOne,
-      decorator_n_plus_one: Detectors::DecoratorNPlusOne
+      decorator_n_plus_one: Detectors::DecoratorNPlusOne,
+      scope_chain_n_plus_one: Detectors::ScopeChainNPlusOne
     }.freeze
 
-    attr_reader :paths, :issues, :association_preloads, :delegation_maps
+    attr_reader :paths, :issues, :association_preloads, :delegation_maps, :scope_maps
 
     def initialize(paths: nil)
       @paths = Array(paths || EagerEye.configuration.app_path)
       @issues = []
       @association_preloads = {}
       @delegation_maps = {}
+      @scope_maps = {}
     end
 
     def run
@@ -48,6 +50,10 @@ module EagerEye
         deleg_parser = DelegationParser.new
         deleg_parser.parse_model(ast, model_name)
         @delegation_maps.merge!(deleg_parser.delegation_maps)
+
+        scope_parser = ScopeParser.new
+        scope_parser.parse_model(ast, model_name)
+        @scope_maps.merge!(scope_parser.scope_maps)
       rescue Errno::ENOENT, Errno::EACCES
         next
       end
@@ -108,6 +114,7 @@ module EagerEye
       args = [ast, file_path]
       args << @association_preloads if detector.is_a?(Detectors::LoopAssociation)
       args << @delegation_maps if detector.is_a?(Detectors::DelegationNPlusOne)
+      args << @scope_maps if detector.is_a?(Detectors::ScopeChainNPlusOne)
       args
     end
 

data/lib/eager_eye/configuration.rb

@@ -9,6 +9,7 @@ module EagerEye
       loop_association serializer_nesting missing_counter_cache
       custom_method_query count_in_iteration callback_query
       pluck_to_array delegation_n_plus_one decorator_n_plus_one
+      scope_chain_n_plus_one
     ].freeze
 
     DEFAULT_SEVERITY_LEVELS = {
@@ -20,7 +21,8 @@ module EagerEye
       callback_query: :warning,
       pluck_to_array: :warning,
       delegation_n_plus_one: :warning,
-      decorator_n_plus_one: :warning
+      decorator_n_plus_one: :warning,
+      scope_chain_n_plus_one: :warning
     }.freeze
 
     VALID_SEVERITIES = %i[info warning error].freeze

data/lib/eager_eye/version.rb

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

data/lib/eager_eye.rb

@@ -6,6 +6,7 @@ require_relative "eager_eye/configuration"
 require_relative "eager_eye/issue"
 require_relative "eager_eye/association_parser"
 require_relative "eager_eye/delegation_parser"
+require_relative "eager_eye/scope_parser"
 require_relative "eager_eye/detectors/base"
 require_relative "eager_eye/detectors/loop_association"
 require_relative "eager_eye/detectors/serializer_nesting"
@@ -16,6 +17,7 @@ require_relative "eager_eye/detectors/callback_query"
 require_relative "eager_eye/detectors/pluck_to_array"
 require_relative "eager_eye/detectors/delegation_n_plus_one"
 require_relative "eager_eye/detectors/decorator_n_plus_one"
+require_relative "eager_eye/detectors/scope_chain_n_plus_one"
 require_relative "eager_eye/comment_parser"
 require_relative "eager_eye/analyzer"
 require_relative "eager_eye/fixers/base"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.7
 platform: ruby
 authors:
 - hamzagedikkaya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-02-25 00:00:00.000000000 Z
+date: 2026-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast