rails-ai-context 0.15.4 → 0.15.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: df16cb513ad22c8fca7405fed9a9b111e7018184e1bd16cd08230dace76a9612
-  data.tar.gz: c4846d86197db0fe697b21cd23ead829a4433abfe444b0a14695433a5ab3d0f0
+  metadata.gz: a9e71509ab3fcf2437fdd2616f8cb88140b478677abdd4ebd7cf6818eb8cead3
+  data.tar.gz: c6847c4a500d762665ea2493c0ee56943bfe2e601b645bf5e9ec97df81f7e495
 SHA512:
-  metadata.gz: dd8e4f99421b6efbc1b589734c03dbf3f167ec6a2779957a357f2924a9ac3bccd3d40c5c8aca442d92e9392cfed40d517b20455f7e9a9dd2f792c543c3c13928
-  data.tar.gz: f970c8cf22b53531143352ea4a8a90d615dbe7530a2a70eabd04dd7fd9e532119a803260681dbb431278ad6bfdc7103e82b487840514afd0c49b4c792e3ce8df
+  metadata.gz: fb13a050a3e8200d1bac5f8d0d59bc1a67469a3fbe904682824541430f9f10755197fa3bd17858c08a2eedcc89f71c9da2dfffea8a8bcd0c177784f85ef511c6
+  data.tar.gz: 4e05a8e22c395b4091d336268aa3e3994f416b11d174e63d922b5979b46695a222176a0871253ddd940638ead977e2917d14998618a91c25ad4a6f891eca9694

data/CHANGELOG.md

@@ -5,6 +5,19 @@ 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.5] - 2026-03-22
+
+### Fixed
+
+- **ERB validation** — now catches missing `<% end %>` by compiling ERB to Ruby then syntax-checking the result (was only checking ERB tag syntax).
+- **Controller namespace format** — accepts both `Bonus::CrisesController` and `bonus/crises` (cross-tool consistency).
+- **Layouts discoverable** — `controller:"layouts"` now works in view tool.
+- **Validate error detail** — Ruby shows up to 5 error lines, JS shows 3 (was truncated to 1).
+- **Invalid/empty regex** — early validation with clear error messages instead of silent fail.
+- **Route count accuracy** — shows filtered count when `app_only:true`, not unfiltered total.
+- **Namespace test lookup** — supports `bonus/crises` format and flat test directories.
+- **Empty inputs** — `near:""` in edit_context and `pattern:""` in search return helpful errors.
+
 ## [0.15.4] - 2026-03-22
 
 ### Fixed

data/README.md

@@ -302,7 +302,7 @@ end
 | `max_tool_response_chars` | `120_000` | Safety cap for MCP tool responses |
 | `excluded_models` | internal Rails models | Models to skip during introspection |
 | `excluded_paths` | `node_modules tmp log vendor .git` | Paths excluded from code search |
-| `sensitive_patterns` | `.env .env.* *.key *.pem config/master.key config/credentials.yml.enc` | File patterns blocked from search and read tools |
+| `sensitive_patterns` | `.env .env.* config/master.key config/credentials.yml.enc config/credentials/*.yml.enc *.pem *.key` | File patterns blocked from search and read tools |
 | `auto_mount` | `false` | Auto-mount HTTP MCP endpoint |
 | `http_path` | `"/mcp"` | HTTP endpoint path |
 | `http_port` | `6029` | HTTP server port |

data/SECURITY.md

@@ -5,7 +5,7 @@
 | Version | Supported          |
 |---------|--------------------|
 | 0.15.x  | :white_check_mark: |
-| < 0.14  | :x:                |
+| < 0.15  | :x:                |
 
 ## Reporting a Vulnerability
 

data/docs/GUIDE.md

@@ -146,22 +146,23 @@ end
 | `app/models/AGENTS.md` | Model reference | Auto-loaded by OpenCode when reading files in `app/models/`. |
 | `app/controllers/AGENTS.md` | Controller reference | Auto-loaded by OpenCode when reading files in `app/controllers/`. |
 
