rails-ai-context 4.2.3 → 4.3.1

data/lib/rails_ai_context/tools/get_turbo_map.rb

@@ -105,7 +105,11 @@ module RailsAiContext
 
       private_class_method def self.format_summary(model_broadcasts, rb_broadcasts, view_subscriptions, view_frames, warnings, filter_label: nil)
         total_broadcasts = model_broadcasts.size + rb_broadcasts.size
+        turbo_data = cached_context[:turbo]
+        turbo_stream_response_count = turbo_data.is_a?(Hash) && !turbo_data[:error] ? turbo_data[:turbo_stream_responses]&.size.to_i : 0
+
         lines = [ "# Turbo Map", "" ]
+        lines << "- **Turbo Stream responses:** #{turbo_stream_response_count} (controller `.turbo_stream.erb` templates)" if turbo_stream_response_count > 0
         lines << "- **Model broadcasts:** #{model_broadcasts.size} (via `broadcasts`, `broadcasts_to`, etc.)"
         lines << "- **Explicit broadcasts:** #{rb_broadcasts.size} (via `broadcast_*_to` calls in .rb files)"
         lines << "- **Stream subscriptions:** #{view_subscriptions.size} (`turbo_stream_from` in views)"
@@ -195,7 +199,9 @@ module RailsAiContext
           lines << ""
         end
 
-        if model_broadcasts.empty? && rb_broadcasts.empty? && view_subscriptions.empty? && view_frames.empty?
+        has_turbo_stream_responses = turbo_data.is_a?(Hash) && turbo_data[:turbo_stream_responses]&.any?
+
+        if model_broadcasts.empty? && rb_broadcasts.empty? && view_subscriptions.empty? && view_frames.empty? && !has_turbo_stream_responses
           if filter_label
             lines << "_No Turbo usage matching #{filter_label}. Try without filter to see all Turbo Streams and Frames._"
           else
@@ -211,6 +217,31 @@ module RailsAiContext
       private_class_method def self.format_full(model_broadcasts, rb_broadcasts, view_subscriptions, view_frames, warnings, filter_label: nil)
         lines = [ "# Turbo Map (Full Detail)", "" ]
 
+        # Turbo Drive Configuration & Stream Responses
+        turbo_data = cached_context[:turbo]
+        if turbo_data.is_a?(Hash) && !turbo_data[:error]
+          drive_parts = []
+          drive_parts << "morph: #{turbo_data[:morph_meta] ? 'yes' : 'no'}" unless turbo_data[:morph_meta].nil?
+          drive_parts << "permanent elements: #{turbo_data[:permanent_elements].size}" if turbo_data[:permanent_elements]&.any?
+          if turbo_data[:turbo_drive_settings].is_a?(Hash) && turbo_data[:turbo_drive_settings].any?
+            turbo_data[:turbo_drive_settings].each { |k, v| drive_parts << "#{k}: #{v}" }
+          end
+          if drive_parts.any?
+            lines << "## Turbo Drive Configuration"
+            drive_parts.each { |p| lines << "- #{p}" }
+            lines << ""
+          end
+
+          # Turbo Stream responses
+          if turbo_data[:turbo_stream_responses]&.any?
+            lines << "## Turbo Stream Responses (#{turbo_data[:turbo_stream_responses].size})"
+            turbo_data[:turbo_stream_responses].each do |resp|
+              lines << "- `#{resp}`"
+            end
+            lines << ""
+          end
+        end
+
         # Model broadcasts with full context
         if model_broadcasts.any?
           lines << "## Model Broadcasts (#{model_broadcasts.size})"
@@ -289,7 +320,9 @@ module RailsAiContext
           lines << ""
         end
 
-        if model_broadcasts.empty? && rb_broadcasts.empty? && view_subscriptions.empty? && view_frames.empty?
+        has_turbo_stream_responses = turbo_data.is_a?(Hash) && turbo_data[:turbo_stream_responses]&.any?
+
+        if model_broadcasts.empty? && rb_broadcasts.empty? && view_subscriptions.empty? && view_frames.empty? && !has_turbo_stream_responses
           if filter_label
             lines << "_No Turbo usage matching #{filter_label}. Try without filter to see all Turbo Streams and Frames._"
           else

data/lib/rails_ai_context/tools/get_view.rb

@@ -87,7 +87,9 @@ 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 << "- #{name} (#{meta[:lines]} lines)#{parts}#{stim}"
+              comps = meta[:components]&.any? ? " components: #{meta[:components].join(', ')}" : ""
+              phlex_tag = meta[:phlex] ? " [phlex]" : ""
+              lines << "- #{name} (#{meta[:lines]} lines#{phlex_tag})#{parts}#{comps}#{stim}"
             end
             ctrl_partials.sort.each do |name, meta|
               lines << "- #{name} (#{meta[:lines]} lines)"
@@ -118,13 +120,29 @@ module RailsAiContext
             lines << "## #{ctrl}/" unless controller && all_dirs.size == 1
             ctrl_templates.sort.each do |name, meta|
               detail_parts = []
-              detail_parts << "renders: #{meta[:partials].join(', ')}" if meta[:partials]&.any?
-              detail_parts << "stimulus: #{meta[:stimulus].join(', ')}" if meta[:stimulus]&.any?
               extra = extract_view_metadata(name)
