rails_pulse 0.1.2 → 0.1.4

data/app/services/rails_pulse/query_analysis_service.rb

@@ -0,0 +1,125 @@
+# Orchestrates comprehensive query analysis using modular analyzers.
+# Coordinates multiple specialized analyzers and consolidates results into actionable insights.
+module RailsPulse
+  class QueryAnalysisService
+    def self.analyze_query(query_id)
+      query = RailsPulse::Query.find(query_id)
+      new(query).analyze
+    end
+
+    def initialize(query)
+      @query = query
+      @operations = fetch_recent_operations
+    end
+
+    def analyze
+      # Run all analyzers
+      results = {
+        analyzed_at: Time.current,
+        query_characteristics: analyze_query_characteristics,
+        index_recommendations: analyze_index_recommendations,
+        n_plus_one_analysis: analyze_n_plus_one,
+        explain_plan: analyze_explain_plan,
+        backtrace_analysis: analyze_backtraces
+      }
+
+      # Generate consolidated suggestions
+      results[:suggestions] = generate_suggestions(results)
+
+      # Build compatible format for query model
+      compatible_results = build_compatible_results(results)
+
+      # Save results to query
+      save_results_to_query(compatible_results)
+
+      results
+    end
+
+    private
+
+    def fetch_recent_operations
+      @query.operations
+            .where("occurred_at > ?", 48.hours.ago)
+            .order(occurred_at: :desc)
+            .limit(50)
+    end
+
+    def analyze_query_characteristics
+      Analysis::QueryCharacteristicsAnalyzer.new(@query, @operations).analyze
+    end
+
+    def analyze_index_recommendations
+      Analysis::IndexRecommendationEngine.new(@query, @operations).analyze
+    end
+
+    def analyze_n_plus_one
+      Analysis::NPlusOneDetector.new(@query, @operations).analyze
+    end
+
+    def analyze_explain_plan
+      return { explain_plan: nil, issues: [] } if @operations.empty?
+      Analysis::ExplainPlanAnalyzer.new(@query, @operations).analyze
+    end
+
+    def analyze_backtraces
+      return {} if @operations.empty?
+      Analysis::BacktraceAnalyzer.new(@query, @operations).analyze
+    end
+
+    def generate_suggestions(analysis_results)
+      Analysis::SuggestionGenerator.new(analysis_results).generate
+    end
+
+    # Build compatible format for query model storage
+    def build_compatible_results(results)
+      characteristics = results[:query_characteristics]
+      explain_result = results[:explain_plan]
+
+      {
+        analyzed_at: results[:analyzed_at],
+        explain_plan: explain_result[:explain_plan],
+        issues: extract_all_issues(characteristics, explain_result),
+        metadata: build_metadata(results),
+        query_stats: extract_query_stats(characteristics),
+        backtrace_analysis: results[:backtrace_analysis],
+        index_recommendations: results[:index_recommendations],
+        n_plus_one_analysis: results[:n_plus_one_analysis],
+        suggestions: results[:suggestions]
+      }
+    end
+
+    def extract_all_issues(characteristics, explain_result)
+      issues = []
+      issues.concat(characteristics[:pattern_issues] || [])
+      issues.concat(explain_result[:issues] || [])
+      issues
+    end
+
+    def extract_query_stats(characteristics)
+      characteristics.except(:pattern_issues)
+    end
+
+    def build_metadata(results)
+      {
+        analyzers_used: results.keys.reject { |k| k.in?([ :analyzed_at, :suggestions ]) },
+        analysis_version: "2.0",
+        total_recommendations: results[:index_recommendations]&.count || 0,
+        n_plus_one_detected: results.dig(:n_plus_one_analysis, :is_likely_n_plus_one) || false
+      }
+    end
+
+    def save_results_to_query(results)
+      @query.update!(
+        analyzed_at: results[:analyzed_at],
+        explain_plan: results[:explain_plan],
+        issues: results[:issues],
+        metadata: results[:metadata],
+        query_stats: results[:query_stats],
+        backtrace_analysis: results[:backtrace_analysis],
+        index_recommendations: results[:index_recommendations],
+        n_plus_one_analysis: results[:n_plus_one_analysis],
+        suggestions: results[:suggestions]
+      )
+    end
+  end
+end

