eager_eye 1.1.4 → 1.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6ae532397830e96202bc3a3d10dc231570f4912d5df67085783123fbb2ebba5e
-  data.tar.gz: ddc5a6971c2c65c73260c59ff8b6acfd52711a2e1ce1d43ea8c44895be729016
+  metadata.gz: 6c6cd435e8db21b831eef924b598c7316fea3a2bc28e2f06744ee0e69d420b9d
+  data.tar.gz: e777d0653d14a48afc157e109ef715150fee54b0f3271b43bf75898bc35d1099
 SHA512:
-  metadata.gz: fa15940b451ebd100e908b4f9931c8f40708f9566451a0f1edb8a65c70fee15f832bd30d91d49f54c9b2f0ebe63cbdf2f5dbabfb5f7894db23d1a2f35c1e95c5
-  data.tar.gz: 80ea91965317596061db8a38cce960ac4f573a4c887de7a2fe88b7a4ad7e43f35f2b18b6716a6487e256e1ed292e8f79c437249e9dbef6cccded54397a7ecf8c
+  metadata.gz: 1817d044118d33fa27712adc6a1548ad2959344b48e76f2a2ba24def749b01fd5f7fe2126df538636df9a2b5ed3915d8001f11020a523e008e66ccc436a4bab7
+  data.tar.gz: 8b69f300d0692d49e63ddc15be8dd473cc4a1ce0c320af23e8145c75c8ad0e28446d3be9e49077111d8718aca9cf2f7ab1889cb83da314c40666ed3e483b0166

data/CHANGELOG.md

@@ -7,6 +7,14 @@ 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

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.4-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/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.1.4
+  version: 1.1.5
 platform: ruby
 authors:
 - hamzagedikkaya