rails_pulse 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a8ed469f307dd71e912b46a4da0a0daa60abbdc262a8dbd12604db00a66335fe
-  data.tar.gz: 8af5a6bf946a86e1bc277c1d719100ace739bb429c07969fd1e4ec31887d6f70
+  metadata.gz: d9f5f4407186848ab8ef3c75d8453dbe44d4f90fd641a50527b26cf9f4b10b82
+  data.tar.gz: 283484459f73560755c16519e808d754bf23d82edb05ef7b98fcc396bdf3ead2
 SHA512:
-  metadata.gz: eceb3af7e713b6049688e50a8fd79af7b2f09c7720158c8d15257235b0c7488ebe7c0eccfaab3cd5dfb5923f80e9a8bd8693d0a9fb2f4e021aca8846c4ac435c
-  data.tar.gz: f982a50256d99968eb68fee89e6380d3f75511e6c3e8e878ddca375919f6a68a201f9ce46c6868e2425c127ae1862ca0d1895b0e01afea89a9af6a4e85d7f0f9
+  metadata.gz: b75fca5343d2080ecc547a5c8b68d2ec7ac431aea54505ad042721262c0f95c5955e229dcd338b011dfcdc208d7e6467b8b1c768efa83379efff5d7deaf13200
+  data.tar.gz: a8e5d530527ed56d61029707825dd74885a742b132b41d5471c981f544c922e0431a6813d61e73db01dc85b17681aeb25c53347358cf93586b1b36af7874ab99

data/app/assets/stylesheets/rails_pulse/components/tags.css

@@ -75,8 +75,8 @@
 }
 
 .tag-add-button {
-  padding: 0.25rem 0.5rem;
-  font-size: 0.875rem;
+  padding: 0.2rem 0.5rem;
+  font-size: 0.8rem;
   line-height: 1.25;
   white-space: nowrap;
 }

data/app/controllers/concerns/chart_table_concern.rb

@@ -38,6 +38,8 @@ module ChartTableConcern
       start_time: @start_time,
       end_time: @end_time,
       start_duration: @start_duration,
+      disabled_tags: session_disabled_tags,
+      show_non_tagged: session[:show_non_tagged] != false,
       **chart_options
     ).to_rails_chart
   end

data/app/controllers/rails_pulse/application_controller.rb

@@ -17,6 +17,7 @@ module RailsPulse
     def set_global_filters
       if params[:clear] == "true"
         session.delete(:global_filters)
+        session[:show_non_tagged] = true  # Reset show_non_tagged to default
       else
         filters = session[:global_filters] || {}
 

data/app/controllers/rails_pulse/dashboard_controller.rb

@@ -1,18 +1,22 @@
 module RailsPulse
   class DashboardController < ApplicationController
     def index
-      @average_query_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: nil).to_metric_card
-      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: nil).to_metric_card
-      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: nil).to_metric_card
-      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: nil).to_metric_card
+      # Get tag filter values from session
+      disabled_tags = session_disabled_tags
+      show_non_tagged = session[:show_non_tagged] != false
+
+      @average_query_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
 
       # Generate chart data for inline rendering
