omnizip 0.3.9 → 0.3.11

data/lib/omnizip/registry.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+module Omnizip
+  # Generic, thread-safe registry base class.
+  #
+  # Subclasses override the hooks +#not_found_error_class+,
+  # +#normalize_key+, and +#label+ to specialize behavior. They may also
+  # override +#register+ / +#get+ to add domain-specific arguments
+  # (e.g., +#register_with_formats+ on FilterRegistry).
+  #
+  # The base class is intentionally minimal: it owns a Hash store and a
+  # Mutex, and exposes the canonical CRUD quintet (+register+, +get+,
+  # +registered?+, +available+, +reset!+). Backward-compat aliases are
+  # provided so existing call sites keep working.
+  class Registry
+    class << self
+      def not_found_error_class
+        Omnizip::Error
+      end
+
+      def normalize_key(key)
+        key.to_sym
+      end
+
+      def label
+        return "Item" unless name
+
+        name.gsub(/^Omnizip::/, "")
+      end
+
+      def register(key, value)
+        synchronize { storage[normalize_key(key)] = value }
+        value
+      end
+
+      # Register a lazy trigger for +key+. When +get(key)+ is called and
+      # the key isn't yet in storage, the block is invoked. The block is
+      # expected to trigger autoload of the implementation file (e.g., by
+      # referencing a constant), which then self-registers.
+      def register_lazy(key, &trigger)
+        lazy_triggers[normalize_key(key)] = trigger
+      end
+
+      def get(key)
+        normalized = normalize_key(key)
+        value = storage[normalized]
+        return value if value
+
+        trigger = lazy_triggers[normalized]
+        if trigger
+          synchronize { lazy_triggers.delete(normalized) }
+          trigger.call
+          value = storage[normalized]
+          return value if value
+        end
+
+        raise not_found_error_class,
+              "#{label} not registered: #{key.inspect}. " \
+              "Available: #{available.join(', ')}"
+      end
+
+      def registered?(key)
+        normalized = normalize_key(key)
+        storage.key?(normalized) || lazy_triggers.key?(normalized)
+      end
+
+      def available
+        storage.keys
+      end
+      alias all available
+      alias strategies available
+
+      def reset!
+        # Clear stored entries but keep lazy triggers — the builtin
+        # autoload list should survive a reset so subsequent +get+ calls
+        # for builtins still trigger the autoload (which re-registers).
+        synchronize { storage.clear }
+      end
+      alias clear reset!
+      alias clear! reset!
+      alias reset reset!
+
+      def entries
+        synchronize { storage.dup }
+      end
+
+      def each(&block)
+        return to_enum(:each) unless block
+
+        entries.each(&block)
+      end
+
+      private
+
+      def storage
+        @storage ||= {}
+      end
+
+      def lazy_triggers
+        @lazy_triggers ||= {}
+      end
+
+      def synchronize(&block)
+        mutex.synchronize(&block)
+      end
+
+      def mutex
+        @mutex ||= Mutex.new
+      end
+    end
+  end
+end

data/lib/omnizip/version.rb

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

data/lib/omnizip/zip/file.rb

@@ -50,7 +50,6 @@ module Omnizip
         @modified = false
         @reader = nil
         @writer = nil
-
         # Load existing archive if it exists
         if ::File.exist?(file_path) && !create
           load_archive
@@ -189,6 +188,18 @@ module Omnizip
       end
       alias_method :length, :size
 
+      # Mark this archive as having unsaved changes. The next +#commit+
+      # or +#close+ will write the archive to disk.
+      def modified!
+        @modified = true
+        self
+      end
+
+      # True if there are unsaved changes.
+      def modified?
+        @modified
+      end
+
       # Check if archive contains an entry
       # @param entry_name [String] Entry name to check
       # @return [Boolean] True if entry exists
@@ -374,8 +385,7 @@ module Omnizip
           extra_field: "",
           comment: "",
         ).tap do |header|
-          # Store original data in header for writing
-          header.instance_variable_set(:@_original_data, data)
+          header.cached_original_data = data
         end
       end
 
@@ -384,7 +394,7 @@ module Omnizip
         return "" if entry.directory?
 
         # Check if we have cached data (for new entries not yet written)
-        cached_data = entry.header.instance_variable_get(:@_original_data)
+        cached_data = entry.header.cached_original_data
         return cached_data if cached_data
 
         # Otherwise read from file
@@ -416,10 +426,11 @@ module Omnizip
           return "" unless compressed_data
 
           # Decompress
-          @reader.send(:decompress_data,
-                       compressed_data,
-                       reader_entry.compression_method,
-                       reader_entry.uncompressed_size)
+          @reader.decompress(
+            compressed_data,
+            reader_entry.compression_method,
+            reader_entry.uncompressed_size,
+          )
         end
       end
 
