karafka-web 0.11.3 → 0.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7b958131581b6fe2fd1a45a68b7366fb842ca15eccd946988516e78b7fcc3cce
-  data.tar.gz: 899898076b7fc09111d8cfd77f5e68fe6158b9bfee3128292f77248f2b5922c8
+  metadata.gz: 24ceb5279e48629e0a1c3a609b1c041323bdedc9faf6cf03dec926c7c0278b4f
+  data.tar.gz: 4ea8f9aef69bb0ee9ac51cc20e132e64e955df2984f6f2359b1c878421793972
 SHA512:
-  metadata.gz: b53b0e2dc696cf5af719ad49b90c130a7122261a8123e5ac8ca4deb31d0f397e49f09d71032df4e840ad3a5f9b5b3115709fbcd08568a0991ee24b0da1b7054e
-  data.tar.gz: b3eeac33c925b9527aef85231a74a255c3d59aedd36f20a174e72b282db744cb6a0779258c9136266f54041ab0e81ce8c95a6a042d4ecc908e0cbba299d651f3
+  metadata.gz: 7861ba705d420ddd875053bb9f687302ad8d2719d332e925cab53075edf267245660e2467934c746f3707327f198538321585b77abe60fcb1bcb49c3c8a1771a
+  data.tar.gz: dd30a59da0ed8590eb471965d1f6e917b15f2a390eb099e4b1527ca1a490dda46695fb3a40cc1e96729f15377a2b74d3a361d593a8b6cf5747fb03ecd4efb461

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Karafka Web Changelog
 
+## 0.11.5 (2025-11-14)
+- [Enhancement] Utilize newly released Roda session management `:env_key` to isolate Karafka Web session from the main application session.
+
+## 0.11.4 (2025-11-01)
+- [Enhancement] Show placeholder rows for partitions with no data during rebalances in health view. The UI now displays all topic partitions (0 to N-1) with "No data available" indicators for partitions currently being rebalanced, preventing confusion from disappearing partitions. Consumer reports now include `partitions_cnt` field extracted from librdkafka statistics. Consumer schema version bumped to 1.5.0 (breaking change).
+- [Enhancement] Track and report UI errors originating from Roda/Puma web processes directly to Kafka errors topic for visibility and debugging. UI errors are dispatched asynchronously from web processes using a dedicated listener.
+- [Enhancement] Require Karafka 2.5.2 at minimum and migrate from string-based execution mode comparisons to the new ExecutionMode object API.
+- [Enhancement] Increase Web UI processing consumer backoff time to 30 seconds when encountering incompatible schema errors to prevent error spam during rolling upgrades.
+- [Enhancement] Add unique `id` field to error reports to track duplicate error occurrences. Error schema version bumped to 1.2.0 while maintaining backward compatibility with older error formats (1.0.0, 1.1.0) in the Web UI.
+- [Enhancement] Add container-aware metrics collection for Docker/Kubernetes environments. The Web UI now reports accurate container memory limits from cgroups (v1 and v2) instead of misleading host metrics, while maintaining full backward compatibility with non-containerized deployments.
+- [Enhancement] Add per-message report migration system to handle schema evolution for consumer reports. This allows transparent migration of old report formats (e.g., schema 1.2.x using `process[:name]`) to current expectations (schema 1.3.0+ using `process[:id]`), ensuring backward compatibility with reports from older karafka-web versions (≤ v0.8.2) that may still exist in Kafka topics.
+- [Change] Reduce `max_messages` for consumer reports processing from 1000 to 200 to prevent excessive memory usage in large-scale deployments. Processing 1000 messages at once can impact memory consumption significantly in big systems, while 200 messages provides better memory efficiency with negligible impact on throughput.
+- [Refactor] Extract metrics collection logic from monolithic Sampler into focused, single-responsibility classes (Metrics::Base, Metrics::Os, Metrics::Container, Metrics::Network, Metrics::Server, Metrics::Jobs) and consumer groups enrichment into dedicated enricher (Enrichers::Base, Enrichers::ConsumerGroups) for improved maintainability and testability.
+- [Testing] Add Docker-based integration tests for container metrics collection. Tests verify cgroup v1/v2 detection, memory limit reading, and fallback behavior across multiple containerized scenarios with different resource constraints.
+- [Fix] Fix "OS memory used" metric on Linux reporting same value as RSS instead of system-wide memory usage. The metric now correctly sums memory usage across all processes (or all container processes when running in Docker/Kubernetes) to match macOS behavior and original design intent.
+- [Fix] Fix crash when processing old consumer reports from schema versions < 1.3.0 (karafka-web ≤ v0.8.2) that used `process[:name]` field instead of `process[:id]`. The error `undefined method 'to_sym' for nil` would occur when these old reports were encountered during upgrades. Reports are now automatically migrated in-place to the current schema format.
+
 ## 0.11.3 (2025-09-29)
 - [Enhancement] Upgrade DaisyUI to 5.1.
 - [Change] Remove Ruby `3.1` support according to the EOL schedule.

