rails-ai-context 0.15.3 → 0.15.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 688db0da1f692aa30e3e0129f13819bb00959f64306c4bb2efabf3ced84dd7f0
-  data.tar.gz: 3338568f68df0f0a74f312e234138580d7b20f607adc6404709c0910580d90c5
+  metadata.gz: df16cb513ad22c8fca7405fed9a9b111e7018184e1bd16cd08230dace76a9612
+  data.tar.gz: c4846d86197db0fe697b21cd23ead829a4433abfe444b0a14695433a5ab3d0f0
 SHA512:
-  metadata.gz: 10302870e79b4ef00cdbe55d48e38f03e602b5bed0ccd5d2cf3b5176fa4daa171e0a9d77f044ec32c95e9bbbbc19a3c8cac3de951900e17edd05f8647ea2ebaa
-  data.tar.gz: bc923eca00f2a7bcdd6fff58aeb7738e1b47b176cf9db1eba7c3f83d7e864aba433d582614f6e63896d0cd9e9042c7b50203473c0915f9f6ea549b49c89d3264
+  metadata.gz: dd8e4f99421b6efbc1b589734c03dbf3f167ec6a2779957a357f2924a9ac3bccd3d40c5c8aca442d92e9392cfed40d517b20455f7e9a9dd2f792c543c3c13928
+  data.tar.gz: f970c8cf22b53531143352ea4a8a90d615dbe7530a2a70eabd04dd7fd9e532119a803260681dbb431278ad6bfdc7103e82b487840514afd0c49b4c792e3ce8df

data/CHANGELOG.md

@@ -5,6 +5,23 @@ 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.4] - 2026-03-22
+
+### Fixed
+
+- **View subfolder paths** — listings now show full relative paths (`bonus/brand_profiles/index.html.erb`) instead of just basenames.
+- **Controller flexible matching** — `"cooks"`, `"CooksController"`, `"cookscontroller"` all resolve (matches other tools' forgiving lookup).
+- **View path traversal** — explicit `..` and absolute path rejection before any filesystem operation.
+- **Schema case-insensitive** — table lookup now case-insensitive (matches models/routes/etc.).
+- **limit:0 silent empty** — uses default instead of returning empty results.
+- **offset past end** — shows "Use `offset:0` to start over" instead of empty response.
+- **Search ordering** — deterministic results via `--sort=path` on ripgrep.
+- **Generated context prepended** — `<!-- BEGIN rails-ai-context -->` section now placed at top of existing files (AI reads top-to-bottom, may truncate at token limits).
+
+### Added
+
+- **Pagination on models, controllers, stimulus** — `limit`/`offset` params (default 50) with "end of results" hints. Prevents token bombs on large apps.
+
 ## [0.15.3] - 2026-03-22
 
 ### Fixed

data/lib/rails_ai_context/serializers/context_file_serializer.rb

@@ -107,8 +107,8 @@ module RailsAiContext
               marked_content
             )
           else
-            # File exists without markers — append marked section
-            "#{existing.chomp}\n\n#{marked_content}"
+            # File exists without markers — prepend our section so AI reads it first
+            "#{marked_content}\n#{existing}"
           end
 
           if new_content == existing

data/lib/rails_ai_context/tools/get_controllers.rb

@@ -20,13 +20,21 @@ module RailsAiContext
             type: "string",
             enum: %w[summary standard full],
             description: "Detail level for controller listing. summary: names + action counts. standard: names + action list (default). full: everything. Ignored when specific controller is given."
+          },
+          limit: {
+            type: "integer",
+            description: "Max controllers to return when listing. Default: 50."
+          },
+          offset: {
+            type: "integer",
+            description: "Skip this many controllers for pagination. Default: 0."
           }
         }
       )
 
       annotations(read_only_hint: true, destructive_hint: false, idempotent_hint: true, open_world_hint: false)
 
