rails_error_dashboard 0.1.29 → 0.1.30

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

@@ -75,9 +75,17 @@
   </td>
   <td onclick="window.location='<%= error_path(error) %>';">
     <% if error.resolved? %>
-      <i class="bi bi-check-circle-fill text-success" title="Resolved"></i>
+      <i class="bi bi-check-circle-fill text-success"
+         data-bs-toggle="tooltip"
+         data-bs-placement="left"
+         data-bs-html="true"
+         title="<%= if error.resolution_comment.present?
+                   "<strong>Resolved</strong><br><em>#{error.resolution_comment.truncate(100).gsub('"', '&quot;').gsub("'", '&#39;')}</em>"
+                 else
+                   'Resolved'
+                 end %>"></i>
     <% else %>
-      <i class="bi bi-exclamation-circle-fill text-danger" title="Unresolved"></i>
+      <i class="bi bi-exclamation-circle-fill text-danger" data-bs-toggle="tooltip" title="Unresolved"></i>
     <% end %>
   </td>
   <td onclick="event.stopPropagation();">

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

@@ -0,0 +1,76 @@
+<%# Displays source code for a backtrace frame with git blame and repository links %>
+<%# Usage: render "source_code", frame: frame, error: @error, index: 0 %>
+
+<% if RailsErrorDashboard.configuration.enable_source_code_integration %>
+  <% source_data = read_source_code(frame) %>
+  <% blame_data = read_git_blame(frame) if RailsErrorDashboard.configuration.enable_git_blame %>
+  <% repo_link = generate_repository_link(frame, error) %>
+
+  <% if source_data && source_data[:lines].present? %>
+    <div class="source-code-viewer mt-2 mb-3 border rounded shadow-sm">
+      <!-- Header with file info and actions -->
+      <div class="source-code-header bg-light border-bottom">
+        <!-- Top row: File path and View Source button -->
+        <div class="d-flex justify-content-between align-items-center px-3 py-2 border-bottom">
+          <div class="d-flex align-items-center gap-2">
+            <i class="bi bi-file-code text-primary"></i>
+            <span class="font-monospace source-file-path fw-medium" style="font-size: 0.9rem;">
+              <%= frame[:short_path] %>:<%= frame[:line_number] %>
+            </span>
+          </div>
+
+          <% if repo_link %>
+            <%= link_to repo_link, target: "_blank", rel: "noopener",
+                class: "btn btn-sm btn-primary",
+                title: "View on #{repo_link.include?('github') ? 'GitHub' : repo_link.include?('gitlab') ? 'GitLab' : 'Bitbucket'}" do %>
+              <i class="bi bi-box-arrow-up-right"></i> View Source
+            <% end %>
+          <% end %>
+        </div>
+
+        <!-- Bottom row: Git blame info (if available) -->
+        <% if blame_data %>
+          <div class="git-blame-info px-3 py-2 bg-white">
+            <div class="d-flex align-items-center gap-3 text-muted" style="font-size: 0.85rem;">
+              <div class="d-flex align-items-center gap-1">
+                <i class="bi bi-person-circle"></i>
+                <span><%= blame_data[:author] %></span>
+              </div>
+              <div class="d-flex align-items-center gap-1">
+                <i class="bi bi-clock"></i>
+                <span><%= time_ago_in_words(blame_data[:date]) %> ago</span>
+              </div>
+              <% if blame_data[:commit_message] %>
+                <div class="d-flex align-items-center gap-1 flex-grow-1">
+                  <i class="bi bi-chat-left-text"></i>
+                  <span class="text-truncate" style="max-width: 400px;" title="<%= blame_data[:commit_message] %>">
+                    <%= blame_data[:commit_message] %>
+                  </span>
+                </div>
+              <% end %>
+            </div>
+          </div>
+        <% end %>
+      </div>
+
+      <!-- Source code lines with syntax highlighting -->
+      <div class="source-code-content bg-white">
+        <div class="code-block" style="max-height: 500px; overflow-y: auto; overflow-x: auto;">
+          <%
+            # Find the error line number for highlighting
+            error_line_number = source_data[:lines].find { |l| l[:highlight] }&.dig(:number)
+            # Get start line number (first line in context)
+            start_line = source_data[:lines].first[:number]
+          %>
+          <pre class="mb-0"><code class="language-<%= source_data[:language] || 'plaintext' %>" data-error-line="<%= error_line_number %>" data-start-line="<%= start_line %>"><%= source_data[:lines].map { |l| l[:content] }.join("\n") %></code></pre>
+        </div>
+      </div>
+    </div>
+  <% elsif source_data && source_data[:error] %>
+    <!-- Error message if source couldn't be read -->
+    <div class="alert alert-warning mt-2 mb-2 py-2">
+      <i class="bi bi-exclamation-triangle"></i>
+      <small>Could not read source: <%= source_data[:error] %></small>
+    </div>
+  <% end %>
+<% end %>

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

