rails_error_dashboard 0.1.3 → 0.1.4

data/app/controllers/rails_error_dashboard/errors_controller.rb

@@ -146,6 +146,22 @@ module RailsErrorDashboard
       @resolution_rate = analytics[:resolution_rate]
       @mobile_errors = analytics[:mobile_errors]
       @api_errors = analytics[:api_errors]
+
+      # Get recurring issues data
+      recurring = Queries::RecurringIssues.call(days)
+      @recurring_data = recurring
+
+      # Get release correlation data
+      correlation = Queries::ErrorCorrelation.new(days: days)
+      @errors_by_version = correlation.errors_by_version
+      @problematic_releases = correlation.problematic_releases
+      @release_comparison = calculate_release_comparison
+
+      # Get MTTR data
+      mttr_data = Queries::MttrStats.call(days)
+      @mttr_stats = mttr_data
+      @overall_mttr = mttr_data[:overall_mttr]
+      @mttr_by_platform = mttr_data[:mttr_by_platform]
     end
 
     def platform_comparison
@@ -221,6 +237,26 @@ module RailsErrorDashboard
 
     private
 
+    def calculate_release_comparison
+      return {} if @errors_by_version.empty? || @errors_by_version.count < 2
+
+      versions_sorted = @errors_by_version.sort_by { |_, data| data[:last_seen] || Time.at(0) }.reverse
+      latest = versions_sorted.first
+      previous = versions_sorted.second
+
+      return {} if latest.nil? || previous.nil?
+
+      {
+        latest_version: latest[0],
+        latest_count: latest[1][:count],
+        latest_critical: latest[1][:critical_count],
+        previous_version: previous[0],
+        previous_count: previous[1][:count],
+        previous_critical: previous[1][:critical_count],
+        change_percentage: previous[1][:count] > 0 ? ((latest[1][:count] - previous[1][:count]).to_f / previous[1][:count] * 100).round(1) : 0.0
+      }
+    end
+
     def filter_params
       {
         error_type: params[:error_type],
@@ -228,11 +264,16 @@ module RailsErrorDashboard
         platform: params[:platform],
         search: params[:search],
         severity: params[:severity],
+        timeframe: params[:timeframe],
+        frequency: params[:frequency],
         # Phase 3: Workflow filter params
         status: params[:status],
         assigned_to: params[:assigned_to],
         priority_level: params[:priority_level],
-        hide_snoozed: params[:hide_snoozed]
+        hide_snoozed: params[:hide_snoozed],
+        # Sorting params
+        sort_by: params[:sort_by],
+        sort_direction: params[:sort_direction]
       }
     end
 

data/app/helpers/rails_error_dashboard/application_helper.rb

@@ -55,5 +55,43 @@ module RailsErrorDashboard
         "var(--text-color)"
       end
     end
+
+    # Returns the current user name for filtering "My Errors"
+    # Uses configured dashboard username or system username
+    # @return [String] Current user identifier
+    def current_user_name
+      RailsErrorDashboard.configuration.dashboard_username || ENV["USER"] || "unknown"
+    end
+
+    # Generates a sortable column header link
+    # @param label [String] The column label to display
+    # @param column [String] The column name to sort by
+    # @return [String] HTML safe link with sort indicator
+    def sortable_header(label, column)
+      current_sort = params[:sort_by]
+      current_direction = params[:sort_direction] || "desc"
+
+      # Determine new direction: if clicking same column, toggle; otherwise default to desc
+      new_direction = if current_sort == column
+        current_direction == "asc" ? "desc" : "asc"
+      else
+        "desc"
+      end
+
+      # Choose icon based on current state
+      icon = if current_sort == column
+        current_direction == "asc" ? "▲" : "▼"
+      else
+        "⇅"  # Unsorted indicator
+      end
+
+      # Preserve existing filter params while adding sort params
+      link_params = params.permit!.to_h.merge(sort_by: column, sort_direction: new_direction)
+
+      link_to errors_path(link_params), class: "text-decoration-none" do
+        content_tag(:span, "#{label} ", class: current_sort == column ? "fw-bold" : "") +
+        content_tag(:span, icon, class: "text-muted small")
+      end
+    end
   end
 end

