omnizip 0.3.53 → 0.3.55

data/lib/omnizip/commands/metadata_command.rb

@@ -84,16 +84,24 @@ module Omnizip
         end
       end
 
+      # The display is driven by the registry's field contract for
+      # :seven_zip — optional lines print only for registered fields.
       def show_seven_zip_entry(entry)
-        puts "Entry: #{entry[:name]}"
-        puts "  Type: #{entry[:directory] ? 'Directory' : 'File'}"
-        puts "  Size: #{format_size(entry[:size])}"
+        registry = Omnizip::Metadata::MetadataRegistry
+        puts "Entry: #{entry[:name]}" if registry.supports?(:seven_zip, :name)
+        puts "  Type: #{entry[:directory] ? 'Directory' : 'File'}" if registry.supports?(:seven_zip, :directory)
+        puts "  Size: #{format_size(entry[:size])}" if registry.supports?(:seven_zip, :size)
         # Per-entry packed sizes are not tracked by the parser
-        if entry[:compressed_size]&.positive?
+        if registry.supports?(:seven_zip, :compressed_size) &&
+            entry[:compressed_size]&.positive?
           puts "  Compressed: #{format_size(entry[:compressed_size])}"
         end
-        puts "  Modified: #{entry[:mtime]}" if entry[:mtime]
-        puts "  CRC32: 0x#{entry[:crc].to_s(16).upcase}" if entry[:crc]
+        if registry.supports?(:seven_zip, :mtime) && entry[:mtime]
+          puts "  Modified: #{entry[:mtime]}"
+        end
+        if registry.supports?(:seven_zip, :crc) && entry[:crc]
+          puts "  CRC32: 0x#{entry[:crc].to_s(16).upcase}"
+        end
       end
 
       def show_metadata(archive, pattern)

data/lib/omnizip/formats/seven_zip/bcj2_stream_decompressor.rb

@@ -46,7 +46,7 @@ module Omnizip
           bcj2_coder = @folder.coders.find { |c| c.method_id == FilterId::BCJ2 }
           compression_coder = find_compression_coder
 
-          raise "BCJ2 coder not found" unless bcj2_coder
+          raise Omnizip::InvalidArchiveError, "BCJ2 coder not found" unless bcj2_coder
 
           # Determine stream layout based on folder structure
           # BCJ2 has 4 input streams: main, call, jump, rc
@@ -217,7 +217,7 @@ module Omnizip
           return packed_data unless algo_sym
 
           algo_class = Omnizip::AlgorithmRegistry.get(algo_sym)
-          raise "Algorithm not found: #{algo_sym}" unless algo_class
+          raise Omnizip::AlgorithmNotFoundError, "Algorithm not found: #{algo_sym}" unless algo_class
 
           # Build decoder options
           options = build_decoder_options(coder, algo_sym)

data/lib/omnizip/formats/seven_zip/coder_chain.rb

@@ -20,7 +20,7 @@ module Omnizip
           # Find the compression method (not a filter) among coders
           # Filters like BCJ, BCJ2 have specific method IDs
           main_coder = find_compression_coder(folder.coders)
-          raise "No compression method found in folder" unless main_coder
+          raise Omnizip::InvalidArchiveError, "No compression method found in folder" unless main_coder
 
           algorithm = algorithm_for_method(main_coder.method_id)
 
@@ -82,8 +82,8 @@ module Omnizip
           when MethodId::DEFLATE64
             :deflate64
           else
-            raise "Unsupported compression method: " \
-                  "0x#{method_id.to_s(16)}"
+            raise Omnizip::UnsupportedFormatError, "Unsupported compression method: " \
+                                                   "0x#{method_id.to_s(16)}"
           end
         end
 
@@ -126,7 +126,7 @@ module Omnizip
           # Get algorithm
           algo_class = AlgorithmRegistry.get(chain_config[:algorithm])
           unless algo_class
-            raise "Algorithm not found: #{chain_config[:algorithm]}"
+            raise Omnizip::AlgorithmNotFoundError, "Algorithm not found: #{chain_config[:algorithm]}"
           end
 
           # Create algorithm instance
@@ -143,7 +143,7 @@ module Omnizip
             pipeline = FilterPipeline.new
             chain_config[:filters].each do |filter_sym|
               filter_class = FilterRegistry.get(filter_sym)