@@ -74,96 +74,32 @@
                 <% end %>
 
                 <% if error.resolved? %>
-                  <span class="badge bg-success">
+                  <span class="badge bg-success"
+                        <% if error.resolution_comment.present? %>
+                          data-bs-toggle="tooltip"
+                          data-bs-placement="top"
+                          data-bs-html="true"
+                          title="<strong>Resolution Notes:</strong><br><%= error.resolution_comment.truncate(150).gsub('"', '&quot;') %>"
+                        <% end %>>
                     <i class="bi bi-check-circle"></i> Resolved
                   </span>
                 <% end %>
               </div>
+
+              <% if error.resolved? && error.resolution_comment.present? %>
+                <div class="mt-2 p-2 bg-success bg-opacity-10 border border-success rounded">
+                  <small class="text-success fw-bold">
+                    <i class="bi bi-journal-text"></i> Resolution Notes:
+                  </small>
+                  <div class="mb-0 small text-muted mt-1">
+                    <%= auto_link_urls(truncate(error.resolution_comment, length: 200), error: error).html_safe %>
+                  </div>
+                </div>
+              <% end %>
             </div>
           </div>
         <% end %>
       </div>
     </div>
   </div>
-
-  <style>
-    .timeline {
-      position: relative;
-      padding-left: 0;
-    }
-
-    .timeline-item {
-      position: relative;
-      padding-left: 40px;
-      padding-bottom: 30px;
-    }
-
-    .timeline-item-last {
-      padding-bottom: 0;
-    }
-
-    .timeline-item::before {
-      content: '';
-      position: absolute;
-      left: 11px;
-      top: 30px;
-      bottom: 0;
-      width: 2px;
-      background: var(--bs-border-color);
-    }
-
-    .timeline-item-last::before {
-      display: none;
-    }
-
-    .timeline-marker {
-      position: absolute;
-      left: 0;
-      top: 0;
-      width: 24px;
-      height: 24px;
-      display: flex;
-      align-items: center;
-      justify-content: center;
-      background: white;
-      border: 2px solid var(--bs-border-color);
-      border-radius: 50%;
-      z-index: 1;
-    }
-
-    .timeline-marker-current {
-      border-color: var(--bs-danger);
-      background: var(--bs-danger-bg-subtle);
-      box-shadow: 0 0 0 4px var(--bs-danger-bg-subtle);
-    }
-
-    .timeline-marker i {
-      font-size: 12px;
-    }
-
-    .timeline-content {
-      background: var(--bs-body-bg);
-      padding: 12px;
-      border-radius: 8px;
-      border: 1px solid var(--bs-border-color);
-    }
-
-    .timeline-item:hover .timeline-content {
-      border-color: var(--bs-primary);
-      background: var(--bs-primary-bg-subtle);
-    }
-
-    /* Dark mode support */
-    body.dark-mode .timeline-marker,
-    html[data-theme="dark"] body .timeline-marker {
-      background: var(--card-bg);
-      border-color: var(--border-color);
-    }
-
-    body.dark-mode .timeline-marker-current,
-    html[data-theme="dark"] body .timeline-marker-current {
-      border-color: var(--bs-danger);
-      background: rgba(220, 53, 69, 0.2);
-    }
-  </style>
 <% end %>

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

@@ -148,6 +148,10 @@
       active_filters << { label: "Unassigned", param: :assigned_to }
     elsif params[:assigned_to] == '__assigned__'
       active_filters << { label: "Assigned", param: :assigned_to }
+      # Show assignee name filter if present
+      if params[:assignee_name].present?
+        active_filters << { label: "Assignee: #{params[:assignee_name]}", param: :assignee_name }
+      end
     end
   %>
 
@@ -185,8 +189,6 @@
             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 %>
@@ -268,44 +270,54 @@
             ['All Assignments', ''],
             ['Unassigned', '__unassigned__'],
             ['Assigned', '__assigned__']
-          ], params[:assigned_to]), class: "form-select" %>
+          ], params[:assigned_to]), class: "form-select", id: "assigned_to_filter" %>
         </div>
 
