roundhouse_ui 0.9.1 → 0.10.0

data/app/views/layouts/roundhouse_ui/application.html.erb

@@ -88,6 +88,39 @@
     .rh-st-paused { background:var(--panel-3); color:var(--muted); }
     .rh-pill { font-size:11px; padding:2px 8px; border-radius:999px; font-family:var(--mono); background:var(--panel-2); color:var(--muted); }
     .rh-pill-warn { background:rgba(226,165,63,.16); color:var(--warn); }
+    .rh-pill-tag { background:var(--panel-2); color:var(--text); border:1px solid var(--line); max-width:22ch; overflow:hidden; text-overflow:ellipsis; white-space:nowrap; vertical-align:middle; }
+    /* jid sits inline after the class now; dim it so it reads as a copy target
+       rather than competing with the job name. Still fully selectable. */
+    /* --muted, not --faint: the jid is secondary but still has to be readable
+       at a glance when you're comparing it against a log line. */
+    .rh-jid { color:var(--muted); opacity:1; font-size:11.5px; letter-spacing:.02em; margin-top:3px; }
+
+    /* Bulk actions. Two scopes — ticked rows vs every match — kept visually
+       separate, because confusing them destroys the wrong jobs. */
+    .rh-actionbar { display:flex; flex-wrap:wrap; align-items:stretch; gap:10px; margin-bottom:14px; }
+    .rh-actiongroup { display:flex; align-items:center; gap:7px; padding:7px 12px; border:1px solid var(--line-soft);
+                      border-radius:11px; background:var(--panel); }
+    .rh-actionlabel { font-size:11px; letter-spacing:.05em; text-transform:uppercase; color:var(--faint);
+                      font-weight:600; margin-right:3px; max-width:34ch; overflow:hidden; text-overflow:ellipsis;
+                      white-space:nowrap; }
+    .rh-actiongroup form { display:inline; margin:0; }
+    /* Icon-only observability link for table rows. Sized like a button so it
+       sits on the Actions baseline without stretching the row. */
+    .rh-trace-ico { display:inline-block; width:24px; text-align:center; padding:5px 0; border-radius:7px;
+                    border:1px solid transparent; text-decoration:none; }
+    .rh-trace-ico:hover { border-color:var(--accent); background:var(--panel-2); }
+    .rh-pill-link { text-decoration:none; border:1px solid transparent; }
+    .rh-pill-link:hover { border-color:var(--accent); color:var(--text); }
+    .rh-pill-link.is-on { border-color:var(--accent); color:var(--text); background:var(--panel-3); }
+
+    /* Quick filters: counts by tag, from the scan that already ran. */
+    .rh-quickfilters { display:flex; flex-wrap:wrap; align-items:center; gap:8px; margin-bottom:14px; }
+    .rh-qf { display:inline-flex; align-items:center; gap:7px; font-size:12px; text-decoration:none;
+             background:var(--panel); border:1px solid var(--line); border-radius:999px; padding:4px 10px; color:var(--muted); }
+    .rh-qf:hover { border-color:var(--accent); color:var(--text); }
+    .rh-qf.is-on { border-color:var(--accent); color:var(--text); background:var(--panel-2); }
+    .rh-qf-n { font-family:var(--mono); font-variant-numeric:tabular-nums; font-size:11px; color:var(--faint); }
+    .rh-qf.is-on .rh-qf-n { color:var(--accent); }
 
     .rh-btn { font:inherit; font-size:12px; background:var(--panel-2); color:var(--text); border:1px solid var(--line); padding:5px 11px; border-radius:7px; cursor:pointer; text-decoration:none; display:inline-block; }
     .rh-btn:hover { border-color:var(--accent); }