@@ -447,7 +458,7 @@ module Omnizip
           next if entry.directory?
 
           header = entry.header
-          cached = header.instance_variable_get(:@_original_data)
+          cached = header.cached_original_data
           entry_data[entry.name] = cached || read_entry_data(entry)
         end
 

data/lib/omnizip.rb

@@ -17,27 +17,30 @@
 #
 
 module Omnizip
-  # Core version - autoloaded
   autoload :VERSION, "omnizip/version"
 
-  # Error classes - autoloaded
   autoload :Error, "omnizip/error"
   autoload :CompressionError, "omnizip/error"
   autoload :DecompressionError, "omnizip/error"
   autoload :AlgorithmNotFoundError, "omnizip/error"
   autoload :UnknownAlgorithmError, "omnizip/error"
+  autoload :UnknownChecksumError, "omnizip/error"
+  autoload :UnknownFilterError, "omnizip/error"
+  autoload :UnknownEncryptionStrategyError, "omnizip/error"
+  autoload :ConversionNotSupportedError, "omnizip/error"
   autoload :UnsupportedFormatError, "omnizip/error"
   autoload :FormatError, "omnizip/error"
   autoload :InvalidArchiveError, "omnizip/error"
+  autoload :IOOperationError, "omnizip/error"
   autoload :IOError, "omnizip/error"
   autoload :ChecksumError, "omnizip/error"
-  autoload :OptimizationNotFound, "omnizip/error"
+  autoload :OptimizationNotFoundError, "omnizip/error"
   autoload :ProgressError, "omnizip/error"
   autoload :ETAError, "omnizip/error"
   autoload :NotLicensedError, "omnizip/error"
   autoload :RarNotAvailableError, "omnizip/error"
 
-  # Core registries
+  autoload :Registry, "omnizip/registry"
   autoload :Algorithm, "omnizip/algorithm"
   autoload :AlgorithmRegistry, "omnizip/algorithm_registry"
   autoload :FormatRegistry, "omnizip/format_registry"
@@ -45,11 +48,9 @@ module Omnizip
   autoload :ChecksumRegistry, "omnizip/checksum_registry"
   autoload :FilterRegistry, "omnizip/filter_registry"
 
-  # Base classes
   autoload :Filter, "omnizip/filter"
   autoload :FilterPipeline, "omnizip/filter_pipeline"
 
-  # Feature modules
   autoload :Buffer, "omnizip/buffer"
   autoload :Pipe, "omnizip/pipe"
   autoload :Chunked, "omnizip/chunked"
@@ -66,15 +67,25 @@ module Omnizip
   autoload :Platform, "omnizip/platform"
   autoload :Profiler, "omnizip/profiler"
   autoload :Commands, "omnizip/commands"
-
-  # Sub-module files
+  autoload :CliOutputFormatter, "omnizip/cli/output_formatter"
+
+  # Implementation namespaces. Each namespace file declares autoloads
+  # for its concrete classes. The classes self-register with their
+  # registry on load (see lib/omnizip/algorithms/lzma.rb etc.).
+  autoload :Algorithms, "omnizip/algorithms"
+  autoload :Filters, "omnizip/filters"
+  autoload :Checksums, "omnizip/checksums"
   autoload :IO, "omnizip/io"
   autoload :Crypto, "omnizip/crypto"
   autoload :Formats, "omnizip/formats"
   autoload :Zip, "omnizip/zip"
+  autoload :Extraction, "omnizip/extraction"
+  autoload :Implementations, "omnizip/implementations"
+  autoload :ArchiveHandler, "omnizip/archive_handler"
+  autoload :ArchiveHandlers, "omnizip/archive_handlers"
+  autoload :Convenience, "omnizip/convenience"
 end
 
-# Models module with autoloaded classes
 module Omnizip
   module Models
     autoload :AlgorithmMetadata, "omnizip/models/algorithm_metadata"
@@ -95,59 +106,15 @@ module Omnizip
   end
 end
 
