rails_pulse 0.2.4 → 0.2.5.pre.pre.2

data/app/views/rails_pulse/job_runs/show.html.erb

@@ -0,0 +1,51 @@
+<%= render 'rails_pulse/components/page_header', taggable: @run %>
+
+<div class="row">
+  <div class="grid-item">
+    <%= render 'rails_pulse/components/panel', title: 'Job Run Details' do %>
+      <dl class="descriptive-list mbs-4">
+        <dt>Job Class</dt>
+        <dd><%= link_to @job.name, job_path(@job) %></dd>
+        <dt>Status</dt>
+        <dd>
+          <%= @run.status.humanize %>
+        </dd>
+        <dt>Occurred At</dt>
+        <dd><%= @run.occurred_at.strftime("%b %d, %Y %l:%M:%S %p") %></dd>
+        <% if @run.enqueued_at %>
+          <dt>Enqueued At</dt>
+          <dd><%= @run.enqueued_at.strftime("%b %d, %Y %l:%M:%S %p") %></dd>
+        <% end %>
+        <dt>Duration</dt>
+        <dd><%= number_to_human(@run.duration || 0, units: { unit: "ms", thousand: "s" }, precision: 2) %></dd>
+        <dt>Attempts</dt>
+        <dd><%= @run.attempts %></dd>
+        <dt>Queue</dt>
+        <dd><%= @job.queue_name || 'default' %></dd>
+      </dl>
+    <% end %>
+  </div>
+
+  <% if @run.error_class.present? %>
+    <div class="grid-item">
+      <%= render 'rails_pulse/components/panel', title: 'Error Details' do %>
+        <dl class="descriptive-list mbs-4">
+          <dt>Error Class</dt>
+          <dd><%= @run.error_class %></dd>
+          <% if @run.error_message.present? %>
+            <dt>Error Message</dt>
+            <dd><%= @run.error_message %></dd>
+          <% end %>
+        </dl>
+      <% end %>
+    </div>
+  <% elsif @run.arguments.present? %>
+    <div class="grid-item">
+      <%= render 'rails_pulse/components/panel', title: 'Arguments' do %>
+        <pre class="mbs-4"><%= JSON.pretty_generate(JSON.parse(@run.arguments)) rescue @run.arguments %></pre>
+      <% end %>
+    </div>
+  <% end %>
+</div>
+
+<%= render 'operations', operations: @operations %>

data/app/views/rails_pulse/jobs/_job_runs_table.html.erb

@@ -0,0 +1,35 @@
+<% columns = [
+  { field: :occurred_at, label: 'Occurred At' },
+  { field: :status, label: 'Status', class: 'w-28' },
+  { field: :duration, label: 'Duration', class: 'w-28' },
+  { field: :attempts, label: 'Attempts', class: 'w-24' },
+  { field: nil, label: 'Tags', class: 'w-32', sortable: false }
+] %>
+
+<table class="table mbs-4" data-controller="rails-pulse--table-sort">
+  <%= render "rails_pulse/components/table_head", columns: columns %>
+
+  <tbody>
+    <% @table_data.each do |run| %>
+      <tr>
+        <td>
+          <%= link_to run.occurred_at.strftime("%b %d, %Y %l:%M %p"), job_run_path(@job, run), data: { turbo_frame: '_top' } %>
+        </td>
+        <td>
+          <%= run.status.humanize %>
+        </td>
+        <td>
+          <%= number_to_human(run.duration || 0, units: { unit: "ms", thousand: "s" }, precision: 2) %>
+        </td>
+        <td>
+          <%= run.attempts %>
+        </td>
+        <td>
+          <%= display_tag_badges(run) %>
+        </td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<%= render "rails_pulse/components/table_pagination" %>

data/app/views/rails_pulse/jobs/_table.html.erb