-      @average_response_time_chart_data = RailsPulse::Dashboard::Charts::AverageResponseTime.new.to_chart_data
-      @p95_response_time_chart_data = RailsPulse::Dashboard::Charts::P95ResponseTime.new.to_chart_data
+      @average_response_time_chart_data = RailsPulse::Dashboard::Charts::AverageResponseTime.new(disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_chart_data
+      @p95_response_time_chart_data = RailsPulse::Dashboard::Charts::P95ResponseTime.new(disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_chart_data
 
       # Generate table data for inline rendering
-      @slow_routes_table_data = RailsPulse::Dashboard::Tables::SlowRoutes.new.to_table_data
-      @slow_queries_table_data = RailsPulse::Dashboard::Tables::SlowQueries.new.to_table_data
+      @slow_routes_table_data = RailsPulse::Dashboard::Tables::SlowRoutes.new(disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_table_data
+      @slow_queries_table_data = RailsPulse::Dashboard::Tables::SlowQueries.new(disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_table_data
     end
   end
 end

data/app/controllers/rails_pulse/queries_controller.rb

@@ -70,7 +70,8 @@ module RailsPulse
     def build_chart_ransack_params(ransack_params)
       base_params = ransack_params.except(:s).merge(
         period_start_gteq: Time.at(@start_time),
-        period_start_lt: Time.at(@end_time)
+        period_start_lt: Time.at(@end_time),
+        summarizable_type_eq: "RailsPulse::Query"
       )
 
       # Only add duration filter if we have a meaningful threshold
@@ -78,8 +79,7 @@ module RailsPulse
 
       if show_action?
         base_params.merge(
-          summarizable_id_eq: @query.id,
-          summarizable_type_eq: "RailsPulse::Query"
+          summarizable_id_eq: @query.id
         )
       else
         base_params
@@ -123,7 +123,8 @@ module RailsPulse
           period_type: period_type,
           start_time: @start_time,
           params: params,
-          disabled_tags: session_disabled_tags
+          disabled_tags: session_disabled_tags,
+          show_non_tagged: session[:show_non_tagged] != false
         ).to_table
       end
     end
@@ -133,9 +134,13 @@ module RailsPulse
     def setup_metric_cards
       return if turbo_frame_request?
 
-      @average_query_times_metric_card = RailsPulse::Queries::Cards::AverageQueryTimes.new(query: @query).to_metric_card
-      @percentile_query_times_metric_card = RailsPulse::Queries::Cards::PercentileQueryTimes.new(query: @query).to_metric_card
-      @execution_rate_metric_card = RailsPulse::Queries::Cards::ExecutionRate.new(query: @query).to_metric_card
+      # Get tag filter values from session
+      disabled_tags = session_disabled_tags
+      show_non_tagged = session[:show_non_tagged] != false
+
+      @average_query_times_metric_card = RailsPulse::Queries::Cards::AverageQueryTimes.new(query: @query, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @percentile_query_times_metric_card = RailsPulse::Queries::Cards::PercentileQueryTimes.new(query: @query, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @execution_rate_metric_card = RailsPulse::Queries::Cards::ExecutionRate.new(query: @query, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
     end
 
     def show_action?

data/app/controllers/rails_pulse/requests_controller.rb

@@ -26,10 +26,14 @@ module RailsPulse
     def setup_metric_cards
       return if  turbo_frame_request?
 
-      @average_response_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: nil).to_metric_card
-      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: nil).to_metric_card
-      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: nil).to_metric_card
-      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: nil).to_metric_card
+      # Get tag filter values from session
+      disabled_tags = session_disabled_tags
+      show_non_tagged = session[:show_non_tagged] != false
+
+      @average_response_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: nil, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
     end
 
 

data/app/controllers/rails_pulse/routes_controller.rb

@@ -20,10 +20,14 @@ module RailsPulse
     def setup_metric_cards
       return if turbo_frame_request?
 
-      @average_query_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: @route).to_metric_card
-      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: @route).to_metric_card
-      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: @route).to_metric_card
-      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: @route).to_metric_card
+      # Get tag filter values from session
+      disabled_tags = session_disabled_tags
+      show_non_tagged = session[:show_non_tagged] != false
+
+      @average_query_times_metric_card = RailsPulse::Routes::Cards::AverageResponseTimes.new(route: @route, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @percentile_response_times_metric_card = RailsPulse::Routes::Cards::PercentileResponseTimes.new(route: @route, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @request_count_totals_metric_card = RailsPulse::Routes::Cards::RequestCountTotals.new(route: @route, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
+      @error_rate_per_route_metric_card = RailsPulse::Routes::Cards::ErrorRatePerRoute.new(route: @route, disabled_tags: disabled_tags, show_non_tagged: show_non_tagged).to_metric_card
     end
 
     def chart_model
@@ -87,7 +91,9 @@ module RailsPulse
       if show_action?
         # Only show requests that belong to time periods where we have route summaries
         # This ensures the table data is consistent with the chart data
-        base_query = apply_tag_filters(@ransack_query.result)
+        # Note: We don't apply tag filters here because we want to show all requests
+        # for this specific route, regardless of individual request tags
+        base_query = @ransack_query.result
           .joins(<<~SQL)
             INNER JOIN rails_pulse_summaries ON
               rails_pulse_summaries.summarizable_id = rails_pulse_requests.route_id AND
@@ -110,7 +116,8 @@ module RailsPulse
           period_type: period_type,
           start_time: @start_time,
           params: params,
-          disabled_tags: session_disabled_tags
+          disabled_tags: session_disabled_tags,
+          show_non_tagged: session[:show_non_tagged] != false
         ).to_table
       end
     end

data/app/models/concerns/rails_pulse/taggable.rb

@@ -0,0 +1,63 @@
+module RailsPulse
+  module Taggable
+    extend ActiveSupport::Concern
+
+    included do
+      # Callbacks
+      before_save :ensure_tags_is_array
+
+      # Scopes with table name qualification to avoid ambiguity
+      scope :with_tag, ->(tag) { where("#{table_name}.tags LIKE ?", "%#{tag}%") }
+      scope :without_tag, ->(tag) { where.not("#{table_name}.tags LIKE ?", "%#{tag}%") }
+      scope :with_tags, -> { where("#{table_name}.tags IS NOT NULL AND #{table_name}.tags != '[]'") }
+    end
+
+    # Tag management methods
+    def tag_list
+      parsed_tags || []
+    end
+
+    def tag_list=(value)
+      self.tags = value.to_json
+    end
+
+    def has_tag?(tag)
+      tag_list.include?(tag.to_s)
+    end
+
+    def add_tag(tag)
+      current_tags = tag_list
+      unless current_tags.include?(tag.to_s)
+        current_tags << tag.to_s
+        self.tag_list = current_tags
+        save
+      end
+    end
+
+    def remove_tag(tag)
+      current_tags = tag_list
+      if current_tags.include?(tag.to_s)
+        current_tags.delete(tag.to_s)
+        self.tag_list = current_tags
+        save
+      end
+    end
+
+    private
+
+    def parsed_tags
+      return [] if tags.nil? || tags.empty?
+      JSON.parse(tags)
+    rescue JSON::ParserError
+      []
+    end
+
+    def ensure_tags_is_array
+      if tags.nil?
+        self.tags = "[]"
+      elsif tags.is_a?(Array)
+        self.tags = tags.to_json
+      end
+    end
+  end
+end

data/app/models/rails_pulse/dashboard/charts/average_response_time.rb

@@ -2,6 +2,11 @@ module RailsPulse
   module Dashboard
     module Charts
       class AverageResponseTime
+        def initialize(disabled_tags: [], show_non_tagged: true)
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
+        end
+
         def to_chart_data
           # Create a range of all dates in the past 2 weeks
           start_date = 2.weeks.ago.beginning_of_day.to_date
@@ -9,11 +14,13 @@ module RailsPulse
           date_range = (start_date..end_date)
 
           # Get the actual data from Summary records (routes)
-          summaries = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Route",
-            period_type: "day",
-            period_start: start_date.beginning_of_day..end_date.end_of_day
-          )
+          summaries = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Route",
+              period_type: "day",
+              period_start: start_date.beginning_of_day..end_date.end_of_day
+            )
 
           # Group by day manually for cross-database compatibility
           actual_data = {}

