eager_eye 1.1.3 → 1.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 928603853f971cca5717d7db1ca527da41562b4aef18c27b37be3b2824a4993e
-  data.tar.gz: b3e7af83cfb0085a6e34ad65e3e633da6d3eaa517d548b530504dbb6f0a7c5c4
+  metadata.gz: 6c6cd435e8db21b831eef924b598c7316fea3a2bc28e2f06744ee0e69d420b9d
+  data.tar.gz: e777d0653d14a48afc157e109ef715150fee54b0f3271b43bf75898bc35d1099
 SHA512:
-  metadata.gz: 1879d46b6a428002adfc1d241b763fb8b59c5f6a0c82656459dbd389fe04d37ffcdf39394185141ce78837cec72386b689147171291a494f7484dbcae0bfe438
-  data.tar.gz: b6cdafbe35419653bd4d374504b1d32475c9cab7fc7ec41d8612c77924589aa1d950c819bd901f0196cdf1de33b2f114bbbbb5fcb11defb7d4624f5043356ee2
+  metadata.gz: 1817d044118d33fa27712adc6a1548ad2959344b48e76f2a2ba24def749b01fd5f7fe2126df538636df9a2b5ed3915d8001f11020a523e008e66ccc436a4bab7
+  data.tar.gz: 8b69f300d0692d49e63ddc15be8dd473cc4a1ce0c320af23e8145c75c8ad0e28446d3be9e49077111d8718aca9cf2f7ab1889cb83da314c40666ed3e483b0166

data/CHANGELOG.md

@@ -7,6 +7,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.1.5] - 2026-01-06
+
+### Changed
+
+- **PluckToArray Severity** - Lower severity to `info` for small collections
+  - `tags`, `settings`, `roles`, `permissions`, `options` etc. now `info` level
+  - Large collections remain `warning`, `.all.pluck` remains `error`
+
+## [1.1.4] - 2026-01-06
+
+### Fixed
+
+- **SerializerNesting False Positive** - Skip ActiveStorage attachments
+  - `user.avatar.attached?`, `user.avatar.variant(...)` no longer flagged
+  - Recognizes `attached?`, `attach`, `blob`, `variant`, `purge` methods
+
 ## [1.1.3] - 2026-01-04
 
 ### Fixed

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.1.3-red.svg" alt="Gem Version"></a>
+  <a href="https://rubygems.org/gems/eager_eye"><img src="https://img.shields.io/badge/gem-v1.1.5-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>

data/lib/eager_eye/detectors/pluck_to_array.rb

@@ -3,6 +3,8 @@
 module EagerEye
   module Detectors
     class PluckToArray < Base
+      SMALL_COLLECTIONS = %w[tags settings options categories roles permissions statuses types priorities].freeze
+
       def self.detector_name
         :pluck_to_array
       end
@@ -13,6 +15,7 @@ module EagerEye
         @pluck_variables = {}
         @map_id_variables = {}
         @critical_pluck_variables = {}
+        @small_collection_variables = {}
 
         return @issues unless ast
 
@@ -38,6 +41,7 @@ module EagerEye
         value = node.children[1]
 
         @critical_pluck_variables[var_name] = node.loc.line if all_pluck_call?(value)
+        @small_collection_variables[var_name] = node.loc.line if small_collection_pluck?(value)
         @pluck_variables[var_name] = node.loc.line if pluck_call?(value)
         @map_id_variables[var_name] = node.loc.line if map_id_call?(value)
       end
@@ -45,8 +49,13 @@ module EagerEye
       def check_where_calls(node)
         return unless where_call?(node)
 
-        add_critical_issue(node) if critical_pluck?(node)
-        add_issue(node) if regular_pluck?(node)
+        if critical_pluck?(node)
+          add_critical_issue(node)
+        elsif small_collection?(node)
+          add_info_issue(node)
+        elsif regular_pluck?(node)
+          add_issue(node)
+        end
       end
 
       def local_variable_assignment?(node) = node.type == :lvasgn
