sidekiq 8.1.6 → 8.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e36b05d6e25f174a0ff8b8a21b4c097a108e4ccf98af88a56ae767cfea1186a
-  data.tar.gz: c58b4f89a1466bac6c9de05cf5ea2aee0152ac42184946b4d1646c6938b351a8
+  metadata.gz: e02b7ba4c390eb2a743bc62a8938cedd64ee940a6e128b2c517d9c3f27649543
+  data.tar.gz: 86b1e8925322fb00b3d443c6aa8762d7d06f9a564276bf1b210c0415c8d571b4
 SHA512:
-  metadata.gz: 3b8690078001bb93fc6e4fb8fe4d52091d89c317d8dcef56d24247882f4c85dbe986a05ae5451aaf70b9ff25c4acbc89b2d15e909763550bbcb13f36e73b2f7f
-  data.tar.gz: 3cabd5a3a2861af5faf70c5f67d8eb93c578d58cb7825dc063e350f30ef37dc3e2c51060a58dcf2fca9915df1bb7fcbfd00d81996ba651c29882c8432545b03a
+  metadata.gz: 63fd4147cf1a9ed566f47266a0c88eb966756285f4404714f2487783a1b754604ab00d5d9b4514002a116598f6005776c6db305c8d66192ec5c8e9b84af71595
+  data.tar.gz: a904f9f6963083f76f5645990f068f31b5f3a1a13376d73744b693d6f2e0e2d551be420d70b9c6f97d3ae3ae6691140d21af6357e208e275228ba0061e92df14

data/Changes.md

