omnizip 0.4.0 → 0.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7a04b0c83ee4e9c6b57823c8db709178ed2b88e073f658fb6573d9f69fa7459f
-  data.tar.gz: 1c5d9b7e86c1d1322cfd8a8193fcec2c8b7a4d28a21b6ee4b6fca3a359bc01e3
+  metadata.gz: 48ed1a8b9fa1d9c51d338c611430fbd78f12de0a9d5d9de3ebff23ac3b9b767c
+  data.tar.gz: 6b173fab258e290030227c869b703631023c621699e810753259b14ef613ef34
 SHA512:
-  metadata.gz: c22355d1136d66e0c6632ce6b4ad57a1b27c8cf826b0f5593318bffd20e85de77ed8072e280a88aa01433ef658fb1adc9139ff1ff8aa6d31735efb6aea1fc9f5
-  data.tar.gz: b06259b9ca3ede7ed15bdfa33537a9d9a17099a59ac5b0d5dc95a4f29a46ead222641abf4e8a17f17a283b1272f99d589c3541d799da0f932fc58a1401aa0d2c
+  metadata.gz: 07fba2b58cca8f0c0429042e4a2813d5618158cb8231625ad1e6f5c32d1bc532d45a8bb952a3087d6274862647dbab0480aca8c96561f24c5a3debf2a1f7dd63
+  data.tar.gz: 8422215bde30fa926b235b20dddbeba195573a82d1678ea98d541be6e37e6e14c7a6ae444e53b1a4505733a6420a4d9d3a53cec3605c751c5bf6eac49678ef86

data/CHANGELOG.md

@@ -7,6 +7,25 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+- RAR recovery-record repair fails loudly instead of pretending:
+  `recover_with_reed_solomon` returned nil and `recover_with_xor`
+  applied a transform the record does not use. Neither unrar (its
+  recvol5/rs16 cover only the separate .rev volumes) nor any public
+  spec carries the in-archive RAR5 RS parameters, so both paths now
+  raise `NotImplementedError` with par2 guidance, surfaced through
+  `ArchiveRepairer` results. The `ozip repair` counterpart (Rust)
+  ships in omnizip-rs with the same honest contract.
+
+### Added
+- `rake rust:build` — one command to build and vendor the Rust
+  backend's cdylib (clones omnizip-rs or reuses a checkout via
+  REPO=); the loader gains the `vendor/` resolution tier behind
+  `OMNIZIP_FFI_DYLIB`. No toolchain, no failure: the gem stays
+  pure Ruby. docs/rust-backend.md documents the tier policy,
+  resolution order, and CI pinning.
+
+
 ### Added
 - Rust-accelerated backend tier: `Omnizip::Backends` is the single
   tier switch between the pure-Ruby cores and the `omnizip-ffi`

data/lib/omnizip/formats/rar/parity_handler.rb

@@ -266,10 +266,16 @@ module Omnizip
         # @param parity_data [String] Parity information
         # @return [String, nil] Recovered data
         def recover_with_reed_solomon(_corrupted_data, _parity_data)
-          # Placeholder for Reed-Solomon recovery
-          # Real implementation would use reed-solomon gem
-          # For now, return nil to indicate recovery not available
-          nil
+          # The in-archive RAR5 recovery record's RS parameters exist
+          # only in the proprietary rar binary — unrar carries no
+          # implementation (its recvol5/rs16 cover only the separate
+          # .rev volume feature), and no independent spec is public.
+          # Fail loudly with actionable guidance instead of pretending
+          # recovery happened.
+          raise NotImplementedError,
+                "in-archive RAR5 Reed-Solomon recovery is not implemented " \
+                "(rar-proprietary format); protect archives with external " \
+                "parity (par2) instead"
         end
 
         # Recover data using XOR (RAR4)
@@ -277,14 +283,15 @@ module Omnizip
         # @param corrupted_data [String] Corrupted block
         # @param parity_data [String] Parity information
         # @return [String] Recovered data
