rails-ai-context 0.15.6 → 0.15.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0eb46f9da0fc24418021dbb2ca4a6850017ab555ac0bbe98b589777ef30593a2
-  data.tar.gz: 5b77c8e0b1c7832c17ea84fffdc96524060fc594e4691f11a84b5e010158f925
+  metadata.gz: 39df765c4841adb33e68870093b8f58d02ff28dfe5e245ee81655902a9b0fb78
+  data.tar.gz: 58385dad4ad55a627d854a2611ed16f951ac0409d54d80e96966efa8b0e1613b
 SHA512:
-  metadata.gz: 13a6f6e8e274cf47210b6cb94f160ba00e4cebe3349eb6c1e2612b4ec53561ab87d99690c9949c4eae389427bf7e80850b1c5a930283792bfa3294010303ec04
-  data.tar.gz: 5ae9e78447e0ec33496b608729be0088bf57f180a37f27690f7eaa0b6ca0a3c1a6637887a747d3de67bec808636a139be88e2b9ba72e71db0e07faa156541182
+  metadata.gz: 49755cc6de2afc6a536b470abea60b5a873a9f73419754f4ad304dd8988b34ebfa65b7d69e9878e2c7bdd3418183e9a71da16007f374161e87f44259c209f5f7
+  data.tar.gz: 83441b234d6f429d4a066ec79dece90fd5685d5ee4a3e5df882241b21b2b79cddde40d9c0c4fa3ac79322bf26764cf694a4beaf8dc9c419c82d9bc96e37009e3

data/CHANGELOG.md

@@ -5,6 +5,18 @@ 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).
 
+## [0.15.7] - 2026-03-22
+
+### Improved
+
+- **Hybrid filter extraction** — controller filters now use reflection for complete names (handles inheritance + skips), with source parsing from the inheritance chain for only/except constraints.
+- **Callback source fallback** — when reflection returns nothing (e.g. CI), falls back to parsing callback declarations from model source files.
+- **ERB validation accuracy** — in-process compilation with `<%=` → `<%` pre-processing and yield wrapper eliminates false positives from block-form helpers.
+- **Schema static parser** — now extracts `null: false`, `default:`, `array: true` from schema.rb columns, and parses `add_foreign_key` declarations.
+- **Array column display** — schema tool shows PostgreSQL array types as `string[]`, `integer[]`, etc.
+- **Concern test lookup** — `rails_get_test_info(model:"PlanLimitable")` searches concern test paths.
+- **Controller flexible matching** — underscore-based normalization handles CamelCase, snake_case, and slash notation consistently.
+
 ## [0.15.6] - 2026-03-22
 
 ### Added

data/docs/GUIDE.md

@@ -404,7 +404,7 @@ Returns test infrastructure details. Optionally filter by model or controller to
 
 | Param | Type | Description |
 |-------|------|-------------|
-| `model` | string | Show tests for a specific model (e.g. `User`). |
+| `model` | string | Show tests for a specific model (e.g. `User`). Also searches concern test paths (`spec/models/concerns/`, `test/models/concerns/`). |
 | `controller` | string | Show tests for a specific controller (e.g. `Cooks`). |
 | `detail` | string | `summary` / `standard` (default) / `full`. |
 
@@ -782,6 +782,12 @@ RailsAiContext.configure do |config|
   # schema.rb / structure.sql parse limit (default: 10MB)
   # config.max_schema_file_size = 10_000_000
 
+  # Total aggregated view content for UI patterns (default: 5MB)
+  # config.max_view_total_size = 5_000_000
+
+  # Per-view file during aggregation (default: 500KB)
+  # config.max_view_file_size = 500_000
+
   # Max search results per call (default: 100)
   # config.max_search_results = 100
 
@@ -888,7 +894,7 @@ These run by default. Fast and cover core Rails structure.
 | `controllers` | Actions, filters (before/after/around with only/except), strong params methods, parent class, API controller detection, concerns. |
 | `tests` | Test framework (rspec/minitest), factories/fixtures with locations and counts, system tests, CI config files, coverage tool, test helpers, VCR cassettes. |
 | `migrations` | Total count, schema version, pending migrations, recent migration history with detected actions (create_table, add_column, etc.), migration statistics. |