@@ -0,0 +1,43 @@
+<% columns = [
+  { field: :name, label: 'Job Class', class: 'w-48', cell_class: 'truncate-cell' },
+  { field: :queue_name, label: 'Queue', class: 'w-32' },
+  { field: :runs_count, label: 'Total Runs', class: 'w-28' },
+  { field: :failures_count, label: 'Failures', class: 'w-24' },
+  { field: :retries_count, label: 'Retries', class: 'w-24' },
+  { field: :avg_duration, label: 'Avg Duration', class: 'w-32' },
+  { field: nil, label: 'Tags', class: 'w-32', sortable: false }
+] %>
+
+<table class="table table-fixed mbs-4" data-controller="rails-pulse--table-sort">
+  <%= render "rails_pulse/components/table_head", columns: columns %>
+
+  <tbody>
+    <% @table_data.each do |job| %>
+      <tr>
+        <td class="truncate-cell">
+          <%= link_to job.name, job_path(job), data: { turbo_frame: '_top' } %>
+        </td>
+        <td>
+          <%= job.queue_name || 'default' %>
+        </td>
+        <td>
+          <%= number_with_delimiter(job.runs_count) %>
+        </td>
+        <td>
+          <%= number_with_delimiter(job.failures_count) %>
+        </td>
+        <td>
+          <%= number_with_delimiter(job.retries_count) %>
+        </td>
+        <td>
+          <%= number_to_human(job.avg_duration || 0, units: { unit: "ms", thousand: "s" }, precision: 2) %>
+        </td>
+        <td>
+          <%= display_tag_badges(job) %>
+        </td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<%= render "rails_pulse/components/table_pagination" %>

data/app/views/rails_pulse/jobs/index.html.erb

@@ -0,0 +1,34 @@
+<%= render 'rails_pulse/components/page_header', show_active_filters: true, show_global_filters: true %>
+
+<% unless turbo_frame_request? %>
+  <div class="row">
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @total_runs_metric_card } %>
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @failure_rate_metric_card } %>
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @average_duration_metric_card } %>
+  </div>
+<% end %>
+
+<%= render 'rails_pulse/components/panel', { title: 'Job Classes' } do %>
+  <%= search_form_for @ransack_query, url: jobs_path, class: "flex items-center justify-between gap mb-4" do |form| %>
+    <div class="flex items-center grow gap">
+      <%= form.search_field :name_cont, placeholder: "Filter by job name", autocomplete: "off", class: "input" %>
+      <%= form.select :queue_name_eq,
+        options_for_select([["All queues", ""]] + @available_queues.map { |q| [q || "default", q] }, params.dig(:q, :queue_name_eq)),
+        {},
+        { class: "input" }
+      %>
+      <%= link_to "Reset", jobs_path, class: "btn btn--borderless show@md" if params.has_key?(:q) %>
+      <%= form.submit "Search", class: "btn show@sm" %>
+    </div>
+  <% end %>
+
+  <% if @jobs.any? %>
+    <%= turbo_frame_tag :index_table do %>
+      <%= render 'rails_pulse/jobs/table' %>
+    <% end %>
+  <% else %>
+    <%= render 'rails_pulse/components/empty_state',
+        title: 'No jobs found',
+        description: 'No background jobs have been executed yet.' %>
+  <% end %>
+<% end %>

data/app/views/rails_pulse/jobs/show.html.erb

@@ -0,0 +1,49 @@
+<%= render 'rails_pulse/components/page_header', taggable: @job, show_active_filters: true %>
+
+<% unless turbo_frame_request? %>
+  <div class="row">
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @total_runs_metric_card } %>
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @failure_rate_metric_card } %>
+    <%= render 'rails_pulse/components/metric_card', { class: "grid-item block", data: @average_duration_metric_card } %>
+  </div>
+<% end %>
+
+<%= render 'rails_pulse/components/panel', { title: 'Job Runs' } do %>
+  <%= search_form_for @ransack_query, url: job_path(@job), class: "flex items-center justify-between gap mb-4", data: { controller: "rails-pulse--custom-range" } do |form| %>
+    <div class="flex items-center grow gap">
+      <%= time_range_selector(form,
+        time_range_options: RailsPulse::JobsController::TIME_RANGE_OPTIONS,
+        selected_time_range: @selected_time_range,
+        mode: :recent_custom
+      ) %>
+
+      <%= form.select :status_eq,
+        options_for_select(
+          [["All Statuses", ""]] + RailsPulse::JobRun::STATUSES.map { |s| [s.humanize, s] },
+          params.dig(:q, :status_eq)
+        ),
+        {},
+        { class: "input" }
+      %>
+
+      <%= form.select :duration_gteq,
+        duration_threshold_filter_options(:job),
+        { selected: params.dig(:q, :duration_gteq) },
+        { class: "input" }
+      %>
+
+      <%= link_to "Reset", job_path(@job), class: "btn btn--borderless show@md" if params.has_key?(:q) %>
+      <%= form.submit "Search", class: "btn show@sm" %>
+    </div>
+  <% end %>
+
+  <% if @recent_runs.any? %>
+    <%= turbo_frame_tag :index_table do %>
+      <%= render 'rails_pulse/jobs/job_runs_table' %>
+    <% end %>
+  <% else %>
+    <%= render 'rails_pulse/components/empty_state',
+        title: 'No runs found',
+        description: 'This job has not been executed yet or no runs match the selected filters.' %>
+  <% end %>
+<% end %>

