omnizip 0.3.13 → 0.3.15

data/lib/omnizip/algorithms/zstandard/xxhash.rb

@@ -0,0 +1,159 @@
+# frozen_string_literal: true
+
+# Copyright (C) 2025 Ribose Inc.
+#
+# Permission is hereby granted, free of charge, to any person obtaining a
+# copy of this software and associated documentation files (the "Software"),
+# to deal in the Software without restriction, including without limitation
+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
+# and/or sell copies of the Software, and to permit persons to whom the
+# Software is furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in
+# all copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+# DEALINGS IN THE SOFTWARE.
+
+module Omnizip
+  module Algorithms
+    class Zstandard
+      # XXHash64 (https://github.com/Cyan4973/xxHash), single-shot.
+      #
+      # Zstandard frame checksums are the low 32 bits of XXHash64 of the
+      # decoded content with seed 0 (the zstd C reference truncates
+      # XXH64_digest to a U32; RFC 8878's "xxh64" wording agrees).
+      module XXHash64
+        MASK64 = 0xFFFFFFFFFFFFFFFF
+        MASK32 = 0xFFFFFFFF
+
+        PRIME64_1 = 0x9E3779B185EBCA87
+        PRIME64_2 = 0xC2B2AE3D27D4EB4F
+        PRIME64_3 = 0x165667B19E3779F9
+        PRIME64_4 = 0x85EBCA77C2B2AE63
+        PRIME64_5 = 0x27D4EB2F165667C5
+
+        module_function
+
+        # Digest `data` with seed 0.
+        #
+        # @param data [String] binary string
+        # @return [Integer] 64-bit digest
+        def digest(data)
+          digest_seeded(data, 0)
+        end
+
+        def digest_seeded(data, seed)
+          len = data.bytesize
+          hash = 0
+          tail_start = 0
+
+          if len >= 32
+            v1 = (seed + PRIME64_1 + PRIME64_2) & MASK64
+            v2 = (seed + PRIME64_2) & MASK64
+            v3 = seed & MASK64
+            v4 = (seed - PRIME64_1) & MASK64
+
+            i = 0
+            while i + 32 <= len
+              v1 = round64(v1, read_u64(data, i))
+              v2 = round64(v2, read_u64(data, i + 8))
+              v3 = round64(v3, read_u64(data, i + 16))
+              v4 = round64(v4, read_u64(data, i + 24))
+              i += 32
+            end
+
+            hash = rotl64(v1, 1)
+            hash = (hash + rotl64(v2, 7)) & MASK64
+            hash = (hash + rotl64(v3, 12)) & MASK64
+            hash = (hash + rotl64(v4, 18)) & MASK64
+
+            hash = merge_round64(hash, v1)
+            hash = merge_round64(hash, v2)
+            hash = merge_round64(hash, v3)
+            hash = merge_round64(hash, v4)
+
+            tail_start = len - (len % 32)
+          else
+            hash = (seed + PRIME64_5) & MASK64
+          end
+
+          hash = (hash + len) & MASK64
+
+          i = tail_start
+          while i + 8 <= len
+            k1 = round64(0, read_u64(data, i))
+            hash ^= k1
+            hash = ((rotl64(hash, 27) * PRIME64_1) & MASK64) + PRIME64_4
+            hash &= MASK64
+            i += 8
+          end
+          if i + 4 <= len
+            hash ^= (read_u32(data, i) * PRIME64_1) & MASK64
+            hash = ((rotl64(hash, 23) * PRIME64_2) & MASK64) + PRIME64_3
+            hash &= MASK64
+            i += 4
+          end
+          while i < len
+            hash ^= (data.getbyte(i) * PRIME64_5) & MASK64
+            hash = (rotl64(hash, 11) * PRIME64_1) & MASK64
+            i += 1
+          end
+
+          hash ^= hash >> 33
+          hash = (hash * PRIME64_2) & MASK64
+          hash ^= hash >> 29
+          hash = (hash * PRIME64_3) & MASK64
+          hash ^= hash >> 32
+          hash
+        end
+
+        # Zstandard frame checksum: low 32 bits of XXHash64.
+        #
+        # @return [Integer] 32-bit checksum
+        def frame_checksum(data)
+          digest(data) & MASK32
+        end
+
+        def read_u64(data, offset)
+          data.getbyte(offset) |
+            (data.getbyte(offset + 1) << 8) |
+            (data.getbyte(offset + 2) << 16) |
+            (data.getbyte(offset + 3) << 24) |
+            (data.getbyte(offset + 4) << 32) |
+            (data.getbyte(offset + 5) << 40) |
+            (data.getbyte(offset + 6) << 48) |
+            (data.getbyte(offset + 7) << 56)
+        end
+
+        def read_u32(data, offset)
+          data.getbyte(offset) |
+            (data.getbyte(offset + 1) << 8) |
+            (data.getbyte(offset + 2) << 16) |
+            (data.getbyte(offset + 3) << 24)
+        end
+
+        def rotl64(value, count)
+          ((value << count) | (value >> (64 - count))) & MASK64
+        end
+
+        def round64(acc, input)
+          acc = ((acc + (input * PRIME64_2)) & MASK64) & MASK64
+          acc = rotl64(acc, 31)
+          (acc * PRIME64_1) & MASK64
+        end
+
+        def merge_round64(acc, value)
+          value = round64(0, value)
+          acc ^= value
+          (((acc * PRIME64_1) & MASK64) + PRIME64_4) & MASK64
+        end
+      end
+    end
+  end
+end

