omnizip 0.3.31 → 0.3.33

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb1ed1142019a375dd71be87daa4c59bdd509640530a183e34b5db31841d9d1b
-  data.tar.gz: a203011eea4196ef77193b642794c1a1eb5b870be4d1e9f66a1a233895d7c4b1
+  metadata.gz: 0f3064a982a38257ff4ef606d6de52925a377e2138a3bacc7cb60033a6ff1ac1
+  data.tar.gz: f24f28f6daac8df45542854f099c071d5b17f52deff99ae7f3bd5d009d6d7c0d
 SHA512:
-  metadata.gz: 17955bf45c57c5a2a6d4117b66617590f9ae5c0527c3f268b79b9be8f353f10b38a51829b60b9bb77cfd374840f694b55b1f56e3777cb8bb629e04e4b3044e17
-  data.tar.gz: 6396d10e54bfd95dcddff3e3475884f8240a3da8c83850156396c2e515b8ad19059094241260adac8684c202e64b5bf41dcf2507a1deda87dc44009661ea3e9d
+  metadata.gz: 7e08082742ca10c5156773d5d279c1e5c7fe6d035cbe06678e2c68509862d89880c1a7f3e40d4e7a69b433e2407546d3735846a7d53a3c139819fa271001b852
+  data.tar.gz: 8d8551cf4cd26ae177b36e5aa67f217323853535ffaed26e4825f537e4384e998629b6084240a2dd82eaad276396dd51b22ab84a4d65f6f9f98475965ca6096e

data/CHANGELOG.md

@@ -7,6 +7,34 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- The convenience API (`compress_file`, `compress_directory`,
+  `extract_archive`, `list_archive`, `read_from_archive`,
+  `add_to_archive`, `remove_from_archive`) now routes archive
+  formats by extension: `.tar` writes/reads real TAR (previously a
+  mislabeled ZIP), `.7z` writes/reads real 7z via a new
+  `ArchiveHandlers::SevenZipHandler` (previously a mislabeled ZIP;
+  archives verify with `7zz`), and read-only formats (`.rar`,
+  `.iso`, `.cpio`) raise `UnsupportedFormatError` instead of
+  silently writing a ZIP under a foreign name. Extensionless
+  outputs keep the ZIP default; an explicit `format:` still wins.
+
+## [0.3.32] - 2026-08-27
+
+### Fixed
+- The `omnizip` executable crashed on every invocation
+  (`uninitialized constant Omnizip::Cli`): the `Cli` autoload was
+  missing and the Thor command classes referenced
+  `Omnizip::Cli::Shared` mid-load, re-triggering the in-progress
+  autoload. The autoload exists now and Shared is defined ahead of
+  the command classes in `lib/omnizip/cli.rb` (its separate file is
+  gone). Verified: `omnizip version`, the documented
+  `compress/decompress output.lzma` round-trip (which also
+  interops with `xz --format=lzma` in both directions), and
+  `archive create backup.7z` producing an archive `7zz` validates.
+
+## [0.3.31] - 2026-08-27
+
 ### Fixed
 - `Omnizip.compress_file` now routes single-file compression by
   output extension: `.gz`, `.bz2`, `.xz`, `.lzma` and `.lz` produce
@@ -15,6 +43,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   `compress_file('input.txt', 'output.lzma', algorithm: :lzma)`
   example previously emitted `PK`-magic data). Unknown extensions
   and explicit archive formats keep the previous behavior.
+- CI: the Windows matrix no longer fails when choco's winrar
+  download 404s (RAR specs skip when it is absent).
 
 ## [0.3.30] - 2026-08-27
 

data/lib/omnizip/archive_handler.rb

@@ -69,6 +69,7 @@ module Omnizip
     LAZY_LOAD_TRIGGERS = {
       zip: -> { Omnizip::ArchiveHandlers::ZipHandler },
       tar: -> { Omnizip::ArchiveHandlers::TarHandler },
+      seven_zip: -> { Omnizip::ArchiveHandlers::SevenZipHandler },
     }.freeze
     private_constant :LAZY_LOAD_TRIGGERS
   end

data/lib/omnizip/archive_handlers/seven_zip_handler.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+module Omnizip
+  module ArchiveHandlers
+    # Adapter that exposes the canonical +create+/+extract_to+/+list+
+    # interface for the 7z format. Wraps +Omnizip::Formats::SevenZip+.
+    class SevenZipHandler
+      # Creates a .7z archive. The yielded proxy only supports
+      # +add(name, source_path)+ for files — directory entries are not
+      # part of the single-file convenience flow.
+      def create(path, &block)
+        Omnizip::Formats::SevenZip.create(path) do |writer|
+          block&.call(FileProxy.new(writer))
+        end
+      end
+
+      def extract_to(path, output_dir, **options)
+        Omnizip::Formats::SevenZip.open(path, options) do |reader|
+          reader.extract_all(output_dir)
+        end
+      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
+        end
+      end
+
+      def read_entry(_path, _entry_name)
+        raise NotImplementedError,
+              "read_from_archive for .7z is not wired; use " \
+              "Formats::SevenZip.open"
+      end
+
+      # Translates the generic +add(name, source_path)+ interface to
+      # the 7z writer's +add_file(file_path, archive_path)+.
+      class FileProxy
+        def initialize(writer)
+          @writer = writer
+        end
+
+        def add(name, source_path = nil)
+          if source_path.nil?
+            raise ArgumentError,
+                  "directory entries are not supported in .7z " \
+                  "convenience compression"
+          end
+
+          @writer.add_file(source_path, name)
+        end
+      end
+    end
+  end
+end
+
+Omnizip::ArchiveHandler.register(:seven_zip,
+                                 Omnizip::ArchiveHandlers::SevenZipHandler.new)

