rails_error_dashboard 0.1.28 → 0.1.30

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

@@ -45,8 +45,23 @@
       </div>
     </div>
 
+    <!-- Unresolved Errors Card -->
+    <div class="col-12 col-md-6 col-lg-4">
+      <div class="card stat-card h-100">
+        <div class="card-body text-center">
+          <div class="stat-label mb-2">UNRESOLVED ERRORS</div>
+          <div class="stat-value text-danger">
+            <%= @stats[:unresolved] %>
+          </div>
+          <small class="text-muted">
+            Pending resolution
+          </small>
+        </div>
+      </div>
+    </div>
+
     <!-- Trend Card -->
-    <div class="col-12 col-lg-4">
+    <div class="col-12 col-md-6 col-lg-4">
       <div class="card stat-card h-100">
         <div class="card-body text-center">
           <div class="stat-label mb-2">ERROR TREND</div>
@@ -60,15 +75,58 @@
         </div>
       </div>
     </div>
+
+    <!-- Resolution Rate Card -->
+    <div class="col-12 col-md-6 col-lg-4">
+      <div class="card stat-card h-100">
+        <div class="card-body text-center">
+          <div class="stat-label mb-2">RESOLUTION RATE</div>
+          <%
+            total_errors = @stats[:resolved] + @stats[:unresolved]
+            resolution_rate = total_errors > 0 ? ((@stats[:resolved].to_f / total_errors) * 100).round(1) : 0.0
+          %>
+          <div class="stat-value <%= resolution_rate >= 80 ? 'text-success' : resolution_rate >= 50 ? 'text-warning' : 'text-danger' %>">
+            <%= resolution_rate %>%
+          </div>
+          <small class="text-muted">
+            <%= @stats[:resolved] %> of <%= total_errors %> resolved
+          </small>
+        </div>
+      </div>
+    </div>
+
+    <!-- Average Resolution Time Card -->
+    <div class="col-12 col-md-6 col-lg-4">
+      <div class="card stat-card h-100">
+        <div class="card-body text-center">
+          <div class="stat-label mb-2">AVG RESOLUTION TIME</div>
+          <% if @stats[:average_resolution_time].present? %>
+            <div class="stat-value text-warning">
+              <%= @stats[:average_resolution_time] %>h
+            </div>
+            <small class="text-muted">
+              Mean time to resolution
+            </small>
+          <% else %>
+            <div class="stat-value text-muted">
+              --
+            </div>
+            <small class="text-muted">
+              No resolved errors yet
+            </small>
+          <% end %>
+        </div>
+      </div>
+    </div>
   </div>
 
-  <!-- Top 5 Errors by Impact (Mobile: stack, Desktop: 2 columns) -->
+  <!-- Top 6 Errors by Impact (Mobile: stack, Desktop: 2 columns) -->
   <% if @stats[:top_errors_by_impact].any? %>
     <div class="card mb-4">
       <div class="card-header bg-white d-flex justify-content-between align-items-center">
         <h5 class="mb-0">
           <i class="bi bi-exclamation-triangle me-2"></i>
-          Top 5 Errors by Impact
+          Top 6 Errors by Impact
         </h5>
         <%= link_to "View All Errors →", errors_path, class: "btn btn-sm btn-outline-primary" %>
       </div>
@@ -174,6 +232,126 @@
     </div>
   <% end %>
 