@@ -2,6 +2,12 @@
 
 [Sidekiq Changes](https://github.com/sidekiq/sidekiq/blob/main/Changes.md) | [Sidekiq Pro Changes](https://github.com/sidekiq/sidekiq/blob/main/Pro-Changes.md) | [Sidekiq Enterprise Changes](https://github.com/sidekiq/sidekiq/blob/main/Ent-Changes.md)
 
+8.1.7
+----------
+
+- Forward compatibililty with Active Job 8.1 [#7019]
+- Many minor fixes and test improvements [hammadxcm]
+
 8.1.6
 ----------
 

data/README.md

@@ -3,6 +3,7 @@ Sidekiq
 
 [![Gem Version](https://badge.fury.io/rb/sidekiq.svg)](https://rubygems.org/gems/sidekiq)
 ![Build](https://github.com/sidekiq/sidekiq/workflows/CI/badge.svg)
+[![Ruby Users Forum](https://img.shields.io/discourse/topics?server=https%3A%2F%2Fwww.rubyforum.org&style=flat&logo=discourse&label=Ruby%20Users%20Forum)](https://www.rubyforum.org/tag/sidekiq)
 
 Simple, efficient background jobs for Ruby.
 
@@ -88,6 +89,7 @@ Useful resources:
 * Product documentation is in the [wiki](https://github.com/sidekiq/sidekiq/wiki).
 * Occasional announcements are made to the [@sidekiq](https://ruby.social/@sidekiq) Mastodon account.
 * The [Sidekiq tag](https://stackoverflow.com/questions/tagged/sidekiq) on Stack Overflow has lots of useful Q & A.
+* Join us in the `sidekiq` tag on the [Ruby Users Forum](https://www.rubyforum.org/tag/sidekiq).
 
 Every Thursday morning is Sidekiq Office Hour: I video chat and answer questions.
 See the [Sidekiq support page](https://sidekiq.org/support/) for details.

data/bin/sidekiqload

@@ -1,5 +1,7 @@
 #!/usr/bin/env ruby
 
+ENV["RUBY_TCP_NO_FAST_FALLBACK"] = "1"
+
 #
 # bin/sidekiqload is a helpful script to load test and
 # performance tune Sidekiq's core. It creates 500,000 no-op

data/lib/active_job/queue_adapters/sidekiq_adapter.rb

@@ -111,7 +111,7 @@ begin
         end
 
         # @api private
-        def stopping? = !!@@stopping
+        def stopping?(job = nil) = !!@@stopping
 
         # Defines a class alias for backwards compatibility with enqueued Active Job jobs.
         # @api private

data/lib/sidekiq/api.rb

@@ -234,7 +234,7 @@ module Sidekiq
     end
 
     class History
-      def initialize(days_previous, start_date = nil, pool: nil)
+      def initialize(days_previous, start_date = nil)
         # we only store five years of data in Redis
         raise ArgumentError if days_previous < 1 || days_previous > (5 * 365)
         @days_previous = days_previous
@@ -627,6 +627,7 @@ module Sidekiq
       Sidekiq.redis do |conn|
         conn.zincrby(@parent.name, at.to_f - score, Sidekiq.dump_json(@item))
       end
+      @score = at.to_f
     end
 
     # Enqueue this job from the scheduled or dead set so it will
@@ -1415,7 +1416,7 @@ module Sidekiq
     private
 
     # Bulk-fetch iteration state for multiple JIDs in a single Redis pipeline.
-    # Returns a Hash of { jid => IterableJobState } for JIDs that have iteration state.
+    # Returns a Hash of { jid => State } for JIDs that have iteration state.
     def bulk_fetch(jids)
       raise ArgumentError unless jids
       jids_to_fetch = Array(jids).compact.uniq
@@ -1452,7 +1453,7 @@ module Sidekiq
         @cursor ||= begin
           Sidekiq.load_json(raw["c"])
         rescue JSON::ParserError
-          @raw["c"]
+          raw["c"]
         end
       end
 

data/lib/sidekiq/client.rb

@@ -142,9 +142,6 @@ module Sidekiq
       raise ArgumentError, "Job 'at' must be a Numeric or an Array of Numeric timestamps" if at && (Array(at).empty? || !Array(at).all? { |entry| entry.is_a?(Numeric) })
       raise ArgumentError, "Job 'at' Array must have same size as 'args' Array" if at.is_a?(Array) && at.size != args.size
 
-      # Use a smaller batch size by default for scheduled jobs since adding to sorted sets is more costly.
-      batch_size = items.delete(:batch_size) || items.delete("batch_size") || (at ? 100 : 1_000)
-
       jid = items.delete("jid")
       raise ArgumentError, "Explicitly passing 'jid' when pushing more than one job is not supported" if jid && args.size > 1
 
@@ -159,6 +156,9 @@ module Sidekiq
         at = args.map { now + rand * spread_interval }
       end
 
+      # Use a smaller batch size by default for scheduled jobs since adding to sorted sets is more costly.
+      batch_size = items.delete(:batch_size) || items.delete("batch_size") || (at ? 100 : 1_000)
+
       normed = normalize_item(items)
       slice_index = 0
       result = args.each_slice(batch_size).flat_map do |slice|

data/lib/sidekiq/job_retry.rb

@@ -72,8 +72,8 @@ module Sidekiq
 
     DEFAULT_MAX_RETRY_ATTEMPTS = 25
 
-    def initialize(capsule)
-      @config = @capsule = capsule
+    def initialize(cfg)
+      @config = cfg
       @max_retries = Sidekiq.default_configuration[:max_retries] || DEFAULT_MAX_RETRY_ATTEMPTS
       @backtrace_cleaner = Sidekiq.default_configuration[:backtrace_cleaner]
     end
@@ -96,7 +96,7 @@ module Sidekiq
       if msg["retry"]
         process_retry(nil, msg, queue, e)
       else
-        @capsule.config.death_handlers.each do |handler|
+        @config[:death_handlers].each do |handler|
           handler.call(msg, e)
         rescue => handler_ex
           handle_exception(handler_ex, {context: "Error calling death handler", job: msg})
@@ -273,7 +273,7 @@ module Sidekiq
     end
 
     def run_death_handlers(job, exception)
-      @capsule.config.death_handlers.each do |handler|
+      @config[:death_handlers].each do |handler|
         handler.call(job, exception)
       rescue => e
         handle_exception(e, {context: "Error calling death handler", job: job})
@@ -288,8 +288,8 @@ module Sidekiq
       redis do |conn|
         conn.multi do |xa|
           xa.zadd("dead", now.to_s, payload)
-          xa.zremrangebyscore("dead", "-inf", now - @capsule.config[:dead_timeout_in_seconds])
-          xa.zremrangebyrank("dead", 0, - @capsule.config[:dead_max_jobs])
+          xa.zremrangebyscore("dead", "-inf", now - @config[:dead_timeout_in_seconds])
+          xa.zremrangebyrank("dead", 0, - @config[:dead_max_jobs])
         end
       end
     end

data/lib/sidekiq/launcher.rb

@@ -235,7 +235,7 @@ module Sidekiq
     MEMORY_GRABBER = case RUBY_PLATFORM
     when /linux/
       ->(pid) {
-        IO.readlines("/proc/#{$$}/status").each do |line|
+        IO.readlines("/proc/#{pid}/status").each do |line|
           next unless line.start_with?("VmRSS:")
           break line.split[1].to_i
         end

data/lib/sidekiq/metrics/query.rb

@@ -169,7 +169,12 @@ module Sidekiq
 
       def fetch_marks(time_range, granularity)
         [].tap do |result|
-          marks = @pool.with { |c| c.hgetall("#{@time.strftime("%Y%m%d")}-marks") }
+          days = (time_range.begin.utc.to_date..time_range.end.utc.to_date)
+          marks = @pool.with { |c|
+            c.pipelined { |pipeline|
+              days.each { |day| pipeline.hgetall("#{day.strftime("%Y%m%d")}-marks") }
+            }
+          }.flat_map(&:to_a)
 
           marks.each do |timestamp, label|
             time = Time.parse(timestamp)

data/lib/sidekiq/paginator.rb

@@ -60,6 +60,9 @@ module Sidekiq
 
     def page_items(items, pageidx = 1, page_size = 25)
       current_page = (pageidx.to_i < 1) ? 1 : pageidx.to_i
+      # A negative page_size makes Array#[] return nil instead of a slice, which
+      # would crash callers iterating the result (e.g. the Web UI Busy page).
+      page_size = 0 if page_size < 0
       pageidx = current_page - 1
       starting = pageidx * page_size
       items = items.to_a

data/lib/sidekiq/processor.rb

@@ -30,15 +30,16 @@ module Sidekiq
     attr_reader :capsule
 
     def initialize(capsule, &block)
-      @config = @capsule = capsule
+      @config = capsule.config
+      @capsule = capsule
       @callback = block
       @down = false
       @done = false
       @job = nil
       @thread = nil
       @reloader = Sidekiq.default_configuration[:reloader]
-      @job_logger = (capsule.config[:job_logger] || Sidekiq::JobLogger).new(capsule.config)
-      @retrier = Sidekiq::JobRetry.new(capsule)
+      @job_logger = (capsule.config[:job_logger] || Sidekiq::JobLogger).new(@config)
+      @retrier = Sidekiq::JobRetry.new(@capsule)
     end
 
     def terminate(wait = false)
@@ -64,7 +65,7 @@ module Sidekiq
     end
 
     def start
-      @thread ||= safe_thread("#{config.name}/processor", &method(:run))
+      @thread ||= safe_thread("#{capsule.name}/processor", &method(:run))
     end
 
     private

data/lib/sidekiq/tui/tabs/base_tab.rb

@@ -140,31 +140,57 @@ module Sidekiq
 
       def render_stats_section(tui, frame, area)
         stats = @data[:stats]
-
         keys = ["Processed", "Failed", "Busy", "Enqueued", "Retries", "Scheduled", "Dead"]
-        values = [
-          stats[:processed],
-          stats[:failed],
-          stats[:busy],
-          stats[:enqueued],
-          stats[:retries],
-          stats[:scheduled],
-          stats[:dead]
-        ]
-
-        # Format keys and values with spacing
-        keys_line = keys.map { |k| t(k).to_s.ljust(12) }.join("  ")
-        values_line = values.map { |v| number_with_delimiter(v).ljust(12) }.join("  ")
+        values = %i[processed failed busy enqueued retries scheduled dead].map { |k| number_with_delimiter(stats[k]) }
+        render_kv_section(tui, frame, area, title: "Statistics", keys:, values:)
+      end
+
+      # Render a bordered paragraph with a line of (translated) keys over
+      # a line of preformatted values, each column ljust'ed to +width+.
+      def render_kv_section(tui, frame, area, title:, keys:, values:, width: 12)
+        keys_line = keys.map { |k| t(k).to_s.ljust(width) }.join("  ")
+        values_line = values.map { |v| v.to_s.ljust(width) }.join("  ")
 
         frame.render_widget(
           tui.paragraph(
             text: [keys_line, values_line],
-            block: tui.block(title: "Statistics", borders: [:all])
+            block: tui.block(title:, borders: [:all])
           ),
           area
         )
       end
 
+      # Split a tab's area into the standard stats header and a fill
+      # area for the tab's main content.
+      def stats_content_split(tui, area)
+        tui.layout_split(
+          area,
+          direction: :vertical,
+          constraints: [
+            tui.constraint_length(4), # Stats
+            tui.constraint_fill(1) # Content
+          ]
+        )
+      end
+
+      def striped_rows(tui, rows)
+        rows.map.with_index { |cells, idx|
+          tui.table_row(
+            cells:,
+            style: idx.even? ? nil : tui.style(bg: :dark_gray)
+          )
+        }
+      end
+
+      def chart_y_axis(tui, y_max, num_labels: 5)
+        labels = (0...num_labels).map { |i| ((y_max * i) / (num_labels - 1)).round.to_s }
+        tui.axis(
+          bounds: [0.0, y_max.to_f],
+          labels: labels,
+          style: tui.style(fg: :white)
+        )
+      end
+
       # [thousands_separator, decimal_separator] per locale.
       # Locales not listed here use the English default [",", "."].
       NUMERIC_SEPARATORS = {

data/lib/sidekiq/tui/tabs/busy.rb

@@ -77,12 +77,7 @@ module Sidekiq
                 tui.constraint_length(6),
                 tui.constraint_length(6)
               ],
-              rows: @data[:busy].map.with_index { |cells, idx|
-                tui.table_row(
-                  cells:,
-                  style: idx.even? ? nil : tui.style(bg: :dark_gray)
-                )
-              }
+              rows: striped_rows(tui, @data[:busy])
             }
           end
         end
@@ -101,16 +96,7 @@ module Sidekiq
           values << format_memory(processes.total_rss)
 
           keys = %w[Processes Threads Busy Utilization RSS]
-          keys_line = keys.map { |k| t(k).to_s.ljust(12) }.join("  ")
-          values_line = values.map { |v| v.to_s.ljust(12) }.join("  ")
-
-          frame.render_widget(
-            tui.paragraph(
-              text: [keys_line, values_line],
-              block: tui.block(title: t("Status"), borders: [:all])
-            ),
-            area
-          )
+          render_kv_section(tui, frame, area, title: t("Status"), keys:, values:)
         end
       end
     end

data/lib/sidekiq/tui/tabs/dead.rb

@@ -8,11 +8,6 @@ module Sidekiq
         include SetTab
 
         def set_class = Sidekiq::DeadSet
-
-        def refresh_data
-          refresh_data_for_stats
-          refresh_data_for_set
-        end
       end
     end
   end

data/lib/sidekiq/tui/tabs/home.rb

@@ -63,31 +63,16 @@ module Sidekiq
           max_value = [@data[:chart][:deltas][:processed].max, @data[:chart][:deltas][:failed].max, 1].max
           y_max = [max_value, 5].max
 
-          processed_data = @data[:chart][:deltas][:processed].each_with_index.map { |value, idx| [idx.to_f, value.to_f] }
-          failed_data = @data[:chart][:deltas][:failed].each_with_index.map { |value, idx| [idx.to_f, value.to_f] }
-
-          datasets = [
+          datasets = {processed: :green, failed: :red}.map { |key, color|
+            data = @data[:chart][:deltas][key].each_with_index.map { |value, idx| [idx.to_f, value.to_f] }
             tui.dataset(
               name: "",
-              data: processed_data,
-              style: tui.style(fg: :green),
-              marker: :dot,
-              graph_type: :line
-            ),
-            tui.dataset(
-              name: "",
-              data: failed_data,
-              style: tui.style(fg: :red),
+              data: data,
+              style: tui.style(fg: color),
               marker: :dot,
               graph_type: :line
             )
-          ]
-
-          num_labels = 5
-          y_labels = (0...num_labels).map do |i|
-            value = ((y_max * i) / (num_labels - 1)).round
-            value.to_s
-          end
+          }
 
           beacon_pulse = (Time.now.to_i % 2 == 0) ? "●" : " "
 
@@ -98,11 +83,7 @@ module Sidekiq
               labels: [],
               style: tui.style(fg: :white)
             ),
-            y_axis: tui.axis(
-              bounds: [0.0, y_max.to_f],
-              labels: y_labels,
-              style: tui.style(fg: :white)
-            ),
+            y_axis: chart_y_axis(tui, y_max),
             block: tui.block(
               title: "Dashboard #{beacon_pulse}",
               borders: [:all]
@@ -119,24 +100,14 @@ module Sidekiq
 
           keys = ["Version", "Uptime", "Connected Clients", "Memory Usage", "Peak Memory"]
           values = [
-            redis_info[:version].to_s,
+            redis_info[:version],
             uptime_value,
-            redis_info[:connected_clients].to_s,
-            redis_info[:used_memory].to_s,
-            redis_info[:peak_memory].to_s
+            redis_info[:connected_clients],
+            redis_info[:used_memory],
+            redis_info[:peak_memory]
           ]
 
-          # Format keys and values with spacing
-          keys_line = keys.map { |k| t(k).ljust(18) }.join("  ")
-          values_line = values.map { |v| v.ljust(18) }.join("  ")
-
-          frame.render_widget(
-            tui.paragraph(
-              text: [keys_line, values_line],
-              block: tui.block(title: "Redis Information", borders: [:all])
-            ),
-            area
-          )
+          render_kv_section(tui, frame, area, title: "Redis Information", keys:, values:, width: 18)
         end
       end
     end

data/lib/sidekiq/tui/tabs/metrics.rb

@@ -33,15 +33,7 @@ module Sidekiq
         end
 
         def render(tui, frame, area)
-          chunks = tui.layout_split(
-            area,
-            direction: :vertical,
-            constraints: [
-              tui.constraint_length(4), # Stats
-              tui.constraint_fill(1) # Chart
-              # TOOD Table
-            ]
-          )
+          chunks = stats_content_split(tui, area)
 
           render_stats_section(tui, frame, chunks[0])
           render_metrics_chart(tui, frame, chunks[1])
@@ -93,11 +85,6 @@ module Sidekiq
               graph_type: :line)
           end
 
-          num_labels = 5
-          y_labels = (0...num_labels).map do |i|
-            value = ((y_max * i) / (num_labels - 1)).round
-            value.to_s
-          end
           xlabels = [
             q.starts_at.iso8601[11..15],
             q.ends_at.iso8601[11..15]
@@ -112,11 +99,7 @@ module Sidekiq
               labels: xlabels,
               style: tui.style(fg: :white)
             ),
-            y_axis: tui.axis(
-              bounds: [0.0, y_max.to_f],
-              labels: y_labels,
-              style: tui.style(fg: :white)
-            ),
+            y_axis: chart_y_axis(tui, y_max),
             block: tui.block(
               title: t(name),
               borders: [:all]

data/lib/sidekiq/tui/tabs/queues.rb

@@ -63,14 +63,7 @@ module Sidekiq
           header = ["☑️", "Queue", "Size", "Latency"].map { |x| t(x) }
           header << "Paused?" if Sidekiq.pro?
 
-          chunks = tui.layout_split(
-            area,
-            direction: :vertical,
-            constraints: [
-              tui.constraint_length(4), # Stats
-              tui.constraint_fill(1) # Table
-            ]
-          )
+          chunks = stats_content_split(tui, area)
 
           render_stats_section(tui, frame, chunks[0])
           render_table(tui, frame, chunks[1]) do
@@ -80,12 +73,7 @@ module Sidekiq
               widths: header.map.with_index { |_, idx|
                 tui.constraint_length((idx == 1) ? 60 : 10)
               },
-              rows: @data[:queues].map.with_index { |cells, idx|
-                tui.table_row(
-                  cells:,
-                  style: idx.even? ? nil : tui.style(bg: :dark_gray)
-                )
-              }
+              rows: striped_rows(tui, @data[:queues])
             }
           end
         end

data/lib/sidekiq/tui/tabs/retries.rb

@@ -8,11 +8,6 @@ module Sidekiq
         include SetTab
 
         def set_class = Sidekiq::RetrySet
-
-        def refresh_data
-          refresh_data_for_stats
-          refresh_data_for_set
-        end
       end
     end
   end

data/lib/sidekiq/tui/tabs/scheduled.rb

@@ -8,11 +8,6 @@ module Sidekiq
         include SetTab
 
         def set_class = Sidekiq::ScheduledSet
-
-        def refresh_data
-          refresh_data_for_stats
-          refresh_data_for_set
-        end
       end
     end
   end

data/lib/sidekiq/tui/tabs/set_tab.rb

@@ -30,6 +30,11 @@ module Sidekiq
           end
         end
 
+        def refresh_data
+          refresh_data_for_stats
+          refresh_data_for_set
+        end
+
         def refresh_data_for_set
           set = set_class.new
           f = current_filter
@@ -52,14 +57,7 @@ module Sidekiq
         end
 
         def render(tui, frame, area)
-          chunks = tui.layout_split(
-            area,
-            direction: :vertical,
-            constraints: [
-              tui.constraint_length(4), # Stats
-              tui.constraint_fill(1)   # Table
-            ]
-          )
+          chunks = stats_content_split(tui, area)
 
           render_stats_section(tui, frame, chunks[0])
           render_table(tui, frame, chunks[1]) do
@@ -74,19 +72,16 @@ module Sidekiq
                 tui.constraint_fill(1)
               ]
             }.tap do |h|
-              rows = @data[:table][:rows].map.with_index { |entry, idx|
-                tui.table_row(
-                  cells: [
-                    selected?(entry) ? "✅" : "",
-                    entry.at,
-                    entry.queue,
-                    entry.display_class,
-                    entry.display_args
-                  ],
-                  style: idx.even? ? nil : tui.style(bg: :dark_gray)
-                )
+              cells = @data[:table][:rows].map { |entry|
+                [
+                  selected?(entry) ? "✅" : "",
+                  entry.at,
+                  entry.queue,
+                  entry.display_class,
+                  entry.display_args
+                ]
               }
-              h[:rows] = rows
+              h[:rows] = striped_rows(tui, cells)
             end
           end
         end

data/lib/sidekiq/tui.rb

@@ -105,63 +105,46 @@ module Sidekiq
                 @tui.constraint_length(4)
               ]
             )
-            content = @tui.block(
-              title: " #{Sidekiq::NAME} ",
-              borders: [:all],
-              title_style: @tui.style(fg: :light_red, modifiers: [:bold]),
-              children: [
-                # TODO convert to table
-                @tui.paragraph(
-                  text: [
-                    @tui.text_line(spans: ["Welcome to the Sidekiq Terminal UI"], alignment: :center),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "Global hotkeys")
-                    ]),
-                    @tui.text_line(spans: []),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "Esc", style: @hotkey_style),
-                      @tui.text_span(content: ": Close this window")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "←/→", style: @hotkey_style),
-                      @tui.text_span(content: ": Move between tabs")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "h/l", style: @hotkey_style),
-                      @tui.text_span(content: ": Move to prev/next page of data")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "j/k", style: @hotkey_style),
-                      @tui.text_span(content: ": Move to prev/next row in current page")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "x", style: @hotkey_style),
-                      @tui.text_span(content: ": Select/deselect current row")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "A", style: @hotkey_style),
-                      @tui.text_span(content: ": Select/deselect All rows in current page")
-                    ]),
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "q", style: @hotkey_style),
-                      @tui.text_span(content: ": Quit")
-                    ])
-                  ]
-                )
-              ]
+
+            help_data = [
+              ["Esc", "Close this window"],
+              ["←/→", "Move between tabs"],
+              ["h/l", "Move to prev/next page of data"],
+              ["j/k", "Move to prev/next row in current page"],
+              ["x", "Select/deselect current row"],
+              ["A", "Select/deselect All rows in current page"],
+              ["q", "Quit"]
+            ]
+
+            # striped styling like other tables in the app
+            help_rows = help_data.map.with_index { |cells, idx|
+              @tui.table_row(
+                cells: cells,
+                style: idx.even? ? nil : @tui.style(bg: :dark_gray)
+              )
+            }
+
+            table = @tui.table(
+              block: @tui.block(
+                title: " #{Sidekiq::NAME} - Welcome to the Sidekiq Terminal UI ",
+                borders: [:all],
+                title_style: @tui.style(fg: :light_red, modifiers: [:bold])
+              ),
+              header: ["Key", "Action"],
+              rows: help_rows,
+              widths: [
+                @tui.constraint_length(15),
+                @tui.constraint_fill(1)
+              ],
+              column_spacing: 1
             )