-      def self.call(controller: nil, action: nil, detail: "standard", server_context: nil)
+      def self.call(controller: nil, action: nil, detail: "standard", limit: nil, offset: 0, server_context: nil)
         data = cached_context[:controllers]
         return text_response("Controller introspection not available. Add :controllers to introspectors.") unless data
         return text_response("Controller introspection failed: #{data[:error]}") if data[:error]
@@ -38,8 +46,13 @@ module RailsAiContext
         app_controller_names = controllers.keys.reject { |name| framework_controllers.include?(name) }.sort
 
         # Specific controller — always full detail (searches ALL controllers including framework)
+        # Flexible matching: "cooks", "CooksController", "cookscontroller" all work
         if controller
-          key = controllers.keys.find { |k| k.downcase == controller.downcase } || controller
+          normalized = controller.downcase.delete_suffix("controller")
+          key = controllers.keys.find { |k|
+            kd = k.downcase
+            kd == controller.downcase || kd.delete_suffix("controller") == normalized
+          } || controller
           info = controllers[key]
           return text_response("Controller '#{controller}' not found. Available: #{app_controller_names.join(', ')}") unless info
           return text_response("Error inspecting #{key}: #{info[:error]}") if info[:error]
@@ -54,33 +67,47 @@ module RailsAiContext
 
         app_controllers = controllers.reject { |name, _| framework_controllers.include?(name) }
 
+        # Pagination
+        total = app_controllers.size
+        offset = [ offset.to_i, 0 ].max
+        limit = limit.nil? ? 50 : [ limit.to_i, 1 ].max
+        all_names = app_controllers.keys.sort
+        paginated_names = all_names.drop(offset).first(limit)
+
+        if paginated_names.empty? && total > 0
+          return text_response("No controllers at offset #{offset}. Total: #{total}. Use `offset:0` to start over.")
+        end
+
+        pagination_hint = offset + limit < total ? "\n_Showing #{paginated_names.size} of #{total}. Use `offset:#{offset + limit}` for more._" : ""
+
         # Listing mode
         case detail
         when "summary"
-          lines = [ "# Controllers (#{app_controllers.size})", "" ]
-          app_controllers.keys.sort.each do |name|
+          lines = [ "# Controllers (#{total})", "" ]
+          paginated_names.each do |name|
             info = app_controllers[name]
             action_count = info[:actions]&.size || 0
             lines << "- **#{name}** — #{action_count} actions"
           end
-          lines << "" << "_Use `controller:\"Name\"` for full detail._"
+          lines << "" << "_Use `controller:\"Name\"` for full detail._#{pagination_hint}"
           text_response(lines.join("\n"))
 
         when "standard"
-          lines = [ "# Controllers (#{app_controllers.size})", "" ]
-          app_controllers.keys.sort.each do |name|
+          lines = [ "# Controllers (#{total})", "" ]
+          paginated_names.each do |name|
             info = app_controllers[name]
             actions = info[:actions]&.join(", ") || "none"
             lines << "- **#{name}** — #{actions}"
           end
-          lines << "" << "_Use `controller:\"Name\"` for filters and strong params, or `detail:\"full\"` for everything._"
+          lines << "" << "_Use `controller:\"Name\"` for filters and strong params, or `detail:\"full\"` for everything._#{pagination_hint}"
           text_response(lines.join("\n"))
 
         when "full"
-          lines = [ "# Controllers (#{app_controllers.size})", "" ]
+          lines = [ "# Controllers (#{total})", "" ]
 
           # Group sibling controllers that share the same parent and identical structure
-          grouped = app_controllers.keys.sort.group_by do |name|
+          paginated_ctrl = app_controllers.select { |k, _| paginated_names.include?(k) }
+          grouped = paginated_ctrl.keys.sort.group_by do |name|
             info = app_controllers[name]
             parent = info[:parent_class]
             # Group by parent + actions + filters + params fingerprint
@@ -121,11 +148,12 @@ module RailsAiContext
               end
             end
           end
+          lines << pagination_hint unless pagination_hint.empty?
           text_response(lines.join("\n"))
 
         else