+  <!-- Correlation Summary (Last 7 days) -->
+  <% if RailsErrorDashboard.configuration.enable_error_correlation && (@problematic_releases.any? || @time_correlated_errors.any? || @multi_error_users.any?) %>
+    <%
+      # Calculate column width based on number of cards present
+      cards_present = [@problematic_releases.any?, @time_correlated_errors.any?, @multi_error_users.any?].count(true)
+      col_class = case cards_present
+                  when 1 then "col-12"
+                  when 2 then "col-12 col-lg-6"
+                  else "col-12 col-lg-4"
+                  end
+    %>
+    <div class="card mb-4">
+      <div class="card-header bg-white d-flex justify-content-between align-items-center">
+        <h5 class="mb-0">
+          <i class="bi bi-diagram-3 me-2"></i>
+          Error Correlation Insights
+        </h5>
+        <%= link_to "Full Analysis →", correlation_errors_path, class: "btn btn-sm btn-outline-primary" %>
+      </div>
+      <div class="card-body">
+        <div class="row g-3">
+          <!-- Problematic Releases -->
+          <% if @problematic_releases.any? %>
+            <div class="<%= col_class %>">
+              <div class="card h-100 border-warning">
+                <div class="card-body">
+                  <h6 class="mb-3">
+                    <i class="bi bi-tag me-2"></i>
+                    Problematic Releases
+                  </h6>
+                  <div class="list-group list-group-flush">
+                    <% @problematic_releases.each do |release| %>
+                      <div class="list-group-item px-0 py-2">
+                        <div class="d-flex justify-content-between align-items-start">
+                          <div class="flex-grow-1">
+                            <code class="small"><%= release[:version] || release[:git_sha]&.truncate(8, omission: '') %></code>
+                            <div class="small text-muted">
+                              <%= release[:error_count] %> error<%= release[:error_count] != 1 ? 's' : '' %> •
+                              <%= release[:unique_types] %> type<%= release[:unique_types] != 1 ? 's' : '' %>
+                            </div>
+                          </div>
+                          <span class="badge bg-danger"><%= release[:severity_score] %></span>
+                        </div>
+                      </div>
+                    <% end %>
+                  </div>
+                </div>
+              </div>
+            </div>
+          <% end %>
+
+          <!-- Time-Correlated Errors -->
+          <% if @time_correlated_errors.any? %>
+            <div class="<%= col_class %>">
+              <div class="card h-100 border-info">
+                <div class="card-body">
+                  <h6 class="mb-3">
+                    <i class="bi bi-clock-history me-2"></i>
+                    Time-Correlated Errors
+                  </h6>
+                  <div class="list-group list-group-flush">
+                    <% @time_correlated_errors.each do |group| %>
+                      <div class="list-group-item px-0 py-2">
+                        <div class="small">
+                          <strong><%= group[:error_types].size %> error types</strong>
+                          <div class="text-muted">
+                            occurred together <%= group[:occurrences] %> time<%= group[:occurrences] != 1 ? 's' : '' %>
+                          </div>
+                          <div class="mt-1">
+                            <% group[:error_types].first(2).each do |type| %>
+                              <code class="small d-block text-truncate"><%= type %></code>
+                            <% end %>
+                            <% if group[:error_types].size > 2 %>
+                              <small class="text-muted">+<%= group[:error_types].size - 2 %> more</small>
+                            <% end %>
+                          </div>
+                        </div>
+                      </div>
+                    <% end %>
+                  </div>
+                </div>
+              </div>
+            </div>
+          <% end %>
+
+          <!-- Multi-Error Users -->
+          <% if @multi_error_users.any? %>
+            <div class="<%= col_class %>">
+              <div class="card h-100 border-danger">
+                <div class="card-body">
+                  <h6 class="mb-3">
+                    <i class="bi bi-people-fill me-2"></i>
+                    Users with Multiple Errors
+                  </h6>
+                  <div class="list-group list-group-flush">
+                    <% @multi_error_users.each do |user| %>
+                      <div class="list-group-item px-0 py-2">
+                        <div class="d-flex justify-content-between align-items-start">
+                          <div class="flex-grow-1">
+                            <div class="small">
+                              User ID: <strong><%= user[:user_id] %></strong>
+                            </div>
+                            <div class="small text-muted">
+                              <%= user[:error_types].size %> different error type<%= user[:error_types].size != 1 ? 's' : '' %>
+                            </div>
+                          </div>
+                          <span class="badge bg-danger"><%= user[:total_errors] %></span>
+                        </div>
+                      </div>
+                    <% end %>
+                  </div>
+                </div>
+              </div>
+            </div>
+          <% end %>
+        </div>
+      </div>
+    </div>
+  <% end %>
+
   <!-- Critical Alerts (Last hour) -->
   <% if @critical_alerts.any? %>
     <div class="alert alert-danger border-danger mb-4" role="alert">

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