-              raise "Filter not found: #{filter_sym}" unless filter_class
+              raise Omnizip::UnknownFilterError, "Filter not found: #{filter_sym}" unless filter_class
 
               pipeline.add_filter(filter_class.new)
             end

data/lib/omnizip/formats/seven_zip/encrypted_header.rb

@@ -69,7 +69,7 @@ module Omnizip
         # @return [EncryptedHeader] Parsed header
         # @raise [RuntimeError] if data is invalid
         def self.from_binary(data)
-          raise "Invalid encrypted header: too short" if data.bytesize < 54
+          raise Omnizip::InvalidArchiveError, "Invalid encrypted header: too short" if data.bytesize < 54
 
           pos = 0
 
@@ -77,7 +77,7 @@ module Omnizip
           marker = data.getbyte(pos)
           pos += 1
           unless marker == PropertyId::ENCODED_HEADER
-            raise "Invalid encrypted header marker: expected #{PropertyId::ENCODED_HEADER}, got #{marker}"
+            raise Omnizip::InvalidArchiveError, "Invalid encrypted header marker: expected #{PropertyId::ENCODED_HEADER}, got #{marker}"
           end
 
           # Read encrypted data size

data/lib/omnizip/formats/seven_zip/file_collector.rb

@@ -23,7 +23,7 @@ module Omnizip
         # @param recursive [Boolean] Recursively add directories
         def add_path(path, archive_path: nil, recursive: true)
           path = File.expand_path(path)
-          raise "Path not found: #{path}" unless File.exist?(path)
+          raise Errno::ENOENT, "Path not found: #{path}" unless File.exist?(path)
 
           @base_path ||= File.dirname(path)
 

data/lib/omnizip/formats/seven_zip/header.rb

@@ -29,14 +29,14 @@ module Omnizip
         def parse(io)
           # Read complete start header (32 bytes)
           header_data = io.read(START_HEADER_SIZE)
-          raise "Invalid .7z file: too short" if header_data.nil? ||
+          raise Omnizip::InvalidArchiveError, "Invalid .7z file: too short" if header_data.nil? ||
             header_data.bytesize < START_HEADER_SIZE
 
           # Validate signature
           signature = header_data[0, SIGNATURE_SIZE]
           unless signature == SIGNATURE
-            raise "Invalid .7z signature: expected #{SIGNATURE.inspect}, " \
-                  "got #{signature.inspect}"
+            raise Omnizip::InvalidArchiveError, "Invalid .7z signature: expected #{SIGNATURE.inspect}, " \
+                                                "got #{signature.inspect}"
           end
 
           # Parse version
@@ -44,7 +44,7 @@ module Omnizip
           @minor_version = header_data.getbyte(7)
 
           unless @major_version == MAJOR_VERSION
-            raise "Unsupported .7z version: #{@major_version}.#{@minor_version}"
+            raise Omnizip::UnsupportedFormatError, "Unsupported .7z version: #{@major_version}.#{@minor_version}"
           end
 
           # Parse start header CRC (bytes 8-11)

data/lib/omnizip/formats/seven_zip/header_encryptor.rb

@@ -83,9 +83,9 @@ module Omnizip
           begin
             decipher.update(encrypted_data) + decipher.final
           rescue OpenSSL::Cipher::CipherError => e
-            raise "Failed to decrypt header: incorrect password or corrupted data (#{e.message})"
+            raise Omnizip::PasswordError, "Failed to decrypt header: incorrect password or corrupted data (#{e.message})"
           rescue StandardError => e
-            raise "Failed to decrypt header: incorrect password or corrupted data (#{e.message})"
+            raise Omnizip::PasswordError, "Failed to decrypt header: incorrect password or corrupted data (#{e.message})"
           end
         end
 

data/lib/omnizip/formats/seven_zip/parser.rb

@@ -82,7 +82,7 @@ module Omnizip
           end
 
           value = read_number
-          raise "Unsupported 32-bit value" if value >= 0x80000000
+          raise Omnizip::InvalidArchiveError, "Unsupported 32-bit value" if value >= 0x80000000
 
           value
         end
@@ -233,7 +233,7 @@ module Omnizip
           end
 
           # Verify required SIZE property was present
