cabriolet 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59d85958b00fa7eb684912e7ec77bfd9ce261a01035ed7ef42c2d6db5b1405a7
-  data.tar.gz: e9fa2123fe7c48778a01018c68f47cdb66f68449fe4fee48074e2b7591d5b9e1
+  metadata.gz: cb8a4d045d1bc49ab6448b9d1890524386e743d27249ad5dd5c5e630140e8a20
+  data.tar.gz: c63202e8fe947a0d14b3f4ffaca42feae67dc04e392ce3f9eac00d90ab0293b1
 SHA512:
-  metadata.gz: ddc9ef226ce8359cbe65ac4e80853dbd8e57f25c6e934e48837a163f614bcdbdf94f18609c788c6396f4b87f93d4e12c1fa326c8f291bc1156ec0eaf0387d377
-  data.tar.gz: 20206a2b5011d4a869d0f4d29085b5b874fe1d34daa00ce14f842d55a71722c19329ea75b1f46adaebf88fc43805a467b089e038aa19fe5736010d1a9ca951ad
+  metadata.gz: 6f6111054718818537222c48d986ef6a6112c89b9ea019ba748176402142316e7c2eef54979904e3bc7078f8d3b809193afcfe4acf971e9498d5cf4e7c4020dd
+  data.tar.gz: 8c0f0b712a10a5751f267cb67afe3c7d369ef6f06df0f961a2e92168f288c667fd97503e4fbfb95531898a8fec4303202e29b093814618c0065a7fbba8e3701f

data/lib/cabriolet/algorithm_factory.rb

@@ -91,7 +91,7 @@ module Cabriolet
     #   compressor = factory.create(3, :compressor,
     #                               io, input, output, 8192)
     def create(type, category, io_system, input, output, buffer_size,
-               **kwargs)
+               **)
       validate_category!(category)
 
       normalized_type = normalize_type(type)
@@ -103,7 +103,7 @@ module Cabriolet
       end
 
       algorithm_info[:class].new(io_system, input, output, buffer_size,
-                                 **kwargs)
+                                 **)
     end
 
     # Check if an algorithm is registered

data/lib/cabriolet/base_compressor.rb

@@ -57,8 +57,8 @@ module Cabriolet
     # @param options [Hash] Format-specific options
     # @return [FileEntry] Added entry
     # @raise [ArgumentError] if file doesn't exist
-    def add_file(source_path, archive_path = nil, **options)
-      @file_manager.add_file(source_path, archive_path, **options)
+    def add_file(source_path, archive_path = nil, **)
+      @file_manager.add_file(source_path, archive_path, **)
     end
 
     # Add file from memory to archive
@@ -67,8 +67,8 @@ module Cabriolet
     # @param archive_path [String] Path in archive
     # @param options [Hash] Format-specific options
     # @return [FileEntry] Added entry
-    def add_data(data, archive_path, **options)
-      @file_manager.add_data(data, archive_path, **options)
+    def add_data(data, archive_path, **)
+      @file_manager.add_data(data, archive_path, **)
     end
 
     # Generate archive (Template Method)
@@ -165,7 +165,7 @@ module Cabriolet
     # @option options [Integer] :window_bits Window size in bits
     # @option options [Integer] :mode Algorithm mode
     # @return [String] Compressed data
-    def compress_data(data, algorithm:, **options)
+    def compress_data(data, algorithm:, **)
       input = System::MemoryHandle.new(data)
       output = System::MemoryHandle.new("", Constants::MODE_WRITE)
 
@@ -176,7 +176,7 @@ module Cabriolet
         input,
         output,
         data.bytesize,
-        **options,
+        **,
       )
 
       compressor.compress

data/lib/cabriolet/cab/decompressor.rb

@@ -36,9 +36,9 @@ module Cabriolet
       # @param output_path [String] Where to write the file
       # @param options [Hash] Extraction options
       # @return [Integer] Number of bytes extracted
