omnizip 0.3.48 → 0.3.50

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ff2a922c26dc80694f6167b0d46e16d28fe34fa9a0cdbcea12f6f5dc57085b6
-  data.tar.gz: f260edbb81b37ba0824fb9a896fc1668a60d3e74ffeba44821670d4304dd065d
+  metadata.gz: 6aa49d6ab2a370eda775a80206da80220936025ece39c3e121471815b309adbd
+  data.tar.gz: 60ab5eb1c7bcd3243f71aed658a85e84b2b37a02d061bf09156f0dd82c18d536
 SHA512:
-  metadata.gz: 0b47b5b5702166234e3c5e5cc132f3f944837904a9f4ce91a560e6c4207de329ec8d1e5ae01028e1b872c4dfbf5fda8b0ffee981ac9698cf500bc114d0a0ae65
-  data.tar.gz: e788bec8df533ea0417eb8db6e762a89ac17908e676558132eb127e1aa18af4c64bdc6da2151d50d772a4754c4a81ced191fa4b1b750fb6df27683d031ad237b
+  metadata.gz: 8e4cc8c3b167c07112ad5f45f3a870eb24b8ffd946cc7f9fe6c4b10bfe376e7efc5a0c975287027646d9bc7aef3f419e67eb1bd40e38824eb3e93219a3f78603
+  data.tar.gz: 7fe55f7e975c40eff0c2109167a0e23fa99f11949489a29f7555632a340b3ea16983091f09bfa6cd948e3351501fd66fb438d3ad90a40953ef514721fc0138f0

data/CHANGELOG.md

@@ -7,6 +7,53 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.49] - 2026-08-31
+
+### Added
+- Read routes for `.rpm`, `.xar`, and `.msi`: the formats had
+  verified readers but no handler and no extension route, so
+  `list_archive('pkg.rpm')` silently fell through to the ZIP
+  reader. Three read-only handlers register them (extraction,
+  listing, entry reading; creation raises truthfully). `.msi`
+  streams are listed exactly as stored — MSI table-stream names
+  are mangled in an MSI-domain encoding the handler does not guess
+  at. The CLI `decompress` command learns the new extensions
+  (`.rpm` previously fell into the single-file LZMA path and failed
+  with `Invalid pb`).
+
+### Fixed
+- The 7z writer silently ignored its documented `filters:` option
+  (preprocessing filters are decode-side only); it now warns that
+  it writes unfiltered data instead of pretending.
+
+## [0.3.48] - 2026-08-31
+
+### Changed
+- `omnizip archive create` routes through the Archive facade: the
+  command's two hand-munged format branches collapse into one
+  `Archive.create` call — the handler registry picks each format's
+  writer and applies its option semantics, and the command only
+  translates CLI vocabulary (414 -> 290 lines). Verified for 7z
+  (7zz byte-identical), RAR5, RAR4, volume splitting, and verbose
+  output.
+
+### Fixed
+- The 7z handler proxy dropped the Builder's explicit directory
+  entries (derive-from-paths no-op), so CLI-created archives lost
+  the `dir/` entries the previous writer path emitted. The proxy
+  maps name-only adds to `add_directory_entry`, which preserves the
+  trailing slash; the Buffer read proxy no longer double-slashes
+  such names.
+- The facade denied RAR passwords (`PASSWORD_FORMATS` listed 7z
+  only) while the RAR5 writer produces unrar-decryptable encrypted
+  archives — `:rar` added (verified with `unrar -p`).
+- RAR4 recovery guard crashed on explicit boolean options:
+  `false&.to_i` still calls `to_i` (safe navigation guards nil
+  only) — Numeric check now.
+- RAR interop specs gate on the unrar command (through its resolved
+  path), not the gem wrapper: CI runners can carry the gem without
+  the binary on PATH.
+
 ## [0.3.47] - 2026-08-31
 
 ### Fixed

data/lib/omnizip/archive_handler.rb

