rails-ai-context 5.12.0 → 5.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4544796594e923abe87c73ee94e07d87c3cec17aa0df7ad2d631dad8fbb4fee5
-  data.tar.gz: 4a41fc88963c28d5f3725ca1f6899c18d2d5b65a9ab201b9210d76f340e8229b
+  metadata.gz: 3747460b78ef9b52d5e2d4bbe59e348599499285fb0215a97ce029ad49b27aa0
+  data.tar.gz: 2f0fff5d8986ca5ead4d74933dac2c7933d2ca7bd7dd1cab11fc3c4c5c756106
 SHA512:
-  metadata.gz: 5bb1a0735533dcac1f9b35e639e15feb46c0f55e72686db352d3f312ccb737392f3783fd931ce0403141ccdb0a4bd46d95d85455519d0ebdb9e21a30c23d9d7d
-  data.tar.gz: 03cc5484cc2774320155ac1272564d58b82e5418a1a8e815f88e32fd7ed849a795f939fb32823beb53557a514050336f239c62bf9f06c0314ec8d96b23eb60cb
+  metadata.gz: b343b7456a8f26c0738ebf3ca19b50758f54e859d82b3c40dba97765bde151f429f7da7b6d7a9abba2ac35438faaa858686c306e4f99482e2fc5a0b8692a7055
+  data.tar.gz: b18a15e6fcb931bc9ae34c340c8c3c2481092273e7c0d1ca3d63da3b40282f2a18988656a96588c2e80ea271ab2f4ebc224124b7872328663eb5bc63da93ea0f

data/.rubocop.yml

@@ -2,7 +2,7 @@ inherit_gem:
   rubocop-rails-omakase: rubocop.yml
 
 AllCops:
-  TargetRubyVersion: 3.2
+  TargetRubyVersion: 3.1
   NewCops: enable
   Exclude:
     - "spec/fixtures/**/*"

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).
 
