omnizip 0.3.45 → 0.3.46

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c60c442839bff64ee405d6eb9c5f3ead4a97fa53df7ffd381ca0508dbb00b5e8
-  data.tar.gz: 22b8abfb94d6403e16ffea3c36115c6c7eb15eed654ef15f8a824453aa2eef20
+  metadata.gz: eb792b03b604528825ce30f9d2ef6c7b6ac077535f5a4bf027498a44138ab692
+  data.tar.gz: 777e4ed705e6aa51b53ca8bccf2b394ae24f6f9b4404325db398a24e13b5a811
 SHA512:
-  metadata.gz: f9a8b45c51e6c36d30273d7ac897eb4854793ed1548d4055dcf90335b836c9d3d1bb4db2faa374cf407345657fd5b459f16bf40021427387ae103ce54f1ed644
-  data.tar.gz: 412293ad5b9d9d3c361fe03f8ed88707f3cd5dc4fc4bc1221198e877a71eb98322ca4cc5664ac329dee117192738d4d1af6b8e79ca8cc080639cf69e83816d89
+  metadata.gz: d512bfb76c29f2ca7a59bc6e46f558c2bc659b38dc1878b50ea879f91f93beff7c52a27a033b002f0a3e27ae2968484a8cb27b68aaa462a86aad9906f360a854
+  data.tar.gz: 25fdb5dacbe8dd8efb6835d916c8b455bcf07197845126dc8b832bbbf80418b80e904c5338a21e0300df6db83fd872fb6faa132f40e9b947ba19496a38eb44d2

data/CHANGELOG.md

@@ -7,6 +7,31 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.45] - 2026-08-31
+
+### Changed
+- The ZIP ArchiveHandler — the seam the facade, the convenience
+  layer, and the CLI all route through — now sits on the native
+  `Formats::Zip` tree instead of the rubyzip-compat
+  `Omnizip::Zip::File` layer (which remains for in-place entry
+  editing and Metadata, by design). The native reader gains
+  `#read_entry` (decompress + CRC-verify, shared with extraction)
+  and the central-directory entry model gains the `Omnizip::Entry`
+  contract plus DOS-time decoding.
+- CLI `archive list` and `archive extract` route through the
+  handler registry instead of hand-rolled format dispatch:
+  `archive list foo.tar` no longer falls into the 7z reader, zip
+  listing no longer opens the compat layer, and every handler's
+  `extract_to` returns the extracted file paths (7z/rar/tar
+  previously returned the reader or nil). Verbose listings show
+  real per-entry compressed sizes for the formats that track them.
+
+### Fixed
+- Parity hardening: a nil PAR2 index crashed five frames deep as a
+  `TypeError`; the repairer boundary now raises a named
+  `ArgumentError` and the spec fixture can no longer select a nil
+  index (the order-coupled CI flake from 0.3.44).
+
 ## [0.3.44] - 2026-08-31
 
 ### Fixed

data/lib/omnizip/archive_handlers/rar_handler.rb

@@ -1,19 +1,23 @@
 # frozen_string_literal: true
 
 require "tmpdir"
+require "tempfile"
 
 module Omnizip
   module ArchiveHandlers
-    # Read-only adapter for the RAR format. RAR compression is
-    # patented software (see link:README[RAR write support]); this
-    # handler exposes extraction/listing/reading only, so the
-    # convenience API and the Archive facade can operate on existing
-    # archives truthfully instead of raising on every operation.
+    # Adapter for the RAR format: extraction/listing/reading, plus
+    # creation through Formats::Rar (RAR5 STORE is unrar-verified;
+    # compressed methods fall back to STORE with a warning — see the
+    # README interop table).
     class RarHandler
-      def create(_path, **_options)
-        raise Omnizip::UnsupportedFormatError,
-              "RAR archives cannot be created (patented format); " \
-              "Omnizip reads RAR archives only"
+      def create(path, **options, &block)
+        proxy = nil
+        result = Omnizip::Formats::Rar.create(path, options) do |writer|
+          proxy = WriterProxy.new(writer)
+          block&.call(proxy)
+        end
+        proxy&.cleanup
+        result
       end
 
       def extract_to(path, output_dir, password: nil, **_)
@@ -47,6 +51,44 @@ module Omnizip
           File.binread(dest)
         end
       end