-            frame.render_widget(content, main_area)
+            frame.render_widget(table, main_area)
             controls = @tui.block(
               title: t("Controls"),
               borders: [:all],
               children: [
                 @tui.paragraph(
-                  text: [
-                    @tui.text_line(spans: [
-                      @tui.text_span(content: "Esc", style: @hotkey_style),
-                      @tui.text_span(content: ": Close  ")
-                    ])
-                  ]
+                  text: [hotkey_line("Esc", "Close  ")]
                 )
               ]
             )
@@ -180,27 +163,36 @@ module Sidekiq
     def render_controls(frame, area)
       active_keys = current_tab.controls.filter { |hash| hash[:description] }
 
-      # Split controls into two lines, 8 is arbitrary
-      # TODO Dynamically split based on term width?
-      first = active_keys[...8]
+      # Dynamically split controls based on terminal width
+      # Estimate space needed: each control needs ~key_length + description_length + 4 chars for ": " and spacing
+      available_width = area.width - 4 # Account for borders
+
       lines = []
-      lines << @tui.text_line(spans: first.map { |hash|
-        [
-          @tui.text_span(content: hash[:display] || hash[:code], style: @hotkey_style),
-          @tui.text_span(content: ": #{t(hash[:description])}  ")
-        ]
-      }.flatten)
+      current_line = []
+      current_width = 0
+
+      active_keys.each do |hash|
+        # Estimate control width: key + ": " + description + "  " (spacing)
+        key_text = hash[:display] || hash[:code]
+        control_width = key_text.length + 2 + t(hash[:description]).length + 2
+
+        if current_width + control_width > available_width && !current_line.empty?
+          # Start a new line
+          lines << controls_line(current_line)
+          current_line = [hash]
+          current_width = control_width
+        else
+          current_line << hash
+          current_width += control_width
+        end
+      end
 
