rails_pulse 0.1.4 → 0.2.2

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/taggable.rb

@@ -0,0 +1,61 @@
+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

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

@@ -2,12 +2,13 @@ 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: [])
           @ransack_query = ransack_query
           @period_type = period_type
           @start_time = start_time
           @params = params
           @query = query
+          @disabled_tags = disabled_tags
         end
 
         def to_table
@@ -21,6 +22,11 @@ module RailsPulse
               period_type: @period_type
             )
 
+          # Apply tag filters by excluding queries with disabled tags
+          @disabled_tags.each do |tag|
+            base_query = base_query.where.not("rails_pulse_queries.tags LIKE ?", "%#{tag}%")
+          end
+
           base_query = base_query.where(summarizable_id: @query.id) if @query
 
           # Apply grouping and aggregation
@@ -29,13 +35,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

data/app/models/rails_pulse/route.rb

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

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

@@ -2,11 +2,12 @@ module RailsPulse
   module Routes
     module Tables
       class Index
-        def initialize(ransack_query:, period_type: nil, start_time:, params:)
+        def initialize(ransack_query:, period_type: nil, start_time:, params:, disabled_tags: [])
           @ransack_query = ransack_query
           @period_type = period_type
           @start_time = start_time
           @params = params
+          @disabled_tags = disabled_tags
         end
 
         def to_table
@@ -22,6 +23,11 @@ module RailsPulse
               period_type: @period_type
             )
 
+          # Apply tag filters by excluding routes with disabled tags
+          @disabled_tags.each do |tag|
+            base_query = base_query.where.not("rails_pulse_routes.tags LIKE ?", "%#{tag}%")
+          end
+
           base_query = base_query.where(summarizable_id: @route.id) if @route
 
           # Apply grouping and aggregation
@@ -31,7 +37,8 @@ module RailsPulse
               "rails_pulse_summaries.summarizable_type",
               "rails_pulse_routes.id",
               "rails_pulse_routes.path",