-      def extract_file(file, output_path, **options)
+      def extract_file(file, output_path, **)
         extractor = Extractor.new(@io_system, self)
-        extractor.extract_file(file, output_path, **options)
+        extractor.extract_file(file, output_path, **)
       end
 
       # Extract all files from the cabinet
@@ -47,9 +47,9 @@ module Cabriolet
       # @param output_dir [String] Directory to extract to
       # @param options [Hash] Extraction options
       # @return [Integer] Number of files extracted
-      def extract_all(cabinet, output_dir, **options)
+      def extract_all(cabinet, output_dir, **)
         extractor = Extractor.new(@io_system, self)
-        extractor.extract_all(cabinet, output_dir, **options)
+        extractor.extract_all(cabinet, output_dir, **)
       end
 
       # Create appropriate decompressor for a folder

data/lib/cabriolet/cab/extractor.rb

@@ -60,7 +60,8 @@ module Cabriolet
         begin
           write_file_data(output_fh, filelen)
         rescue DecompressionError
-          handle_extraction_error(output_fh, output_path, file.filename, salvage, filelen)
+          handle_extraction_error(output_fh, output_path, file.filename,
+                                  salvage, filelen)
         ensure
           output_fh.close
         end
@@ -72,6 +73,7 @@ module Cabriolet
       def reset_state
         @current_input&.close
         @current_input = nil
+        @current_decomp&.free # Free decompressor buffers to prevent memory leaks
         @current_decomp = nil
         @current_folder = nil
         @current_offset = 0
@@ -136,6 +138,9 @@ module Cabriolet
         warn "Salvage: #{failed_count} file(s) skipped due to extraction errors" if failed_count.positive?
 
         count
+      ensure
+        # Clean up resources to prevent memory leaks
+        reset_state
       end
 
       private
@@ -264,7 +269,8 @@ module Cabriolet
       # @param filelen [Integer] Number of bytes to write
       def write_file_data(output_fh, filelen)
         unless @current_decomp
-          raise DecompressionError, "Decompressor not available (state was reset)"
+          raise DecompressionError,
+                "Decompressor not available (state was reset)"
         end
 
         @current_decomp.instance_variable_set(:@output, output_fh)
@@ -279,7 +285,8 @@ module Cabriolet
       # @param filename [String] Filename for error messages
       # @param salvage [Boolean] Salvage mode flag
       # @raise [DecompressionError] If not in salvage mode
-      def handle_extraction_error(output_fh, output_path, filename, salvage, _filelen)
+      def handle_extraction_error(output_fh, output_path, filename, salvage,
+_filelen)
         output_fh.close
         if salvage
           ::File.write(output_path, "", mode: "wb")

data/lib/cabriolet/cab/parser.rb

@@ -23,6 +23,9 @@ module Cabriolet
         cabinet = parse_handle(handle, filename)
         @io_system.close(handle)
         cabinet
+      rescue StandardError
+        @io_system.close(handle) if handle
+        raise
       end
 
       # Parse a CAB from an already-open handle

data/lib/cabriolet/cli.rb

@@ -422,11 +422,11 @@ module Cabriolet
     # @param command [Symbol] Command to execute
     # @param file [String] File path
     # @param args [Array] Additional arguments
-    def run_dispatcher(command, file, *args, **options)
+    def run_dispatcher(command, file, *, **options)
       setup_verbose(options[:verbose])
 
       dispatcher = Commands::CommandDispatcher.new(**options)
-      dispatcher.dispatch(command, file, *args, **options)
+      dispatcher.dispatch(command, file, *, **options)
     end
 
     # Run command with explicit format override
@@ -435,12 +435,12 @@ module Cabriolet
     # @param format [Symbol] Format to force
     # @param file [String] File path
     # @param args [Array] Additional arguments
