rails_error_dashboard 0.1.3 → 0.1.5

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
@@ -219,8 +235,32 @@ module RailsErrorDashboard
       @platform_specific_errors = correlation.platform_specific_errors
     end
 
+    def settings
+      @config = RailsErrorDashboard.configuration
+    end
+
     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 +268,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/layouts/rails_error_dashboard.html.erb

@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html>
   <head>
-    <title>Error Dashboard</title>
+    <title><%= Rails.application.class.module_parent_name %> - Error Dashboard</title>
     <meta name="viewport" content="width=device-width,initial-scale=1">
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
@@ -736,7 +736,10 @@
             <i class="bi bi-list fs-4"></i>
           </button>
           <a class="navbar-brand" href="<%= root_path %>">
-            <i class="bi bi-bug-fill"></i> Error Dashboard
+            <i class="bi bi-bug-fill"></i>
+            <span class="d-none d-sm-inline"><%= Rails.application.class.module_parent_name %></span>
+            <span class="d-none d-md-inline text-white-50 mx-2">|</span>
+            <span class="d-none d-md-inline">Error Dashboard</span>
           </a>
         </div>
         <div class="d-flex align-items-center gap-3">
@@ -771,6 +774,11 @@
                   <i class="bi bi-graph-up"></i> Analytics
                 <% end %>
               </li>
+              <li class="nav-item">
+                <%= link_to settings_path, class: "nav-link #{request.path == settings_path ? 'active' : ''}" do %>
+                  <i class="bi bi-gear"></i> Settings
+                <% end %>
+              </li>
             </ul>
 
             <h6 class="mt-4">QUICK FILTERS</h6>
@@ -780,6 +788,16 @@
                   <i class="bi bi-exclamation-circle"></i> Unresolved
                 <% end %>
               </li>
+              <li class="nav-item">
+                <%= link_to errors_path(severity: 'critical'), class: "nav-link" do %>
+                  <i class="bi bi-exclamation-triangle-fill text-danger"></i> Critical
+                <% end %>
+              </li>
+              <li class="nav-item">
+                <%= link_to errors_path(priority_level: 'high'), class: "nav-link" do %>
+                  <i class="bi bi-flag-fill text-warning"></i> High Priority
+                <% end %>
+              </li>
               <li class="nav-item">
                 <%= link_to errors_path(platform: 'iOS'), class: "nav-link" do %>
                   <i class="bi bi-phone"></i> iOS Errors

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

@@ -143,6 +143,14 @@
               });
             </script>
           </div>
+          <div class="card-footer bg-white border-top-0">
+            <div class="d-flex gap-2 flex-wrap">
+              <small class="text-muted me-2">Quick Links:</small>
+              <% @errors_by_platform.keys.each do |platform| %>
+                <%= link_to platform, errors_path(platform: platform), class: "btn btn-sm btn-outline-secondary" %>
+              <% end %>
+            </div>
+          </div>
         </div>
       </div>
     </div>
@@ -249,6 +257,7 @@
                     <th>User</th>
                     <th>Error Count</th>
                     <th>Percentage</th>
+                    <th>Actions</th>
                   </tr>
                 </thead>
                 <tbody>
@@ -269,6 +278,9 @@
                           </div>
                         </div>
                       </td>
+                      <td>
+                        <%= link_to "View Errors", errors_path(search: email), class: "btn btn-sm btn-outline-primary" %>
+                      </td>
                     </tr>
                   <% end %>
                 </tbody>
@@ -330,4 +342,309 @@
       </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>
+                    <th>Actions</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>
+                      <td>
+                        <%= link_to "View", errors_path(severity: severity), class: "btn btn-sm btn-outline-primary" %>
+                      </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/correlation.html.erb

@@ -82,6 +82,7 @@
                 <th>Critical Errors</th>
                 <th>Unique Error Types</th>
                 <th>Platforms</th>
+                <th>Actions</th>
               </tr>
             </thead>
             <tbody>
@@ -101,6 +102,9 @@
                       <span class="badge bg-secondary text-capitalize me-1"><%= platform %></span>
                     <% end %>
                   </td>
+                  <td>
+                    <%= link_to "View", errors_path(search: release[:version]), class: "btn btn-sm btn-outline-primary" %>
+                  </td>
                 </tr>
               <% end %>
             </tbody>
@@ -218,6 +222,7 @@
                 <th>Different Error Types</th>
                 <th>Total Errors</th>
                 <th>Error Types</th>
+                <th>Actions</th>
               </tr>
             </thead>
             <tbody>
@@ -238,6 +243,9 @@
                       <span class="text-muted small">+<%= user_data[:error_types].count - 3 %> more</span>
                     <% end %>
                   </td>
+                  <td>
+                    <%= link_to "View", errors_path(search: user_data[:user_email]), class: "btn btn-sm btn-outline-primary" %>
+                  </td>
                 </tr>
               <% end %>
             </tbody>