-              detail_parts << "ivars: #{extra[:ivars].join(', ')}" if extra[:ivars]&.any?
-              detail_parts << "turbo: #{extra[:turbo].join(', ')}" if extra[:turbo]&.any?
+
+              if meta[:phlex]
+                # Phlex views: show components, helpers, stimulus, ivars
+                # Prefer introspector-level data, fall back to extract_view_metadata
+                components = meta[:components]&.any? ? meta[:components] : extra[:components]
+                helpers = meta[:helpers]&.any? ? meta[:helpers] : extra[:helpers]
+                detail_parts << "ivars: #{extra[:ivars].join(', ')}" if extra[:ivars]&.any?
+                detail_parts << "components: #{components.join(', ')}" if components&.any?
+                detail_parts << "helpers: #{helpers.join(', ')}" if helpers&.any?
+                detail_parts << "stimulus: #{meta[:stimulus].join(', ')}" if meta[:stimulus]&.any?
+                detail_parts << "turbo: #{extra[:turbo].join(', ')}" if extra[:turbo]&.any?
+              else
+                # ERB/Haml/Slim views: existing behavior
+                detail_parts << "renders: #{meta[:partials].join(', ')}" if meta[:partials]&.any?
+                detail_parts << "stimulus: #{meta[:stimulus].join(', ')}" if meta[:stimulus]&.any?
+                detail_parts << "ivars: #{extra[:ivars].join(', ')}" if extra[:ivars]&.any?
+                detail_parts << "turbo: #{extra[:turbo].join(', ')}" if extra[:turbo]&.any?
+              end
+
+              phlex_tag = meta[:phlex] ? " [phlex]" : ""
               details = detail_parts.any? ? " — #{detail_parts.join(' | ')}" : ""
-              lines << "- **#{name}** (#{meta[:lines]} lines)#{details}"
+              lines << "- **#{name}** (#{meta[:lines]} lines#{phlex_tag})#{details}"
             end
             ctrl_partials.sort.each do |name, meta|
               fields = meta[:fields]&.any? ? " fields: #{meta[:fields].join(', ')}" : ""
@@ -269,7 +287,7 @@ module RailsAiContext
       # Extract instance variables and Turbo wiring from a view template
       private_class_method def self.extract_view_metadata(relative_path)
         content = read_view_content(relative_path)
-        return { ivars: [], turbo: [] } if content.nil? || content.include?("(file not found)")
+        return { ivars: [], turbo: [], components: [], helpers: [] } if content.nil? || content.include?("(file not found)")
 
         # Instance variables used in template
         ivars = content.scan(/@(\w+)/).flatten.uniq.reject { |v| %w[output_buffer virtual_path _request].include?(v) }.sort
@@ -284,9 +302,47 @@ module RailsAiContext
           turbo << "stream:#{val}" unless val.start_with?('"') || val.start_with?("'") || turbo.any? { |t| t.include?(val) }
         end
 
-        { ivars: ivars, turbo: turbo.uniq }
+        result = { ivars: ivars, turbo: turbo.uniq }
+
+        # For Phlex views (.rb), extract component renders and helper calls
+        if relative_path.end_with?(".rb") && phlex_view_content?(content)
+          result[:components] = extract_phlex_components(content)
+          result[:helpers] = extract_phlex_helpers(content)
+        end
+
+        result
       rescue
-        { ivars: [], turbo: [] }
+        { ivars: [], turbo: [], components: [], helpers: [] }
+      end
+
+      # Detect if content is a Phlex view class
+      private_class_method def self.phlex_view_content?(content)
+        content.match?(/class\s+\S+\s*<\s*\S+/) && content.match?(/def\s+view_template\b/)
+      end
+
+      # Extract component render calls from Phlex Ruby DSL
+      private_class_method def self.extract_phlex_components(content)
+        components = Set.new
+        content.scan(/render[\s(]+([A-Z]\w+(?:::\w+)*)\.new/).each do |match|
+          components << match[0]
+        end
+        components.to_a.sort
+      end
+
+      # Extract helper method calls from Phlex views
+      PHLEX_HELPERS = %w[
+        link_to image_tag content_for button_to form_with form_for
+        content_tag tag number_to_currency number_to_human
+        time_ago_in_words distance_of_time_in_words
+        truncate pluralize raw sanitize dom_id
+      ].freeze
+
+      private_class_method def self.extract_phlex_helpers(content)
+        helpers = []
+        PHLEX_HELPERS.each do |method|
+          helpers << method if content.match?(/\b#{method}\b/)
+        end
+        helpers
       end
 
       # Scan templates that render a partial to extract locals keys

data/lib/rails_ai_context/tools/migration_advisor.rb

@@ -130,6 +130,8 @@ module RailsAiContext
           opts = options ? ", #{options}" : ""
           class_name = migration_class_name("add", table, column)
 
+          lines << "**Run:** `bin/rails generate migration #{class_name} #{column}:#{type}`"
+          lines << ""
           lines << "```ruby"
           lines << "# rails generate migration #{class_name} #{column}:#{type}"
           lines << "class #{class_name} < ActiveRecord::Migration[#{rails_version}]"
@@ -165,6 +167,8 @@ module RailsAiContext
           # Check if column is referenced
           col_type = find_column_type(table, column, schema) || type || "string"
 
+          lines << "**Run:** `bin/rails generate migration #{class_name} #{column}:#{col_type}`"
+          lines << ""
           lines << "**Warning:** `remove_column` is irreversible without specifying the column type."
           lines << ""
           lines << "```ruby"