data/app/views/rails_pulse/operations/_operation_analysis_application.html.erb

@@ -6,35 +6,37 @@
   <dd><%= html_escape(action) %></dd>
 <% end %>
 
-<% total_db_time = operation.request.operations
-     .where(operation_type: ["sql"])
-     .sum(:duration) %>
-<% total_view_time = operation.request.operations
-     .where(operation_type: ["template", "partial", "layout", "collection"])
-     .sum(:duration) %>
-<% pure_controller_time = operation.duration - total_db_time - total_view_time %>
+<% if parent %>
+  <% total_db_time = parent.operations
+       .where(operation_type: ["sql"])
+       .sum(:duration) %>
+  <% total_view_time = parent.operations
+       .where(operation_type: ["template", "partial", "layout", "collection"])
+       .sum(:duration) %>
+  <% pure_controller_time = operation.duration - total_db_time - total_view_time %>
 
-<% if pure_controller_time > 0 %>
-  <dt>Pure Logic Time</dt>
-  <dd>
-    <%= pure_controller_time.round(1) %>ms
-    <% if pure_controller_time > 100 %>
-      - consider optimization
-    <% end %>
-  </dd>
-<% end %>
+  <% if pure_controller_time > 0 %>
+    <dt>Pure Logic Time</dt>
+    <dd>
+      <%= pure_controller_time.round(1) %>ms
+      <% if pure_controller_time > 100 %>
+        - consider optimization
+      <% end %>
+    </dd>
+  <% end %>
 
-<% db_operations_count = operation.request.operations
-     .where(operation_type: ["sql"])
-     .count %>
-<% if db_operations_count > 0 %>
-  <dt>Database Queries</dt>
-  <dd>
-    <%= db_operations_count %>
-    <% if db_operations_count > 10 %>
-      - potential N+1 queries
-    <% end %>
-  </dd>
+  <% db_operations_count = parent.operations
+       .where(operation_type: ["sql"])
+       .count %>
+  <% if db_operations_count > 0 %>
+    <dt>Database Queries</dt>
+    <dd>
+      <%= db_operations_count %>
+      <% if db_operations_count > 10 %>
+        - potential N+1 queries
+      <% end %>
+    </dd>
+  <% end %>
 <% end %>
 
 <% if operation.codebase_location.present? %>

data/app/views/rails_pulse/operations/_operation_analysis_view.html.erb

@@ -21,16 +21,18 @@
   <dd>Partial template - ensure data is pre-loaded in controller</dd>
 <% end %>
 
-<% view_start = operation.occurred_at %>
-<% view_end = operation.occurred_at + operation.duration %>
-<% concurrent_db_ops = operation.request.operations
-     .where(operation_type: ["sql"])
-     .where("occurred_at >= ? AND occurred_at <= ?", view_start, view_end)
-     .count %>
+<% if parent %>
+  <% view_start = operation.occurred_at %>
+  <% view_end = operation.occurred_at + operation.duration %>
+  <% concurrent_db_ops = parent.operations
+       .where(operation_type: ["sql"])
+       .where("occurred_at >= ? AND occurred_at <= ?", view_start, view_end)
+       .count %>
 
-<% if concurrent_db_ops > 0 %>
-  <dt>Queries During Rendering</dt>
-  <dd><%= concurrent_db_ops %> database <%= 'query'.pluralize(concurrent_db_ops) %></dd>
+  <% if concurrent_db_ops > 0 %>
+    <dt>Queries During Rendering</dt>
+    <dd><%= concurrent_db_ops %> database <%= 'query'.pluralize(concurrent_db_ops) %></dd>
+  <% end %>
 <% end %>
 
 <% if operation.duration > 50 %>