-| `config` | Cache store, session store, timezone, middleware stack, initializers, credentials keys, CurrentAttributes classes. |
+| `config` | Cache store, session store, timezone, queue adapter, mailer settings, middleware stack, initializers, credentials status, CurrentAttributes classes. |
 | `stimulus` | Stimulus controllers with targets, values (with types), actions, outlets, classes. Extracted from JS/TS files. |
 | `view_templates` | View file contents, partial references, Stimulus data attributes, UI pattern extraction, model field usage in partials. |
 | `design_tokens` | Auto-detects CSS framework (Tailwind v3/v4, Bootstrap, Sass, plain CSS) and extracts design tokens from config files and built CSS. |

data/lib/rails_ai_context/introspectors/config_introspector.rb

@@ -3,7 +3,7 @@
 module RailsAiContext
   module Introspectors
     # Extracts application configuration: cache store, session store,
-    # timezone, middleware stack, initializers, credentials keys.
+    # timezone, middleware stack, initializers, credentials status.
     class ConfigIntrospector
       attr_reader :app
 

data/lib/rails_ai_context/introspectors/controller_introspector.rb

@@ -150,30 +150,63 @@ module RailsAiContext
         actions.sort
       end
 
-      # Prefer source-based parsing for filters — always reflects current file state.
-      # Falls back to reflection for controllers without readable source files.
+      # Hybrid approach: reflection for complete filter names (handles inheritance + skips),
+      # source parsing from inheritance chain for only/except constraints.
       def extract_filters(ctrl, source = nil)
+        if ctrl.respond_to?(:_process_action_callbacks)
+          reflection_filters = ctrl._process_action_callbacks.filter_map do |cb|
+            next if cb.filter.is_a?(Proc) || cb.filter.to_s.start_with?("_")
+            next if excluded_filters.include?(cb.filter.to_s)
+            { name: cb.filter.to_s, kind: cb.kind.to_s }
+          end
+
+          if reflection_filters.any?
+            # Collect only/except constraints from source files in the inheritance chain
+            source_constraints = collect_source_constraints(ctrl, source)
+            reflection_filters.each do |f|
+              if (sc = source_constraints[f[:name]])
+                f[:only] = sc[:only] if sc[:only]&.any?
+                f[:except] = sc[:except] if sc[:except]&.any?
+              end
+            end
+            return reflection_filters
+          end
+        end
+
+        # Fallback to source parsing when reflection is unavailable
         if source
           filters = extract_filters_from_source(source)
           return filters if filters.any?
         end
-        return [] unless ctrl.respond_to?(:_process_action_callbacks)
-
-        ctrl._process_action_callbacks.filter_map do |cb|
-          next if cb.filter.is_a?(Proc) || cb.filter.to_s.start_with?("_")
-          next if excluded_filters.include?(cb.filter.to_s)
-
-          filter = { name: cb.filter.to_s, kind: cb.kind.to_s }
-          filter[:only] = cb.instance_variable_get(:@if)&.filter_map { |c| extract_action_condition(c) }&.flatten
-          filter[:except] = cb.instance_variable_get(:@unless)&.filter_map { |c| extract_action_condition(c) }&.flatten
-          filter.delete(:only) if filter[:only]&.empty?
-          filter.delete(:except) if filter[:except]&.empty?
-          filter
-        end
+
+        []
       rescue
         []
       end
 
+      # Walk up the controller inheritance chain and collect filter constraints from source files
+      def collect_source_constraints(ctrl, current_source = nil)
+        constraints = {}
+        klass = ctrl
+        while klass && klass.name
+          break if klass.name.start_with?("ActionController::", "AbstractController::")
+          break if klass == ActionController::Base
+          break if defined?(ActionController::API) && klass == ActionController::API
+
+          src = (klass == ctrl) ? (current_source || read_source(klass)) : read_source(klass)
+          if src
+            extract_filters_from_source(src).each do |sf|
+              # First definition wins (most specific controller in chain)
+              constraints[sf[:name]] ||= sf
+            end
+          end
+          klass = klass.superclass
+        end
+        constraints
+      rescue
+        {}
+      end
+
       def extract_filters_from_source(source)
         filters = []
         source.each_line do |line|

data/lib/rails_ai_context/introspectors/model_introspector.rb

