rails-ai-context 5.9.0 → 5.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f300193f1fb692df6f1404e884f9efddbafc85b2cadee89adaa995f49e69ca0
-  data.tar.gz: befb6b7b12702fe0ff805786d1e7628d1c2cca0fd1a7f85d7a655a0521bde69a
+  metadata.gz: 63fa43e2446217a8b226ba3fd59b0345c7e4fec6300acd0ab877354339187dac
+  data.tar.gz: 0d12192067351b3e28e1ae8ba16792be31a8e340e356b4b261249345a2295fa3
 SHA512:
-  metadata.gz: 95950aab08d9547799c7271d838cb145deee30a7b29930ffc8bbe2c83ff8b130a1ecb8eb53722d025c91658d7ea9365c5a5bb81aafce796aa51b28c91b0980e8
-  data.tar.gz: 43a6e75ff67650e5ab91b95cab532fc9a09fa1c60e52728825a145101fcdd54e2aabb59d250fcdb5c3eede77309b9c6030023f9ba3857a45dc9b1824031790c0
+  metadata.gz: 6d3f4378d84b890464cdda8c550195430987c05970b892184bda610fb727f7614d943b46141e4b3f7b39642503deacf71b76fca933923aada849e1c39f887fef
+  data.tar.gz: 4632cbf34995fb74922c19f89988d7bd4a5dfb3cdac98ab78e8ab3d08bac11f84c3cab0df68364b46740c133fdda22bd7311400dd5381d857b611cb4de2fe6e7

data/CHANGELOG.md