-    def run_with_format(command, format, file, *args, **options)
+    def run_with_format(command, format, file, *, **options)
       setup_verbose(options[:verbose])
       options[:format] = format.to_s
 
       dispatcher = Commands::CommandDispatcher.new(**options)
-      dispatcher.dispatch(command, file, *args, **options)
+      dispatcher.dispatch(command, file, *, **options)
     end
 
     # Detect format from output file extension

data/lib/cabriolet/decompressors/lzx.rb

@@ -178,6 +178,23 @@ module Cabriolet
         @output_length = length if length.positive?
       end
 
+      # Free resources used by the decompressor
+      #
+      # Releases large memory buffers to prevent memory leaks when
+      # the decompressor is no longer needed.
+      #
+      # @return [void]
+      def free
+        @window = nil
+        @e8_buf = nil
+        @pending_frame_data = nil
+        @bitstream = nil
+        @maintree_lengths = nil
+        @length_lengths = nil
+        @pretree_lengths = nil
+        @aligned_lengths = nil
+      end
+
       # Decompress LZX data
       #
       # Per libmspack lzxd.c: the decompressor always decodes full frames
@@ -201,7 +218,9 @@ module Cabriolet
         if @pending_frame_data
           avail = @pending_frame_data.bytesize - @pending_frame_offset
           write_amount = [bytes, avail].min
-          io_system.write(output, @pending_frame_data[@pending_frame_offset, write_amount])
+          io_system.write(output,
+                          @pending_frame_data[@pending_frame_offset,
+                                              write_amount])
           total_written += write_amount
           @offset += write_amount
           @pending_frame_offset += write_amount

data/lib/cabriolet/decompressors/mszip.rb

@@ -83,6 +83,21 @@ salvage: false, **_kwargs)
         @debug_mszip_symbols = ENV.fetch("DEBUG_MSZIP_SYMBOLS", nil)
       end
 
+      # Free resources used by the decompressor
+      #
+      # Releases memory buffers to prevent memory leaks when
+      # the decompressor is no longer needed.
+      #
+      # @return [void]
+      def free
+        @window = nil
+        @bitstream = nil
+        @literal_lengths = nil
+        @distance_lengths = nil
+        @literal_tree = nil
+        @distance_tree = nil
+      end
+
       # Decompress MSZIP data
       #
       # @param bytes [Integer] Number of bytes to decompress

data/lib/cabriolet/decompressors/quantum.rb

@@ -2,30 +2,16 @@
 
 require_relative "../quantum_shared"
 
-# Compatibility shim for String#bytesplice (added in Ruby 3.2)
-unless String.method_defined?(:bytesplice)
-  module StringBytespliceCompat
-    # Compatibility implementation of bytesplice for Ruby < 3.2
-    # Uses clear/append which is slower but works with mutable strings
-    def bytesplice(index, length, other_string, other_index = 0,
-other_length = nil)
-      other_length ||= other_string.bytesize
-
-      # Build new string content
-      prefix = byteslice(0, index)
-      middle = other_string.byteslice(other_index, other_length)
-      suffix = byteslice((index + length)..-1)
-      new_content = prefix + middle + suffix
-
-      # Modify receiver in place
-      clear
-      self << new_content
-
-      self
+# Helper for 5-argument bytesplice (added in Ruby 3.3)
+# Ruby 3.2 has bytesplice but only 2-3 argument forms
+unless String.method_defined?(:window_splice)
+  class String
+    # Copy bytes from source string into self at specified position
+    # Works on all Ruby versions including 3.2 which lacks 5-arg bytesplice
+    def window_splice(dest_idx, dest_len, src, src_idx, src_len)
+      self[dest_idx, dest_len] = src.byteslice(src_idx, src_len)
     end
   end
-
-  String.prepend(StringBytespliceCompat)
 end
 
 module Cabriolet
@@ -61,11 +47,7 @@ module Cabriolet
         @window_size = 1 << window_bits
 
         # Initialize window (must be binary to avoid UTF-8 character vs byte mismatch)
