omnizip 0.3.42 → 0.3.43

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 060755f6c69bca46bcb999d0d36471045ae3d13e260cfa3c73601a8d440b1634
-  data.tar.gz: b524b77c579c824009821ca5d1bdb47327cfc55f3ebd1dab4ea8574d1ec3c631
+  metadata.gz: 436437db6697f7363305eda4e6a3e2b78b922aa12b3c43e29756efe187310b60
+  data.tar.gz: 6e0b17e143c50beae378cdb5b7902e773d888bca14183fcf83e61dfeabe56a5e
 SHA512:
-  metadata.gz: 237f14e0c479844a473d1830e956882a0d250f6c733522bcd9f3d36b7ba10b474b4564902791aecd52948ad330b82bb0bb7e61dd97db607bf3dcd85ab3dc847b
-  data.tar.gz: cfe4637c8ad1223ccb8b9735fc1494d61937b636f3d79dd7fbd2c1f59f6f35eb38686152ee56732f0cb29cfbf3c12f7ddfb6dae5bd6c146b8d833fca5315a00a
+  metadata.gz: f1e20ae301093cacb8c582e1b554a678c72cefa6bf21095d129bebc14b8c4095a68a87da653ef5ea16cb65c00de9edd6ae7b0d92191e49c25e05438ceba264ab
+  data.tar.gz: a6ad938ab6bdfd75586bcef013ba056cc523681ef34449d03bc44daf18a591e6485b292b7c62bd17c72c13c20c7ff8c45caf4faf6735d5cff79a0c2f4a5a010b

data/CHANGELOG.md

@@ -7,6 +7,26 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.42] - 2026-08-29
+
+### Changed
+- `Formats::Zip::Reader` is usable immediately: entries parse on
+  first use, so `extract_all`/listing no longer silently do nothing
+  when `#read` was not called first. `#read` remains for eager
+  loading.
+- `Temp::ArchiveHelper` writes zips through the native
+  `Formats::Zip::Writer` instead of the rubyzip-compat
+  `Omnizip::Zip::File` seam (the Metadata subsystem remains on
+  compat deliberately as an in-place central-directory editor).
+- The RAR extraction spill cache installs an `at_exit` sweeper on
+  first use, reclaiming spill directories at normal process exit.
+- Module docs tell the truth: `Formats::Rar` claimed read-only with
+  external tools required; `Formats::Iso` claimed read-only while
+  carrying the verified writer. New `CONTEXT.md` records the domain
+  glossary (primary parser vs adapters, native vs external decode,
+  interop-verified vs Omnizip-internal codecs, handler routes, the
+  compat seam, bridges).
+
 ## [0.3.41] - 2026-08-29
 
 ### Changed

data/CONTEXT.md

@@ -19,7 +19,11 @@ here.
   extract/list/read. `.rar`, `.cpio`, `.iso` are read-only routes;
   creating them raises truthfully.
 
-## Reading and decoding
+## Reading and decode
+
+- **Reader invariant — always usable** — every Reader
+  (`Formats::Zip`, `Formats::SevenZip`, `Formats::Rar`) parses on
+  first use; calling `#open` first is never required, only eager.
 
 - **Native decode** — decompression implemented in Ruby inside
   omnizip. RAR native decode is **CRC-gated**: the stored header CRC

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

@@ -12,7 +12,7 @@ module Omnizip
       class Reader
         include Omnizip::Formats::Rar::Constants
 
-        attr_reader :file_path, :header, :entries, :archive_info,
+        attr_reader :file_path, :header, :archive_info,
                     :volume_manager
 
         # Initialize reader with file path
@@ -25,6 +25,7 @@ module Omnizip
           @archive_info = Models::RarArchive.new(file_path)
           @volume_manager = VolumeManager.new(file_path)
           @use_native = true # Prefer native decompression
+          @opened = false
         end
 
         # Open and parse RAR archive
@@ -34,14 +35,24 @@ module Omnizip
           File.open(@file_path, "rb") do |io|
             parse_archive(io)
           end
+          @opened = true
           self
         end
 
+        # All entries, parsing on first use — a Reader is always
+        # usable; #open remains for eager loading
+        #
+        # @return [Array<Models::RarEntry>] File entries
+        def entries
+          ensure_open
+          @entries
+        end
+
         # List all files in archive
         #
         # @return [Array<Models::RarEntry>] File entries
         def list_files
-          @entries
+          entries
         end
 
         # Extract file to output path
