judges 0.60.3 → 0.60.4

data/lib/judges/commands/update.rb

@@ -7,8 +7,8 @@ require 'backtrace'
 require 'elapsed'
 require 'factbase'
 require 'factbase/churn'
-require 'factbase/logged'
 require 'factbase/fact_as_yaml'
+require 'factbase/logged'
 require 'logger'
 require 'tago'
 require 'timeout'
@@ -21,7 +21,7 @@ require_relative '../../judges/to_rel'
 
 # The +update+ command.
 #
-# This class is instantiated by the +bin/judge+ command line interface. You
+# This class is instantiated by the +bin/judges+ command line interface. You
 # are not supposed to instantiate it yourself.
 #
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
@@ -40,31 +40,13 @@ class Judges::Update
   # @param [Array] args List of command line arguments
   # @raise [RuntimeError] If not exactly two arguments provided or directory is missing
   def run(opts, args)
-    raise 'Exactly two arguments required' unless args.size == 2
+    raise(ArgumentError, 'Exactly two arguments required') unless args.size == 2
     dir = args[0]
-    raise "The directory is absent: #{dir.to_rel}" unless File.exist?(dir)
+    raise(StandardError, "The directory is absent: #{dir.to_rel}") unless File.exist?(dir)
     impex = Judges::Impex.new(@loog, args[1])
     fb = impex.import(strict: false)
     fb = Factbase::Logged.new(fb, @loog) if opts['log']