-      last = active_keys[8...]
-      lines << if last && last.size > 0
-        @tui.text_line(spans: last.map { |hash|
-          [
-            @tui.text_span(content: hash[:display] || hash[:code], style: @hotkey_style),
-            @tui.text_span(content: ": #{t(hash[:description])}  ")
-          ]
-        }.flatten)
-      else
-        @tui.text_line(spans: [])
+      # Add the last line
+      lines << controls_line(current_line) unless current_line.empty?
+
+      # Ensure we have at least 2 lines for layout consistency
+      while lines.length < 2
+        lines << @tui.text_line(spans: [])
       end
 
       footer = [
@@ -225,6 +217,22 @@ module Sidekiq
       frame.render_widget(controls, area)
     end
 
+    def hotkey_line(key, description)
+      @tui.text_line(spans: [
+        @tui.text_span(content: key, style: @hotkey_style),
+        @tui.text_span(content: ": #{description}")
+      ])
+    end
+
+    def controls_line(keys)
+      @tui.text_line(spans: keys.map { |hash|
+        [
+          @tui.text_span(content: hash[:display] || hash[:code], style: @hotkey_style),
+          @tui.text_span(content: ": #{t(hash[:description])}  ")
+        ]
+      }.flatten)
+    end
+
     def handle_input
       # We shouldn't need more than 10 FPS for a data-oriented app.
       # This throttles down our CPU usage. Default is 60 FPS.

data/lib/sidekiq/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Sidekiq
-  VERSION = "8.1.6"
+  VERSION = "8.1.7"
   MAJOR = 8
 
   def self.gem_version

data/sidekiq.gemspec

@@ -20,7 +20,8 @@ Gem::Specification.new do |gem|
     "documentation_uri" => "https://github.com/sidekiq/sidekiq/wiki",
     "changelog_uri" => "https://github.com/sidekiq/sidekiq/blob/main/Changes.md",
     "source_code_uri" => "https://github.com/sidekiq/sidekiq",
-    "rubygems_mfa_required" => "true"
+    "rubygems_mfa_required" => "true",
+    "mailing_list_uri" => "https://www.rubyforum.org/tag/sidekiq"
   }
 
   gem.add_dependency "redis-client", ">= 0.29.0"