@@ -145,7 +145,7 @@ module RailsAiContext
           after_commit after_rollback
         ]
 
-        callback_types.each_with_object({}) do |type, hash|
+        result = callback_types.each_with_object({}) do |type, hash|
           callbacks = model.send(:"_#{type}_callbacks").reject do |cb|
             cb.filter.to_s.start_with?(*EXCLUDED_CALLBACKS) || cb.filter.is_a?(Proc)
           end
@@ -154,6 +154,29 @@ module RailsAiContext
 
           hash[type.to_s] = callbacks.map { |cb| cb.filter.to_s }
         end
+
+        # If reflection returned nothing, fall back to source parsing
+        return result if result.any?
+        extract_callbacks_from_source(model)
+      rescue
+        extract_callbacks_from_source(model)
+      end
+
+      # Parse callback declarations from model source file
+      def extract_callbacks_from_source(model)
+        source_path = model_source_path(model)
+        return {} unless source_path && File.exist?(source_path)
+
+        source = File.read(source_path)
+        callbacks = {}
+        source.each_line do |line|
+          if (match = line.match(/\A\s*(before_validation|after_validation|before_save|after_save|before_create|after_create|before_update|after_update|before_destroy|after_destroy|after_commit|after_rollback)\s+:(\w+)/))
+            type = match[1]
+            method_name = match[2]
+            (callbacks[type] ||= []) << method_name
+          end
+        end
+        callbacks
       rescue
         {}
       end
@@ -168,13 +191,18 @@ module RailsAiContext
 
       def framework_concern?(name)
         return true if name.nil?
-        return true if %w[Kernel JSON PP Marshal MessagePack].include?(name)
+        return true if %w[Kernel JSON PP Marshal MessagePack].any? { |prefix| name == prefix || name.start_with?("#{prefix}::") }
+        return true if name.start_with?("ActiveModel::", "ActiveRecord::", "ActiveSupport::")
         RailsAiContext.configuration.excluded_concerns.any? { |pattern| name.match?(pattern) }
       end
 
       def extract_public_class_methods(model)
+        scope_names = extract_scopes(model).map(&:to_s)
         (model.methods - ActiveRecord::Base.methods - Object.methods)
-          .reject { |m| m.to_s.start_with?("_", "autosave") }
+          .reject { |m|
+            ms = m.to_s
+            ms.start_with?("_", "autosave") || scope_names.include?(ms)
+          }
           .sort
           .first(30) # Cap to avoid noise
           .map(&:to_s)

data/lib/rails_ai_context/introspectors/schema_introspector.rb

@@ -179,7 +179,14 @@ module RailsAiContext
             next if current_table.start_with?("ar_internal_metadata", "schema_migrations")
             tables[current_table] = { columns: [], indexes: [], foreign_keys: [] }
           elsif current_table && (match = line.match(/t\.(\w+)\s+"(\w+)"/))