@@ -5,6 +5,26 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [5.9.1] — 2026-04-20
+
+### Fixed — `GetConcern` missed plural concern names (#78)
+
+Thanks to [@johan--](https://github.com/johan--) for the report and fix.
+
+`rails_get_concern`'s includer search built its include-pattern regex via `String#classify`, which singularizes its input. Concerns with intentionally plural module names — `WorksheetImports`, `PaperTrailEvents`, `SoftDeletables`, etc. — got demodulized to `WorksheetImport` and the `include WorksheetImports` line in the model never matched. The tool reported no includers even when the concern was in use.
+
+Switched to `String#camelize`, which normalizes case (so lowercase input like `plan_limitable` → `PlanLimitable` still works) **without** singularizing. This also restores consistency with the three other `.camelize` calls already used in `get_concern.rb` for the same "file basename / module name → class name" conversion. Covered by a new spec in `get_concern_spec.rb` that exercises the plural-name case end-to-end.
+
+### Fixed — internal invariant compliance
+
+- **`validate.rb` now routes all Prism parses through `AstCache`.** The Ruby syntax validator was calling `Prism.parse_file` directly and the ERB + semantic-visitor paths `Prism.parse` on string input, bypassing the cache entirely for the first and violating the "all Prism parses must flow through `RailsAiContext::AstCache`" invariant (`.results/3-identify-architecture.json:33`) for all three. Now uses `AstCache.parse(path)` for on-disk sources (picking up the existing size-cap + content-hash caching) and `AstCache.parse_string(source)` for synthetic strings. Note: `AstCache.parse` enforces a 5 MB `MAX_PARSE_SIZE` cap; Ruby files above that size fall through the existing `rescue` to the `ruby -c` subprocess validator, which returns errors but not Prism warnings — a graceful degradation that affects only pathologically large source files.
+- **`Listeners::BaseListener` uses `Confidence::INFERRED` constant.** Replaced three hardcoded `"[INFERRED]"` literals in `extract_first_symbol`, `extract_key`, and `extract_value` with `RailsAiContext::Confidence::INFERRED`. Value is identical; constant reference prevents drift if the marker string is ever versioned.
+- **Diagnostic `$stderr.puts` in `rescue` blocks now `ENV["DEBUG"]`-gated.** 12 previously-unconditional stderr writes across `tools/diagnose.rb` (5), `tools/review_changes.rb` (4), and `serializers/stack_overview_helper.rb` (3) were logging under normal operation whenever an optional context-enrichment step failed. These were never visible to most users but polluted stderr in MCP/CLI logs. Now silent unless `DEBUG=1`, matching the convention used everywhere else in the gem.
+
+### Added
+
+- **Prism-discipline regression spec** (`spec/lib/rails_ai_context/ast_cache_discipline_spec.rb`). Scans every `lib/**/*.rb` file (excluding `ast_cache.rb`) for direct `Prism.parse` / `Prism.parse_file` / `Prism.parse_string` calls and fails if any are found. Prevents re-introduction of the bypass that `validate.rb` had.
+
 ## [5.9.0] — 2026-04-16
 
 ### Fixed — Cursor chat agent didn't detect rules

data/Rakefile

@@ -43,4 +43,32 @@ RSpec::Core::RakeTask.new(:e2e) do |t|
   ENV["E2E"] = "1"
 end
 
+namespace :e2e do
+  desc "Run e2e specs in parallel via parallel_tests (opt-in; uses all CPUs)"
+  task :parallel do
+    # Opt-in parallelization. Each parallel_tests worker is a separate
+    # process — it will rebuild its own shared-fixture memoization and
+    # its own built .gem artefact. Still a net win because rspec-level
+    # wall-clock scales with the slowest worker, not the sum.
+    #
+    # Caveats:
+    #   - postgres_install_spec is excluded here because its DB name is
+    #     derived from E2E_DB_SUFFIX and workers would collide unless you
+    #     pre-create per-worker databases. Run `rake e2e` separately if
+    #     you need postgres coverage.
+    #   - Shared BUNDLE_PATH is NOT used in parallel mode — two workers
+    #     writing to the same bundle path can corrupt native extension
+    #     builds. Each worker gets its own implicit bundle directory.
+    require "shellwords"
+    spec_files = Dir["spec/e2e/**/*_spec.rb"].reject { |p| p.include?("postgres_install_spec") }
+    cmd = [
+      "bundle", "exec", "parallel_rspec",
+      "--serialize-stdout",
+      "--"
+    ] + spec_files
+    env = { "E2E" => "1", "BUNDLE_PATH" => nil }
+    sh(env, *cmd) { |ok, _res| abort("parallel_rspec failed") unless ok }
+  end
+end
+
 task default: :spec

data/lib/rails_ai_context/introspectors/listeners/base_listener.rb

@@ -23,7 +23,7 @@ module RailsAiContext
           case arg
           when Prism::SymbolNode then arg.value.to_sym
           when Prism::StringNode then arg.unescaped.to_sym
-          else "[INFERRED]"
+          else RailsAiContext::Confidence::INFERRED
           end
         end
 
@@ -56,7 +56,7 @@ module RailsAiContext
           case node
           when Prism::SymbolNode then node.value.to_sym
           when Prism::StringNode then node.unescaped.to_sym
-          else "[INFERRED]"
+          else RailsAiContext::Confidence::INFERRED
           end
         end
 
@@ -74,7 +74,7 @@ module RailsAiContext
           when Prism::ArrayNode          then node.elements.map { |e| extract_value(e) }
           when Prism::HashNode           then hash_node_to_hash(node)
           when Prism::KeywordHashNode    then hash_node_to_hash(node)
-          else "[INFERRED]"
+          else RailsAiContext::Confidence::INFERRED
           end
         end
 

data/lib/rails_ai_context/serializers/stack_overview_helper.rb

@@ -186,7 +186,7 @@ module RailsAiContext
           .map { |f| File.basename(f, ".rb").camelize }
           .reject { |s| s == "ApplicationService" }
       rescue => e
-        $stderr.puts "[rails-ai-context] Service file scan skipped: #{e.message}"
+        $stderr.puts "[rails-ai-context] Service file scan skipped: #{e.message}" if ENV["DEBUG"]
         []
       end
 
@@ -198,7 +198,7 @@ module RailsAiContext
           .map { |f| File.basename(f, ".rb").camelize }
           .reject { |j| j == "ApplicationJob" }
       rescue => e
-        $stderr.puts "[rails-ai-context] Job file scan skipped: #{e.message}"
+        $stderr.puts "[rails-ai-context] Job file scan skipped: #{e.message}" if ENV["DEBUG"]
         []
       end
 
@@ -208,7 +208,7 @@ module RailsAiContext
         return [] unless File.exist?(app_ctrl_file)
         File.read(app_ctrl_file).scan(/before_action\s+:([\w!?]+)/).flatten
       rescue => e
-        $stderr.puts "[rails-ai-context] Before actions scan skipped: #{e.message}"
+        $stderr.puts "[rails-ai-context] Before actions scan skipped: #{e.message}" if ENV["DEBUG"]
         []
       end
     end

data/lib/rails_ai_context/tools/diagnose.rb

@@ -265,7 +265,7 @@ module RailsAiContext
                   lines << text
                   lines << ""
                 end
-              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}"; end
+              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}" if ENV["DEBUG"]; end
             end
           end
 
@@ -284,7 +284,7 @@ module RailsAiContext
                   lines << text
                   lines << ""
                 end
-              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}"; end
+              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}" if ENV["DEBUG"]; end
             end
           end
 
@@ -298,7 +298,7 @@ module RailsAiContext
                 lines << text
                 lines << ""
               end
-            rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}"; end
+            rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}" if ENV["DEBUG"]; end
           end
 
           lines
@@ -329,7 +329,7 @@ module RailsAiContext
                          "This variable may not be set in all code paths — check if it's assigned before use, " \
                          "or use `#{receiver}&.#{method}` for safe navigation."
                 end