@@ -277,6 +310,7 @@
           .then(function (d) {
             setConn(true);
             apply(d);
+            if (d.queue_names) QUEUE_NAMES = d.queue_names; // feeds the ⌘K palette
             var now = Date.now();
             var backlog = d.enqueued + d.scheduled + d.retries;
             if (lastProcessed != null) {
@@ -352,7 +386,63 @@
         try { localStorage.setItem("rh-width", next); } catch (_) {}
         syncWidth();
       });
-      // command palette (⌘K)
+      // command palette (⌘K). Two halves: static navigation, and search actions
+      // built from whatever you type — paste a JID, a queue, or a squad and the
+      // palette offers the searches that will actually find it, rather than
+      // making you pick a page first and search again once you're there.
+      // Capability-gated exactly like the sidebar: a backend without a retry
+      // set must not be offered a search into one.
+      var SETS = [
+        { label: "Dead",      path: "<%= dead_set_path %>",  icon: "✕" },
+        <% if RoundhouseUi.backend.supports?(:retries) %>{ label: "Retries", path: "<%= retries_path %>", icon: "↻" },<% end %>
+        { label: "Scheduled", path: "<%= scheduled_path %>", icon: "◔" },
+        { label: "Errors",    path: "<%= errors_path %>",    icon: "⚠" }
+      ];
+      // Declared tag vocabulary and live queue names, so the palette can tell a
+      // squad from a queue from a free-text term.
+      var TAG_VOCAB = <%== (RoundhouseUi::Tags.filters || {}).to_json %>;
+      // Filled by the stats poll, which already loads queues for its count, so
+      // knowing the queue names costs no extra Redis work.
+      var QUEUE_NAMES = [];
+
+      function palQuery(q) {
+        var out = [], seen = {};
+        // A 12+ char hex string is a Sidekiq JID — offer the sets directly.
+        if (/^[0-9a-f]{12,}$/i.test(q)) {
+          SETS.forEach(function (s) {
+            out.push({ label: "Find job " + q + " in " + s.label, icon: s.icon,
+                       path: s.path + "?q=" + encodeURIComponent(q) });
+          });
+          return out;
+        }
+        // Exact squad / tag value → the structured filter, which is stricter
+        // and safer than a substring match.
+        Object.keys(TAG_VOCAB).forEach(function (key) {
+          TAG_VOCAB[key].forEach(function (value) {
+            if (value.toLowerCase().indexOf(q) !== 0) return;
+            seen[value] = true;
+            SETS.forEach(function (s) {
+              out.push({ label: key + ": " + value + " — " + s.label, icon: "◆",
+                         path: s.path + "?tag=" + encodeURIComponent(key + ":" + value) });
+            });
+          });
+        });
+        // Known queue name → filter by queue.
+        QUEUE_NAMES.forEach(function (name) {
+          if (name.toLowerCase().indexOf(q) !== 0) return;
+          SETS.slice(0, 3).forEach(function (s) {
+            out.push({ label: "Queue " + name + " — " + s.label, icon: "≣",
+                       path: s.path + "?queue=" + encodeURIComponent(name) });
+          });
+        });
+        // Always offer plain text search as a fallback.
+        SETS.forEach(function (s) {
+          out.push({ label: "Search " + s.label + " for “" + q + "”", icon: s.icon,
+                     path: s.path + "?q=" + encodeURIComponent(q) });
+        });
+        return out;
+      }
+
       var CMDS = [
         { label: "Dashboard", path: "<%= root_path %>", icon: "▦" },
         { label: "Metrics", path: "<%= metrics_path %>", icon: "▤" },
@@ -383,8 +473,11 @@
       }
       function palRender() {
         var input = palInputEl(), list = palListEl(); if (!input || !list) return;
-        var q = input.value.toLowerCase();
-        palFiltered = CMDS.filter(function (c) { return c.label.toLowerCase().indexOf(q) !== -1; });
+        var q = input.value.trim().toLowerCase();
+        var nav = CMDS.filter(function (c) { return c.label.toLowerCase().indexOf(q) !== -1; });
+        // Search actions lead once you've typed: with a query, finding the job
+        // is the intent; jumping to a page is the fallback.
+        palFiltered = q ? palQuery(q).concat(nav) : nav;
         if (palSel >= palFiltered.length) palSel = Math.max(0, palFiltered.length - 1);
         list.innerHTML = "";
         palFiltered.forEach(function (c, i) {
@@ -433,6 +526,17 @@
       });
 
       document.addEventListener("turbo:load", function () { startOnce(); syncTheme(); syncWidth(); setActiveNav(); restoreChartInterval(); wireChart(); draw(); });
+
+      // Dead set "select all". Delegated from document rather than bound to the
+      // checkbox, because a Turbo visit (searching, paging, arriving via the
+      // nav) replaces <body> — the old element and its listener are discarded,
+      // and a re-inserted body script carries a nonce the original page's CSP
+      // does not accept. document survives both, so this binds once and keeps
+      // working. See issue #26.
+      document.addEventListener("change", function (e) {
+        if (!e.target || e.target.id !== "rh-all") return;
+        document.querySelectorAll(".rh-check").forEach(function (cb) { cb.checked = e.target.checked; });
+      });
       document.addEventListener("DOMContentLoaded", function () { startOnce(); syncTheme(); syncWidth(); setActiveNav(); restoreChartInterval(); wireChart(); });
       document.addEventListener("visibilitychange", function () { if (!document.hidden) poll(); });
     })();
