rails_pulse 0.1.1 → 0.1.2

data/app/models/rails_pulse/routes/cards/average_response_times.rb

@@ -7,39 +7,61 @@ module RailsPulse
         end
 
         def to_metric_card
-          requests = if @route
-            RailsPulse::Request.where(route: @route)
-          else
-            RailsPulse::Request.all
-          end
+          last_7_days = 7.days.ago.beginning_of_day
+          previous_7_days = 14.days.ago.beginning_of_day
 
-          requests = requests.where("occurred_at >= ?", 2.weeks.ago.beginning_of_day)
+          # Single query to get all aggregated metrics with conditional sums
+          base_query = RailsPulse::Summary.where(
+            summarizable_type: "RailsPulse::Route",
+            period_type: "day",
+            period_start: 2.weeks.ago.beginning_of_day..Time.current
+          )
+          base_query = base_query.where(summarizable_id: @route.id) if @route
 
-          # Calculate overall average response time
-          average_response_time = requests.average(:duration)&.round(0) || 0
+          metrics = base_query.select(
+            "SUM(avg_duration * count) AS total_weighted_duration",
+            "SUM(count) AS total_requests",
+            "SUM(CASE WHEN period_start >= '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN avg_duration * count ELSE 0 END) AS current_weighted_duration",
+            "SUM(CASE WHEN period_start >= '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN count ELSE 0 END) AS current_requests",
+            "SUM(CASE WHEN period_start >= '#{previous_7_days.strftime('%Y-%m-%d %H:%M:%S')}' AND period_start < '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN avg_duration * count ELSE 0 END) AS previous_weighted_duration",
+            "SUM(CASE WHEN period_start >= '#{previous_7_days.strftime('%Y-%m-%d %H:%M:%S')}' AND period_start < '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN count ELSE 0 END) AS previous_requests"
+          ).take
 
-          # Calculate trend by comparing last 7 days vs previous 7 days
-          last_7_days = 7.days.ago.beginning_of_day
-          previous_7_days = 14.days.ago.beginning_of_day
-          current_period_avg = requests.where("occurred_at >= ?", last_7_days).average(:duration) || 0
-          previous_period_avg = requests.where("occurred_at >= ? AND occurred_at < ?", previous_7_days, last_7_days).average(:duration) || 0
+          # Calculate metrics from single query result
+          average_response_time = metrics.total_requests.to_i > 0 ? (metrics.total_weighted_duration / metrics.total_requests).round(0) : 0
+          current_period_avg = metrics.current_requests.to_i > 0 ? (metrics.current_weighted_duration / metrics.current_requests) : 0
+          previous_period_avg = metrics.previous_requests.to_i > 0 ? (metrics.previous_weighted_duration / metrics.previous_requests) : 0
 
-          percentage = previous_period_avg.zero? ?  0 : ((previous_period_avg - current_period_avg) / previous_period_avg * 100).abs.round(1)
-          trend_icon = percentage < 0.1 ?  "move-right" : current_period_avg < previous_period_avg ? "trending-down" : "trending-up"
+          percentage = previous_period_avg.zero? ? 0 : ((previous_period_avg - current_period_avg) / previous_period_avg * 100).abs.round(1)
+          trend_icon = percentage < 0.1 ? "move-right" : current_period_avg < previous_period_avg ? "trending-down" : "trending-up"
           trend_amount = previous_period_avg.zero? ? "0%" : "#{percentage}%"
 