-              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}"; end
+              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}" if ENV["DEBUG"]; end
             end
           end
 
@@ -346,7 +346,7 @@ module RailsAiContext
                          "The record with the given ID doesn't exist or doesn't belong to the current user. " \
                          "Check if the record was deleted or if the user is authorized to access it."
                 end
-              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}"; end
+              rescue => e; $stderr.puts "[rails-ai-context] Diagnosis step skipped: #{e.message}" if ENV["DEBUG"]; end
             end
           end
 

data/lib/rails_ai_context/tools/get_concern.rb

@@ -470,9 +470,11 @@ module RailsAiContext
 
         max_size = RailsAiContext.configuration.max_file_size
         # Build pattern: match `include ConcernName` or `include ModuleName::ConcernName`
-        # Handle both simple and namespaced concern names
-        # Classify to handle lowercase input: "plan_limitable" → "PlanLimitable"
-        simple_name = concern_name.demodulize.classify
+        # Handle both simple and namespaced concern names.
+        # Use `camelize` (not `classify`) — `classify` singularizes, which drops
+        # the final `s` from plural concern names like `WorksheetImports` and
+        # then fails to match `include WorksheetImports` in the model.
+        simple_name = concern_name.demodulize.camelize
         pattern = /^\s*include\s+(?:\w+::)*#{Regexp.escape(simple_name)}\b/
 
         search_dirs.each do |dir|

data/lib/rails_ai_context/tools/review_changes.rb

@@ -186,7 +186,7 @@ module RailsAiContext
               result = GetModelDetails.call(model: model_name, detail: "standard")
               text = result.content.first[:text]
               lines << "" << "**Model context:** #{model_name}" unless text.include?("not found")
-            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}"; end
+            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}" if ENV["DEBUG"]; end
 
           when :controller
             ctrl_name = File.basename(file, ".rb").camelize
@@ -195,7 +195,7 @@ module RailsAiContext
               result = GetRoutes.call(controller: snake, detail: "summary")
               text = result.content.first[:text]
               lines << "" << "**Routes:**" << text unless text.include?("not found") || text.include?("No routes")
-            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}"; end
+            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}" if ENV["DEBUG"]; end
 
           when :migration
             # Parse migration for table/column info
@@ -211,7 +211,7 @@ module RailsAiContext
                       result = GetSchema.call(table: t, detail: "summary")
                       text = result.content.first[:text]
                       lines << "  #{t}: #{text.lines.first&.strip}" unless text.include?("not found")
-                    rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}"; end
+                    rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}" if ENV["DEBUG"]; end
                   end
                 end
               end
@@ -221,7 +221,7 @@ module RailsAiContext
             begin
               result = GetRoutes.call(detail: "summary")
               lines << "" << "**Current routes:** #{result.content.first[:text].lines.first&.strip}"
-            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}"; end
+            rescue => e; $stderr.puts "[rails-ai-context] Context lookup skipped: #{e.message}" if ENV["DEBUG"]; end
           end
 
           lines << ""

data/lib/rails_ai_context/tools/validate.rb

@@ -170,7 +170,7 @@ module RailsAiContext
       end
 
       private_class_method def self.validate_ruby_prism(full_path)
-        result = Prism.parse_file(full_path.to_s)
+        result = AstCache.parse(full_path.to_s)
         basename = File.basename(full_path.to_s)
         warnings = result.warnings.map do |w|
           "#{basename}:#{w.location.start_line}:#{w.location.start_column}: warning: #{w.message}"
@@ -213,7 +213,7 @@ module RailsAiContext
         erb_src.force_encoding("UTF-8")
         compiled = "# encoding: utf-8\ndef __erb_syntax_check\n#{erb_src}\nend"
 
-        result = Prism.parse(compiled)
+        result = AstCache.parse_string(compiled)
         if result.success?
           [ true, nil, [] ]
         else
@@ -457,7 +457,7 @@ module RailsAiContext
           content
         end
 
-        result = Prism.parse(source)
+        result = AstCache.parse_string(source)
         visitor = RailsSemanticVisitor.new
         result.value.accept(visitor)
         visitor

data/lib/rails_ai_context/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsAiContext
-  VERSION = "5.9.0"
+  VERSION = "5.9.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-ai-context
 version: !ruby/object:Gem::Version
-  version: 5.9.0
+  version: 5.9.1
 platform: ruby
 authors:
 - crisnahine
@@ -181,6 +181,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.4'
+- !ruby/object:Gem::Dependency
+  name: parallel_tests
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 description: |
   rails-ai-context turns your running Rails app into the source of truth for AI
   coding assistants. Instead of guessing from training data or stale file reads,