-    options = Judges::Options.new(timeout: opts['timeout']&.to_i, lifetime: opts['lifetime']&.to_i)
-    if options.lifetime && options.timeout && options.lifetime < options.timeout * 1.1
-      raise "The --timeout=#{options.timeout} must be at least 10 percent smaller than --lifetime=#{options.lifetime}"
-    end
-    options += Judges::Options.new(opts['option'])
-    if opts['options-file']
-      options += Judges::Options.new(
-        File.readlines(opts['options-file'])
-          .compact
-          .reject(&:empty?)
-          .map { |ln| ln.strip.split('=', 1).map(&:strip).join('=') }
-      )
-      @loog.debug("Options loaded from #{opts['options-file']}")
-    end
-    if options.empty?
-      @loog.debug('No options provided by the --option flag')
-    else
-      @loog.debug("The following options provided:\n\t#{options.to_s.gsub("\n", "\n\t")}")
-    end
+    options = build_options(opts)
     judges = Judges::Judges.new(
       dir, opts['lib'], @loog,
       epoch: @epoch, shuffle: opts['shuffle'], boost: opts['boost'],
@@ -77,7 +59,7 @@ class Judges::Update
       end
     rescue Timeout::Error, Timeout::ExitException => e
       @loog.error("Terminated due to --lifetime=#{opts['lifetime']}")
-      raise e unless opts['quiet']
+      raise(e) unless opts['quiet']
       @loog.info("Had to stop due to the --lifetime=#{opts['lifetime']}")
     ensure
       impex.export(fb)
@@ -90,20 +72,52 @@ class Judges::Update
 
   private
 
-  def loop_them(judges, fb, churn, opts, options)
-    c = 0
-    ch = Factbase::Churn.new
-    errors = []
-    statistics = opts['statistics'] ? Judges::Statistics.new : nil
+  def build_options(opts)
+    options = Judges::Options.new(timeout: opts['timeout']&.to_i, lifetime: opts['lifetime']&.to_i)
+    if options.lifetime && options.timeout && options.lifetime < options.timeout * 1.1
+      raise(
+        StandardError,
+        "The --timeout=#{options.timeout} must be at least 10 percent smaller than --lifetime=#{options.lifetime}"
+      )
+    end
+    options += Judges::Options.new(opts['option'])
+    if opts['options-file']
+      options += Judges::Options.new(
+        File.readlines(opts['options-file'])
+          .compact
+          .reject(&:empty?)
+          .map { |ln| ln.strip.split('=', 1).map(&:strip).join('=') }
+      )
+      @loog.debug("Options loaded from #{opts['options-file']}")
+    end
+    if options.empty?
+      @loog.debug('No options provided by the --option flag')
+    else
+      @loog.debug("The following options provided:\n\t#{options.to_s.gsub("\n", "\n\t")}")
+    end
+    options
+  end
+
+  def log_summary(opts, fb)
     sum = fb.query('(eq what "judges-summary")').each.to_a
     if sum.empty?
       @loog.info('Summary fact not found') unless opts['summary'] == 'off'
     else
       @loog.info("Summary fact found:\n\t#{Factbase::FactAsYaml.new(sum.first).to_s.gsub("\n", "\n\t")}")
     end
-    if !sum.empty? && opts['summary'] == 'add' && fb.query('(eq what "judges-summary")').delete!
-      @loog.info('Summary fact deleted')
-    end
+    return if sum.empty?
+    return unless opts['summary'] == 'add'
+    fb.query('(eq what "judges-summary")').delete!
+    @loog.info('Summary fact deleted')
+  end
+
+  # rubocop:disable Metrics/MethodLength
+  def loop_them(judges, fb, churn, opts, options)
+    c = 0
+    ch = Factbase::Churn.new
+    errors = []
+    statistics = opts['statistics'] ? Judges::Statistics.new : nil
+    log_summary(opts, fb)
     elapsed(@loog, level: Logger::INFO) do
       loop do
         c += 1
@@ -131,11 +145,12 @@ class Judges::Update
         end
         @loog.info("The cycle #{c} did #{delta}")
       end
-      throw :"👍 Update completed in #{c} cycle(s), did #{ch}"
+      throw(:"👍 Update completed in #{c} cycle(s), did #{ch}")
     end
     statistics&.report(@loog)
     summarize(fb, ch, errors, c) if %w[add append].include?(opts['summary'])
   end
+  # rubocop:enable Metrics/MethodLength
 
   # Update the summary.
   # @param [Factbase] fb The factbase
@@ -186,58 +201,71 @@ class Judges::Update
     used = 0
     elapsed(@loog, level: Logger::INFO) do
       done =
-        judges.each_with_index do |judge, i|
-          if opts['fail-fast'] && !errors.empty?
-            @loog.info("Not running #{judge.name.inspect} due to #{errors.count} errors above, in --fail-fast mode")
-            statistics&.record(judge.name, 0, 'SKIPPED (fail-fast)') if include?(opts, judge.name)
-            next
-          end
-          if opts['lifetime'] && opts['timeout']
-            remained = @start + opts['lifetime'] - Time.now
-            if remained < opts['timeout'].to_f / 16
-              @loog.info("Not running #{judge.name.inspect}, not enough time left (just #{remained.seconds})")
-              statistics&.record(judge.name, 0, 'SKIPPED (timeout)') if include?(opts, judge.name)
-              next
-            end
-          end
-          next unless include?(opts, judge.name)
-          @loog.info("\n👉 Running #{judge.name} (##{i}) at #{judge.dir.to_rel} (#{@start.ago} already)...")
-          used += 1
-          start_time = Time.now
-          result = 'OK'
-          impact = nil
-          elapsed(@loog, level: Logger::INFO) do
-            impact = one_judge(opts, fb, judge, global, options, errors)
-            delta += impact
-            churn.append(impact.inserted, impact.deleted, impact.added)
-            throw :"👍 The '#{judge.name}' judge made zero changes to #{fb.size} facts" if impact.zero?
-            throw :"👍 The '#{judge.name}' judge #{impact} out of #{fb.size} facts"
-          end
-        rescue StandardError, SyntaxError => e
-          if e.is_a?(RuntimeError) && e.message == 'skip'
-            result = 'SKIPPED'
-          else
-            @loog.warn(Backtrace.new(e))
-            errors << e.message
-            result = 'ERROR'
+        judges.each_with_index do |judge, idx|
+          result = run_judge_in_cycle(judge, idx, opts, fb, churn, options, errors, global, statistics)
+          if result
+            used += 1
+            delta += result if result.is_a?(Factbase::Churn)
           end
-        ensure
-          statistics&.record(judge.name, Time.now - start_time, result, impact) if start_time
         end
-      throw :"👍 #{done} judge(s) processed" if errors.empty?
-      throw :"❌ #{done} judge(s) processed with #{errors.size} errors"
+      throw(:"👍 #{done} judge(s) processed") if errors.empty?
+      throw(:"❌ #{done} judge(s) processed with #{errors.size} errors")
     end
     if used.zero?
-      raise 'No judges were used, while at least one expected to run' if opts['expect-judges']
+      raise(StandardError, 'No judges were used, while at least one expected to run') if opts['expect-judges']
       @loog.info('No judges were used (looks like an error); not failing because of --no-expect-judges')
     end
     unless errors.empty?
-      raise "Failed to update correctly (#{errors.size} errors)" unless opts['quiet']
+      raise(StandardError, "Failed to update correctly (#{errors.size} errors)") unless opts['quiet']
       @loog.info('Not failing because of the --quiet flag provided')
     end
     delta
   end
 
+  def run_judge_in_cycle(judge, idx, opts, fb, churn, options, errors, global, statistics)
+    return if skip_judge?(judge, idx, opts, errors, statistics)
+    return unless include?(opts, judge.name)
+    @loog.info("\n👉 Running #{judge.name} (##{idx}) at #{judge.dir.to_rel} (#{@start.ago} already)...")
+    start = Time.now
+    result = 'OK'
+    impact = nil
+    elapsed(@loog, level: Logger::INFO) do
+      impact = one_judge(opts, fb, judge, global, options, errors)
+      churn.append(impact.inserted, impact.deleted, impact.added)
+      throw(:"👍 The '#{judge.name}' judge made zero changes to #{fb.size} facts") if impact.zero?
+      throw(:"👍 The '#{judge.name}' judge #{impact} out of #{fb.size} facts")
+    end
+    impact
+  rescue StandardError, SyntaxError => e
+    if e.is_a?(RuntimeError) && e.message == 'skip'
+      result = 'SKIPPED'
+    else
+      @loog.warn(Backtrace.new(e))
+      errors << e.message
+      result = 'ERROR'
+    end
+    impact || true
+  ensure
+    statistics&.record(judge.name, Time.now - start, result, impact) if start
+  end
+
+  def skip_judge?(judge, _idx, opts, errors, statistics)
+    if opts['fail-fast'] && !errors.empty?
+      @loog.info("Not running #{judge.name.inspect} due to #{errors.count} errors above, in --fail-fast mode")
+      statistics&.record(judge.name, 0, 'SKIPPED (fail-fast)') if include?(opts, judge.name)
+      return true
+    end
+    if opts['lifetime'] && opts['timeout']
+      remained = @start + opts['lifetime'] - Time.now
+      if remained < opts['timeout'].to_f / 16
+        @loog.info("Not running #{judge.name.inspect}, not enough time left (just #{remained.seconds})")
+        statistics&.record(judge.name, 0, 'SKIPPED (timeout)') if include?(opts, judge.name)
+        return true
+      end
+    end
+    false
+  end
+
   # Run a single judge.
   #
   # @param [Hash] opts The command line options
@@ -253,7 +281,7 @@ class Judges::Update
     fb = Factbase::Tallied.new(fb)
     begin
       if opts['lifetime'] && Time.now - @start > opts['lifetime']
-        throw :"👎 The '#{judge.name}' judge skipped, no time left"
+        throw(:"👎 The '#{judge.name}' judge skipped, no time left")
       end
       Timeout.timeout(opts['timeout']) do
         judge.run(fb, global, local, options)

data/lib/judges/commands/upload.rb

@@ -3,15 +3,15 @@
 # SPDX-FileCopyrightText: Copyright (c) 2024-2026 Yegor Bugayenko
 # SPDX-License-Identifier: MIT
 
-require 'typhoeus'
-require 'iri'
 require 'baza-rb'
 require 'elapsed'
+require 'iri'
+require 'typhoeus'
 require_relative '../../judges'
 
 # The +upload+ command, to send a durable to Zerocracy.
 #
-# This class is instantiated by the +bin/judge+ command line interface. You
+# This class is instantiated by the +bin/judges+ command line interface. You
 # are not supposed to instantiate it yourself.
 #
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
@@ -28,11 +28,12 @@ class Judges::Upload
   # @param [Hash] opts Command line options (start with '--')
   # @param [Array] args List of command line arguments
   # @raise [RuntimeError] If not exactly two arguments provided
+  # rubocop:disable Metrics/MethodLength
   def run(opts, args)
-    raise 'Exactly two arguments required' unless args.size == 2
+    raise(ArgumentError, 'Exactly two arguments required') unless args.size == 2
     jname = args[0]
     path = args[1]
-    raise "File not found: #{path}" unless File.exist?(path)
+    raise(StandardError, "File not found: #{path}") unless File.exist?(path)
     name = File.basename(path)
     baza = BazaRb.new(
       opts['host'], opts['port'].to_i, opts['token'],
@@ -44,8 +45,6 @@ class Judges::Upload
     elapsed(@loog, level: Logger::INFO) do
       id = baza.durable_find(jname, name)
       if id.nil? || id.to_s.strip.empty?
-        # Block form of Dir.mkdir causes error Errno::EACCESS on windows
-        # so we use non-block form
         tmp = Dir.mktmpdir
         begin
           f = File.join(tmp, name)
@@ -61,10 +60,11 @@ class Judges::Upload
       baza.durable_lock(id, opts['owner'] || 'default')
       begin
         baza.durable_save(id, path)
-        throw :"👍 Uploaded #{path} to existing durable '#{name}' in '#{jname}' (ID: #{id}, #{size} bytes)"
+        throw(:"👍 Uploaded #{path} to existing durable '#{name}' in '#{jname}' (ID: #{id}, #{size} bytes)")
       ensure
         baza.durable_unlock(id, opts['owner'] || 'default')
       end
     end
   end
+  # rubocop:enable Metrics/MethodLength
 end

data/lib/judges/impex.rb

@@ -3,9 +3,9 @@
 # SPDX-FileCopyrightText: Copyright (c) 2024-2026 Yegor Bugayenko
 # SPDX-License-Identifier: MIT
 
+require 'elapsed'
 require 'factbase'
 require 'fileutils'
-require 'elapsed'
 require_relative '../judges'
 require_relative '../judges/to_rel'
 
@@ -51,10 +51,10 @@ class Judges::Impex
     if File.exist?(@file)
       elapsed(@loog, level: Logger::INFO) do
         fb.import(File.binread(@file))
-        throw :"The factbase imported from #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)"
+        throw(:"The factbase imported from #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)")
       end
     else
-      raise "The factbase is absent at #{@file.to_rel}" if strict
+      raise(StandardError, "The factbase is absent at #{@file.to_rel}") if strict
       @loog.info("Nothing to import from #{@file.to_rel} (file not found)")
     end
     fb
@@ -74,10 +74,10 @@ class Judges::Impex
   #   # ... populate fb with some data ...
   #   impex.import_to(fb) # Adds data from file to existing facts
   def import_to(fb)
-    raise "The factbase is absent at #{@file.to_rel}" unless File.exist?(@file)
+    raise(StandardError, "The factbase is absent at #{@file.to_rel}") unless File.exist?(@file)
     elapsed(@loog, level: Logger::INFO) do
       fb.import(File.binread(@file))
-      throw :"The factbase loaded from #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)"
+      throw(:"The factbase loaded from #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)")
     end
   end
 
@@ -97,7 +97,7 @@ class Judges::Impex
     elapsed(@loog, level: Logger::INFO) do
       FileUtils.mkdir_p(File.dirname(@file))
       File.binwrite(@file, fb.export)
-      throw :"Factbase exported to #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)"
+      throw(:"Factbase exported to #{@file.to_rel} (#{File.size(@file)} bytes, #{fb.size} facts)")
     end
   end
 end

data/lib/judges/judge.rb

@@ -4,13 +4,13 @@
 # SPDX-License-Identifier: MIT
 
 require 'elapsed'
-require 'tago'
-require 'timeout'
 require 'factbase/tallied'
 require 'octokit'
+require 'tago'
+require 'timeout'
 require_relative '../judges'
-require_relative '../judges/to_rel'
 require_relative '../judges/pretty_exception'
+require_relative '../judges/to_rel'
 
 # A single judge.
 # Author:: Yegor Bugayenko (yegor256@gmail.com)
@@ -57,42 +57,44 @@ class Judges::Judge
   # @param [Judges::Options] options Command-line options object
   # @return [nil] Nothing
   # @raise [RuntimeError] If the lib directory doesn't exist, the script can't be loaded, or execution fails
+  # rubocop:disable Metrics/MethodLength
   def run(fb, global, local, options)
     $fb = fb
     $judge = File.basename(@dir)
     $options = options
     $loog = @loog
     $global = global
-    $global.delete(:fb) # to make sure Tallied is always actual
+    $global.delete(:fb)
     $local = local
     $epoch = @epoch
     $kickoff = Time.now
     options.to_h.each { |k, v| ENV.store(k.to_s, v.to_s) }
     unless @lib.nil?
-      raise "Lib dir #{@lib.to_rel} is absent" unless File.exist?(@lib)
-      raise "Lib #{@lib.to_rel} is not a directory" unless File.directory?(@lib)
+      raise(StandardError, "Lib dir #{@lib.to_rel} is absent") unless File.exist?(@lib)
+      raise(StandardError, "Lib #{@lib.to_rel} is not a directory") unless File.directory?(@lib)
       Dir.glob(File.join(@lib, '*.rb')).each do |f|
         require_relative(File.absolute_path(f))
       end
     end
     s = File.join(@dir, script)
-    raise "Can't load '#{s}'" unless File.exist?(s)
+    raise(StandardError, "Can't load '#{s}'") unless File.exist?(s)
     elapsed(@loog, good: "#{$judge} completed", level: Logger::INFO) do
       load(s, true)
       nil
       # rubocop:disable Lint/RescueException
     rescue Exception => e
       # rubocop:enable Lint/RescueException
-      raise e if e.is_a?(RuntimeError) && e.message == 'skip'
+      raise(e) if e.is_a?(RuntimeError) && e.message == 'skip'
       e = Judges::PrettyException.new(e) if e.is_a?(Octokit::ServerError)
       @loog.error(Backtrace.new(e))
-      raise e if e.is_a?(StandardError)
-      raise e if e.is_a?(Timeout::ExitException)
-      raise "#{e.message} (#{e.class.name})"
+      raise(e) if e.is_a?(StandardError)
+      raise(e) if e.is_a?(Timeout::ExitException)
+      raise(StandardError, "#{e.message} (#{e.class.name})")
     ensure
       $fb = $judge = $options = $loog = $epoch = $kickoff = nil
     end
   end
+  # rubocop:enable Metrics/MethodLength
 
   # Returns the name of the judge.
   #
@@ -113,7 +115,7 @@ class Judges::Judge
   def script
     b = "#{File.basename(@dir)}.rb"
     files = Dir.glob(File.join(@dir, '*.rb')).map { |f| File.basename(f) }
-    raise "No #{b} script in #{@dir.to_rel} among #{files}" unless files.include?(b)
+    raise(StandardError, "No #{b} script in #{@dir.to_rel} among #{files}") unless files.include?(b)
     b
   end
 

data/lib/judges/judges.rb

@@ -55,7 +55,7 @@ class Judges::Judges
   # @raise [RuntimeError] If no judge directory exists with the given name
   def get(name)
     d = File.absolute_path(File.join(@dir, name))
-    raise "Judge #{name} doesn't exist in #{@dir}" unless File.exist?(d)
+    raise(StandardError, "Judge #{name} doesn't exist in #{@dir}") unless File.exist?(d)
     Judges::Judge.new(d, @lib, @loog, epoch: @epoch)
   end
 
@@ -73,30 +73,11 @@ class Judges::Judges
   # @return [Enumerator] Returns an enumerator if no block is given
   def each(&)
     return to_enum(__method__) unless block_given?
-    list =
-      Dir.glob(File.join(@dir, '*')).each.to_a.map do |d|
-        next unless File.directory?(d)
-        b = File.basename(d)
-        next unless File.exist?(File.join(d, "#{b}.rb"))
-        Judges::Judge.new(File.absolute_path(d), @lib, @loog, epoch: @epoch)
-      end
-    list.compact!
-    list.sort_by!(&:name)
-    all = list.each_with_index.to_a
-    good = all.dup
-    mapping =
-      all
-           .map { |a| [a[0].name, a[1], a[1]] }
-           .reject { |a| !@shuffle.empty? && a[0].start_with?(@shuffle) }
-           .to_h { |a| [a[1], a[2]] }
-    positions = mapping.values.shuffle(random: Random.new(@seed))
-    mapping.keys.zip(positions).to_h.each do |before, after|
-      good[after] = all[before]
-    end
+    good = reorder_judges
     boosted = []
     demoted = []
     normal = []
-    good.map { |a| a[0] }.each do |j|
+    good.each do |j|
       if fits?(j.name, @boost)
         boosted.append(j)
       elsif fits?(j.name, @demote)
@@ -105,8 +86,7 @@ class Judges::Judges
         normal.append(j)
       end
     end
-    ret = boosted + normal + demoted
-    ret.each(&)
+    (boosted + normal + demoted).each(&)
   end
 
   # Iterates over all judges while tracking their index position.
@@ -120,7 +100,7 @@ class Judges::Judges
   def each_with_index
     idx = 0
     each do |p|
-      yield [p, idx]
+      yield([p, idx])
       idx += 1
     end
     idx
@@ -128,6 +108,31 @@ class Judges::Judges
 
   private
 
+  def reorder_judges
+    list = discover_judges
+    list.sort_by!(&:name)
+    all = list.each_with_index.to_a
+    good = all.dup
+    mapping =
+      all
+        .map { |a| [a[0].name, a[1], a[1]] }
+        .reject { |a| !@shuffle.empty? && a[0].start_with?(@shuffle) }
+        .to_h { |a| [a[1], a[2]] }
+    positions = mapping.values.shuffle(random: Random.new(@seed))
+    mapping.keys.zip(positions).to_h.each do |before, after|
+      good[after] = all[before]
+    end
+    good.map { |a| a[0] }
+  end
+
+  def discover_judges
+    Dir.glob(File.join(@dir, '*')).each.to_a.filter_map do |d|
+      next unless File.directory?(d)
+      next unless File.exist?(File.join(d, "#{File.basename(d)}.rb"))
+      Judges::Judge.new(File.absolute_path(d), @lib, @loog, epoch: @epoch)
+    end
+  end
+
   # Checks if a judge name matches any of the given patterns.
   # Patterns can contain '*' wildcards which are converted to '.*' regex patterns.
   #

data/lib/judges/options.rb

@@ -111,49 +111,40 @@ class Judges::Options
   #   options = Judges::Options.new("token=abc123,max_speed=100,debug")
   #   options.to_h # => { TOKEN: "abc123", MAX_SPEED: 100, DEBUG: "true" }
   def to_h
-    @to_h ||=
-      begin
-        pp = @pairs || []
-        pp = pp.split(',') if pp.is_a?(String)
-        if pp.is_a?(Array)
-          pp = pp
-            .compact
-            .map(&:strip)
-            .reject(&:empty?)
-            .map { |s| s.split('=', 2) }
-            .map { |a| a.size == 1 ? [a[0], nil] : a }
-            .reject { |a| a[0].empty? }
-            .to_h
-        end
-        pp
-          .reject { |k, _| k.nil? }
-          .compact
-          .reject { |k, _| k.is_a?(String) && k.empty? }
-          .to_h
-          .transform_values { |v| v.nil? ? 'true' : v }
-          .transform_values { |v| v.is_a?(String) ? v.strip : v }
-          .transform_values { |v| v.is_a?(String) && v.match?(/^[0-9]+$/) ? v.to_i : v }
-          .transform_keys { |k| k.to_s.strip.upcase.to_sym }
-      end
+    @to_h ||= normalize_to_h
+  end
+
+  private
+
+  def normalize_to_h
+    pp = parse_pairs
+    pp
+      .reject { |k, _| k.nil? }
+      .compact
+      .reject { |k, _| k.is_a?(String) && k.empty? }
+      .to_h
+      .transform_values { |v| v.nil? ? 'true' : v }
+      .transform_values { |v| v.is_a?(String) ? v.strip : v }
+      .transform_values { |v| v.is_a?(String) && v.match?(/^[0-9]+$/) ? v.to_i : v }
+      .transform_keys { |k| k.to_s.strip.upcase.to_sym }
+  end
+
+  def parse_pairs
+    pp = @pairs || []
+    pp = pp.split(',') if pp.is_a?(String)
+    if pp.is_a?(Array)
+      pp = pp
+        .compact
+        .map(&:strip)
+        .reject(&:empty?)
+        .map { |s| s.split('=', 2) }
+        .map { |a| a.size == 1 ? [a[0], nil] : a }
+        .reject { |a| a[0].empty? }
+        .to_h
+    end
+    pp
   end
 
-  # Get option by name.
-  #
-  # This method is implemented using the 'others' gem, which provides
-  # dynamic method handling. It allows accessing options as method calls.
-  # Method names are automatically converted to uppercase symbols to match
-  # the keys in the options hash.
-  #
-  # @!method method_missing(method_name, *args)
-  #   Dynamic method to access option values
-  #   @param [Symbol] method_name The name of the option to retrieve
-  #   @param [Array] args Additional arguments (unused)
-  #   @return [Object, nil] The value of the option, or nil if not found
-  # @example Access options as methods
-  #   options = Judges::Options.new(["token=abc123", "max_speed=100"])
-  #   options.token # => "abc123"
-  #   options.max_speed # => 100
-  #   options.missing_option # => nil
   others do |*args|
     to_h[args[0].upcase.to_sym]
   end

data/lib/judges/pretty_exception.rb

@@ -7,7 +7,7 @@ require 'delegate'
 require 'ellipsized'
 require_relative '../judges'
 
-# Decorates the exception for show ellipsed message.
+# Decorates the exception to show an ellipsized message.
 class Judges::PrettyException < SimpleDelegator
   undef_method :class
   undef_method :instance_of?

data/lib/judges/statistics.rb

@@ -32,12 +32,7 @@ class Judges::Statistics
   # @param [Churn] churn The churn for this run (can be nil)
   def record(name, time, result, churn = nil)
     unless @data[name]
-      @data[name] = {
-        total_time: 0.0,
-        cycles: 0,
-        results: [],
-        total_churn: nil
-      }
+      @data[name] = { total_time: 0.0, cycles: 0, results: [], total_churn: nil }
     end
     stats = @data[name]
     stats[:total_time] += time
@@ -62,8 +57,10 @@ class Judges::Statistics
         format(fmt, 'Judge', 'Seconds', 'Cycles', 'Changes', 'Results'),
         format(fmt, '---', '---', '---', '---', '---'),
         @data.sort_by { |_, stats| stats[:total_time] }.reverse.map do |name, stats|
-          format(fmt, name, format('%.3f', stats[:total_time]), stats[:cycles],
-                 stats[:total_churn] ? stats[:total_churn].to_s : 'N/A', summarize(stats[:results]))
+          format(
+            fmt, name, format('%.3f', stats[:total_time]), stats[:cycles],
+            stats[:total_churn] ? stats[:total_churn].to_s : 'N/A', summarize(stats[:results])
+          )
         end.join("\n  ")
       ].join("\n  ")
     )

data/lib/judges.rb

@@ -8,5 +8,5 @@
 # Copyright:: Copyright (c) 2024-2026 Yegor Bugayenko
 # License:: MIT
 module Judges
-  VERSION = '0.60.3' unless const_defined?(:VERSION)
+  VERSION = '0.60.4' unless const_defined?(:VERSION)
 end