omnizip 0.3.30 → 0.3.31

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20b634d20f6abd7d5c44314d94b89e5008b1ad34e4c0efa14a01ef72a71c0723
-  data.tar.gz: 58d5380b5f2a98f92361395a55738047313f66c90bb9e662a56a47ee20333f11
+  metadata.gz: eb1ed1142019a375dd71be87daa4c59bdd509640530a183e34b5db31841d9d1b
+  data.tar.gz: a203011eea4196ef77193b642794c1a1eb5b870be4d1e9f66a1a233895d7c4b1
 SHA512:
-  metadata.gz: efcdcc1f1f3cf2774a6c955531f5a6747f4bad4464e0521f27e89d6b5c97b31bedcd1e3e2315f421bb7bea29a4aa73ca4f75dcee8eec702516e8c2b5349fb115
-  data.tar.gz: 96d041e3847fa402dbf86cfa3f86b7ba7b9b12f754471e10a8c60de8e966decbb0cfeb3711f71cfb905bf7acfea62a30128a1fb573f4f52e8e8e29b37fecc00b
+  metadata.gz: 17955bf45c57c5a2a6d4117b66617590f9ae5c0527c3f268b79b9be8f353f10b38a51829b60b9bb77cfd374840f694b55b1f56e3777cb8bb629e04e4b3044e17
+  data.tar.gz: 6396d10e54bfd95dcddff3e3475884f8240a3da8c83850156396c2e515b8ad19059094241260adac8684c202e64b5bf41dcf2507a1deda87dc44009661ea3e9d

data/CHANGELOG.md

@@ -7,7 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [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.
+
+## [0.3.30] - 2026-08-27
 
 ### Changed
 - Re-enabled LZMA2 in the 7-Zip header-encryption and split-archive

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/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.31"
 end

metadata

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