omnizip 0.3.34 → 0.3.35

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad5f2e41132ed61e2e80173f7550d75f3f0c8711d264e36a605fbee28e4a322e
-  data.tar.gz: 6d2b1989b06bb359cf48f2a5d24e8eea4b697cde22404c144cfef72357fdfb2d
+  metadata.gz: e76fab599398d72d3eb4999e7d516dabb26e1ca625dbc75afb35d48b7d037dcf
+  data.tar.gz: 6afeb78f1f806ba12a7577bfe8c6fd2f6ba127047ec8546b0513be5516a7420a
 SHA512:
-  metadata.gz: 38afce515d452d0dd4b1ace8b32608b59ff72e86d76d7d0ccfe3985f2d8cae6c24f4e657dc2e48d92b1b9194de2bca7e6d38eedb499255f1b8dcc3da5e2660db
-  data.tar.gz: dab546e8c4db730e7d257d29c89e903768e2112e858a87ef55daf3ac64305e584c715b3042a60e5caef7428357a8b12d4c2609e8ed2baa15612a5ada53ed3337
+  metadata.gz: 7a46395d5faa8ed51438bb680839490c99dd15c01f2409b60a8cf9241af8269150b0fb98694fc176c330af90ded71bea99e64f0c11770f0c9d8259abf1719b8f
+  data.tar.gz: bec34c361902bc7954b98d4d9d1f50e53cdac36bb9b800b3d6d0109a1ffc9e36e5211fba84cb38f829ef54c27a3eba79f03b0df8be39af0026c378f5feeee773

data/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### 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/version.rb

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

metadata

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