-          raise "Missing SIZE property in pack info" unless sizes_read
+          raise Omnizip::InvalidArchiveError, "Missing SIZE property in pack info" unless sizes_read
 
           # K_END is optional in some 7-Zip versions
           read_byte if !eof? && peek_byte == PropertyId::K_END
@@ -259,7 +259,7 @@ module Omnizip
             end
           else
             # External folders not supported
-            raise "External folders not supported"
+            raise Omnizip::UnsupportedFormatError, "External folders not supported"
           end
         end
 
@@ -268,7 +268,7 @@ module Omnizip
         # @param folder [Models::Folder] Folder to populate
         def read_folder(folder)
           num_coders = read_number
-          raise "Too many coders" if num_coders > Constants::MAX_NUM_CODERS
+          raise Omnizip::InvalidArchiveError, "Too many coders" if num_coders > Constants::MAX_NUM_CODERS
 
           num_in_streams = 0
           num_out_streams = 0
@@ -386,8 +386,8 @@ module Omnizip
           end
 
           # Verify required properties were present
-          raise "Missing FOLDER property in unpack info" unless folders_read
-          raise "Missing CODERS_UNPACK_SIZE property in unpack info" unless unpack_sizes_read
+          raise Omnizip::InvalidArchiveError, "Missing FOLDER property in unpack info" unless folders_read
+          raise Omnizip::InvalidArchiveError, "Missing CODERS_UNPACK_SIZE property in unpack info" unless unpack_sizes_read
 
           # K_END is optional in some 7-Zip versions
           read_byte if !eof? && peek_byte == PropertyId::K_END
@@ -611,8 +611,8 @@ module Omnizip
           actual = read_byte
           return if actual == expected
 
-          raise "Expected property 0x#{expected.to_s(16)}, " \
-                "got 0x#{actual.to_s(16)}"
+          raise Omnizip::InvalidArchiveError, "Expected property 0x#{expected.to_s(16)}, " \
+                                              "got 0x#{actual.to_s(16)}"
         end
 
         # Skip size field

data/lib/omnizip/formats/seven_zip/reader.rb

@@ -223,18 +223,13 @@ module Omnizip
                 EncryptedHeader.from_binary(next_header_data)
                 # If we got here, it's encrypted - try to decrypt
                 next_header_data = decrypt_header(next_header_data)
-              rescue RuntimeError => e
-                # Re-raise password-related errors
-                if e.message.include?("Password required") || e.message.include?("incorrect password")
-                  raise
-                end
-
-                # Not encrypted - it's a compressed header
-                # Decompress it using the encoded stream info
-                next_header_data = decompress_encoded_header(io,
-                                                             next_header_data)
+              rescue Omnizip::PasswordError
+                # Decryption refused or failed — a password problem,
+                # never a compressed header
+                raise
               rescue StandardError
-                # Parsing error - not an encrypted header, try decompression
+                # Not an encrypted header — it's a compressed header;
+                # decompress it using the encoded stream info
                 next_header_data = decompress_encoded_header(io,
                                                              next_header_data)
               end
@@ -250,8 +245,8 @@ module Omnizip
             @stream_info, @entries = parse_metadata(parser)
           rescue StandardError => e
             if @headers_decrypted
-              raise "Failed to decrypt headers: incorrect password " \
-                    "or corrupted data (#{e.message})"
+              raise Omnizip::PasswordError, "Failed to decrypt headers: incorrect password " \
+                                            "or corrupted data (#{e.message})"
             end
 
             raise
@@ -271,7 +266,7 @@ module Omnizip
           @encrypted_header = EncryptedHeader.from_binary(encrypted_data)
 
           unless @password
-            raise "Archive headers are encrypted. Password required to access."
+            raise Omnizip::PasswordError, "Archive headers are encrypted. Password required to access."
           end
 
           # Decrypt using password
@@ -287,7 +282,7 @@ module Omnizip
           @headers_decrypted = true
           decrypted
         rescue OpenSSL::Cipher::CipherError => e
-          raise "Failed to decrypt headers: incorrect password (#{e.message})"
+          raise Omnizip::PasswordError, "Failed to decrypt headers: incorrect password (#{e.message})"
         end
 
         # Decompress encoded (compressed) header