data/web/locales/en.yml

@@ -108,3 +108,4 @@ en:
   View: View
   Token: Token
   ElapsedTime: Elapsed Time
+  ClickToLearnMore: Click to learn more

data/web/locales/hi.yml

@@ -8,42 +8,57 @@ hi:
   AreYouSureDeleteJob: क्या आप इस कार्य को हटाना चाहते है?
   AreYouSureDeleteQueue: क्या आप %{queue} कतार को हटाना चाहते है?
   Arguments: अर्गुमेन्ट्स्
+  BackToApp: ऐप पर वापस जाएँ
   Busy: व्यस्थ
   Class: क्लास
   Connections: कनेक्श्न
+  CreatedAt: बनाया गया
   CurrentMessagesInQueue: <span class='title'>%{queue}</span> कतार मे वर्तमान कार्य
   Dashboard: डैशबोर्ड
   Dead: निष्प्राण
   DeadJobs: निष्प्राण कार्य
   Delete: हटाओ
   DeleteAll: सब हटाओ
+  Deploy: परिनियोजन
   Enqueued: कतारबद्ध
   Error: एरर
   ErrorBacktrace: एरर बैकट्रेस
   ErrorClass: एरर क्लास
   ErrorMessage: एरर संदेश
+  ExecutionTime: निष्पादन समय
   Extras: अतिरिक्त
   Failed: असफल
   Failures: असफलता
