mysql_genius 0.4.1 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7858ff8adbac9ab758eee1f82b0302f67cc09357ad087f3a3102a6646307f9d
-  data.tar.gz: 72053d61ff33d5fbf986a7a60445e36ae821ec9c2368ef4c9a0e4768636cb6ec
+  metadata.gz: 48fbc36b280f63255eebb38587db28d24e38150b53faaff13efeddb810de8c70
+  data.tar.gz: a55a5115090bc136e99d854dc13c231670293a363f8d441951d98191c794d9e6
 SHA512:
-  metadata.gz: 8dd7d033566d1ba8112253768dd491a5083f2daf57f2eb96231f668f229bac16572d97735d86a873efe6d9b44b7fdc649a31647ba10470c877a5c353dd0ebf53
-  data.tar.gz: b25231d00d367295acc3c84e612bd4345fc3a954a48aa7b93a17c8fec50bb5804936e9605b1663971566542626c93f65fc49522900e36f667ff6d6073143d11c
+  metadata.gz: 9d9de9d97d05814ff37b0ed88ee1846656ba3f1ff323f094ff776a3abb4be358d12204810e9068d2f6e99daea88d007f5a406f27681b7c1574a38395b0889c7e
+  data.tar.gz: cb5c725008371288dff5e8a32a66851914a39de71f0b647ade81bde743e01f7fe5f7ae2664715da9602b7e024b5609c3df53f37c1817faa73e7a4d801a26f292

data/.rubocop.yml

@@ -17,6 +17,9 @@ RSpec/MessageSpies:
 RSpec/VerifiedDoubles:
   Enabled: false
 
+RSpec/VerifiedDoubleReference:
+  Enabled: false
+
 RSpec/ExampleLength:
   Max: 25
 

data/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Changelog
 
+## 0.5.1
+
+### Fixed
+- **ERB templates missing from `mysql_genius-core` gem package.** The gemspec glob only matched `*.rb` files, excluding the shared ERB templates. The dashboard crashed with `MissingTemplate` error when installed from RubyGems. Fixed by changing the glob to `*.{rb,erb}`.
+
+## 0.5.0
+
+### Changed
+- **ERB templates moved into `mysql_genius-core`.** All 11 view files (`dashboard.html.erb` and 10 partials) have been extracted from `app/views/mysql_genius/queries/` into `gems/mysql_genius-core/lib/mysql_genius/core/views/mysql_genius/queries/`. The index template is renamed to `dashboard.html.erb`. The engine registers `MysqlGenius::Core.views_path` before `:add_view_paths` so Rails finds templates in both view roots. Non-Rails adapters (Phase 2b `mysql_genius-desktop` sidecar) can register this same path with their own view loader and implement `path_for`/`render_partial` to reuse the templates.
+- **`QueriesController#index`** now sets `@framework_version_major` and `@framework_version_minor` instance variables (replacing direct `Rails::VERSION` references in the template) and explicitly renders `"mysql_genius/queries/dashboard"`.
+- **`SharedViewHelpers`** — new concern providing `path_for(name)` and `render_partial(name)` as the 2-method contract the shared templates depend on. `render_partial` delegates to `view_context.render(partial: "mysql_genius/queries/#{name}")`.
+- Extracted 5 AI prompt builders from the `AiFeatures` concern into `MysqlGenius::Core::Ai::{DescribeQuery, SchemaReview, RewriteQuery, IndexAdvisor, MigrationRisk}` plus a shared `Core::Ai::SchemaContextBuilder` helper. `anomaly_detection` and `root_cause` remain in the Rails concern because they depend on the Redis-backed `SlowQueryMonitor`.
+- Extracted `QueriesController#columns` logic into `MysqlGenius::Core::Analysis::Columns` with a tagged-result struct. Retires the `masked_column?` helper added in the 0.4.1 hotfix.
+- `MysqlGenius::Core::Ai::Config` gains a `domain_context:` field. The Rails adapter defaults it to a Rails-specific string; `mysql_genius-desktop` will default to empty.
+- `mysql_genius` now declares runtime dependency on `mysql_genius-core ~> 0.5.0` (was `~> 0.4.0`).
+
+### Added
+- Integration test suite at `spec/dummy/` + `spec/rails_helper.rb` + `spec/requests/`. Boots a minimal Rails engine dummy app and dispatches real HTTP requests against the mounted engine via `Rack::Test`. Dedicated regression specs at `spec/regressions/` pin the two Phase 1b latent bugs (`Core::Connection::ActiveRecordAdapter` boot-order and `QueriesController#masked_column?` helper deletion) so they can never silently return.
+- `CLAUDE.md` updated: the "no Rails boot in tests" rule is relaxed to a two-tier model (unit specs stub AR, integration specs boot Rails via `spec/dummy/`).
+
+### Internal
+- `MysqlGenius::Core.views_path` — new public module method returning the absolute path to the shared ERB template directory.
+
 ## 0.4.1
 
 ### Fixed

