eager_eye 1.3.1 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: edf21044e597735154b0b1acdcda9230749bf027e8b8acafe70c6aaa1da22b62
-  data.tar.gz: b7d6b716ff42f9e160c292a54ecf0344027c73e24eae29f6aecd34aae27d60ac
+  metadata.gz: cf62f7a72d83da98d52ebb908f4d90e64b5833bf7a06e93f70fdf048c5e4acef
+  data.tar.gz: c6e7bf356eb50ca8c3b38fd25370bba07be422936e1077630be0a514aa5ba9db
 SHA512:
-  metadata.gz: 1849caab263614bdb85ed1ddcd15f63ae4199cd978ae56e2c8a690ca0e4c0e4616851f9a09f835d95e61adb457e35d5d07fbb3cb189c6394812f7064901cfaed
-  data.tar.gz: fe737b046298753a2fb9739f26307d46fbe4852a47f2a147fae56169259824dff4ced9f3f4cc3c6a37c58dbaea03785d2a1003d230d6b5c7cba56c4224ad011e
+  metadata.gz: 3b2501ee3da617e3c141f44563c6bd0313862b174a19637f2e563bdfa5419424a95cc31748c0ef0b57d01513075d468385806561ba3b8e83581d2fcbd6596f34
+  data.tar.gz: 2ab7ec6aef2c2d89b83b8b29f7de5849404af0351f5d3cbd585b2447b369da25ba08129e9c7b88f5004669c0ee17190df181c7980f4c20e0bb5c25c4dbd124ba

data/CHANGELOG.md