data/lib/omnizip/archive_handlers.rb

@@ -7,5 +7,6 @@ module Omnizip
   module ArchiveHandlers
     autoload :ZipHandler, "omnizip/archive_handlers/zip_handler"
     autoload :TarHandler, "omnizip/archive_handlers/tar_handler"
+    autoload :SevenZipHandler, "omnizip/archive_handlers/seven_zip_handler"
   end
 end

data/lib/omnizip/cli.rb

@@ -19,9 +19,38 @@
 require "thor"
 
 module Omnizip
+  # Shared helpers used by the Thor command groups defined below
+  # (ProfileCommands, ArchiveCommands, Cli). Defined here because the
+  # command classes include it at class-body time; a separate file
+  # under Omnizip::Cli would re-trigger this file's autoload
+  # mid-load.
+  module Shared
+    # Print a formatted error message and exit nonzero.
+    #
+    # @param error [StandardError] the error to display
+    def handle_error(error)
+      warn Omnizip::CliOutputFormatter.format_error(error)
+      exit 1
+    end
+
+    # Format a byte count with a binary-unit suffix.
+    #
+    # @param bytes [Integer] the byte count
+    # @return [String] human-readable size (e.g. "1.5 MB")
+    def format_bytes(bytes)
+      return "0 B" if bytes.zero?
+
+      units = %w[B KB MB GB TB]
+      exp = (Math.log(bytes) / Math.log(1024)).to_i
+      exp = [exp, units.size - 1].min
+
+      "%.1f %s" % [bytes.to_f / (1024**exp), units[exp]]
+    end
+  end
+
   # Profile commands subcommand group
   class ProfileCommands < Thor
-    include Omnizip::Cli::Shared
+    include Shared
 
     class << self
       def exit_on_failure?
@@ -69,7 +98,7 @@ module Omnizip
 
   # Archive commands subcommand group
   class ArchiveCommands < Thor
-    include Omnizip::Cli::Shared
+    include Shared
 
     class << self
       def exit_on_failure?
@@ -298,7 +327,7 @@ module Omnizip
   # Provides Thor-based CLI commands for compressing and decompressing
   # files using various compression algorithms.
   class Cli < Thor
-    include Omnizip::Cli::Shared
+    include Shared
 
     class << self
       def exit_on_failure?

data/lib/omnizip/convenience.rb

@@ -40,7 +40,7 @@ module Omnizip
         return output_path
       end
 
-      Omnizip::ArchiveHandler.for(format).create(output_path) do |archive|
+      Omnizip::ArchiveHandler.for(resolve_archive_format(output_path, format)).create(output_path) do |archive|
         archive.add(::File.basename(input_path), input_path)
       end
 
@@ -69,7 +69,7 @@ module Omnizip
         apply_profile(first_file, options)
       end
 
-      Omnizip::ArchiveHandler.for(format).create(output_path) do |archive|
+      Omnizip::ArchiveHandler.for(resolve_archive_format(output_path, format)).create(output_path) do |archive|
         add_directory_contents(archive, input_dir, "", recursive: recursive)
       end
 
@@ -87,7 +87,7 @@ module Omnizip
     def extract_archive(archive_path, output_dir, format: DEFAULT_FORMAT,
                         overwrite: false, **options)
       require_archive!(archive_path)
-      Omnizip::ArchiveHandler.for(format)
+      Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
         .extract_to(archive_path, output_dir,
                     overwrite: overwrite, **options)
     end
@@ -102,7 +102,7 @@ module Omnizip
     def list_archive(archive_path, format: DEFAULT_FORMAT, details: false,
                      **options)
       require_archive!(archive_path)
-      Omnizip::ArchiveHandler.for(format)
+      Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
         .list(archive_path, details: details, **options)
     end
 
@@ -114,7 +114,7 @@ module Omnizip
     # @return [String] Entry contents
     def read_from_archive(archive_path, entry_name, format: DEFAULT_FORMAT)
       require_archive!(archive_path)
-      Omnizip::ArchiveHandler.for(format).read_entry(archive_path, entry_name)
+      Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format)).read_entry(archive_path, entry_name)
     end
 
     # Add a file to an existing archive.