@@ -500,7 +604,7 @@
 
   <div id="rh-palette" class="rh-palette" data-turbo-permanent hidden>
     <div class="rh-palette-box">
-      <input id="rh-palette-input" type="text" placeholder="Jump to…  (type to filter, ↑↓ to move, ↵ to go)" autocomplete="off">
+      <input id="rh-palette-input" type="text" placeholder="Search a job ID, queue, squad, or class — or jump to a page" autocomplete="off">
       <div id="rh-palette-list" class="rh-palette-list"></div>
     </div>
   </div>

data/app/views/roundhouse_ui/dashboard/show.html.erb

@@ -57,7 +57,7 @@
     <% else %>
       <% @top_errors.each do |g| %>
         <div class="rh-insight-row">
-          <div class="rh-insight-main"><%= link_to g[:klass], errors_path(q: g[:klass]), class: "rh-joblink" %> <%= error_trace_link(klass: g[:klass], error: g[:error]) %><br><span class="rh-err"><%= g[:error] %></span></div>
+          <div class="rh-insight-main"><%= link_to g[:klass], errors_path(q: g[:klass]), class: "rh-joblink" %> <%= tag_badges(tags_for_class(g[:klass])) %> <%= error_trace_link(klass: g[:klass], error: g[:error]) %><br><span class="rh-err"><%= g[:error] %></span></div>
           <span class="rh-pill"><%= number_with_delimiter g[:count] %></span>
         </div>
       <% end %>

data/app/views/roundhouse_ui/dead/index.html.erb

@@ -2,39 +2,60 @@
 
 <form method="get" action="<%= dead_set_path %>" class="rh-search">
   <input type="search" name="q" value="<%= @query %>" placeholder="search class, jid, error, or arg value…">
+  <% if params[:tag].present? %><input type="hidden" name="tag" value="<%= params[:tag] %>"><% end %>
+  <% if params[:queue].present? %><input type="hidden" name="queue" value="<%= params[:queue] %>"><% end %>
 </form>
 
-<h2 class="rh-h2">Dead set · <%= number_with_delimiter @total %> jobs</h2>
+<%= render "roundhouse_ui/shared/tag_filter" %>
 