data/Gemfile

@@ -2,8 +2,6 @@
 
 source 'https://rubygems.org'
 
-plugin 'diffend'
-
 gemspec
 
 group :test do
@@ -16,4 +14,5 @@ group :test do
   gem 'rack-test'
   gem 'rspec'
   gem 'simplecov'
+  gem 'yard-lint'
 end

data/Gemfile.lock

@@ -1,41 +1,41 @@
 PATH
   remote: .
   specs:
-    karafka-web (0.11.3)
+    karafka-web (0.11.5)
       erubi (~> 1.4)
-      karafka (>= 2.5.0, < 2.6.0)
+      karafka (>= 2.5.2, < 2.6.0)
       karafka-core (>= 2.5.0, < 2.6.0)
-      roda (~> 3.68, >= 3.69)
+      roda (~> 3.69, >= 3.69)
       tilt (~> 2.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.2.2.2)
+    activesupport (8.1.1)
       base64
-      benchmark (>= 0.3)
       bigdecimal
       concurrent-ruby (~> 1.0, >= 1.3.1)
       connection_pool (>= 2.2.5)
       drb
       i18n (>= 1.6, < 2)
+      json
       logger (>= 1.4.2)
       minitest (>= 5.1)
       securerandom (>= 0.3)
       tzinfo (~> 2.0, >= 2.0.5)
-    base64 (0.2.0)
-    benchmark (0.4.1)
-    bigdecimal (3.1.9)
+      uri (>= 0.13.1)
+    base64 (0.3.0)
+    bigdecimal (3.3.1)
     byebug (12.0.0)
     concurrent-ruby (1.3.5)
-    connection_pool (2.5.3)
+    connection_pool (2.5.4)
     diff-lcs (1.6.2)
     docile (1.4.1)
     drb (2.2.3)
     erubi (1.13.1)
-    et-orbi (1.2.11)
+    et-orbi (1.4.0)
       tzinfo
-    factory_bot (6.5.5)
+    factory_bot (6.5.6)
       activesupport (>= 6.1.0)
     ffi (1.17.2)
     ffi (1.17.2-aarch64-linux-gnu)
@@ -48,43 +48,54 @@ GEM
     ffi (1.17.2-x86_64-darwin)
     ffi (1.17.2-x86_64-linux-gnu)
     ffi (1.17.2-x86_64-linux-musl)
-    fugit (1.11.2)
-      et-orbi (~> 1, >= 1.2.11)
+    fugit (1.12.1)
+      et-orbi (~> 1.4)
       raabro (~> 1.4)
     i18n (1.14.7)
       concurrent-ruby (~> 1.0)
-    karafka (2.5.0)
+    json (2.16.0)
+    karafka (2.5.2)
       base64 (~> 0.2)