@@ -317,7 +312,7 @@ module Omnizip
               parser.read_unpack_info(stream_info)
             end
           else
-            raise "Unexpected property in encoded header: 0x#{type.to_s(16)}"
+            raise Omnizip::InvalidArchiveError, "Unexpected property in encoded header: 0x#{type.to_s(16)}"
           end
 
           # Decompress the header using the stream info
@@ -341,7 +336,7 @@ module Omnizip
 
           # Read main header
           type = parser.read_byte
-          raise "Expected Header, got 0x#{type.to_s(16)}" unless
+          raise Omnizip::InvalidArchiveError, "Expected Header, got 0x#{type.to_s(16)}" unless
             type == PropertyId::HEADER
 
           # Parse header sections
@@ -529,7 +524,7 @@ module Omnizip
             crc = Omnizip::Checksums::Crc32.new
             crc.update(data)
             unless crc.value == entry.crc
-              raise "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
+              raise Omnizip::ChecksumError, "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
             end
           end
 
@@ -589,7 +584,7 @@ module Omnizip
               crc = Omnizip::Checksums::Crc32.new
               crc.update(data)
               unless crc.value == entry.crc
-                raise "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
+                raise Omnizip::ChecksumError, "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
               end
             end
 

data/lib/omnizip/formats/seven_zip/split_archive_reader.rb

@@ -167,7 +167,7 @@ module Omnizip
             volume_num += 1
           end
 
-          raise "No volumes found for #{@base_path}" if @volumes.empty?
+          raise Errno::ENOENT, "No volumes found for #{@base_path}" if @volumes.empty?
         end
 
         # Detect volumes with alpha naming (.aa, .ab, ...)
@@ -185,7 +185,7 @@ module Omnizip
             volume_num += 1
           end
 
-          raise "No volumes found for #{@base_path}" if @volumes.empty?
+          raise Errno::ENOENT, "No volumes found for #{@base_path}" if @volumes.empty?
         end
 
         # Open all volume files
@@ -293,7 +293,7 @@ module Omnizip
 
           # Read main header
           type = parser.read_byte
-          raise "Expected Header, got 0x#{type.to_s(16)}" unless
+          raise Omnizip::InvalidArchiveError, "Expected Header, got 0x#{type.to_s(16)}" unless
             type == PropertyId::HEADER
 
           # Parse header sections
@@ -372,7 +372,7 @@ module Omnizip
               parser.read_unpack_info(stream_info)
             end
           else
-            raise "Unexpected property in encoded header: 0x#{type.to_s(16)}"
+            raise Omnizip::InvalidArchiveError, "Unexpected property in encoded header: 0x#{type.to_s(16)}"
           end
 
           # Decompress the header using the stream info
@@ -491,7 +491,7 @@ module Omnizip
               crc = Omnizip::Checksums::Crc32.new
               crc.update(data)
               unless crc.value == entry.crc
-                raise "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
+                raise Omnizip::ChecksumError, "CRC mismatch for #{entry.name}: expected 0x#{entry.crc.to_s(16)}, got 0x#{crc.value.to_s(16)}"
               end
             end
 

data/lib/omnizip/formats/seven_zip/stream_compressor.rb

@@ -44,7 +44,7 @@ module Omnizip
           # Apply compression algorithm
           if @algorithm && @algorithm != :copy
             algo_class = Omnizip::AlgorithmRegistry.get(@algorithm)
-            raise "Algorithm not found: #{@algorithm}" unless algo_class
+            raise Omnizip::AlgorithmNotFoundError, "Algorithm not found: #{@algorithm}" unless algo_class
 
             encoder = algo_class.new
             input_io = StringIO.new(result)

data/lib/omnizip/formats/seven_zip/stream_decompressor.rb

@@ -57,7 +57,7 @@ module Omnizip
 
           # Get algorithm class
           algo_class = Omnizip::AlgorithmRegistry.get(algo_sym)
-          raise "Algorithm not found: #{algo_sym}" unless algo_class
+          raise Omnizip::AlgorithmNotFoundError, "Algorithm not found: #{algo_sym}" unless algo_class
 
           # Decompress
           input_io = StringIO.new(packed_data)
@@ -80,8 +80,8 @@ module Omnizip
 
               # BCJ2 requires special handling with multiple streams
               if filter_sym == :bcj2