data/app/models/rails_error_dashboard/error_log.rb

@@ -133,6 +133,11 @@ module RailsErrorDashboard
       SystemStackError
       SignalException
       ActiveRecord::StatementInvalid
+      LoadError
+      SyntaxError
+      ActiveRecord::ConnectionNotEstablished
+      Redis::ConnectionError
+      OpenSSL::SSL::SSLError
     ].freeze
 
     HIGH_SEVERITY_ERROR_TYPES = %w[
@@ -141,6 +146,11 @@ module RailsErrorDashboard
       TypeError
       NoMethodError
       NameError
+      ZeroDivisionError
+      FloatDomainError
+      IndexError
+      KeyError
+      RangeError
     ].freeze
 
     MEDIUM_SEVERITY_ERROR_TYPES = %w[
@@ -148,6 +158,10 @@ module RailsErrorDashboard
       Timeout::Error
       Net::ReadTimeout
       Net::OpenTimeout
+      ActiveRecord::RecordNotUnique
+      JSON::ParserError
+      CSV::MalformedCSVError
+      Errno::ECONNREFUSED
     ].freeze
 
     # Find existing error by hash or create new one

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

@@ -1,16 +1,54 @@
 <!DOCTYPE html>
 <html>
 <head>
-  <title>Rails error dashboard</title>
+  <title>Rails Error Dashboard</title>
   <%= csrf_meta_tags %>
   <%= csp_meta_tag %>
 
   <%= yield :head %>
 
   <%= stylesheet_link_tag    "rails_error_dashboard/application", media: "all" %>
+  <script src="https://cdn.jsdelivr.net/npm/chart.js@4.4.0/dist/chart.umd.min.js"></script>
+  <script src="https://cdn.jsdelivr.net/npm/chartkick@5.0.1/dist/chartkick.min.js"></script>
 </head>
 <body>
 
+<!-- Navigation -->
+<nav class="navbar navbar-expand-lg navbar-dark bg-dark">
+  <div class="container-fluid">
+    <%= link_to "Error Dashboard", root_path, class: "navbar-brand" %>
+    <button class="navbar-toggler" type="button" data-bs-toggle="collapse" data-bs-target="#navbarNav">
+      <span class="navbar-toggler-icon"></span>
+    </button>
+    <div class="collapse navbar-collapse" id="navbarNav">
+      <ul class="navbar-nav me-auto">
+        <li class="nav-item">
+          <%= link_to overview_path, class: "nav-link #{controller_name == 'errors' && action_name == 'overview' ? 'active' : ''}" do %>
+            <i class="bi bi-speedometer2"></i> Dashboard
+          <% end %>
+        </li>
+        <li class="nav-item">
+          <%= link_to errors_path, class: "nav-link #{controller_name == 'errors' && action_name == 'index' ? 'active' : ''}" do %>
+            <i class="bi bi-list-ul"></i> Errors
+          <% end %>
+        </li>
+        <li class="nav-item">
+          <%= link_to analytics_errors_path, class: "nav-link #{controller_name == 'errors' && action_name == 'analytics' ? 'active' : ''}" do %>
+            <i class="bi bi-graph-up"></i> Analytics
+          <% end %>
+        </li>
+      </ul>
+      <ul class="navbar-nav">
+        <li class="nav-item">
+          <a class="nav-link" href="#" id="theme-toggle">
+            <i class="bi bi-moon-stars"></i>
+          </a>
+        </li>
+      </ul>
+    </div>
+  </div>
+</nav>
+
 <%= yield %>
 
 </body>

data/app/views/rails_error_dashboard/errors/analytics.html.erb

@@ -330,4 +330,305 @@
       </div>
     </div>
   </div>