data/lib/omnizip/algorithms/zstandard.rb

@@ -48,22 +48,17 @@ module Omnizip
       autoload :Encoder, "omnizip/algorithms/zstandard/encoder"
       autoload :Decoder, "omnizip/algorithms/zstandard/decoder"
       autoload :Huffman, "omnizip/algorithms/zstandard/huffman"
+      autoload :HuffmanTableReader, "omnizip/algorithms/zstandard/huffman"
       autoload :HuffmanEncoder, "omnizip/algorithms/zstandard/huffman_encoder"
-      autoload :Literals, "omnizip/algorithms/zstandard/literals"
+      autoload :LiteralsDecoder, "omnizip/algorithms/zstandard/literals"
       autoload :LiteralsEncoder, "omnizip/algorithms/zstandard/literals_encoder"
-      autoload :Sequences, "omnizip/algorithms/zstandard/sequences"
+      autoload :SequencesDecoder, "omnizip/algorithms/zstandard/sequences"
+      autoload :SequenceExecutor, "omnizip/algorithms/zstandard/sequences"
+      autoload :XXHash64, "omnizip/algorithms/zstandard/xxhash"
 
-      # Frame module
+      # Frame and FSE modules
       autoload :Frame, "omnizip/algorithms/zstandard/frame"
-
-      # Nested Frame module classes
-      autoload :FrameHeader, "omnizip/algorithms/zstandard/frame/header"
-      autoload :FrameBlock, "omnizip/algorithms/zstandard/frame/block"
-
-      # Nested FSE module classes
-      autoload :FseEncoder, "omnizip/algorithms/zstandard/fse/encoder"
-      autoload :FseTable, "omnizip/algorithms/zstandard/fse/table"
-      autoload :FseBitstream, "omnizip/algorithms/zstandard/fse/bitstream"
+      autoload :FSE, "omnizip/algorithms/zstandard/fse"
 
       # Get algorithm metadata
       #
@@ -96,9 +91,7 @@ module Omnizip
       # @param options [Models::CompressionOptions] Decompression options
       # @return [void]
       def decompress(input_stream, output_stream, _options = nil)
-        if output_stream.respond_to?(:set_encoding)
-          output_stream.set_encoding(Encoding::BINARY)
-        end
+        output_stream.set_encoding(Encoding::BINARY)
         decoder = Decoder.new(input_stream)
         decompressed = decoder.decode_stream
         output_stream.write(decompressed)
@@ -108,18 +101,21 @@ module Omnizip
 
       # Build encoder options from compression options
       #
-      # @param options [Models::CompressionOptions, nil] Compression opts
+      # Always sets :level, defaulting to the zstd default (3) so that
+      # `compress(data)` never silently degrades to stored frames
+      # (issue #27). Accepts a Hash (as forwarded by Algorithm.compress)
+      # or a Models::CompressionOptions.
+      #
+      # @param options [Models::CompressionOptions, Hash, nil]
       # @return [Hash] Encoder options
       def build_encoder_options(options)
