omnizip 0.3.30 → 0.3.32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20b634d20f6abd7d5c44314d94b89e5008b1ad34e4c0efa14a01ef72a71c0723
-  data.tar.gz: 58d5380b5f2a98f92361395a55738047313f66c90bb9e662a56a47ee20333f11
+  metadata.gz: a20cd2785f3068dabfe29b7cb3e9b88503b151839ea076fd4d4c727708e5a440
+  data.tar.gz: 7db78180133b7b7d3e62dff82588036c73dabc80c1957d892ffbff96fd83e657
 SHA512:
-  metadata.gz: efcdcc1f1f3cf2774a6c955531f5a6747f4bad4464e0521f27e89d6b5c97b31bedcd1e3e2315f421bb7bea29a4aa73ca4f75dcee8eec702516e8c2b5349fb115
-  data.tar.gz: 96d041e3847fa402dbf86cfa3f86b7ba7b9b12f754471e10a8c60de8e966decbb0cfeb3711f71cfb905bf7acfea62a30128a1fb573f4f52e8e8e29b37fecc00b
+  metadata.gz: 601948e2be2303a5c82573728118fe4fdb6bd123ce983f6f92776373e3215e96c1b29ab02217356353a8978335af54233093ede1aeb9aa8665a57f1f189a9525
+  data.tar.gz: ac68a261028b8a92a948bb0f93caed9997e5e3d13c885962d1857104c65f16d2da2eba38996aac90fb0a17da57d9fa8b7e1c806956e22d65d36765ebcf9f84c7

data/CHANGELOG.md