+
+  <!-- Recurring Issues Analysis -->
+  <div class="card mb-4">
+    <div class="card-header bg-white">
+      <h5 class="mb-0"><i class="bi bi-arrow-repeat"></i> Recurring Issues</h5>
+    </div>
+    <div class="card-body">
+      <% if @recurring_data[:high_frequency_errors].any? %>
+        <h6 class="text-muted mb-3">High Frequency Errors</h6>
+        <div class="table-responsive mb-4">
+          <table class="table table-sm">
+            <thead>
+              <tr>
+                <th>Error Type</th>
+                <th>Total Occurrences</th>
+                <th>Duration</th>
+                <th>First Seen</th>
+                <th>Last Seen</th>
+                <th>Status</th>
+              </tr>
+            </thead>
+            <tbody>
+              <% @recurring_data[:high_frequency_errors].first(10).each do |error| %>
+                <tr>
+                  <td><code class="small"><%= error[:error_type] %></code></td>
+                  <td><span class="badge bg-danger"><%= error[:total_occurrences] %></span></td>
+                  <td><%= error[:duration_days] %> days</td>
+                  <td><small class="text-muted"><%= error[:first_seen].strftime("%b %d, %Y") %></small></td>
+                  <td><small class="text-muted"><%= error[:last_seen].strftime("%b %d, %Y %H:%M") %></small></td>
+                  <td>
+                    <% if error[:still_active] %>
+                      <span class="badge bg-warning">Active</span>
+                    <% else %>
+                      <span class="badge bg-secondary">Inactive</span>
+                    <% end %>
+                  </td>
+                </tr>
+              <% end %>
+            </tbody>
+          </table>
+        </div>
+      <% else %>
+        <p class="text-muted">No high-frequency errors found in the last <%= @days %> days.</p>
+      <% end %>
+
+      <% if @recurring_data[:persistent_errors].any? %>
+        <h6 class="text-muted mb-3 mt-4">Persistent Unresolved Errors</h6>
+        <div class="alert alert-info">
+          <i class="bi bi-info-circle"></i> These errors have been unresolved for more than 7 days.
+        </div>
+        <div class="table-responsive">
+          <table class="table table-sm">
+            <thead>
+              <tr>
+                <th>Error Type</th>
+                <th>Message</th>
+                <th>Platform</th>
+                <th>Count</th>
+                <th>Age</th>
+                <th>Actions</th>
+              </tr>
+            </thead>
+            <tbody>
+              <% @recurring_data[:persistent_errors].first(10).each do |error| %>
+                <tr>
+                  <td><code class="small"><%= error[:error_type] %></code></td>
+                  <td class="text-muted small"><%= error[:message] %></td>
+                  <td><%= error[:platform] %></td>
+                  <td><span class="badge bg-secondary"><%= error[:occurrence_count] %>x</span></td>
+                  <td>
+                    <span class="badge bg-warning"><%= error[:age_days] %> days</span>
+                  </td>
+                  <td>
+                    <%= link_to "View", error_path(error[:id]), class: "btn btn-sm btn-outline-primary" %>
+                  </td>
+                </tr>
+              <% end %>
+            </tbody>
+          </table>
+        </div>
+      <% end %>
+    </div>
+  </div>
+
+  <!-- Release Comparison Charts -->
+  <% if @errors_by_version.present? && @errors_by_version.count >= 2 %>
+    <div class="row g-4 mb-4">
+      <!-- Errors by Version Chart -->
+      <div class="col-md-8">
+        <div class="card">
+          <div class="card-header bg-white">
+            <h5 class="mb-0"><i class="bi bi-git"></i> Errors by Release Version</h5>
+          </div>
+          <div class="card-body">
+            <div id="errors-by-version-chart"></div>
+            <script>
+              document.addEventListener('DOMContentLoaded', function() {
+                const versionData = <%= raw @errors_by_version.transform_values { |v| v[:count] }.to_json %>;
+                const colors = window.getChartColors();
+
+                new Chartkick.ColumnChart("errors-by-version-chart", versionData, {
+                  colors: ["#8B5CF6", "#EF4444", "#F59E0B", "#10B981"],
+                  height: "300px",
+                  xtitle: "Version",
+                  ytitle: "Error Count",
+                  library: window.getChartLibraryOptions()
+                });
+              });
+            </script>
+          </div>
+        </div>
+      </div>
+
+      <!-- Release Comparison Card -->
+      <div class="col-md-4">
+        <div class="card">
+          <div class="card-header bg-white">
+            <h5 class="mb-0">Latest vs Previous</h5>
+          </div>
+          <div class="card-body">
+            <% if @release_comparison.present? %>
+              <div class="mb-3">
+                <small class="text-muted">Latest Version</small>
+                <h4><%= @release_comparison[:latest_version] %></h4>
+                <p class="mb-0">
+                  <span class="badge bg-danger"><%= @release_comparison[:latest_count] %> errors</span>
+                  <% if @release_comparison[:latest_critical] > 0 %>
+                    <span class="badge bg-dark ms-1"><%= @release_comparison[:latest_critical] %> critical</span>
+                  <% end %>
+                </p>
+              </div>
+              <div class="mb-3">
+                <small class="text-muted">Previous Version</small>
+                <h4><%= @release_comparison[:previous_version] %></h4>
+                <p class="mb-0">
+                  <span class="badge bg-secondary"><%= @release_comparison[:previous_count] %> errors</span>
+                  <% if @release_comparison[:previous_critical] > 0 %>
+                    <span class="badge bg-secondary ms-1"><%= @release_comparison[:previous_critical] %> critical</span>
+                  <% end %>
+                </p>
+              </div>
+              <hr>
+              <div>
+                <small class="text-muted">Change</small>
+                <h4 class="<%= @release_comparison[:change_percentage] > 0 ? 'text-danger' : 'text-success' %>">
+                  <%= @release_comparison[:change_percentage] > 0 ? '↑' : '↓' %>
+                  <%= @release_comparison[:change_percentage].abs %>%
+                </h4>
+              </div>
+            <% else %>
+              <p class="text-muted">Not enough release data for comparison.</p>
+            <% end %>
+          </div>
+        </div>
+      </div>
+    </div>
+
+    <!-- Problematic Releases Alert -->
+    <% if @problematic_releases.any? %>
+      <div class="alert alert-warning mb-4">
+        <h5 class="alert-heading"><i class="bi bi-exclamation-triangle"></i> Problematic Releases Detected</h5>
+        <p>The following releases have significantly higher error rates (&gt;2x average):</p>
+        <ul class="mb-0">
+          <% @problematic_releases.each do |release| %>
+            <li>
+              <strong><%= release[:version] %></strong>:
+              <%= release[:error_count] %> errors
+              (<%= release[:critical_count] %> critical,
+              +<%= release[:deviation_from_avg] %>% from average)
+            </li>
+          <% end %>
+        </ul>
+      </div>
+    <% end %>
+  <% end %>
+
+  <!-- MTTR (Mean Time to Resolution) -->
+  <% if @mttr_stats[:total_resolved] > 0 %>
+    <div class="card mb-4">
+      <div class="card-header bg-white">
+        <h5 class="mb-0"><i class="bi bi-clock-history"></i> Resolution Performance (MTTR)</h5>
+      </div>
+      <div class="card-body">
+        <div class="row g-3 mb-4">
+          <!-- Overall MTTR -->
+          <div class="col-md-3">
+            <div class="card border-info">
+              <div class="card-body text-center">
+                <small class="text-muted d-block">Overall MTTR</small>
+                <h3 class="text-info mb-0"><%= @mttr_stats[:overall_mttr] %>h</h3>
+                <small class="text-muted">Average resolution time</small>
+              </div>
+            </div>
+          </div>
+
+          <!-- Fastest -->
+          <div class="col-md-3">
+            <div class="card border-success">
+              <div class="card-body text-center">
+                <small class="text-muted d-block">Fastest</small>
+                <h3 class="text-success mb-0"><%= @mttr_stats[:fastest_resolution] || 0 %>m</h3>
+                <small class="text-muted">Best resolution time</small>
+              </div>
+            </div>
+          </div>
+
+          <!-- Slowest -->
+          <div class="col-md-3">
+            <div class="card border-danger">
+              <div class="card-body text-center">
+                <small class="text-muted d-block">Slowest</small>
+                <h3 class="text-danger mb-0"><%= @mttr_stats[:slowest_resolution] || 0 %>h</h3>
+                <small class="text-muted">Longest resolution time</small>
+              </div>
+            </div>
+          </div>
+
+          <!-- Total Resolved -->
+          <div class="col-md-3">
+            <div class="card border-secondary">
+              <div class="card-body text-center">
+                <small class="text-muted d-block">Total Resolved</small>
+                <h3 class="text-secondary mb-0"><%= @mttr_stats[:total_resolved] %></h3>
+                <small class="text-muted">In last <%= @days %> days</small>
+              </div>
+            </div>
+          </div>
+        </div>
+
+        <!-- MTTR by Platform -->
+        <% if @mttr_by_platform.present? && @mttr_by_platform.any? %>
+          <div class="mt-4">
+            <h6 class="text-muted mb-3">MTTR by Platform</h6>
+            <div id="mttr-by-platform-chart"></div>
+            <script>
+              document.addEventListener('DOMContentLoaded', function() {
+                const colors = window.getChartColors();
+                new Chartkick.BarChart("mttr-by-platform-chart",
+                  <%= raw @mttr_by_platform.to_json %>, {
+                  suffix: " hours",
+                  height: "200px",
+                  colors: ["#8B5CF6"],
+                  library: window.getChartLibraryOptions()
+                });
+              });
+            </script>
+          </div>
+        <% end %>
+
+        <!-- MTTR by Severity -->
+        <% if @mttr_stats[:mttr_by_severity].present? && @mttr_stats[:mttr_by_severity].any? %>
+          <div class="mt-4">
+            <h6 class="text-muted mb-3">MTTR by Severity</h6>
+            <div class="table-responsive">
+              <table class="table table-sm">
+                <thead>
+                  <tr>
+                    <th>Severity</th>
+                    <th>Average Time to Resolution</th>
+                  </tr>
+                </thead>
+                <tbody>
+                  <% @mttr_stats[:mttr_by_severity].each do |severity, hours| %>
+                    <tr>
+                      <td>
+                        <span class="badge bg-<%= severity_color(severity) %>">
+                          <%= severity.to_s.capitalize %>
+                        </span>
+                      </td>
+                      <td><strong><%= hours %> hours</strong></td>
+                    </tr>
+                  <% end %>
+                </tbody>
+              </table>
+            </div>
+          </div>
+        <% end %>
+
+        <!-- MTTR Trend -->
+        <% if @mttr_stats[:mttr_trend].present? && @mttr_stats[:mttr_trend].any? %>
+          <div class="mt-4">
+            <h6 class="text-muted mb-3">MTTR Trend (Weekly)</h6>
+            <div id="mttr-trend-chart"></div>
+            <script>
+              document.addEventListener('DOMContentLoaded', function() {
+                const colors = window.getChartColors();
+                new Chartkick.LineChart("mttr-trend-chart",
+                  <%= raw @mttr_stats[:mttr_trend].to_json %>, {
+                  suffix: " hours",
+                  height: "250px",
+                  colors: ["#10B981"],
+                  curve: false,
+                  library: window.getChartLibraryOptions()
+                });
+              });
+            </script>
+          </div>
+        <% end %>
+      </div>
+    </div>
+  <% end %>
 </div>