-        return {} if options.nil?
-
-        opts = {}
-
-        if options.respond_to?(:level)
-          opts[:level] = map_compression_level(options.level)
-        end
+        level = case options
+                when nil then nil
+                when Hash then options[:level]
+                else options.level if options.respond_to?(:level)
+                end
 
-        opts
+        { level: map_compression_level(level) }
       end
 
       # Map generic compression level (0-9) to Zstd level (1-22)

data/lib/omnizip/algorithms.rb

@@ -1,7 +1,20 @@
 # frozen_string_literal: true
 
 module Omnizip
-  # Compression algorithm implementations
+  # High-level compression algorithm facades.
+  #
+  # Each top-level constant in this namespace (Deflate, BZip2, LZMA,
+  # LZMA2, Zstandard, PPMd7, PPMd8) is a facade that exposes the
+  # streaming +.compress+/+.decompress+ contract from +Omnizip::Algorithm+.
+  # Format-specific readers/writers call these facades for the actual
+  # compression work; the facades themselves do not know about archive
+  # container structure.
+  #
+  # Bit-level SDK ports (state machines, range coders, match finders)
+  # live under +Omnizip::Implementations+, not here. This namespace
+  # owns the *streaming contract*; that namespace owns the *primitives*.
+  # The boundary is: +Algorithms+ calls +Implementations+, never the
+  # reverse.
   module Algorithms
     autoload :PPMdBase, "omnizip/algorithms/ppmd_base"
     autoload :LZMA2Const, "omnizip/algorithms/lzma2/constants"

data/lib/omnizip/buffer.rb

@@ -46,7 +46,7 @@ module Omnizip
       #     archive.add('dir/file2.txt', 'more content')
       #   end
       #   File.binwrite('output.zip', buffer.string)
-      # rubocop:disable Naming/BlockForwarding, Style/ArgumentsForwarding -- Ruby 3.0 compatibility
+      # rubocop:disable-next Naming/BlockForwarding, Style/ArgumentsForwarding -- Ruby 3.0 compatibility
       def create(format = :zip, **options, &block)
         buffer = StringIO.new(String.new(encoding: Encoding::BINARY))
 
@@ -61,7 +61,6 @@ module Omnizip
 
         buffer.tap(&:rewind)
       end
-      # rubocop:enable Naming/BlockForwarding, Style/ArgumentsForwarding
 
       # Open archive from memory
       #

data/lib/omnizip/commands/archive_list_command.rb

@@ -185,29 +185,31 @@ module Omnizip
         end
       end
 
-      # Helper methods to handle different entry types
+      # Helper methods to handle different entry types. Each delegates
+      # to the Omnizip::Entry contract; format-specific accessors stay
+      # on the format entry classes themselves.
       def entry_name(entry)
-        entry.respond_to?(:name) ? entry.name : entry.to_s
+        entry.entry_name || entry.to_s
       end
 
       def entry_directory?(entry)
-        entry.respond_to?(:directory?) && entry.directory?
+        entry.entry_directory?
       end
 
       def entry_size(entry)
-        entry.respond_to?(:size) ? entry.size : 0
+        entry.entry_size || 0
       end
 
       def entry_compressed_size(entry)
-        entry.respond_to?(:compressed_size) ? entry.compressed_size : nil
+        entry.compressed_size if entry.is_a?(Omnizip::Formats::SevenZip::Models::FileEntry)
       end
 
       def entry_has_stream?(entry)
-        entry.respond_to?(:has_stream?) ? entry.has_stream? : true
+        entry.has_stream? if entry.is_a?(Omnizip::Formats::SevenZip::Models::FileEntry)
       end
 
       def entry_mtime(entry)
-        entry.respond_to?(:mtime) ? entry.mtime : nil
+        entry.entry_mtime
       end
     end
   end

data/lib/omnizip/convenience.rb

@@ -152,12 +152,11 @@ module Omnizip
     end
 
     # Create a RAR archive (requires RAR license — see NotLicensedError).
-    # rubocop:disable Naming/BlockForwarding, Style/ArgumentsForwarding -- Ruby 3.0 compatibility
+    # rubocop:disable-next Naming/BlockForwarding, Style/ArgumentsForwarding -- Ruby 3.0 compatibility
     def create_rar(archive_path, **options, &block)
       options[:version] ||= 5
       Omnizip::Formats::Rar.create(archive_path, options, &block)
     end