+## [5.13.0] - 2026-07-10
+
+### Added
+
+- **Ruby 3.1 and Rails 7.0 are now supported.** `required_ruby_version` drops to `>= 3.1.0` and the `railties` floor to `>= 7.0`, widening the test matrix to Ruby 3.1-4.0 x Rails 7.0-8.1. Rails 7.0 is exercised on Ruby 3.1, its only supported pairing (7.0 predates official Ruby 3.2+ support). Reaching parity required:
+  - A vendored `Data.define` backport (`lib/rails_ai_context/polyfill/data.rb`) for the value objects `Doctor::Check`, `SchemaHint`, and `HydrationResult`. `Data` is a Ruby 3.2+ class; the shim is inert on 3.2+ (guarded on `Data.respond_to?(:define)`) and activates only on 3.1, preserving keyword construction, the `super`-forwarding custom initialize, frozen instances, and value equality.
+  - `rails_search_code` no longer passes the Ruby 3.2+ `timeout:` keyword to `Regexp.new` on Ruby 3.1, where it raised `TypeError: no implicit conversion of Hash into String` and broke every search. The per-pattern ReDoS timeout is applied only when the runtime supports it (`Regexp.respond_to?(:timeout)`); 3.1 builds the pattern without it.
+  - Every version-sensitive Rails call in the 40 introspectors was already `respond_to?`/`defined?`-guarded or rescue-wrapped, so no introspector changed to run on Rails 7.0.
+  - The dev/test Gemfile pins `sqlite3 ~> 1.4` on Rails 7.0 (whose adapter predates the sqlite3 2.x line) and always loads `logger` (concurrent-ruby 1.3.5 dropped the implicit `require "logger"` that Rails < 7.1 relies on at boot).
+
+## [5.12.1] - 2026-07-10
+
+### Fixed
+
+- `rails_query` no longer fails on MySQL with `Transaction characteristics can't be changed while a transaction is in progress`. `execute_mysql` issued `SET TRANSACTION READ ONLY` inside `conn.transaction`, but Rails materializes the lazy `BEGIN` before the first in-block statement, so MySQL (error 1568) rejected the SET on every call -- a 100% failure rate for plain queries and `explain: true`. The SET now runs before the transaction opens; without `GLOBAL`/`SESSION` scope it applies only to the next transaction, which the query transaction immediately consumes, so the read-only guard still holds and nothing leaks onto the pooled connection. PostgreSQL (where `SET TRANSACTION` applies to the current transaction) and SQLite are unaffected. (#89)
+- The static `schema.rb` parser now recognizes the PostgreSQL `timestamptz` and `tsvector` column types. `t.timestamptz` and `t.tsvector` columns were dropped from the parsed schema because the DSL listener's column-type set omitted them, so schema-backed tools under-reported columns on Postgres apps using `timestamp with time zone` or full-text search vectors. (#88)
+
 ## [5.12.0] - 2026-06-29
 
 ### Fixed

data/CONTRIBUTING.md

@@ -64,7 +64,8 @@ The `ToolRunner` (`lib/rails_ai_context/cli/tool_runner.rb`) handles CLI executi
 ## Code Style
 
 - Follow `rubocop-rails-omakase` style (run `bundle exec rubocop`)
-- Ruby 3.2+ features welcome (pattern matching, etc.)
+- Target Ruby 3.1 (the gem's floor). Ruby 3.2+ features (`Data.define`, `Regexp` `timeout:`, etc.) need a runtime-guarded fallback so 3.1 keeps working
+- Support Rails 7.0 through 8.1. Guard Rails 7.1+ APIs with `respond_to?`/`defined?` so introspectors degrade cleanly on 7.0
 - Every introspector must return a Hash and never raise - wrap errors in `{ error: msg }`
 - MCP tools return `MCP::Tool::Response` objects
 - All tools must be prefixed with `rails_` and annotated as read-only

data/README.md

@@ -19,8 +19,8 @@
 [![CI](https://github.com/crisnahine/rails-ai-context/actions/workflows/ci.yml/badge.svg)](https://github.com/crisnahine/rails-ai-context/actions)
 [![MCP Registry](https://img.shields.io/badge/MCP_Registry-listed-green)](https://registry.modelcontextprotocol.io)
 <br>
-[![Ruby](https://img.shields.io/badge/Ruby-3.2%20%7C%203.3%20%7C%203.4-CC342D)](https://github.com/crisnahine/rails-ai-context)
-[![Rails](https://img.shields.io/badge/Rails-7.1%20%7C%207.2%20%7C%208.0-CC0000)](https://github.com/crisnahine/rails-ai-context)
+[![Ruby](https://img.shields.io/badge/Ruby-3.1%20%7C%203.2%20%7C%203.3%20%7C%203.4-CC342D)](https://github.com/crisnahine/rails-ai-context)
+[![Rails](https://img.shields.io/badge/Rails-7.0%20%7C%207.1%20%7C%207.2%20%7C%208.0-CC0000)](https://github.com/crisnahine/rails-ai-context)
 [![Tests](https://img.shields.io/badge/Tests-2257%20passing-brightgreen)](https://github.com/crisnahine/rails-ai-context/actions)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
@@ -531,7 +531,7 @@ end
 
 ## Requirements
 
-- **Ruby** >= 3.2 &nbsp;&nbsp; **Rails** >= 7.1
+- **Ruby** >= 3.1 &nbsp;&nbsp; **Rails** >= 7.0
 - **Optional:** `brakeman` for security scanning, `listen` for watch mode, `ripgrep` for fast search
 
 <br>

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

@@ -8,10 +8,10 @@ module RailsAiContext
       class SchemaDslListener < BaseListener
         COLUMN_TYPES = %w[
           string integer text boolean datetime date decimal float binary
-          references belongs_to jsonb json uuid bigint timestamp time
+          references belongs_to jsonb json uuid bigint timestamp timestamptz time
           inet cidr macaddr hstore ltree numrange tsrange daterange
           bit bit_varying money oid xml point line lseg box path
-          polygon circle interval serial virtual primary_key
+          polygon circle interval serial tsvector virtual primary_key
         ].to_set.freeze
 
         def on_call_node_enter(node)

data/lib/rails_ai_context/polyfill/data.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+# Backport of the Ruby 3.2+ immutable value class Data for Ruby 3.1, which
+# ships no Data at all. Inert on Ruby >= 3.2: the guard sees the built-in
+# Data.define and defines nothing, so every runtime from 3.2 up keeps the real
+# implementation and only 3.1 gets this shim.
+#
+# The value-setting initialize lives on this base class so a block-defined
+# initialize on a subclass can call super into it (the HydrationResult idiom).
+# Members are keyword- or positionally-constructable, instances are frozen, and
+# equality is by value, matching how Doctor::Check, SchemaHint, and
+# HydrationResult are built and compared.
+unless defined?(Data) && Data.respond_to?(:define)
+  class Data
+    class << self
+      def define(*members, &block)
+        members = members.map(&:to_sym)
+        raise ArgumentError, "duplicate member" if members.uniq.length != members.length
+
+        subclass = ::Class.new(self)
+        subclass.instance_variable_set(:@members, members.freeze)
+        members.each do |name|
+          subclass.define_method(name) { instance_variable_get(:"@#{name}") }
+        end
+        subclass.class_eval(&block) if block
+        subclass
+      end
+
+      def members
+        @members || []
+      end
+    end
+
+    def initialize(*args, **kwargs)
+      members = self.class.members
+      if kwargs.empty?
+        unless args.length == members.length
+          raise ArgumentError, "wrong number of arguments (given #{args.length}, expected #{members.length})"
+        end
+
+        members.each_with_index { |m, i| instance_variable_set(:"@#{m}", args[i]) }
+      else
+        unless args.empty?
+          raise ArgumentError, "wrong number of arguments (given #{args.length}, expected 0)"
+        end
+
+        extra = kwargs.keys - members
+        unless extra.empty?
+          raise ArgumentError, "unknown keyword#{extra.length > 1 ? 's' : ''}: #{extra.map(&:inspect).join(', ')}"
+        end
+
+        missing = members - kwargs.keys
+        unless missing.empty?
+          raise ArgumentError, "missing keyword#{missing.length > 1 ? 's' : ''}: #{missing.map(&:inspect).join(', ')}"
+        end
+
+        members.each { |m| instance_variable_set(:"@#{m}", kwargs[m]) }
+      end
+      freeze
+    end
+
+    def members
+      self.class.members
+    end
+
+    def to_h(&block)
+      pairs = self.class.members.map { |m| [ m, instance_variable_get(:"@#{m}") ] }
+      return pairs.to_h unless block
+
+      pairs.each_with_object({}) do |(k, v), acc|
+        nk, nv = block.call(k, v)
+        acc[nk] = nv
+      end
+    end
+
+    def with(**kwargs)
+      extra = kwargs.keys - self.class.members
+      unless extra.empty?
+        raise ArgumentError, "unknown keyword#{extra.length > 1 ? 's' : ''}: #{extra.map(&:inspect).join(', ')}"
+      end
+
+      self.class.new(**to_h.merge(kwargs))
+    end
+
+    def deconstruct
+      self.class.members.map { |m| instance_variable_get(:"@#{m}") }
+    end
+
+    def deconstruct_keys(keys)
+      return to_h if keys.nil?
+
+      keys.each_with_object({}) do |k, acc|
+        acc[k] = instance_variable_get(:"@#{k}") if self.class.members.include?(k)
+      end
+    end
+
+    def ==(other)
+      other.class == self.class && other.to_h == to_h
+    end
+    alias_method :eql?, :==
+
+    def hash
+      [ self.class, to_h ].hash
+    end
+
+    def inspect
+      pairs = self.class.members.map { |m| "#{m}=#{instance_variable_get(:"@#{m}").inspect}" }
+      "#<data #{self.class.name}#{pairs.empty? ? '' : " #{pairs.join(', ')}"}>"
+    end
+    alias_method :to_s, :inspect
+  end
+end

data/lib/rails_ai_context/tools/query.rb

@@ -307,9 +307,16 @@ module RailsAiContext
           sql
         end
 
+        # The SET must run BEFORE the transaction opens: Rails materializes
+        # the lazy BEGIN ahead of the first in-block statement, and MySQL
+        # rejects changing transaction characteristics mid-transaction
+        # (error 1568). Without GLOBAL/SESSION scope the SET applies only to
+        # the next transaction, which the block below immediately consumes,
+        # so the read-only characteristic cannot leak onto the pooled
+        # connection. (#89)
         result = nil
+        conn.execute("SET TRANSACTION READ ONLY")
         conn.transaction do
-          conn.execute("SET TRANSACTION READ ONLY")
           result = conn.select_all(hinted_sql)
           raise ActiveRecord::Rollback
         end

data/lib/rails_ai_context/tools/search_code.rb

@@ -5,6 +5,10 @@ require "open3"
 module RailsAiContext
   module Tools
     class SearchCode < BaseTool
+      # Per-pattern Regexp timeout (a ReDoS guard) is Ruby 3.2+. Ruby 3.1 has
+      # no per-match timeout, so the pattern is built without one there.
+      REGEXP_TIMEOUT_SUPPORTED = Regexp.respond_to?(:timeout)
+
       tool_name "rails_search_code"
       description "Search the Rails codebase with smart modes. " \
         "Use match_type:\"trace\" to see where a method is defined, who calls it, and what it calls - in one call. " \
@@ -102,7 +106,7 @@ module RailsAiContext
 
         # Validate regex syntax early
         begin
-          Regexp.new(search_pattern, timeout: 1)
+          build_regexp(search_pattern, timeout: 1)
         rescue RegexpError => e
           return text_response("Invalid regex pattern: #{e.message}")
         end
@@ -174,6 +178,17 @@ module RailsAiContext
         end
       end
 
+      # Build a Regexp, applying the ReDoS timeout only on runtimes that
+      # support it (Ruby 3.2+). Keeps a single construction path for both the
+      # syntax-validation probe and the actual Ruby-fallback search.
+      private_class_method def self.build_regexp(pattern, options = nil, timeout:)
+        if REGEXP_TIMEOUT_SUPPORTED
+          Regexp.new(pattern, options, timeout: timeout)
+        else
+          Regexp.new(pattern, options)
+        end
+      end
+
       private_class_method def self.ripgrep_available?
         return @rg_available unless @rg_available.nil?
         @rg_available = system("which rg > /dev/null 2>&1")
@@ -244,7 +259,7 @@ module RailsAiContext
       private_class_method def self.search_with_ruby(pattern, search_path, file_type, max_results, root, exclude_tests: false)
         results = []
         begin
-          regex = Regexp.new(pattern, Regexp::IGNORECASE, timeout: 2)
+          regex = build_regexp(pattern, Regexp::IGNORECASE, timeout: 2)
         rescue RegexpError => e
           return [ { file: "error", line_number: 0, content: "Invalid regex: #{e.message}" } ]
         end

data/lib/rails_ai_context/version.rb

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

data/lib/rails_ai_context.rb

@@ -2,10 +2,15 @@
 
 require "zeitwerk"
 
+# Provide Data.define on Ruby 3.1 (no-op on 3.2+) before any value object is
+# autoloaded. Defines a top-level constant, so it stays outside Zeitwerk.
+require_relative "rails_ai_context/polyfill/data"
+
 loader = Zeitwerk::Loader.for_gem(warn_on_extra_files: false)
 loader.inflector.inflect("devops_introspector" => "DevOpsIntrospector", "cli" => "CLI", "vfs" => "VFS")
 loader.ignore("#{__dir__}/generators")
 loader.ignore("#{__dir__}/rails-ai-context.rb")
+loader.ignore("#{__dir__}/rails_ai_context/polyfill")
 loader.setup
 
 module RailsAiContext

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-ai-context
 version: !ruby/object:Gem::Version
-  version: 5.12.0
+  version: 5.13.0
 platform: ruby
 authors:
 - crisnahine
@@ -35,7 +35,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -45,7 +45,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.0'
     - - "<"
       - !ruby/object:Gem::Version
         version: '9.0'
@@ -329,6 +329,7 @@ files:
 - lib/rails_ai_context/live_reload.rb
 - lib/rails_ai_context/mcp_config_generator.rb
 - lib/rails_ai_context/middleware.rb
+- lib/rails_ai_context/polyfill/data.rb
 - lib/rails_ai_context/resources.rb
 - lib/rails_ai_context/safe_file.rb
 - lib/rails_ai_context/schema_hint.rb
@@ -423,7 +424,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.2.0
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="