data/app/views/rails_error_dashboard/errors/index.html.erb

@@ -1,5 +1,7 @@
-<!-- Subscribe to Turbo Stream updates -->
-<%= turbo_stream_from "error_list" %>
+<!-- Subscribe to Turbo Stream updates (only if ActionCable is available) -->
+<% if defined?(ActionCable) %>
+  <%= turbo_stream_from "error_list" %>
+<% end %>
 
 <div class="py-4">
   <div class="d-flex justify-content-between align-items-center mb-4">
@@ -124,6 +126,16 @@
       <h5 class="mb-0">Filters & Search</h5>
     </div>
     <div class="card-body">
+      <!-- Quick Filter Buttons -->
+      <div class="d-flex gap-2 mb-3">
+        <%= link_to "All Errors", errors_path,
+            class: "btn btn-sm #{params[:assigned_to].blank? ? 'btn-primary' : 'btn-outline-primary'}" %>
+        <%= link_to "Unassigned", errors_path(assigned_to: '__unassigned__'),
+            class: "btn btn-sm #{params[:assigned_to] == '__unassigned__' ? 'btn-primary' : 'btn-outline-primary'}" %>
+        <%= link_to "My Errors", errors_path(assigned_to: current_user_name),
+            class: "btn btn-sm #{params[:assigned_to] == current_user_name ? 'btn-primary' : 'btn-outline-primary'}" %>
+      </div>
+
       <%= form_with url: errors_path, method: :get, class: "row g-3", data: { turbo: false } do %>
         <div class="col-md-4">
           <%= text_field_tag :search, params[:search], placeholder: "Search errors...", class: "form-control" %>