-    # rubocop:enable Naming/BlockForwarding, Style/ArgumentsForwarding
 
     private
 

data/lib/omnizip/entry.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Omnizip
+  # Mix-in that establishes the common contract every archive entry
+  # satisfies, regardless of format.
+  #
+  # Each format-specific entry class (Zip::Entry, Tar::Entry,
+  # SevenZip::Models::FileEntry, Iso::DirectoryRecord, Ole::Dirent,
+  # Xar::Entry, Rpm::Entry, Msi::Entry, Cpio::Entry, Rar3::Entry,
+  # Rar5::Entry, Xz::Entry, Metadata::EntryMetadata) includes this
+  # module so callers can rely on a single narrow interface instead
+  # of duck-typing through `respond_to?(:name) / :path / :filename`
+  # cascades.
+  #
+  # Including classes MUST override the four methods below. The
+  # defaults raise `NotImplementedError` to surface missing
+  # implementations at call time rather than at include time.
+  module Entry
+    # @return [String, nil] in-archive path, or nil if the entry has
+    #   no name (e.g. empty anti-entry)
+    def entry_name
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_name"
+    end
+
+    # @return [Boolean] true if the entry is a directory
+    def entry_directory?
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_directory?"
+    end
+
+    # @return [Integer, nil] uncompressed size in bytes
+    def entry_size
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_size"
+    end
+
+    # @return [Time, nil] modification time
+    def entry_mtime
+      raise NotImplementedError,
+            "#{self.class} must implement #entry_mtime"
+    end
+  end
+end

data/lib/omnizip/extraction/selective_extractor.rb

@@ -184,18 +184,10 @@ module Omnizip
 
       # Get filename from entry
       #
-      # @param entry [Object] Entry
+      # @param entry [#entry_name] Entry (Omnizip::Entry or compatible)
       # @return [String] Filename
       def entry_filename(entry)
-        if entry.respond_to?(:name)
-          entry.name
-        elsif entry.respond_to?(:path)
-          entry.path
-        elsif entry.respond_to?(:filename)
-          entry.filename
-        else
-          entry.to_s
-        end
+        entry.entry_name || entry.to_s
       end
 
       # Update progress tracker

data/lib/omnizip/filter_pipeline.rb

@@ -36,7 +36,7 @@ module Omnizip
     # Filters are applied in the order they are added during encoding,
     # and in reverse order during decoding.
     #
-    # @param filter [Filters::FilterBase] Filter instance to add
+    # @param filter [Omnizip::Filter] Filter instance to add
     # @return [self] For method chaining
     def add_filter(filter)
       @filters << filter

data/lib/omnizip/filter_registry.rb

@@ -4,16 +4,18 @@ module Omnizip
   class FilterRegistry < Omnizip::Registry
     DEFAULT_FORMATS = %i[xz seven_zip].freeze
 
+    # Single source of truth for builtin filters. Each entry binds the
+    # filter name to its class and the formats that support it.
     BUILTIN_FILTERS = {
-      "bcj-x86": "Omnizip::Filters::BcjX86",
-      "bcj-arm": "Omnizip::Filters::BcjArm",
-      "bcj-arm64": "Omnizip::Filters::BcjArm64",
-      "bcj-ia64": "Omnizip::Filters::BcjIa64",
-      "bcj-ppc": "Omnizip::Filters::BcjPpc",
-      "bcj-sparc": "Omnizip::Filters::BcjSparc",
-      bcj: "Omnizip::Filters::BCJ",
-      bcj2: "Omnizip::Filters::Bcj2",
-      delta: "Omnizip::Filters::Delta",
+      "bcj-x86": { const: "Omnizip::Filters::BcjX86",   formats: %i[xz seven_zip] },
+      "bcj-arm": { const: "Omnizip::Filters::BcjArm",   formats: %i[xz seven_zip] },
+      "bcj-arm64": { const: "Omnizip::Filters::BcjArm64", formats: [:seven_zip] },
+      "bcj-ia64": { const: "Omnizip::Filters::BcjIa64", formats: %i[xz seven_zip] },
+      "bcj-ppc": { const: "Omnizip::Filters::BcjPpc", formats: %i[xz seven_zip] },
+      "bcj-sparc": { const: "Omnizip::Filters::BcjSparc", formats: %i[xz seven_zip] },
+      bcj: { const: "Omnizip::Filters::BCJ", formats: [:seven_zip] },
+      bcj2: { const: "Omnizip::Filters::Bcj2", formats: [:seven_zip] },
+      delta: { const: "Omnizip::Filters::Delta", formats: %i[xz seven_zip] },
     }.freeze
 
     class << self