-        <div class="col-md-2">
-          <%= select_tag :priority_level, options_for_select([
-            ['All Priorities', ''],
-            ['🔴 Critical (P3)', 3],
-            ['🟠 High (P2)', 2],
-            ['🟡 Medium (P1)', 1],
-            ['⚪ Low (P0)', 0]
-          ], params[:priority_level]), class: "form-select" %>
+        <!-- Assignee Name Filter - Only shown when "Assigned" is selected -->
+        <div class="col-md-2" id="assignee_name_filter" style="<%= params[:assigned_to] == '__assigned__' ? '' : 'display: none;' %>">
+          <%= select_tag :assignee_name, options_for_select(
+            [['All Assignees', '']] + @assignees.map { |name| [name, name] },
+            params[:assignee_name]
+          ), class: "form-select", placeholder: "Filter by assignee..." %>
         </div>
 
-        <div class="col-auto">
-          <div class="form-check mt-2">
-            <%
-              # Determine checkbox state based on params
-              # If unresolved param is missing or nil, default to checked (true)
-              # If unresolved param is explicitly "false" or "0", uncheck it
-              is_checked = if params[:unresolved].nil?
-                true  # Default to checked when first loading
-              else
-                params[:unresolved] != "false" && params[:unresolved] != "0"
-              end
-            %>
-            <%= check_box_tag :unresolved, "1", is_checked, class: "form-check-input", id: "unresolved_checkbox" %>
-            <%= label_tag :unresolved, "Unresolved only", class: "form-check-label" %>
-          </div>
+        <div class="col-md-2">
+          <%= select_tag :priority_level, options_for_select(
+            [['All Priorities', '']] + RailsErrorDashboard::ErrorLog.priority_options(include_emoji: true),
+            params[:priority_level]
+          ), class: "form-select" %>
         </div>
 
-        <div class="col-auto">
-          <div class="form-check mt-2">
-            <%= check_box_tag :hide_snoozed, "1", params[:hide_snoozed] == "1", class: "form-check-input" %>
-            <%= label_tag :hide_snoozed, "Hide snoozed", class: "form-check-label" %>
+        <!-- Checkboxes on their own row -->
+        <div class="col-12 mt-2">
+          <div class="row g-3">
+            <div class="col-auto">
+              <div class="form-check">
+                <%
+                  # Determine checkbox state based on params
+                  # If unresolved param is missing or nil, default to checked (true)
+                  # If unresolved param is explicitly "false" or "0", uncheck it
+                  is_checked = if params[:unresolved].nil?
+                    true  # Default to checked when first loading
+                  else
+                    params[:unresolved] != "false" && params[:unresolved] != "0"
+                  end
+                %>
+                <%= check_box_tag :unresolved, "1", is_checked, class: "form-check-input", id: "unresolved_checkbox" %>
+                <%= label_tag :unresolved, "Unresolved only", class: "form-check-label" %>
+              </div>
+            </div>
+
+            <div class="col-auto">
+              <div class="form-check">
+                <%= check_box_tag :hide_snoozed, "1", params[:hide_snoozed] == "1", class: "form-check-input" %>
+                <%= label_tag :hide_snoozed, "Hide snoozed", class: "form-check-label" %>
+              </div>
+            </div>
           </div>
         </div>
 
-        <div class="col-12">
+        <div class="col-12 mt-3">
           <%= submit_tag "Apply Filters", class: "btn btn-primary" %>
           <%= link_to "Clear", errors_path, class: "btn btn-outline-secondary" %>
         </div>
@@ -600,4 +612,25 @@
       document.title = `(${unresolvedCount}) ${document.title}`;
     }
   });
+
+  // Toggle assignee name filter visibility based on assigned_to selection
+  document.addEventListener('DOMContentLoaded', function() {
+    const assignedToFilter = document.getElementById('assigned_to_filter');
+    const assigneeNameFilter = document.getElementById('assignee_name_filter');
+
+    if (assignedToFilter && assigneeNameFilter) {
+      assignedToFilter.addEventListener('change', function() {
+        if (this.value === '__assigned__') {
+          assigneeNameFilter.style.display = 'block';
+        } else {
+          assigneeNameFilter.style.display = 'none';
+          // Reset assignee name filter when hiding
+          const assigneeNameSelect = assigneeNameFilter.querySelector('select');
+          if (assigneeNameSelect) {
+            assigneeNameSelect.value = '';
+          }
+        }
+      });
+    }
+  });
 </script>

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: {