data/Gemfile

@@ -16,6 +16,8 @@ end
 group :development, :test do
   gem "rake"
   gem "rspec", "~> 3.0"
+  gem "rspec-rails"
+  gem "rack-test"
   gem "rubocop"
   gem "rubocop-shopify"
   gem "rubocop-rspec"

data/app/controllers/concerns/mysql_genius/ai_features.rb

@@ -47,20 +47,7 @@ module MysqlGenius
       sql = params[:sql].to_s.strip
       return render(json: { error: "SQL is required." }, status: :unprocessable_entity) if sql.blank?
 
-      messages = [
-        { role: "system", content: <<~PROMPT },
-          You are a MySQL query explainer. Given a SQL query, explain in plain English:
-          1. What the query does (tables involved, joins, filters, aggregations)
-          2. How data flows through the query
-          3. Any subtle behaviors (implicit type casts, NULL handling in NOT IN, DISTINCT effects, etc.)
-          4. Potential performance concerns visible from the SQL structure alone
-          #{ai_domain_context}
-          Respond with JSON: {"explanation": "your plain-English explanation using markdown formatting"}
-        PROMPT
-        { role: "user", content: sql },
-      ]
-
-      result = ai_client.chat(messages: messages)
+      result = MysqlGenius::Core::Ai::DescribeQuery.new(ai_client, ai_config_for_core).call(sql)
       render(json: result)
     rescue StandardError => e
       render(json: { error: "Explanation failed: #{e.message}" }, status: :unprocessable_entity)
@@ -69,42 +56,7 @@ module MysqlGenius
     def schema_review
       return ai_not_configured unless mysql_genius_config.ai_enabled?
 