data/app/views/layouts/rails_pulse/_sidebar_menu.html.erb

@@ -5,7 +5,6 @@
     </div>
     <div class="flex flex-col text-start leading-tight overflow-hidden">
       <span class="overflow-ellipsis font-semibold">Rails Pulse</span>
-      <span class="overflow-ellipsis text-xs">Open Source</span>
     </div>
   </a>
 

data/app/views/layouts/rails_pulse/application.html.erb

@@ -22,7 +22,6 @@
         <div class="hide@md" data-controller="rails-pulse--dialog">
           <button type="button" class="btn btn--icon" data-action="rails-pulse--dialog#showModal">
             <%= rails_pulse_icon 'menu', width: '20' %>
-            <span class="sr-only">Open menu</span>
           </button>
 
         <dialog class="sheet sheet--left" style="--sheet-size: 288px;" data-rails-pulse--dialog-target="menu" data-action="click->rails-pulse--dialog#closeOnClickOutside">
@@ -31,7 +30,6 @@
               <%= link_to rails_pulse.root_path, class: "btn sidebar-menu__button" do %>
                 <div class="flex flex-col text-start leading-tight overflow-hidden">
                   <span class="overflow-ellipsis font-semibold">Rails Pulse</span>
-                  <span class="overflow-ellipsis text-xs">Open Source</span>
                 </div>
               <% end %>
               <div class="sidebar-menu__content">

data/app/views/rails_pulse/components/_breadcrumbs.html.erb

@@ -1,4 +1,4 @@
-<nav class="breadcrumb" aria-label="Breadcrumb">
+<nav class="breadcrumb mis-2" aria-label="Breadcrumb">
   <% breadcrumbs.each_with_index do |crumb, index| %>
     <% if crumb[:current] %>
       <span class="text-primary" aria-disabled="true" aria-current="page" role="link"><%= crumb[:title] %></span>

data/app/views/rails_pulse/components/_code_panel.html.erb

@@ -2,11 +2,25 @@
   title ||= nil
 %>
 
-<div>
+<div
+  data-controller="rails-pulse--collapsible"
+  data-rails-pulse--collapsible-collapsed-class="collapsed"
+  class="collapsible-code"
+>
   <% if title %>
-    <h2 class="grow font-semibold leading-none mbe-1 uppercase text-xs"><%= title %></h2>
+    <h2 class="grow font-semibold leading-none mbe-1 uppercase text-xs">
+      <%= title %>
+      <button
+        type="button"
+        class="collapsible-toggle"
+        data-rails-pulse--collapsible-target="toggle"
+        data-action="click->rails-pulse--collapsible#toggle"
+      >
+        show more
+      </button>
+    </h2>
   <% end %>
-  <div class="prose max-i-none">
+  <div class="prose max-i-none" data-rails-pulse--collapsible-target="content">
     <pre class="mbs-0" style="white-space: normal; margin-block-end: 0; margin-block-start: 0;"><code><%= html_escape(value) %></code></pre>
   </div>
 </div>

data/app/views/rails_pulse/components/_empty_state.html.erb

@@ -1,7 +1,7 @@
 <div class="flex items-center justify-center pbs-12 pbe-12 pis-6 pie-6 mb-8">
   <div class="flex items-center gap">
     <div class="shrink-0">
-      <img src="<%= asset_path('search.svg') %>" class="w-32 h-24 opacity-50" alt="No data available" />
+      <img src="<%= asset_path('search.svg') %>" class="w-48 h-48" alt="No data available" />
     </div>
     <div class="text-subtle pis-8">
       <p class="text-lg font-semibold mbe-2"><%= title %></p>

data/app/views/rails_pulse/components/_metric_card.html.erb