@@ -378,7 +378,7 @@
     const times = <%= raw @resolution_times.values.map { |v| v || 0 }.to_json %>;
 
     new Chart(resolutionTimeCtx, {
-      type: 'horizontalBar',
+      type: 'bar',
       data: {
         labels: platforms,
         datasets: [{
@@ -390,6 +390,7 @@
         }]
       },
       options: {
+        indexAxis: 'y', // Horizontal bar chart
         responsive: true,
         maintainAspectRatio: false,
         plugins: {

data/app/views/rails_error_dashboard/errors/settings/_value_badge.html.erb

@@ -0,0 +1,286 @@
+<%
+  # Render value badge based on type
+  case type
+  when :boolean
+    if value
+%>
+    <span class="badge bg-success fs-6">
+      <i class="bi bi-check-circle"></i> Enabled
+    </span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">
+      <i class="bi bi-x-circle"></i> Disabled
+    </span>
+<%
+    end
+
+  when :integer
+    if value.present?
+%>
+    <span class="badge bg-info fs-6"><%= value %><%= " #{unit}" if unit %></span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :float
+    if value.present?
+%>
+    <span class="badge bg-info fs-6"><%= value %><%= unit %></span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :percentage
+    if value.present?
+%>
+    <span class="badge bg-info fs-6"><%= (value * 100).to_i %>%</span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :string
+    if value.present?
+%>
+    <code class="fs-6"><%= value %></code>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :symbol
+    if value.present?
+%>
+    <code class="fs-6">:<%= value %></code>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :url
+    if value.present?
+%>
+    <code class="fs-6 text-truncate" style="max-width: 200px; display: inline-block;"><%= value %></code>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :array
+    if value.present? && value.any?
+%>
+    <span class="badge bg-info fs-6"><%= value.count %> item<%= value.count != 1 ? 's' : '' %></span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Empty</span>
+<%
+    end
+
+  when :hash
+    if value.present? && value.any?
+%>
+    <span class="badge bg-info fs-6"><%= value.count %> rule<%= value.count != 1 ? 's' : '' %></span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Empty</span>
+<%
+    end
+
+  when :git_sha
+    if value.present?
+%>
+    <span class="fs-6"><%= git_commit_link(value) %></span>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+
+  when :application_name
+    # Show both configured and auto-detected application name
+    configured_value = value
+    auto_detected = defined?(Rails) ? Rails.application.class.module_parent_name : nil
+    effective_value = configured_value.presence || auto_detected
+    is_auto_detected = configured_value.blank? && auto_detected.present?
+
+    if effective_value.present?
+%>
+    <div>
+      <code class="fs-6"><%= effective_value %></code>
+      <% if is_auto_detected %>
+        <small class="text-success d-block">
+          <i class="bi bi-magic"></i> Auto-detected
+        </small>
+      <% else %>
+        <small class="text-muted d-block">(Configured)</small>
+      <% end %>
+    </div>
+<%
+    else
+%>
+    <div>
+      <span class="badge bg-secondary fs-6">Not set</span>
+      <small class="text-muted d-block">Rails app name unavailable</small>
+    </div>
+<%
+    end
+
+  when :database_name
+    # Show the active database connection name
+    begin
+      if RailsErrorDashboard.configuration.use_separate_database
+        # Using separate database
+        db_config_name = RailsErrorDashboard.configuration.database || 'error_dashboard'
+        db_config = ActiveRecord::Base.configurations.configs_for(env_name: Rails.env, name: db_config_name.to_s)
+        if db_config
+          db_name = db_config.database
+          status_badge = 'bg-success'
+          status_text = "Separate DB: #{db_config_name}"
+        else
+          db_name = db_config_name
+          status_badge = 'bg-warning'
+          status_text = "Configured but not found"
+        end
+      else
+        # Using primary database
+        db_config = ActiveRecord::Base.configurations.configs_for(env_name: Rails.env, name: 'primary')
+        if db_config
+          db_name = db_config.database
+          status_badge = 'bg-info'
+          status_text = "Shared DB (primary)"
+        else
+          # Fallback to current connection
+          db_name = RailsErrorDashboard::ErrorLog.connection.current_database rescue 'Unknown'
+          status_badge = 'bg-info'
+          status_text = "Shared DB"
+        end
+      end
+%>
+    <div>
+      <span class="badge <%= status_badge %> fs-6"><%= File.basename(db_name.to_s) %></span>
+      <small class="text-muted d-block"><%= status_text %></small>
+    </div>
+<%
+    rescue => e
+%>
+    <div>
+      <span class="badge bg-secondary fs-6">Unknown</span>
+      <small class="text-muted d-block">Unable to detect</small>
+    </div>
+<%
+    end
+
+  when :user_model
+    # Show both configured and auto-detected values
+    configured_value = value
+    effective_value = RailsErrorDashboard.configuration.effective_user_model
+    is_auto_detected = configured_value.blank? && effective_value.present?
+
+    if effective_value.present?
+%>
+    <div>
+      <code class="fs-6"><%= effective_value %></code>
+      <% if is_auto_detected %>
+        <small class="text-success d-block">
+          <i class="bi bi-magic"></i> Auto-detected
+        </small>
+      <% else %>
+        <small class="text-muted d-block">(Configured)</small>
+      <% end %>
+    </div>
+<%
+    else
+%>
+    <div>
+      <span class="badge bg-secondary fs-6">Not found</span>
+      <small class="text-muted d-block">No User model detected</small>
+    </div>
+<%
+    end
+
+  when :total_users
+    # Show both configured and auto-detected values
+    configured_value = value
+    effective_value = RailsErrorDashboard.configuration.effective_total_users
+    is_auto_detected = configured_value.blank? && effective_value.present?
+
+    if effective_value.present?
+%>
+    <div>
+      <span class="badge <%= is_auto_detected ? 'bg-success' : 'bg-info' %> fs-6"><%= number_with_delimiter(effective_value) %> users</span>
+      <% if is_auto_detected %>
+        <small class="text-success d-block">
+          <i class="bi bi-magic"></i> Auto-detected
+        </small>
+      <% else %>
+        <small class="text-muted d-block">(Configured)</small>
+      <% end %>
+    </div>
+<%
+    else
+%>
+    <div>
+      <span class="badge bg-secondary fs-6">Not available</span>
+      <small class="text-muted d-block">Unable to query count</small>
+    </div>
+<%
+    end
+
+  when :retention_days
+    # Show retention configuration with manual cleanup instructions
+    if value.present?
+%>
+    <div>
+      <span class="badge bg-warning text-dark fs-6"><%= value %> days</span>
+      <small class="text-danger d-block">
+        <i class="bi bi-exclamation-triangle"></i> Manual cleanup required
+      </small>
+      <small class="text-muted d-block mt-1">
+        Run: <code class="small">rails error_dashboard:cleanup_resolved DAYS=<%= value %></code>
+      </small>
+    </div>
+<%
+    else
+%>
+    <div>
+      <span class="badge bg-success fs-6"><i class="bi bi-infinity"></i> Keep Forever</span>
+      <small class="text-muted d-block">No automatic deletion</small>
+      <small class="text-muted d-block mt-1">
+        To cleanup: <code class="small">rails error_dashboard:cleanup_resolved DAYS=90</code>
+      </small>
+    </div>
+<%
+    end
+
+  else
+    # Default rendering
+    if value.present?
+%>
+    <code class="fs-6"><%= value %></code>
+<%
+    else
+%>
+    <span class="badge bg-secondary fs-6">Not set</span>
+<%
+    end
+  end
+%>