-          list = app_controllers.keys.sort.map { |c| "- #{c}" }.join("\n")
-          text_response("# Controllers (#{app_controllers.size})\n\n#{list}")
+          list = paginated_names.map { |c| "- #{c}" }.join("\n")
+          text_response("# Controllers (#{total})\n\n#{list}#{pagination_hint}")
         end
       end
 

data/lib/rails_ai_context/tools/get_model_details.rb

@@ -16,13 +16,21 @@ module RailsAiContext
             type: "string",
             enum: %w[summary standard full],
             description: "Detail level for model listing. summary: names only. standard: names + association/validation counts (default). full: names + full association list. Ignored when specific model is given (always returns full)."
+          },
+          limit: {
+            type: "integer",
+            description: "Max models to return when listing. Default: 50."
+          },
+          offset: {
+            type: "integer",
+            description: "Skip this many models for pagination. Default: 0."
           }
         }
       )
 
       annotations(read_only_hint: true, destructive_hint: false, idempotent_hint: true, open_world_hint: false)
 
-      def self.call(model: nil, detail: "standard", server_context: nil)
+      def self.call(model: nil, detail: "standard", limit: nil, offset: 0, server_context: nil)
         models = cached_context[:models]
         return text_response("Model introspection not available. Add :models to introspectors.") unless models
         return text_response("Model introspection failed: #{models[:error]}") if models[:error]
@@ -36,15 +44,28 @@ module RailsAiContext
           return text_response(format_model(key, data))
         end
 
+        # Pagination
+        total = models.size
+        offset = [ offset.to_i, 0 ].max
+        limit = normalize_limit(limit, 50)
+        all_names = models.keys.sort
+        paginated = all_names.drop(offset).first(limit)
+
+        if paginated.empty? && total > 0
+          return text_response("No models at offset #{offset}. Total: #{total}. Use `offset:0` to start over.")
+        end
+
+        pagination_hint = offset + limit < total ? "\n_Showing #{paginated.size} of #{total}. Use `offset:#{offset + limit}` for more._" : ""
+
         # Listing mode
         case detail
         when "summary"
-          model_list = models.keys.sort.map { |m| "- #{m}" }.join("\n")
-          text_response("# Available models (#{models.size})\n\n#{model_list}\n\n_Use `model:\"Name\"` for full detail._")
+          model_list = paginated.map { |m| "- #{m}" }.join("\n")
+          text_response("# Available models (#{total})\n\n#{model_list}\n\n_Use `model:\"Name\"` for full detail._#{pagination_hint}")
 
         when "standard"
-          lines = [ "# Models (#{models.size})", "" ]
-          models.keys.sort.each do |name|
+          lines = [ "# Models (#{total})", "" ]
+          paginated.each do |name|
             data = models[name]
             next if data[:error]
             assoc_count = (data[:associations] || []).size
@@ -53,12 +74,12 @@ module RailsAiContext
             line += " — #{assoc_count} associations, #{val_count} validations" if assoc_count > 0 || val_count > 0
             lines << line
           end
-          lines << "" << "_Use `model:\"Name\"` for full detail, or `detail:\"full\"` for association lists._"
+          lines << "" << "_Use `model:\"Name\"` for full detail, or `detail:\"full\"` for association lists._#{pagination_hint}"
           text_response(lines.join("\n"))
 
         when "full"
-          lines = [ "# Models (#{models.size})", "" ]
-          models.keys.sort.each do |name|
+          lines = [ "# Models (#{total})", "" ]
+          paginated.each do |name|
             data = models[name]
             next if data[:error]
             assocs = (data[:associations] || []).map { |a| "#{a[:type]} :#{a[:name]}" }.join(", ")
@@ -67,15 +88,21 @@ module RailsAiContext
             line += " — #{assocs}" unless assocs.empty?
             lines << line
           end