data/app/models/rails_pulse/dashboard/charts/p95_response_time.rb

@@ -2,6 +2,11 @@ module RailsPulse
   module Dashboard
     module Charts
       class P95ResponseTime
+        def initialize(disabled_tags: [], show_non_tagged: true)
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
+        end
+
         def to_chart_data
           # Create a range of all dates in the past 2 weeks
           start_date = 2.weeks.ago.beginning_of_day.to_date
@@ -9,11 +14,13 @@ module RailsPulse
           date_range = (start_date..end_date)
 
           # Get the actual data from Summary records (queries for P95)
-          summaries = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Query",
-            period_type: "day",
-            period_start: start_date.beginning_of_day..end_date.end_of_day
-          )
+          summaries = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Query",
+              period_type: "day",
+              period_start: start_date.beginning_of_day..end_date.end_of_day
+            )
 
           actual_data = summaries
             .group_by_day(:period_start, time_zone: Time.zone)

data/app/models/rails_pulse/dashboard/tables/slow_queries.rb

@@ -3,6 +3,12 @@ module RailsPulse
     module Tables
       class SlowQueries
         include RailsPulse::FormattingHelper
+
+        def initialize(disabled_tags: [], show_non_tagged: true)
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
+        end
+
         def to_table_data
           # Get data for this week
           this_week_start = 1.week.ago.beginning_of_week