@@ -64,6 +73,16 @@ module EagerEye
         receiver.is_a?(Parser::AST::Node) && receiver.type == :send && receiver.children[1] == :all
       end
 
+      def small_collection_pluck?(node)
+        return false unless pluck_call?(node)
+
+        receiver = node.children[0]
+        return false unless receiver.is_a?(Parser::AST::Node) && receiver.type == :send
+
+        method_name = receiver.children[1].to_s
+        SMALL_COLLECTIONS.any? { |c| method_name.include?(c) }
+      end
+
       def map_id_call?(node)
         node.is_a?(Parser::AST::Node) && (block_map?(node) || send_map?(node))
       end
@@ -92,6 +111,10 @@ module EagerEye
         node.children[2..].any? { |arg| critical_pluck_in_hash?(arg) }
       end
 
+      def small_collection?(node)
+        node.children[2..].any? { |arg| small_collection_in_hash?(arg) }
+      end
+
       def pluck_var_in_hash?(node)
         return false unless node.is_a?(Parser::AST::Node) && node.type == :hash
 
@@ -104,6 +127,12 @@ module EagerEye
         node.children.any? { |pair| pair.type == :pair && critical_value?(pair.children[1]) }
       end
 
+      def small_collection_in_hash?(node)
+        return false unless node.is_a?(Parser::AST::Node) && node.type == :hash
+
+        node.children.any? { |pair| pair.type == :pair && small_collection_value?(pair.children[1]) }
+      end
+
       def pluck_value?(value)
         value.type == :lvar && (@pluck_variables.key?(value.children[0]) || @map_id_variables.key?(value.children[0]))
       end
@@ -112,6 +141,10 @@ module EagerEye
         value.type == :lvar ? @critical_pluck_variables.key?(value.children[0]) : all_pluck_call?(value)
       end
 
+      def small_collection_value?(value)
+        value.type == :lvar && @small_collection_variables.key?(value.children[0])
+      end
+
       def add_issue(node)
         @issues << create_issue(
           file_path: @file_path,
@@ -131,6 +164,16 @@ module EagerEye
           severity: :error
         )
       end
+
+      def add_info_issue(node)
+        @issues << create_issue(
+          file_path: @file_path,
+          line_number: node.loc.line,
+          message: "Small collection pluck may be acceptable for few records",
+          suggestion: "Consider `.select(:id)` for consistency, but pluck is fine for small collections",
+          severity: :info
+        )
+      end
     end
   end
 end

data/lib/eager_eye/detectors/serializer_nesting.rb

@@ -6,6 +6,8 @@ module EagerEye
       SERIALIZER_PATTERNS = %w[ActiveModel::Serializer ActiveModelSerializers::Model Blueprinter::Base Alba::Resource].freeze
       ATTRIBUTE_METHODS = %i[attribute field attributes].freeze
       OBJECT_REFS = %i[object record resource].freeze
+      ACTIVE_STORAGE_METHODS = %i[attached? attach attachment attachments blob blobs purge purge_later variant
+                                  preview].freeze
       HAS_MANY_ASSOCIATIONS = %w[
         authors users owners creators admins members customers clients
         posts articles comments categories tags children companies organizations
@@ -100,8 +102,11 @@ module EagerEye
       end
 
       def find_association_in_block(block_body, file_path, issues)
+        storage_lines = collect_active_storage_lines(block_body)
+
         traverse_ast(block_body) do |node|
           next unless node.type == :send
+          next if storage_lines.include?(node.loc.line)
 
           receiver = node.children[0]
           method_name = node.children[1]
@@ -116,6 +121,16 @@ module EagerEye
         end
       end
 
+      def collect_active_storage_lines(block_body)
+        lines = Set.new
+        traverse_ast(block_body) do |node|
+          next unless node.type == :send && ACTIVE_STORAGE_METHODS.include?(node.children[1])
+
+          lines << node.loc.line
+        end
+        lines
+      end
+
       def object_reference?(node)
         return false unless node
 

data/lib/eager_eye/version.rb

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

metadata

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