omnizip 0.3.51 → 0.3.52

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b268c5b315ddcfff3347f6843e30e7ca180b1b3bae14b21a7a721bd2ccf3f6e
-  data.tar.gz: 2d150420264453af6e1ba5114b32a31b1a67c12a020f41435a037747dec2bfde
+  metadata.gz: 8002bf03f99715929f352dfb92a91a10d02f6dac7152006a6cb87ca20b3647ae
+  data.tar.gz: b095258b0b0af358ae9c0a21482a6f8b29de1620bb2162d64dd8794f29eb8f59
 SHA512:
-  metadata.gz: '053812e04bf42b6fe5335d57a5183c90b19a3ed8b414ee57ac9a83831a04f15e7225ecc79d29bc42e78e692bf7fe6f60b88ba234a3fc92eb0e781e7a357acd14'
-  data.tar.gz: dd2e32bd6ea8732be35a2f6c7325ed23b922860c051caf7666df93edd7f7da1342bdc1f6c9928346a404a7a1c707c509bd74d1b281ddd07cdc4a591c67e8a05d
+  metadata.gz: 527be0108ee0fa63b6e7ebafdeaafd05430a17c08e0d59b0f731db7a3b76401da96daec205947c7c3b5baa07aba886515959513ef3d4b31438889761219e958d
+  data.tar.gz: e1f8665f545fc81a4459cbdffbb9e5d5aca219a177a5b2d2ac3bd47c589a2ceb11b02d8cfdc748ac39381417f12682bf53cf14105dcbcd3d04bd59b77d7f9e87

data/CHANGELOG.md

@@ -7,6 +7,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.51] - 2026-09-01
+
+### Changed
+- Profile resolution lives in one place: `Omnizip::Profile` gains
+  `resolve`, `apply`, and `first_file_among`, and the convenience
+  methods plus the `archive create` command collapse onto them. The
+  two prior copies had drifted — the command only accepted `"auto"`
+  strings and could not pass a `CompressionProfile` through.
+- Both direct converter strategies (ZIP↔7z) now extract through
+  `Omnizip.extract_archive` and write through a shared
+  `ConversionStrategy#repack_tree` on the Archive facade — they were
+  the last library code instantiating format readers/writers
+  directly, and the 7z→ZIP path required `#open` against the
+  reader-always-usable invariant.
+- `create_rar` routes through the RAR handler (unrar-verified) with
+  the generic writer block interface; its stale "requires RAR
+  license" docstring is gone. `decompress_file`'s three-arm case
+  with magic `:xz`/`:zstandard` markers becomes a
+  `SINGLE_FILE_DECOMPRESSORS` table mirroring the compressor table.
+
+### Fixed
+- macOS CI failed at setup from 2026-09-01: Homebrew disabled the
+  `rar` cask (Gatekeeper check), breaking `brew install rar`. The
+  workflow now installs it separately and non-fatally, like the
+  Windows branch already did for winrar; RAR interop specs skip
+  when unrar is absent.
+
 ## [0.3.50] - 2026-08-31
 
 ### Added

data/lib/omnizip/archive_handlers/seven_zip_handler.rb

@@ -31,7 +31,7 @@ module Omnizip
             entries.map do |e|
               { name: e.name, size: e.size, directory: e.is_dir,
                 compressed_size: (e.compressed_size if e.compressed_size&.positive?),
-                mtime: e.mtime }
+                mtime: e.mtime, crc: e.crc }
             end
           else
             entries.map(&:name)

data/lib/omnizip/cli/output_formatter.rb

@@ -44,19 +44,27 @@ module Omnizip
         ].join("\n")
       end
 
-      # Format file size in human-readable format.
+      # Format file size in human-readable format. This is THE byte
+      # formatter — commands, the CLI, and display-oriented models
+      # delegate here instead of carrying private copies.
       #
-      # @param bytes [Integer] Size in bytes
-      # @return [String] Formatted size
+      # @param bytes [Integer, nil] Size in bytes
+      # @return [String] Formatted size ("512 B", "1.5 KB", "2.0 MB")
       def format_size(bytes)
+        return "0 B" if bytes.nil? || bytes.zero?
+
         units = %w[B KB MB GB TB]
-        return "0 B" if bytes.zero?
+        size = bytes.to_f
+        unit_index = 0
+
+        while size >= 1024.0 && unit_index < units.size - 1
+          size /= 1024.0
+          unit_index += 1
+        end
 