-            tables[current_table][:columns] << { name: match[2], type: match[1] }
+            col = { name: match[2], type: match[1] }
+            col[:null] = false if line.include?("null: false")
+            if (default_match = line.match(/default:\s*("[^"]*"|\{[^}]*\}|\[[^\]]*\]|-?\d+(?:\.\d+)?|true|false)/))
+              raw = default_match[1]
+              col[:default] = raw.start_with?('"') ? raw[1..-2] : raw
+            end
+            col[:array] = true if line.include?("array: true")
+            tables[current_table][:columns] << col
           elsif current_table && (match = line.match(/t\.index\s+\[([^\]]*)\]/))
             cols = match[1].scan(/["'](\w+)["']/).flatten
             unique = line.include?("unique: true")
@@ -199,6 +206,17 @@ module RailsAiContext
             unique = rest.include?("unique: true")
             idx_name = rest.match(/name:\s*"(\w+)"/)&.send(:[], 1)
             tables[table_name]&.dig(:indexes)&.push({ name: idx_name, columns: cols, unique: unique }.compact) if cols.any?
+          elsif (match = line.match(/add_foreign_key\s+"(\w+)",\s+"(\w+)"/))
+            from_table = match[1]
+            to_table = match[2]
+            column_match = line.match(/column:\s*"(\w+)"/)
+            column = column_match ? column_match[1] : "#{to_table.singularize}_id"
+            pk_match = line.match(/primary_key:\s*"(\w+)"/)
+            primary_key = pk_match ? pk_match[1] : "id"
+            tables[from_table]&.dig(:foreign_keys)&.push({
+              from_table: from_table, to_table: to_table,
+              column: column, primary_key: primary_key
+            })
           end
         end
 

data/lib/rails_ai_context/introspectors/test_introspector.rb

@@ -155,7 +155,7 @@ module RailsAiContext
 
       def detect_test_files
         categories = {}
-        %w[models controllers requests system services integration features].each do |cat|
+        %w[models models/concerns controllers requests system services integration features].each do |cat|
           %w[spec test].each do |base|
             dir = File.join(root, base, cat)
             next unless Dir.exist?(dir)

data/lib/rails_ai_context/tools/get_controllers.rb

@@ -49,12 +49,14 @@ module RailsAiContext
         # Flexible matching: "cooks", "CooksController", "cookscontroller" all work
         if controller
           # Accept multiple formats: "CooksController", "cooks", "bonus/crises", "Bonus::CrisesController"
-          normalized = controller.downcase.delete_suffix("controller").tr("/", "::")
+          # Use underscore for CamelCase→snake_case: "OmniauthCallbacks" → "omniauth_callbacks"
+          # Also match on plain downcase to handle "userscontroller" → "users"
+          input_snake = controller.gsub("/", "::").underscore.delete_suffix("_controller")
+          input_down = controller.downcase.delete_suffix("controller").tr("/", "::")
           key = controllers.keys.find { |k|
-            kd = k.downcase
-            kd == controller.downcase ||
-              kd.delete_suffix("controller") == normalized ||
-              kd.delete_suffix("controller").tr("::", "/") == controller.downcase.delete_suffix("controller")
+            key_snake = k.underscore.delete_suffix("_controller")
+            key_down = k.downcase.delete_suffix("controller")
+            key_snake == input_snake || key_down == input_down
           } || controller
           info = controllers[key]
           unless info

data/lib/rails_ai_context/tools/get_model_details.rb

@@ -129,12 +129,17 @@ module RailsAiContext
           end
         end
 
-        # Validations — compress repeated inclusion lists
+        # Validations — compress repeated inclusion lists, deduplicate same kind+attribute
         if data[:validations]&.any?
           lines << "" << "## Validations"
+          # Deduplicate validations with same kind and attributes (e.g. implicit belongs_to + explicit validates :user, presence)
+          seen_validations = Set.new
           # Track seen inclusion arrays to avoid repeating long lists
           seen_inclusions = {}
           data[:validations].each do |v|
+            dedup_key = "#{v[:kind]}:#{v[:attributes].sort.join(',')}"
+            next if seen_validations.include?(dedup_key)
+            seen_validations << dedup_key
             attrs = v[:attributes].join(", ")
             if v[:options]&.any?
               compressed_opts = v[:options].map do |k, val|

data/lib/rails_ai_context/tools/get_schema.rb

@@ -136,7 +136,8 @@ module RailsAiContext
 
         columns.each do |col|
           nullable = col.key?(:null) ? (col[:null] ? "yes" : "**NO**") : "yes"
-          line = "| #{col[:name]} | #{col[:type]} | #{nullable}"
+          col_type = col[:array] ? "#{col[:type]}[]" : col[:type].to_s
+          line = "| #{col[:name]} | #{col_type} | #{nullable}"
           line += " | #{col[:default]}" if has_defaults
           lines << "#{line} |"
         end

data/lib/rails_ai_context/tools/get_test_info.rb

@@ -130,7 +130,9 @@ module RailsAiContext
         when :model
           [
             "spec/models/#{snake}_spec.rb",
-            "test/models/#{snake}_test.rb"
+            "test/models/#{snake}_test.rb",
+            "spec/models/concerns/#{snake}_spec.rb",
+            "test/models/concerns/#{snake}_test.rb"
           ]
         when :controller
           [

data/lib/rails_ai_context/tools/validate.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "open3"
+require "erb"
 
 module RailsAiContext
   module Tools
@@ -106,28 +107,44 @@ module RailsAiContext
       end
 
       # Validate ERB by compiling to Ruby source then syntax-checking the result.
-      # ERB.new(...).src only validates ERB tag syntax — it does NOT catch missing <% end %>.
-      # So we compile to Ruby, then run ruby -c on the compiled output to catch block mismatches.
+      # Catches missing <% end %>, unclosed blocks, and mismatched do/end.
+      #
+      # Two key pre-processing steps avoid false positives:
+      # 1. Convert `<%= ... %>` to `<% ... %>` — prevents the `_buf << ( helper do ).to_s`
+      #    ambiguity that standard ERB compilation creates for block-form helpers
+      #    like `<%= link_to ... do %>`, `<%= form_with ... do |f| %>`, etc.
+      # 2. Wrap compiled source in a method def — makes `yield` syntactically valid.
       private_class_method def self.validate_erb(full_path)
-        # Step 1: Compile ERB to Ruby source
-        compile_script = "require 'erb'; print ERB.new(File.read(ARGV[0])).src"
-        compiled, compile_status = Open3.capture2e("ruby", "-e", compile_script, full_path.to_s)
+        return [ false, "file too large" ] if File.size(full_path) > RailsAiContext.configuration.max_file_size
 
-        unless compile_status.success?
-          error = compiled.lines.reject { |l| l.strip.empty? }.first&.strip || "ERB syntax error"
-          return [ false, error ]
-        end
+        content = File.binread(full_path).force_encoding("UTF-8")
+
+        # Pre-process: convert output tags to non-output for syntax-only checking.
+        # This is safe because we only check structure (do/end, if/end matching),
+        # not whether output is correct.
+        processed = content.gsub("<%=", "<%")
+
+        # Compile ERB to Ruby, wrapped in a method so `yield` is valid syntax.
+        # Force UTF-8 on .src output — ERB may return ASCII-8BIT which breaks
+        # concatenation with UTF-8 strings when non-ASCII bytes (emoji, etc.) are present.
+        erb_src = +ERB.new(processed).src
+        erb_src.force_encoding("UTF-8")
+        compiled = "# encoding: utf-8\ndef __erb_syntax_check\n#{erb_src}\nend"
 
-        # Step 2: Syntax-check the compiled Ruby to catch missing end, unclosed blocks, etc.
         check_result, check_status = Open3.capture2e("ruby", "-c", "-", stdin_data: compiled)
         if check_status.success?
           [ true, nil ]
         else
-          error = check_result.lines.reject { |l| l.strip.empty? || l.include?("Syntax OK") }.first&.strip || "ERB error"
-          # Make the error more helpful — it's from compiled source, translate back
-          error = error.sub(/^-:/, "compiled ERB line ")
-          [ false, error ]
+          # Adjust line numbers: subtract 1 for the wrapper def line
+          error = check_result.lines
+            .reject { |l| l.strip.empty? || l.include?("Syntax OK") }
+            .first(5)
+            .map { |l| l.strip.sub(/-:(\d+):/) { "ruby: -:#{$1.to_i - 1}:" } }
+          msg = error.any? ? error.join("\n") : "ERB syntax error"
+          [ false, msg ]
         end
+      rescue => e
+        [ false, "ERB check error: #{e.message}" ]
       end
 
       # Validate JavaScript syntax via `node -c` (no shell — uses Open3 array form)

data/lib/rails_ai_context/version.rb

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

data/server.json

@@ -7,11 +7,11 @@
     "url": "https://github.com/crisnahine/rails-ai-context",
     "source": "github"
   },
-  "version": "0.15.5",
+  "version": "0.15.6",
   "packages": [
     {
       "registryType": "mcpb",
-      "identifier": "https://github.com/crisnahine/rails-ai-context/releases/download/v0.15.5/rails-ai-context-mcp.mcpb",
+      "identifier": "https://github.com/crisnahine/rails-ai-context/releases/download/v0.15.6/rails-ai-context-mcp.mcpb",
       "fileSha256": "dd711a0ad6c4de943ae4da94eaf59a6dc9494b9d57f726e24649ed4e2f156990",
       "transport": {
         "type": "stdio"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-ai-context
 version: !ruby/object:Gem::Version
-  version: 0.15.6
+  version: 0.15.7
 platform: ruby
 authors:
 - crisnahine