@@ -86,7 +88,6 @@ module Omnizip
 
         trigger = lazy_triggers[normalized]
         if trigger
-          synchronize { lazy_triggers.delete(normalized) }
           trigger.call
           entry = storage[normalized]
           return entry if entry
@@ -98,12 +99,13 @@ module Omnizip
   end
 end
 
-# Lazy triggers — calling the Filters::Registry.register_all re-runs
-# the registration of every builtin filter. Idempotent: register just
-# overwrites the storage entry. Survives reset! because the trigger
-# proc is held by FilterRegistry, not the storage.
-Omnizip::FilterRegistry::BUILTIN_FILTERS.each_key do |name|
+# Lazy triggers — each trigger resolves the filter's class (firing
+# autoload) and registers it with its declared formats. Idempotent:
+# register overwrites the storage entry. Survives reset! because the
+# trigger proc is held by FilterRegistry, not the storage.
+Omnizip::FilterRegistry::BUILTIN_FILTERS.each do |name, spec|
   Omnizip::FilterRegistry.register_lazy(name) do
-    Omnizip::Filters::Registry.register_all
+    klass = Object.const_get(spec[:const])
+    Omnizip::FilterRegistry.register(name, klass, formats: spec[:formats])
   end
 end

data/lib/omnizip/filters/bcj2.rb

@@ -34,7 +34,7 @@ module Omnizip
     # NOTE: Currently only decoding (decompression) is implemented.
     # Encoding is extremely complex and will be added in a future version.
     # For compression use cases, the simpler BCJ-x86 filter is recommended.
-    class Bcj2 < FilterBase
+    class Bcj2 < Omnizip::Filter
       # Encode is not currently supported for BCJ2.
       #
       # @param _data [String] Binary data to encode

data/lib/omnizip/filters/bcj_arm.rb

@@ -28,7 +28,7 @@ module Omnizip
     # The filter improves compression by making branch targets
     # position-independent. The offset in ARM BL instructions is stored
     # as word offset (divided by 4), and is relative to PC+8.
-    class BcjArm < FilterBase
+    class BcjArm < Omnizip::Filter
       # ARM BL (Branch and Link) opcode
       OPCODE_BL = 0xEB
 

data/lib/omnizip/filters/bcj_arm64.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets and
     # page-aligned addresses position-independent.
-    class BcjArm64 < FilterBase
+    class BcjArm64 < Omnizip::Filter
       # B/BL instruction base opcode
       OPCODE_B_BL = 0x94000000
 

data/lib/omnizip/filters/bcj_ia64.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjIa64 < FilterBase
+    class BcjIa64 < Omnizip::Filter
       # Size of IA-64 instruction bundle (16 bytes, little-endian)
       BUNDLE_SIZE = 16
 

data/lib/omnizip/filters/bcj_ppc.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjPpc < FilterBase
+    class BcjPpc < Omnizip::Filter
       # PPC B/BL instruction base (0x48000000)
       OPCODE_BASE = 0x48000000
 

data/lib/omnizip/filters/bcj_sparc.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter improves compression by making branch targets
     # position-independent.
-    class BcjSparc < FilterBase
+    class BcjSparc < Omnizip::Filter
       # Size of SPARC instruction (4 bytes, big-endian)
       INSTRUCTION_SIZE = 4
 

data/lib/omnizip/filters/bcj_x86.rb

@@ -27,7 +27,7 @@ module Omnizip
     #
     # The filter is reversible and works on 5-byte boundaries (1-byte
     # opcode + 4-byte address).
-    class BcjX86 < FilterBase
+    class BcjX86 < Omnizip::Filter
       # x86 CALL opcode
       OPCODE_CALL = 0xE8
 

data/lib/omnizip/filters.rb