@@ -51,6 +62,7 @@ module Omnizip
         # @param password [String, nil] Optional password
         # @raise [RuntimeError] if entry not found or extraction fails
         def extract_entry(entry_name, output_path, password: nil)
+          ensure_open
           entry = @entries.find { |e| e.name == entry_name }
           raise "Entry not found: #{entry_name}" unless entry
 
@@ -79,6 +91,7 @@ module Omnizip
         # @param password [String, nil] Optional password
         # @raise [RuntimeError] on extraction error
         def extract_all(output_dir, password: nil)
+          ensure_open
           FileUtils.mkdir_p(output_dir)
 
           # Use decompressor to extract all
@@ -126,6 +139,11 @@ module Omnizip
 
         private
 
+        # Parse the archive once, on demand
+        def ensure_open
+          open unless @opened
+        end
+
         # Parse RAR archive structure
         #
         # @param io [IO] Input stream

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

@@ -28,6 +28,7 @@ module Omnizip
           @offset = options[:offset] || 0
           @solid_cache = {}
           @solid_cache_extracted = {}
+          @opened = false
         end
 
         # Open and parse .7z archive
@@ -79,10 +80,12 @@ module Omnizip
           @split_reader&.volumes || [@file_path]
         end
 
-        # List all files in archive
+        # List all files in archive, parsing on first use — a Reader
+        # is always usable; #open remains for eager loading
         #
         # @return [Array<Models::FileEntry>] File entries
         def list_files
+          ensure_open
           @entries.reject { |e| e.name.nil? || e.name.empty? }
         end
 
@@ -92,6 +95,7 @@ module Omnizip
         # @param output_path [String] Destination path
         # @raise [RuntimeError] if entry not found or extraction fails
         def extract_entry(entry_name, output_path)
+          ensure_open
           # Delegate to split reader if available
           if @split_reader
             @split_reader.extract_entry(entry_name, output_path)
@@ -132,6 +136,7 @@ module Omnizip
         # @param output_dir [String] Destination directory
         # @raise [RuntimeError] on extraction error
         def extract_all(output_dir)
+          ensure_open
           FileUtils.mkdir_p(output_dir)
 
           real_out = File.realpath(output_dir)
@@ -187,6 +192,11 @@ module Omnizip
 
         private
 
+        # Parse the archive once, on demand
+        def ensure_open
+          open unless @opened
+        end
+
         # Parse .7z archive structure
         #
         # @param io [io] Input stream
@@ -236,7 +246,16 @@ module Omnizip
 
           # Parse metadata
           parser = Parser.new(next_header_data)
-          @stream_info, @entries = parse_metadata(parser)
+          begin
+            @stream_info, @entries = parse_metadata(parser)
+          rescue StandardError => e
+            if @headers_decrypted
+              raise "Failed to decrypt headers: incorrect password " \
+                    "or corrupted data (#{e.message})"
+            end
+
+            raise
+          end
 
           # Map entries to their folders/streams
           map_entries_to_streams
@@ -257,11 +276,16 @@ module Omnizip
 
           # Decrypt using password
           encryptor = HeaderEncryptor.new(@password)
-          encryptor.decrypt(
+          decrypted = encryptor.decrypt(
             @encrypted_header.encrypted_data,
             @encrypted_header.salt,
             @encrypted_header.iv,
           )
+          # AES-CBC 'succeeds' with any key and yields garbage; wrong
+          # passwords surface as later header-parse failures, so mark
+          # the archive and translate those into the password error
+          @headers_decrypted = true
+          decrypted
         rescue OpenSSL::Cipher::CipherError => e
           raise "Failed to decrypt headers: incorrect password (#{e.message})"
         end
@@ -393,6 +417,20 @@ module Omnizip
             entry.folder_index = folder_idx
             entry.file_index = i
             entry.size = @stream_info.unpack_sizes[stream_idx] if @stream_info.unpack_sizes[stream_idx]
+
+            # Single-stream folders own their packed bytes outright,
+            # so the entry's compressed size is exact. Solid folders
+            # share packed bytes across streams — leave nil rather
+            # than fabricate a split.
+            num_in_folder = @stream_info.num_unpack_streams_in_folders[folder_idx]
+            if num_in_folder == 1
+              folder = @stream_info.folders[folder_idx]
+              packed = folder.pack_stream_indices.sum do |pi|
+                @stream_info.pack_sizes[pi] || 0
+              end
+              entry.compressed_size = packed
+            end
+
             stream_idx += 1
           end
         end

data/lib/omnizip/version.rb

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

metadata

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