rails-ai-context 4.5.2 → 4.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d53c26bdd73eac84be7e9a49ea08c2a50dd0230f969cb29ae240cf2d08b9e4c3
-  data.tar.gz: a65a518fab277ff246401cffb32701fb36048f65034b068e20f4b3fe76ddc4eb
+  metadata.gz: 7a6993d2929f87b6f9c29dffd5f16124a1d3ccb15ac3f8ba8f100364bf0feb06
+  data.tar.gz: '08c5fe8f51d126c1f7cc52ca9cc8ca0dc2edf5d1dba5f569b8ec5afaeecbb0b4'
 SHA512:
-  metadata.gz: 0504e0e68f018d325189ccdfd153b7097d1204c906241a9a0dfea41586272b0c2ae296f81704c4e6c4ac9bb414aaab83c5de35e226b7aabff52b984d1bec451e
-  data.tar.gz: f2c8a762bd3520edd09e719337043be0a9b7158d0099076118226012cea980ca1877c48459d78810c8e9a1c9d91bf526aad4a2c334521e19c5ac7b715bc44b1e
+  metadata.gz: 94ec45e9fd089ba075c1b1c2457bfbf6fc9f2fc9c6da31b2ccafc7e51e64ca9579a390e25b56816a0c1bf4023f3ecbf092124082d53681029dcea3a5446065f5
+  data.tar.gz: fbab834f047ab3ae3392501964b6e0de1c090af063872473df3e3b074a161536c306cf4afd51daf8e44df8f127ddbb94fd885b15f21610f0884e27159ae118e3

data/CHANGELOG.md

