rails_error_dashboard 0.1.28 → 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/_user_errors_table.html.erb

@@ -0,0 +1,70 @@
+<div class="table-responsive">
+  <table class="table table-hover mb-0">
+    <thead class="table-light">
+      <tr>
+        <% if show_rank %>
+          <th>Rank</th>
+        <% end %>
+        <th>User</th>
+        <% if show_error_type_count %>
+          <th>Different Error Types</th>
+        <% end %>
+        <th>Error Count</th>
+        <% if show_percentage %>
+          <th>Percentage</th>
+        <% end %>
+        <% if show_error_types %>
+          <th>Error Types</th>
+        <% end %>
+        <th>Actions</th>
+      </tr>
+    </thead>
+    <tbody>
+      <% users.each_with_index do |user_data, index| %>
+        <tr>
+          <% if show_rank %>
+            <td><strong>#<%= index + 1 %></strong></td>
+          <% end %>
+          <td><%= user_data[:email] || user_data[:user_email] %></td>
+          <% if show_error_type_count %>
+            <td>
+              <span class="badge bg-warning text-dark">
+                <%= user_data[:error_type_count] %> types
+              </span>
+            </td>
+          <% end %>
+          <td>
+            <span class="badge bg-danger"><%= user_data[:count] || user_data[:total_errors] %></span>
+          </td>
+          <% if show_percentage && total_errors.present? %>
+            <td>
+              <div class="progress" style="height: 20px;">
+                <div class="progress-bar bg-danger"
+                     role="progressbar"
+                     style="width: <%= ((user_data[:count] || user_data[:total_errors]).to_f / total_errors * 100).round(1) %>%"
+                     aria-valuenow="<%= user_data[:count] || user_data[:total_errors] %>"
+                     aria-valuemin="0"
+                     aria-valuemax="100">
+                  <%= ((user_data[:count] || user_data[:total_errors]).to_f / total_errors * 100).round(1) %>%
+                </div>
+              </div>
+            </td>
+          <% end %>
+          <% if show_error_types && user_data[:error_types].present? %>
+            <td>
+              <% user_data[:error_types].first(3).each do |error_type| %>
+                <code class="small me-1"><%= error_type %></code>
+              <% end %>
+              <% if user_data[:error_types].count > 3 %>
+                <span class="text-muted small">+<%= user_data[:error_types].count - 3 %> more</span>
+              <% end %>
+            </td>
+          <% end %>
+          <td>
+            <%= link_to "View Errors", errors_path(user_id: user_data[:user_id]), class: "btn btn-sm btn-outline-primary" %>
+          </td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
+</div>

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

@@ -266,43 +266,15 @@
             <h5 class="mb-0"><i class="bi bi-people"></i> Top 10 Affected Users</h5>
           </div>
           <div class="card-body p-0">
-            <div class="table-responsive">
-              <table class="table table-hover mb-0">
-                <thead class="table-light">
-                  <tr>
-                    <th>Rank</th>
-                    <th>User</th>
-                    <th>Error Count</th>
-                    <th>Percentage</th>
-                    <th>Actions</th>
-                  </tr>
-                </thead>
-                <tbody>
-                  <% @top_users.each_with_index do |(email, count), index| %>
-                    <tr>
-                      <td><strong>#<%= index + 1 %></strong></td>
-                      <td><%= email %></td>
-                      <td><span class="badge bg-danger"><%= count %></span></td>
-                      <td>
-                        <div class="progress" style="height: 20px;">
-                          <div class="progress-bar bg-danger"
-                               role="progressbar"
-                               style="width: <%= (count.to_f / @error_stats[:total] * 100).round(1) %>%"
-                               aria-valuenow="<%= count %>"
-                               aria-valuemin="0"
-                               aria-valuemax="<%= @error_stats[:total] %>">
-                            <%= (count.to_f / @error_stats[:total] * 100).round(1) %>%
-                          </div>
-                        </div>
-                      </td>
-                      <td>
-                        <%= link_to "View Errors", errors_path(search: email), class: "btn btn-sm btn-outline-primary" %>
-                      </td>
-                    </tr>
-                  <% end %>
-                </tbody>
-              </table>
-            </div>
+            <%= render partial: 'user_errors_table',
+                       locals: {
+                         users: @top_users,
+                         show_rank: true,
+                         show_error_type_count: false,
+                         show_percentage: true,
+                         show_error_types: false,
+                         total_errors: @error_stats[:total]
+                       } %>
           </div>
         </div>
       </div>

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

@@ -216,43 +216,17 @@
         <small class="text-muted">Users experiencing 2+ different error types</small>
       </div>
       <div class="card-body">
-        <div class="table-responsive">
-          <table class="table table-hover">
-            <thead>
-              <tr>
-                <th>User</th>
-                <th>Different Error Types</th>
-                <th>Total Errors</th>
-                <th>Error Types</th>
-                <th>Actions</th>
-              </tr>
-            </thead>
-            <tbody>
-              <% @multi_error_users.first(20).each do |user_data| %>
-                <tr>
-                  <td><%= user_data[:user_email] %></td>
-                  <td>
-                    <span class="badge bg-warning text-dark">
-                      <%= user_data[:error_type_count] %> types
-                    </span>
-                  </td>
-                  <td><%= user_data[:total_errors] %></td>
-                  <td>
-                    <% user_data[:error_types].first(3).each do |error_type| %>
-                      <code class="small me-1"><%= error_type %></code>
-                    <% end %>
-                    <% if user_data[:error_types].count > 3 %>
-                      <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>
-          </table>
-        </div>
+        <%= render partial: 'user_errors_table',
+                   locals: {
+                     users: @multi_error_users.first(20),
+                     show_rank: false,
+                     show_error_type_count: true,
+                     show_percentage: false,
+                     show_error_types: true,
+                     total_errors: nil
+                   } %>
+      </div>
+      <div class="card-body pt-0 border-top">
         <% if @multi_error_users.count > 20 %>
           <p class="text-muted small mb-0">
             Showing 20 of <%= @multi_error_users.count %> users

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>