-      karafka-core (>= 2.5.2, < 2.6.0)
-      karafka-rdkafka (>= 0.19.5)
-      waterdrop (>= 2.8.3, < 3.0.0)
+      karafka-core (>= 2.5.6, < 2.6.0)
+      karafka-rdkafka (>= 0.22.0)
+      waterdrop (>= 2.8.9, < 3.0.0)
       zeitwerk (~> 2.3)
-    karafka-core (2.5.2)
-      karafka-rdkafka (>= 0.19.2, < 0.21.0)
+    karafka-core (2.5.7)
+      karafka-rdkafka (>= 0.20.0)
       logger (>= 1.6.0)
-    karafka-rdkafka (0.20.1)
-      ffi (~> 1.15)
+    karafka-rdkafka (0.23.0)
+      ffi (~> 1.17.1)
+      json (> 2.0)
       logger
       mini_portile2 (~> 2.6)
       rake (> 12)
-    karafka-rdkafka (0.20.1-arm64-darwin)
-      ffi (~> 1.15)
+    karafka-rdkafka (0.23.0-aarch64-linux-gnu)
+      ffi (~> 1.17.1)
+      json (> 2.0)
       logger
       mini_portile2 (~> 2.6)
       rake (> 12)
-    karafka-rdkafka (0.20.1-x86_64-linux-gnu)
-      ffi (~> 1.15)
+    karafka-rdkafka (0.23.0-arm64-darwin)
+      ffi (~> 1.17.1)
+      json (> 2.0)
       logger
       mini_portile2 (~> 2.6)
       rake (> 12)
-    karafka-rdkafka (0.20.1-x86_64-linux-musl)
-      ffi (~> 1.15)
+    karafka-rdkafka (0.23.0-x86_64-linux-gnu)
+      ffi (~> 1.17.1)
+      json (> 2.0)
+      logger
+      mini_portile2 (~> 2.6)
+      rake (> 12)
+    karafka-rdkafka (0.23.0-x86_64-linux-musl)
+      ffi (~> 1.17.1)
+      json (> 2.0)
       logger
       mini_portile2 (~> 2.6)
       rake (> 12)
     logger (1.7.0)
     mini_portile2 (2.8.9)
-    minitest (5.25.5)
+    minitest (5.26.1)
     nokogiri (1.18.10)
       mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
@@ -107,44 +118,49 @@ GEM
     ostruct (0.6.3)
     raabro (1.4.0)
     racc (1.8.1)
-    rack (3.1.16)
+    rack (3.2.4)
     rack-test (2.2.0)
       rack (>= 1.3)
     rackup (0.2.3)
       rack (>= 3.0.0.beta1)
       webrick
-    rake (13.3.0)
-    roda (3.92.0)
+    rake (13.3.1)
+    roda (3.98.0)
       rack
-    rspec (3.13.1)
+    rspec (3.13.2)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
       rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.4)
+    rspec-core (3.13.6)
       rspec-support (~> 3.13.0)
     rspec-expectations (3.13.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.5)
+    rspec-mocks (3.13.7)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-support (3.13.4)
+    rspec-support (3.13.6)
     securerandom (0.4.1)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
       simplecov_json_formatter (~> 0.1)
-    simplecov-html (0.13.1)
+    simplecov-html (0.13.2)
     simplecov_json_formatter (0.1.4)
-    tilt (2.6.0)
+    tilt (2.6.1)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    waterdrop (2.8.4)
+    uri (1.1.1)
+    waterdrop (2.8.13)
       karafka-core (>= 2.4.9, < 3.0.0)
-      karafka-rdkafka (>= 0.19.2)
+      karafka-rdkafka (>= 0.20.0)
       zeitwerk (~> 2.3)
     webrick (1.9.1)
-    zeitwerk (2.6.18)
+    yard (0.9.37)
+    yard-lint (1.2.3)
+      yard (~> 0.9)
+      zeitwerk (~> 2.6)
+    zeitwerk (2.7.3)
 
 PLATFORMS
   aarch64-linux-gnu
