rails_pulse 0.1.4 → 0.2.3

data/app/javascript/rails_pulse/controllers/custom_range_controller.js

@@ -0,0 +1,115 @@
+import { Controller } from "@hotwired/stimulus"
+
+export default class extends Controller {
+  static targets = ["selectWrapper", "pickerWrapper"]
+
+  connect() {
+    const selectElement = this.selectWrapperTarget.querySelector('select')
+    if (!selectElement) return
+
+    // Check if we're in recent_custom mode
+    const mode = selectElement.dataset.mode
+
+    if (mode === 'recent_custom') {
+      // In recent_custom mode, "recent" shows no picker, "custom" shows picker
+      if (selectElement.value === "custom") {
+        this.showPicker()
+        this.initializeDatePicker()
+      }
+    } else {
+      // In preset mode, "custom" shows picker
+      if (selectElement.value === "custom") {
+        this.showPicker()
+        this.initializeDatePicker()
+      }
+    }
+  }
+
+  // When time range is selected from dropdown
+  handleChange(event) {
+    const mode = event.target.dataset.mode
+
+    if (mode === 'recent_custom') {
+      // In recent_custom mode
+      if (event.target.value === "custom") {
+        this.showPicker()
+        this.openDatePicker()
+      } else {
+        // "recent" is selected - hide picker
+        this.pickerWrapperTarget.style.display = "none"
+        this.selectWrapperTarget.style.display = "block"
+      }
+    } else {
+      // In preset mode
+      if (event.target.value === "custom") {
+        this.showPicker()
+        this.openDatePicker()
+      }
+    }
+  }
+
+  // Show picker, hide select
+  showPicker() {
+    this.selectWrapperTarget.style.display = "none"
+    this.pickerWrapperTarget.style.display = "flex"
+  }
+
+  // Open the flatpickr calendar
+  openDatePicker() {
+    // Wait a bit for the DOM to update and flatpickr to initialize
+    setTimeout(() => {
+      // Find the original hidden input that has the datepicker controller
+      const hiddenInput = this.pickerWrapperTarget.querySelector('input[name*="custom_date_range"]')
+      if (!hiddenInput) return
+
+      // Get the datepicker controller from the hidden input
+      const datepickerController = this.application.getControllerForElementAndIdentifier(
+        hiddenInput,
+        'rails-pulse--datepicker'
+      )
+
+      if (datepickerController && datepickerController.flatpickr) {
+        datepickerController.flatpickr.open()
+      }
+    }, 50)
+  }
+
+  // Show select, hide picker
+  showSelect() {
+    this.pickerWrapperTarget.style.display = "none"
+    this.selectWrapperTarget.style.display = "block"
+
+    // Reset select to default value based on mode
+    const selectElement = this.selectWrapperTarget.querySelector('select')
+    if (selectElement) {
+      const mode = selectElement.dataset.mode
+      if (mode === 'recent_custom') {
+        selectElement.value = "recent"
+      } else {
+        selectElement.value = "last_day"
+      }
+    }
+  }
+
+  // Initialize flatpickr with existing date value
+  initializeDatePicker() {
+    const dateInput = this.pickerWrapperTarget.querySelector('input[type="text"]')
+    if (!dateInput || !dateInput.value) return
+
+    // Get the datepicker controller
+    const datepickerController = this.application.getControllerForElementAndIdentifier(
+      dateInput,
+      'rails-pulse--datepicker'
+    )
+
+    if (datepickerController && datepickerController.flatpickr) {
+      const value = dateInput.value
+      // Parse the "start to end" format
+      if (value.includes(' to ')) {
+        const [start, end] = value.split(' to ').map(d => d.trim())
+        // Set the dates in flatpickr
+        datepickerController.flatpickr.setDate([start, end], false)
+      }
+    }
+  }
+}

data/app/javascript/rails_pulse/controllers/datepicker_controller.js