+  Failure: असफलता
   GoBack: ← पीछे
   History: वृत्तान्त
   Job: कार्य
   Jobs: कार्य
   Kill: नष्ट करे
+  KillAll: सभी को समाप्त करें
+  Language: भाषा
+  LastDashboardUpdateTemplateLiteral: "नवीनतम पोल: संसाधित: PROCESSED_COUNT. असफल: FAILED_COUNT."
   LastRetry: अंतिम पुन:प्रयास
+  Latency: विलंब
   LivePoll: लाईव सर्वेक्षण
   MemoryUsage: मेमरी उपयोग
+  Name: नाम
   Namespace: नामस्थान
   NextRetry: अगला पुन:प्रयास
   NoDeadJobsFound: कोई निष्प्राण कार्य नही पाए गए
   NoRetriesFound: कोई पुनर्प्रयास नही पाए गए
   NoScheduledFound: कोई परिगणित कार्य नही पाए गए
+  NotYetEnqueued: अभी तक कतार में नहीं जोड़ा गया
   OneMonth: १ महीना
   OneWeek: १ सप्ताह
   OriginallyFailed: पहिले से विफल
+  Pause: रोकें
   Paused: थमे हुए
   PeakMemoryUsage: अधिकतम मेमरी उपयोग
+  Plugins: प्लगइन्स
   PollingInterval: सर्वेक्षण अंतराल
+  PollingIntervalMilliseconds: सर्वेक्षण अंतराल (मिलीसेकंड)
+  Process: प्रक्रिया
   Processed: कार्रवाई कृत
   Processes: प्रोसेसेस्
   Queue: कतार
@@ -57,6 +72,7 @@ hi:
   RetryNow: पुन:प्रयास करे
   Scheduled: परिगणित
   ScheduledJobs: परिगणित कार्य
+  Seconds: सेकंड
   ShowAll: सब दिखाएं
   SixMonths: ६ महीने
   Size: आकार
@@ -65,13 +81,31 @@ hi:
   Stop: रोको
   StopAll: सब रोको
   StopPolling: सर्वेक्षण रोको
+  Success: सफल
+  Summary: सारांश
   Thread: थ्रेड
   Threads: थ्रेड्स्
   ThreeMonths: ३ महीने
   Time: समय
+  Unpause: पुनः प्रारंभ करें
   Uptime: उपरिकाल (दिवस)
+  Utilization: उपयोग
   Version: वर्जन
   When: कब
   Worker: वर्कर
   active: सक्रिय
   idle: निष्क्रिय
+  Metrics: मेट्रिक्स
+  NoDataFound: कोई डेटा नहीं मिला
+  TotalExecutionTime: कुल निष्पादन समय
+  AvgExecutionTime: औसत निष्पादन समय
+  Context: संदर्भ
+  NoJobMetricsFound: हाल के कार्यों के लिए कोई मेट्रिक्स नहीं मिले
+  Filter: फ़िल्टर
+  AnyJobContent: कोई भी कार्य सामग्री
+  Profiles: प्रोफ़ाइल
+  Data: डेटा
+  View: देखें
+  Token: टोकन
+  ElapsedTime: बीता हुआ समय
+  ClickToLearnMore: अधिक जानने के लिए क्लिक करें

data/web/locales/ta.yml

@@ -5,73 +5,107 @@ ta:
   AddToQueue: வரிசையில் சேர்
   AddAllToQueue: அனைத்தையும் வரிசையில் சேர்
   AreYouSure: நீங்கள் உறுதியாக இருக்கிறீர்களா?
-  AreYouSureDeleteJob: நீ இந்த வேலையை நீக்க வேண்டும் என்று உறுதியாக இருக்கிறீர்களா?
-  AreYouSureDeleteQueue: நீங்கள் %{queue} வரிசையில் நீக்க வேண்டும் என்பதில் உறுதியாக இருக்கிறீர்களா?
-  Arguments: வாதங்கள்,
-  Busy: பணிமிகுதி
+  AreYouSureDeleteJob: இந்த பணியை நீக்க விரும்புகிறீர்களா?
+  AreYouSureDeleteQueue: "%{queue} வரிசையை நீக்க விரும்புகிறீர்களா? இது வரிசையிலுள்ள அனைத்து பணிகளையும் நீக்கிவிடும், எதிர்காலத்தில் மேலும் பணிகளை அனுப்பினால் இது மீண்டும் தோன்றும்."
+  Arguments: அளவுருக்கள்
+  BackToApp: பயன்பாட்டிற்குத் திரும்பு
+  Busy: செயலில்
   Class: வகுப்பு
   Connections: இணைப்புகள்
-  CurrentMessagesInQueue: தற்போதைய வேலைகள் <span class='title'>%{queue}</span>
-  Dashboard: டாஷ்போர்டு
-  Dead: இறந்துபோன
-  DeadJobs: டெட் வேலைகள்
+  CreatedAt: உருவாக்கப்பட்ட நேரம்
+  CurrentMessagesInQueue: <span class='title'>%{queue}</span> இல் உள்ள தற்போதைய பணிகள்
+  Dashboard: தகவற்பலகை
+  Dead: முடிந்தது
+  DeadJobs: முடிந்த பணிகள்
   Delete: நீக்கு
-  DeleteAll: அனைத்து நீக்கு
-  Enqueued: வரிசைப்படுத்தப்பட்டவை
+  DeleteAll: அனைத்தையும் நீக்கு
+  Deploy: வரிசைப்படுத்து
+  Enqueued: வரிசையிடப்பட்டது
   Error: பிழை
-  ErrorBacktrace: பிழை பின்தேடுலை
+  ErrorBacktrace: பிழை பின்தடம்
   ErrorClass: பிழை வகுப்பு
   ErrorMessage: பிழை செய்தி
-  Extras: உபரி
-  Failed: தோல்வி
-  Failures: தோல்விகள்
-  GoBack: பின்புறம்
+  ExecutionTime: செயல்படுத்தும் நேரம்
+  Extras: கூடுதல்
+  Failed: தோல்வியடைந்தது
+  Failures: தோல்விகள்
+  Failure: தோல்வி
+  GoBack: ← பின் செல்
   History: வரலாறு