-        exp = (Math.log(bytes) / Math.log(1024)).floor
-        exp = [exp, units.length - 1].min
+        return format("%d B", bytes) if unit_index.zero?
 
-        size = (bytes.to_f / (1024**exp)).round(2)
-        "#{size} #{units[exp]}"
+        format("%.1f %s", size, units[unit_index])
       end
 
       # Format error message for display.

data/lib/omnizip/cli.rb

@@ -60,13 +60,7 @@ module Omnizip
     # @param bytes [Integer] the byte count
     # @return [String] human-readable size (e.g. "1.5 MB")
     def format_bytes(bytes)
-      return "0 B" if bytes.zero?
-
-      units = %w[B KB MB GB TB]
-      exp = (Math.log(bytes) / Math.log(1024)).to_i
-      exp = [exp, units.size - 1].min
-
-      "%.1f %s" % [bytes.to_f / (1024**exp), units[exp]]
+      Omnizip::CliOutputFormatter.format_size(bytes)
     end
   end
 

data/lib/omnizip/commands/archive_create_command.rb

@@ -201,16 +201,7 @@ module Omnizip
       end
 
       def format_bytes(bytes)
-        units = %w[B KB MB GB]
-        size = bytes.to_f
-        unit_idx = 0
-
-        while size >= 1024 && unit_idx < units.length - 1
-          size /= 1024.0
-          unit_idx += 1
-        end
-
-        format("%.2f %s", size, units[unit_idx])
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
 
       def parse_volume_size(size_str)

data/lib/omnizip/commands/archive_extract_command.rb

@@ -135,17 +135,10 @@ module Omnizip
       end
 
       def extract_with_patterns(archive_file, output_dir, verbose)
-        # Determine archive type and open appropriately
-        archive = case File.extname(archive_file).downcase
-                  when ".zip"
-                    Omnizip::Zip::File.open(archive_file)
-                  when ".rar"
-                    Formats::Rar::Reader.new(archive_file).open
-                  when ".tar"
-                    Formats::Tar::Reader.new(archive_file).read
-                  else
-                    Formats::SevenZip::Reader.new(archive_file).open
-                  end
+        # Route through the facade: the extension routes pick each
+        # format's reader, and the reader session holds no open
+        # resources to close
+        archive = Omnizip::Archive.open(archive_file)
 
         # Build filter chain
         filter = Extraction::FilterChain.new
@@ -193,8 +186,6 @@ module Omnizip
         end
 
         extracted.size
-      ensure
-        archive.close if archive.is_a?(Omnizip::Zip::File)
       end
 
       def extract_gzip(archive_file, output_dir, verbose)

data/lib/omnizip/commands/archive_list_command.rb

@@ -157,24 +157,7 @@ module Omnizip
       end
 
       def format_bytes(bytes)
-        return "0 B" if bytes.zero?
-
-        units = %w[B KB MB GB]
-        size = bytes.to_f
-        unit_idx = 0
-
-        while size >= 1024 && unit_idx < units.length - 1
-          size /= 1024.0
-          unit_idx += 1
-        end
-
-        if size < 10
-          format("%.2f %s", size, units[unit_idx])
-        elsif size < 100
-          format("%.1f %s", size, units[unit_idx])
-        else
-          format("%.0f %s", size, units[unit_idx])
-        end
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
 
       # Helper methods to handle different entry types. Each delegates

data/lib/omnizip/commands/metadata_command.rb

@@ -54,7 +54,8 @@ module Omnizip
         %i[comment set_mtime chmod set_attribute].none? { |key| options[key] }
       end
 
-      # Read-only view for 7z archives; the edit path is zip-only
+      # Read-only view for 7z archives through the handler registry;
+      # the edit path is zip-only
       def show_seven_zip_metadata(archive_path, pattern)
         if options[:comment] || options[:set_mtime] ||
             options[:chmod] || options[:set_attribute]
@@ -62,37 +63,37 @@ module Omnizip
                 "Metadata editing is only supported for ZIP archives"
         end
 
-        reader = Omnizip::Formats::SevenZip::Reader.new(archive_path)
-        reader.open
+        entries = Omnizip.list_archive(archive_path, details: true)
 
         if pattern
-          entries = reader.list_files.select do |e|
-            File.fnmatch(pattern, e.name)
+          entries = entries.select do |entry|
+            File.fnmatch(pattern, entry[:name])
           end
           if entries.empty?
             warn "No entries match pattern: #{pattern}"
             return
           end
 