@@ -68,6 +68,9 @@ module Omnizip
     # file, which then self-registers at the bottom of the file.
     LAZY_LOAD_TRIGGERS = {
       zip: -> { Omnizip::ArchiveHandlers::ZipHandler },
+      rpm: -> { Omnizip::ArchiveHandlers::RpmHandler },
+      xar: -> { Omnizip::ArchiveHandlers::XarHandler },
+      ole: -> { Omnizip::ArchiveHandlers::OleHandler },
       tar: -> { Omnizip::ArchiveHandlers::TarHandler },
       seven_zip: -> { Omnizip::ArchiveHandlers::SevenZipHandler },
       rar: -> { Omnizip::ArchiveHandlers::RarHandler },

data/lib/omnizip/archive_handlers/ole_handler.rb

@@ -0,0 +1,37 @@
+# frozen_string: true
+
+module Omnizip
+  module ArchiveHandlers
+    # Read-only adapter for OLE compound documents ( MSI installers,
+    # legacy Office binaries): stream listing, reading, and
+    # extraction. Stream names are listed exactly as stored — MSI
+    # installers mangle their table-stream names in a
+    # MSI-domain encoding this handler does not decode.
+    class OleHandler
+      def create(_path, **_options)
+        raise Omnizip::UnsupportedFormatError,
+              "compound documents cannot be created through the " \
+              "convenience API; use Omnizip::Formats::Ole directly"
+      end
+
+      def extract_to(path, output_dir, **_)
+        Omnizip::Formats::Ole.extract(path, output_dir)
+        Dir.glob(File.join(output_dir, "**", "*"))
+          .select { |f| File.file?(f) }
+      end
+
+      def list(path, details: false, **_)
+        entries = Omnizip::Formats::Ole.list(path)
+        return entries unless details
+
+        entries.map { |name| { name: name, size: nil, directory: false } }
+      end
+
+      def read_entry(path, entry_name, **_)
+        Omnizip::Formats::Ole.read(path, entry_name)
+      end
+    end
+  end
+end
+
+Omnizip::ArchiveHandler.register(:ole, Omnizip::ArchiveHandlers::OleHandler.new)

data/lib/omnizip/archive_handlers/rpm_handler.rb

@@ -0,0 +1,45 @@
+# frozen_string: true
+
+require "tmpdir"
+
+module Omnizip
+  module ArchiveHandlers
+    # Read-only adapter for the RPM package format: extraction and
+    # listing over the CPIO payload; package metadata lives in
+    # Formats::Rpm.info. RPM writing exists at the format level but
+    # is not exposed through the convenience archive API.
+    class RpmHandler
+      def create(_path, **_options)
+        raise Omnizip::UnsupportedFormatError,
+              "rpm archives cannot be created through the convenience " \
+              "API; use Omnizip::Formats::Rpm::Writer directly"
+      end
+
+      def extract_to(path, output_dir, **_)
+        Omnizip::Formats::Rpm.extract(path, output_dir)
+        Dir.glob(File.join(output_dir, "**", "*"))
+          .select { |f| File.file?(f) }
+      end
+
+      def list(path, details: false, **_)
+        entries = Omnizip::Formats::Rpm.list(path)
+        return entries unless details
+
+        entries.map { |name| { name: name, size: nil, directory: false } }
+      end
+
+      def read_entry(path, entry_name, **_)
+        Dir.mktmpdir("omnizip-rpm-entry") do |dir|
+          Omnizip::Formats::Rpm.extract(path, dir)
+          dest = File.join(dir, entry_name)
+          raise Errno::ENOENT, "Entry not found: #{entry_name}" unless
+            File.exist?(dest)
+
+          File.binread(dest)
+        end
+      end
+    end
+  end
+end
+
+Omnizip::ArchiveHandler.register(:rpm, Omnizip::ArchiveHandlers::RpmHandler.new)

data/lib/omnizip/archive_handlers/xar_handler.rb

@@ -0,0 +1,49 @@
+# frozen_string: true
+
+require "tmpdir"
+
+module Omnizip
+  module ArchiveHandlers
+    # Read-only adapter for the XAR archive format: extraction and
+    # listing over the XML TOC. XAR writing exists at the format
+    # level but is not exposed through the convenience archive API.
+    class XarHandler
+      def create(_path, **_options)
+        raise Omnizip::UnsupportedFormatError,
+              "xar archives cannot be created through the convenience " \
+              "API; use Omnizip::Formats::Xar.create directly"
+      end
+
+      def extract_to(path, output_dir, **_)
+        Omnizip::Formats::Xar.extract(path, output_dir)
+        Dir.glob(File.join(output_dir, "**", "*"))
+          .select { |f| File.file?(f) }
+      end
+
+      def list(path, details: false, **_)
+        entries = Omnizip::Formats::Xar.list(path)
+        if details
+          entries.map do |e|
+            { name: e.name, size: e.size,
+              directory: false }
+          end
+        else
+          entries.map(&:name)
+        end
+      end
+
+      def read_entry(path, entry_name, **_)
+        Dir.mktmpdir("omnizip-xar-entry") do |dir|
+          Omnizip::Formats::Xar.extract(path, dir)
+          dest = File.join(dir, entry_name)
+          raise Errno::ENOENT, "Entry not found: #{entry_name}" unless
+            File.exist?(dest)
+
+          File.binread(dest)
+        end
+      end
+    end
+  end
+end
+
+Omnizip::ArchiveHandler.register(:xar, Omnizip::ArchiveHandlers::XarHandler.new)

data/lib/omnizip/archive_handlers.rb

@@ -10,6 +10,9 @@ module Omnizip
     autoload :SevenZipHandler, "omnizip/archive_handlers/seven_zip_handler"
     autoload :RarHandler, "omnizip/archive_handlers/rar_handler"
     autoload :CpioHandler, "omnizip/archive_handlers/cpio_handler"
+    autoload :RpmHandler, "omnizip/archive_handlers/rpm_handler"
+    autoload :XarHandler, "omnizip/archive_handlers/xar_handler"
+    autoload :OleHandler, "omnizip/archive_handlers/ole_handler"
     autoload :IsoHandler, "omnizip/archive_handlers/iso_handler"
   end
 end

data/lib/omnizip/commands/decompress_command.rb

@@ -80,7 +80,7 @@ module Omnizip
         end
       end
 
-      ARCHIVE_EXTENSIONS = %w[.zip .7z .tar .rar .cpio .iso].freeze
+      ARCHIVE_EXTENSIONS = %w[.zip .7z .tar .rar .cpio .iso .rpm .xar .msi].freeze
 
       private
 

data/lib/omnizip/convenience.rb

@@ -236,7 +236,7 @@ module Omnizip
     # creation raises truthfully instead (the format-level writers
     # exist for CPIO/ISO but are not part of the archive API; RAR
     # moved to the writable routes when its handler gained create).
-    READ_ONLY_FORMAT_EXTENSIONS = [".iso", ".cpio"].freeze
+    READ_ONLY_FORMAT_EXTENSIONS = %w[.iso .cpio .rpm .xar .msi].freeze
 
     # Extensions whose format has a READ-ONLY handler: extraction and
     # listing route to it, while creation keeps raising.
@@ -244,6 +244,9 @@ module Omnizip
       ".rar" => :rar,
       ".cpio" => :cpio,
       ".iso" => :iso,
+      ".rpm" => :rpm,
+      ".xar" => :xar,
+      ".msi" => :ole,
     }.freeze
 
     # Extension -> single-file decompressor (stream interface).