-<% if @query.present? %>
-  <div class="rh-bulkbar">
-    <%= button_to "↻ Retry all matching", bulk_all_dead_path, method: :post, params: { op: "retry", q: @query }, class: "rh-btn", form_class: "rh-inline", data: { turbo_confirm: "Retry every dead job matching “#{@query}” (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
-    <%= button_to "✕ Delete all matching", bulk_all_dead_path, method: :post, params: { op: "delete", q: @query }, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Permanently delete every dead job matching “#{@query}” (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
-    <span class="rh-sub">acts on every match for “<%= @query %>”, not just this page</span>
-  </div>
-<% end %>
+<%= set_heading("Dead set", showing: @jobs.size, total: @total, query: @query, tag: @tag) %>
 
-<%= form_with url: bulk_dead_path, method: :post, data: { turbo: false } do %>
-  <div class="rh-bulkbar">
-    <button type="submit" name="op" value="retry" class="rh-btn">↻ Retry selected</button>
-    <button type="submit" name="op" value="delete" class="rh-btn rh-btn-danger">✕ Delete selected</button>
-    <span class="rh-sub">tick rows, then choose an action</span>
+<%# One toolbar, two scopes. They used to be stacked bars whose captions were the
+    only thing distinguishing "the rows you ticked" from "every job that matches",
+    which is a dangerous distinction to bury. The selected-rows buttons live here
+    but submit the form wrapping the table, via the HTML form= attribute. %>
+<div class="rh-actionbar">
+  <div class="rh-actiongroup">
+    <span class="rh-actionlabel">Selected rows</span>
+    <button type="submit" form="rh-bulk-dead" name="op" value="retry" class="rh-btn">↻ Retry</button>
+    <button type="submit" form="rh-bulk-dead" name="op" value="delete" class="rh-btn rh-btn-danger">✕ Delete</button>
   </div>
+
+  <% if any_filter?(@query, @tag) %>
+    <div class="rh-actiongroup">
+      <span class="rh-actionlabel" title="<%= filter_description(@query, @tag) %>">
+        All <%= number_with_delimiter @total %> <%= filter_description(@query, @tag) %>
+      </span>
+      <%= button_to "↻ Retry", bulk_all_dead_path, method: :post, params: { op: "retry", q: @query, tag: params[:tag], queue: params[:queue] }, class: "rh-btn", form_class: "rh-inline", data: { turbo_confirm: "Retry every dead job #{filter_description(@query, @tag)} (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
+      <%= button_to "✕ Delete", bulk_all_dead_path, method: :post, params: { op: "delete", q: @query, tag: params[:tag], queue: params[:queue] }, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Permanently delete every dead job #{filter_description(@query, @tag)} (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
+    </div>
+  <% end %>
+</div>
+
+<%= form_with url: bulk_dead_path, method: :post, id: "rh-bulk-dead", data: { turbo: false } do %>
   <table class="rh-table">
-    <thead><tr><th style="width:28px"><input type="checkbox" id="rh-all" aria-label="select all"></th><th>Job</th><th>Last error</th><th class="r">Died</th></tr></thead>
+    <thead><tr><th style="width:28px"><input type="checkbox" id="rh-all" aria-label="select all"></th><th>Job</th><% if tag_column? %><th><%= tag_column_label %></th><% end %><th>Queue</th><th>Last error</th><th class="r">Died</th><th class="r">Actions</th></tr></thead>
     <tbody>
       <% if @jobs.empty? %>
-        <tr><td colspan="4" class="rh-empty"><%= @query.present? ? "No dead jobs match “#{@query}”." : "Dead set is empty 🎉" %></td></tr>
+        <tr><td colspan="<%= tag_column? ? 7 : 6 %>" class="rh-empty"><%= any_filter?(@query, @tag) ? "No dead jobs #{filter_description(@query, @tag)}." : "Dead set is empty 🎉" %></td></tr>
       <% else %>
         <% @jobs.each do |job| %>
           <tr>
             <td><input type="checkbox" name="jids[]" value="<%= job.jid %>" class="rh-check" aria-label="select <%= job.jid %>"></td>
-            <td><%= link_to job.klass, job_path(set: "dead", jid: job.jid), class: "rh-joblink" %><br><span class="rh-sub"><%= job.jid %></span> <%= trace_link(klass: job.klass, jid: job.jid, queue: job.queue) %><% if RoundhouseUi.allow_job_editing %> <%= link_to "edit", edit_job_path(set: "dead", jid: job.jid), class: "rh-trace" %><% end %></td>
+            <td><%= job_identity(job.klass, job.jid, job_path(set: "dead", jid: job.jid)) %></td>
+            <% if tag_column? %><td><%= tag_cell(tags_for(job)) %></td><% end %>
+            <td><%= queue_pill(job.queue, link: true) %></td>
             <td>
               <span class="rh-err"><%= job.item["error_class"] %></span>
-              <% if job.item["error_message"].present? %><br><span class="rh-sub"><%= truncate(job.item["error_message"], length: 90) %></span><% end %>
+              <% if job.item["error_message"].present? %><br><span class="rh-sub" title="<%= job.item["error_message"] %>"><%= truncate(job.item["error_message"], length: 90) %></span><% end %>
+            </td>
+            <td class="r"><%= job_time(job.at, overdue: "#{time_ago_in_words(job.at)} ago") %></td>
+            <td class="r">
+              <%= trace_icon(klass: job.klass, jid: job.jid, queue: job.queue) %>
+              <% if RoundhouseUi.allow_job_editing %><%= link_to "Edit", edit_job_path(set: "dead", jid: job.jid), class: "rh-btn" %><% end %>
+              <%= button_to "Retry", retry_dead_job_path(job.jid), method: :post, class: "rh-btn", form_class: "rh-inline" %>
+              <%= button_to "Delete", delete_dead_job_path(job.jid), method: :post, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Delete #{job.jid}?" } %>
             </td>
-            <td class="r"><%= time_ago_in_words(job.at) %> ago</td>
           </tr>
         <% end %>
       <% end %>
@@ -43,12 +64,5 @@
 <% end %>
 <%= render "roundhouse_ui/shared/pager" %>
 
-<script nonce="<%= content_nonce %>">
-  (function () {
-    var all = document.getElementById("rh-all");
-    if (!all) return;
-    all.addEventListener("change", function () {
-      document.querySelectorAll(".rh-check").forEach(function (cb) { cb.checked = all.checked; });
-    });
-  })();
-</script>
+<%# The select-all handler lives in the layout head, delegated from document, so
+    it survives Turbo body swaps. Don't reintroduce a body script here. #26 %>

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

@@ -1,23 +1,44 @@
 <% content_for :title, "Errors" %>
 
 <form method="get" action="<%= errors_path %>" class="rh-search">
-  <input type="search" name="q" value="<%= @query %>" placeholder="filter by job class or error…">
+  <input type="search" name="q" value="<%= @query %>" placeholder="filter by job class, error, or squad…">
+  <% if params[:tag].present? %><input type="hidden" name="tag" value="<%= params[:tag] %>"><% end %>
 </form>
 
-<h2 class="rh-h2">Errors · <%= @groups.size %> issues</h2>
+<%# Counts come from the same scan that built the groups, so this costs no extra
+    Redis work. Each chip narrows the list below rather than sending you elsewhere. %>
+<%= render "roundhouse_ui/shared/tag_filter", counts: @tag_counts %>
+
+<h2 class="rh-h2">
+  Errors · <%= @groups.size %> issues
+  <% if @tag %><span class="hint">filtered to <%= @tag[0] %>: <%= @tag[1] %></span><% end %>
+</h2>
+
 <table class="rh-table">
-  <thead><tr><th>Issue</th><th class="r">Occurrences</th><th>In</th><th>Queues</th><th class="r">Last seen</th></tr></thead>
+  <thead>
+    <tr>
+      <th>Issue</th>
+      <% if tag_column? %><th><%= tag_column_label %></th><% end %>
+      <th class="r">Occurrences</th>
+      <th>In</th>
+      <th>Queues</th>
+      <th class="r">Last seen</th>
+    </tr>
+  </thead>
   <tbody>
     <% if @groups.empty? %>
-      <tr><td colspan="5" class="rh-empty"><%= @query.present? ? "No issues match “#{@query}”." : "No failing jobs 🎉" %></td></tr>
+      <tr><td colspan="<%= tag_column? ? 6 : 5 %>" class="rh-empty">
+        <%= @query.present? || @tag ? "No issues match the current filter." : "No failing jobs 🎉" %>
+      </td></tr>
     <% else %>
       <% @groups.each do |g| %>
         <tr>
-          <td><%= g[:klass] %> <%= error_trace_link(klass: g[:klass], error: g[:error]) %><br><span class="rh-err"><%= g[:error] %></span></td>
+          <td><%= g[:klass] %> <%= error_trace_icon(klass: g[:klass], error: g[:error]) %><br><span class="rh-err"><%= g[:error] %></span></td>
+          <% if tag_column? %><td><%= tag_cell(@group_tags[g[:klass]]) %></td><% end %>
           <td class="r"><%= number_with_delimiter g[:count] %></td>
           <td><span class="rh-sub"><%= g[:sources].join(", ") %></span></td>
-          <td><span class="rh-sub"><%= g[:queues].join(", ") %></span></td>
-          <td class="r"><%= g[:last_at] ? "#{time_ago_in_words(g[:last_at])} ago" : "—" %></td>
+          <td><%= safe_join(g[:queues].map { |q| queue_pill(q) }, " ") %></td>
+          <td class="r"><%= job_time(g[:last_at], overdue: "#{time_ago_in_words(g[:last_at])} ago" ) if g[:last_at] %><%= content_tag(:span, "—", class: "rh-sub") unless g[:last_at] %></td>
         </tr>
       <% end %>
     <% end %>

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

@@ -5,6 +5,7 @@
   <span class="rh-tag"><%= @set %></span>
   <span class="rh-tag">queue <%= @entry.queue %></span>
   <% if item["retry_count"] %><span class="rh-tag">retry <%= item["retry_count"] %></span><% end %>
+  <%= tag_badges(tags_for(@entry)) %>
   <span class="rh-tag rh-mono"><%= @entry.jid %></span>
 </div>
 

data/app/views/roundhouse_ui/queues/index.html.erb

@@ -1,5 +1,9 @@
 <% content_for :title, "Queues" %>
 
+<form method="get" action="<%= queues_path %>" class="rh-search">
+  <input type="search" name="q" value="<%= @query %>" placeholder="filter queues by name…">
+</form>
+
 <% if RoundhouseUi.allow_job_editing %>
   <div class="rh-toolbar"><%= link_to "+ Enqueue job", new_job_path, class: "rh-btn rh-btn-primary" %></div>
 <% end %>
@@ -16,12 +20,12 @@
   <thead><tr><th>Queue</th><th>State</th><th class="r">Size</th><th class="r">Latency</th><th class="r">Actions</th></tr></thead>
   <tbody>
     <% if @queues.empty? %>
-      <tr><td colspan="5" class="rh-empty">No active queues</td></tr>
+      <tr><td colspan="5" class="rh-empty"><%= @query.present? ? "No queues match “#{@query}”." : "No active queues" %></td></tr>
     <% else %>
       <% @queues.each do |q| %>
         <% paused = @paused.include?(q.name) %>
         <tr>
-          <td><%= q.name %></td>
+          <td><%= queue_pill(q.name) %></td>
           <td><% if paused %><span class="rh-pill rh-pill-warn">paused</span><% else %><span class="rh-sub">active</span><% end %></td>
           <td class="r"><%= number_with_delimiter q.size %></td>
           <td class="r <%= "rh-lat-warn" if q.latency > 60 %>"><%= q.latency.round(2) %>s</td>

data/app/views/roundhouse_ui/retries/index.html.erb

@@ -2,34 +2,45 @@
 
 <form method="get" action="<%= retries_path %>" class="rh-search">
   <input type="search" name="q" value="<%= @query %>" placeholder="search class, jid, error, or arg value…">
+  <% if params[:tag].present? %><input type="hidden" name="tag" value="<%= params[:tag] %>"><% end %>
+  <% if params[:queue].present? %><input type="hidden" name="queue" value="<%= params[:queue] %>"><% end %>
 </form>
 
-<h2 class="rh-h2">Retries · <%= number_with_delimiter @total %> jobs</h2>
+<%= render "roundhouse_ui/shared/tag_filter" %>
 
-<% if @query.present? %>
-  <div class="rh-bulkbar">
-    <%= button_to "↻ Run all matching now", bulk_all_retries_path, method: :post, params: { op: "retry", q: @query }, class: "rh-btn", form_class: "rh-inline", data: { turbo_confirm: "Run every retry matching “#{@query}” now (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
-    <%= button_to "✕ Delete all matching", bulk_all_retries_path, method: :post, params: { op: "delete", q: @query }, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Permanently delete every retry matching “#{@query}” (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
-    <span class="rh-sub">acts on every match for “<%= @query %>”, not just this page</span>
+<%= set_heading("Retries", showing: @jobs.size, total: @total, query: @query, tag: @tag) %>
+
+<% if any_filter?(@query, @tag) %>
+  <div class="rh-actionbar">
+    <div class="rh-actiongroup">
+      <span class="rh-actionlabel" title="<%= filter_description(@query, @tag) %>">
+        All <%= number_with_delimiter @total %> <%= filter_description(@query, @tag) %>
+      </span>
+      <%= button_to "↻ Run now", bulk_all_retries_path, method: :post, params: { op: "retry", q: @query, tag: params[:tag], queue: params[:queue] }, class: "rh-btn", form_class: "rh-inline", data: { turbo_confirm: "Run every retry #{filter_description(@query, @tag)} now (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
+      <%= button_to "✕ Delete", bulk_all_retries_path, method: :post, params: { op: "delete", q: @query, tag: params[:tag], queue: params[:queue] }, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Permanently delete every retry #{filter_description(@query, @tag)} (up to #{RoundhouseUi::JobSetBrowsing::BULK_CAP})?" } %>
+    </div>
   </div>
 <% end %>
 
 <table class="rh-table">
-  <thead><tr><th>Job</th><th>Last error</th><th class="r">Attempt</th><th class="r">Next try</th><th class="r">Actions</th></tr></thead>
+  <thead><tr><th>Job</th><% if tag_column? %><th><%= tag_column_label %></th><% end %><th>Queue</th><th>Last error</th><th class="r">Attempt</th><th class="r">Next try</th><th class="r">Actions</th></tr></thead>
   <tbody>
     <% if @jobs.empty? %>
-      <tr><td colspan="5" class="rh-empty"><%= @query.present? ? "No retries match “#{@query}”." : "Nothing waiting to retry 🎉" %></td></tr>
+      <tr><td colspan="<%= tag_column? ? 7 : 6 %>" class="rh-empty"><%= any_filter?(@query, @tag) ? "No retries #{filter_description(@query, @tag)}." : "Nothing waiting to retry 🎉" %></td></tr>
     <% else %>
       <% @jobs.each do |job| %>
         <tr>
-          <td><%= link_to job.klass, job_path(set: "retry", jid: job.jid), class: "rh-joblink" %><br><span class="rh-sub"><%= job.jid %></span> <%= trace_link(klass: job.klass, jid: job.jid, queue: job.queue) %></td>
+          <td><%= job_identity(job.klass, job.jid, job_path(set: "retry", jid: job.jid)) %></td>
+          <% if tag_column? %><td><%= tag_cell(tags_for(job)) %></td><% end %>
+          <td><%= queue_pill(job.queue, link: true) %></td>
           <td>
             <span class="rh-err"><%= job.item["error_class"] %></span>
-            <% if job.item["error_message"].present? %><br><span class="rh-sub"><%= truncate(job.item["error_message"], length: 90) %></span><% end %>
+            <% if job.item["error_message"].present? %><br><span class="rh-sub" title="<%= job.item["error_message"] %>"><%= truncate(job.item["error_message"], length: 90) %></span><% end %>
           </td>
-          <td class="r">#<%= job.item["retry_count"].to_i + 1 %></td>
-          <td class="r"><span class="rh-sub"><%= job.at && job.at > Time.now ? "in #{distance_of_time_in_words(Time.now, job.at)}" : "now" %></span></td>
+          <td class="r"><%= job.item["retry_count"].to_i + 1 %></td>
+          <td class="r"><%= job_time(job.at, overdue: "now") %></td>
           <td class="r">
+            <%= trace_icon(klass: job.klass, jid: job.jid, queue: job.queue) %>
             <% if RoundhouseUi.allow_job_editing %><%= link_to "Edit", edit_job_path(set: "retry", jid: job.jid), class: "rh-btn" %><% end %>
             <%= button_to "Run now", run_retry_path(job.jid), method: :post, class: "rh-btn", form_class: "rh-inline" %>
             <%= button_to "Delete", delete_retry_path(job.jid), method: :post, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Delete #{job.jid}?" } %>

data/app/views/roundhouse_ui/scheduled/index.html.erb

@@ -2,21 +2,27 @@
 
 <form method="get" action="<%= scheduled_path %>" class="rh-search">
   <input type="search" name="q" value="<%= @query %>" placeholder="search class, jid, or arg value…">
+  <% if params[:tag].present? %><input type="hidden" name="tag" value="<%= params[:tag] %>"><% end %>
+  <% if params[:queue].present? %><input type="hidden" name="queue" value="<%= params[:queue] %>"><% end %>
 </form>
 
-<h2 class="rh-h2">Scheduled · <%= number_with_delimiter @total %> jobs</h2>
+<%= render "roundhouse_ui/shared/tag_filter" %>
+
+<%= set_heading("Scheduled", showing: @jobs.size, total: @total, query: @query, tag: @tag) %>
 <table class="rh-table">
-  <thead><tr><th>Job</th><th>Queue</th><th class="r">Runs</th><th class="r">Actions</th></tr></thead>
+  <thead><tr><th>Job</th><% if tag_column? %><th><%= tag_column_label %></th><% end %><th>Queue</th><th class="r">Runs</th><th class="r">Actions</th></tr></thead>
   <tbody>
     <% if @jobs.empty? %>
-      <tr><td colspan="4" class="rh-empty"><%= @query.present? ? "No scheduled jobs match “#{@query}”." : "Nothing scheduled" %></td></tr>
+      <tr><td colspan="<%= tag_column? ? 5 : 4 %>" class="rh-empty"><%= any_filter?(@query, @tag) ? "No scheduled jobs #{filter_description(@query, @tag)}." : "Nothing scheduled" %></td></tr>
     <% else %>
       <% @jobs.each do |job| %>
         <tr>
-          <td><%= link_to job.klass, job_path(set: "scheduled", jid: job.jid), class: "rh-joblink" %><br><span class="rh-sub"><%= job.jid %></span></td>
-          <td><span class="rh-sub"><%= job.queue %></span></td>
-          <td class="r"><span class="rh-sub"><%= job.at && job.at > Time.now ? "in #{distance_of_time_in_words(Time.now, job.at)}" : "now (overdue)" %></span></td>
+          <td><%= job_identity(job.klass, job.jid, job_path(set: "scheduled", jid: job.jid)) %></td>
+          <% if tag_column? %><td><%= tag_cell(tags_for(job)) %></td><% end %>
+          <td><%= queue_pill(job.queue, link: true) %></td>
+          <td class="r"><%= job_time(job.at) %></td>
           <td class="r">
+            <%= trace_icon(klass: job.klass, jid: job.jid, queue: job.queue) %>
             <% if RoundhouseUi.allow_job_editing %><%= link_to "Edit", edit_job_path(set: "scheduled", jid: job.jid), class: "rh-btn" %><% end %>
             <%= button_to "Enqueue now", enqueue_scheduled_path(job.jid), method: :post, class: "rh-btn", form_class: "rh-inline" %>
             <%= button_to "Delete", delete_scheduled_path(job.jid), method: :post, class: "rh-btn rh-btn-danger", form_class: "rh-inline", data: { turbo_confirm: "Delete #{job.jid}?" } %>

data/app/views/roundhouse_ui/shared/_pager.html.erb

@@ -1,13 +1,13 @@
 <% if @page > 1 || @has_next %>
   <div class="rh-pager">
     <% if @page > 1 %>
-      <%= link_to "← Prev", url_for(page: @page - 1, q: @query.presence), class: "rh-btn" %>
+      <%= link_to "← Prev", url_for(page: @page - 1, q: @query.presence, tag: params[:tag].presence, queue: params[:queue].presence), class: "rh-btn" %>
     <% else %>
       <span class="rh-btn" style="opacity:.4; pointer-events:none">← Prev</span>
     <% end %>
     <span class="rh-sub">Page <%= @page %></span>
     <% if @has_next %>
-      <%= link_to "Next →", url_for(page: @page + 1, q: @query.presence), class: "rh-btn" %>
+      <%= link_to "Next →", url_for(page: @page + 1, q: @query.presence, tag: params[:tag].presence, queue: params[:queue].presence), class: "rh-btn" %>
     <% else %>
       <span class="rh-btn" style="opacity:.4; pointer-events:none">Next →</span>
     <% end %>

data/app/views/roundhouse_ui/shared/_tag_filter.html.erb

@@ -0,0 +1,30 @@
+<%# Tag filter chips (ADR 0002 step 4), shared by the job sets and Errors.
+    Errors passes `counts` because grouping already scanned every entry; the job
+    sets don't, since browse deliberately reads only enough to fill one page and
+    counting would turn a 25-row render into a full-set scan. %>
+<%# .presence matters: an empty counts hash is truthy, so treating it as "we have
+    counts" while falling back to the uncounted vocabulary made the two branches
+    disagree and blew up sorting a Hash-of-Arrays as if it were Hash-of-counts. %>
+<% counts = local_assigns[:counts].presence %>
+<% if RoundhouseUi.job_tags %>
+  <% vocabulary = counts || tag_vocabulary %>
+  <% if vocabulary.present? %>
+    <div class="rh-quickfilters">
+      <% vocabulary.each do |key, values| %>
+        <% values = values.sort_by { |_, count| -count } if counts %>
+        <% values.each do |value, count| %>
+          <% active = @tag == [ key.to_s, value.to_s ] %>
+          <%= link_to url_for(only_path: true, page: nil, q: @query.presence,
+                              tag: (active ? nil : "#{key}:#{value}"),
+                              queue: params[:queue].presence),
+                class: "rh-qf#{' is-on' if active}", title: "#{key}: #{value}" do %>
+            <%= value %><% if count %><span class="rh-qf-n"><%= number_with_delimiter count %></span><% end %>
+          <% end %>
+        <% end %>
+      <% end %>
+      <% if @tag %>
+        <%= link_to "clear", url_for(only_path: true, tag: nil, page: nil, q: @query.presence, queue: params[:queue].presence), class: "rh-sub" %>
+      <% end %>
+    </div>
+  <% end %>
+<% end %>

data/lib/roundhouse_ui/error_groups.rb

@@ -14,6 +14,7 @@ module RoundhouseUi
     def initialize(query: nil, limit: DEFAULT_SCAN_LIMIT)
       @query = query.to_s.strip
       @limit = limit
+      @tag_cache = {}
     end
 
     def call
@@ -34,12 +35,22 @@ module RoundhouseUi
       end
 
       list = groups.values.sort_by { |g| -g[:count] }
-      list = list.select { |g| "#{g[:klass]} #{g[:error]}".downcase.include?(@query.downcase) } if @query.present?
+      list = list.select { |g| matches?(g) } if @query.present?
       Result.new(groups: list, scanned: scanned, truncated: truncated)
     end
 
     private
 
+    # Tag values are part of the haystack here for the same reason they are on
+    # the job sets: typing a squad name should find that squad's failures. A
+    # group's entries all share a class, so the tag is constant for the group
+    # and resolves once per group rather than per scanned entry.
+    def matches?(group)
+      haystack = [ group[:klass], group[:error] ]
+      haystack.concat(Tags.for(klass: group[:klass], item: {}, cache: @tag_cache).values) if RoundhouseUi.job_tags
+      haystack.join(" ").downcase.include?(@query.downcase)
+    end
+
     # Sidekiq's native sets, plus the sidekiq-failures `failed` set when opted in
     # and loaded. Its FailureSet is a Sidekiq::JobSet, so it iterates like the rest.
     def sources