-          lines << "" << "_Use `model:\"Name\"` for validations, scopes, callbacks, and more._"
+          lines << "" << "_Use `model:\"Name\"` for validations, scopes, callbacks, and more._#{pagination_hint}"
           text_response(lines.join("\n"))
 
         else
-          model_list = models.keys.sort.map { |m| "- #{m}" }.join("\n")
-          text_response("# Available models (#{models.size})\n\n#{model_list}")
+          model_list = paginated.map { |m| "- #{m}" }.join("\n")
+          text_response("# Available models (#{total})\n\n#{model_list}#{pagination_hint}")
         end
       end
 
+      private_class_method def self.normalize_limit(limit, default)
+        return default if limit.nil?
+        val = limit.to_i
+        val < 1 ? default : val
+      end
+
       private_class_method def self.format_model(name, data)
         lines = [ "# #{name}", "" ]
         lines << "**Table:** `#{data[:table_name]}`" if data[:table_name]

data/lib/rails_ai_context/tools/get_schema.rb

@@ -47,24 +47,29 @@ module RailsAiContext
 
         tables = schema[:tables] || {}
 
-        # Single table — always full detail (existing behavior)
-        if table
-          table_data = tables[table]
-          return text_response("Table '#{table}' not found. Available: #{tables.keys.sort.join(', ')}") unless table_data
-          output = format == "json" ? table_data.to_json : format_table_markdown(table, table_data)
-          return text_response(output)
-        end
-
         # Return full JSON if requested (existing behavior)
         return text_response(schema.to_json) if format == "json" && detail == "full"
 
         total = tables.size
         offset = [ offset.to_i, 0 ].max
 
+        # Single table — case-insensitive lookup
+        if table
+          table_key = tables.keys.find { |k| k.downcase == table.downcase } || table
+          table_data = tables[table_key]
+          return text_response("Table '#{table}' not found. Available: #{tables.keys.sort.join(', ')}") unless table_data
+          output = format == "json" ? table_data.to_json : format_table_markdown(table_key, table_data)
+          return text_response(output)
+        end
+
         case detail
         when "summary"
           limit ||= 50
+          limit = 50 if limit.to_i < 1
           paginated = tables.keys.sort.drop(offset).first(limit)
+          if paginated.empty? && total > 0
+            return text_response("No tables at offset #{offset}. Total: #{total}. Use `offset:0` to start over.")
+          end
           lines = [ "# Schema Summary (#{total} tables)", "" ]
           paginated.each do |name|
             data = tables[name]
@@ -77,6 +82,7 @@ module RailsAiContext
 
         when "standard"
           limit ||= 15
+          limit = 15 if limit.to_i < 1
           paginated = tables.keys.sort.drop(offset).first(limit)
           if paginated.empty?
             return text_response("No tables at offset #{offset}. Total tables: #{total}. Use `offset:0` to start from the beginning.")
@@ -96,9 +102,12 @@ module RailsAiContext
           text_response(lines.join("\n"))
 
         when "full"
-          # Existing behavior — but now with optional pagination
           limit ||= 5
+          limit = 5 if limit.to_i < 1
           paginated = tables.keys.sort.drop(offset).first(limit)
+          if paginated.empty? && total > 0
+            return text_response("No tables at offset #{offset}. Total: #{total}. Use `offset:0` to start over.")
+          end
           lines = [ "# Schema Full Detail (#{paginated.size} of #{total} tables)", "" ]
           paginated.each do |name|
             lines << format_table_markdown(name, tables[name])

data/lib/rails_ai_context/tools/get_stimulus.rb

@@ -16,36 +16,56 @@ module RailsAiContext
             type: "string",
             enum: %w[summary standard full],
             description: "Detail level. summary: names + counts. standard: names + targets + actions (default). full: everything including values, outlets, classes."
+          },
+          limit: {
+            type: "integer",
+            description: "Max controllers to return when listing. Default: 50."
+          },
+          offset: {
+            type: "integer",
+            description: "Skip this many controllers for pagination. Default: 0."
           }
         }
       )
 
       annotations(read_only_hint: true, destructive_hint: false, idempotent_hint: true, open_world_hint: false)
 