@@ -7,8 +7,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### 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
+  the matching format instead of silently writing a ZIP under a
+  foreign extension (the documented
+  `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
+
 ### Changed
 - Re-enabled LZMA2 in the 7-Zip header-encryption and split-archive
   specs (the multi-chunk encoder landed in 0.3.19; the specs had

data/TODO.refactor/00-overview.md

@@ -32,7 +32,7 @@ as absolute rules:
 | 10 | `10-format-detector-ocp.md` | medium | **DONE** |
 | 11 | `11-convenience-decoupling.md` | medium | **DONE** (ArchiveHandler dispatcher + ZipHandler + TarHandler) |
 | 12 | `12-thread-safety.md` | covered by 01 | n/a |
-| 13 | `13-lutaml-model-migration.md` | medium | **PARTIAL** — CompressionOptions and AlgorithmMetadata migrated; ConversionOptions pending lutaml symbol-type support |
+| 13 | `13-lutaml-model-migration.md` | medium | **DONE** — six models migrated; follow-ups 2+3 landed in 0.3.28; 1 and 4 decided (deferred rename / migration declined with reasons) |
 | 14 | `14-add-missing-specs.md` | high | **DONE** |
 
 ## Anti-pattern counts (lib/)

data/TODO.refactor/13-lutaml-model-migration.md

@@ -129,18 +129,30 @@ is the intended end state, not unfinished work.
 
 ## Follow-up work
 
-1. **Rename the 17.** If they are reports and projections, `to_h` is
-   the wrong name — that is the real defect. Suggested groupings:
-   `to_report` for the computed ones, `to_tag_map` / `to_attributes`
-   for projections, `to_xml_attributes` / `to_fields` for wire-format
-   builders. Each rename is itself a public-API break.
-2. **Fix `apply` in `CompressionOptions` and `AlgorithmMetadata`.**
-   Both iterate `self.class.attributes.each do |attr| ... attr.name`,
-   but `attributes` is a Hash, so the block gets a two-element Array
-   and `attr.name` raises `NoMethodError`. Neither has a caller or a
-   spec today, which is why it went unnoticed. Fix is `each_key`, plus
-   a spec per class.
-3. **Extract the shared `apply` loop** once (2) has landed — three
-   classes now carry a similar body.
-4. **Consider `FilterConfig` and the profile classes** for a proper
-   migration; both were judged too high-blast-radius to bundle here.
+1. **Rename the 17 — DECIDED: deferred to a deliberate major/minor
+   release.** The names below remain the plan if that release
+   happens: `to_report` for the computed ones, `to_tag_map` /
+   `to_attributes` for projections, `to_xml_attributes` / `to_fields`
+   for wire-format builders. Each rename is a public-API break
+   (every one of the 17 is a public class method), so it cannot ship
+   silently in a patch; the 17 staying is the intended end state for
+   now, per "Why 17 stay" above.
+2. **DONE (0.3.28, PR #42).** `apply` fixed in both classes — all
+   three options models now share the tested
+   `Models::AttributeApply` concern, which also delivers (3).
+3. **DONE (0.3.28, PR #42)** — see (2).
+4. **Considered — decided against, with reasons.**
+   - `FilterConfig` is a behavior-heavy domain object, not a
+     serialization model: `filter_instance` reaches into
+     `FilterRegistry`, `id_for_format`/`bcj?`/`delta?` are domain
+     queries, `properties` defaults to a binary String (lutaml's
+     `:string` is UTF-8-oriented, so the attribute would need a
+     custom type), and initialization accepts two alias keys
+     (`:name`/`:name_sym`). Its `to_h` is a projection with no
+     round-trip consumer. Migration would fight the framework to
+     achieve nothing the class needs.
+   - The profile classes are abstract domain objects: subclasses
+     override `suitable_for?`, `apply_to` mutates an options object,
+     five validators run at construction, and `to_h` coexists with
+     `to_s`/`inspect` as display projections. No wire format round
+     trips them. Same verdict.

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

@@ -31,6 +31,15 @@ module Omnizip
         return Omnizip::Chunked.compress_file(input_path, output_path, **options)
       end
 
+      # Single-file compression formats route by output extension
+      # (unless an archive format was requested explicitly): writing
+      # output.lzma through the archive path silently produced a ZIP
+      # under a foreign extension.
+      if format == DEFAULT_FORMAT && (handler = single_file_handler(output_path))
+        handler.call(input_path, output_path, options)
+        return output_path
+      end
+
       Omnizip::ArchiveHandler.for(format).create(output_path) do |archive|
         archive.add(::File.basename(input_path), input_path)
       end
@@ -160,6 +169,43 @@ module Omnizip
       Omnizip::Formats::Rar.create(archive_path, options, &block)
     end
 
+    # Extension -> single-file compressor map. Each lambda takes
+    # (input_path, output_path, options). These formats are streams,
+    # not archives, so they bypass ArchiveHandler.
+    SINGLE_FILE_COMPRESSORS = {
+      ".gz" => lambda do |input, output, options|
+        Omnizip::Formats::Gzip.compress(input, output, options)
+      end,
+      ".bz2" => lambda do |input, output, options|
+        Omnizip::Formats::Bzip2File.compress(input, output, options)
+      end,
+      ".xz" => lambda do |input, output, options|
+        Omnizip::Formats::Xz.create(::File.binread(input), output, options)
+      end,
+      ".lzma" => lambda do |input, output, options|
+        ::File.open(input, "rb") do |input_io|
+          ::File.open(output, "wb") do |output_io|
+            Omnizip::Formats::LzmaAlone.compress_stream(input_io, output_io,
+                                                        options)
+          end
+        end
+      end,
+      ".lz" => lambda do |input, output, options|
+        ::File.open(input, "rb") do |input_io|
+          ::File.open(output, "wb") do |output_io|
+            Omnizip::Formats::Lzip.compress_stream(input_io, output_io,
+                                                   options)
+          end
+        end
+      end,
+    }.freeze
+
+    # The single-file compressor matching the output extension, or nil.
+    def single_file_handler(output_path)
+      ext = ::File.extname(output_path).downcase
+      SINGLE_FILE_COMPRESSORS[ext]
+    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.30"
+  VERSION = "0.3.32"
 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.30
+  version: 0.3.32
 platform: ruby
 authors:
 - Ribose Inc.
@@ -339,7 +339,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