@@ -7,6 +7,38 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.3.2] - 2026-07-09
+
+
+### Fixed
+
+- **Unparseable files are reported once, by name — not as raw parser dumps.**
+  A file the parser gem cannot lex (e.g. a model holding a binary string
+  literal whose escapes are invalid UTF-8) used to trigger the default
+  diagnostic consumer, which dumped a three-line error block to stderr on
+  *every* analysis pass — three times per scan — labelled `(string):63` with
+  no hint of which file was affected, while the file itself was silently
+  dropped from analysis. All parsing now goes through the new
+  `EagerEye::SourceParser` (quiet diagnostics, buffers named after the real
+  file), and the analyzer emits a single clear line per file instead:
+  `EagerEye: Skipped unparseable file app/models/coupon.rb: literal contains
+  escape sequences incompatible with UTF-8`. Skipped files and their reasons
+  are also exposed programmatically via `Analyzer#skipped_files` (reset on
+  every `#run`). Unknown magic encoding comments (`# encoding: utf8` — a
+  `Parser::UnknownEncodingInMagicComment`, which is an ArgumentError rather
+  than a SyntaxError and previously crashed the whole run) are handled the
+  same way, and an unparseable `db/schema.rb` now warns explicitly that
+  schema-aware column checks are disabled instead of degrading precision in
+  silence.
+- **No more `parser/current` version-deviation warning.** EagerEye now
+  requires the grammar matching the running Ruby's *minor* version
+  (`parser/ruby33` on any 3.3.x) instead of `parser/current`, which insists
+  on an exact patch match and warned on every load ("parser/current is
+  loading parser/ruby33, which recognizes 3.3.4-compliant syntax, but you
+  are running 3.3.1"). Grammar only changes between minor versions, so
+  behaviour is identical; Rubies without a bundled grammar file fall back to
+  `parser/current` as before.
+
 ## [1.3.1] - 2026-06-12
 
 ### Changed — precision (fewer false positives)

data/lib/eager_eye/analyzer.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "parser/current"
+require_relative "source_parser"
 
 module EagerEye
   class Analyzer
@@ -32,7 +32,7 @@ module EagerEye
 
     attr_reader :paths, :issues, :association_preloads, :association_names, :method_queries, :delegation_maps,
                 :scope_maps, :uniqueness_models, :associations_by_model, :all_columns, :columns_by_model,
-                :serializer_usage
+                :serializer_usage, :skipped_files
 
     def initialize(paths: nil)
       @paths = Array(paths || EagerEye.configuration.app_path)
@@ -47,10 +47,12 @@ module EagerEye
       @all_columns = Set.new
       @columns_by_model = {}
       @serializer_usage = SerializerUsageParser.new
+      @skipped_files = {}
     end
 
     def run
       @issues = []
+      @skipped_files = {}
       collect_schema
       collect_model_metadata
       collect_serializer_usage
@@ -73,7 +75,7 @@ module EagerEye
     # this to stay silent on associations preloaded at all render sites.
     def collect_serializer_usage
       ruby_files.each do |file_path|
-        ast = parse_source(File.read(file_path))
+        ast = parse_source(File.read(file_path), file_path)
         @serializer_usage.parse_file(ast) if ast
       rescue Errno::ENOENT, Errno::EACCES
         next
@@ -82,7 +84,7 @@ module EagerEye
 
     def collect_model_metadata
       model_files.each do |file_path|
-        ast = parse_source(File.read(file_path))
+        ast = parse_source(File.read(file_path), file_path)
         next unless ast
 
         model_name = extract_model_name(file_path)
@@ -143,7 +145,7 @@ module EagerEye
 
     def analyze_file(file_path)
       source = File.read(file_path)
-      ast = parse_source(source)
+      ast = parse_source(source, file_path)
       return unless ast
 
       comment_parser = CommentParser.new(source)
@@ -160,12 +162,24 @@ module EagerEye
       warn "EagerEye: Could not read file #{file_path}: #{e.message}"
     end
 
-    def parse_source(source)
-      Parser::CurrentRuby.parse(source)
-    rescue Parser::SyntaxError
+    # A file the parser cannot handle (syntax error, binary literal whose
+    # escapes are invalid UTF-8, ...) is skipped from analysis entirely. Each
+    # such file is recorded in #skipped_files and warned about exactly once,
+    # even though every file is parsed by multiple passes.
+    def parse_source(source, file_path)
+      SourceParser.parse(source, file_path)
+    rescue Parser::SyntaxError, Parser::UnknownEncodingInMagicComment, EncodingError => e
+      register_unparseable(file_path, e.message)
       nil
     end
 
+    def register_unparseable(file_path, message)
+      return if @skipped_files.key?(file_path)
+
+      @skipped_files[file_path] = message
+      warn "EagerEye: Skipped unparseable file #{file_path}: #{message}"
+    end
+
     def detector_args(detector, ast, file_path)
       extra = DETECTOR_EXTRA_ARGS.find { |klass, _| detector.is_a?(klass) }&.last || []
       [ast, file_path, *extra.map { |name| instance_variable_get(:"@#{name}") }]

data/lib/eager_eye/detectors/base.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "parser/current"
+require_relative "../source_parser"
 
 module EagerEye
   module Detectors
@@ -43,9 +43,12 @@ module EagerEye
         node.children.each { |child| traverse_ast(child, &block) }
       end
 
-      def parse_source(source)
-        Parser::CurrentRuby.parse(source)
-      rescue Parser::SyntaxError
+      # Convenience for detector subclasses; returns nil (without any stderr
+      # output) when the source cannot be parsed. Pass file_path so any
+      # diagnostics you inspect on the raised error name the real file.
+      def parse_source(source, file_path = "(string)")
+        SourceParser.parse(source, file_path)
+      rescue Parser::SyntaxError, Parser::UnknownEncodingInMagicComment, EncodingError
         nil
       end
 

data/lib/eager_eye/schema_parser.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "parser/current"
+require_relative "source_parser"
 
 module EagerEye
   # Parses db/schema.rb to learn the real column names of each table. Columns are
@@ -31,7 +31,7 @@ module EagerEye
       schema = locate_schema(start_path)
       return false unless schema
 
-      ast = parse(File.read(schema))
+      ast = parse(File.read(schema), schema)
       return false unless ast
 
       walk(ast)
@@ -62,9 +62,12 @@ module EagerEye
       nil
     end
 
-    def parse(source)
-      Parser::CurrentRuby.parse(source)
-    rescue Parser::SyntaxError
+    # An unparseable schema silently disables the column disambiguation that
+    # filters most false positives, so the failure must not be swallowed.
+    def parse(source, file_path)
+      SourceParser.parse(source, file_path)
+    rescue Parser::SyntaxError, Parser::UnknownEncodingInMagicComment, EncodingError => e
+      warn "EagerEye: Could not parse schema #{file_path}: #{e.message} (schema-aware column checks disabled)"
       nil
     end
 

data/lib/eager_eye/serializer_usage_parser.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "parser/current"
+require_relative "source_parser"
 
 module EagerEye
   # Serializers are the worst false-positive source because the detector sees the

data/lib/eager_eye/version.rb

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

data/lib/eager_eye.rb

@@ -2,6 +2,7 @@
 
 require "set"
 require_relative "eager_eye/version"
+require_relative "eager_eye/source_parser"
 require_relative "eager_eye/configuration"
 require_relative "eager_eye/issue"
 require_relative "eager_eye/baseline"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eager_eye
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.2
 platform: ruby
 authors:
 - hamzagedikkaya
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-12 00:00:00.000000000 Z
+date: 2026-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ast