omnizip 0.3.34 → 0.3.36

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad5f2e41132ed61e2e80173f7550d75f3f0c8711d264e36a605fbee28e4a322e
-  data.tar.gz: 6d2b1989b06bb359cf48f2a5d24e8eea4b697cde22404c144cfef72357fdfb2d
+  metadata.gz: 53eb9dfabf6b8f299e86de445f0b2e487dcb12531ad6c901d35f89b306e53e97
+  data.tar.gz: 2df326cef1acba3e9de77ee349c4714ba3ba20c559beab62637dd70b2d55c1d8
 SHA512:
-  metadata.gz: 38afce515d452d0dd4b1ace8b32608b59ff72e86d76d7d0ccfe3985f2d8cae6c24f4e657dc2e48d92b1b9194de2bca7e6d38eedb499255f1b8dcc3da5e2660db
-  data.tar.gz: dab546e8c4db730e7d257d29c89e903768e2112e858a87ef55daf3ac64305e584c715b3042a60e5caef7428357a8b12d4c2609e8ed2baa15612a5ada53ed3337
+  metadata.gz: 182da8c8f1f5ccf84552f1bb7741f00340165d90bf8a19edcb3299dcdff02d99644b98b87fa13969dd62090c70f9973c97950267b816a3cceda988c53df2430b
+  data.tar.gz: e8745b56c4aded2fa04e81d0dd0ec011360f4593d590b4127aa284fac62b42766c39e26ce53f67b9f77e6e7cbe024345465a89af544b4594213e5d8811d297a2

data/CHANGELOG.md

@@ -7,6 +7,25 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- `add_to_archive`/`remove_from_archive` errors name the resolved
+  format (`Format :seven_zip does not support adding entries`)
+  instead of always blaming `:zip`, the default parameter the caller
+  never passed.
+
+## [0.3.35] - 2026-08-28
+
+### Fixed
+- `Omnizip.list_archive` on `.7z` returned raw Reader objects (the
+  SevenZip.open facade discards block return values) and
+  `read_from_archive` on `.7z` raised NotImplementedError. Both work
+  now: list returns entry names (or name/size/directory/mtime hashes
+  with `details: true`) and read returns entry bytes, wired through
+  the reader's `extract_entry_data`; missing entries raise
+  `Errno::ENOENT` like the ZIP handler.
+
+## [0.3.34] - 2026-08-27
+
 ### Fixed
 - `Omnizip.compress_directory` to `.7z` and `.tar` (both documented)
   works now: the 7z handler treats directory entries as the no-op

data/lib/omnizip/archive_handlers/seven_zip_handler.rb

@@ -21,16 +21,40 @@ module Omnizip
       end
 
       def list(path, details: false, **options)
-        Omnizip::Formats::SevenZip.open(path, options) do |reader|
-          files = reader.list_files
-          return details ? files.map { |f| { name: f } } : files
+        with_reader(path, options) do |reader|
+          entries = reader.list_files
+          if details
+            entries.map do |e|
+              { name: e.name, size: e.size, directory: e.is_dir,
+                mtime: e.mtime }
+            end
+          else
+            entries.map(&:name)
+          end
         end
       end
 
-      def read_entry(_path, _entry_name)
-        raise NotImplementedError,
-              "read_from_archive for .7z is not wired; use " \
-              "Formats::SevenZip.open"
+      def read_entry(path, entry_name, **options)
+        with_reader(path, options) do |reader|
+          entry = reader.list_files.find { |e| e.name == entry_name }
+          raise Errno::ENOENT, "Entry not found: #{entry_name}" unless entry
+
+          reader.extract_entry_data(File.open(path, "rb"), entry)
+        end
+      end
+
+      private
+
+      # The SevenZip.open facade returns the reader, discarding the
+      # block's value — so drive the Reader directly.
+      def with_reader(path, options)
+        reader = Omnizip::Formats::SevenZip::Reader.new(path, options)
+        reader.open
+        begin
+          yield reader
+        ensure
+          reader.split_reader&.close
+        end
       end
 
       # Translates the generic +add(name, source_path)+ interface to

data/lib/omnizip/convenience.rb

@@ -131,13 +131,14 @@ module Omnizip
         raise Errno::ENOENT, "Source file not found: #{source_path}"
       end
 
-      handler = Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
+      resolved = resolve_archive_format(archive_path, format)
+      handler = Omnizip::ArchiveHandler.for(resolved)
       # allowed: handler is a registered duck; missing add_entry raises below
       if handler.respond_to?(:add_entry)
         handler.add_entry(archive_path, entry_name, source_path)
       else
         raise Omnizip::UnsupportedFormatError,
-              "Format #{format.inspect} does not support adding entries"
+              "Format #{resolved.inspect} does not support adding entries"
       end
 
       archive_path
@@ -151,11 +152,12 @@ module Omnizip
     # @return [String] +archive_path+
     def remove_from_archive(archive_path, entry_name, format: DEFAULT_FORMAT)
       require_archive!(archive_path)
-      handler = Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
+      resolved = resolve_archive_format(archive_path, format)
+      handler = Omnizip::ArchiveHandler.for(resolved)
       # allowed: handler is a registered duck; missing remove_entry raises below
       unless handler.respond_to?(:remove_entry)
         raise Omnizip::UnsupportedFormatError,
-              "Format #{format.inspect} does not support removing entries"
+              "Format #{resolved.inspect} does not support removing entries"
       end
 
       handler.remove_entry(archive_path, entry_name)

data/lib/omnizip/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.34
+  version: 0.3.36
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-08-27 00:00:00.000000000 Z
+date: 2026-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64