-              "rails_pulse_routes.method"
+              "rails_pulse_routes.method",
+              "rails_pulse_routes.tags"
             )
             .select(
               "rails_pulse_summaries.summarizable_id",
@@ -39,6 +46,7 @@ module RailsPulse
               "rails_pulse_routes.id as route_id",
               "rails_pulse_routes.path",
               "rails_pulse_routes.method as route_method",
+              "rails_pulse_routes.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 count",

data/app/services/rails_pulse/summary_service.rb

@@ -74,6 +74,7 @@ module RailsPulse
       route_groups = Request
         .where(occurred_at: start_time...end_time)
         .where.not(route_id: nil)
+        .joins(:route)
         .group(:route_id)
 
       # Calculate basic aggregates
@@ -132,6 +133,7 @@ module RailsPulse
       query_groups = Operation
         .where(occurred_at: start_time...end_time)
         .where.not(query_id: nil)
+        .joins(:query)
         .group(:query_id)
 
       basic_stats = query_groups.pluck(

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

@@ -0,0 +1,84 @@
+<% global_filters = session_global_filters %>
+<% has_global_filters = global_filters['start_time'].present? && global_filters['end_time'].present? %>
+<% has_performance_filter = global_filters['performance_threshold'].present? %>
+<% disabled_tags = global_filters['disabled_tags'] || [] %>
+<% has_tag_filters = disabled_tags.any? %>
+<% has_any_filters = has_global_filters || has_performance_filter  %>
+<% current_date_range = has_global_filters ? "#{global_filters['start_time']} to #{global_filters['end_time']}" : "" %>
+<% current_threshold = global_filters['performance_threshold'] %>
+
+<div data-controller="rails-pulse--global-filters" data-rails-pulse--global-filters-active-value="<%= has_any_filters %>">
+  <%= link_to '#', "aria-label": "Global filters", role: "button", data: { action: "rails-pulse--global-filters#open", "rails-pulse--global-filters-target": "indicator" } do %>
+    <%= rails_pulse_icon has_any_filters ? 'list-filter-plus' : 'list-filter', width: '20' %>
+  <% end %>
+
+  <div data-rails-pulse--global-filters-target="wrapper" data-action="click->rails-pulse--global-filters#closeOnClickOutside" style="display: none; position: fixed; inset: 0; background-color: rgba(0, 0, 0, 0.8); z-index: 1000; align-items: center; justify-content: center;">
+    <div class="dialog" data-rails-pulse--global-filters-target="dialog" style="position: relative; opacity: 1; transform: scale(1);">
+      <div class="dialog__content">
+        <h2 class="text-lg font-semibold mb-4">Global Filters</h2>
+        <p class="text-sm text-subtle mb-4">Set default time filters that persist across all pages. These can be overridden by page-specific filters.</p>
+
+        <%= form_with url: settings_global_filters_path, method: :patch, local: true, data: { action: "submit->rails-pulse--global-filters#submit" } do |form| %>
+          <div class="flex flex-col gap mb-4">
+            <div class="flex flex-col gap" style="--row-gap: 0.5rem">
+              <label for="global_date_range" class="text-sm font-medium">Date Range</label>
+              <input
+                type="text"
+                name="date_range"
+                id="global_date_range"
+                value="<%= current_date_range %>"
+                placeholder="Pick date range"
+                class="input"
+                data-controller="rails-pulse--datepicker"
+                data-rails-pulse--datepicker-mode-value="range"
+                data-rails-pulse--datepicker-show-months-value="2"
+                data-rails-pulse--datepicker-type-value="datetime"
+                data-rails-pulse--global-filters-target="dateRange"
+              />
+            </div>
+
+            <div class="flex flex-col gap" style="--row-gap: 0.5rem">
+              <label for="global_performance_threshold" class="text-sm font-medium">Performance Threshold</label>
+              <select name="performance_threshold" id="global_performance_threshold" class="input">
+                <option value="">All Requests</option>
+                <option value="slow" <%= 'selected' if current_threshold == 'slow' %>>Slow and Above</option>
+                <option value="very_slow" <%= 'selected' if current_threshold == 'very_slow' %>>Very Slow and Above</option>
+                <option value="critical" <%= 'selected' if current_threshold == 'critical' %>>Critical Only</option>
+              </select>
+              <p class="text-xs text-subtle">Filter requests/queries by performance threshold</p>
+            </div>
+
+            <div class="flex flex-col gap" style="--row-gap: 0.5rem">
+              <label class="text-sm font-medium">Tag Visibility</label>
+              <p class="text-xs text-subtle">Toggle visibility of tagged data. Unchecked tags will be hidden from all views.</p>
+              <div class="flex flex-col gap" style="--row-gap: 0.75rem">
+                <% tags = RailsPulse.configuration.tags + ["non_tagged"] %>
+                <% tags.each do |tag| %>
+                  <div class="flex items-center gap">
+                    <input
+                      type="checkbox"
+                      name="enabled_tags[]"
+                      id="tag_<%= tag %>"
+                      value="<%= tag %>"
+                      <%= 'checked="checked"' if tag == "non_tagged" ? session[:show_non_tagged] != false : !disabled_tags.include?(tag) %>
+                      class="switch"
+                      role="switch"
+                    />
+                    <label class="text-sm font-medium" for="tag_<%= tag %>">
+                      <%= tag.humanize %>
+                    </label>
+                  </div>
+                <% end %>
+              </div>
+            </div>
+          </div>
+
+          <div class="flex items-center justify-end gap">
+            <%= form.button "Clear", type: "submit", name: "clear", value: "true", class: "btn btn--borderless", formnovalidate: true %>
+            <%= form.submit "Apply Filters", class: "btn" %>
+          </div>
+        <% end %>
+      </div>
+    </div>
+  </div>
+</div>

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

@@ -8,12 +8,12 @@
   <span class="overflow-ellipsis">Routes</span>
 <% end %>
 
-<%= link_to requests_path, class: 'btn sidebar-menu__button' do %>
-  <%= rails_pulse_icon 'audio-lines', width: '16' %>
-  <span class="overflow-ellipsis">Requests</span>
-<% end %>
-
 <%= link_to queries_path, class: 'btn sidebar-menu__button' do %>
   <%= rails_pulse_icon 'database', width: '16' %>
   <span class="overflow-ellipsis">Queries</span>
 <% end %>
+
+<%= link_to requests_path, class: 'btn sidebar-menu__button' do %>
+  <%= rails_pulse_icon 'audio-lines', width: '16' %>
+  <span class="overflow-ellipsis">Requests</span>
+<% end %>

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

@@ -53,10 +53,14 @@
         </nav>
       </div>
 
-      <div class="flex items-center" data-controller="rails-pulse--color-scheme">
-        <%= link_to '#', "aria-label": "Toggle color scheme", role: "button", data: { action: "rails-pulse--color-scheme#toggle" } do %>
-          <%= rails_pulse_icon 'sun', width: '48' %>
-        <% end %>
+      <div class="flex items-center gap" style="--column-gap: 0.5rem">
+        <%= render 'layouts/rails_pulse/global_filters' %>
+
+        <div data-controller="rails-pulse--color-scheme">
+          <%= link_to '#', "aria-label": "Toggle color scheme", role: "button", data: { action: "rails-pulse--color-scheme#toggle" } do %>
+            <%= rails_pulse_icon 'sun', width: '48' %>
+          <% end %>
+        </div>
       </div>
     </header>
 
@@ -65,6 +69,5 @@
         <%= yield %>
       </div>
     </main>
-
   </body>
 </html>

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

@@ -0,0 +1,20 @@
+<div class="breadcrumb-container">
+  <nav class="breadcrumb mis-2 mbs-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>
+      <% else %>
+        <%= link_to crumb[:title], crumb[:path] %>
+      <% end %>
+      <% unless index == breadcrumbs.length - 1 %>
+        <%= rails_pulse_icon 'chevron-right', width: 14, height: 14, class: "breadcrumb-separator mbe-1" %>
+      <% end %>
+    <% end %>
+  </nav>
+
+  <% if defined?(taggable) && taggable.present? %>
+    <div class="breadcrumb-tags">
+      <%= render 'rails_pulse/tags/tag_manager', taggable: taggable %>
+    </div>
+  <% end %>
+</div>

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

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/breadcrumbs' %>
+<%= render 'rails_pulse/components/page_header' %>
 
 <h1>Operation <%= @operation.id %></h1>
 

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

@@ -1,7 +1,8 @@
 <% columns = [
   { field: :normalized_sql, label: 'Query', class: 'w-auto' },
   { field: :avg_duration_sort, label: 'Average Query Time', class: 'w-44' },
-  { field: :execution_count_sort, label: 'Executions', class: 'w-24' }
+  { field: :execution_count_sort, label: 'Executions', class: 'w-24' },
+  { field: nil, label: 'Tags', class: 'w-32' }
 ] %>
 
 <table class="table mbs-4" data-controller="rails-pulse--table-sort">
@@ -17,6 +18,7 @@
         </td>
         <td class="whitespace-nowrap"><%= summary.avg_duration.to_i %> ms</td>
         <td class="whitespace-nowrap"><%= number_with_delimiter summary.execution_count %></td>
+        <td class="whitespace-nowrap"><%= display_tag_badges(summary.tags) %></td>
       </tr>
     <% end %>
   </tbody>

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

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/breadcrumbs' %>
+<%= render 'rails_pulse/components/page_header' %>
 
 <% unless turbo_frame_request? %>
   <div class="row">
@@ -13,13 +13,9 @@
   data-rails-pulse--index-chart-id-value="average_query_times_chart"
 >
   <%= render 'rails_pulse/components/panel', { title: 'Average Query Time', } do %>
-    <%= search_form_for @ransack_query, url: queries_path, class: "flex items-center justify-between gap mb-4" do |form| %>
+    <%= search_form_for @ransack_query, url: queries_path, class: "flex items-center justify-between gap mb-4", data: { controller: "rails-pulse--custom-range" } do |form| %>
       <div class="flex items-center grow gap">
-        <%= form.select :period_start_range,
-          RailsPulse::QueriesController::TIME_RANGE_OPTIONS,
-          { selected: @selected_time_range },
-          { class: "input" }
-        %>
+        <%= time_range_selector(form, time_range_options: RailsPulse::QueriesController::TIME_RANGE_OPTIONS, selected_time_range: @selected_time_range) %>
         <%= form.select :avg_duration,
           duration_options(:query),
           { selected: @selected_response_range },

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

@@ -1,4 +1,4 @@
-<%= render 'rails_pulse/components/breadcrumbs' %>
+<%= render 'rails_pulse/components/page_header', taggable: @query %>
 
 <% unless turbo_frame_request? %>
   <div class="row">
@@ -13,13 +13,9 @@
   data-rails-pulse--index-chart-id-value="query_responses_chart"
 >
   <%= render 'rails_pulse/components/panel', { title: 'Query Responses' } do %>
-    <%= search_form_for @ransack_query, url: query_path(@query), class: "flex items-center justify-between gap mb-4" do |form| %>
+    <%= search_form_for @ransack_query, url: query_path(@query), class: "flex items-center justify-between gap mb-4", data: { controller: "rails-pulse--custom-range" } do |form| %>
       <div class="flex items-center grow gap">
-        <%= form.select :period_start_range,
-          RailsPulse::RoutesController::TIME_RANGE_OPTIONS,
-          { selected: @selected_time_range },
-          { class: "input" }
-        %>
+        <%= time_range_selector(form, time_range_options: RailsPulse::RoutesController::TIME_RANGE_OPTIONS, selected_time_range: @selected_time_range) %>
         <%= form.select :duration,
           duration_options(:query),
           { selected: @selected_response_range },

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

@@ -2,7 +2,8 @@
   { field: :occurred_at, label: 'Timestamp', class: 'w-36' },
   { field: :route_path, label: 'Route', class: 'w-auto' },
   { field: :duration, label: 'Response Time', class: 'w-36' },
-  { field: :status, label: 'Status', class: 'w-20' }
+  { field: :status, label: 'Status', class: 'w-20' },
+  { field: nil, label: 'Tags', class: 'w-32' }
 ] %>
 
 <table class="table mbs-4" data-controller="rails-pulse--table-sort">
@@ -34,6 +35,7 @@
         <td class="whitespace-nowrap">
           <span class="text-green-600"><%= request.status %></span>
         </td>
+        <td class="whitespace-nowrap"><%= display_tag_badges(request) %></td>
       </tr>
     <% end %>
   </tbody>