@@ -10,6 +16,7 @@ module RailsPulse
 
           # Fetch query data from Summary records for this week
           query_data = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
             .joins("INNER JOIN rails_pulse_queries ON rails_pulse_queries.id = rails_pulse_summaries.summarizable_id")
             .where(
               summarizable_type: "RailsPulse::Query",

data/app/models/rails_pulse/dashboard/tables/slow_routes.rb

@@ -4,6 +4,11 @@ module RailsPulse
       class SlowRoutes
         include RailsPulse::FormattingHelper
 
+        def initialize(disabled_tags: [], show_non_tagged: true)
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
+        end
+
         def to_table_data
           # Get data for this week
           this_week_start = 1.week.ago.beginning_of_week
@@ -11,6 +16,7 @@ module RailsPulse
 
           # Fetch route data from Summary records for this week
           route_data = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
             .joins("INNER JOIN rails_pulse_routes ON rails_pulse_routes.id = rails_pulse_summaries.summarizable_id")
             .where(
               summarizable_type: "RailsPulse::Route",

data/app/models/rails_pulse/queries/cards/average_query_times.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Queries
     module Cards
       class AverageQueryTimes
-        def initialize(query: nil)
+        def initialize(query: nil, disabled_tags: [], show_non_tagged: true)
           @query = query
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all aggregated metrics with conditional sums
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Query",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Query",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           metrics = base_query.select(

data/app/models/rails_pulse/queries/cards/execution_rate.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Queries
     module Cards
       class ExecutionRate
-        def initialize(query: nil)
+        def initialize(query: nil, disabled_tags: [], show_non_tagged: true)
           @query = query
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -12,20 +14,24 @@ module RailsPulse
 
           # Get the most common period type for this query, or fall back to "day"
           period_type = if @query
-            RailsPulse::Summary.where(
-              summarizable_type: "RailsPulse::Query",
-              summarizable_id: @query.id
-            ).group(:period_type).count.max_by(&:last)&.first || "day"
+            RailsPulse::Summary
+              .with_tag_filters(@disabled_tags, @show_non_tagged)
+              .where(
+                summarizable_type: "RailsPulse::Query",
+                summarizable_id: @query.id
+              ).group(:period_type).count.max_by(&:last)&.first || "day"
           else
             "day"
           end
 
           # Single query to get all count metrics with conditional aggregation
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Query",
-            period_type: period_type,
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Query",
+              period_type: period_type,
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           metrics = base_query.select(

data/app/models/rails_pulse/queries/cards/percentile_query_times.rb

@@ -2,8 +2,10 @@ module RailsPulse
   module Queries
     module Cards
       class PercentileQueryTimes
-        def initialize(query: nil)
+        def initialize(query: nil, disabled_tags: [], show_non_tagged: true)
           @query = query
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # Single query to get all P95 metrics with conditional aggregation
-          base_query = RailsPulse::Summary.where(
-            summarizable_type: "RailsPulse::Query",
-            period_type: "day",
-            period_start: 2.weeks.ago.beginning_of_day..Time.current
-          )
+          base_query = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Query",
+              period_type: "day",
+              period_start: 2.weeks.ago.beginning_of_day..Time.current
+            )
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           metrics = base_query.select(

data/app/models/rails_pulse/queries/charts/average_query_times.rb

@@ -2,18 +2,21 @@ module RailsPulse
   module Queries
     module Charts
       class AverageQueryTimes
-        def initialize(ransack_query:, period_type: nil, query: nil, start_time: nil, end_time: nil, start_duration: nil)
+        def initialize(ransack_query:, period_type: nil, query: nil, start_time: nil, end_time: nil, start_duration: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @query = query
           @start_time = start_time
           @end_time = end_time
           @start_duration = start_duration
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_rails_chart
-          # The ransack query already contains the correct filters, just add period_type
+          # The ransack query already contains the correct filters, just add period_type and tag filters
           summaries = @ransack_query.result(distinct: false)
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
             .where(period_type: @period_type)
             .group(:period_start)
             .having("AVG(avg_duration) > ?", @start_duration || 0)

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

@@ -2,13 +2,14 @@ module RailsPulse
   module Queries
     module Tables
       class Index
-        def initialize(ransack_query:, period_type: nil, start_time:, params:, query: nil, disabled_tags: [])
+        def initialize(ransack_query:, period_type: nil, start_time:, params:, query: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @start_time = start_time
           @params = params
           @query = query
           @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_table
@@ -23,10 +24,19 @@ module RailsPulse
             )
 
           # Apply tag filters by excluding queries with disabled tags
-          @disabled_tags.each do |tag|
+          # Separate "non_tagged" from actual tags (it's a virtual tag)
+          actual_disabled_tags = @disabled_tags.reject { |tag| tag == "non_tagged" }
+
+          # Exclude queries with actual disabled tags
+          actual_disabled_tags.each do |tag|
             base_query = base_query.where.not("rails_pulse_queries.tags LIKE ?", "%#{tag}%")
           end
 
+          # Exclude non-tagged queries if show_non_tagged is false
+          unless @show_non_tagged
+            base_query = base_query.where("rails_pulse_queries.tags IS NOT NULL AND rails_pulse_queries.tags != '[]'")
+          end
+
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           # Apply grouping and aggregation

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

@@ -2,21 +2,27 @@ module RailsPulse
   module Requests
     module Charts
       class AverageResponseTimes
-        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil)
+        def initialize(ransack_query:, period_type: nil, route: nil, start_time: nil, end_time: nil, start_duration: nil, disabled_tags: [], show_non_tagged: true)
           @ransack_query = ransack_query
           @period_type = period_type
           @route = route
           @start_time = start_time
           @end_time = end_time
           @start_duration = start_duration
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_rails_chart
-          summaries = @ransack_query.result(distinct: false).where(
-            summarizable_type: "RailsPulse::Request",
-            summarizable_id: 0,  # Overall request summaries
-            period_type: @period_type
-          )
+          # Note: Overall request summaries (summarizable_id: 0) are not filtered by tags
+          # as they aggregate all requests regardless of route tags
+          summaries = @ransack_query.result(distinct: false)
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .where(
+              summarizable_type: "RailsPulse::Request",
+              summarizable_id: 0,  # Overall request summaries
+              period_type: @period_type
+            )
 
           summaries = summaries
             .group(:period_start)

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

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class AverageResponseTimes
-        def initialize(route:)
+        def initialize(route:, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.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 = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .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(

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

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class ErrorRatePerRoute
-        def initialize(route: nil)
+        def initialize(route: nil, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.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 = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .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(

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

@@ -2,8 +2,10 @@ module RailsPulse
   module Routes
     module Cards
       class PercentileResponseTimes
-        def initialize(route: nil)
+        def initialize(route: nil, disabled_tags: [], show_non_tagged: true)
           @route = route
+          @disabled_tags = disabled_tags
+          @show_non_tagged = show_non_tagged
         end
 
         def to_metric_card
@@ -11,11 +13,13 @@ module RailsPulse
           previous_7_days = 14.days.ago.beginning_of_day
 
           # 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 = RailsPulse::Summary
+            .with_tag_filters(@disabled_tags, @show_non_tagged)
+            .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(