data/app/views/rails_pulse/operations/show.html.erb

@@ -32,19 +32,21 @@
         <%= @operation.duration.round(2) %> ms
       </dd>
 
-      <dt>Request Impact</dt>
-      <dd>
-        <% impact = (@operation.duration / @request.duration * 100).round(1) %>
-        <%= impact %>% of total request time
-      </dd>
+      <% if @parent %>
+        <dt><%= @request ? 'Request' : 'Job Run' %> Impact</dt>
+        <dd>
+          <% impact = (@operation.duration / @parent.duration * 100).round(1) %>
+          <%= impact %>% of total <%= @request ? 'request' : 'job run' %> time
+        </dd>
+      <% end %>
 
       <dt>Occurred At</dt>
       <dd><%= human_readable_occurred_at @operation.occurred_at %></dd>
 
       <% begin %>
-        <%= render partial: "operation_analysis_#{categorize_operation(@operation.operation_type)}", locals: { operation: @operation } %>
+        <%= render partial: "operation_analysis_#{categorize_operation(@operation.operation_type)}", locals: { operation: @operation, parent: @parent } %>
       <% rescue ActionView::MissingTemplate %>
-        <%= render partial: "operation_analysis_generic", locals: { operation: @operation } %>
+        <%= render partial: "operation_analysis_generic", locals: { operation: @operation, parent: @parent } %>
       <% end %>
 
       <% if @performance_context.any? %>
@@ -59,7 +61,7 @@
       <% if @related_operations.any? %>
         <dt>Related Operations</dt>
         <dd>
-          <%= pluralize(@related_operations.count, 'similar operation') %> in this request
+          <%= pluralize(@related_operations.count, 'similar operation') %> in this <%= @request ? 'request' : 'job run' %>
           <% if @related_operations.count > 2 %>
             <br>Potential N+1 query pattern detected
           <% end %>

data/app/views/rails_pulse/queries/_table.html.erb

@@ -1,17 +1,17 @@
 <% columns = [
-  { field: :normalized_sql, label: 'Query', class: 'w-auto' },
+  { field: :normalized_sql, label: 'Query', class: 'w-48', cell_class: 'truncate-cell' },
   { field: :avg_duration_sort, label: 'Average Query Time', class: 'w-44' },
   { field: :execution_count_sort, label: 'Executions', class: 'w-24' },
   { field: nil, label: 'Tags', class: 'w-32' }
 ] %>
 
-<table class="table mbs-4" data-controller="rails-pulse--table-sort">
+<table class="table table-fixed mbs-4" data-controller="rails-pulse--table-sort">
   <%= render "rails_pulse/components/table_head", columns: columns %>
 
   <tbody>
     <% @table_data.each do |summary| %>
       <tr>
-        <td class="whitespace-nowrap">
+        <td class="truncate-cell">
           <div>
             <%= link_to html_escape(truncate_sql(summary.normalized_sql)), query_path(summary.query_id), data: { turbo_frame: '_top', } %>
           </div>

data/app/views/rails_pulse/requests/_table.html.erb

@@ -1,12 +1,12 @@
 <% columns = [
-  { field: :occurred_at, label: 'Timestamp', class: 'w-36' },
-  { field: :route_path, label: 'Route', class: 'w-auto' },
+  { field: :route_path, label: 'Route', cell_class: 'truncate-cell' },
+  { field: :occurred_at, label: 'Occurred At', class: 'w-48' },
   { field: :duration, label: 'Response Time', class: 'w-36' },
   { field: :status, label: 'Status', class: 'w-20' },
   { field: nil, label: 'Tags', class: 'w-32' }
 ] %>
 
-<table class="table mbs-4" data-controller="rails-pulse--table-sort">
+<table class="table table-fixed mbs-4" data-controller="rails-pulse--table-sort">
   <%= render "rails_pulse/components/table_head", columns: columns %>
 
   <tbody>
@@ -21,11 +21,11 @@
         end
       %>
       <tr>
-        <td class="whitespace-nowrap">
-          <%= link_to human_readable_occurred_at(request.occurred_at), request_path(request), data: { turbo_frame: '_top' } %>
+        <td class="truncate-cell">
+          <%= link_to "#{request.route.path} #{request.route.method}", route_path(request.route), data: { turbo_frame: '_top' } %>
         </td>
         <td class="whitespace-nowrap">