@@ -4,19 +4,42 @@
   context = data[:context]
   title = data[:title]
   summary = data[:summary]
-  line_chart_data = data[:line_chart_data]
+  chart_data = data[:chart_data]
   trend_icon = data[:trend_icon]
   trend_amount = data[:trend_amount]
   trend_text = data[:trend_text]
 %>
 <div class="grid-item" data-controller="rails-pulse--chart" id="<%= id %>">
   <%= render 'rails_pulse/components/panel', { title: title, card_classes: 'card-compact' } do %>
-    <div class="row mbs-2" style="height: 50px; margin-bottom: 0;">
+    <div class="row mbs-2" style="--columns: 2; align-items: center; margin-bottom: 0;">
       <div class="grid-item">
         <h4 class="text-xl mbs-1 font-bold"><%= summary %></h4>
       </div>
       <div class="grid-item">
-        <%= line_chart line_chart_data, height: "100%", options: sparkline_chart_options %>
+        <div class="chart-container chart-container--slim">
+          <%
+            # Match chart color to trending icon - use hex equivalents of CSS variables
+            # These colors match the badge colors exactly
+            chart_color = case trend_icon
+            when "trending-up"
+              "#dc2626" # equivalent to --red-600 (negative trend)
+            when "trending-down"
+              "#16a34a" # equivalent to --green-600 (positive trend)
+            else
+              "#ffc91f" # equivalent to --zinc-900 (neutral/primary)
+            end
+
+            chart_options = sparkline_chart_options.deep_merge(
+              color: [chart_color],
+              series: {
+                itemStyle: {
+                  color: chart_color
+                }
+              }
+            )
+          %>
+          <%= bar_chart chart_data, height: "100%", options: chart_options %>
+        </div>
       </div>
     </div>
     <div class="mbs-2" style="height: 10px;">
@@ -30,9 +53,9 @@
           end
       %>
       <div class="flex items-center justify-between">
-        <span class="badge <%= badge %> p-0">
+        <span class="badge <%= badge %> badge--trend p-0">
           <%= rails_pulse_icon trend_icon, height: "15px", width: "15px", class: "mie-2" %>
-          <%= trend_amount %>
+          <span class="badge__trend-amount"><%= trend_amount %></span>
           <p class="mis-2 text-subtle text-xs"><%= trend_text %></p>
         </span>
       </div>

data/app/views/rails_pulse/components/_operation_details_popover.html.erb

@@ -26,7 +26,7 @@
         <div>
           <h4 class="text-xs font-medium text-subtle uppercase">Occurred At</h4>
           <div class="text-sm">
-            <%= operation.occurred_at.strftime("%H:%M:%S.%L") %>
+            <%= operation.occurred_at.getlocal.strftime("%H:%M:%S.%L") %>
           </div>
         </div>
       <% end %>

data/app/views/rails_pulse/components/_panel.html.erb

@@ -15,7 +15,7 @@
       <% end %>
       <div class="flex gap items-center">
         <% if help_heading %>
-          <div data-controller="rails-pulse--popover" data-rails-pulse--popover-placement-value="bottom-end">
+          <div data-controller="rails-pulse--popover" data-rails-pulse--popover-placement-value="bottom-start">
             <a href="#"
               data-rails-pulse--popover-target="button"
               data-action="rails-pulse--popover#toggle"

data/app/views/rails_pulse/components/_sparkline_stats.html.erb

@@ -1,10 +1,8 @@
-<div class="row mbs-2" style="height: 50px; margin-bottom: 0;">
-  <div class="grid-item">
-    <h4 class="text-xl mbs-1 font-bold"><%= summary %></h4>
-  </div>
-  <div class="grid-item">
-    <%= line_chart line_chart_data, height: "100%", options: sparkline_chart_options %>
-  </div>
+<div class="mbs-2">
+  <h4 class="text-xl mbs-1 font-bold"><%= summary %></h4>
+</div>
+<div class="chart-container chart-container--slim">
+  <%= bar_chart chart_data, height: "100%", options: sparkline_chart_options %>
 </div>
 <div>
   <span class="badge badge--<%= trend_direction == "down" ? "positive" : "negative" %>-inverse p-0">