-          entries.each do |entry|
-            puts "Entry: #{entry.name}"
-            puts "  Type: #{entry.is_dir ? 'Directory' : 'File'}"
-            puts "  Size: #{format_size(entry.size)}"
-            # Per-entry packed sizes are not tracked by the parser
-            if 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
-          end
+          entries.each { |entry| show_seven_zip_entry(entry) }
         else
-          entries = reader.list_files
-          files = entries.reject(&:is_dir)
+          files = entries.reject { |entry| entry[:directory] }
           puts "Archive: #{archive_path}"
           puts "Entries: #{entries.size} " \
                "(#{files.size} files, #{entries.size - files.size} dirs)"
-          puts "Total size: #{format_size(files.sum(&:size))}"
+          puts "Total size: #{format_size(files.sum { |entry| entry[:size] })}"
+        end
+      end
+
+      def show_seven_zip_entry(entry)
+        puts "Entry: #{entry[:name]}"
+        puts "  Type: #{entry[:directory] ? 'Directory' : 'File'}"
+        puts "  Size: #{format_size(entry[:size])}"
+        # Per-entry packed sizes are not tracked by the parser
+        if 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]
       end
 
       def show_metadata(archive, pattern)
@@ -199,13 +200,7 @@ module Omnizip
       end
 
       def format_size(bytes)
-        return "0 B" if bytes.zero?
-
-        units = %w[B KB MB GB TB]
-        exp = (Math.log(bytes) / Math.log(1024)).to_i
-        exp = [exp, units.size - 1].min
-
-        "%.1f %s" % [bytes.to_f / (1024**exp), units[exp]]
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
     end
   end

data/lib/omnizip/commands/parity_create_command.rb

@@ -107,13 +107,7 @@ module Omnizip
       # @param bytes [Integer] Size in bytes
       # @return [String] Formatted size
       def format_size(bytes)
-        if bytes < 1024
-          "#{bytes} B"
-        elsif bytes < 1024 * 1024
-          "#{(bytes / 1024.0).round(1)} KB"
-        else
-          "#{(bytes / (1024.0 * 1024)).round(1)} MB"
-        end
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
     end
   end

data/lib/omnizip/extraction/selective_extractor.rb

@@ -33,7 +33,10 @@ module Omnizip
         FileUtils.mkdir_p(dest)
         extracted = []
 
-        entries_to_extract = list_matches
+        # Directory entries carry no content to extract
+        entries_to_extract = list_matches.reject do |entry|
+          entry_filename(entry).end_with?("/")
+        end
         total = entries_to_extract.size
         current = 0
 
@@ -162,7 +165,7 @@ module Omnizip
           entry.get_input_stream.read
         # allowed: some archives read entry content from the archive
         elsif @archive.respond_to?(:read)
-          @archive.read(entry)
+          @archive.read(entry_filename(entry))
         else
           raise Error, "Cannot read entry content"
         end
@@ -191,10 +194,19 @@ module Omnizip
 
       # Get filename from entry
       #
-      # @param entry [#entry_name] Entry (Omnizip::Entry or compatible)
+      # @param entry [#entry_name, #name] Entry (legacy Omnizip::Entry,
+      #   Archive::Entry from a reader session, or compatible)
       # @return [String] Filename
       def entry_filename(entry)
-        entry.entry_name || entry.to_s
+        # allowed: entries come from a caller-supplied archive
+        if entry.respond_to?(:entry_name)
+          entry.entry_name
+        # allowed: reader-session entries expose name, not entry_name
+        elsif entry.respond_to?(:name)
+          entry.name
+        else
+          entry.to_s
+        end
       end
 
       # Update progress tracker

data/lib/omnizip/models/conversion_result.rb

@@ -122,13 +122,7 @@ module Omnizip
       end
 
       def format_size(bytes)
-        return "0 B" if bytes.zero?
-
-        units = %w[B KB MB GB TB]
-        exp = (Math.log(bytes) / Math.log(1024)).to_i
-        exp = [exp, units.size - 1].min
-
-        "%.1f %s" % [bytes.to_f / (1024**exp), units[exp]]
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
     end
   end

data/lib/omnizip/parity/par2_creator.rb

@@ -161,40 +161,6 @@ progress: nil)
         Digest::MD5.digest("#{Time.now.to_f}#{rand}")
       end
 