-      def self.call(controller: nil, detail: "standard", server_context: nil)
+      def self.call(controller: nil, detail: "standard", limit: nil, offset: 0, server_context: nil)
         data = cached_context[:stimulus]
         return text_response("Stimulus introspection not available. Add :stimulus to introspectors.") unless data
         return text_response("Stimulus introspection failed: #{data[:error]}") if data[:error]
 
-        controllers = data[:controllers] || []
-        return text_response("No Stimulus controllers found.") if controllers.empty?
+        all_controllers = data[:controllers] || []
+        return text_response("No Stimulus controllers found.") if all_controllers.empty?
 
         # Specific controller — accepts both dash and underscore naming
         # (HTML uses data-controller="weekly-chart", file is weekly_chart_controller.js)
         if controller
           normalized = controller.downcase.tr("-", "_")
-          ctrl = controllers.find { |c| c[:name]&.downcase&.tr("-", "_") == normalized }
-          return text_response("Controller '#{controller}' not found. Available: #{controllers.map { |c| c[:name] }.sort.join(', ')}\n\n_Note: use dashes in HTML (`data-controller=\"my-name\"`) but underscores for lookup (`controller:\"my_name\"`)._") unless ctrl
+          ctrl = all_controllers.find { |c| c[:name]&.downcase&.tr("-", "_") == normalized }
+          return text_response("Controller '#{controller}' not found. Available: #{all_controllers.map { |c| c[:name] }.sort.join(', ')}\n\n_Note: use dashes in HTML (`data-controller=\"my-name\"`) but underscores for lookup (`controller:\"my_name\"`)._") unless ctrl
           return text_response(format_controller_full(ctrl))
         end
 
+        # Pagination
+        total = all_controllers.size
+        offset_val = [ offset.to_i, 0 ].max
+        limit_val = limit.nil? ? 50 : [ limit.to_i, 1 ].max
+        sorted_all = all_controllers.sort_by { |c| c[:name]&.to_s || "" }
+        controllers = sorted_all.drop(offset_val).first(limit_val)
+
+        if controllers.empty? && total > 0
+          return text_response("No controllers at offset #{offset_val}. Total: #{total}. Use `offset:0` to start over.")
+        end
+
+        pagination_hint = offset_val + limit_val < total ? "\n_Showing #{controllers.size} of #{total}. Use `offset:#{offset_val + limit_val}` for more._" : ""
+
         case detail
         when "summary"