@@ -149,6 +161,31 @@
           ], params[:severity]), class: "form-select" %>
         </div>
 
+        <!-- Time Range Filter -->
+        <div class="col-md-2">
+          <%= select_tag :timeframe, options_for_select([
+            ['All Time', ''],
+            ['Last Hour', 'last_hour'],
+            ['Today', 'today'],
+            ['Yesterday', 'yesterday'],
+            ['Last 7 Days', 'last_7_days'],
+            ['Last 30 Days', 'last_30_days'],
+            ['Last 90 Days', 'last_90_days']
+          ], params[:timeframe]), class: "form-select" %>
+        </div>
+
+        <!-- Frequency Filter -->
+        <div class="col-md-2">
+          <%= select_tag :frequency, options_for_select([
+            ['All Frequencies', ''],
+            ['Once', 'once'],
+            ['2-9 Times', 'few'],
+            ['10-99 Times', 'frequent'],
+            ['100+ Times', 'very_frequent'],
+            ['Recurring (Active)', 'recurring']
+          ], params[:frequency]), class: "form-select" %>
+        </div>
+
         <!-- Phase 3: Workflow Filters -->
         <div class="col-md-2">
           <%= select_tag :status, options_for_select([
@@ -253,13 +290,13 @@
                 <th style="width: 40px;">
                   <input type="checkbox" id="select-all" class="form-check-input">
                 </th>
-                <th>Severity</th>
-                <th>Error Type</th>
+                <th><%= sortable_header("Severity", "severity") %></th>
+                <th><%= sortable_header("Error Type", "error_type") %></th>
                 <th>Message</th>
-                <th>Occurrences</th>
-                <th>First / Last Seen</th>
+                <th><%= sortable_header("Occurrences", "occurrence_count") %></th>
+                <th><%= sortable_header("First / Last Seen", "last_seen_at") %></th>
                 <% if @platforms.size > 1 %>
-                  <th>Platform</th>
+                  <th><%= sortable_header("Platform", "platform") %></th>
                 <% end %>
                 <th>Status</th>
                 <th></th>

data/config/routes.rb

@@ -1,5 +1,8 @@
 RailsErrorDashboard::Engine.routes.draw do
-  root to: "errors#index"
+  root to: "errors#overview"
+
+  # Dashboard overview
+  get "overview", to: "errors#overview", as: :overview
 
   resources :errors, only: [ :index, :show ] do
     member do