-      table = params[:table].to_s.strip
-      connection = ActiveRecord::Base.connection
-
-      tables_to_review = table.present? ? [table] : queryable_tables.first(20)
-      schema_desc = tables_to_review.map do |t|
-        next unless connection.tables.include?(t)
-
-        cols = connection.columns(t).map { |c| "#{c.name} #{c.sql_type}#{" NOT NULL" unless c.null}#{" DEFAULT #{c.default}" if c.default}" }
-        pk = connection.primary_key(t)
-        indexes = connection.indexes(t).map { |idx| "#{"UNIQUE " if idx.unique}INDEX #{idx.name} (#{idx.columns.join(", ")})" }
-        row_count = connection.exec_query("SELECT TABLE_ROWS FROM information_schema.tables WHERE table_schema = #{connection.quote(connection.current_database)} AND table_name = #{connection.quote(t)}").rows.first&.first
-        desc = "Table: #{t} (~#{row_count} rows)\n"
-        desc += "Primary Key: #{pk || "NONE"}\n"
-        desc += "Columns: #{cols.join(", ")}\n"
-        desc += "Indexes: #{indexes.any? ? indexes.join(", ") : "NONE"}"
-        desc
-      end.compact.join("\n\n")
-
-      messages = [
-        { role: "system", content: <<~PROMPT },
-          You are a MySQL schema reviewer for a Ruby on Rails application. Analyze the following schema and identify anti-patterns and improvement opportunities. Look for:
-          - Inappropriate column types (VARCHAR(255) for short values, TEXT where VARCHAR suffices, INT for booleans)
-          - Missing indexes on foreign key columns or frequently filtered columns
-          - Missing NOT NULL constraints where NULLs are unlikely
-          - ENUM columns that should be lookup tables
-          - Missing created_at/updated_at timestamps
-          - Tables without a PRIMARY KEY
-          - Overly wide indexes or redundant indexes
-          - Column naming inconsistencies
-          #{ai_domain_context}
-          Respond with JSON: {"findings": "markdown-formatted findings organized by severity (Critical, Warning, Suggestion). Include specific ALTER TABLE statements where applicable."}
-        PROMPT
-        { role: "user", content: schema_desc },
-      ]
-
-      result = ai_client.chat(messages: messages)
+      result = MysqlGenius::Core::Ai::SchemaReview.new(ai_client, ai_config_for_core, rails_connection).call(params[:table].to_s.strip.presence)
       render(json: result)
     rescue StandardError => e
       render(json: { error: "Schema review failed: #{e.message}" }, status: :unprocessable_entity)
@@ -116,31 +68,7 @@ module MysqlGenius
       sql = params[:sql].to_s.strip
       return render(json: { error: "SQL is required." }, status: :unprocessable_entity) if sql.blank?
 
-      schema = build_schema_for_query(sql)
-
-      messages = [
-        { role: "system", content: <<~PROMPT },
-          You are a MySQL query rewrite expert. Analyze the SQL for anti-patterns and suggest a rewritten version. Look for:
-          - SELECT * when specific columns would suffice
-          - Correlated subqueries that could be JOINs
-          - OR conditions preventing index use (suggest UNION ALL)
-          - LIKE '%prefix' patterns (leading wildcard)
-          - Implicit type conversions in WHERE clauses
-          - NOT IN with NULLable columns (suggest NOT EXISTS)
-          - ORDER BY on non-indexed columns with LIMIT
-          - Unnecessary DISTINCT
-          - Functions on indexed columns in WHERE (e.g., DATE(created_at) instead of range)
-
-          Available schema:
-          #{schema}
-          #{ai_domain_context}
-
-          Respond with JSON: {"original": "the original SQL", "rewritten": "the improved SQL", "changes": "markdown list of each change and why it helps"}
-        PROMPT
-        { role: "user", content: sql },
-      ]
-
-      result = ai_client.chat(messages: messages)
+      result = MysqlGenius::Core::Ai::RewriteQuery.new(ai_client, ai_config_for_core, rails_connection).call(sql)
       render(json: result)
     rescue StandardError => e
       render(json: { error: "Rewrite failed: #{e.message}" }, status: :unprocessable_entity)
@@ -153,33 +81,7 @@ module MysqlGenius
       explain_rows = Array(params[:explain_rows]).map { |row| row.respond_to?(:values) ? row.values : Array(row) }
       return render(json: { error: "SQL and EXPLAIN output are required." }, status: :unprocessable_entity) if sql.blank? || explain_rows.blank?
 
-      connection = ActiveRecord::Base.connection
-      tables_in_query = MysqlGenius::Core::SqlValidator.extract_table_references(sql, connection)
-
-      index_detail = tables_in_query.map do |t|
-        indexes = connection.indexes(t).map { |idx| "#{"UNIQUE " if idx.unique}INDEX #{idx.name} (#{idx.columns.join(", ")})" }
-        stats = connection.exec_query("SELECT INDEX_NAME, COLUMN_NAME, CARDINALITY, SEQ_IN_INDEX FROM information_schema.STATISTICS WHERE TABLE_SCHEMA = #{connection.quote(connection.current_database)} AND TABLE_NAME = #{connection.quote(t)} ORDER BY INDEX_NAME, SEQ_IN_INDEX")
-        cardinality = stats.rows.map { |r| "#{r[0]}.#{r[1]}: cardinality=#{r[2]}" }.join(", ")
-        row_count = connection.exec_query("SELECT TABLE_ROWS FROM information_schema.tables WHERE table_schema = #{connection.quote(connection.current_database)} AND table_name = #{connection.quote(t)}").rows.first&.first
-        "Table: #{t} (~#{row_count} rows)\nIndexes: #{indexes.any? ? indexes.join("; ") : "NONE"}\nCardinality: #{cardinality}"
-      end.join("\n\n")
-
-      messages = [
-        { role: "system", content: <<~PROMPT },
-          You are a MySQL index advisor. Given a query, its EXPLAIN output, and current index/cardinality information, suggest optimal indexes. Consider:
-          - Composite index column ordering (most selective first, or matching query order)
-          - Covering indexes to avoid table lookups
-          - Partial indexes for long string columns
-          - Write-side costs (if this is a high-write table, note the INSERT/UPDATE overhead)
-          - Whether existing indexes could be extended rather than creating new ones
-          #{ai_domain_context}
-
-          Respond with JSON: {"indexes": "markdown-formatted recommendations with exact CREATE INDEX statements, rationale for column ordering, and estimated impact. Include any indexes that should be DROPPED as part of the change."}
-        PROMPT
-        { role: "user", content: "Query:\n#{sql}\n\nEXPLAIN:\n#{explain_rows.map { |r| r.join(" | ") }.join("\n")}\n\nCurrent Indexes:\n#{index_detail}" },
-      ]
-
-      result = ai_client.chat(messages: messages)
+      result = MysqlGenius::Core::Ai::IndexAdvisor.new(ai_client, ai_config_for_core, rails_connection).call(sql, explain_rows)
       render(json: result)
     rescue StandardError => e
       render(json: { error: "Index advisor failed: #{e.message}" }, status: :unprocessable_entity)
@@ -323,54 +225,7 @@ module MysqlGenius
       migration_sql = params[:migration].to_s.strip
       return render(json: { error: "Migration SQL or Ruby code is required." }, status: :unprocessable_entity) if migration_sql.blank?
 
-      connection = ActiveRecord::Base.connection
-
-      # Try to identify tables mentioned in the migration
-      table_names = migration_sql.scan(/(?:create_table|add_column|remove_column|add_index|remove_index|rename_column|change_column|alter\s+table)\s+[:\"]?(\w+)/i).flatten.uniq
-      table_names += migration_sql.scan(/ALTER\s+TABLE\s+`?(\w+)`?/i).flatten
-
-      table_info = table_names.uniq.map do |t|
-        next unless connection.tables.include?(t)
-
-        row_count = connection.exec_query("SELECT TABLE_ROWS FROM information_schema.tables WHERE table_schema = #{connection.quote(connection.current_database)} AND table_name = #{connection.quote(t)}").rows.first&.first
-        indexes = connection.indexes(t).map { |idx| "#{idx.name} (#{idx.columns.join(", ")})" }
-        "Table: #{t} (~#{row_count} rows, #{indexes.size} indexes)"
-      end.compact.join("\n")
-
-      # Current active queries on those tables
-      active = ""
-      begin
-        results = connection.exec_query(<<~SQL)
-          SELECT DIGEST_TEXT, COUNT_STAR AS calls, ROUND(AVG_TIMER_WAIT / 1000000000, 1) AS avg_ms
-          FROM performance_schema.events_statements_summary_by_digest
-          WHERE SCHEMA_NAME = #{connection.quote(connection.current_database)}
-            AND DIGEST_TEXT IS NOT NULL
-            AND COUNT_STAR > 10
-          ORDER BY COUNT_STAR DESC LIMIT 20
-        SQL
-        matching = results.rows.select { |r| table_names.any? { |t| r[0].to_s.downcase.include?(t.downcase) } }
-        active = matching.map { |r| "calls=#{r[1]} avg=#{r[2]}ms: #{r[0].to_s.truncate(200)}" }.join("\n")
-      rescue ActiveRecord::StatementInvalid
-        # performance_schema may be unavailable
-      end
-
-      messages = [
-        { role: "system", content: <<~PROMPT },
-          You are a MySQL migration risk assessor. Given a Rails migration or DDL, evaluate:
-          1. Will this lock the table? For how long given the row count?
-          2. Is this safe to run during traffic, or does it need a maintenance window?
-          3. Should pt-online-schema-change or gh-ost be used instead?
-          4. Will it break or degrade any of the active queries against this table?
-          5. Are there any data loss risks?
-          6. What is the recommended deployment strategy?
-          #{ai_domain_context}
-
-          Respond with JSON: {"risk_level": "low|medium|high|critical", "assessment": "markdown-formatted risk assessment with specific recommendations and estimated lock duration"}
-        PROMPT
-        { role: "user", content: "Migration:\n#{migration_sql}\n\nAffected Tables:\n#{table_info.presence || "Could not determine"}\n\nActive Queries on These Tables:\n#{active.presence || "None found or performance_schema unavailable"}" },
-      ]
-
-      result = ai_client.chat(messages: messages)
+      result = MysqlGenius::Core::Ai::MigrationRisk.new(ai_client, ai_config_for_core, rails_connection).call(migration_sql)
       render(json: result)
     rescue StandardError => e
       render(json: { error: "Migration risk assessment failed: #{e.message}" }, status: :unprocessable_entity)
@@ -378,6 +233,10 @@ module MysqlGenius
 
     private
 
+    RAILS_DOMAIN_CONTEXT = <<~CTX
+      This is a Ruby on Rails application. Do NOT recommend adding foreign key constraints (FOREIGN KEY / REFERENCES); Rails handles referential integrity at the application layer. DO recommend indexes on foreign key columns for join performance.
+    CTX
+
     def ai_client
       MysqlGenius::Core::Ai::Client.new(ai_config_for_core)
     end
@@ -391,6 +250,7 @@ module MysqlGenius
         model: cfg.ai_model,
         auth_style: cfg.ai_auth_style,
         system_context: cfg.ai_system_context,
+        domain_context: RAILS_DOMAIN_CONTEXT,
       )
     end
 
@@ -398,21 +258,14 @@ module MysqlGenius
       render(json: { error: "AI features are not configured." }, status: :not_found)
     end
 
-    def ai_domain_context
-      parts = []
-      parts << "This is a Ruby on Rails application. Do NOT recommend adding foreign key constraints (FOREIGN KEY / REFERENCES); Rails handles referential integrity at the application layer. DO recommend indexes on foreign key columns for join performance."
-      ctx = mysql_genius_config.ai_system_context
-      parts << "Domain context:\n#{ctx}" if ctx.present?
-      "\n" + parts.join("\n")
+    def rails_connection
+      MysqlGenius::Core::Connection::ActiveRecordAdapter.new(ActiveRecord::Base.connection)
     end
 
-    def build_schema_for_query(sql)
-      connection = ActiveRecord::Base.connection
-      tables = MysqlGenius::Core::SqlValidator.extract_table_references(sql, connection)
-      tables.map do |t|
-        cols = connection.columns(t).map { |c| "#{c.name} (#{c.type})" }
-        "#{t}: #{cols.join(", ")}"
-      end.join("\n")
+    def ai_domain_context
+      cfg = mysql_genius_config
+      ctx = cfg.ai_system_context
+      ctx.present? ? "\nDomain context:\n#{ctx}" : ""
     end
   end
 end

data/app/controllers/concerns/mysql_genius/shared_view_helpers.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module MysqlGenius
+  module SharedViewHelpers
+    extend ActiveSupport::Concern
+
+    included do
+      helper_method :path_for, :render_partial
+    end
+
+    # URL path helper for shared templates.
+    #   path_for(:execute) # => "/mysql_genius/execute" (from engine route helpers)
+    def path_for(name)
+      mysql_genius.public_send("#{name}_path")
+    end
+
+    # Partial renderer for shared templates.
+    #   render_partial(:tab_dashboard) # => view_context.render partial: "mysql_genius/queries/tab_dashboard"
+    def render_partial(name)
+      view_context.render(partial: "mysql_genius/queries/#{name}")
+    end
+  end
+end

data/app/controllers/mysql_genius/queries_controller.rb

@@ -5,6 +5,7 @@ module MysqlGenius
     include QueryExecution
     include DatabaseAnalysis
     include AiFeatures
+    include SharedViewHelpers
 
     def index
       @featured_tables = if mysql_genius_config.featured_tables.any?
@@ -14,23 +15,24 @@ module MysqlGenius
       end
       @all_tables = queryable_tables.sort
       @ai_enabled = mysql_genius_config.ai_enabled?
+      @framework_version_major = Rails::VERSION::MAJOR
+      @framework_version_minor = Rails::VERSION::MINOR
+      render("mysql_genius/queries/dashboard")
     end
 
     def columns
-      table = params[:table]
-      if mysql_genius_config.blocked_tables.include?(table)
-        return render(json: { error: "Table '#{table}' is not available for querying." }, status: :forbidden)
-      end
-
-      unless ActiveRecord::Base.connection.tables.include?(table)
-        return render(json: { error: "Table '#{table}' does not exist." }, status: :not_found)
-      end
+      result = MysqlGenius::Core::Analysis::Columns.new(
+        rails_connection,
+        blocked_tables: mysql_genius_config.blocked_tables,
+        masked_column_patterns: mysql_genius_config.masked_column_patterns,
+        default_columns: mysql_genius_config.default_columns,
+      ).call(table: params[:table])
 
-      defaults = mysql_genius_config.default_columns[table] || []
-      cols = ActiveRecord::Base.connection.columns(table).reject { |c| masked_column?(c.name) }.map do |c|
-        { name: c.name, type: c.type.to_s, default: defaults.empty? || defaults.include?(c.name) }
+      case result.status
+      when :ok        then render(json: result.columns)
+      when :blocked   then render(json: { error: result.error_message }, status: :forbidden)
+      when :not_found then render(json: { error: result.error_message }, status: :not_found)
       end
-      render(json: cols)
     end
 
     def slow_queries
@@ -58,14 +60,8 @@ module MysqlGenius
       ActiveRecord::Base.connection.tables - mysql_genius_config.blocked_tables
     end
 
-    # Delegates to Core::SqlValidator's 2-arg class method. A bare
-    # `masked_column?(name)` call survives on line 30 because this helper
-    # reintroduces the 1-arg instance method the controller's `columns`
-    # action depends on. Without this helper, `columns` raises NoMethodError
-    # at runtime (Phase 1b regression — Core::SqlValidator.masked_column?
-    # became a 2-arg class method but the call site wasn't updated).
-    def masked_column?(name)
-      MysqlGenius::Core::SqlValidator.masked_column?(name, mysql_genius_config.masked_column_patterns)
+    def rails_connection
+      MysqlGenius::Core::Connection::ActiveRecordAdapter.new(ActiveRecord::Base.connection)
     end
   end
 end

data/lib/mysql_genius/engine.rb

@@ -4,6 +4,10 @@ module MysqlGenius
   class Engine < ::Rails::Engine
     isolate_namespace MysqlGenius
 
+    initializer "mysql_genius.register_core_views", before: :add_view_paths do
+      paths["app/views"] << MysqlGenius::Core.views_path
+    end
+
     config.after_initialize do
       if MysqlGenius.configuration.redis_url.present?
         require "mysql_genius/slow_query_monitor"

data/lib/mysql_genius/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MysqlGenius
-  VERSION = "0.4.1"
+  VERSION = "0.5.1"
 end

data/mysql_genius.gemspec

@@ -31,6 +31,6 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency("activerecord", ">= 5.2", "< 9")
-  spec.add_dependency("mysql_genius-core", "~> 0.4.0")
+  spec.add_dependency("mysql_genius-core", "~> 0.5.0")
   spec.add_dependency("railties", ">= 5.2", "< 9")
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mysql_genius
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.1
 platform: ruby
 authors:
 - Antarr Byrd
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-11 00:00:00.000000000 Z
+date: 2026-04-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -36,14 +36,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.5.0
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
@@ -88,20 +88,10 @@ files:
 - app/controllers/concerns/mysql_genius/ai_features.rb
 - app/controllers/concerns/mysql_genius/database_analysis.rb
 - app/controllers/concerns/mysql_genius/query_execution.rb
+- app/controllers/concerns/mysql_genius/shared_view_helpers.rb
 - app/controllers/mysql_genius/base_controller.rb
 - app/controllers/mysql_genius/queries_controller.rb
 - app/views/layouts/mysql_genius/application.html.erb
-- app/views/mysql_genius/queries/_shared_results.html.erb
-- app/views/mysql_genius/queries/_tab_ai_tools.html.erb
-- app/views/mysql_genius/queries/_tab_dashboard.html.erb
-- app/views/mysql_genius/queries/_tab_duplicate_indexes.html.erb
-- app/views/mysql_genius/queries/_tab_query_explorer.html.erb
-- app/views/mysql_genius/queries/_tab_query_stats.html.erb
-- app/views/mysql_genius/queries/_tab_server.html.erb
-- app/views/mysql_genius/queries/_tab_slow_queries.html.erb
-- app/views/mysql_genius/queries/_tab_table_sizes.html.erb
-- app/views/mysql_genius/queries/_tab_unused_indexes.html.erb
-- app/views/mysql_genius/queries/index.html.erb
 - bin/console
 - bin/setup
 - config/routes.rb

data/app/views/mysql_genius/queries/_shared_results.html.erb

@@ -1,56 +0,0 @@
-<!-- Explain Results -->
-<div id="explain-results" class="mg-mt mg-hidden">
-  <div class="mg-card">
-    <div class="mg-card-header">
-      <span><strong>&#128270; EXPLAIN Output</strong></span>
-      <div>
-        <% if @ai_enabled %>
-          <button id="explain-optimize" class="mg-btn mg-btn-outline mg-btn-sm">&#9889; AI Optimization</button>
-          <button id="explain-index-advisor" class="mg-btn mg-btn-outline mg-btn-sm">&#9889; Index Advisor</button>
-        <% end %>
-        <button id="explain-close" class="mg-btn mg-btn-outline-secondary mg-btn-sm">&#10005; Close</button>
-      </div>
-    </div>
-    <div class="mg-card-body">
-      <div class="mg-table-wrap">
-        <table class="mg-table">
-          <thead id="explain-thead"></thead>
-          <tbody id="explain-tbody"></tbody>
-        </table>
-      </div>
-      <div id="optimize-results" class="mg-hidden mg-mt">
-        <div id="optimize-content" class="mg-alert mg-alert-info"></div>
-      </div>
-    </div>
-  </div>
-</div>
-
-<!-- Results Area -->
-<div id="query-results" class="mg-mt">
-  <div id="results-alert" class="mg-hidden"></div>
-  <div id="results-stats" class="mg-mb mg-hidden">
-    <span id="results-row-count" class="mg-badge mg-badge-info"></span>
-    <span id="results-time" class="mg-badge mg-badge-secondary"></span>
-    <span id="results-truncated" class="mg-badge mg-badge-warning mg-hidden">Results truncated</span>
-  </div>
-  <div id="results-table-wrapper" class="mg-table-wrap mg-hidden">
-    <table class="mg-table">
-      <thead id="results-thead"></thead>
-      <tbody id="results-tbody"></tbody>
-    </table>
-  </div>
-  <div id="results-empty" class="mg-text-center mg-text-muted mg-hidden">No rows returned.</div>
-</div>
-
-<!-- AI Query Analysis Results -->
-<div id="ai-query-result" class="mg-mt mg-hidden">
-  <div class="mg-card">
-    <div class="mg-card-header">
-      <span id="ai-query-title"><strong>&#9889; AI Analysis</strong></span>
-      <button id="ai-query-close" class="mg-btn mg-btn-outline-secondary mg-btn-sm">&#10005;</button>
-    </div>
-    <div class="mg-card-body">
-      <div id="ai-query-content" style="font-size:13px;"></div>
-    </div>
-  </div>
-</div>

data/app/views/mysql_genius/queries/_tab_ai_tools.html.erb

@@ -1,43 +0,0 @@
-<!-- AI Tools Tab -->
-<div class="mg-tab-content" id="tab-aitools">
-  <!-- Schema Review -->
-  <div class="mg-card mg-mb">
-    <div class="mg-card-header"><strong>&#9889; Schema Review</strong></div>
-    <div class="mg-card-body">
-      <div class="mg-text-muted mg-mb" style="font-size:12px;">Analyze your schema for anti-patterns: inappropriate column types, missing indexes, naming inconsistencies, and more.</div>
-      <div class="mg-row" style="align-items:flex-end;">
-        <div class="mg-col-4 mg-field">
-          <label for="schema-table">Table (leave blank for all)</label>
-          <select id="schema-table">
-            <option value="">All tables (top 20)</option>
-            <% @all_tables.each do |table| %>
-              <option value="<%= table %>"><%= table %></option>
-            <% end %>
-          </select>
-        </div>
-        <div class="mg-field">
-          <button id="schema-review-btn" class="mg-btn mg-btn-primary mg-btn-sm">&#9889; Analyze Schema</button>
-        </div>
-      </div>
-      <div id="schema-result" class="mg-mt mg-hidden">
-        <div id="schema-result-content" style="font-size:13px;"></div>
-      </div>
-    </div>
-  </div>
-
-  <!-- Migration Risk Assessment -->
-  <div class="mg-card">
-    <div class="mg-card-header"><strong>&#9889; Migration Risk Assessment</strong></div>
-    <div class="mg-card-body">
-      <div class="mg-text-muted mg-mb" style="font-size:12px;">Paste a Rails migration or DDL and get a risk assessment: lock duration, impact on active queries, deployment strategy.</div>
-      <div class="mg-field">
-        <textarea id="migration-input" rows="8" placeholder="class AddIndexToUsers < ActiveRecord::Migration[7.0]&#10;  def change&#10;    add_index :users, :email, unique: true&#10;  end&#10;end"></textarea>
-      </div>
-      <button id="migration-assess-btn" class="mg-btn mg-btn-primary mg-btn-sm">&#9889; Assess Risk</button>
-      <div id="migration-result" class="mg-mt mg-hidden">
-        <div id="migration-risk-badge" style="margin-bottom:8px;"></div>
-        <div id="migration-result-content" style="font-size:13px;"></div>
-      </div>
-    </div>
-  </div>
-</div>