-          sparkline_data = requests
-            .group_by_week(:occurred_at, time_zone: "UTC")
-            .average(:duration)
-            .each_with_object({}) do |(date, avg), hash|
-              formatted_date = date.strftime("%b %-d")
-              value = avg&.round(0) || 0
-              hash[formatted_date] = {
-                value: value
+          # Separate query for sparkline data - manually calculate weighted averages by week
+          sparkline_data = {}
+          base_query.each do |summary|
+            week_start = summary.period_start.beginning_of_week
+            formatted_date = week_start.strftime("%b %-d")
+
+            if sparkline_data[formatted_date]
+              sparkline_data[formatted_date][:total_weighted] += (summary.avg_duration || 0) * (summary.count || 0)
+              sparkline_data[formatted_date][:total_count] += (summary.count || 0)
+            else
+              sparkline_data[formatted_date] = {
+                total_weighted: (summary.avg_duration || 0) * (summary.count || 0),
+                total_count: (summary.count || 0)
               }
             end
+          end
+
+          # Convert to final format
+          sparkline_data = sparkline_data.transform_values do |data|
+            weighted_avg = data[:total_count] > 0 ? (data[:total_weighted] / data[:total_count]).round(0) : 0
+            { value: weighted_avg }
+          end
 
           {
+            id: "average_response_times",
+            context: "routes",
             title: "Average Response Time",
             summary: "#{average_response_time} ms",
             line_chart_data: sparkline_data,

data/app/models/rails_pulse/routes/cards/error_rate_per_route.rb

@@ -7,60 +7,53 @@ module RailsPulse
         end
 
         def to_metric_card
-          # Calculate error rate for each route or a specific route
-          routes = if @route
-            RailsPulse::Route.where(id: @route)
-          else
-            RailsPulse::Route.all
-          end
+          last_7_days = 7.days.ago.beginning_of_day
+          previous_7_days = 14.days.ago.beginning_of_day
 
-          routes = routes.where("occurred_at >= ?", 2.weeks.ago.beginning_of_day)
+          # Single query to get all error metrics with conditional aggregation
+          base_query = RailsPulse::Summary.where(
+            summarizable_type: "RailsPulse::Route",
+            period_type: "day",
+            period_start: 2.weeks.ago.beginning_of_day..Time.current
+          )
+          base_query = base_query.where(summarizable_id: @route.id) if @route
 
-          error_rates = routes.joins(:requests)
-            .select("rails_pulse_routes.id, rails_pulse_routes.path, COUNT(rails_pulse_requests.id) as total_requests, SUM(CASE WHEN rails_pulse_requests.is_error = true THEN 1 ELSE 0 END) as error_count")
-            .group("rails_pulse_routes.id, rails_pulse_routes.path")
-            .map do |route|
-              error_rate = route.error_count.to_f / route.total_requests * 100
-              {
-                path: route.path,
-                error_rate: error_rate.round(2)
-              }
-            end
+          metrics = base_query.select(
+            "SUM(error_count) AS total_errors",
+            "SUM(count) AS total_requests",
+            "SUM(CASE WHEN period_start >= '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN error_count ELSE 0 END) AS current_errors",
+            "SUM(CASE WHEN period_start >= '#{previous_7_days.strftime('%Y-%m-%d %H:%M:%S')}' AND period_start < '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN error_count ELSE 0 END) AS previous_errors"
+          ).take
 
-          # Calculate overall error rate summary as errors per day
-          requests = @route ? RailsPulse::Request.where(route: @route) : RailsPulse::Request.all
-          total_errors = requests.where(is_error: true).count
-          min_time = requests.minimum(:occurred_at)
-          max_time = requests.maximum(:occurred_at)
-          total_days = min_time && max_time && min_time != max_time ? (max_time - min_time) / 1.day : 1
-          errors_per_day = total_errors / total_days
-          error_rate_summary = "#{errors_per_day.round(2)} / day"
+          # Calculate metrics from single query result
+          total_errors = metrics.total_errors || 0
+          total_requests = metrics.total_requests || 0
+          current_period_errors = metrics.current_errors || 0
+          previous_period_errors = metrics.previous_errors || 0
 
-          # Generate sparkline data
-          sparkline_data = requests
-            .where(is_error: true)
-            .group_by_week(:occurred_at, time_zone: "UTC")
-            .count
-            .each_with_object({}) do |(date, count), hash|
-              formatted_date = date.strftime("%b %-d")
-              hash[formatted_date] = {
-                value: count
-              }
-            end
+          # Calculate overall error rate percentage
+          overall_error_rate = total_requests > 0 ? (total_errors.to_f / total_requests * 100).round(2) : 0
 
-          # Determine trend direction and amount
-          last_7_days = 7.days.ago.beginning_of_day
-          previous_7_days = 14.days.ago.beginning_of_day
-          current_period_errors = requests.where("occurred_at >= ? AND is_error = ?", last_7_days, true).count
-          previous_period_errors = requests.where("occurred_at >= ? AND occurred_at < ? AND is_error = ?", previous_7_days, last_7_days, true).count
+          # Calculate trend
+          percentage = previous_period_errors.zero? ? 0 : ((previous_period_errors - current_period_errors) / previous_period_errors.to_f * 100).abs.round(1)
+          trend_icon = percentage < 0.1 ? "move-right" : current_period_errors < previous_period_errors ? "trending-down" : "trending-up"
+          trend_amount = previous_period_errors.zero? ? "0%" : "#{percentage}%"
 
-          trend_amount = previous_period_errors.zero? ? "0%" : "#{((current_period_errors - previous_period_errors) / previous_period_errors.to_f * 100).round(1)}%"
-          trend_icon = trend_amount.to_f < 0.1 ? "move-right" : current_period_errors < previous_period_errors ? "trending-down" : "trending-up"
+          # Separate query for sparkline data - group by week using Rails
+          sparkline_data = base_query
+            .group_by_week(:period_start, time_zone: "UTC")
+            .sum(:error_count)
+            .each_with_object({}) do |(week_start, total_errors), hash|
+              formatted_date = week_start.strftime("%b %-d")
+              value = total_errors || 0
+              hash[formatted_date] = { value: value }
+            end
 
           {
+            id: "error_rate_per_route",
+            context: "routes",
             title: "Error Rate Per Route",
-            data: error_rates,
-            summary: error_rate_summary,
+            summary: "#{overall_error_rate}%",
             line_chart_data: sparkline_data,
             trend_icon: trend_icon,
             trend_amount: trend_amount,

data/app/models/rails_pulse/routes/cards/percentile_response_times.rb

@@ -7,60 +7,47 @@ module RailsPulse
         end
 
         def to_metric_card
-          requests = if @route
-            RailsPulse::Request.where(route: @route)
-          else
-            RailsPulse::Request.all
-          end
-
-          requests = requests.where("occurred_at >= ?", 2.weeks.ago.beginning_of_day)
-
-          # Calculate overall 95th percentile response time
-          count = requests.count
-          percentile_95th = if count > 0
-            requests.select("duration").order("duration").limit(1).offset((count * 0.95).floor).pluck(:duration).first.round(0) || 0
-          else
-            0
-          end
-
-          # Calculate trend by comparing last 7 days vs previous 7 days for 95th percentile
           last_7_days = 7.days.ago.beginning_of_day
           previous_7_days = 14.days.ago.beginning_of_day
 
-          current_period = requests.where("occurred_at >= ?", last_7_days)
-          current_count = current_period.count
-          current_period_95th = if current_count > 0
-            current_period.select("duration").order("duration").limit(1).offset((current_count * 0.95).floor).pluck(:duration).first || 0
-          else
-            0
-          end
-
-          previous_period = requests.where("occurred_at >= ? AND occurred_at < ?", previous_7_days, last_7_days)
-          previous_count = previous_period.count
-          previous_period_95th = if previous_count > 0
-            previous_period.select("duration").order("duration").limit(1).offset((previous_count * 0.95).floor).pluck(:duration).first || 0
-          else
-            0
-          end
-
-          percentage = previous_period_95th.zero? ?  0 : ((previous_period_95th - current_period_95th) / previous_period_95th * 100).abs.round(1)
-          trend_icon = percentage < 0.1 ?  "move-right" : current_period_95th < previous_period_95th ? "trending-down" : "trending-up"
-          trend_amount = previous_period_95th.zero? ? "0%" : "#{percentage}%"
-
-          sparkline_data = requests
-            .group_by_week(:occurred_at, time_zone: "UTC")
-            .average(:duration)
-            .each_with_object({}) do |(date, avg), hash|
-              formatted_date = date.strftime("%b %-d")
-              value = avg&.round(0) || 0
-              hash[formatted_date] = {
-                value: value
-              }
+          # Single query to get all P95 metrics with conditional aggregation
+          base_query = RailsPulse::Summary.where(
+            summarizable_type: "RailsPulse::Route",
+            period_type: "day",
+            period_start: 2.weeks.ago.beginning_of_day..Time.current
+          )
+          base_query = base_query.where(summarizable_id: @route.id) if @route
+
+          metrics = base_query.select(
+            "AVG(p95_duration) AS overall_p95",
+            "AVG(CASE WHEN period_start >= '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN p95_duration ELSE NULL END) AS current_p95",
+            "AVG(CASE WHEN period_start >= '#{previous_7_days.strftime('%Y-%m-%d %H:%M:%S')}' AND period_start < '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN p95_duration ELSE NULL END) AS previous_p95"
+          ).take
+
+          # Calculate metrics from single query result
+          p95_response_time = (metrics.overall_p95 || 0).round(0)
+          current_period_p95 = metrics.current_p95 || 0
+          previous_period_p95 = metrics.previous_p95 || 0
+
+          percentage = previous_period_p95.zero? ? 0 : ((previous_period_p95 - current_period_p95) / previous_period_p95 * 100).abs.round(1)
+          trend_icon = percentage < 0.1 ? "move-right" : current_period_p95 < previous_period_p95 ? "trending-down" : "trending-up"
+          trend_amount = previous_period_p95.zero? ? "0%" : "#{percentage}%"
+
+          # Separate query for sparkline data - group by week using Rails
+          sparkline_data = base_query
+            .group_by_week(:period_start, time_zone: "UTC")
+            .average(:p95_duration)
+            .each_with_object({}) do |(week_start, avg_p95), hash|
+              formatted_date = week_start.strftime("%b %-d")
+              value = (avg_p95 || 0).round(0)
+              hash[formatted_date] = { value: value }
             end
 
           {
+            id: "percentile_response_times",
+            context: "routes",
             title: "95th Percentile Response Time",
-            summary: "#{percentile_95th} ms",
+            summary: "#{p95_response_time} ms",
             line_chart_data: sparkline_data,
             trend_icon: trend_icon,
             trend_amount: trend_amount,

data/app/models/rails_pulse/routes/cards/request_count_totals.rb

@@ -7,44 +7,49 @@ module RailsPulse
         end
 
         def to_metric_card
-          requests = if @route
-            RailsPulse::Request.where(route: @route)
-          else
-            RailsPulse::Request.all
-          end
+          last_7_days = 7.days.ago.beginning_of_day
+          previous_7_days = 14.days.ago.beginning_of_day
 
-          requests = requests.where("occurred_at >= ?", 2.weeks.ago.beginning_of_day)
+          # Single query to get all count metrics with conditional aggregation
+          base_query = RailsPulse::Summary.where(
+            summarizable_type: "RailsPulse::Route",
+            period_type: "day",
+            period_start: 2.weeks.ago.beginning_of_day..Time.current
+          )
+          base_query = base_query.where(summarizable_id: @route.id) if @route
 
-          # Calculate total request count
-          total_request_count = requests.count
+          metrics = base_query.select(
+            "SUM(count) AS total_count",
+            "SUM(CASE WHEN period_start >= '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN count ELSE 0 END) AS current_count",
+            "SUM(CASE WHEN period_start >= '#{previous_7_days.strftime('%Y-%m-%d %H:%M:%S')}' AND period_start < '#{last_7_days.strftime('%Y-%m-%d %H:%M:%S')}' THEN count ELSE 0 END) AS previous_count"
+          ).take
 
-          # Calculate trend by comparing last 7 days vs previous 7 days
-          last_7_days = 7.days.ago.beginning_of_day
-          previous_7_days = 14.days.ago.beginning_of_day
-          current_period_count = requests.where("occurred_at >= ?", last_7_days).count
-          previous_period_count = requests.where("occurred_at >= ? AND occurred_at < ?", previous_7_days, last_7_days).count
+          # Calculate metrics from single query result
+          total_request_count = metrics.total_count || 0
+          current_period_count = metrics.current_count || 0
+          previous_period_count = metrics.previous_count || 0
 
           percentage = previous_period_count.zero? ? 0 : ((previous_period_count - current_period_count) / previous_period_count.to_f * 100).abs.round(1)
           trend_icon = percentage < 0.1 ? "move-right" : current_period_count < previous_period_count ? "trending-down" : "trending-up"
           trend_amount = previous_period_count.zero? ? "0%" : "#{percentage}%"
 
-          sparkline_data = requests
-            .group_by_week(:occurred_at, time_zone: "UTC")
-            .count
-            .each_with_object({}) do |(date, count), hash|
-              formatted_date = date.strftime("%b %-d")
-              hash[formatted_date] = {
-                value: count
-              }
+          # Separate query for sparkline data - group by week using Rails
+          sparkline_data = base_query
+            .group_by_week(:period_start, time_zone: "UTC")
+            .sum(:count)
+            .each_with_object({}) do |(week_start, total_count), hash|
+              formatted_date = week_start.strftime("%b %-d")
+              value = total_count || 0
+              hash[formatted_date] = { value: value }
             end
 
-          # Calculate average requests per minute
-          min_time = requests.minimum(:occurred_at)
-          max_time = requests.maximum(:occurred_at)
-          total_minutes = min_time && max_time && min_time != max_time ? (max_time - min_time) / 60.0 : 1
+          # Calculate average requests per minute over 2-week period
+          total_minutes = 2.weeks / 1.minute
           average_requests_per_minute = total_request_count / total_minutes
 
           {
+            id: "request_count_totals",
+            context: "routes",
             title: "Request Count Total",
             summary: "#{average_requests_per_minute.round(2)} / min",
             line_chart_data: sparkline_data,

data/app/models/rails_pulse/routes/charts/average_response_times.rb

@@ -2,112 +2,35 @@ module RailsPulse
   module Routes
     module Charts
       class AverageResponseTimes
-        def initialize(ransack_query:, group_by: :group_by_day, route: nil)
+        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil)
           @ransack_query = ransack_query
-          @group_by = group_by
+          @period_type = period_type
           @route = route
+          @start_time = start_time
+          @end_time = end_time
+          @start_duration = start_duration
         end
 
         def to_rails_chart
-          # Get actual data using existing logic
-          actual_data = if @route
-            # These are the requests for the specific route so it will just be a collection of Requests that we can
-            # filter and sort using the attributes on each Request
-            @ransack_query.result(distinct: false)
-              .public_send(@group_by, "occurred_at", series: true, time_zone: "UTC")
-              .average(:duration)
-          else
-            # Use the existing query structure with left_joins from ransack
-            @ransack_query.result(distinct: false)
-              .left_joins(:requests)
-              .public_send(
-                @group_by,
-                "rails_pulse_requests.occurred_at",
-                series: true,
-                time_zone: "UTC"
-              )
-              .average("rails_pulse_requests.duration")
-          end
-
-          # Create full time range and fill in missing periods
-          fill_missing_periods(actual_data)
-        end
-
-        private
-
-        def fill_missing_periods(actual_data)
-          # Extract actual time range from ransack query conditions
-          start_time, end_time = extract_time_range_from_ransack
-
-          # Create time range based on grouping type
-          case @group_by
-          when :group_by_hour
-            time_range = generate_hour_range(start_time, end_time)
-          else # :group_by_day
-            time_range = generate_day_range(start_time, end_time)
-          end
-
-          # Fill in all periods with zero values for missing periods
-          time_range.each_with_object({}) do |period, result|
-            occurred_at = period.is_a?(String) ? Time.parse(period) : period
-            occurred_at = (occurred_at.is_a?(Time) || occurred_at.is_a?(Date)) ? occurred_at : Time.current
-
-            normalized_occurred_at =
-              case @group_by
-              when :group_by_hour
-                occurred_at&.beginning_of_hour || occurred_at
-              when :group_by_day
-                occurred_at&.beginning_of_day || occurred_at
-              else
-                occurred_at
-              end
-
-            # Use actual data if available, otherwise default to 0
-            average_duration = actual_data[period] || 0
-            result[normalized_occurred_at.to_i] = {
-              value: average_duration.to_f
-            }
-          end
-        end
-
-        def generate_day_range(start_time, end_time)
-          (start_time.to_date..end_time.to_date).map(&:beginning_of_day)
-        end
-
-        def generate_hour_range(start_time, end_time)
-          current = start_time
-          hours = []
-          while current <= end_time
-            hours << current
-            current += 1.hour
-          end
-          hours
-        end
-
-        def extract_time_range_from_ransack
-          # Extract time range from ransack conditions
-          conditions = @ransack_query.conditions
-
-          if @route
-            # For specific route queries, look for occurred_at conditions
-            start_condition = conditions.find { |c| c.a.first == "occurred_at" && c.p == "gteq" }
-            end_condition = conditions.find { |c| c.a.first == "occurred_at" && c.p == "lt" }
-          else
-            # For general route queries, look for requests_occurred_at conditions
-            start_condition = conditions.find { |c| c.a.first == "requests_occurred_at" && c.p == "gteq" }
-            end_condition = conditions.find { |c| c.a.first == "requests_occurred_at" && c.p == "lt" }
-          end
-
-          start_time = start_condition&.v || 2.weeks.ago
-          end_time = end_condition&.v || Time.current
-
-          # Normalize time boundaries based on grouping
-          case @group_by
-          when :group_by_hour
-            [ start_time.beginning_of_hour, end_time.beginning_of_hour ]
-          else
-            [ start_time.beginning_of_day, end_time.beginning_of_day ]
+          summaries = @ransack_query.result(distinct: false).where(
+            summarizable_type: "RailsPulse::Route",
+            period_type: @period_type
+          )
+
+          summaries = summaries.where(summarizable_id: @route.id) if @route
+          summaries = summaries
+            .group(:period_start)
+            .having("AVG(avg_duration) > ?", @start_duration || 0)
+            .average(:avg_duration)
+            .transform_keys(&:to_i)
+
+          # Pad missing data points with zeros
+          step = @period_type == :hour ? 1.hour : 1.day
+          data = {}
+          (@start_time.to_i..@end_time.to_i).step(step) do |timestamp|
+            data[timestamp.to_i] = summaries[timestamp.to_i].to_f.round(2)
           end
+          data
         end
       end
     end

data/app/models/rails_pulse/routes/tables/index.rb

@@ -2,60 +2,77 @@ module RailsPulse
   module Routes
     module Tables
       class Index
-        def initialize(ransack_query:, start_time:, params:)
+        def initialize(ransack_query:, period_type: nil, start_time:, params:)
           @ransack_query = ransack_query
+          @period_type = period_type
           @start_time = start_time
           @params = params
         end
 
         def to_table
-          # Pre-calculate values to avoid SQL injection and improve readability
-          minutes_elapsed = calculate_minutes_elapsed
+          # Check if we have explicit ransack sorts
+          has_sorts = @ransack_query.sorts.any?
 
-          # Get thresholds with safe defaults to avoid nil access errors
-          config = RailsPulse.configuration rescue nil
-          thresholds = config&.route_thresholds || { slow: 500, very_slow: 1500, critical: 3000 }
-
-          requests_per_minute_divisor = minutes_elapsed > 0 ? minutes_elapsed : 1
+          base_query = @ransack_query.result(distinct: false)
+            .joins("INNER JOIN rails_pulse_routes ON rails_pulse_routes.id = rails_pulse_summaries.summarizable_id")
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: @period_type
+            )
 
-          status_sql = build_status_sql(thresholds)
+          base_query = base_query.where(summarizable_id: @route.id) if @route
 
-          @ransack_query.result(distinct: false)
-            .left_joins(:requests)
-            .group("rails_pulse_routes.id")
+          # Apply grouping and aggregation
+          grouped_query = base_query
+            .group(
+              "rails_pulse_summaries.summarizable_id",
+              "rails_pulse_summaries.summarizable_type",
+              "rails_pulse_routes.id",
+              "rails_pulse_routes.path",
+              "rails_pulse_routes.method"
+            )
             .select(
-              "rails_pulse_routes.*",
-              "COALESCE(AVG(rails_pulse_requests.duration), 0) AS average_response_time_ms",
-              "COUNT(rails_pulse_requests.id) AS request_count",
-              "COALESCE(COUNT(rails_pulse_requests.id) / #{requests_per_minute_divisor}, 0) AS requests_per_minute",
-              "COALESCE(SUM(CASE WHEN rails_pulse_requests.is_error = true THEN 1 ELSE 0 END), 0) AS error_count",
-              "CASE WHEN COUNT(rails_pulse_requests.id) > 0 THEN ROUND((COALESCE(SUM(CASE WHEN rails_pulse_requests.is_error = true THEN 1 ELSE 0 END), 0) * 100.0) / COUNT(rails_pulse_requests.id), 2) ELSE 0 END AS error_rate_percentage",
-              "COALESCE(MAX(rails_pulse_requests.duration), 0) AS max_response_time_ms",
-              "#{status_sql} AS status_indicator"
+              "rails_pulse_summaries.summarizable_id",
+              "rails_pulse_summaries.summarizable_type",
+              "rails_pulse_routes.id as route_id",
+              "rails_pulse_routes.path",
+              "rails_pulse_routes.method as route_method",
+              "AVG(rails_pulse_summaries.avg_duration) as avg_duration",
+              "MAX(rails_pulse_summaries.max_duration) as max_duration",
+              "SUM(rails_pulse_summaries.count) as count",
+              "SUM(rails_pulse_summaries.error_count) as error_count",
+              "SUM(rails_pulse_summaries.success_count) as success_count"
             )
-        end
-
-        private
 
-        def calculate_minutes_elapsed
-          start_timestamp = Time.at(@start_time.to_i).utc
-          ((Time.current.utc - start_timestamp) / 60.0).round(2)
-        end
+          # Apply sorting based on ransack sorts or use default
+          if has_sorts
+            # Apply custom sorting based on ransack parameters
+            sort = @ransack_query.sorts.first
+            direction = sort.dir == "desc" ? :desc : :asc
 
-        def build_status_sql(thresholds)
-          # Ensure all thresholds have default values
-          slow = thresholds[:slow] || 500
-          very_slow = thresholds[:very_slow] || 1500
-          critical = thresholds[:critical] || 3000
+            case sort.name
+            when "avg_duration_sort"
+              grouped_query = grouped_query.order(Arel.sql("AVG(rails_pulse_summaries.avg_duration)").send(direction))
+            when "max_duration_sort"
+              grouped_query = grouped_query.order(Arel.sql("MAX(rails_pulse_summaries.max_duration)").send(direction))
+            when "count_sort"
+              grouped_query = grouped_query.order(Arel.sql("SUM(rails_pulse_summaries.count)").send(direction))
+            when "requests_per_minute"
+              grouped_query = grouped_query.order(Arel.sql("SUM(rails_pulse_summaries.count) / 60.0").send(direction))
+            when "error_rate_percentage"
+              grouped_query = grouped_query.order(Arel.sql("(SUM(rails_pulse_summaries.error_count) * 100.0) / SUM(rails_pulse_summaries.count)").send(direction))
+            when "route_path"
+              grouped_query = grouped_query.order(Arel.sql("rails_pulse_routes.path").send(direction))
+            else
+              # Unknown sort field, fallback to default
+              grouped_query = grouped_query.order(Arel.sql("AVG(rails_pulse_summaries.avg_duration)").desc)
+            end
+          else
+            # Apply default sort when no explicit sort is provided (matches controller default_table_sort)
+            grouped_query = grouped_query.order(Arel.sql("AVG(rails_pulse_summaries.avg_duration)").desc)
+          end
 
-          <<-SQL.squish
-            CASE
-              WHEN COALESCE(AVG(rails_pulse_requests.duration), 0) >= #{critical} THEN 3
-              WHEN COALESCE(AVG(rails_pulse_requests.duration), 0) >= #{very_slow} THEN 2
-              WHEN COALESCE(AVG(rails_pulse_requests.duration), 0) >= #{slow} THEN 1
-              ELSE 0
-            END
-          SQL
+          grouped_query
         end
       end
     end