omnizip 0.3.33 → 0.3.35

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f3064a982a38257ff4ef606d6de52925a377e2138a3bacc7cb60033a6ff1ac1
-  data.tar.gz: f24f28f6daac8df45542854f099c071d5b17f52deff99ae7f3bd5d009d6d7c0d
+  metadata.gz: e76fab599398d72d3eb4999e7d516dabb26e1ca625dbc75afb35d48b7d037dcf
+  data.tar.gz: 6afeb78f1f806ba12a7577bfe8c6fd2f6ba127047ec8546b0513be5516a7420a
 SHA512:
-  metadata.gz: 7e08082742ca10c5156773d5d279c1e5c7fe6d035cbe06678e2c68509862d89880c1a7f3e40d4e7a69b433e2407546d3735846a7d53a3c139819fa271001b852
-  data.tar.gz: 8d8551cf4cd26ae177b36e5aa67f217323853535ffaed26e4825f537e4384e998629b6084240a2dd82eaad276396dd51b22ab84a4d65f6f9f98475965ca6096e
+  metadata.gz: 7a46395d5faa8ed51438bb680839490c99dd15c01f2409b60a8cf9241af8269150b0fb98694fc176c330af90ded71bea99e64f0c11770f0c9d8259abf1719b8f
+  data.tar.gz: bec34c361902bc7954b98d4d9d1f50e53cdac36bb9b800b3d6d0109a1ffc9e36e5211fba84cb38f829ef54c27a3eba79f03b0df8be39af0026c378f5feeee773

data/CHANGELOG.md

@@ -7,6 +7,28 @@ 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
+  they are in the format (7z derives the tree from file paths;
+  nested paths verified against `7zz`), and the Tar handler routes
+  bare directory entries to the writer's `add_directory` instead of
+  misreading them as CWD-relative paths. Both round-trip nested
+  subdirectories through `extract_archive`.
+
+## [0.3.33] - 2026-08-27
+
 ### Fixed
 - The convenience API (`compress_file`, `compress_directory`,
   `extract_archive`, `list_archive`, `read_from_archive`,

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
@@ -40,12 +64,12 @@ module Omnizip
           @writer = writer
         end
 
+        # Directory entries are a no-op: 7z archives derive the
+        # directory tree from the files' archive paths ("sub/f.txt"
+        # implies "sub/"), and the writer/reader handle slashed
+        # paths (verified against 7zz).
         def add(name, source_path = nil)
-          if source_path.nil?
-            raise ArgumentError,
-                  "directory entries are not supported in .7z " \
-                  "convenience compression"
-          end
+          return if source_path.nil?
 
           @writer.add_file(source_path, name)
         end

data/lib/omnizip/archive_handlers/tar_handler.rb

@@ -6,7 +6,9 @@ module Omnizip
     # /+list+ interface for the TAR format. Wraps +Omnizip::Formats::Tar+.
     class TarHandler
       def create(path, &block)
-        Omnizip::Formats::Tar.create(path, &block)
+        Omnizip::Formats::Tar.create(path) do |writer|
+          block&.call(EntryProxy.new(writer))
+        end
       end
 
       def open(path, &block)
@@ -34,6 +36,22 @@ module Omnizip
         end
       end
 
+      # Translates the generic +add(name, source_path)+ interface:
+      # a bare +add(name)+ is a directory ENTRY (no source on disk
+      # relative to the caller), which the Tar writer's own +add+
+      # would misread as a CWD-relative path.
+      class EntryProxy
+        def initialize(writer)
+          @writer = writer
+        end
+
+        def add(name, source_path = nil)
+          return @writer.add_directory(name) if source_path.nil?
+
+          @writer.add(name, source_path)
+        end
+      end
+
       def read_entry(path, entry_name)
         data = nil
         Omnizip::Formats::Tar.open(path) do |reader|

data/lib/omnizip/version.rb

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

metadata

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