@@ -170,6 +186,7 @@ DEPENDENCIES
   rackup (~> 0.2)
   rspec
   simplecov
+  yard-lint
 
 BUNDLED WITH
-   2.6.9
+   2.7.1

data/bin/integrations

@@ -0,0 +1,44 @@
+#!/usr/bin/env bash
+
+# Integration test runner for container metrics
+# Runs tests inside Docker containers with various memory/CPU configurations
+
+set -e
+
+echo "================================================"
+echo "Container Metrics Integration Tests"
+echo "================================================"
+echo ""
+
+# Build the test image first
+echo "Building integration test Docker image..."
+docker compose -f spec/integrations/docker-compose.yml build
+
+echo ""
+echo "Running integration tests in containers..."
+echo ""
+
+# Run all test scenarios
+# Note: Each container runs independently and exits after tests complete
+docker compose -f spec/integrations/docker-compose.yml up \
+  --abort-on-container-exit \
+  --exit-code-from test-container-limited \
+  test-container-limited
+
+docker compose -f spec/integrations/docker-compose.yml up \
+  --abort-on-container-exit \
+  --exit-code-from test-container-unlimited \
+  test-container-unlimited
+
+docker compose -f spec/integrations/docker-compose.yml up \
+  --abort-on-container-exit \
+  --exit-code-from test-container-strict \
+  test-container-strict
+
+echo ""
+echo "================================================"
+echo "All integration tests completed successfully!"
+echo "================================================"
+
+# Clean up
+docker compose -f spec/integrations/docker-compose.yml down

data/bin/rspecs

@@ -2,5 +2,9 @@
 
 set -e
 
-SPECS_TYPE=regular bundle exec rspec --exclude-pattern "**/pro/**/*_spec.rb"
-SPECS_TYPE=pro bundle exec rspec spec/lib/karafka/web/pro
+SPECS_TYPE=regular bundle exec rspec \
+  spec/lib \
+  --exclude-pattern "**/pro/**/*_spec.rb"
+
+SPECS_TYPE=pro bundle exec rspec \
+  spec/lib/karafka/web/pro

data/docker-compose.yml

@@ -1,7 +1,7 @@
 services:
   kafka:
     container_name: kafka
-    image: confluentinc/cp-kafka:8.0.1
+    image: confluentinc/cp-kafka:8.1.0
 
     ports:
       - 9092:9092

data/karafka-web.gemspec

@@ -17,9 +17,9 @@ Gem::Specification.new do |spec|
   spec.licenses    = %w[LGPL-3.0-only Commercial]
 
   spec.add_dependency 'erubi', '~> 1.4'
-  spec.add_dependency 'karafka', '>= 2.5.0', '< 2.6.0'
+  spec.add_dependency 'karafka', '>= 2.5.2', '< 2.6.0'
   spec.add_dependency 'karafka-core', '>= 2.5.0', '< 2.6.0'
-  spec.add_dependency 'roda', '~> 3.68', '>= 3.69'
+  spec.add_dependency 'roda', '~> 3.69', '>= 3.69'
   spec.add_dependency 'tilt', '~> 2.0'
 
   spec.add_development_dependency 'rackup', '~> 0.2'

data/lib/karafka/web/app.rb

@@ -6,9 +6,8 @@ module Karafka
     class App
       class << self
         # @param env [Hash] Rack env
-        # @param block [Proc] Rack block
-        def call(env, &block)
-          engine.call(env, &block)
+        def call(env, &)
+          engine.call(env, &)
         end
 
         # @return [Class] regular or pro Web engine

data/lib/karafka/web/cli/help.rb

@@ -10,7 +10,7 @@ module Karafka
         # Print available commands
         def call
           # Find the longest command for alignment purposes
-          max_command_length = self.class.commands.map(&:name).map(&:size).max
+          max_command_length = self.class.commands.map { |command| command.name.size }.max
 
           puts 'Karafka Web UI commands:'
 

