roundhouse_ui 0.9.0 → 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/backends/sidekiq.rb

@@ -14,10 +14,17 @@ module RoundhouseUi
       def name = "Sidekiq"
 
       # Capabilities let the UI hide what a backend can't do. Sidekiq supports
-      # all the sets/views; pause is NOT native (needs the fetcher), so it does
-      # not advertise :native_pause — the "not enforced" warning still applies.
+      # all the sets/views. On OSS Sidekiq pause is NOT native (it needs our
+      # fetcher), so :native_pause is withheld and the "not enforced" warning
+      # applies — but Sidekiq Pro ships its own enforced pause, so there it is
+      # advertised and the warning drops away.
       CAPABILITIES = %i[retries dead scheduled busy workers redis capsules].freeze
-      def supports?(capability) = CAPABILITIES.include?(capability)
+
+      def supports?(capability)
+        return RoundhouseUi::Pause.native? if capability == :native_pause
+
+        CAPABILITIES.include?(capability)
+      end
 
       def stats        = ::Sidekiq::Stats.new
       def queues       = ::Sidekiq::Queue.all

data/lib/roundhouse_ui/cancel_middleware.rb

@@ -7,9 +7,15 @@ module RoundhouseUi
   #   Sidekiq.configure_server do |config|
   #     config.server_middleware { |chain| chain.add RoundhouseUi::CancelMiddleware }
   #   end
+  #
+  # Hot-path cost: near zero. Cancellation.pending? answers "anything cancelled
+  # at all?" from a process-local gate (one EXISTS per CHECK_EVERY seconds, not
+  # per job); the exact per-job SISMEMBER only runs while cancellations are
+  # actually pending. State lives on the Cancellation module because Sidekiq
+  # builds a fresh middleware instance per job — ivars here wouldn't survive.
   class CancelMiddleware
     def call(_worker, job, _queue)
-      if RoundhouseUi::Cancellation.cancelled?(job["jid"])
+      if RoundhouseUi::Cancellation.pending? && RoundhouseUi::Cancellation.cancelled?(job["jid"])
         RoundhouseUi::Cancellation.clear!(job["jid"])
         return # acknowledge without running
       end

data/lib/roundhouse_ui/cancellation.rb

@@ -12,6 +12,9 @@ module RoundhouseUi
   module Cancellation
     KEY = "roundhouse:cancelled"
     TTL = 86_400 # seconds
+    CHECK_EVERY = 2.0 # seconds — max staleness of the "nothing is cancelled" gate
+
+    @gate = Mutex.new # guards @pending / @checked_at (see .pending?)
 
     module_function
 
@@ -20,6 +23,12 @@ module RoundhouseUi
         conn.call("SADD", KEY, jid.to_s)
         conn.call("EXPIRE", KEY, TTL)
       end
+      # Bust the local gate: the cancelling process sees its own cancel
+      # immediately; other processes converge within CHECK_EVERY.
+      @gate.synchronize do
+        @pending = true
+        @checked_at = monotonic_now
+      end
     end
 
     def cancelled?(jid)
@@ -33,5 +42,39 @@ module RoundhouseUi
     def cancelled_jids
       Sidekiq.redis { |conn| conn.call("SMEMBERS", KEY) }
     end
+
+    # The middleware's hot-path gate: "is anything cancelled at all?" — almost
+    # always no, so answering it with one EXISTS per process per CHECK_EVERY
+    # (instead of a SISMEMBER per job) takes the common case to zero Redis
+    # round-trips. While cancellations are pending, the middleware still does
+    # the exact per-job SISMEMBER. A cancel issued by *another* process is
+    # invisible here for up to CHECK_EVERY — acceptable because cancellation
+    # is cooperative and racy by design (a job may already be running when the
+    # flag lands), and cancel! busts the local gate so the cancelling process
+    # itself is always exact.
+    def pending?
+      now = monotonic_now
+      @gate.synchronize do
+        if @checked_at.nil? || now - @checked_at >= CHECK_EVERY
+          @pending = any?
+          @checked_at = now
+        end
+        @pending
+      end
+    end
+
+    def any?
+      Sidekiq.redis { |conn| conn.call("EXISTS", KEY) } == 1
+    end
+
+    # Forget the cached gate state (tests; or after flushing Redis by hand).
+    def reset_gate!
+      @gate.synchronize do
+        @pending = nil
+        @checked_at = nil
+      end
+    end
+
+    def monotonic_now = Process.clock_gettime(Process::CLOCK_MONOTONIC)
   end
 end