+
+      # Translates the generic +add(name, source_path)+ /
+      # +add_data(name, data)+ / +add_directory(name)+ interface onto
+      # the RAR writers. The writers take file paths, so in-memory
+      # content spills to a temporary file.
+      class WriterProxy
+        def initialize(writer)
+          @writer = writer
+          @spill = []
+        end
+
+        def add(name, source_path = nil)
+          if source_path
+            @writer.add_file(source_path, name)
+          else
+            @writer.add_directory_entry(name)
+          end
+        end
+
+        def add_data(name, data)
+          file = Tempfile.new(["omnizip_rar_entry", File.extname(name)])
+          file.binmode
+          file.write(data)
+          file.close
+          @spill << file
+          @writer.add_file(file.path, name)
+        end
+
+        def add_directory(name)
+          @writer.add_directory_entry(name)
+        end
+
+        # Remove the temporary files add_data spilled
+        def cleanup
+          @spill.each(&:unlink)
+          @spill.clear
+        end
+      end
     end
   end
 end

data/lib/omnizip/cli.rb

@@ -355,6 +355,18 @@ module Omnizip
     rescue StandardError => e
       handle_error(e)
     end
+
+    desc "repair ARCHIVE OUTPUT", "Repair a damaged archive"
+    long_desc <<~DESC
+      Repair a damaged archive (currently RAR archives via recovery
+      records and .rev volumes). OUTPUT is the repaired archive path.
+    DESC
+    option :verbose, type: :boolean, default: false, aliases: "-v"
+    def repair(archive, output)
+      Omnizip::Commands::ArchiveRepairCommand.new(options).run(archive, output)
+    rescue StandardError => e
+      handle_error(e)
+    end
   end
 
   # Command-line interface for Omnizip.

data/lib/omnizip/formats/rar/rar5/header.rb

@@ -106,8 +106,10 @@ header_data: "")
 
         # File header
         class FileHeader < Header
-          # File header flags
-          FILE_HAS_ATTRIBUTES = 0x0001
+          # File header flags. Per the RAR 5.0 spec, 0x0001 marks a
+          # directory entry (what this codebase once mislabeled as
+          # "has attributes" — attributes are an always-present field)
+          FILE_IS_DIRECTORY = 0x0001
           FILE_HAS_MTIME = 0x0002
           FILE_HAS_CRC32 = 0x0004
 
@@ -119,23 +121,27 @@ header_data: "")
 
           def initialize(filename:, file_size:, compressed_size:,
                          compression_method: 0, dict_size: 0,
-                         flags: 0, mtime: nil, crc32: nil, extra_area: nil)
+                         flags: 0, mtime: nil, crc32: nil,
+                         directory: false, extra_area: nil)
             # Build file flags based on what's provided
             file_flags = 0
+            file_flags |= FILE_IS_DIRECTORY if directory
             file_flags |= FILE_HAS_MTIME if mtime
             file_flags |= FILE_HAS_CRC32 if crc32
 
             # Build header data with file information
             data = build_file_data(filename, file_size,
                                    compression_method, dict_size,
-                                   file_flags, mtime, crc32)
+                                   file_flags, mtime, crc32,
+                                   directory: directory)
             super(HEADER_TYPE_FILE, flags: flags, data_area_size: compressed_size, header_data: data, extra_area: extra_area)
           end
 
           private
 
           def build_file_data(filename, file_size, compression_method,
-                              dict_size, file_flags, mtime, crc32)
+                              dict_size, file_flags, mtime, crc32,
+                              directory: false)
             data = []
 
             # File flags (VINT)
@@ -145,8 +151,9 @@ header_data: "")
             data.concat(VINT.encode(file_size))
 
             # Attributes (VINT) - ALWAYS present in RAR5.
-            # Unix host: standard regular-file mode 0o100644.
-            data.concat(VINT.encode(0o100644))
+            # Unix host: directory mode for directory entries,
+            # regular-file 0o100644 otherwise.
+            data.concat(VINT.encode(directory ? 0o040755 : 0o100644))
 
             # mtime (optional) - only if FILE_HAS_MTIME flag is set
             # RAR5 stores mtime as Unix timestamp (seconds since epoch)

data/lib/omnizip/formats/rar/rar5/writer.rb

@@ -121,6 +121,22 @@ module Omnizip
             }
           end
 