data/lib/karafka/web/config.rb

@@ -171,6 +171,14 @@ module Karafka
             Tracking::Producers::Listeners::Errors.new
           ]
         end
+
+        setting :ui do
+          # Listeners needed for the Web UI to track errors from Roda/Puma
+          # These dispatch directly to Kafka since there's no background reporter in web processes
+          setting :listeners, default: [
+            Tracking::Ui::Errors.new
+          ]
+        end
       end
 
       # States processing related settings
@@ -202,6 +210,12 @@ module Karafka
           # Cookie key name
           setting :key, default: '_karafka_session'
 
+          # Rack middleware session env key. We use independent key from "rack.session" here to
+          # prevent our data from leaking to the main app (when mounted) and the other way around.
+          # This also prevents us from overloading the session object with extra data that could
+          # cause it to go beyond a cookie limit.
+          setting :env_key, default: 'karafka.session'
+
           # Secret for the session cookie
           setting :secret, default: SecureRandom.hex(32)
         end

data/lib/karafka/web/contracts/base.rb

@@ -13,10 +13,8 @@ module Karafka
             return super if block_given?
 
             super do |config|
-              config.error_messages = YAML.safe_load(
-                File.read(
-                  File.join(Karafka::Web.gem_root, 'config', 'locales', 'errors.yml')
-                )
+              config.error_messages = YAML.safe_load_file(
+                File.join(Karafka::Web.gem_root, 'config', 'locales', 'errors.yml')
               ).fetch('en').fetch('validations').fetch('web')
             end
           end

data/lib/karafka/web/contracts/config.rb

@@ -19,7 +19,7 @@ module Karafka
             required(:name) { |val| val.is_a?(String) && TOPIC_REGEXP.match?(val) }
 
             required(:config) do |val|
-              val.is_a?(Hash) && !val.empty? && val.keys.all? { |key| key.is_a?(Symbol) }
+              val.is_a?(Hash) && !val.empty? && val.keys.all?(Symbol)
             end
           end
 
@@ -28,7 +28,7 @@ module Karafka
               required(:name) { |val| val.is_a?(String) && TOPIC_REGEXP.match?(val) }
 
               required(:config) do |val|
-                val.is_a?(Hash) && !val.empty? && val.keys.all? { |key| key.is_a?(Symbol) }
+                val.is_a?(Hash) && !val.empty? && val.keys.all?(Symbol)
               end
             end
 
@@ -36,7 +36,7 @@ module Karafka
               required(:name) { |val| val.is_a?(String) && TOPIC_REGEXP.match?(val) }
 
               required(:config) do |val|
-                val.is_a?(Hash) && !val.empty? && val.keys.all? { |key| key.is_a?(Symbol) }
+                val.is_a?(Hash) && !val.empty? && val.keys.all?(Symbol)
               end
             end
 
@@ -44,7 +44,7 @@ module Karafka
               required(:name) { |val| val.is_a?(String) && TOPIC_REGEXP.match?(val) }
 
               required(:config) do |val|
-                val.is_a?(Hash) && !val.empty? && val.keys.all? { |key| key.is_a?(Symbol) }
+                val.is_a?(Hash) && !val.empty? && val.keys.all?(Symbol)
               end
             end
 
@@ -52,7 +52,7 @@ module Karafka
               required(:name) { |val| val.is_a?(String) && TOPIC_REGEXP.match?(val) }
 
               required(:config) do |val|
-                val.is_a?(Hash) && !val.empty? && val.keys.all? { |key| key.is_a?(Symbol) }
+                val.is_a?(Hash) && !val.empty? && val.keys.all?(Symbol)
               end
             end
           end
@@ -91,6 +91,7 @@ module Karafka
         nested(:ui) do
           nested(:sessions) do
             required(:key) { |val| val.is_a?(String) && !val.empty? }
+            required(:env_key) { |val| val.is_a?(String) && !val.empty? }
             required(:secret) { |val| val.is_a?(String) && val.length >= 64 }
           end
 

data/lib/karafka/web/inflector.rb

@@ -13,7 +13,7 @@ module Karafka
 
       private_constant :MIGRATION_ABSPATH_REGEXP, :MIGRATION_BASENAME_REGEXP
 
-      # @param [String] basename of the file to be loaded
+      # @param basename [String] basename of the file to be loaded
       # @param abspath [String] absolute path of the file to be loaded
       # @return [String] Constant name to be used for given file
       def camelize(basename, abspath)

data/lib/karafka/web/management/actions/enable.rb

@@ -61,7 +61,7 @@ module Karafka
                   # Since we materialize state in intervals, we can poll for half of this time
                   # without impacting the reporting responsiveness
                   max_wait_time ::Karafka::Web.config.processing.interval / 2
-                  max_messages 1_000
+                  max_messages 200
                   consumer ::Karafka::Web::Processing::Consumer
                   # This needs to be true in order not to reload the consumer in dev. This consumer
                   # should not be affected by the end user development process
@@ -73,6 +73,13 @@ module Karafka
                   # consumer group name would be renamed and we would start consuming all
                   # historical
                   initial_offset 'latest'
+                  # Increase backoff time on errors. Incompatible schema errors are not recoverable
+                  # until rolling upgrade completes, so we use a longer max timeout to prevent
+                  # spamming errors in logs.
+                  # We set this ourselves so user settings do not impact frequency of retrying
+                  pause_timeout 5_000
+                  pause_max_timeout 60_000
+                  pause_with_exponential_backoff true
                   # We use the defaults + our config alterations that may not align with what
                   # user wants for his topics.
                   kafka kafka_config
@@ -124,6 +131,12 @@ module Karafka
 
               ::Karafka::Web.producer.monitor.subscribe(listener)
             end
+
+            # Installs all the UI related listeners for tracking errors from web processes
+            # These listen on Karafka monitor to catch instrumented UI errors
+            ::Karafka::Web.config.tracking.ui.listeners.each do |listener|
+              ::Karafka.monitor.subscribe(listener)
+            end
           end
 
           # In most cases we want to close the producer if possible.

data/lib/karafka/web/management/migrations/consumers_reports/1761645571_rename_process_name_to_id.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Karafka
+  module Web
+    module Management
+      module Migrations
+        # Consumer reports migrations
+        module ConsumersReports
+          # Migrates consumer reports from schema < 1.3.0 that used process[:name] to process[:id]
+          #
+          # In schema versions 1.2.x and earlier (karafka-web <= v0.8.2), the process identifier
+          # was stored in the :name field. Starting with schema 1.3.0 (karafka-web v0.9.0+),
+          # this was renamed to :id for consistency.
+          #
+          # This migration ensures old reports can be processed by current aggregators that
+          # expect the :id field.
+          class RenameProcessNameToId < Base
+            # Apply to all schema versions before 1.3.0
+            self.versions_until = '1.3.0'
+            self.type = :consumers_reports
+
+            # @param report [Hash] consumer report to migrate
+            def migrate(report)
+              # If :id already exists, nothing to do (already migrated or newer schema)
+              return if report[:process][:id]
+
+              # Rename :name to :id
+              # Both :name (in schema < 1.3.0) and :id (in schema >= 1.3.0) were always
+              # required fields, so we don't need nil checks for valid reports
+              report[:process][:id] = report[:process][:name]
+              report[:process].delete(:name)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/karafka/web/management/migrator.rb

@@ -65,8 +65,6 @@ module Karafka
               'consumers metrics newer than supported'
             )
           end