-        def recover_with_xor(corrupted_data, parity_data)
-          # Simple XOR recovery
-          result = corrupted_data.bytes.map.with_index do |byte, i|
-            parity_byte = parity_data.bytes[i % parity_data.size]
-            byte ^ parity_byte
-          end
-
-          result.pack("C*")
+        def recover_with_xor(_corrupted_data, _parity_data)
+          # RAR4's integrated recovery record is likewise rar-proprietary
+          # (unrar has no repair path for it either); a cyclic XOR with
+          # arbitrary parity bytes is not the record's actual transform
+          # and would silently fabricate corrupt bytes. Refuse honestly.
+          raise NotImplementedError,
+                "in-archive RAR4 recovery-record repair is not implemented " \
+                "(rar-proprietary format); protect archives with external " \
+                "parity (par2) instead"
         end
       end
     end

data/lib/omnizip/version.rb

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

data/lib/tasks/rust_backend.rake

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+# One command to build the Rust-accelerated backend's cdylib into
+# the gem's vendor dir (TODO.ref-parity/62 — the distribution
+# story). Requires git + a Rust toolchain; the gem stays fully
+# functional (pure Ruby) without ever running this.
+#
+#   bundle exec rake rust:build
+#   bundle exec rake rust:build REPO=/path/to/omnizip-rs
+desc "Build the omnizip-ffi cdylib into vendor/ for the Rust tier"
+task "rust:build" do
+  repo = ENV.fetch("REPO", nil)
+  gem_root = File.expand_path("../..", __dir__)
+  build = File.join(gem_root, "build", "rust")
+
+  require "fileutils"
+  FileUtils.mkdir_p(build)
+  src = if repo
+          File.expand_path(repo)
+        else
+          target = File.join(build, "omnizip-rs")
+          if File.directory?(File.join(target, ".git"))
+            system("git", "-C", target, "fetch", "--quiet", "origin") or abort "git fetch failed in #{target}"
+            system("git", "-C", target, "reset", "--quiet", "--hard", "origin/main") or abort "git reset failed"
+          else
+            system("git", "clone", "--quiet", "--depth", "1",
+                   "https://github.com/omnizip/omnizip-rs.git", target) or abort "clone failed"
+          end
+          target
+        end
+
+  abort "#{src} has no omnizip-ffi crate" unless File.directory?(File.join(src, "omnizip-ffi"))
+
+  system("cargo", "build", "--release", "-p", "omnizip-ffi", chdir: src) or abort "cargo build failed"
+
+  vendor = File.join(gem_root, "vendor")
+  FileUtils.mkdir_p(vendor)
+  %w[dylib so dll].each do |ext|
+    candidate = File.join(src, "target", "release", "libomnizip_ffi.#{ext}")
+    next unless File.file?(candidate)
+
+    FileUtils.cp(candidate, vendor)
+    puts "vendored: #{File.join(vendor, "libomnizip_ffi.#{ext}")}"
+    puts "tier: OMNIZIP_BACKEND=auto now uses the Rust decoders"
+    exit 0
+  end
+  abort "cdylib not found under #{src}/target/release"
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.2
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-09-19 00:00:00.000000000 Z
+date: 2026-09-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -211,6 +211,7 @@ files:
 - docs/reference/index.adoc
 - docs/resources/faq.adoc
 - docs/resources/quick-reference.adoc
+- docs/rust-backend.md
 - docs/troubleshooting/index.adoc
 - docs/xar_format.md
 - examples/api_comparison.rb
@@ -722,6 +723,7 @@ files:
 - lib/omnizip/zip/input_stream.rb
 - lib/omnizip/zip/output_stream.rb
 - lib/tasks/msi_test_files.rake
+- lib/tasks/rust_backend.rake
 - readme-docs/advanced-features.adoc
 - readme-docs/api-usage.adoc
 - readme-docs/architecture.adoc