rails-ai-context 4.3.0 → 4.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7cd3960b55660ccf3b1c8da3c6af0aeef18ba57a9a906f02cd136b976bf66976
-  data.tar.gz: '0185228442a21266ba75f01375d8883dc176b6b140b764e59aafa8044fa7faed'
+  metadata.gz: 7ba5dfa48fbbc7efaa498640785dd290523f434ed1f7244849f31046f8359e62
+  data.tar.gz: 2dca436c86b470b51b38e745a47c64ee3908b7e342677c842940b9515dcf57f2
 SHA512:
-  metadata.gz: 117c13423c12bd5e4a204290716639c5f30fcd960290c5cd3535c5b37911567f7678341bb15f5dcff88e75f6c321a740fe8c4d7adb0a5d5a5936aa2c8b679bc0
-  data.tar.gz: 101b71249f3e49a40ec0bd8417a71bd76ea7b1907b36d94abf4182ead812fd76f33e9b353c9b14f1e78de680f6a1eeebf0659097cc3ec7afe4f06e7b48bf2017
+  metadata.gz: 64c07284bc9d61136ff8dee427396fd0237fc628e9ca8cd08524022a7d9031521360b49526a93205cb6c3990284e9096aedead5d88d2fc0d75ad2f58475d30c6
+  data.tar.gz: d82fbfea6afbe2e7c44f22afa1b8f4201fbb6cf9fda17362675f46000e911bece69a3384133b19bc30d5ffe3c553e4cbb935cbd6031df92f21c6dfc3eb32aa78

data/CHANGELOG.md

@@ -5,21 +5,59 @@ 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).
 
+## [4.3.1] — 2026-04-02
+
+### Fixed
+- **performance_check false positives** — now parses `t.index` inside `create_table` blocks (was only parsing `add_index` outside blocks, missing inline indexes)
+- **review_changes overflow** — capped at 20 files with 30 diff lines each; remaining files listed without diff to prevent 200K+ char responses
+- **get_context ivar cross-check** — now follows `render :other_template` references (create rendering :new on failure no longer shows false positives)
+- **generate_test setup block** — always generates `setup do` with factory/fixture/inline fallback; minitest tests no longer reference undefined instance variables
+- **session_context auto-tracking** — `text_response()` now auto-records every tool call; `session_context(action:"status")` shows what was queried without manual `mark:` calls
+- **search_code AI file exclusion** — excludes CLAUDE.md, AGENTS.md, .claude/, .cursor/, .cursorrules, .github/copilot-instructions.md, .ai-context.json from results
+- **diagnose output truncation** — per-section size limits (3K chars each) + total output cap (20K) prevent overflow
+- **diagnose NameError classification** — `NameError: uninitialized constant` now correctly classified as `:name_error`, not `:nil_reference`
+- **diagnose specific inference** — identifies nil receivers, missing `authenticate_user!`, and `set_*` before_actions from code context
+- **onboard purpose inference** — quick mode now infers app purpose from models, jobs, services, gems (e.g., "news aggregation app with RSS, YouTube, Reddit ingestion")
+- **onboard adapter resolution** — resolves `static_parse` adapter name from config or gems instead of showing internal implementation detail
+- **security_scan transparency** — "no warnings" response now lists which check categories were run (e.g., "SQL injection, XSS, mass assignment")
+- **read_logs filename filter** — `available_log_files` now rejects filenames with non-standard characters
+- **Phlex view support** — get_view detects Phlex views (.rb), extracts component renders and helper calls
+- **Component introspector Phlex** — discovers Phlex components alongside ViewComponent
+- **Schema introspector array columns** — detects PostgreSQL `array: true` columns from schema.rb
+- **search_code regex injection** — `definition` and `class` match types now escape user input with `Regexp.escape` (previously raw interpolation could crash with metacharacters like `(`, `[`, `{`)
+- **sensitive file bypass on macOS** — all 3 `sensitive_file?` implementations now use `FNM_CASEFOLD` flag; `.ENV`, `Master.Key`, `.PEM` variants no longer bypass the block on case-insensitive filesystems
+- **doctor silent exception swallowing** — `rescue nil` replaced with `rescue StandardError` + stderr logging; broken health checks are now reported instead of silently skipped
+- **context file race condition** — `write_plain` and `write_with_markers` now use atomic write (temp file + rename) to prevent partial writes from concurrent generators
+- **performance_introspector O(n*m) scan** — `detect_model_all_in_controllers` now builds a single combined regex instead of scanning each controller once per model
+- **HTTP transport non-loopback warning** — MCP server now logs a warning when `http_bind` is set to a non-loopback address (no authentication on the HTTP transport)
+
+### Added
+- **`rails_runtime_info`** — live runtime state: DB connection pool, table sizes (PG/MySQL/SQLite), pending migrations, cache stats (Redis hit rate + memory), Sidekiq queue depth, job adapter detection
+- **`rails_session_context`** — session-aware context tracking with auto-recording; `action:"status"` shows what tools were called, `action:"summary"` for compressed recap, `action:"reset"` to clear
+- **`auto_compress` helper** — BaseTool method that auto-downgrades detail when response approaches 85% of max chars
+- **`not_found_response` dedup** — no longer suggests the exact same string the user typed
+- **get_frontend_stack Hotwire** — reports Stimulus controllers, Turbo config, importmap pins for Hotwire/importmap apps (not just React/Vue)
+- **get_component_catalog guidance** — returns actionable message for partial-based apps: "Use get_partial_interface or get_view"
+- **get_context feature enrichment** — `feature:` mode now also searches controllers and services by name when analyze_feature misses them
+- **Fingerprinter gem development** — includes gem lib/ directory mtime when using path gem (local dev cache invalidation)
+
+### Changed
+- Tool count: 37 → 39
+- Test count: 1052 → 1170
+- Standard preset now includes turbo, auth, accessibility, performance, i18n (was 14 introspectors, now 19)
+
 ## [4.3.0] — 2026-04-01
 
 ### Added