-        @window = if String.method_defined?(:bytesplice)
-                    ("\0" * @window_size).b
-                  else
-                    String.new("\0" * @window_size, encoding: Encoding::BINARY)
-                  end
+        @window = ("\0" * @window_size).b
         @window_posn = 0
         @frame_todo = FRAME_SIZE
 
@@ -82,6 +64,21 @@ module Cabriolet
         initialize_models
       end
 
+      # Free resources used by the decompressor
+      #
+      # Releases memory buffers to prevent memory leaks when
+      # the decompressor is no longer needed.
+      #
+      # @return [void]
+      def free
+        @window = nil
+        @bitstream = nil
+        @m0sym = @m1sym = @m2sym = @m3sym = nil
+        @m4sym = @m5sym = @m6sym = @m6lsym = nil
+        @model0 = @model1 = @model2 = @model3 = nil
+        @model4 = @model5 = @model6 = @model6len = nil
+      end
+
       # Decompress Quantum data
       #
       # @param bytes [Integer] Number of bytes to decompress
@@ -402,7 +399,7 @@ module Cabriolet
         end
       end
 
-      # Bulk copy using bytesplice for better performance on longer matches
+      # Bulk copy using window_splice for better performance on longer matches
       def copy_match_bulk(offset, length)
         if offset > @window_posn
           # Match wraps around window
@@ -417,21 +414,23 @@ module Cabriolet
 
           if copy_len < length
             # Copy from end, then from beginning
-            @window.bytesplice(@window_posn, copy_len, @window, src_pos,
-                               copy_len)
+            @window.window_splice(@window_posn, copy_len, @window, src_pos,
+                                  copy_len)
             @window_posn += copy_len
             remaining = length - copy_len
-            @window.bytesplice(@window_posn, remaining, @window, 0, remaining)
+            @window.window_splice(@window_posn, remaining, @window, 0,
+                                  remaining)
             @window_posn += remaining
           else
             # Copy entirely from end
-            @window.bytesplice(@window_posn, length, @window, src_pos, length)
+            @window.window_splice(@window_posn, length, @window, src_pos,
+                                  length)
             @window_posn += length
           end
         else
-          # Normal copy - use bytesplice for bulk operation
+          # Normal copy - use window_splice for bulk operation
           src_pos = @window_posn - offset
-          @window.bytesplice(@window_posn, length, @window, src_pos, length)
+          @window.window_splice(@window_posn, length, @window, src_pos, length)
           @window_posn += length
         end
       end

data/lib/cabriolet/file_manager.rb

@@ -29,13 +29,13 @@ module Cabriolet
     # @param options [Hash] Format-specific options
     # @return [FileEntry] Added entry
     # @raise [ArgumentError] if file doesn't exist
-    def add_file(source_path, archive_path = nil, **options)
+    def add_file(source_path, archive_path = nil, **)
       archive_path ||= File.basename(source_path)
 
       entry = FileEntry.new(
         source: source_path,
         archive_path: archive_path,
-        **options,
+        **,
       )
 
       @entries << entry
@@ -48,11 +48,11 @@ module Cabriolet
     # @param archive_path [String] Path in archive
     # @param options [Hash] Format-specific options
     # @return [FileEntry] Added entry
-    def add_data(data, archive_path, **options)
+    def add_data(data, archive_path, **)
       entry = FileEntry.new(
         data: data,
         archive_path: archive_path,
-        **options,
+        **,
       )
 
       @entries << entry

data/lib/cabriolet/format_base.rb

@@ -44,7 +44,7 @@ module Cabriolet
     # @param size [Integer] Data size
     # @param options [Hash] Additional options for the compressor
     # @return [Object] Compressor instance