@@ -3,7 +3,6 @@
 module Omnizip
   # Filter implementations for preprocessing data before compression
   module Filters
-    autoload :FilterBase, "omnizip/filters/filter_base"
     autoload :BCJ, "omnizip/filters/bcj"
     autoload :BcjX86, "omnizip/filters/bcj_x86"
     autoload :Bcj2, "omnizip/filters/bcj2"
@@ -17,6 +16,5 @@ module Omnizip
     autoload :BcjSparc, "omnizip/filters/bcj_sparc"
     autoload :BcjIa64, "omnizip/filters/bcj_ia64"
     autoload :Delta, "omnizip/filters/delta"
-    autoload :Registry, "omnizip/filters/registry"
   end
 end

data/lib/omnizip/formats/bzip2_file.rb

@@ -112,12 +112,6 @@ module Omnizip
           output_io.write(decompressed_io.read)
         end
 
-        # Register BZIP2 format when loaded
-        def register!
-          FormatRegistry.register(".bz2", Omnizip::Formats::Bzip2File)
-          FormatRegistry.register(".bzip2", Omnizip::Formats::Bzip2File)
-        end
-
         private
 
         # Write BZIP2 header
@@ -163,4 +157,3 @@ module Omnizip
     end
   end
 end
-

data/lib/omnizip/formats/cpio/entry.rb

@@ -8,6 +8,7 @@ module Omnizip
       # Represents a file, directory, or special file in a CPIO archive.
       # Supports newc, CRC, and ODC formats.
       class Entry
+        include Omnizip::Entry
         include Omnizip::Formats::Cpio::Constants
 
         # @return [String] Magic number identifying format
@@ -58,6 +59,11 @@ module Omnizip
         # @return [String] File data
         attr_accessor :data
 
+        def entry_name = name
+        def entry_directory? = directory?
+        def entry_size = filesize
+        def entry_mtime = mtime
+
         # Initialize CPIO entry
         #
         # @param attributes [Hash] Entry attributes

data/lib/omnizip/formats/cpio.rb

@@ -126,13 +126,7 @@ module Omnizip
             }
           end
         end
-
-        # Auto-register CPIO format when loaded
-        def register!
-          FormatRegistry.register(".cpio", Reader)
-        end
       end
     end
   end
 end
-

data/lib/omnizip/formats/gzip.rb

@@ -127,12 +127,6 @@ module Omnizip
           metadata
         end
 
-        # Register GZIP format when loaded
-        def register!
-          FormatRegistry.register(".gz", Omnizip::Formats::Gzip)
-          FormatRegistry.register(".gzip", Omnizip::Formats::Gzip)
-        end
-
         private
 
         # Write GZIP header
@@ -231,4 +225,3 @@ module Omnizip
     end
   end
 end
-

data/lib/omnizip/formats/iso/directory_record.rb

@@ -6,9 +6,16 @@ module Omnizip
       # ISO 9660 Directory Record
       # Represents a file or directory entry
       class DirectoryRecord
+        include Omnizip::Entry
+
         attr_accessor :full_path
         attr_reader :length, :extended_attr_length, :location, :data_length, :recording_date, :flags, :file_unit_size, :interleave_gap_size, :volume_sequence_number, :name, :system_use
 
+        def entry_name = full_path || name
+        def entry_directory? = directory?
+        def entry_size = data_length
+        def entry_mtime = recording_date
+
         # Set the full path for this record, used by Reader when walking
         # a directory tree. Allows callers to look up records by path
         # without re-joining parent names.

data/lib/omnizip/formats/iso.rb

@@ -127,11 +127,6 @@ module Omnizip
 
         writer.write
       end
-
-      # Auto-register ISO format when loaded
-      def self.register!
-        FormatRegistry.register(".iso", Reader)
-      end
     end
   end
 end

data/lib/omnizip/formats/lzip.rb

@@ -132,12 +132,6 @@ module Omnizip
           }
         end
 
-        # Register LZIP format when loaded
-        def register!
-          FormatRegistry.register(".lz", Omnizip::Formats::Lzip)
-          FormatRegistry.register(".lzip", Omnizip::Formats::Lzip)
-        end
-
         # Detect if data is LZIP format
         #
         # @param data [String] Data to check
@@ -149,4 +143,3 @@ module Omnizip
     end
   end
 end
-