-- **`rails_onboard`** — narrative app walkthrough (quick/standard/full) for new developers and AI agents: stack, data model, authentication, key flows, jobs, frontend, testing, getting started
-- **`rails_generate_test`** — generates test scaffolding matching project patterns: detects framework (RSpec/Minitest), factories vs fixtures, assertion style, Devise helpers; covers validations, associations, scopes, enums, request specs with routes
-- **`rails_diagnose`** — one-call error diagnosis: parses error → classifies (nil_reference, record_not_found, validation_failure, routing, strong_params, schema_mismatch) → gathers controller/model/schema context → shows recent git changes → pulls error logs → suggests fix
-- **`rails_review_changes`** — PR/commit review context: classifies changed files, pulls per-file context (model → schema, controller → routes, migration → affected models), detects warnings (missing indexes, removed validations, no test changes)
-- **Improved AI instructions** — workflow sequencing (step-by-step tool call order), `detail` parameter guidance, anti-patterns section, `get_context` promoted as power tool, app-specific footer rules from conventions introspector
-- **Compact tool name list** — root files (CLAUDE.md, AGENTS.md) now include all 37 tool names in a dense format that fits within the 150-line compact mode limit
+- **`rails_onboard`** — narrative app walkthrough (quick/standard/full)
+- **`rails_generate_test`** — test scaffolding matching project patterns
+- **`rails_diagnose`** — one-call error diagnosis with classification + context + git + logs
+- **`rails_review_changes`** — PR/commit review with per-file context + warnings
+- **Improved AI instructions** — workflow sequencing, detail guidance, anti-patterns, get_context as power tool
 
 ### Changed
 - Tool count: 33 → 37
 - Test count: 1016 → 1052
-- Root file render order: commands and rules before tool guide to prevent truncation
-- Cursor rules description: fixed stale "25 tools" → "37 tools"
 
 ## [4.2.3] — 2026-04-01
 