-    def create_compressor(algorithm, input, output, size, **options)
+    def create_compressor(algorithm, input, output, size, **)
       @algorithm_factory.create(
         algorithm,
         :compressor,
@@ -52,7 +52,7 @@ module Cabriolet
         input,
         output,
         size,
-        **options,
+        **,
       )
     end
 
@@ -64,7 +64,7 @@ module Cabriolet
     # @param size [Integer] Data size
     # @param options [Hash] Additional options for the decompressor
     # @return [Object] Decompressor instance
-    def create_decompressor(algorithm, input, output, size, **options)
+    def create_decompressor(algorithm, input, output, size, **)
       @algorithm_factory.create(
         algorithm,
         :decompressor,
@@ -72,7 +72,7 @@ module Cabriolet
         input,
         output,
         size,
-        **options,
+        **,
       )
     end
   end

data/lib/cabriolet/hlp/compressor.rb

@@ -45,8 +45,8 @@ module Cabriolet
       # @param output_file [String] Output file path
       # @param options [Hash] Format options
       # @return [Integer] Bytes written
-      def generate(output_file, **options)
-        @quickhelp.generate(output_file, **options)
+      def generate(output_file, **)
+        @quickhelp.generate(output_file, **)
       end
 
       # Create a Windows Help format HLP file

data/lib/cabriolet/plugin.rb

@@ -190,10 +190,10 @@ module Cabriolet
     # @example Register a format-specific decompressor
     #   register_algorithm(:special, SpecialDecompressor,
     #                     category: :decompressor, format: :cab)
-    def register_algorithm(type, klass, **options)
+    def register_algorithm(type, klass, **)
       raise PluginError, "Plugin manager not available" unless @manager
 
-      Cabriolet.algorithm_factory.register(type, klass, **options)
+      Cabriolet.algorithm_factory.register(type, klass, **)
     end
 
     # Register a format handler

data/lib/cabriolet/streaming.rb

@@ -188,9 +188,9 @@ module Cabriolet
       # @param paths [Array<String>] Array of archive paths
       # @yield [file, archive_path] Yields each file with its archive path
       # @return [Hash] Processing statistics
-      def process_archives(paths, &block)
+      def process_archives(paths, &)
         paths.each do |path|
-          process_archive(path, &block)
+          process_archive(path, &)
         end
 
         @stats

data/lib/cabriolet/validator.rb

@@ -324,9 +324,9 @@ module Cabriolet
       }
     end
 
-    def to_json(*args)
+    def to_json(*)
       require "json"
-      to_h.to_json(*args)
+      to_h.to_json(*)
     end
   end
 end

data/lib/cabriolet/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Cabriolet
-  VERSION = "0.2.3"
+  VERSION = "0.2.4"
 end

data/lib/cabriolet.rb

@@ -169,7 +169,7 @@ module Cabriolet
     # @example
     #   archive = Cabriolet.open('unknown.archive')
     #   archive.files.each { |f| puts f.name }
-    def open(path, **options)
+    def open(path, **)
       parser_class = FormatDetector.parser_for(path)
 
       unless parser_class
@@ -178,7 +178,7 @@ module Cabriolet
               "Unable to detect format or no parser available for: #{path} (detected: #{format || 'unknown'})"
       end
 
-      parser_class.new(**options).parse(path)
+      parser_class.new(**).parse(path)
     end
 
     # Detect format of an archive file
@@ -209,9 +209,9 @@ module Cabriolet
     # @example Parallel extraction with 8 workers
     #   stats = Cabriolet.extract('file.chm', 'docs/', workers: 8)
     #   puts "Extracted #{stats[:extracted]} files"
-    def extract(archive_path, output_dir, **options)
+    def extract(archive_path, output_dir, **)
       archive = open(archive_path)
-      extractor = Extraction::Extractor.new(archive, output_dir, **options)
+      extractor = Extraction::Extractor.new(archive, output_dir, **)
       extractor.extract_all
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cabriolet
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-03-06 00:00:00.000000000 Z
+date: 2026-03-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bindata