@@ -5,6 +5,22 @@ 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.6.0] — 2026-04-04
+
+### Added
+- **Integration test suite** — 3 purpose-built Rails 8 apps exercising every gem feature end-to-end:
+  - `full_app` — comprehensive app (38 gems, 14 models, 15 controllers, 26 views, 5 jobs, 3 mailers, multi-database, ViewComponent, Stimulus, STI, polymorphic, AASM, PaperTrail, FriendlyId, encrypted attributes, CurrentAttributes, Flipper feature flags, Sentry monitoring, Pundit auth, Ransack search, Dry-rb, acts_as_tenant, Docker, Kamal, GitHub Actions CI, RSpec + FactoryBot)
+  - `api_app` — API-only app (Products/Orders/OrderItems, namespaced API v1 routes, CLI tool_mode)
+  - `minimal_app` — bare minimum app (single model, graceful degradation testing)
+- **Master test runner** (`test_apps/run_all_tests.sh`) — validates Doctor, context generation, all 33 introspectors, all 39 MCP tools, Rake tasks, MCP server startup, and app-specific pattern detection across all 3 apps (222 tests)
+- All 3 test apps achieve **100/100 AI Readiness Score**
+
+### Fixed
+- **Standalone CLI `full_gem_path` crash** — `Gem.loaded_specs.delete_if { |_, spec| !spec.default_gem? }` in the exe file cleared gem specs needed by MCP SDK at runtime (`json-schema` gem's `full_gem_path` returned nil). Added `!ENV["BUNDLE_BIN_PATH"]` guard so cleanup only runs in true standalone mode, not under `bundle exec`. This bug affected ALL `rails-ai-context tool` commands in standalone mode.
+
+### Changed
+- Test count: 1621 RSpec examples + 222 integration tests across 3 apps
+
 ## [4.5.2] — 2026-04-04
 
 ### Added
@@ -30,8 +46,24 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **CLI error messages** — Clean error messages for all CLI error paths
 - **Rake/init parity** — `rake ai:context` and `init` command now match generator output
 
+### Refactored
+- **SLOP audit: ~640 lines removed** — comprehensive audit eliminating superfluous abstractions, dead code, and duplicated patterns
+- **CompactSerializerHelper** — extracted shared logic from ClaudeSerializer and OpencodeSerializer, eliminating ~75% duplication
+- **StackOverviewHelper consolidation** — moved `project_root`, `detect_service_files`, `detect_job_files`, `detect_before_actions`, `scope_names`, `notable_gems_list`, `arch_labels_hash`, `pattern_labels_hash`, `write_rule_files` into shared module, replacing 30+ duplicate copies across 6 serializers
+- **Atomic file writes** — `write_rule_files` uses temp file + rename for crash-safe context file generation
+- **ConventionDetector → ConventionIntrospector** — renamed for naming consistency with all 33 other introspectors
+- **MarkdownEscape inlined** — single-use module inlined into MarkdownSerializer as private method
+- **RulesSerializer deleted** — dead code never called by ContextFileSerializer
+- **BaseTool cleanup** — removed dead `auto_compress`, `app_size`, `session_queried?` methods
+- **IntrospectionError deleted** — exception class never raised anywhere
+- **mobile_paths config removed** — config option never read by any introspector, tool, or serializer
+- **server_version** — changed from attr_accessor to method delegating to `VERSION` constant
+- **Configuration constants** — extracted `DEFAULT_EXCLUDED_FILTERS`, `DEFAULT_EXCLUDED_MIDDLEWARE`, `DEFAULT_EXCLUDED_CONCERNS` as frozen constants
+- **Detail spec consolidation** — merged 5 detail spec files into their base spec counterparts
+- **Orphaned spec cleanup** — removed `gem_introspector_spec.rb` duplicate (canonical spec already exists under introspectors/)
+
 ### Changed
-- Test count: 1658 examples (76 new tests for Phase 2 features)
+- Test count: 1621 examples (consolidated from 1658 — no coverage lost, only duplicate/orphaned specs removed)
 
 ## [4.4.0] — 2026-04-03
 

data/CLAUDE.md

@@ -11,12 +11,11 @@ structure to AI assistants via the Model Context Protocol (MCP).
 - `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/` — 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/serializers/` — Output formatters (claude, claude_rules, opencode, opencode_rules, cursor_rules, copilot, copilot_instructions, markdown, JSON, context_file_serializer, compact_serializer_helper, 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)
 - `lib/rails_ai_context/server.rb` — MCP server configuration (stdio + HTTP transports)
 - `lib/rails_ai_context/middleware.rb` — Rack middleware for auto-mounting MCP HTTP endpoint
 - `lib/rails_ai_context/safe_file.rb` — Safe file reading with size limits and error handling
-- `lib/rails_ai_context/markdown_escape.rb` — Escapes markdown special characters in dynamic content
 - `lib/rails_ai_context/fingerprinter.rb` — SHA256 file fingerprinting for cache invalidation
 - `lib/rails_ai_context/doctor.rb` — Diagnostic checks and AI readiness scoring
 - `lib/rails_ai_context/live_reload.rb` — MCP live reload: watches files, invalidates caches, notifies AI clients
@@ -68,7 +67,7 @@ structure to AI assistants via the Model Context Protocol (MCP).
 ## Testing
 
 ```bash
-bundle exec rspec           # Run specs (1658 examples)
+bundle exec rspec           # Run specs (1621 examples)
 bundle exec rubocop         # Lint
 ```
 

data/README.md

@@ -19,7 +19,7 @@
 [![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-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)
-[![Tests](https://img.shields.io/badge/Tests-1529%20passing-brightgreen)](https://github.com/crisnahine/rails-ai-context/actions)
+[![Tests](https://img.shields.io/badge/Tests-1621%20passing-brightgreen)](https://github.com/crisnahine/rails-ai-context/actions)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](LICENSE)
 
 </div>
@@ -444,7 +444,7 @@ end
 ## About
 
 Built by a Rails developer with 10+ years of production experience.<br>
-1529 tests. 39 tools. 33 introspectors. Standalone or in-Gemfile.<br>
+1621 tests. 39 tools. 33 introspectors. Standalone or in-Gemfile.<br>
 MIT licensed. [Contributions welcome.](CONTRIBUTING.md)
 
 <br>

data/SECURITY.md

@@ -4,6 +4,9 @@
 
 | Version | Supported          |
 |---------|--------------------|
+| 4.6.x   | :white_check_mark: |
+| 4.5.x   | :white_check_mark: |
+| 4.4.x   | :white_check_mark: |
 | 4.3.x   | :white_check_mark: |
 | 4.2.x   | :white_check_mark: (4.2.1 includes security hardening) |
 | 4.1.x   | :white_check_mark: |

data/exe/rails-ai-context

@@ -6,7 +6,11 @@
 # may need different versions of those deps (e.g., bigdecimal 4.1.0 activated here
 # but Gemfile.lock wants 4.0.1). Clear non-default gem activations so Bundler can
 # resolve cleanly. The $LOAD_PATH entries remain (Bundler.setup replaces them anyway).
-if defined?(Gem) && Gem.respond_to?(:loaded_specs)
+#
+# Skip this when running under `bundle exec` — Bundler has already resolved gems
+# correctly, and clearing specs breaks gems that look up their own path at runtime
+# (e.g., MCP SDK reads Gem.loaded_specs["json-schema"].full_gem_path).
+if defined?(Gem) && Gem.respond_to?(:loaded_specs) && !ENV["BUNDLE_BIN_PATH"]
   Gem.loaded_specs.delete_if { |_, spec| !spec.default_gem? }
 end
 

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

@@ -257,7 +257,6 @@ module RailsAiContext
             # ── Frontend Framework Detection ─────────────────────────────────
             # Auto-detected from package.json, config/vite.json, etc. Override only if needed.
             # config.frontend_paths = ["app/frontend", "../web-client"]
-            # config.mobile_paths = ["../mobile-app"]
         SECTION
       }.freeze
 

data/lib/rails_ai_context/configuration.rb

@@ -13,11 +13,11 @@ module RailsAiContext
     # All YAML-supported keys (explicit allowlist for safety)
     YAML_KEYS = %i[
       ai_tools tool_mode preset context_mode generate_root_files claude_max_lines
-      server_name server_version cache_ttl max_tool_response_chars
+      server_name cache_ttl max_tool_response_chars
       live_reload live_reload_debounce auto_mount http_path http_bind http_port
       output_dir skip_tools excluded_models excluded_controllers
       excluded_route_prefixes excluded_filters excluded_middleware excluded_paths
-      sensitive_patterns search_extensions concern_paths frontend_paths mobile_paths
+      sensitive_patterns search_extensions concern_paths frontend_paths
       max_file_size max_test_file_size max_schema_file_size max_view_total_size
       max_view_file_size max_search_results max_validate_files
       query_timeout query_row_limit query_redacted_columns allow_query_in_production
@@ -79,7 +79,11 @@ module RailsAiContext
     }.freeze
 
     # MCP server settings
-    attr_accessor :server_name, :server_version
+    attr_accessor :server_name
+
+    def server_version
+      RailsAiContext::VERSION
+    end
 
     # Which introspectors to run
     attr_accessor :introspectors
@@ -151,6 +155,33 @@ module RailsAiContext
     # Tool invocation mode: :mcp (MCP primary + CLI fallback) or :cli (CLI only)
     attr_accessor :tool_mode
 
+    DEFAULT_EXCLUDED_FILTERS = %w[
+      verify_authenticity_token verify_same_origin_request
+      turbo_tracking_request_id handle_unverified_request
+      mark_for_same_origin_verification
+    ].freeze
+
+    DEFAULT_EXCLUDED_MIDDLEWARE = %w[
+      Rack::Sendfile ActionDispatch::Static ActionDispatch::Executor
+      ActionDispatch::ServerTiming Rack::Runtime
+      ActionDispatch::RequestId ActionDispatch::RemoteIp
+      Rails::Rack::Logger ActionDispatch::ShowExceptions
+      ActionDispatch::DebugExceptions ActionDispatch::Callbacks
+      ActionDispatch::Cookies ActionDispatch::Session::CookieStore
+      ActionDispatch::Flash ActionDispatch::ContentSecurityPolicy::Middleware
+      ActionDispatch::PermissionsPolicy::Middleware ActionDispatch::ActionableExceptions
+      Rack::Head Rack::ConditionalGet Rack::ETag Rack::TempfileReaper
+      ActiveRecord::Migration::CheckPending ActionDispatch::HostAuthorization
+      Rack::MethodOverride ActionDispatch::Session::AbstractSecureStore
+    ].freeze
+
+    DEFAULT_EXCLUDED_CONCERNS = [
+      /::Generated/,
+      /\A(ActiveRecord|ActiveModel|ActiveSupport|ActionText|ActionMailbox|ActiveStorage)/,
+      /\A(ActionDispatch|ActionController|ActionView|AbstractController)/,
+      /\A(Devise::Models|Devise::Orm|Bullet::|Turbo::|GlobalID::|Rolify::)/
+    ].freeze
+
     # Filtering — customize what's hidden from AI output
     attr_accessor :excluded_controllers   # Controller classes hidden from listings (e.g. DeviseController)
     attr_accessor :excluded_route_prefixes # Route controller prefixes hidden with app_only (e.g. action_mailbox/)
@@ -164,7 +195,6 @@ module RailsAiContext
 
     # Frontend framework detection (optional overrides — auto-detected if nil)
     attr_accessor :frontend_paths         # User-declared frontend dirs (e.g. ["app/frontend", "../web-client"])
-    attr_accessor :mobile_paths           # User-declared mobile dirs (e.g. ["../mobile-app"])
 
     # Database query tool settings (rails_query)
     attr_accessor :query_timeout              # Statement timeout in seconds (default: 5)
@@ -177,7 +207,6 @@ module RailsAiContext
 
     def initialize
       @server_name         = "rails-ai-context"
-      @server_version      = RailsAiContext::VERSION
       @introspectors       = PRESETS[:full].dup
       @excluded_paths      = %w[node_modules tmp log vendor .git doc docs]
       @sensitive_patterns  = %w[
@@ -211,30 +240,9 @@ module RailsAiContext
       @max_validate_files       = 50
       @excluded_controllers     = %w[DeviseController Devise::OmniauthCallbacksController]
       @excluded_route_prefixes  = %w[action_mailbox/ active_storage/ rails/ conductor/ devise/ turbo/]
-      @excluded_concerns        = [
-        /::Generated/,
-        /\A(ActiveRecord|ActiveModel|ActiveSupport|ActionText|ActionMailbox|ActiveStorage)/,
-        /\A(ActionDispatch|ActionController|ActionView|AbstractController)/,
-        /\A(Devise::Models|Devise::Orm|Bullet::|Turbo::|GlobalID::|Rolify::)/
-      ]
-      @excluded_filters         = %w[
-        verify_authenticity_token verify_same_origin_request
-        turbo_tracking_request_id handle_unverified_request
-        mark_for_same_origin_verification
-      ]
-      @excluded_middleware      = %w[
-        Rack::Sendfile ActionDispatch::Static ActionDispatch::Executor
-        ActionDispatch::ServerTiming Rack::Runtime
-        ActionDispatch::RequestId ActionDispatch::RemoteIp
-        Rails::Rack::Logger ActionDispatch::ShowExceptions
-        ActionDispatch::DebugExceptions ActionDispatch::Callbacks
-        ActionDispatch::Cookies ActionDispatch::Session::CookieStore
-        ActionDispatch::Flash ActionDispatch::ContentSecurityPolicy::Middleware
-        ActionDispatch::PermissionsPolicy::Middleware ActionDispatch::ActionableExceptions
-        Rack::Head Rack::ConditionalGet Rack::ETag Rack::TempfileReaper
-        ActiveRecord::Migration::CheckPending ActionDispatch::HostAuthorization
-        Rack::MethodOverride ActionDispatch::Session::AbstractSecureStore
-      ]
+      @excluded_concerns        = DEFAULT_EXCLUDED_CONCERNS.dup
+      @excluded_filters         = DEFAULT_EXCLUDED_FILTERS.dup
+      @excluded_middleware      = DEFAULT_EXCLUDED_MIDDLEWARE.dup
       @custom_tools             = []
       @skip_tools               = []
       @ai_tools                 = nil
@@ -242,7 +250,6 @@ module RailsAiContext
       @search_extensions        = %w[rb js erb yml yaml json ts tsx vue svelte haml slim]
       @concern_paths            = %w[app/models/concerns app/controllers/concerns]
       @frontend_paths           = nil
-      @mobile_paths             = nil
       @query_timeout            = 5
       @query_row_limit          = 100
       @query_redacted_columns   = %w[

data/lib/rails_ai_context/introspector.rb

@@ -63,7 +63,7 @@ module RailsAiContext
       when :routes      then Introspectors::RouteIntrospector.new(app)
       when :jobs        then Introspectors::JobIntrospector.new(app)
       when :gems        then Introspectors::GemIntrospector.new(app)
-      when :conventions then Introspectors::ConventionDetector.new(app)
+      when :conventions then Introspectors::ConventionIntrospector.new(app)
       when :stimulus       then Introspectors::StimulusIntrospector.new(app)
       when :database_stats then Introspectors::DatabaseStatsIntrospector.new(app)
       when :controllers    then Introspectors::ControllerIntrospector.new(app)

data/lib/rails_ai_context/introspectors/{convention_detector.rb → convention_introspector.rb}

@@ -4,7 +4,7 @@ module RailsAiContext
   module Introspectors
     # Detects high-level Rails conventions and patterns in use,
     # giving AI assistants critical context about the app's architecture.
-    class ConventionDetector
+    class ConventionIntrospector
       attr_reader :app
 
       def initialize(app)

data/lib/rails_ai_context/serializers/claude_rules_serializer.rb

@@ -19,34 +19,18 @@ module RailsAiContext
       # @return [Hash] { written: [paths], skipped: [paths] }
       def call(output_dir)
         rules_dir = File.join(output_dir, ".claude", "rules")
-        FileUtils.mkdir_p(rules_dir)
-
-        written = []
-        skipped = []
 
         files = {
-          "rails-context.md" => render_context_overview,
-          "rails-schema.md" => render_schema_reference,
-          "rails-models.md" => render_models_reference,
-          "rails-ui-patterns.md" => render_ui_patterns_reference,
-          "rails-mcp-tools.md" => render_mcp_tools_reference,
-          "rails-components.md" => render_components_reference,
-          "rails-accessibility.md" => render_accessibility_reference
+          File.join(rules_dir, "rails-context.md") => render_context_overview,
+          File.join(rules_dir, "rails-schema.md") => render_schema_reference,
+          File.join(rules_dir, "rails-models.md") => render_models_reference,
+          File.join(rules_dir, "rails-ui-patterns.md") => render_ui_patterns_reference,
+          File.join(rules_dir, "rails-mcp-tools.md") => render_mcp_tools_reference,
+          File.join(rules_dir, "rails-components.md") => render_components_reference,
+          File.join(rules_dir, "rails-accessibility.md") => render_accessibility_reference
         }
 
-        files.each do |filename, content|
-          next unless content
-
-          filepath = File.join(rules_dir, filename)
-          if File.exist?(filepath) && File.read(filepath) == content
-            skipped << filepath
-          else
-            File.write(filepath, content)
-            written << filepath
-          end
-        end
-
-        { written: written, skipped: skipped }
+        write_rule_files(files)
       end
 
       private
@@ -74,17 +58,8 @@ module RailsAiContext
         lines.concat(full_preset_stack_lines)
 
         # ApplicationController before_actions — apply to all controllers
-        begin
-          root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-          app_ctrl_file = File.join(root, "app", "controllers", "application_controller.rb")
-          if File.exist?(app_ctrl_file)
-            source = File.read(app_ctrl_file)
-            before_actions = source.scan(/before_action\s+:([\w!?]+)/).flatten
-            if before_actions.any?
-              lines << "" << "**Global before_actions:** #{before_actions.join(', ')}"
-            end
-          end
-        rescue => e; $stderr.puts "[rails-ai-context] Serializer section skipped: #{e.message}"; end
+        before_actions = detect_before_actions
+        lines << "" << "**Global before_actions:** #{before_actions.join(', ')}" if before_actions.any?
 
         lines << ""
         lines << "ALWAYS use MCP tools for context — do NOT read reference files directly."
@@ -206,7 +181,7 @@ module RailsAiContext
 
           # Include scopes so agents know available query methods
           scopes = data[:scopes] || []
-          scope_names = scopes.map { |s| s.is_a?(Hash) ? s[:name] : s }
+          scope_names = scope_names(scopes)
           lines << "  scopes: #{scope_names.join(', ')}" if scopes.any?
 
           # Instance methods — filter Devise/framework internals that add noise
@@ -248,8 +223,7 @@ module RailsAiContext
 
         # Shared partials — so agents reuse them instead of recreating
         begin
-          root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-          shared_dir = File.join(root, "app", "views", "shared")
+          shared_dir = File.join(project_root, "app", "views", "shared")
           if Dir.exist?(shared_dir)
             partials = Dir.glob(File.join(shared_dir, "_*.html.erb"))
               .map { |f| File.basename(f) }
@@ -263,8 +237,7 @@ module RailsAiContext
 
         # Helpers — so agents use existing helpers instead of creating new ones
         begin
-          root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-          helper_file = File.join(root, "app", "helpers", "application_helper.rb")
+          helper_file = File.join(project_root, "app", "helpers", "application_helper.rb")
           if File.exist?(helper_file)
             helper_methods = File.read(helper_file).scan(/def\s+(\w+)/).flatten
             if helper_methods.any?

data/lib/rails_ai_context/serializers/claude_serializer.rb

@@ -10,6 +10,7 @@ module RailsAiContext
       include StackOverviewHelper
       include DesignSystemHelper
       include ToolGuideHelper
+      include CompactSerializerHelper
 
       attr_reader :context
 
@@ -38,118 +39,9 @@ module RailsAiContext
         lines.concat(render_commands)
         lines.concat(render_warnings)
         lines.concat(render_footer)
-        lines.concat(render_mcp_guide_compact)
+        lines.concat(render_tools_guide_compact)
 
-        # Enforce max lines
-        max = RailsAiContext.configuration.claude_max_lines
-        if lines.size > max
-          lines = lines.first(max - 2)
-          lines << ""
-          lines << "_Context trimmed. Use MCP tools for full details._"
-        end
-
-        lines.join("\n")
-      end
-
-      def render_header
-        [
-          "# #{context[:app_name]} — AI Context",
-          "",
-          "> Rails #{context[:rails_version]} | Ruby #{context[:ruby_version]}",
-          "> Generated by rails-ai-context v#{RailsAiContext::VERSION}",
-          ""
-        ]
-      end
-
-      def render_stack_overview
-        lines = [ "## Stack" ]
-
-        schema = context[:schema]
-        if schema && !schema[:error]
-          lines << "- Database: #{schema[:adapter]} — #{schema[:total_tables]} tables"
-        end
-
-        models = context[:models]
-        if models.is_a?(Hash) && !models[:error]
-          lines << "- Models: #{models.size}"
-        end
-
-        routes = context[:routes]
-        if routes && !routes[:error]
-          # Count only app controllers (exclude internal Rails, Devise, Turbo routes)
-          internal = %w[action_mailbox/ active_storage/ rails/ conductor/ devise/ turbo/]
-          app_ctrls = (routes[:by_controller] || {}).keys.reject { |k| internal.any? { |p| k.downcase.start_with?(p) } }
-          lines << "- Routes: #{routes[:total_routes]} across #{app_ctrls.size} controllers"
-        end
-
-        jobs = context[:jobs]
-        if jobs.is_a?(Hash) && !jobs[:error]
-          job_count = jobs[:jobs]&.size || 0
-          mailer_count = jobs[:mailers]&.size || 0
-          channel_count = jobs[:channels]&.size || 0
-          parts = []
-          parts << "#{job_count} jobs" if job_count > 0
-          parts << "#{mailer_count} mailers" if mailer_count > 0
-          parts << "#{channel_count} channels" if channel_count > 0
-          lines << "- Async: #{parts.join(', ')}" if parts.any?
-        end
-
-        migrations = context[:migrations]
-        if migrations.is_a?(Hash) && !migrations[:error]
-          pending = migrations[:pending]
-          lines << "- Migrations: #{migrations[:total]} total, #{pending&.size || 0} pending"
-        end
-
-        lines.concat(full_preset_stack_lines)
-
-        lines << ""
-        lines
-      end
-
-      def render_key_models
-        models = context[:models]
-        return [] unless models.is_a?(Hash) && !models[:error] && models.any?
-
-        max_show = 15
-        lines = [ "## Key models (#{models.size} total)" ]
-        models.keys.sort.first(max_show).each do |name|
-          data = models[name]
-          assoc_count = (data[:associations] || []).size
-          val_count = (data[:validations] || []).size
-          top_assocs = (data[:associations] || []).map { |a| "#{a[:type]} :#{a[:name]}" }.join(", ")
-          line = "- **#{name}**"
-          line += " (#{assoc_count}a, #{val_count}v)" if assoc_count > 0 || val_count > 0
-          line += " — #{top_assocs}" if top_assocs && !top_assocs.empty?
-          lines << line
-          scopes = (data[:scopes] || [])
-          constants = (data[:constants] || [])
-          if scopes.any? || constants.any?
-            extras = []
-            scope_names = scopes.map { |s| s.is_a?(Hash) ? s[:name] : s }
-            extras << "scopes: #{scope_names.join(', ')}" if scopes.any?
-            constants.each { |c| extras << "#{c[:name]}: #{c[:values].join(', ')}" }
-            lines << "  #{extras.join(' | ')}"
-          end
-        end
-        lines << "- _...#{models.size - max_show} more (use `rails_get_model_details` tool)_" if models.size > max_show
-        lines << ""
-        lines
-      end
-
-      def render_notable_gems
-        gems = context[:gems]
-        return [] unless gems.is_a?(Hash) && !gems[:error]
-        notable = gems[:notable_gems] || gems[:notable] || gems[:detected] || []
-        return [] if notable.empty?
-
-        lines = [ "## Gems" ]
-        grouped = notable.group_by { |g| g[:category]&.to_s || "other" }
-        grouped.each do |category, gem_list|
-          names = gem_list.map { |g| g[:name] }.join(", ")
-          lines << "- **#{category}**: #{names}"
-        end
-        lines << ""
-        lines
+        enforce_max_lines(lines)
       end
 
       def render_architecture
@@ -160,94 +52,35 @@ module RailsAiContext
         patterns = conv[:patterns] || []
         return [] if arch.empty? && patterns.empty?
 
-        arch_labels = RailsAiContext::Tools::GetConventions::ARCH_LABELS rescue {}
-        pattern_labels = RailsAiContext::Tools::GetConventions::PATTERN_LABELS rescue {}
+        arch_labels = arch_labels_hash
+        pattern_labels = pattern_labels_hash
 
         lines = [ "## Architecture" ]
         arch.each { |p| lines << "- #{arch_labels[p] || p}" }
         patterns.first(8).each { |p| lines << "- #{pattern_labels[p] || p}" }
 
-        # List service objects and jobs from conventions directory_structure
         dir_struct = conv[:directory_structure] || {}
 
         if dir_struct["app/services"]
-          begin
-            root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-            service_files = Dir.glob(File.join(root, "app", "services", "*.rb"))
-              .map { |f| File.basename(f, ".rb").camelize }
-              .reject { |s| s == "ApplicationService" }
-            lines << "" << "**Services:** #{service_files.join(', ')}" if service_files.any?
-          rescue => e; $stderr.puts "[rails-ai-context] Serializer section skipped: #{e.message}"; end
+          services = detect_service_files
+          lines << "" << "**Services:** #{services.join(', ')}" if services.any?
         end
 
         if dir_struct["app/jobs"]
-          begin
-            root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-            job_files = Dir.glob(File.join(root, "app", "jobs", "*.rb"))
-              .map { |f| File.basename(f, ".rb").camelize }
-              .reject { |j| j == "ApplicationJob" }
-            lines << "**Jobs:** #{job_files.join(', ')}" if job_files.any?
-          rescue => e; $stderr.puts "[rails-ai-context] Serializer section skipped: #{e.message}"; end
+          jobs = detect_job_files
+          lines << "**Jobs:** #{jobs.join(', ')}" if jobs.any?
         end
 
         lines << ""
         lines
       end
 
-      def render_ui_patterns
-        render_design_system(context, max_lines: 30)
-      end
-
-      def render_mcp_guide
-        render_tools_guide
-      end
-
-      def render_mcp_guide_compact
-        render_tools_guide_compact
-      end
-
-      def render_conventions
-        conv = context[:conventions]
-        return [] unless conv.is_a?(Hash) && !conv[:error]
-
-        config_files = conv[:config_files] || []
-        return [] if config_files.empty?
-
-        lines = [ "## Key config files" ]
-        config_files.first(5).each { |f| lines << "- `#{f}`" }
-        lines << ""
-        lines
-      end
-
-      def render_commands
-        test_cmd = detect_test_command
-        [
-          "## Commands",
-          "- `bin/dev` — start dev server",
-          "- `#{test_cmd}` — run tests",
-          "- `rails db:migrate` — run pending migrations",
-          ""
-        ]
-      end
-
-      def render_warnings
-        warnings = context[:_warnings]
-        return [] if warnings.nil? || warnings.empty?
-
-        lines = [ "", "## Warnings", "" ]
-        warnings.each do |w|
-          lines << "- **#{w[:introspector]}** skipped: #{w[:error]}"
-        end
-        lines
-      end
-
       def render_footer
         test_cmd = detect_test_command
         lines = [ "## Rules" ]
         lines << "- Run `#{test_cmd}` after changes"
         lines << "- Do NOT re-read files to verify edits — trust your Edit, validate syntax only"
 
-        # App-specific conventions from introspection
         conv = context[:conventions]
         if conv.is_a?(Hash) && !conv[:error]
           arch = conv[:architecture] || []
@@ -258,24 +91,13 @@ module RailsAiContext
           lines << "- Use query objects for complex queries" if patterns.include?("query_objects")
         end
 
-        # Stimulus auto-register if detected
         stimulus = context[:stimulus]
         if stimulus.is_a?(Hash) && !stimulus[:error] && (stimulus[:controllers]&.any? || stimulus[:total_controllers]&.positive?)
           lines << "- Stimulus controllers auto-register — no manual import in controllers/index.js needed"
         end
 
-        # Global before_actions — critical for understanding controller flow
-        begin
-          root = defined?(Rails) ? Rails.root.to_s : Dir.pwd
-          app_ctrl_file = File.join(root, "app", "controllers", "application_controller.rb")
-          if File.exist?(app_ctrl_file)
-            source = File.read(app_ctrl_file)
-            before_actions = source.scan(/before_action\s+:([\w!?]+)/).flatten
-            if before_actions.any?
-              lines << "- Global before_actions: #{before_actions.join(', ')}"
-            end
-          end
-        rescue => e; $stderr.puts "[rails-ai-context] Serializer section skipped: #{e.message}"; end
+        before_actions = detect_before_actions
+        lines << "- Global before_actions: #{before_actions.join(', ')}" if before_actions.any?
 
         lines << ""
         lines
@@ -284,8 +106,6 @@ module RailsAiContext
 
     # Internal: full-mode Claude serializer (wraps MarkdownSerializer with behavioral rules)
     class FullClaudeSerializer < MarkdownSerializer
-      include TestCommandDetection
-
       private
 
       def header