data/CLAUDE.md

@@ -9,7 +9,7 @@ structure to AI assistants via the Model Context Protocol (MCP).
 - `lib/rails_ai_context/configuration.rb` — User-facing config with presets (:standard, :full)
 - `lib/rails_ai_context/introspector.rb` — Orchestrates sub-introspectors
 - `lib/rails_ai_context/introspectors/` — 33 introspectors (schema, models, routes, jobs, gems, conventions, stimulus, database_stats, controllers, views, view_templates, design_tokens, turbo, i18n, config, active_storage, action_text, auth, api, tests, rake_tasks, assets, devops, action_mailbox, migrations, seeds, middleware, engines, multi_database, components, accessibility, performance, frontend_frameworks)
-- `lib/rails_ai_context/tools/` — 37 MCP tools using the official mcp SDK
+- `lib/rails_ai_context/tools/` — 39 MCP tools using the official mcp SDK
 - `lib/rails_ai_context/cli/` — CLI tool runner (`tool_runner.rb`) — executes MCP tools from rake/Thor
 - `lib/rails_ai_context/serializers/` — Output formatters (claude, claude_rules, opencode, opencode_rules, cursor_rules, copilot, copilot_instructions, rules, markdown, JSON, context_file_serializer, test_command_detection, tool_guide_helper, design_system_helper, stack_overview_helper)
 - `lib/rails_ai_context/resources.rb` — MCP resources (static data AI clients read directly)
@@ -40,12 +40,12 @@ structure to AI assistants via the Model Context Protocol (MCP).
 13. **Per-tool split rules** — `.claude/rules/`, `.cursor/rules/`, `.github/instructions/`
 14. **Section markers** — root file content wrapped in `<!-- BEGIN/END rails-ai-context -->` to preserve user content
 15. **generate_root_files toggle** — when false, skip root files (CLAUDE.md, etc.), only generate split rules
-16. **custom_tools API** — `config.custom_tools` array lets users register additional MCP::Tool subclasses alongside the 37 built-in tools
+16. **custom_tools API** — `config.custom_tools` array lets users register additional MCP::Tool subclasses alongside the 39 built-in tools
 17. **Design system extraction** — view templates analyzed for canonical examples, color palette, typography, responsive patterns, interactive states, dark mode
 18. **skip_tools API** — `config.skip_tools` array lets users exclude specific built-in tools (e.g. `%w[rails_security_scan]`)
 19. **Security scanning** — optional Brakeman integration via `rails_security_scan` tool (graceful degradation if not installed)
 20. **tool_mode config** — `:mcp` (default, MCP primary + CLI fallback) or `:cli` (CLI only, no MCP server needed). Selected during install.
-21. **CLI tool access** — all 37 MCP tools callable from terminal: `rails ai:tool[schema]`, `rails-ai-context tool schema`. Tool name resolution: `schema` → `get_schema` → `rails_get_schema`.
+21. **CLI tool access** — all 39 MCP tools callable from terminal: `rails ai:tool[schema]`, `rails-ai-context tool schema`. Tool name resolution: `schema` → `get_schema` → `rails_get_schema`.
 22. **Shared ToolGuideHelper** — serializers use a shared module for tool reference sections, rendering MCP or CLI syntax based on `tool_mode`
 23. **Component catalog** — ViewComponent/Phlex introspection: props, slots, previews, sidecar assets, usage examples via `rails_get_component_catalog`
 24. **Accessibility scanning** — ARIA attributes, semantic HTML, screen reader text, alt text, landmark roles, accessibility score via AccessibilityIntrospector
@@ -60,7 +60,7 @@ structure to AI assistants via the Model Context Protocol (MCP).
 ## Testing
 
 ```bash
-bundle exec rspec           # Run specs (1052 examples)
+bundle exec rspec           # Run specs (1170 examples)
 bundle exec rubocop         # Lint
 ```
 

data/CONTRIBUTING.md