@@ -131,7 +131,7 @@ module Omnizip
         raise Errno::ENOENT, "Source file not found: #{source_path}"
       end
 
-      handler = Omnizip::ArchiveHandler.for(format)
+      handler = Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
       # 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)
@@ -151,7 +151,7 @@ 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(format)
+      handler = Omnizip::ArchiveHandler.for(resolve_archive_format(archive_path, format))
       # allowed: handler is a registered duck; missing remove_entry raises below
       unless handler.respond_to?(:remove_entry)
         raise Omnizip::UnsupportedFormatError,
@@ -172,6 +172,20 @@ module Omnizip
     # Extension -> single-file compressor map. Each lambda takes
     # (input_path, output_path, options). These formats are streams,
     # not archives, so they bypass ArchiveHandler.
+    # Extensions naming archive formats with a registered handler;
+    # routed when the caller did not pass an explicit +format:+.
+    ARCHIVE_FORMAT_EXTENSIONS = {
+      ".zip" => :zip,
+      ".tar" => :tar,
+      ".7z" => :seven_zip,
+    }.freeze
+
+    # Extensions naming real formats this gem can READ but not write
+    # through the convenience API. Writing them a ZIP under a foreign
+    # name was silent corruption; failing truthfully is the only
+    # honest behavior.
+    READ_ONLY_FORMAT_EXTENSIONS = [".rar", ".iso", ".cpio"].freeze
+
     SINGLE_FILE_COMPRESSORS = {
       ".gz" => lambda do |input, output, options|
         Omnizip::Formats::Gzip.compress(input, output, options)
@@ -206,6 +220,26 @@ module Omnizip
       SINGLE_FILE_COMPRESSORS[ext]
     end
 
+    # Archive format for a path: an explicit +format+ wins unless it
+    # is the default; otherwise the extension routes. Known-but-
+    # unwritable extensions raise instead of receiving a mislabeled
+    # ZIP.
+    def resolve_archive_format(path, format)
+      return format unless format == DEFAULT_FORMAT
+
+      ext = ::File.extname(path).downcase
+      routed = ARCHIVE_FORMAT_EXTENSIONS[ext]
+      return routed if routed
+
+      if READ_ONLY_FORMAT_EXTENSIONS.include?(ext)
+        raise Omnizip::UnsupportedFormatError,
+              "#{ext} archives cannot be written by the convenience " \
+              "API (read-only support); use the format-specific reader"
+      end
+
+      DEFAULT_FORMAT
+    end
+
     private
 
     def require_archive!(archive_path)

data/lib/omnizip/version.rb

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

data/lib/omnizip.rb

@@ -67,6 +67,7 @@ module Omnizip
   autoload :Profiler, "omnizip/profiler"
   autoload :Commands, "omnizip/commands"
   autoload :CliOutputFormatter, "omnizip/cli/output_formatter"
+  autoload :Cli, "omnizip/cli"
 
   # Implementation namespaces. Each namespace file declares autoloads
   # for its concrete classes. The classes self-register with their

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.31
+  version: 0.3.33
 platform: ruby
 authors:
 - Ribose Inc.
@@ -321,6 +321,7 @@ files:
 - lib/omnizip/algorithms/zstandard/xxhash.rb
 - lib/omnizip/archive_handler.rb
 - lib/omnizip/archive_handlers.rb
+- lib/omnizip/archive_handlers/seven_zip_handler.rb
 - lib/omnizip/archive_handlers/tar_handler.rb
 - lib/omnizip/archive_handlers/zip_handler.rb
 - lib/omnizip/buffer.rb
@@ -339,7 +340,6 @@ files:
 - lib/omnizip/chunked/writer.rb
 - lib/omnizip/cli.rb
 - lib/omnizip/cli/output_formatter.rb
-- lib/omnizip/cli/shared.rb
 - lib/omnizip/commands.rb
 - lib/omnizip/commands/.keep
 - lib/omnizip/commands/archive_create_command.rb

data/lib/omnizip/cli/shared.rb

@@ -1,31 +0,0 @@
-# frozen_string_literal: true
-
-module Omnizip
-  class Cli
-    # Shared helpers used by the Thor command groups defined in
-    # lib/omnizip/cli.rb (ProfileCommands, ArchiveCommands, Cli).
-    module Shared
-      # Print a formatted error message and exit nonzero.
-      #
-      # @param error [StandardError] the error to display
-      def handle_error(error)
-        warn Omnizip::CliOutputFormatter.format_error(error)
-        exit 1
-      end
-
-      # Format a byte count with a binary-unit suffix.
-      #
-      # @param bytes [Integer] the byte count
-      # @return [String] human-readable size (e.g. "1.5 MB")
-      def format_bytes(bytes)
-        return "0 B" if bytes.zero?
-
-        units = %w[B KB MB GB TB]
-        exp = (Math.log(bytes) / Math.log(1024)).to_i
-        exp = [exp, units.size - 1].min
-
-        "%.1f %s" % [bytes.to_f / (1024**exp), units[exp]]
-      end
-    end
-  end
-end