data/lib/omnizip/converter/conversion_registry.rb

@@ -45,5 +45,6 @@ module Omnizip
     # Register built-in strategies
     ConversionRegistry.register(ZipToSevenZipStrategy)
     ConversionRegistry.register(SevenZipToZipStrategy)
+    ConversionRegistry.register(ExtractRepackStrategy)
   end
 end

data/lib/omnizip/converter/extract_repack_strategy.rb

@@ -0,0 +1,61 @@
+# frozen_string: true
+
+require "tmpdir"
+
+module Omnizip
+  module Converter
+    # Fallback conversion strategy: extract any routed source format
+    # and repack the tree into any writable target format, both
+    # through the handler registry. Registered after the direct
+    # entry-copy strategies, which keep precedence for their pairs.
+    class ExtractRepackStrategy < ConversionStrategy
+      WRITABLE_FORMATS = %i[zip seven_zip tar rar].freeze
+
+      # Any routed source extension converts to any writable target
+      def self.can_convert?(source, _target)
+        !source_format_for(source).nil? &&
+          WRITABLE_FORMATS.include?(target_format_for(_target))
+      end
+
+      # Routed read-side format for an extension, or nil
+      def self.source_format_for(path)
+        ext = File.extname(path).downcase
+        Convenience::ARCHIVE_FORMAT_EXTENSIONS[ext] ||
+          Convenience::READ_ARCHIVE_FORMAT_EXTENSIONS[ext]
+      end
+
+      # Writable format for a target extension, or nil
+      def self.target_format_for(path)
+        Convenience::ARCHIVE_FORMAT_EXTENSIONS[File.extname(path).downcase]
+      end
+
+      def convert
+        start_time = Time.now
+
+        Dir.mktmpdir("omnizip_convert_repack") do |tmp|
+          extracted = Omnizip.extract_archive(source_path, tmp)
+
+          Omnizip::Archive.create(target_path,
+                                  format: self.class.target_format_for(target_path)) do |b|
+            Dir.glob(File.join(tmp, "**", "*")).each do |path|
+              next if File.directory?(path)
+
+              b.add_file(path, path.delete_prefix("#{tmp}/"))
+            end
+          end
+
+          entry_count = extracted.size
+          create_result(start_time, entry_count)
+        end
+      end
+
+      def source_format
+        self.class.source_format_for(source_path) || :unknown
+      end
+
+      def target_format
+        self.class.target_format_for(target_path) || :unknown
+      end
+    end
+  end
+end