+          # Add a name-only directory entry (empty trees survive
+          # archive creation instead of vanishing)
+          #
+          # @param archive_path [String] Directory path inside the
+          #   archive (trailing slash optional)
+          # @return [self]
+          def add_directory_entry(archive_path)
+            @files << {
+              input: nil,
+              archive: archive_path.chomp("/"),
+              mtime: Time.now,
+              directory: true,
+            }
+            self
+          end
+
           # Add directory recursively
           #
           # @param dir_path [String] Directory path
@@ -227,11 +243,18 @@ module Omnizip
           # @param files [Array<Hash>] Files to compress in solid mode
           # @return [void]
           def write_solid_block(io, files)
+            # Directory entries carry no data; write their headers
+            # separately so the solid stream indexes stay aligned
+            files.each do |file|
+              write_directory_file_entry(io, file) if file[:directory]
+            end
+            solid_files = files.reject { |f| f[:directory] }
+
             # Create solid manager
             manager = Solid::SolidManager.new(level: @options[:level])
 
             # Add all files to solid stream
-            files.each do |file|
+            solid_files.each do |file|
               data = File.binread(file[:input])
               manager.add_file(file[:archive], data, mtime: file[:mtime],
                                                      stat: file[:stat])
@@ -242,7 +265,7 @@ module Omnizip
 
             # Write each file header with references to the solid block
             # In RAR5, all files share the same compressed data
-            files.each_with_index do |file, idx|
+            solid_files.each_with_index do |file, idx|
               file_info = result[:files][idx]
 
               # Calculate CRC32 if needed (note: only for STORE in non-solid)
@@ -286,6 +309,8 @@ module Omnizip
           # @param io [IO] Output stream
           # @param file [Hash] File information
           def write_file_entry(io, file)
+            return write_directory_file_entry(io, file) if file[:directory]
+
             # Read file data
             data = File.binread(file[:input])
 
@@ -350,6 +375,24 @@ module Omnizip
             io.write(final_data)
           end
 
+          # Write a directory entry: FileFlags directory bit, zero
+          # sizes, no data area
+          #
+          # @param io [IO] Output stream
+          # @param file [Hash] Entry with :archive name and :mtime
+          def write_directory_file_entry(io, file)
+            header = FileHeader.new(
+              filename: file[:archive],
+              file_size: 0,
+              compressed_size: 0,
+              compression_method: Compression::Store::METHOD,
+              dict_size: 0,
+              mtime: @options[:include_mtime] ? file[:mtime] : nil,
+              directory: true,
+            )
+            io.write(header.encode)
+          end
+
           # Write End header
           #
           # @param io [IO] Output stream

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

@@ -93,6 +93,21 @@ module Omnizip
           warn_unimplemented_options
         end
 
+        # Add a name-only directory entry (empty trees survive
+        # archive creation instead of vanishing)
+        #
+        # @param archive_path [String] Directory path inside the
+        #   archive (trailing slash optional)
+        # @return [self]
+        def add_directory_entry(archive_path)
+          @files << {
+            source: nil,
+            archive_path: archive_path.chomp("/"),
+            directory: true,
+          }
+          self
+        end
+
         # Add file to archive
         #
         # @param file_path [String] Path to file
@@ -241,10 +256,17 @@ module Omnizip
             (directory ? METHOD_STORE : select_compression_method(file_data))
           compressed_data = compress_data(file_data, method)
 
-          stat = File.stat(file_path)
-          file_attr = directory ? 0o040755 : stat.mode
-          file_time = dos_time(stat.mtime)
-          data_crc = Zlib.crc32(file_data)
+          if directory && file_path.nil?
+            # Name-only directory entry: no source to stat
+            file_attr = 0o040755
+            file_time = dos_time(Time.now)
+            data_crc = 0
+          else
+            stat = File.stat(file_path)
+            file_attr = directory ? 0o040755 : stat.mode
+            file_time = dos_time(stat.mtime)
+            data_crc = Zlib.crc32(file_data)
+          end
 
           name_bytes = archive_path.encode("UTF-8").bytes
 

data/lib/omnizip/formats/rar.rb

@@ -174,8 +174,8 @@ module Omnizip
         # @param archive_path [String] Path to corrupted RAR archive
         # @param output_path [String] Path for repaired archive
         # @return [ArchiveRepairer::RepairResult] Repair result
-        def repair(archive_path, output_path)
-          ArchiveRepairer.new.repair(archive_path, output_path)
+        def repair(archive_path, output_path, options = {})
+          ArchiveRepairer.new.repair(archive_path, output_path, options)
         end
 
         private

data/lib/omnizip/version.rb

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

metadata

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