-                raise "BCJ2 archives require multi-stream decompression which is not yet implemented. " \
-                      "Please use the 7z command-line tool for this archive."
+                raise Omnizip::UnsupportedFormatError, "BCJ2 archives require multi-stream decompression which is not yet implemented. " \
+                                                       "Please use the 7z command-line tool for this archive."
               end
 
               filter = filter_class.new
@@ -109,8 +109,8 @@ module Omnizip
             actual_crc = crc.value
 
             unless actual_crc == expected_crc
-              raise "CRC mismatch: expected 0x#{expected_crc.to_s(16)}, " \
-                    "got 0x#{actual_crc.to_s(16)}"
+              raise Omnizip::ChecksumError, "CRC mismatch: expected 0x#{expected_crc.to_s(16)}, " \
+                                            "got 0x#{actual_crc.to_s(16)}"
             end
           end
 

data/lib/omnizip/formats/seven_zip/writer.rb

@@ -299,7 +299,7 @@ module Omnizip
         # @return [String] Encrypted header with metadata
         def encrypt_header(header_data)
           unless @options[:password]
-            raise "Password required for header encryption"
+            raise Omnizip::PasswordError, "Password required for header encryption"
           end
 
           encryptor = HeaderEncryptor.new(@options[:password])

data/lib/omnizip/formats/zip/writer.rb

@@ -92,14 +92,17 @@ module Omnizip
           @entries << entry
         end
 
-        # Add data directly to the archive
-        def add_data(archive_path, data, stat = nil)
+        # Add data directly to the archive. +compression_method+
+        # (a ZIP method code) overrides the archive-wide default for
+        # this entry alone.
+        def add_data(archive_path, data, stat = nil, compression_method: nil)
           entry = create_entry(
             filename: archive_path,
             uncompressed_data: data,
             stat: stat,
           )
 
+          entry[:compression_method] = compression_method
           @entries << entry
         end
 
@@ -188,13 +191,17 @@ module Omnizip
         def write_to_io(io, compression_method: COMPRESSION_DEFLATE, level: 6)
           local_header_offsets = []
 
-          # Write local file headers and data
+          # Write local file headers and data. An entry-level
+          # :compression_method overrides the archive-wide default
+          # (used by the buffer layer's per-entry :store option).
           entries.each do |entry|
             offset = io.pos
             local_header_offsets << offset
 
+            entry_method = entry[:compression_method] || compression_method
+
             # Create local file header
-            local_header = create_local_header(entry, compression_method)
+            local_header = create_local_header(entry, entry_method)
 
             # Compress data if not a directory
             if entry[:directory]
@@ -205,7 +212,7 @@ module Omnizip
             else
               compressed_data = compress_data(
                 entry[:uncompressed_data],
-                compression_method,
+                entry_method,
                 level,
               )
               entry[:compressed_size] = compressed_data.bytesize
@@ -232,7 +239,7 @@ module Omnizip
           entries.each_with_index do |entry, index|
             central_header = create_central_header(
               entry,
-              compression_method,
+              entry[:compression_method] || compression_method,
               local_header_offsets[index],
             )
             io.write(central_header.to_binary)

data/lib/omnizip/implementations/seven_zip/lzma/decoder.rb

@@ -136,19 +136,19 @@ module Omnizip
           # Parse LZMA header
           def parse_header
             props = @input.getbyte
-            raise "Invalid LZMA header: missing properties" unless props
+            raise Omnizip::DecompressionError, "Invalid LZMA header: missing properties" unless props
 
             @lc = props % 9
             rem = props / 9
             @lp = rem % 5
             @pb = rem / 5
 
-            raise "Invalid LZMA properties: pb=#{@pb} > 4" if @pb > 4
+            raise Omnizip::DecompressionError, "Invalid LZMA properties: pb=#{@pb} > 4" if @pb > 4
 
             @dict_size = 0
             4.times do |i|
               byte = @input.getbyte
-              raise "Invalid LZMA header: missing dictionary size" unless byte
+              raise Omnizip::DecompressionError, "Invalid LZMA header: missing dictionary size" unless byte
 
               @dict_size |= (byte << (i * 8))
             end
@@ -159,7 +159,7 @@ module Omnizip
             @uncompressed_size = 0
             8.times do |i|
               byte = @input.getbyte