data/lib/omnizip/converter.rb

@@ -11,6 +11,8 @@ module Omnizip
     autoload :SevenZipToZipStrategy,
              "omnizip/converter/seven_zip_to_zip_strategy"
     autoload :ConversionRegistry, "omnizip/converter/conversion_registry"
+    autoload :ExtractRepackStrategy,
+             "omnizip/converter/extract_repack_strategy"
 
     class << self
       # Convert archive from one format to another

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

@@ -37,6 +37,16 @@ module Omnizip
           }.merge(options)
           @collector = FileCollector.new
           @entries = []
+
+          # Preprocessing filters are decode-side only: the write
+          # path does not apply them, so say so instead of silently
+          # ignoring the option
+          filters = @options[:filters]
+          return unless filters && !filters.empty?
+
+          warn "omnizip: the 7z writer does not apply filters " \
+               "(#{Array(filters).join(', ')}); writing unfiltered data. " \
+               "Filters apply when reading archives that contain them."
         end
 
         def add_file(file_path, archive_path = nil)

data/lib/omnizip/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.48
+  version: 0.3.50
 platform: ruby
 authors:
 - Ribose Inc.
@@ -327,9 +327,12 @@ files:
 - lib/omnizip/archive_handlers.rb
 - lib/omnizip/archive_handlers/cpio_handler.rb
 - lib/omnizip/archive_handlers/iso_handler.rb
+- lib/omnizip/archive_handlers/ole_handler.rb
 - lib/omnizip/archive_handlers/rar_handler.rb
+- lib/omnizip/archive_handlers/rpm_handler.rb
 - lib/omnizip/archive_handlers/seven_zip_handler.rb
 - lib/omnizip/archive_handlers/tar_handler.rb
+- lib/omnizip/archive_handlers/xar_handler.rb
 - lib/omnizip/archive_handlers/zip_handler.rb
 - lib/omnizip/buffer.rb
 - lib/omnizip/buffer/memory_archive.rb
@@ -368,6 +371,7 @@ files:
 - lib/omnizip/converter.rb
 - lib/omnizip/converter/conversion_registry.rb
 - lib/omnizip/converter/conversion_strategy.rb
+- lib/omnizip/converter/extract_repack_strategy.rb
 - lib/omnizip/converter/seven_zip_to_zip_strategy.rb
 - lib/omnizip/converter/zip_to_seven_zip_strategy.rb
 - lib/omnizip/crypto.rb