-      # Analyze file and calculate hashes
-      #
-      # @param file_path [String] Path to file
-      # @return [FileInfo] File information
-      def analyze_file(file_path)
-        File.open(file_path, "rb") do |io|
-          file_size = io.size
-
-          # Calculate hash of first 16KB
-          first_16k = io.read(16384) || ""
-          hash_16k = Digest::MD5.digest(first_16k)
-
-          # Calculate full file hash
-          io.rewind
-          hash_full = Digest::MD5.file(file_path).digest
-
-          # Generate file ID
-          file_id = Digest::MD5.digest("#{File.basename(file_path)}#{file_size}")
-
-          # Read file blocks
-          io.rewind
-          blocks = read_file_blocks(io)
-
-          FileInfo.new(
-            path: file_path,
-            file_id: file_id,
-            hash_16k: hash_16k,
-            hash_full: hash_full,
-            size: file_size,
-            blocks: blocks,
-          )
-        end
-      end
-
       # Read file data into blocks
       #
       # @param io [IO] File IO object

data/lib/omnizip/parity/par2_verifier.rb

@@ -69,6 +69,11 @@ module Omnizip
       # @param par2_file [String] Path to .par2 index file
       # @raise [ArgumentError] if file doesn't exist
       def initialize(par2_file)
+        if par2_file.nil?
+          raise ArgumentError,
+                "PAR2 file is nil — no index .par2 was produced for this set"
+        end
+
         raise ArgumentError, "PAR2 file not found: #{par2_file}" unless
           File.exist?(par2_file)
 

data/lib/omnizip/profiler/report_generator.rb

@@ -145,18 +145,7 @@ module Omnizip
       end
 
       def format_bytes(bytes)
-        return "0 B" if bytes.nil? || bytes.zero?
-
-        units = %w[B KB MB GB]
-        size = bytes.to_f
-        unit_index = 0
-
-        while size >= 1024.0 && unit_index < units.size - 1
-          size /= 1024.0
-          unit_index += 1
-        end
-
-        format("%.2f %s", size, units[unit_index])
+        Omnizip::CliOutputFormatter.format_size(bytes)
       end
 
       def truncate(string, max_length)

data/lib/omnizip/profiler.rb

@@ -186,18 +186,7 @@ module Omnizip
     end
 
     def format_bytes(bytes)
-      return "0 B" if bytes.zero?
-
-      units = %w[B KB MB GB]
-      size = bytes.to_f
-      unit_index = 0
-
-      while size >= 1024.0 && unit_index < units.size - 1
-        size /= 1024.0
-        unit_index += 1
-      end
-
-      format("%.2f %s", size, units[unit_index])
+      Omnizip::CliOutputFormatter.format_size(bytes)
     end
   end
 end

data/lib/omnizip/temp/safe_extract.rb

@@ -79,37 +79,9 @@ module Omnizip
       private
 
       def extract_to_temp(temp_dir)
-        # Detect archive format and extract
-        case detect_format
-        when :zip
-          extract_zip(temp_dir)
-        when :seven_zip
-          extract_7z(temp_dir)
-        else
-          raise Omnizip::UnsupportedFormatError,
-                "Unknown archive format: #{@archive_path}"
-        end
-      end
-
-      def extract_zip(dest)
-        Omnizip::Zip::File.open(@archive_path) do |zip|
-          zip.each do |entry|
-            entry_path = File.join(dest, entry.name)
-
-            if entry.directory?
-              FileUtils.mkdir_p(entry_path)
-            else
-              FileUtils.mkdir_p(File.dirname(entry_path))
-              zip.extract(entry, entry_path)
-            end
-          end
-        end
-      end
-
-      def extract_7z(dest)
-        # Placeholder for 7z extraction
-        # Would use Omnizip::SevenZip::File when available
-        raise NotImplementedError, "7z extraction not yet implemented"
+        # Every routed format extracts through the handler registry,
+        # same as the convenience API
+        Omnizip.extract_archive(@archive_path, temp_dir)
       end
 
       def move_to_destination(temp_dir)
@@ -147,18 +119,6 @@ module Omnizip
         end
         count
       end
-
-      def detect_format
-        File.open(@archive_path, "rb") do |f|
-          magic = f.read(4)
-          case magic
-          when "PK\x03\x04"
-            :zip
-          when "7z\xBC\xAF"
-            :seven_zip
-          end
-        end
-      end
     end
   end
 end

data/lib/omnizip/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Omnizip
-  VERSION = "0.3.51"
+  VERSION = "0.3.52"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.51
+  version: 0.3.52
 platform: ruby
 authors:
 - Ribose Inc.