-  Job: வேலை
-  Jobs: வேலை வாய்ப்புகள்
-  Kill: கொல்
-  LastRetry: கடைசியாக, மீண்டும் முயற்சிக்கவும்
-  LivePoll: நேரடி கணிப்பு
+  Job: பணி
+  Jobs: பணிகள்
+  Kill: நிறுத்து
+  KillAll: அனைத்தையும் நிறுத்து
+  Language: மொழி
+  LastDashboardUpdateTemplateLiteral: "கடைசி சோதனை: செயலாக்கப்பட்டவை: PROCESSED_COUNT. தோல்வியடைந்தவை: FAILED_COUNT."
+  LastRetry: கடைசி மறுமுயற்சி
+  Latency: தாமதம்
+  LivePoll: நேரடி சோதனை
   MemoryUsage: நினைவக பயன்பாடு
+  Name: பெயர்
   Namespace: பெயர்வெளி
-  NextRetry: அடுத்த, மீண்டும் முயற்சிக்கவும்
-  NoDeadJobsFound: இறந்த வேலை எதுவும் இல்லை
-  NoRetriesFound: இல்லை மீண்டும் காணப்படவில்லை
-  NoScheduledFound: திட்டமிட்ட வேலைகள் காணப்படவில்லை
+  NextRetry: அடுத்த மறுமுயற்சி
+  NoDeadJobsFound: முடிந்த பணிகள் எதுவும் கிடைக்கவில்லை
+  NoRetriesFound: மறுமுயற்சிகள் எதுவும் கிடைக்கவில்லை
+  NoScheduledFound: திட்டமிடப்பட்ட பணிகள் எதுவும் கிடைக்கவில்லை
+  NotYetEnqueued: இன்னும் வரிசையிடப்படவில்லை
   OneMonth: 1 மாதம்
   OneWeek: 1 வாரம்
-  OriginallyFailed: முதலில் தோல்வி
-  Paused: தற்காலிக பணிநிறுத்தம்
-  PeakMemoryUsage: உச்ச நினைவக பயன்பாடு
-  PollingInterval: வாக்குப்பதிவு இடைவெளி
-  Processed: நிறையுற்றது
+  OriginallyFailed: முதலில் தோல்வியடைந்தது
+  Pause: இடைநிறுத்து
+  Paused: இடைநிறுத்தப்பட்டது
+  PeakMemoryUsage: அதிகபட்ச நினைவக பயன்பாடு
+  Plugins: செருகுநிரல்கள்
+  PollingInterval: சோதனை இடைவெளி
+  PollingIntervalMilliseconds: சோதனை இடைவெளி மில்லிவினாடிகள்
+  Process: செயல்முறை
+  Processed: செயலாக்கப்பட்டது
   Processes: செயல்முறைகள்
   Queue: வரிசை
-  Queues: வரிசை
-  Quiet: அமைதியான
-  QuietAll: அமைதியான அனைத்து
-  Realtime: நேரலை
-  Retries: மீண்டும் முயற்சிக்க,
-  RetryAll: அனைத்து, மீண்டும் முயற்சிக்கவும்
-  RetryCount: கணிப்பீடு, மீண்டும் முயற்சிக்கவும்
-  RetryNow: இப்போது மீண்டும் முயற்சி செய்க
-  Scheduled: திட்டமிடப்பட்ட
-  ScheduledJobs: திட்டமிட்ட வேலைகள்
-  ShowAll: அனைத்து காட்டு
+  Queues: வரிசைகள்
+  Quiet: அமைதி
+  QuietAll: அனைத்தையும் அமைதிப்படுத்து
+  Realtime: நிகழ்நேரம்
+  Retries: மறுமுயற்சிகள்
+  RetryAll: அனைத்தையும் மறுமுயற்சி செய்
+  RetryCount: மறுமுயற்சி எண்ணிக்கை
+  RetryNow: இப்போது மறுமுயற்சி செய்
+  Scheduled: திட்டமிடப்பட்டது
+  ScheduledJobs: திட்டமிடப்பட்ட பணிகள்
+  Seconds: வினாடிகள்
+  ShowAll: அனைத்தையும் காட்டு
   SixMonths: 6 மாதங்கள்
   Size: அளவு
-  Started: தொடங்குதல்
-  Status: நிலைமை
+  Started: தொடங்கப்பட்டது
+  Status: நிலை
   Stop: நிறுத்து
-  StopAll: நிறுத்து அனைத்து
-  StopPolling: நிறுத்து வாக்குப்பதிவு
+  StopAll: அனைத்தையும் நிறுத்து
+  StopPolling: சோதனையை நிறுத்து
+  Success: வெற்றி
+  Summary: சுருக்கம்
   Thread: நூல்
   Threads: நூல்கள்
   ThreeMonths: 3 மாதங்கள்
   Time: நேரம்
-  Uptime: இயக்க நேரம் (நாட்கள்)
+  Unpause: இடைநிறுத்தத்தை நீக்கு
+  Uptime: இயங்கிய நேரம் (நாட்கள்)
+  Utilization: பயன்பாட்டு விகிதம்
   Version: பதிப்பு
-  When: எப்பொழுது?
+  When: எப்போது
   Worker: பணியாளர்
   active: செயலில்
-  idle: முடங்கு நேரம்
+  idle: செயலற்று
+  Metrics: அளவீடுகள்
+  NoDataFound: தரவு எதுவும் கிடைக்கவில்லை
+  TotalExecutionTime: மொத்த செயல்படுத்தும் நேரம்
+  AvgExecutionTime: சராசரி செயல்படுத்தும் நேரம்
+  Context: சூழல்
+  NoJobMetricsFound: சமீபத்திய பணி அளவீடுகள் எதுவும் கிடைக்கவில்லை
+  Filter: வடிகட்டி
+  AnyJobContent: எந்த பணி உள்ளடக்கமும்
+  Profiles: சுயவிவரங்கள்
+  Data: தரவு
+  View: பார்வை
+  Token: டோக்கன்
+  ElapsedTime: கடந்த நேரம்
+  ClickToLearnMore: மேலும் அறிய கிளிக் செய்யவும்