-### Cursor (6 files)
+### Cursor (5 files)
 
 | File | Purpose | Notes |
 |------|---------|-------|
 | `.cursor/rules/rails-project.mdc` | Project overview | `alwaysApply: true` — loaded in every conversation. |
 | `.cursor/rules/rails-models.mdc` | Model reference | `globs: app/models/**/*.rb` — auto-attaches when editing models. |
 | `.cursor/rules/rails-controllers.mdc` | Controller reference | `globs: app/controllers/**/*.rb` — auto-attaches when editing controllers. |
-| `.cursor/rules/rails-ui-patterns.mdc` | UI patterns and design tokens | `alwaysApply: true` — loaded in every conversation. |
+| `.cursor/rules/rails-ui-patterns.mdc` | UI patterns and design tokens | `globs: app/views/**/*.erb` — loaded when editing views. |
 | `.cursor/rules/rails-mcp-tools.mdc` | MCP tool reference | `alwaysApply: true` — always available. |
 
-### Windsurf (3 files)
+### Windsurf (4 files)
 
 | File | Purpose | Notes |
 |------|---------|-------|
 | `.windsurfrules` | Main context file | Hard-capped at 5,800 chars (Windsurf's 6K limit). Truncated silently if exceeded. |
 | `.windsurf/rules/rails-context.md` | Project overview | New Windsurf rules format. |
+| `.windsurf/rules/rails-ui-patterns.md` | UI patterns and design tokens | Loaded when editing views. |
 | `.windsurf/rules/rails-mcp-tools.md` | MCP tool reference | Compact — respects 6K per-file limit. |
 
 ### GitHub Copilot (6 files)
@@ -172,7 +173,7 @@ end
 | `.github/instructions/rails-models.instructions.md` | Model context | `applyTo: app/models/**/*.rb` — loaded when editing models. |
 | `.github/instructions/rails-controllers.instructions.md` | Controller context | `applyTo: app/controllers/**/*.rb` — loaded when editing controllers. |
 | `.github/instructions/rails-context.instructions.md` | Project context and conventions | `applyTo: **/*` — loaded everywhere. |
-| `.github/instructions/rails-ui-patterns.instructions.md` | UI patterns and design tokens | `applyTo: **/*` — loaded everywhere. |
+| `.github/instructions/rails-ui-patterns.instructions.md` | UI patterns and design tokens | `applyTo: app/views/**/*.erb` — loaded when editing views. |
 | `.github/instructions/rails-mcp-tools.instructions.md` | MCP tool reference | `applyTo: **/*` — loaded everywhere. |
 
 ### Generic (1 file)
@@ -359,8 +360,11 @@ Returns controller details: actions, filters, strong params, concerns.
 
 | Param | Type | Description |
 |-------|------|-------------|
-| `controller` | string | Specific controller name (e.g. `UsersController`). Case-insensitive. |
+| `controller` | string | Specific controller name (e.g. `UsersController`, `cooks`, `bonus/crises`). Case-insensitive, flexible format. |
+| `action` | string | Specific action (e.g. `index`). Requires controller. Returns source code with applicable filters. |
 | `detail` | string | `summary` / `standard` (default) / `full`. Ignored when controller is specified. |
+| `limit` | integer | Max controllers to return when listing. Default: 50. |
+| `offset` | integer | Skip this many controllers for pagination. Default: 0. |
 
 **Examples:**
 
@@ -382,7 +386,7 @@ rails_get_controllers(detail: "full")
 
 Returns application configuration. No parameters.
 
-**Returns:** cache store, session store, timezone, middleware stack, initializers, credentials keys, current attributes.
+**Returns:** cache store, session store, timezone, queue adapter, mailer settings, custom middleware, initializers, current attributes.
 
 ```
 rails_get_config()
@@ -391,20 +395,35 @@ rails_get_config()
 
 ### rails_get_test_info
 
-Returns test infrastructure details. No parameters.
+Returns test infrastructure details. Optionally filter by model or controller to find existing tests.
 
-**Returns:** test framework (rspec/minitest), factories/fixtures with locations and counts, system tests, CI config, coverage tool, test helpers.
+**Parameters:**
+
+| Param | Type | Description |
+|-------|------|-------------|
+| `model` | string | Show tests for a specific model (e.g. `User`). |
+| `controller` | string | Show tests for a specific controller (e.g. `Cooks`). |
+| `detail` | string | `summary` / `standard` (default) / `full`. |
 
 ```
 rails_get_test_info()
-  → Framework: rspec, Factories: spec/factories (12 files), CI: .github/workflows/ci.yml, ...
+  → Framework: rspec, Factories: spec/factories (12 files), CI: .github/workflows/ci.yml
+
+rails_get_test_info(model: "User")
+  → Shows spec/models/user_spec.rb test names (summary/standard) or full source (full)
 ```
 
 ### rails_get_gems
 
-Returns notable gems categorized by function. No parameters.
+Returns notable gems categorized by function.
+
+**Parameters:**
+
+| Param | Type | Description |
+|-------|------|-------------|
+| `category` | string | Filter by category: `auth`, `jobs`, `frontend`, `api`, `database`, `files`, `testing`, `deploy`, `all` (default). |
 
-**Returns:** 70+ recognized gems grouped by category (auth, background_jobs, admin, monitoring, search, pagination, etc.) with versions and descriptions.
+**Returns:** Notable gems grouped by category with descriptions.
 
 ```
 rails_get_gems()

data/lib/rails_ai_context/tools/get_controllers.rb

@@ -48,13 +48,19 @@ module RailsAiContext
         # Specific controller — always full detail (searches ALL controllers including framework)
         # Flexible matching: "cooks", "CooksController", "cookscontroller" all work
         if controller
-          normalized = controller.downcase.delete_suffix("controller")
+          # Accept multiple formats: "CooksController", "cooks", "bonus/crises", "Bonus::CrisesController"
+          normalized = controller.downcase.delete_suffix("controller").tr("/", "::")
           key = controllers.keys.find { |k|
             kd = k.downcase
-            kd == controller.downcase || kd.delete_suffix("controller") == normalized
+            kd == controller.downcase ||
+              kd.delete_suffix("controller") == normalized ||
+              kd.delete_suffix("controller").tr("::", "/") == controller.downcase.delete_suffix("controller")
           } || controller
           info = controllers[key]
-          return text_response("Controller '#{controller}' not found. Available: #{app_controller_names.join(', ')}") unless info
+          unless info
+            available = app_controller_names.any? ? "Available: #{app_controller_names.join(', ')}" : "No controllers discovered."
+            return text_response("Controller '#{controller}' not found. #{available}")
+          end
           return text_response("Error inspecting #{key}: #{info[:error]}") if info[:error]
 
           # Specific action — return source code

data/lib/rails_ai_context/tools/get_edit_context.rb

@@ -32,6 +32,11 @@ module RailsAiContext
       SENSITIVE_PATTERNS = nil # uses configuration.sensitive_patterns
 
       def self.call(file:, near:, context_lines: 5, server_context: nil)
+        # Reject empty search term
+        if near.nil? || near.strip.empty?
+          return text_response("The `near` parameter is required. Provide a method name, keyword, or string to find.")
+        end
+
         full_path = Rails.root.join(file)
 
         # Block access to sensitive files (secrets, keys, credentials)

data/lib/rails_ai_context/tools/get_routes.rb

@@ -65,6 +65,7 @@ module RailsAiContext
 
         # Combine PUT/PATCH duplicates (Rails generates both for update routes)
         by_controller = by_controller.transform_values { |actions| dedupe_put_patch(actions) }
+        filtered_total = by_controller.values.sum(&:size)
 
         case detail
         when "summary"
@@ -72,7 +73,7 @@ module RailsAiContext
           app_routes = controller ? by_controller : by_controller.reject { |k, _| FRAMEWORK_PREFIXES.any? { |p| k.downcase.start_with?(p) } }
           framework_routes = controller ? {} : by_controller.select { |k, _| FRAMEWORK_PREFIXES.any? { |p| k.downcase.start_with?(p) } }
 
-          lines = [ "# Routes Summary (#{routes[:total_routes]} total)", "" ]
+          lines = [ "# Routes Summary (#{filtered_total} routes)", "" ]
 
           # Group sibling routes with identical verb patterns (e.g., bonus/*)
           grouped = app_routes.keys.sort.group_by do |ctrl|
@@ -118,7 +119,7 @@ module RailsAiContext
           app_routes = controller ? by_controller : by_controller.reject { |k, _| FRAMEWORK_PREFIXES.any? { |p| k.downcase.start_with?(p) } }
           framework_routes = controller ? {} : by_controller.select { |k, _| FRAMEWORK_PREFIXES.any? { |p| k.downcase.start_with?(p) } }
 
-          lines = [ "# Routes (#{routes[:total_routes]} total)", "" ]
+          lines = [ "# Routes (#{filtered_total} routes)", "" ]
           count = 0
 
           # Group identical sibling route sets (e.g. bonus/*)
@@ -181,7 +182,7 @@ module RailsAiContext
         when "full"
           # Existing full table behavior
           limit ||= 200
-          lines = [ "# Routes Full Detail (#{routes[:total_routes]} total)", "" ]
+          lines = [ "# Routes Full Detail (#{filtered_total} routes)", "" ]
           lines << "| Verb | Path | Controller#Action | Name |"
           lines << "|------|------|-------------------|------|"
           count = 0

data/lib/rails_ai_context/tools/get_test_info.rb

@@ -122,7 +122,8 @@ module RailsAiContext
       MAX_TEST_FILE_SIZE = 500_000 # 500KB safety limit
 
       private_class_method def self.find_test_file(name, type, detail = "full")
-        snake = name.to_s.underscore.sub(/_controller$/, "")
+        # Normalize: accept "Bonus::CrisesController", "bonus/crises", "Crises"
+        snake = name.to_s.tr("/", "::").underscore.sub(/_controller$/, "")
         candidates = case type
         when :model
           [
@@ -133,7 +134,9 @@ module RailsAiContext
           [
             "spec/controllers/#{snake}_controller_spec.rb",
             "spec/requests/#{snake}_spec.rb",
-            "test/controllers/#{snake}_controller_test.rb"
+            "test/controllers/#{snake}_controller_test.rb",
+            # Also try without namespace prefix for flat test dirs
+            "spec/requests/#{snake.split('/').last}_spec.rb"
           ]
         end
 

data/lib/rails_ai_context/tools/get_view.rb

@@ -43,6 +43,11 @@ module RailsAiContext
         end
 
         # Filter by controller (also checks partials for directories like "shared/")
+        # Special case: "layouts" reads from app/views/layouts/ (excluded from normal listing)
+        if controller&.downcase == "layouts"
+          return list_layouts(detail)
+        end
+
         if controller
           ctrl_lower = controller.downcase
           filtered_templates = templates.select { |k, _| k.downcase.start_with?(ctrl_lower + "/") }
@@ -135,6 +140,27 @@ module RailsAiContext
         end
       end
 
+      private_class_method def self.list_layouts(detail)
+        layouts_dir = Rails.root.join("app", "views", "layouts")
+        return text_response("No app/views/layouts/ directory found.") unless Dir.exist?(layouts_dir)
+
+        files = Dir.glob(File.join(layouts_dir, "*")).reject { |f| File.directory?(f) }.sort
+        return text_response("No layout files found.") if files.empty?
+
+        lines = [ "# Layouts (#{files.size} files)", "" ]
+        files.each do |path|
+          relative = "layouts/#{File.basename(path)}"
+          if detail == "full"
+            content = File.read(path, encoding: "UTF-8", invalid: :replace, undef: :replace) rescue "(error reading)"
+            lines << "## #{relative}" << "```erb" << strip_svg(content) << "```" << ""
+          else
+            line_count = (File.readlines(path).size rescue 0)
+            lines << "- #{relative} (#{line_count} lines)"
+          end
+        end
+        text_response(lines.join("\n"))
+      end
+
       MAX_FILE_SIZE = 2_000_000 # 2MB safety limit
 
       private_class_method def self.read_view_file(path)

data/lib/rails_ai_context/tools/search_code.rb

@@ -41,6 +41,18 @@ module RailsAiContext
       def self.call(pattern:, path: nil, file_type: nil, max_results: 30, context_lines: 0, server_context: nil)
         root = Rails.root.to_s
 
+        # Reject empty or whitespace-only patterns
+        if pattern.nil? || pattern.strip.empty?
+          return text_response("Pattern is required. Provide a search term or regex.")
+        end
+
+        # Validate regex syntax early
+        begin
+          Regexp.new(pattern, timeout: 1)
+        rescue RegexpError => e
+          return text_response("Invalid regex pattern: #{e.message}")
+        end
+
         # Validate file_type to prevent injection
         if file_type && !file_type.match?(/\A[a-zA-Z0-9]+\z/)
           return text_response("Invalid file_type: must contain only alphanumeric characters.")

data/lib/rails_ai_context/tools/validate.rb

@@ -93,20 +93,37 @@ module RailsAiContext
         if status.success?
           [ true, nil ]
         else
-          error = result.lines.reject { |l| l.strip.empty? }.first&.strip || "syntax error"
-          error = error.sub(full_path.to_s, File.basename(full_path.to_s))
+          # Show up to 5 non-empty error lines for full context (ruby -c gives multi-line errors)
+          error_lines = result.lines
+            .reject { |l| l.strip.empty? || l.include?("Syntax OK") }
+            .first(5)
+            .map { |l| l.strip.sub(full_path.to_s, File.basename(full_path.to_s)) }
+          error = error_lines.any? ? error_lines.join("\n") : "syntax error"
           [ false, error ]
         end
       end
 
-      # Validate ERB syntax by compiling the template (no shell — uses Open3 array form)
+      # 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.
       private_class_method def self.validate_erb(full_path)
-        script = "require 'erb'; ERB.new(File.read(ARGV[0])).src"
-        result, status = Open3.capture2e("ruby", "-e", script, full_path.to_s)
-        if status.success?
+        # 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)
+
+        unless compile_status.success?
+          error = compiled.lines.reject { |l| l.strip.empty? }.first&.strip || "ERB syntax error"
+          return [ false, error ]
+        end
+
+        # 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 = result.lines.reject { |l| l.strip.empty? }.first&.strip || "ERB syntax error"
+          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 ]
         end
       end
@@ -120,8 +137,12 @@ module RailsAiContext
           if status.success?
             [ true, nil ]
           else
-            error = result.lines.reject { |l| l.strip.empty? }.first&.strip || "syntax error"
-            error = error.sub(full_path.to_s, File.basename(full_path.to_s))
+            # Show up to 3 non-empty error lines for context
+            error_lines = result.lines
+              .reject { |l| l.strip.empty? }
+              .first(3)
+              .map { |l| l.strip.sub(full_path.to_s, File.basename(full_path.to_s)) }
+            error = error_lines.any? ? error_lines.join("\n") : "syntax error"
             [ false, error ]
           end
         else

data/lib/rails_ai_context/version.rb

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

data/server.json

@@ -7,11 +7,11 @@
     "url": "https://github.com/crisnahine/rails-ai-context",
     "source": "github"
   },
-  "version": "0.15.2",
+  "version": "0.15.4",
   "packages": [
     {
       "registryType": "mcpb",
-      "identifier": "https://github.com/crisnahine/rails-ai-context/releases/download/v0.15.2/rails-ai-context-mcp.mcpb",
+      "identifier": "https://github.com/crisnahine/rails-ai-context/releases/download/v0.15.4/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.4
+  version: 0.15.5
 platform: ruby
 authors:
 - crisnahine