-          <%= link_to "#{request.route.path} #{request.route.method}", route_path(request.route), data: { turbo_frame: '_top' } %>
+          <%= link_to human_readable_occurred_at(request.occurred_at), request_path(request), data: { turbo_frame: '_top' } %>
         </td>
         <td class="whitespace-nowrap">
           <span class="<%= performance_class %> font-medium">

data/app/views/rails_pulse/routes/_table.html.erb

@@ -1,18 +1,18 @@
 <% columns = [
-  { field: :route_path, label: 'Route', class: 'w-auto' },
+  { field: :route_path, label: 'Route', class: 'w-48', cell_class: 'truncate-cell' },
   { field: :avg_duration_sort, label: 'Average Response Time', class: 'w-48' },
   { field: :max_duration_sort, label: 'Max Response Time', class: 'w-44' },
   { field: :count_sort, label: 'Requests', class: 'w-24' },
   { field: nil, label: 'Tags', class: 'w-32' }
 ] %>
 
-<table class="table mbs-4" data-controller="rails-pulse--table-sort">
+<table class="table table-fixed mbs-4" data-controller="rails-pulse--table-sort">
   <%= render "rails_pulse/components/table_head", columns: columns %>
 
   <tbody>
     <% @table_data.each do |summary| %>
       <tr>
-        <td class="whitespace-nowrap"><%= link_to "#{summary.path} #{summary.route_method}", route_path(summary.route_id), data: { turbo_frame: '_top' } %></td>
+        <td class="truncate-cell"><%= link_to "#{summary.path} #{summary.route_method}", route_path(summary.route_id), data: { turbo_frame: '_top' } %></td>
         <td class="whitespace-nowrap"><%= summary.avg_duration.to_i %> ms</td>
         <td class="whitespace-nowrap"><%= summary.max_duration.to_i %> ms</td>
         <td class="whitespace-nowrap"><%= number_with_delimiter summary.count %></td>

data/app/views/rails_pulse/routes/show.html.erb

@@ -15,7 +15,7 @@
   data-rails-pulse--index-chart-id-value="route_responses_chart"
 >
   <div class="grid-item">
-    <%= render 'rails_pulse/components/panel', { title: 'Route Reqeusts', } do %>
+    <%= render 'rails_pulse/components/panel', { title: 'Route Requests', } do %>
       <%= search_form_for @ransack_query, url: route_path(@route), class: "flex items-center justify-between gap mb-4", data: { controller: "rails-pulse--custom-range" } do |form| %>
         <div class="flex items-center grow gap">
           <%= time_range_selector(form, time_range_options: RailsPulse::RoutesController::TIME_RANGE_OPTIONS, selected_time_range: @selected_time_range) %>

data/app/views/rails_pulse/tags/_tag_manager.html.erb

@@ -7,21 +7,14 @@
 
 <div
   id="tag_manager_<%= taggable_type %>_<%= taggable.id %>"
-  class="tag-manager"
+  class="tag-manager card pi-3 pb-2 text-sm shadow-xs"
 >
+  <% if current_tags.present? %>
+    Tags:
+    <% end %>
   <div class="tag-list">
     <% current_tags.each do |tag| %>
-      <span class="badge badge--secondary font-normal">
-        <%= tag %>
-        <%= button_to remove_tag_path(taggable_type, taggable.id, tag: tag),
-              method: :delete,
-              class: "tag-remove",
-              data: {
-                turbo_frame: "_top"
-              } do %>
-          <span aria-hidden="true">&times;</span>
-        <% end %>
-      </span>
+      <%= render_tag_badge(tag, variant: :secondary, removable: true, taggable_type: taggable_type, taggable_id: taggable.id) %>
     <% end %>
 
     <% if available_to_add.any? %>
@@ -35,7 +28,7 @@
           aria-haspopup="true"
           aria-controls="tag_menu_<%= taggable_type %>_<%= taggable.id %>"
         >
-          + tag
+          tag +
         </button>
 
         <div
@@ -62,7 +55,7 @@
                     },
                     style: "cursor: pointer;",
                     role: "menuitem" do %>
-                <%= tag %>
+                <%= tag.humanize %>
               <% end %>
             <% end %>
           </div>