data/app/views/rails_pulse/components/_table_head.html.erb

@@ -8,10 +8,15 @@
           <% sort_params = {} %>
           <% sort_params[:zoom_start_time] = @zoom_start if @zoom_start.present? %>
           <% sort_params[:zoom_end_time] = @zoom_end if @zoom_end.present? %>
+          <% sort_params[:selected_column_time] = params[:selected_column_time] if params[:selected_column_time].present? %>
           <%= sort_link @ransack_query, column[:field], column[:label],
             sort_params.merge(
               class: "flex items-center",
-              data: { turbo_prefetch: "false", turbo_action: "replace" }
+              data: {
+                turbo_prefetch: "false",
+                turbo_frame: "index_table",
+                action: "click->rails-pulse--table-sort#updateUrl"
+              }
             ) %>
         <% end %>
       </th>

data/app/views/rails_pulse/dashboard/index.html.erb

@@ -6,7 +6,7 @@
 </div>
 
 <div class="row">
-  <div class="grid-item" style="height:300px">
+  <div class="grid-item">
     <%= render 'rails_pulse/components/panel', {
       title: 'Average Response Time',
       card_classes: 'b-full',
@@ -71,7 +71,7 @@
   <div class="grid-item">
     <%= render 'rails_pulse/components/panel', {
       title: 'Slowest Queries This Week',
-      help_heading: 'Slowest Queries', 
+      help_heading: 'Slowest Queries',
       help_text: 'This panel shows the slowest database queries in your application this week, including average execution time and when they were last seen.',
       actions: [{ url: queries_path, icon: 'external-link', title: 'View details', data: { turbo_frame: '_top' } }],
       card_classes: 'table-container'

data/app/views/rails_pulse/operations/show.html.erb

@@ -4,23 +4,25 @@
 
 <div class="card">
   <%= turbo_frame_tag "operation_#{@operation.id}_details" do %>
+    <% if @operation.operation_type == "sql" %>
+      <div class="mbe-4">
+        <%= render 'rails_pulse/components/code_panel', { value: @operation.label } %>
+      </div>
+    <% end %>
+
     <dl class="descriptive-list">
-      <dt>
-        <% if @operation.operation_type == "sql" %>
-          Query
-        <% elsif ["template", "partial", "layout", "collection"].include?(@operation.operation_type) %>
-          View Render
-        <% else %>
-          <%= html_escape(@operation.label) %>
-        <% end %>
-      </dt>
-      <dd>
-        <% if @operation.operation_type == "sql" %>
-          <%= render 'rails_pulse/components/code_panel', { value: @operation.label } %>
-        <% else %>
+      <% if @operation.operation_type != "sql" %>
+        <dt>
+          <% if ["template", "partial", "layout", "collection"].include?(@operation.operation_type) %>
+            View Render
+          <% else %>
+            <%= html_escape(@operation.label) %>
+          <% end %>
+        </dt>
+        <dd>
           <%= html_escape(@operation.label) %>
-        <% end %>
-      </dd>
+        </dd>
+      <% end %>
 
       <dt>Operation Type</dt>
       <dd><pre><%= @operation.operation_type %></pre></dd>

data/app/views/rails_pulse/queries/_analysis_error.html.erb

@@ -0,0 +1,15 @@
+<div class="panel panel--danger">
+  <div>
+    <h3 class="text-sm bold">Analysis Failed</h3>
+    <p class="text-sm mt-1">
+      <%= error_message %>
+    </p>
+    <div class="mt-3">
+      <%= button_to analyze_query_path(query), method: :post,
+          data: { turbo_frame: "query_analysis" },
+          class: "btn btn--sm" do %>
+        Try Again
+      <% end %>
+    </div>
+  </div>
+</div>

data/app/views/rails_pulse/queries/_analysis_prompt.html.erb

@@ -0,0 +1,27 @@
+<div class="center py-4">
+  <h3 class="text-lg mb-2">Analyze Query Performance</h3>
+  <p class="mb-4">
+    Get detailed insights about this query's performance, potential issues, and optimization suggestions.
+  </p>
+
+  <% if query.has_recent_operations? %>
+    <%= button_to analyze_query_path(query), method: :post,
+        data: { turbo_frame: "query_analysis" },
+        class: "btn btn--primary" do %>
+      Analyze Query
+    <% end %>
+
+    <p class="text-sm mt-2">
+      Analysis will examine recent executions and provide performance insights.
+    </p>
+  <% else %>
+    <div class="badge badge--warning p-3 mt-3">
+      <div>
+        <h4 class="text-sm bold">No Recent Data Available</h4>
+        <p class="text-sm mt-1">
+          This query hasn't been executed recently. Analysis requires recent execution data to provide meaningful insights.
+        </p>
+      </div>
+    </div>
+  <% end %>
+</div>

data/app/views/rails_pulse/queries/_analysis_results.html.erb

@@ -0,0 +1,117 @@
+<div class="row">
+  <div>
+    <!-- Query Characteristics -->
+    <% if query.query_stats.present? %>
+      <h5>Query Characteristics</h5>
+      <dl class="descriptive-list mbs-4">
+        <dt>Last Analyzed</dt>
+        <dd><%= time_ago_in_words(query.analyzed_at) %></dd>
+        <dt>Query Type</dt>
+        <dd><%= query.query_stats['query_type'] %></dd>
+        <dt>Tables</dt>
+        <dd><%= query.query_stats['table_count'] || 0 %></dd>
+        <dt>Joins</dt>
+        <dd><%= query.query_stats['join_count'] || 0 %></dd>
+        <dt>
+          Complexity Score
+          <div data-controller="rails-pulse--popover" data-rails-pulse--popover-placement-value="bottom-start" style="display: inline-block; margin-left: 4px;">
+            <a href="#"
+              data-rails-pulse--popover-target="button"
+              data-action="rails-pulse--popover#toggle"
+              data-popovertarget="complexity-score-popover"
+              style="color: var(--gray-500); vertical-align: top;">
+              <%= rails_pulse_icon 'info', height: "14px" %>
+            </a>
+
+            <div popover class="popover card" data-rails-pulse--popover-target="menu" style="max-width: 22rem">
+              <div class="flex flex-col">
+                <h3 class="font-semibold leading-none mbe-2 uppercase text-sm">Complexity Score</h3>
+                <p class="text-sm text-subtle mbe-3">
+                  A calculated score representing query complexity based on multiple factors:
+                </p>
+                <ul class="text-sm text-subtle" style="list-style-type: disc; padding-left: 16px; margin: 0;">
+                  <li>Tables: +2 points per table</li>
+                  <li>Joins: +3 points per join</li>
+                  <li>WHERE conditions: +1 point per condition, +2 per function</li>
+                  <li>UNIONs: +4 points each</li>
+                  <li>Subqueries: +5 points each</li>
+                </ul>
+                <p class="text-sm text-subtle mbs-3">
+                  Higher scores indicate more complex queries that may need optimization.
+                </p>
+              </div>
+            </div>
+          </div>
+        </dt>
+        <dd><%= query.query_stats['estimated_complexity'] || 0 %></dd>
+        <dt>Has LIMIT</dt>
+        <dd><%= query.query_stats['has_limit'] ? 'Yes' : 'No' %></dd>
+        <dt>Has ORDER BY</dt>
+        <dd><%= query.query_stats['has_order_by'] ? 'Yes' : 'No' %></dd>
+        <dt>Has GROUP BY</dt>
+        <dd><%= query.query_stats['has_group_by'] ? 'Yes' : 'No' %></dd>
+        <dt>Has Subqueries</dt>
+        <dd><%= query.query_stats['has_subqueries'] ? 'Yes' : 'No' %></dd>
+      </dl>
+    <% end %>
+  </div>
+
+  <div>
+    <!-- Execution Analysis -->
+    <% if query.backtrace_analysis.present? %>
+      <h5>Execution Analysis</h5>
+      <dl class="descriptive-list mbs-4">
+        <dt>Total Executions</dt>
+        <dd><%= query.backtrace_analysis['total_executions'] || 0 %></dd>
+        <dt>Unique Locations</dt>
+        <dd><%= query.backtrace_analysis['unique_locations'] || 0 %></dd>
+        <dt>Execution Frequency</dt>
+        <dd><%= query.backtrace_analysis['execution_frequency'] || 0 %>/hour</dd>
+        <dt>Most Common Location</dt>
+        <dd><code><%= query.backtrace_analysis['most_common_location']['location'] || 'N/A' %></code></dd>
+        <dt>N+1 Pattern Detected</dt>
+        <dd>
+          <% if query.backtrace_analysis['potential_n_plus_one'] %>
+            Yes
+          <% else %>
+            No
+          <% end %>
+        </dd>
+      </dl>
+    <% end %>
+  </div>
+</div>
+
+
+<!-- Issues Summary -->
+<% if query.issues.present? && query.issues.any? %>
+  <hr class="mb-4" />
+  <h3 class="text-lg bold">Issues Detected</h3>
+  <ul style="list-style-type: disc; padding-left: 20px;">
+    <% query.issues.each do |issue| %>
+      <li><%= issue['description'] %></li>
+    <% end %>
+  </ul>
+<% end %>
+
+
+<!-- Suggestions -->
+<% if query.suggestions.present? && query.suggestions.any? %>
+  <hr class="mb-4" />
+  <h3 class="text-lg bold">Optimization Suggestions</h3>
+  <ul style="list-style-type: disc; padding-left: 20px;">
+    <% query.suggestions.each do |suggestion| %>
+      <li>
+        <%= suggestion['action'] %>.
+        <%= suggestion['benefit'] if suggestion['benefit'].present? %>
+      </li>
+    <% end %>
+  </ul>
+<% end %>
+
+<!-- EXPLAIN Plan -->
+<% if query.explain_plan.present? %>
+  <hr class="mb-4" />
+  <h3 class="text-lg bold">Execution Plan</h3>
+  <pre class="text-sm" style="overflow: scroll"><%= query.explain_plan %></pre>
+<% end %>

data/app/views/rails_pulse/queries/_analysis_section.html.erb

@@ -0,0 +1,39 @@
+<%
+  analyze_actions = []
+  analyze_actions << {
+    url: analyze_query_path(query),
+    title: query.analyzed? ? "Re-analyze query" : "Analyze query performance",
+    icon: "refresh-cw",
+    data: {
+      turbo_method: "post",
+      turbo_frame: "query_analysis"
+    }
+  }
+%>
+
+<%= render 'rails_pulse/components/panel', {
+  title: 'Query Analysis',
+  actions: analyze_actions
+} do %>
+  <% if local_assigns[:error_message] %>
+    <%= render 'rails_pulse/queries/analysis_error', error_message: error_message, query: query %>
+  <% elsif query.analyzed? %>
+    <%= render 'rails_pulse/queries/analysis_results', query: query %>
+  <% else %>
+    <div class="flex items-center justify-center pbs-12 pbe-12 pis-6 pie-6 mb-8">
+      <div class="flex items-center gap">
+        <div class="shrink-0">
+          <img src="<%= asset_path('search.svg') %>" class="w-48 h-48" alt="No data available" />
+        </div>
+        <div class="text-subtle pis-8">
+          <p class="text-lg font-semibold mbe-2">Analyze Query Performance</p>
+          <% if query.has_recent_operations? %>
+            <p class="text-sm mbe-2"> Get detailed insights about this query's performance, potential issues, and optimization suggestions. </p>
+          <% else %>
+            <p class="text-sm mbe-2">No recent data available for analysis</p>
+          <% end %>
+        </div>
+      </div>
+    </div>
+  <% end %>
+<% end %>

data/app/views/rails_pulse/queries/_show_table.html.erb

@@ -1,16 +1,44 @@
 <% columns = [
-  { field: :occurred_at, label: 'Timestamp', class: 'w-auto' },
-  { field: :duration, label: 'Duration', class: 'w-32'}
+  { field: :period_start, label: 'Time Period', class: 'w-auto' },
+  { field: :count, label: 'Executions', class: 'w-32'},
+  { field: :avg_duration, label: 'Avg Duration', class: 'w-32'},
+  { field: :min_duration, label: 'Min Duration', class: 'w-32'},
+  { field: :max_duration, label: 'Max Duration', class: 'w-32'}
 ] %>
 
-<table class="table mbs-4">
+<table class="table mbs-4" data-controller="rails-pulse--table-sort">
   <%= render "rails_pulse/components/table_head", columns: columns %>
 
   <tbody>
-    <% @table_data.each do |query| %>
+    <% @table_data.each do |summary| %>
+      <%
+        # Determine performance class based on average duration
+        avg_duration_ms = summary.avg_duration&.round(2) || 0
+        performance_class = case avg_duration_ms
+        when 0..10 then "text-green-600"
+        when 10..50 then "text-yellow-600"
+        when 50..100 then "text-orange-600"
+        else "text-red-600"
+        end
+      %>
       <tr>
-        <td class="whitespace-nowrap"><%= human_readable_occurred_at(query.occurred_at)  %></td>
-        <td class="whitespace-nowrap"><%= query.duration.round(2) %> ms</td>
+        <td class="whitespace-nowrap">
+          <%= human_readable_summary_period(summary) %>
+        </td>
+        <td class="whitespace-nowrap text-center">
+          <span class="font-medium"><%= summary.count %></span>
+        </td>
+        <td class="whitespace-nowrap">
+          <span class="<%= performance_class %> font-medium">
+            <%= avg_duration_ms %> ms
+          </span>
+        </td>
+        <td class="whitespace-nowrap text-center">
+          <%= summary.min_duration&.round(2) || 0 %> ms
+        </td>
+        <td class="whitespace-nowrap text-center">
+          <%= summary.max_duration&.round(2) || 0 %> ms
+        </td>
       </tr>
     <% end %>
   </tbody>

data/app/views/rails_pulse/queries/_table.html.erb

@@ -1,12 +1,10 @@
 <% columns = [
   { field: :normalized_sql, label: 'Query', class: 'w-auto' },
-  { field: :execution_count_sort, label: 'Executions', class: 'w-24' },
-  { field: :avg_duration_sort, label: 'Avg Time', class: 'w-24' },
-  { field: :total_time_consumed_sort, label: 'Total Time', class: 'w-28' },
-  { field: :performance_status, label: 'Status', class: 'w-16', sortable: false }
+  { field: :avg_duration_sort, label: 'Average Query Time', class: 'w-44' },
+  { field: :execution_count_sort, label: 'Executions', class: 'w-24' }
 ] %>
 
-<table class="table mbs-4">
+<table class="table mbs-4" data-controller="rails-pulse--table-sort">
   <%= render "rails_pulse/components/table_head", columns: columns %>
 
   <tbody>
@@ -17,10 +15,8 @@
             <%= link_to html_escape(truncate_sql(summary.normalized_sql)), query_path(summary.query_id), data: { turbo_frame: '_top', } %>
           </div>
         </td>
-        <td class="whitespace-nowrap"><%= number_with_delimiter summary.execution_count %></td>
         <td class="whitespace-nowrap"><%= summary.avg_duration.to_i %> ms</td>
-        <td class="whitespace-nowrap"><%= number_with_delimiter summary.total_time_consumed.to_i %> ms</td>
-        <td class="whitespace-nowrap text-center"><%= query_status_indicator(summary.avg_duration) %></td>
+        <td class="whitespace-nowrap"><%= number_with_delimiter summary.execution_count %></td>
       </tr>
     <% end %>
   </tbody>