-              raise "Invalid LZMA header: missing uncompressed size" unless byte
+              raise Omnizip::DecompressionError, "Invalid LZMA header: missing uncompressed size" unless byte
 
               @uncompressed_size |= (byte * (1 << (i * 8)))
             end
@@ -328,7 +328,7 @@ module Omnizip
               # Decoder returns distance before +1, so add 1 to get actual distance
               distance += 1
 
-              raise "Invalid distance: #{distance}" if distance >= @dict_size && @dict_full
+              raise Omnizip::DecompressionError, "Invalid distance: #{distance}" if distance >= @dict_size && @dict_full
 
               # Update reps
               @reps[3] = @reps[2]

data/lib/omnizip/implementations/seven_zip/lzma/encoder.rb

@@ -309,7 +309,7 @@ module Omnizip
           # @return [void]
           def encode_match(match, data)
             # Defensive check: distance must be >= 1
-            raise "Invalid match distance: #{match.distance}" if match.distance < 1
+            raise Omnizip::CompressionError, "Invalid match distance: #{match.distance}" if match.distance < 1
 
             pos_state = @pos & ((1 << @pb) - 1)
 

data/lib/omnizip/implementations/seven_zip/lzma/range_decoder.rb

@@ -158,7 +158,7 @@ module Omnizip
           def init_decoder
             # Read first byte (should be 0 for valid LZMA stream)
             first = read_byte
-            raise "Invalid LZMA stream: first byte not 0" unless first.zero?
+            raise Omnizip::DecompressionError, "Invalid LZMA stream: first byte not 0" unless first.zero?
 
             # Read 4 bytes for initial code value
             @code = 0

data/lib/omnizip/implementations/seven_zip/lzma/state_machine.rb

@@ -108,7 +108,7 @@ module Omnizip
             when 9
               CATEGORY_SHORT_REP
             else
-              raise "Invalid state: #{@index}"
+              raise Omnizip::CompressionError, "Invalid state: #{@index}"
             end
           end
 

data/lib/omnizip/metadata/metadata_registry.rb

@@ -55,10 +55,17 @@ module Omnizip
                                 extra_field
                               ])
 
-    # Register 7z format metadata support (limited)
+    # Register 7z format metadata support: the readable fields the
+    # handler detail contract carries (name/size/directory always
+    # present; compressed_size when tracked, mtime/crc when stored).
+    # Editing is ZIP-only — the metadata command raises truthfully.
     MetadataRegistry.register(:seven_zip, %i[
+                                name
+                                size
+                                directory
+                                compressed_size
                                 mtime
-                                attributes
+                                crc
                               ])
   end
 end

data/lib/omnizip/parallel/engine.rb

@@ -31,17 +31,21 @@ module Omnizip
       # @return [Array<Fractor::WorkResult>] the failed results, for
       #   the caller to decide how to surface.
       def run(work_items)
-        pool = Fractor::WorkerPool.new(
+        pool = WorkerPool.new(
           worker_class: @worker_class,
           num_workers: @threads,
           continuous: false,
         )
         pool.start
-        pool.submit_batch(work_items)
-        pool.run
+        begin
+          pool.submit_batch(work_items)
+          pool.run
 
-        pool.successful_results.each { |r| yield r if block_given? }
-        pool.failed_results
+          pool.successful_results.each { |r| yield r if block_given? }
+          pool.failed_results
+        ensure
+          pool.shutdown
+        end
       end
     end
   end

data/lib/omnizip/parallel/job_scheduler.rb

@@ -38,6 +38,10 @@ module Omnizip
           schedule_dynamic(jobs, worker_count)
         when :static
           schedule_static(jobs, worker_count)
+        when :round_robin
+          schedule_round_robin(jobs, worker_count)
+        when :bin_packing
+          schedule_bin_packing(jobs, worker_count)
         end
       end
 
@@ -99,7 +103,7 @@ bytes_per_second: 10_000_000)
       #
       # @raise [ArgumentError] if strategy is invalid
       def validate_strategy!
-        valid_strategies = %i[dynamic static]
+        valid_strategies = %i[dynamic static round_robin bin_packing]
         return if valid_strategies.include?(@strategy)
 
         raise ArgumentError,