data/web/views/_footer.html.erb

@@ -11,7 +11,7 @@
         <p id="serverUtcTime"><%= server_utc_time %></p>
       </li>
       <li>
-        <p><a rel="help" href="https://github.com/sidekiq/sidekiq/wiki">docs</a></p>
+        <p><a rel="help" href="https://sidekiq.org/wiki">docs</a></p>
       </li>
       <li>
         <form id="locale-form" action="<%= root_path %>change_locale" method="post">

data/web/views/_job_info.html.erb

@@ -9,7 +9,7 @@
         <tr>
           <th><%= t('Queue') %></th>
           <td>
-            <a href="<%= root_path %>queues/<%= job.queue %>"><%= job.queue %></a>
+            <a href="<%= root_path %>queues/<%= CGI.escape(job.queue) %>"><%= h job.queue %></a>
           </td>
         </tr>
         <tr>

data/web/views/busy.html.erb

@@ -45,7 +45,7 @@
       <thead>
         <th><%= t('Name') %></th>
         <th><%= t('Started') %></th>
-        <th class="col-sm-1"><%= t('RSS') %><a target="blank" href="https://github.com/sidekiq/sidekiq/wiki/Memory#rss"><span class="info-circle" title="Click to learn more about RSS">?</span></a></th>
+        <th class="col-sm-1"><%= t('RSS') %><a target="blank" href="https://sidekiq.org/wiki/Memory#rss"><span class="info-circle" title="<%= t("ClickToLearnMore") %>">?</span></a></th>
         <th class="col-sm-1"><%= t('Threads') %></th>
         <th class="col-sm-1"><%= t('Busy') %></th>
         <th>&nbsp;</th>
@@ -132,7 +132,7 @@
             <td><%= thread %></td>
             <td><%= job.jid %></td>
             <td>
-              <a href="<%= root_path %>queues/<%= work.queue %>"><%= work.queue %></a>
+              <a href="<%= root_path %>queues/<%= CGI.escape(work.queue) %>"><%= h work.queue %></a>
             </td>
             <td>
               <%= job.display_class %>

data/web/views/metrics.html.erb

@@ -7,7 +7,7 @@
   <header>
     <h1>
       <%= t('Metrics') %>
-      <a target="blank" href="https://github.com/sidekiq/sidekiq/wiki/Metrics"><span class="info-circle" title="Click to learn more about metrics">?</span></a>
+      <a target="blank" href="https://sidekiq.org/wiki/Metrics"><span class="info-circle" title="<%= t("ClickToLearnMore") %>">?</span></a>
     </h1>
 
     <%= erb :_metrics_period_select, locals: { periods: @periods, period: @period, path: "#{root_path}metrics" } %>

data/web/views/metrics_for_job.html.erb

@@ -15,7 +15,7 @@
       <h1>
         <a href="<%= root_path %>metrics?period=<%= @period %>"><%= t('Metrics') %></a> /
         <%= h @name %>
-        <a target="blank" href="https://github.com/sidekiq/sidekiq/wiki/Metrics"><span class="info-circle" title="Click to learn more about metrics">?</span></a>
+        <a target="blank" href="https://sidekiq.org/wiki/Metrics"><span class="info-circle" title="<%= t("ClickToLearnMore") %>">?</span></a>
       </h1>
       <%= erb :_metrics_period_select, locals: { periods: @periods, period: @period, path: "#{root_path}metrics/#{@name}" } %>
     </header>

data/web/views/morgue.html.erb

@@ -37,7 +37,7 @@
                 <a href="<%= root_path %>morgue/<%= job_params(entry.item, entry.score) %>"><%= relative_time(entry.at) %></a>
               </td>
               <td>
-                <a href="<%= root_path %>queues/<%= entry.queue %>"><%= entry.queue %></a>
+                <a href="<%= root_path %>queues/<%= CGI.escape(entry.queue) if entry.queue %>"><%= h entry.queue %></a>
               </td>
               <td>
                 <%= entry.display_class %>

data/web/views/profiles.html.erb

@@ -2,7 +2,7 @@
   <header>
     <h1>
       <%= t('Profiles') %>
-      <a target="blank" href="https://github.com/sidekiq/sidekiq/wiki/Profiling"><span class="info-circle" title="Click to learn more about profiling">?</span></a>
+      <a target="blank" href="https://sidekiq.org/wiki/Profiling"><span class="info-circle" title="<%= t("ClickToLearnMore") %>">?</span></a>
     </h1>
   </header>
 

data/web/views/retries.html.erb

@@ -39,7 +39,7 @@
               </td>
               <td><%= entry['retry_count'] %></td>
               <td>
-                <a href="<%= root_path %>queues/<%= entry.queue %>"><%= entry.queue %></a>
+                <a href="<%= root_path %>queues/<%= CGI.escape(entry.queue) %>"><%= h entry.queue %></a>
               </td>
               <td>
                 <%= entry.display_class %>

data/web/views/scheduled.html.erb

@@ -36,7 +36,7 @@
                 <a href="<%= root_path %>scheduled/<%= job_params(entry.item, entry.score) %>"><%= relative_time(entry.at) %></a>
               </td>
               <td>
-                <a href="<%= root_path %>queues/<%= entry.queue %>"><%= entry.queue %></a>
+                <a href="<%= root_path %>queues/<%= CGI.escape(entry.queue) %>"><%= h entry.queue %></a>
               </td>
               <td>
                 <%= entry.display_class %>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sidekiq
 version: !ruby/object:Gem::Version
-  version: 8.1.6
+  version: 8.1.7
 platform: ruby
 authors:
 - Mike Perham
@@ -247,6 +247,7 @@ metadata:
   changelog_uri: https://github.com/sidekiq/sidekiq/blob/main/Changes.md
   source_code_uri: https://github.com/sidekiq/sidekiq
   rubygems_mfa_required: 'true'
+  mailing_list_uri: https://www.rubyforum.org/tag/sidekiq
 rdoc_options: []
 require_paths:
 - lib
@@ -261,7 +262,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 4.0.16
 specification_version: 4
 summary: Simple, efficient background processing for Ruby
 test_files: []