-
-          true
         end
 
         # Applies migrations if needed and mutates the in-memory data
@@ -76,6 +74,9 @@ module Karafka
           any_migrations = false
 
           Migrations::Base.sorted_descendants.each do |migration_class|
+            # Skip report migrations - they are applied per-message by ReportsMigrator
+            next if migration_class.type == :consumers_reports
+
             data = send(migration_class.type)
 
             next unless migration_class.applicable?(data[:schema_version])

data/lib/karafka/web/pro/commanding/commands/base.rb

@@ -50,7 +50,7 @@ module Karafka
             #   being executed in the embedded or swarm processes since there the signaling is
             #   handled differently (either via the main process or supervisor).
             def standalone?
-              Karafka::Server.execution_mode == :standalone
+              Karafka::Server.execution_mode.standalone?
             end
 
             # @return [String] id of the current consumer process

data/lib/karafka/web/pro/commanding/contracts/config.rb

@@ -12,10 +12,8 @@ module Karafka
           # Makes sure, all the expected commanding config is defined as it should be
           class Config < ::Karafka::Contracts::Base
             configure do |config|
-              config.error_messages = YAML.safe_load(
-                File.read(
-                  File.join(Karafka::Web.gem_root, 'config', 'locales', 'pro_errors.yml')
-                )
+              config.error_messages = YAML.safe_load_file(
+                File.join(Karafka::Web.gem_root, 'config', 'locales', 'pro_errors.yml')
               ).fetch('en').fetch('validations').fetch('config')
             end
 

data/lib/karafka/web/pro/commanding/handlers/partitions/tracker.rb

@@ -41,17 +41,16 @@ module Karafka
               # @param subscription_group_id [String] id of the subscription group for which we
               #   want to get all the requests. Subscription groups ids (not names) are unique
               #   within the application, so it is unique "enough".
-              # @param block [Proc]
               #
               # @yieldparam [Request] given command request for the requested subscription group
-              def each_for(subscription_group_id, &block)
+              def each_for(subscription_group_id, &)
                 requests = nil
 
                 @mutex.synchronize do
                   requests = @requests.delete(subscription_group_id)
                 end
 
-                (requests || EMPTY_ARRAY).each(&block)
+                (requests || EMPTY_ARRAY).each(&)
               end
             end
           end

data/lib/karafka/web/pro/ui/controllers/cluster_controller.rb

@@ -30,7 +30,7 @@ module Karafka
 
             # Displays selected broker configuration
             #
-            # @param broker_id [String] id of the broker
+            # @param broker_id [String]
             def show(broker_id)
               @broker = Models::Broker.find(broker_id)
 

data/lib/karafka/web/pro/ui/controllers/scheduled_messages/schedules_controller.rb

@@ -21,8 +21,7 @@ module Karafka
                 # cluster
                 candidates = Karafka::App
                              .routes
-                             .map(&:topics)
-                             .map(&:to_a)
+                             .map { |route| route.topics.to_a }
                              .flatten
                              .select(&:scheduled_messages?)
                              .reject { |topic| topic.name.end_with?(states_postfix) }

data/lib/karafka/web/pro/ui/controllers/topics/distributions_controller.rb

@@ -65,9 +65,7 @@ module Karafka
                     topic_name,
                     partition_count
                   )
-                rescue Rdkafka::RdkafkaError => e
-                  @form_error = e
-                rescue Rdkafka::Config::ConfigError => e
+                rescue Rdkafka::RdkafkaError, Rdkafka::Config::ConfigError => e
                   @form_error = e
                 end
 

data/lib/karafka/web/pro/ui/lib/branding/contracts/config.rb

@@ -14,10 +14,8 @@ module Karafka
               # Makes sure, all the expected UI branding config is defined as it should be
               class Config < ::Karafka::Contracts::Base
                 configure do |config|
-                  config.error_messages = YAML.safe_load(
-                    File.read(
-                      File.join(Karafka::Web.gem_root, 'config', 'locales', 'pro_errors.yml')
-                    )
+                  config.error_messages = YAML.safe_load_file(
+                    File.join(Karafka::Web.gem_root, 'config', 'locales', 'pro_errors.yml')
                   ).fetch('en').fetch('validations').fetch('config')
                 end