@@ -20,7 +20,7 @@ The test suite uses [Combustion](https://github.com/pat/combustion) to boot a mi
 lib/rails_ai_context/
 ├── cli/               # CLI tool runner (tool_runner.rb) — executes MCP tools from rake/Thor
 ├── introspectors/     # 33 introspectors (schema, models, routes, etc.)
-├── tools/             # 37 MCP tools with detail levels and pagination
+├── tools/             # 39 MCP tools with detail levels and pagination
 ├── serializers/       # Per-assistant formatters + shared ToolGuideHelper
 ├── server.rb          # MCP server setup (stdio + HTTP)
 ├── live_reload.rb     # MCP live reload (file watcher + cache invalidation)

data/README.md

@@ -8,19 +8,19 @@
 [![MCP Registry](https://img.shields.io/badge/MCP_Registry-listed-green)](https://registry.modelcontextprotocol.io)
 [![Ruby](https://img.shields.io/badge/Ruby-3.2%20%7C%203.3%20%7C%203.4-red)](https://github.com/crisnahine/rails-ai-context)
 [![Rails](https://img.shields.io/badge/Rails-7.1%20%7C%207.2%20%7C%208.0-red)](https://github.com/crisnahine/rails-ai-context)
-[![Tests](https://img.shields.io/badge/Tests-1052%20passing-brightgreen)](https://github.com/crisnahine/rails-ai-context/actions)
+[![Tests](https://img.shields.io/badge/Tests-1170%20passing-brightgreen)](https://github.com/crisnahine/rails-ai-context/actions)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
 **Works with:** Claude Code • Cursor • GitHub Copilot • OpenCode • Any terminal
 
-> Built by a Rails developer with 10+ years of production experience. AI assisted — the same way it assists me shipping features at work. I designed the architecture, made every decision, reviewed every line, and wrote 1052 tests. This gem exists because I understand Rails deeply enough to know exactly what AI agents get wrong and what context they need to get it right.
+> Built by a Rails developer with 10+ years of production experience. AI assisted — the same way it assists me shipping features at work. I designed the architecture, made every decision, reviewed every line, and wrote 1170 tests. This gem exists because I understand Rails deeply enough to know exactly what AI agents get wrong and what context they need to get it right.
 
 ```bash
 gem "rails-ai-context", group: :development
 rails generate rails_ai_context:install
 ```
 
-That's it. Your AI now has 37 tools that understand your entire Rails app — via MCP server or CLI. Zero config.
+That's it. Your AI now has 39 tools that understand your entire Rails app — via MCP server or CLI. Zero config.
 
 > **[Full Guide →](docs/GUIDE.md)** — every command, every parameter, every configuration option.
 
@@ -50,7 +50,7 @@ rails 'ai:tool[schema]' table=users
 rails 'ai:tool[analyze_feature]' feature=billing
 ```
 
-Same 37 tools. Same output. AI agents run these as shell commands. **Works in any terminal, any AI tool, any workflow.** No MCP client required.
+Same 39 tools. Same output. AI agents run these as shell commands. **Works in any terminal, any AI tool, any workflow.** No MCP client required.
 
 ---
 
@@ -174,7 +174,7 @@ Tested on a real Rails 8 app (5 models, 19 controllers, 95 routes):
 
 ---
 
-## 37 Tools
+## 39 Tools
 
 Every tool is **read-only** and returns structured, token-efficient data.
 
@@ -246,7 +246,7 @@ Every tool is **read-only** and returns structured, token-efficient data.
          ▼                  ▼              ▼
 ┌──────────────────┐ ┌────────────┐ ┌────────────────────┐
 │  Static Files     │ │  MCP Server │ │  CLI Tools          │
-│  CLAUDE.md        │ │  37 tools   │ │  Same 37 tools      │
+│  CLAUDE.md        │ │  39 tools   │ │  Same 39 tools      │
 │  .cursor/rules/   │ │  stdio/HTTP │ │  No server needed   │
 │  .github/instr... │ │  .mcp.json  │ │  rails 'ai:tool[X]' │
 └──────────────────┘ └────────────┘ └────────────────────┘
@@ -282,7 +282,7 @@ MCP auto-discovery: `.mcp.json` is detected automatically by Claude Code and Cur
 | Command | What it does |
 |---------|-------------|
 | `rails ai:context` | Generate context files for your AI tools |
-| `rails 'ai:tool[NAME]'` | Run any of the 37 tools from the CLI |
+| `rails 'ai:tool[NAME]'` | Run any of the 39 tools from the CLI |
 | `rails ai:tool` | List all available tools with short names |
 | `rails ai:serve` | Start MCP server (stdio) |
 | `rails ai:doctor` | Diagnostics + AI readiness score |

data/SECURITY.md

@@ -4,6 +4,7 @@
 
 | Version | Supported          |
 |---------|--------------------|
+| 4.3.x   | :white_check_mark: |
 | 4.2.x   | :white_check_mark: (4.2.1 includes security hardening) |
 | 4.1.x   | :white_check_mark: |
 | 4.0.x   | :white_check_mark: |
@@ -26,7 +27,7 @@ If you discover a security vulnerability in rails-ai-context, please report it r
 
 ## Security Design
 
-- All 37 MCP tools are **read-only** and never modify your application or database.
+- All 39 MCP tools are **read-only** and never modify your application or database.
 - **Sensitive file blocking** — configurable `sensitive_patterns` blocks access to `.env`, `*.key`, `*.pem`, `credentials.yml.enc` across all search and read tools. Patterns are checked in `rails_search_code`, `rails_get_edit_context`, and all new tools.
 - **Path traversal protection** — all file-reading tools validate paths with `File.realpath()` against `Rails.root` to prevent directory escape.
 - **Command injection prevention** — code search uses `Open3.capture2` with array arguments (never shell strings). The `--` flag separator prevents pattern injection.

data/docs/GUIDE.md

@@ -252,7 +252,7 @@ rails ai:context:claude           # Use this instead (no quoting needed)
 
 ## CLI Tools
 
-All 37 MCP tools can be run directly from the terminal — no MCP server or AI client needed.
+All 39 MCP tools can be run directly from the terminal — no MCP server or AI client needed.
 
 ### Rake
 
@@ -316,7 +316,7 @@ The `tool_mode` is selected during `rails generate rails_ai_context:install`.
 
 ## MCP Tools — Full Reference
 
-All 37 tools are **read-only** and **idempotent** — they never modify your application or database.
+All 39 tools are **read-only** and **idempotent** — they never modify your application or database.
 
 ### rails_get_schema
 
@@ -1117,7 +1117,7 @@ RailsAiContext.configure do |config|
 end
 ```
 
-Both transports are **read-only** — they expose the same 37 tools and never modify your app.
+Both transports are **read-only** — they expose the same 39 tools and never modify your app.
 
 ---
 

data/lib/generators/rails_ai_context/install/install_generator.rb

@@ -436,9 +436,9 @@ module RailsAiContext
         say ""
         say "Commands:", :yellow
         say "  rails ai:context         # Regenerate context files"
-        say "  rails 'ai:tool[schema]'    # Run any of the 37 tools from CLI"
+        say "  rails 'ai:tool[schema]'    # Run any of the 39 tools from CLI"
         if @tool_mode == :mcp
-          say "  rails ai:serve           # Start MCP server (37 live tools)"
+          say "  rails ai:serve           # Start MCP server (39 live tools)"
         end
         say "  rails ai:doctor          # Check AI readiness"
         say "  rails ai:inspect         # Print introspection summary"

data/lib/rails_ai_context/configuration.rb

@@ -3,7 +3,9 @@
 module RailsAiContext
   class Configuration
     PRESETS = {
-      standard: %i[schema models routes jobs gems conventions controllers tests migrations stimulus view_templates design_tokens config components],
+      standard: %i[schema models routes jobs gems conventions controllers tests migrations stimulus
+                   view_templates design_tokens config components
+                   turbo auth accessibility performance i18n],
       full: %i[schema models routes jobs gems conventions stimulus controllers views view_templates design_tokens turbo
                i18n config active_storage action_text auth api tests rake_tasks assets
                devops action_mailbox migrations seeds middleware engines multi_database
@@ -111,7 +113,7 @@ module RailsAiContext
       @server_name         = "rails-ai-context"
       @server_version      = RailsAiContext::VERSION
       @introspectors       = PRESETS[:full].dup
-      @excluded_paths      = %w[node_modules tmp log vendor .git]
+      @excluded_paths      = %w[node_modules tmp log vendor .git doc docs]
       @sensitive_patterns  = %w[
         .env .env.* config/master.key config/credentials.yml.enc
         config/credentials/*.yml.enc *.pem *.key

data/lib/rails_ai_context/doctor.rb

@@ -38,7 +38,12 @@ module RailsAiContext
     end
 
     def run
-      results = CHECKS.filter_map { |check| send(check) rescue nil }
+      results = CHECKS.filter_map do |check|
+        send(check)
+      rescue StandardError => e
+        $stderr.puts "[rails-ai-context] Doctor check #{check} failed: #{e.class}: #{e.message}"
+        nil
+      end
       score = compute_score(results)
       { checks: results, score: score }
     end

data/lib/rails_ai_context/fingerprinter.rb

@@ -36,6 +36,17 @@ module RailsAiContext
         root = app.root.to_s
         digest = Digest::SHA256.new
 
+        # Include the gem's own version so cache invalidates during gem development
+        digest.update(RailsAiContext::VERSION)
+
+        # Include gem lib directory mtime when using a local/path gem (development mode)
+        gem_lib = File.expand_path("../../..", __FILE__)
+        if gem_lib.start_with?(root) || (defined?(Bundler) && local_gem_path?)
+          Dir.glob(File.join(gem_lib, "**/*.rb")).sort.each do |path|
+            digest.update(File.mtime(path).to_f.to_s)
+          end
+        end
+
         WATCHED_FILES.each do |file|
           path = File.join(root, file)
           digest.update(File.mtime(path).to_f.to_s) if File.exist?(path)
@@ -53,6 +64,19 @@ module RailsAiContext
         digest.hexdigest
       end
 
+      private
+
+      # Detect if this gem is loaded via a local path (path: in Gemfile)
+      def local_gem_path?
+        spec = Bundler.rubygems.find_name("rails-ai-context").first
+        return false unless spec
+        spec.source.is_a?(Bundler::Source::Path)
+      rescue
+        false
+      end
+
+      public
+
       def changed?(app, previous)
         compute(app) != previous
       end

data/lib/rails_ai_context/introspectors/component_introspector.rb

@@ -49,11 +49,15 @@ module RailsAiContext
         class_name = extract_class_name(content)
         return nil unless class_name
 
+        props = extract_props(content)
+        enum_values = extract_enum_values(content)
+        attach_enum_values_to_props(props, enum_values, content)
+
         component = {
           name: class_name,
           file: relative,
           type: detect_component_type(content),
-          props: extract_props(content),
+          props: props,
           slots: extract_slots(content)
         }
 
@@ -67,21 +71,60 @@ module RailsAiContext
       end
 
       def extract_class_name(content)
-        match = content.match(/class\s+(\w+)/)
-        match[1] if match
+        # Extract fully qualified class name (e.g., Components::Articles::Article)
+        match = content.match(/class\s+([\w:]+)/)
+        return nil unless match
+
+        full_name = match[1]
+        # Return the last meaningful segment for display, but keep namespace context
+        # e.g., "Components::Articles::Article" → "Articles::Article"
+        #        "RubyUI::Button" → "Button"
+        #        "AlertComponent" → "AlertComponent"
+        parts = full_name.split("::")
+        if parts.size > 2 && parts.first == "Components"
+          parts[1..].join("::")
+        elsif parts.size > 1 && %w[Components RubyUI].include?(parts.first)
+          parts.last
+        else
+          full_name
+        end
       end
 
       def detect_component_type(content)
-        if content.match?(/< (ViewComponent::Base|ApplicationComponent)/)
+        if content.match?(/< (ViewComponent::Base|ApplicationComponent)\b/)
           :view_component
-        elsif content.match?(/< (Phlex::HTML|Phlex::SVG|ApplicationView|ApplicationComponent)/) &&
-              content.match?(/def (view_)?template/)
+        elsif content.match?(/< (Phlex::HTML|Phlex::SVG|ApplicationView|ApplicationComponent)\b/) ||
+              (content.match?(/< \S+/) && inherits_from_phlex_base?(content))
           :phlex
         else
           :unknown
         end
       end
 
+      def inherits_from_phlex_base?(content)
+        parent_match = content.match(/class\s+\S+\s*<\s*(\S+)/)
+        return false unless parent_match
+
+        parent_class = parent_match[1]
+        @phlex_bases ||= detect_phlex_bases
+        @phlex_bases.include?(parent_class)
+      end
+
+      def detect_phlex_bases
+        bases = Set.new
+        return bases unless Dir.exist?(components_dir)
+
+        Dir.glob(File.join(components_dir, "**/*.rb")).each do |path|
+          content = File.read(path, encoding: "UTF-8", invalid: :replace, undef: :replace)
+          if content.match?(/< (Phlex::HTML|Phlex::SVG)\b/)
+            match = content.match(/class\s+(\S+)\s*</)
+            bases << match[1] if match
+          end
+        end
+
+        bases
+      end
+
       def extract_props(content)
         # Extract from initialize method parameters
         init_match = content.match(/def initialize\(([^)]*)\)/m)
@@ -128,7 +171,7 @@ module RailsAiContext
         end
 
         # Phlex slots: def slot_name(&block)
-        if content.match?(/< Phlex/)
+        if detect_component_type(content) == :phlex
           content.scan(/def\s+(\w+)\s*\(\s*&\s*\w*\s*\)/).each do |name,|
             next if %w[initialize template view_template before_template after_template].include?(name)
             slots << { name: name, type: :phlex_slot }
@@ -138,6 +181,78 @@ module RailsAiContext
         slots
       end
 
+      # Extracts enumerable values from constants and case statements.
+      # Returns a hash mapping downcased constant/variable names to arrays of symbol values.
+      # Detects three patterns:
+      #   1. Hash constants: VARIANTS = { primary: "...", secondary: "..." } -> keys
+      #   2. Array constants: SIZES = [:sm, :md, :lg] -> elements
+      #   3. Case statements: case @variant; when :primary; when :secondary -> when values
+      def extract_enum_values(content)
+        enums = {}
+
+        # Pattern 1: Hash constants — NAME = { key: "value", ... }
+        content.scan(/([A-Z][A-Z_0-9]*)\s*=\s*\{([^}]*)\}/m) do |name, body|
+          keys = body.scan(/(\w+):/).map(&:first)
+          enums[name.downcase] = keys if keys.any?
+        end
+
+        # Pattern 2: Array constants — NAME = [:sym, :sym, ...]
+        content.scan(/([A-Z][A-Z_0-9]*)\s*=\s*\[([^\]]*)\]/) do |name, body|
+          values = body.scan(/:(\w+)/).map(&:first)
+          enums[name.downcase] = values if values.any?
+        end
+
+        # Pattern 3: Case statements — case @ivar; when :val1 ... when :val2
+        # Use a non-greedy match that stops at the next `end`, `case`, or `def` keyword
+        content.scan(/case\s+@(\w+)\s*\n(.*?)(?=\n\s*(?:end|case|def)\b)/m) do |ivar, block|
+          values = block.scan(/when\s+:(\w+)/).map(&:first)
+          next if values.empty?
+          # Merge with existing values for same ivar (handles multiple case blocks)
+          existing = enums[ivar] || []
+          enums[ivar] = (existing + values).uniq
+        end
+
+        enums
+      end
+
+      # Matches extracted enum values to props by:
+      #   1. Direct ivar match: prop "variant" matches case @variant values
+      #   2. Constant name match: prop "size" matches SIZES constant, prop "variant" matches VARIANTS constant
+      #   3. Constant usage in initialize: @size referenced as SIZES[@size] matches prop "size"
+      def attach_enum_values_to_props(props, enum_values, content)
+        props.each do |prop|
+          name = prop[:name]
+          values = nil
+
+          # Direct match: prop name matches case @ivar
+          values = enum_values[name] if enum_values.key?(name)
+
+          # Constant name match: prop "size" -> SIZES, prop "variant" -> VARIANTS/COLORS
+          unless values
+            # Try pluralized forms and common naming patterns
+            candidates = [ name.upcase + "S", name.upcase + "ES", name.upcase ]
+            candidates.each do |candidate|
+              if enum_values.key?(candidate.downcase)
+                values = enum_values[candidate.downcase]
+                break
+              end
+            end
+          end
+
+          # Constant usage match: find CONST[@ivar] patterns in the file
+          unless values
+            content.scan(/([A-Z][A-Z_0-9]*)\[@#{name}\]/) do |const_name,|
+              if enum_values.key?(const_name.downcase)
+                values = enum_values[const_name.downcase]
+                break
+              end
+            end
+          end
+
+          prop[:values] = values if values&.any?
+        end
+      end
+
       def find_preview(component_path, class_name)
         # Check common preview locations
         preview_name = class_name.sub(/Component\z/, "").underscore

data/lib/rails_ai_context/introspectors/performance_introspector.rb

@@ -54,6 +54,10 @@ module RailsAiContext
             elsif (idx = line.match(/add_index\s+"#{Regexp.escape(current_table)}",\s+(?:"(\w+)"|\[([^\]]+)\])/))
               col_name = idx[1] || idx[2]&.gsub(/["'\s]/, "")
               tables[current_table][:indexes] << col_name
+            elsif (tidx = line.match(/t\.index\s+\[([^\]]+)\]/))
+              # t.index ["col_name"] inside create_table block
+              cols = tidx[1].gsub(/["'\s]/, "").split(",")
+              cols.each { |c| tables[current_table][:indexes] << c }
             end
           end
 
@@ -226,20 +230,23 @@ module RailsAiContext
 
         return findings if model_names.empty?
 
+        # Build a single regex matching any model's .all call to avoid O(n*m) scanning
+        escaped_names = model_names.map { |n| Regexp.escape(n) }
+        combined_pattern = /(#{escaped_names.join("|")})\.all\b/
+
         Dir.glob(File.join(controllers_dir, "**/*.rb")).each do |path|
           content = File.read(path, encoding: "UTF-8", invalid: :replace, undef: :replace)
           relative = path.sub("#{root}/", "")
 
-          model_names.each do |model_name|
-            content.scan(/#{Regexp.escape(model_name)}\.all\b/).each do
-              findings << {
-                controller: relative,
-                model: model_name,
-                suggestion: "#{model_name}.all loads all records into memory. Consider pagination or scoping."
-              }
-            end
+          content.scan(combined_pattern).each do |match|
+            model_name = match[0]
+            findings << {
+              controller: relative,
+              model: model_name,
+              suggestion: "#{model_name}.all loads all records into memory. Consider pagination or scoping."
+            }
           end
-        rescue
+        rescue StandardError
           next
         end
 
@@ -277,7 +284,8 @@ module RailsAiContext
         total_issues = result[:n_plus_one_risks].size +
                        result[:missing_counter_cache].size +
                        result[:missing_fk_indexes].size +
-                       result[:model_all_in_controllers].size
+                       result[:model_all_in_controllers].size +
+                       result[:eager_load_candidates].size
 
         {
           total_issues: total_issues,