-# Algorithms (with registration - must be required explicitly)
-require_relative "omnizip/algorithms/ppmd_base"
-require_relative "omnizip/algorithms/lzma"
-require_relative "omnizip/algorithms/lzma2"
-require_relative "omnizip/algorithms/ppmd7"
-require_relative "omnizip/algorithms/ppmd8"
-require_relative "omnizip/algorithms/bzip2"
-require_relative "omnizip/algorithms/deflate"
-require_relative "omnizip/algorithms/deflate64"
-require_relative "omnizip/algorithms/zstandard"
-
-# Filters implementations (with registration)
-require_relative "omnizip/filters/bcj"
-require_relative "omnizip/filters/bcj_x86"
-require_relative "omnizip/filters/bcj2"
-require_relative "omnizip/filters/bcj_arm"
-require_relative "omnizip/filters/bcj_arm64"
-require_relative "omnizip/filters/bcj_ppc"
-require_relative "omnizip/filters/bcj_sparc"
-require_relative "omnizip/filters/bcj_ia64"
-require_relative "omnizip/filters/delta"
-require_relative "omnizip/filters/registry"
-
-# Checksum implementations (with registration)
-require_relative "omnizip/checksums/crc_base"
-require_relative "omnizip/checksums/crc32"
-require_relative "omnizip/checksums/crc64"
-
-# Archive formats registry (with autoload declarations for lazy loading)
-require_relative "omnizip/formats"
-
-# Archive formats (with registration - must be required explicitly)
-require_relative "omnizip/formats/seven_zip"
-require_relative "omnizip/formats/zip"
-require_relative "omnizip/formats/rar"
-require_relative "omnizip/formats/tar"
-require_relative "omnizip/formats/gzip"
-require_relative "omnizip/formats/bzip2_file"
-require_relative "omnizip/formats/xz"
-
-# MSI format (must be loaded after OLE to override .msi registration)
-require_relative "omnizip/formats/msi"
-
-# Platform-specific features
 module Omnizip
   module Platform
     autoload :NtfsStreams, "omnizip/platform/ntfs_streams"
   end
 
-  # Implementations module with autoloaded classes
   module Implementations
     autoload :SevenZip, "omnizip/implementations/seven_zip"
     autoload :XZUtils, "omnizip/implementations/xz_utils"
+
     module SevenZip
       module LZMA
         autoload :StateMachine,
@@ -175,7 +142,8 @@ module Omnizip
   end
 end
 
-# Load convenience module to extend Omnizip with utility methods
-require "omnizip/convenience"
-
-# Formats auto-register when their files are loaded (see individual format files)
+# Trigger autoload of Convenience so its methods are available on the
+# Omnizip module (it calls `extend Convenience` at the bottom of the
+# file). Without this, Omnizip.compress_file and friends would not be
+# defined until the first manual reference.
+Omnizip::Convenience

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omnizip
 version: !ruby/object:Gem::Version
-  version: 0.3.9
+  version: 0.3.11
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-24 00:00:00.000000000 Z
+date: 2026-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base64
@@ -120,6 +120,21 @@ files:
 - LICENSE
 - README.adoc
 - Rakefile
+- TODO.refactor/00-overview.md
+- TODO.refactor/01-registry-base-class.md
+- TODO.refactor/02-error-hierarchy.md
+- TODO.refactor/03-autoload-entry-point.md
+- TODO.refactor/04-filter-base-consolidation.md
+- TODO.refactor/05-send-private-methods.md
+- TODO.refactor/06-instance-variable-access.md
+- TODO.refactor/07-respond-to-replacement.md
+- TODO.refactor/08-spec-doubles.md
+- TODO.refactor/09-cli-shared-module.md
+- TODO.refactor/10-format-detector-ocp.md
+- TODO.refactor/11-convenience-decoupling.md
+- TODO.refactor/12-thread-safety.md
+- TODO.refactor/13-lutaml-model-migration.md
+- TODO.refactor/14-add-missing-specs.md
 - benchmark/README.md
 - benchmark/benchmark_suite.rb
 - benchmark/compression_bench.rb
@@ -301,6 +316,10 @@ files:
 - lib/omnizip/algorithms/zstandard/literals.rb
 - lib/omnizip/algorithms/zstandard/literals_encoder.rb
 - lib/omnizip/algorithms/zstandard/sequences.rb
+- lib/omnizip/archive_handler.rb
+- lib/omnizip/archive_handlers.rb
+- lib/omnizip/archive_handlers/tar_handler.rb
+- lib/omnizip/archive_handlers/zip_handler.rb
 - lib/omnizip/buffer.rb
 - lib/omnizip/buffer/memory_archive.rb
 - lib/omnizip/buffer/memory_extractor.rb
@@ -317,6 +336,7 @@ 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
@@ -581,6 +601,7 @@ files:
 - lib/omnizip/io.rb
 - lib/omnizip/io/buffered_input.rb
 - lib/omnizip/io/buffered_output.rb
+- lib/omnizip/io/source.rb
 - lib/omnizip/io/stream_manager.rb
 - lib/omnizip/link_handler.rb
 - lib/omnizip/link_handler/hard_link.rb
@@ -668,6 +689,7 @@ files:
 - lib/omnizip/progress/progress_reporter.rb
 - lib/omnizip/progress/progress_tracker.rb
 - lib/omnizip/progress/silent_reporter.rb
+- lib/omnizip/registry.rb
 - lib/omnizip/rubyzip_compat.rb
 - lib/omnizip/temp.rb
 - lib/omnizip/temp/safe_extract.rb