@@ -0,0 +1,48 @@
+import { Controller } from "@hotwired/stimulus"
+import flatpickr from "flatpickr"
+
+export default class extends Controller {
+  static targets = [ "details" ]
+  static values  = {
+    type: String, disable: Array,
+    mode: { type: String, default: "single" },
+    showMonths: { type: Number, default: 1 },
+    dateFormat: { type: String, default: "F d, Y" },
+    dateTimeFormat: { type: String, default: "M d, Y h:i K" }
+  }
+
+  connect() {
+    if (this.typeValue == "time") {
+      this.flatpickr = flatpickr(this.element, this.#timeOptions)
+    } else if (this.typeValue == "datetime") {
+      this.flatpickr = flatpickr(this.element, this.#dateTimeOptions)
+    } else {
+      this.flatpickr = flatpickr(this.element, this.#basicOptions)
+    }
+  }
+
+  disconnect() {
+    this.flatpickr.destroy()
+  }
+
+  get #timeOptions() {
+    return { dateFormat: "H:i", enableTime: true, noCalendar: true }
+  }
+
+  get #dateTimeOptions() {
+    return { ...this.#baseOptions, altFormat: this.dateTimeFormatValue, dateFormat: "Y-m-d H:i", enableTime: true }
+  }
+
+  get #basicOptions() {
+    return { ...this.#baseOptions, altFormat: this.dateFormatValue, dateFormat: "Y-m-d" }
+  }
+
+  get #baseOptions() {
+    return {
+      altInput: true,
+      disable: this.disableValue,
+      mode: this.modeValue,
+      showMonths: this.showMonthsValue
+    }
+  }
+}

data/app/javascript/rails_pulse/controllers/global_filters_controller.js

@@ -0,0 +1,110 @@
+import { Controller } from "@hotwired/stimulus"
+
+export default class extends Controller {
+  static targets = ["wrapper", "dialog", "dateRange", "indicator", "form"]
+  static values = {
+    active: { type: Boolean, default: false }
+  }
+
+  connect() {
+    this.updateIndicator()
+  }
+
+  // Open the global filters dialog
+  open(event) {
+    event.preventDefault()
+
+    // If there's a value in the date range input, make sure flatpickr knows about it
+    if (this.dateRangeTarget.value) {
+      const datepickerController = this.application.getControllerForElementAndIdentifier(
+        this.dateRangeTarget,
+        'rails-pulse--datepicker'
+      )
+
+      if (datepickerController && datepickerController.flatpickr) {
+        const value = this.dateRangeTarget.value
+        // Parse the "start to end" format
+        if (value.includes(' to ')) {
+          const [start, end] = value.split(' to ').map(d => d.trim())
+          // Set the dates in flatpickr
+          datepickerController.flatpickr.setDate([start, end], false)
+        }
+      }
+    }
+
+    this.wrapperTarget.style.display = 'flex'
+    // Prevent body scroll when dialog is open
+    document.body.style.overflow = 'hidden'
+  }
+
+  // Close the dialog
+  close(event) {
+    if (event) {
+      event.preventDefault()
+    }
+    this.wrapperTarget.style.display = 'none'
+    // Restore body scroll
+    document.body.style.overflow = ''
+  }
+
+  // Close dialog when clicking outside
+  closeOnClickOutside(event) {
+    if (event.target === this.wrapperTarget) {
+      this.close(event)
+    }
+  }
+
+  // Handle form submission - parse date range and add individual params
+  submit(event) {
+    // If clear button was clicked, let it through as-is
+    if (event.submitter && event.submitter.name === "clear") {
+      return
+    }
+
+    const dateRangeValue = this.dateRangeTarget.value
+    const form = event.target
+
+    // Parse date range if provided
+    if (dateRangeValue && dateRangeValue.includes(' to ')) {
+      const [startTime, endTime] = dateRangeValue.split(' to ').map(d => d.trim())
+
+      // Remove any existing hidden inputs
+      form.querySelectorAll('input[name="start_time"], input[name="end_time"]').forEach(el => el.remove())
+
+      // Add new hidden inputs
+      const startInput = document.createElement('input')
+      startInput.type = 'hidden'
+      startInput.name = 'start_time'
+      startInput.value = startTime
+      form.appendChild(startInput)
+
+      const endInput = document.createElement('input')
+      endInput.type = 'hidden'
+      endInput.name = 'end_time'
+      endInput.value = endTime
+      form.appendChild(endInput)
+    }
+
+    // Tag switches are already being submitted as enabled_tags[]
+    // The controller will convert these to disabled_tags
+    // No additional processing needed here
+
+    // No validation needed - user can apply any combination of filters
+  }
+
+  // Update visual indicator based on activeValue
+  updateIndicator() {
+    if (this.hasIndicatorTarget) {
+      if (this.activeValue) {
+        this.indicatorTarget.classList.add("global-filters-active")
+      } else {
+        this.indicatorTarget.classList.remove("global-filters-active")
+      }
+    }
+  }
+
+  // Called when activeValue changes
+  activeValueChanged() {
+    this.updateIndicator()
+  }
+}

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,12 +2,14 @@ module RailsPulse
   module Queries
     module Tables
       class Index
-        def initialize(ransack_query:, period_type: nil, start_time:, params:, query: nil)
+        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
@@ -21,6 +23,20 @@ module RailsPulse
               period_type: @period_type
             )
 
+          # Apply tag filters by excluding queries with disabled tags
+          # 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
@@ -29,13 +45,15 @@ module RailsPulse
               "rails_pulse_summaries.summarizable_id",
               "rails_pulse_summaries.summarizable_type",
               "rails_pulse_queries.id",
-              "rails_pulse_queries.normalized_sql"
+              "rails_pulse_queries.normalized_sql",
+              "rails_pulse_queries.tags"
             )
             .select(
               "rails_pulse_summaries.summarizable_id",
               "rails_pulse_summaries.summarizable_type",
               "rails_pulse_queries.id as query_id",
               "rails_pulse_queries.normalized_sql",
+              "rails_pulse_queries.tags",
               "AVG(rails_pulse_summaries.avg_duration) as avg_duration",
               "MAX(rails_pulse_summaries.max_duration) as max_duration",
               "SUM(rails_pulse_summaries.count) as execution_count",

data/app/models/rails_pulse/query.rb

@@ -1,5 +1,7 @@
 module RailsPulse
   class Query < RailsPulse::ApplicationRecord
+    include Taggable
+
     self.table_name = "rails_pulse_queries"
 
     # Associations

data/app/models/rails_pulse/request.rb

@@ -1,5 +1,7 @@
 module RailsPulse
   class Request < RailsPulse::ApplicationRecord
+    include Taggable
+
     self.table_name = "rails_pulse_requests"
 
     # Associations
@@ -17,7 +19,7 @@ module RailsPulse
     before_create :set_request_uuid
 
     def self.ransackable_attributes(auth_object = nil)
-      %w[id route_id occurred_at duration status status_indicator route_path]
+      %w[id route_id occurred_at duration status status_category status_indicator route_path]
     end
 
     def self.ransackable_associations(auth_object = nil)
@@ -32,6 +34,12 @@ module RailsPulse
       Arel.sql("rails_pulse_routes.path")
     end
 
+    ransacker :status_category do |parent|
+      # Returns the first digit of the status code (2, 3, 4, or 5)
+      # Use FLOOR instead of CAST for cross-database compatibility
+      Arel.sql("FLOOR(#{parent.table[:status].name} / 100)")
+    end
+
     ransacker :status_indicator do |parent|
       # Calculate status indicator based on request_thresholds with safe defaults
       config = RailsPulse.configuration rescue nil