-          sorted = controllers.sort_by { |c| c[:name]&.to_s || "" }
-          active = sorted.select { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
-          empty = sorted.reject { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
+          active = controllers.select { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
+          empty = controllers.reject { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
 
-          lines = [ "# Stimulus Controllers (#{controllers.size})", "" ]
+          lines = [ "# Stimulus Controllers (#{total})", "" ]
           active.each do |ctrl|
             targets = (ctrl[:targets] || []).size
             actions = (ctrl[:actions] || []).size
@@ -55,15 +75,14 @@ module RailsAiContext
             names = empty.map { |c| c[:name] }.join(", ")
             lines << "- _#{names}_ (lifecycle only)"
           end
-          lines << "" << "_Use `controller:\"name\"` for full detail._"
+          lines << "" << "_Use `controller:\"name\"` for full detail._#{pagination_hint}"
           text_response(lines.join("\n"))
 
         when "standard"
-          sorted = controllers.sort_by { |c| c[:name]&.to_s || "" }
-          active = sorted.select { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
-          empty = sorted.reject { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
+          active = controllers.select { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
+          empty = controllers.reject { |c| (c[:targets] || []).any? || (c[:actions] || []).any? }
 
-          lines = [ "# Stimulus Controllers (#{controllers.size})", "" ]
+          lines = [ "# Stimulus Controllers (#{total})", "" ]
           active.each do |ctrl|
             lines << "## #{ctrl[:name]}"
             lines << "- Targets: #{(ctrl[:targets] || []).join(', ')}" if ctrl[:targets]&.any?
@@ -74,12 +93,13 @@ module RailsAiContext
             names = empty.map { |c| c[:name] }.join(", ")
             lines << "_Lifecycle only (no targets/actions): #{names}_"
           end
+          lines << pagination_hint unless pagination_hint.empty?
           text_response(lines.join("\n"))
 
         when "full"
-          lines = [ "# Stimulus Controllers (#{controllers.size})", "" ]
+          lines = [ "# Stimulus Controllers (#{total})", "" ]
           lines << "_HTML naming: `data-controller=\"my-name\"` (dashes in HTML, underscores in filenames)_" << ""
-          controllers.sort_by { |c| c[:name]&.to_s || "" }.each do |ctrl|
+          controllers.each do |ctrl|
             lines << format_controller_full(ctrl) << ""
           end
           text_response(lines.join("\n"))

data/lib/rails_ai_context/tools/get_view.rb

@@ -70,10 +70,10 @@ module RailsAiContext
             ctrl_templates.sort.each do |name, meta|
               parts = meta[:partials]&.any? ? " renders: #{meta[:partials].join(', ')}" : ""
               stim = meta[:stimulus]&.any? ? " stimulus: #{meta[:stimulus].join(', ')}" : ""
-              lines << "- #{File.basename(name)} (#{meta[:lines]} lines)#{parts}#{stim}"
+              lines << "- #{name} (#{meta[:lines]} lines)#{parts}#{stim}"
             end
             ctrl_partials.sort.each do |name, meta|
-              lines << "- #{File.basename(name)} (#{meta[:lines]} lines)"
+              lines << "- #{name} (#{meta[:lines]} lines)"
             end
             lines << ""
           end
@@ -91,12 +91,12 @@ module RailsAiContext
             ctrl_templates.sort.each do |name, meta|
               parts = meta[:partials]&.any? ? " renders: #{meta[:partials].join(', ')}" : ""
               stim = meta[:stimulus]&.any? ? " stimulus: #{meta[:stimulus].join(', ')}" : ""
-              lines << "- #{File.basename(name)} (#{meta[:lines]} lines)#{parts}#{stim}"
+              lines << "- #{name} (#{meta[:lines]} lines)#{parts}#{stim}"
             end
             ctrl_partials.sort.each do |name, meta|
               fields = meta[:fields]&.any? ? " fields: #{meta[:fields].join(', ')}" : ""
               helpers = meta[:helpers]&.any? ? " helpers: #{meta[:helpers].join(', ')}" : ""
-              lines << "- #{File.basename(name)} (#{meta[:lines]} lines)#{fields}#{helpers}"
+              lines << "- #{name} (#{meta[:lines]} lines)#{fields}#{helpers}"
             end
             lines << ""
           end
@@ -138,6 +138,11 @@ module RailsAiContext
       MAX_FILE_SIZE = 2_000_000 # 2MB safety limit
 
       private_class_method def self.read_view_file(path)
+        # Reject path traversal attempts before any filesystem operation
+        if path.include?("..") || path.start_with?("/")
+          return text_response("Path not allowed: #{path}")
+        end
+
         views_dir = Rails.root.join("app", "views")
         full_path = views_dir.join(path)
 

data/lib/rails_ai_context/tools/search_code.rb

@@ -90,7 +90,7 @@ module RailsAiContext
       end
 
       private_class_method def self.search_with_ripgrep(pattern, search_path, file_type, max_results, root, ctx_lines = 0)
-        cmd = [ "rg", "--no-heading", "--line-number", "--max-count", max_results.to_s ]
+        cmd = [ "rg", "--no-heading", "--line-number", "--sort=path", "--max-count", max_results.to_s ]
         if ctx_lines > 0
           cmd.push("-C", ctx_lines.to_s)
           # Use colon separator for context lines so parse_rg_output handles them correctly

data/lib/rails_ai_context/version.rb

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

metadata

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