omnizip 0.3.7 → 0.3.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77e13c00351fadfa082ef58c3053379870550819c1f0b1ca8c51f6de35be21ad
-  data.tar.gz: b18a4614b01c6d0cd974103f7eb5aba464caf0ff467d4a108eacc5744dc44382
+  metadata.gz: eb35ab3f87d9e1a1d4f96c65a86b9c5dcbd5df0f4da48443158dbe16d8e896e0
+  data.tar.gz: f9c8deab498097486bb07092f479b5097959cbc7f6b6ca171bc79996b08b6b91
 SHA512:
-  metadata.gz: 16b86cdedfe871db80695c62dad8fb63236aeb43662e0064ad76115986887ae5fe30bba1a827d5ef2f9f93ab6746c821f4a2fae2ea188dd1bb3425c3460f54e8
-  data.tar.gz: '088baddfb374607a12572d440662c9406aca7254e50a9ce3e8a2965135c56fbdde617b054ff2e0f7ccade0ba9916f71b594dce17e8add271f77487fdb9f4a185'
+  metadata.gz: b518a237a3f41aac1d2e30d88cbc0e872320141d7f705d2bc33db3a2fbe22c5f784b8219c88001c54706571d07a3aa858f068c249e1da20efeb7c2afaca2f234
+  data.tar.gz: a18702ca45df7e9b6176b397c33d0e852a3a8a3880f58be16567aa13be7472c52b4774595a7fe1b90f3ef3929bab29d70e18ea22e4184add6a21b2c38c501909

data/.rubocop_todo.yml

@@ -1,25 +1,23 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2026-02-23 11:56:10 UTC using RuboCop version 1.81.6.
+# on 2026-03-19 08:28:00 UTC using RuboCop version 1.85.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 69
+# Offense count: 1
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'omnizip.gemspec'
+
+# Offense count: 5
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: EnforcedStyle, IndentationWidth.
 # SupportedStyles: with_first_argument, with_fixed_indentation
 Layout/ArgumentAlignment:
-  Enabled: false
-
-# Offense count: 1
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_element, with_fixed_indentation
-Layout/ArrayAlignment:
   Exclude:
-    - 'lib/omnizip/implementations/seven_zip/lzma/decoder.rb'
+    - 'spec/omnizip/formats/seven_zip/embedded_search_spec.rb'
 
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
@@ -30,31 +28,19 @@ Layout/BlockAlignment:
     - 'lib/omnizip/algorithms/lzma/lzip_decoder.rb'
     - 'lib/omnizip/formats/xz_impl/block_decoder.rb'
 
-# Offense count: 10
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EmptyLineBetweenMethodDefs, EmptyLineBetweenClassDefs, EmptyLineBetweenModuleDefs, DefLikeMacros, AllowAdjacentOneLineDefs, NumberOfEmptyLines.
-Layout/EmptyLineBetweenDefs:
-  Exclude:
-    - 'lib/omnizip.rb'
-    - 'lib/omnizip/formats/rar.rb'
-
-# Offense count: 17
+# Offense count: 1485
 # This cop supports safe autocorrection (--autocorrect).
-Layout/EmptyLines:
-  Enabled: false
-
-# Offense count: 1831
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: Max, AllowHeredoc, AllowURI, AllowQualifiedName, URISchemes, IgnoreCopDirectives, AllowedPatterns, SplitStrings.
+# Configuration parameters: Max, AllowHeredoc, AllowURI, AllowQualifiedName, URISchemes, AllowRBSInlineAnnotation, AllowCopDirectives, AllowedPatterns, SplitStrings.
 # URISchemes: http, https
 Layout/LineLength:
   Enabled: false
 
-# Offense count: 69
+# Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowInHeredoc.
 Layout/TrailingWhitespace:
-  Enabled: false
+  Exclude:
+    - 'spec/omnizip/formats/seven_zip/embedded_search_spec.rb'
 
 # Offense count: 13
 # Configuration parameters: AllowedMethods.
@@ -71,15 +57,16 @@ Lint/ConstantDefinitionInBlock:
     - 'spec/omnizip/seven_zip_tool_compatibility_spec.rb'
     - 'spec/omnizip/xz_tool_compatibility_spec.rb'
 
-# Offense count: 34
+# Offense count: 36
 # Configuration parameters: IgnoreLiteralBranches, IgnoreConstantBranches, IgnoreDuplicateElseBranch.
 Lint/DuplicateBranch:
   Enabled: false
 
-# Offense count: 8
+# Offense count: 9
 Lint/DuplicateMethods:
   Exclude:
     - 'lib/omnizip/algorithms/lzma/xz_range_encoder_exact.rb'
+    - 'lib/omnizip/formats/msi/reader.rb'
     - 'lib/omnizip/formats/xz.rb'
     - 'lib/omnizip/implementations/xz_utils/lzma2/encoder.rb'
     - 'lib/omnizip/models/filter_config.rb'
@@ -127,13 +114,6 @@ Lint/StructNewOverride:
     - 'lib/omnizip/parallel/job_queue.rb'
     - 'lib/omnizip/parity/par2_creator.rb'
 
-# Offense count: 1
-# Configuration parameters: AllowedPatterns.
-# AllowedPatterns: (?-mix:(exactly|at_least|at_most)\(\d+\)\.times)
-Lint/UnreachableLoop:
-  Exclude:
-    - 'lib/omnizip/algorithms/lzma/range_decoder.rb'
-
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowUnusedKeywordArguments, IgnoreEmptyMethods, IgnoreNotImplementedMethods, NotImplementedExceptions.
@@ -148,38 +128,38 @@ Lint/UselessConstantScoping:
   Exclude:
     - 'lib/omnizip/algorithms/lzma/optimal_encoder.rb'
 
-# Offense count: 552
+# Offense count: 553
 # Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes, Max.
 Metrics/AbcSize:
   Enabled: false
 
-# Offense count: 18
+# Offense count: 20
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 # AllowedMethods: refine
 Metrics/BlockLength:
-  Max: 206
+  Max: 81
 
 # Offense count: 5
 # Configuration parameters: CountBlocks, CountModifierForms.
 Metrics/BlockNesting:
   Max: 6
 
-# Offense count: 266
+# Offense count: 263
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/CyclomaticComplexity:
   Enabled: false
 
-# Offense count: 948
+# Offense count: 961
 # Configuration parameters: CountComments, CountAsOne, AllowedMethods, AllowedPatterns.
 Metrics/MethodLength:
-  Max: 434
+  Max: 238
 
 # Offense count: 33
 # Configuration parameters: CountKeywordArgs, MaxOptionalParameters.
 Metrics/ParameterLists:
   Max: 20
 
-# Offense count: 187
+# Offense count: 182
 # Configuration parameters: AllowedMethods, AllowedPatterns, Max.
 Metrics/PerceivedComplexity:
   Enabled: false
@@ -193,22 +173,19 @@ Naming/MethodParameterName:
 # Offense count: 38
 # Configuration parameters: Mode, AllowedMethods, AllowedPatterns, AllowBangMethods, WaywardPredicates.
 # AllowedMethods: call
-# WaywardPredicates: nonzero?
+# WaywardPredicates: infinite?, nonzero?
 Naming/PredicateMethod:
   Enabled: false
 
-# Offense count: 42
+# Offense count: 37
 # Configuration parameters: EnforcedStyle, CheckMethodNames, CheckSymbols, AllowedIdentifiers, AllowedPatterns.
 # SupportedStyles: snake_case, normalcase, non_integer
 # AllowedIdentifiers: TLS1_1, TLS1_2, capture3, iso8601, rfc1123_date, rfc822, rfc2822, rfc3339, x86_64
 Naming/VariableNumber:
   Exclude:
     - 'lib/omnizip/algorithms/lzma/bit_model.rb'
-    - 'lib/omnizip/algorithms/lzma/distance_coder.rb'
-    - 'lib/omnizip/algorithms/lzma/literal_decoder.rb'
     - 'lib/omnizip/algorithms/lzma/range_encoder.rb'
     - 'lib/omnizip/algorithms/lzma/xz_buffered_range_encoder.rb'
-    - 'lib/omnizip/algorithms/lzma/xz_utils_decoder.rb'
     - 'lib/omnizip/formats/rar/rar5/compression/lzss.rb'
     - 'lib/omnizip/formats/xz_impl/block_decoder.rb'
     - 'lib/omnizip/implementations/seven_zip/lzma/range_encoder.rb'
@@ -218,13 +195,14 @@ Naming/VariableNumber:
     - 'spec/omnizip/parity/reed_solomon_encoder_spec.rb'
     - 'spec/omnizip/parity/reed_solomon_matrix_spec.rb'
 
-# Offense count: 8
+# Offense count: 9
 # Configuration parameters: MinSize.
 Performance/CollectionLiteralInLoop:
   Exclude:
     - 'lib/omnizip/commands/archive_create_command.rb'
     - 'lib/omnizip/formats/iso/directory_builder.rb'
     - 'lib/omnizip/formats/iso/writer.rb'
+    - 'lib/omnizip/formats/msi/constants.rb'
     - 'lib/omnizip/formats/seven_zip/header_writer.rb'
     - 'lib/omnizip/parity/models/file_description_packet.rb'
     - 'lib/omnizip/parity/par2_repairer.rb'
@@ -235,12 +213,14 @@ Performance/MapMethodChain:
   Exclude:
     - 'lib/omnizip/commands/archive_create_command.rb'
 
-# Offense count: 7
+# Offense count: 11
 Security/Open:
   Exclude:
     - 'lib/omnizip/formats/cpio.rb'
     - 'lib/omnizip/formats/iso.rb'
+    - 'lib/omnizip/formats/msi.rb'
     - 'lib/omnizip/zip/file.rb'
+    - 'lib/tasks/msi_test_files.rake'
 
 # Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
@@ -260,6 +240,20 @@ Style/EmptyElse:
     - 'lib/omnizip/algorithms/ppmd7/encoder.rb'
     - 'lib/omnizip/formats/iso/volume_descriptor.rb'
 
+# Offense count: 11
+Style/FileOpen:
+  Exclude:
+    - 'lib/omnizip/chunked/writer.rb'
+    - 'lib/omnizip/formats/iso/reader.rb'
+    - 'lib/omnizip/formats/rar/rar5/multi_volume/volume_writer.rb'
+    - 'lib/omnizip/formats/rpm.rb'
+    - 'lib/omnizip/formats/seven_zip/split_archive_writer.rb'
+    - 'lib/omnizip/formats/tar/writer.rb'
+    - 'lib/omnizip/formats/xar/reader.rb'
+    - 'lib/omnizip/zip/input_stream.rb'
+    - 'lib/omnizip/zip/output_stream.rb'
+    - 'spec/omnizip/formats/rpm_spec.rb'
+
 # Offense count: 161
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: MaxUnannotatedPlaceholdersAllowed, Mode, AllowedMethods, AllowedPatterns.
@@ -273,6 +267,13 @@ Style/MixinUsage:
     - 'spec/scripts/debug_7z_structure.rb'
     - 'spec/scripts/debug_writer_output.rb'
 
+# Offense count: 3
+# Configuration parameters: AllowedClasses.
+Style/OneClassPerFile:
+  Exclude:
+    - 'lib/omnizip.rb'
+    - 'lib/omnizip/algorithms/lzma2.rb'
+
 # Offense count: 3
 # Configuration parameters: AllowedMethods.
 # AllowedMethods: respond_to_missing?

data/Gemfile

@@ -5,6 +5,8 @@ source "https://rubygems.org"
 # Specify your gem's dependencies in omnizip.gemspec
 gemspec
 
+# For spec testing
+gem "csv"
 gem "rake"
 gem "rspec"
 gem "rubocop"

data/Rakefile

@@ -9,4 +9,7 @@ require "rubocop/rake_task"
 
 RuboCop::RakeTask.new
 
+# Load custom rake tasks
+Dir["lib/tasks/*.rake"].each { |task| load task }
+
 task default: %i[spec rubocop]

data/lib/omnizip/algorithms/lzma/bit_model.rb

@@ -37,7 +37,7 @@ module Omnizip
         MAX_PROB = 1 << 11 # 4096
         BIT_MODEL_TOTAL = 0x800 # XZ Utils RC_BIT_MODEL_TOTAL = 2048
 
-        attr_reader :probability
+        attr_accessor :probability
 
         # Initialize a new bit probability model
         #

data/lib/omnizip/algorithms/lzma/distance_coder.rb

@@ -49,6 +49,8 @@ module Omnizip
           end
 
           # Position encoders for slots 4-13
+          # SDK convention: size = NUM_FULL_DISTANCES - END_POS_MODEL_INDEX = 128 - 14 = 114
+          # Indexed by: base - slot - 1 (matches LZMA SDK and XZ Utils)
           @pos_encoders = Array.new(NUM_FULL_DISTANCES - END_POS_MODEL_INDEX) do
             BitModel.new
           end
@@ -71,19 +73,11 @@ module Omnizip
         #
         # @return [void]
         def reset_models
-          if ENV.fetch("DEBUG_RESET_MODELS",
-                       nil) && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-            puts "    [DistanceCoder.reset_models] Resetting #{@slot_encoders.size} len_states, each with #{@slot_encoders[0]&.size || '?'} models"
-          end
           @slot_encoders.each do |len_state_models|
             len_state_models.each(&:reset)
           end
           @pos_encoders.each(&:reset)
           @align_encoder.each(&:reset)
-          if ENV.fetch("DEBUG_RESET_MODELS",
-                       nil) && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-            puts "    [DistanceCoder.reset_models] Done resetting"
-          end
         end
 
         # Encode a match distance using SDK-compatible encoding
@@ -95,11 +89,6 @@ module Omnizip
         def encode(range_encoder, distance, len_state)
           slot = get_dist_slot(distance)
 
-          if ENV["LZMA_DEBUG_ENCODE"]
-            puts "[DistanceCoder.encode] distance=#{distance} slot=#{slot} len_state=#{len_state}"
-            puts "[DistanceCoder.encode] CALLING encode_tree with symbol=#{slot}"
-          end
-
           # Encode the slot using the appropriate slot encoder
           encode_tree(range_encoder, @slot_encoders[len_state], slot,
                       NUM_DIST_SLOT_BITS)
@@ -111,6 +100,7 @@ module Omnizip
 
             if slot < END_POS_MODEL_INDEX
               # Slots 4-13: Use position encoders (reverse tree encoding)
+              # SDK/XZ Utils convention: base_idx = base - slot - 1
               encode_reverse_tree(range_encoder,
                                   @pos_encoders,
                                   base - slot - 1,
@@ -138,61 +128,24 @@ module Omnizip
         # @param len_state [Integer] Length state for slot selection
         # @return [Integer] Decoded distance value (before adding 1)
         def decode(range_decoder, len_state)
-          # DEBUG: Trace specific calls to find corruption
-          $distance_decode_count ||= 0
-          debug_calls = (320..330)
-          debug_this = debug_calls.include?($distance_decode_count)
-          trace_326 = ($distance_decode_count == 326)
-          trace_325 = ($distance_decode_count == 325)
-
-          # DEBUG: Trace large distances (> 100000)
-          trace_large = $distance_decode_count.between?(25,
-                                                        35) || $distance_decode_count.between?(
-                                                          315, 330
-                                                        )
-
-          # DEBUG: Trace all when LZMA_DEBUG_DISTANCE is set
-          trace_all = ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-
-          if (trace_325 || trace_large || trace_all) && ENV.fetch(
-            "LZMA_DEBUG_DISTANCE", nil
-          )
-            puts "  [DistanceCoder.decode ##{$distance_decode_count}] START - len_state=#{len_state}"
-            puts "    BEFORE: range=#{range_decoder.range.inspect}, code=#{range_decoder.code.inspect}"
-          end
-
           slot = decode_tree(range_decoder, @slot_encoders[len_state],
                              NUM_DIST_SLOT_BITS)
 
-          if (debug_this || trace_large || trace_all) && ENV.fetch(
-            "LZMA_DEBUG_DISTANCE", nil
-          )
-            puts "  [DistanceCoder.decode ##{$distance_decode_count}] len_state=#{len_state}, slot=#{slot}"
-            puts "    @slot_encoders[#{len_state}] object_id=#{@slot_encoders[len_state].object_id}"
-          end
-
           # Decode extra bits based on slot
           if slot < START_POS_MODEL_INDEX
             # Slots 0-3: No extra bits
-            $distance_decode_count += 1
-            if debug_this && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-              puts "    -> distance=#{slot}"
-            end
             slot
           else
             footer_bits = (slot >> 1) - 1
 
             if slot < END_POS_MODEL_INDEX
               # Slots 4-13: Use position encoders (reverse tree decoding)
+              # SDK/XZ Utils convention: base_idx = base - slot - 1
               base = (2 | (slot & 1)) << footer_bits
               result = base + decode_reverse_tree(range_decoder,
                                                   @pos_encoders,
                                                   base - slot - 1,
                                                   footer_bits)
-              $distance_decode_count += 1
-              if debug_this && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-                puts "    -> distance=#{result} (slot #{slot})"
-              end
             else
               # Slots 14+: Fixed direct bits + aligned bits
               # XZ Utils pattern (lzma_decoder.c:500-514):
@@ -225,23 +178,10 @@ module Omnizip
                                              @align_encoder,
                                              0,
                                              DIST_ALIGN_BITS)
-              if trace_326 && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-                puts "    TRACE_326: low_bits=#{low_bits}"
-              end
 
               # Final result: (result << 4) + low_bits
               # NOTE: slot value is NOT added (XZ Utils pattern - line 513 adds symbol for EOPM check only)
               result = (result << DIST_ALIGN_BITS) + low_bits
-              $distance_decode_count += 1
-              if (debug_this || trace_large) && ENV.fetch(
-                "LZMA_DEBUG_DISTANCE", nil
-              )
-                puts "    -> slot=#{slot}, result_after_direct=#{result >> DIST_ALIGN_BITS}, low_bits=#{low_bits}, distance=#{result}"
-              end
-              if result > 100000
-                puts "    [LARGE_DISTANCE ##{$distance_decode_count}] distance=#{result}, slot=#{slot}" if ENV["LZMA_DEBUG_DISTANCE"]
-                puts "      BEFORE: range_decoder.range=#{range_decoder.range}, range_decoder.code=#{range_decoder.code}" if ENV["LZMA_DEBUG_DISTANCE"]
-              end
             end
             result
           end
@@ -321,29 +261,11 @@ module Omnizip
         # @return [void]
         def encode_tree(range_encoder, models, symbol, num_bits)
           m = 1
-          trace_all = ENV.fetch("TRACE_ALL_SLOT_ENCODE", nil)
-          iteration = 0
-
-          if trace_all && ENV.fetch("LZMA_DEBUG_ENCODE", nil)
-            puts "    [encode_tree START] RECEIVED symbol=#{symbol}, num_bits=#{num_bits}"
-            puts "      BEFORE: range=#{range_encoder.range}, low=#{range_encoder.low}"
-          end
-
           (num_bits - 1).downto(0) do |i|
-            iteration += 1
             bit = (symbol >> i) & 1
-            if trace_all && ENV.fetch("LZMA_DEBUG_ENCODE", nil)
-              model_idx = m
-              puts "      [#{iteration}/#{num_bits}] i=#{i}, bit=#{bit}, m=#{m}, model_idx=#{model_idx}, prob=#{models[m].probability}"
-            end
             range_encoder.encode_bit(models[m], bit)
             m = (m << 1) | bit
           end
-
-          if trace_all && ENV.fetch("LZMA_DEBUG_ENCODE", nil)
-            puts "      AFTER: range=#{range_encoder.range}, low=#{range_encoder.low}"
-            puts "    [encode_tree END] ENCODED symbol=#{symbol}"
-          end
         end
 
         # Decode a value using a tree of bit models
@@ -355,29 +277,10 @@ module Omnizip
         def decode_tree(range_decoder, models, num_bits)
           m = 1
           symbol = 0
-          trace_this = (num_bits == 6 && ENV.fetch("TRACE_SLOT_DECODE",
-                                                   nil)) || ($distance_decode_count == 28)
-          trace_all = ENV.fetch("TRACE_ALL_SLOT_DECODE", nil)
-          iteration = 0
-
-          if (trace_this || trace_all) && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-            puts "    [decode_tree START] num_bits=#{num_bits}, range=#{range_decoder.range}, code=#{range_decoder.code}"
-            puts "      models array object_id=#{models.object_id}"
-          end
-
           (num_bits - 1).downto(0) do |i|
-            iteration += 1
-            model = models[m]
-            bit = range_decoder.decode_bit(model)
+            bit = range_decoder.decode_bit(models[m])
             m = (m << 1) | bit
             symbol |= (bit << i)
-            if (trace_this || trace_all) && ENV.fetch("LZMA_DEBUG_DISTANCE",
-                                                      nil)
-              puts "      [#{iteration}/#{num_bits}] i=#{i}, bit=#{bit}, m=#{m}, model.object_id=#{model.object_id}, prob=#{model.probability}, symbol=#{symbol}"
-            end
-          end
-          if (trace_this || trace_all) && ENV.fetch("LZMA_DEBUG_DISTANCE", nil)
-            puts "    [decode_tree END] symbol=#{symbol}"
           end
           symbol
         end

data/lib/omnizip/algorithms/lzma/length_coder.rb

@@ -67,67 +67,25 @@ module Omnizip
         # @param pos_state [Integer] Position state for tree selection
         # @return [void]
         def encode(range_encoder, length, pos_state)
-          trace_encode = ENV.fetch("LZMA_DEBUG_ENCODE",
-                                   nil) && ENV.fetch("TRACE_LENGTH_CODER", nil)
-
-          if trace_encode
-            puts "    [LengthCoder.encode] START: length=#{length}, pos_state=#{pos_state}"
-            puts "      @choice.prob=#{@choice.probability} @choice2.prob=#{@choice2.probability}"
-          end
-
           if length < LEN_LOW_SYMBOLS
             # 0-7: Use low tree
-            if trace_encode
-              puts "      Using LOW tree (length #{length} < #{LEN_LOW_SYMBOLS})"
-              puts "      Encoding choice=0 with prob=#{@choice.probability}"
-            end
             range_encoder.encode_bit(@choice, 0)
-            if trace_encode
-              puts "      After choice: @choice.prob=#{@choice.probability}"
-            end
             encode_tree(range_encoder, @low[pos_state], length,
                         NUM_LEN_LOW_BITS)
           elsif length < LEN_LOW_SYMBOLS + LEN_MID_SYMBOLS
             # 8-15: Use mid tree
-            if trace_encode
-              puts "      Using MID tree (length #{length} < #{LEN_LOW_SYMBOLS + LEN_MID_SYMBOLS})"
-              puts "      Encoding choice=1 with prob=#{@choice.probability}"
-            end
             range_encoder.encode_bit(@choice, 1)
-            if trace_encode
-              puts "      After choice: @choice.prob=#{@choice.probability}"
-              puts "      Encoding choice2=0 with prob=#{@choice2.probability}"
-            end
             range_encoder.encode_bit(@choice2, 0)
-            if trace_encode
-              puts "      After choice2: @choice2.prob=#{@choice2.probability}"
-            end
             encode_tree(range_encoder, @mid[pos_state],
                         length - LEN_LOW_SYMBOLS, NUM_LEN_MID_BITS)
           else
             # 16+: Use high tree
-            if trace_encode
-              puts "      Using HIGH tree (length #{length} >= #{LEN_LOW_SYMBOLS + LEN_MID_SYMBOLS})"
-              puts "      Encoding choice=1 with prob=#{@choice.probability}"
-            end
             range_encoder.encode_bit(@choice, 1)
-            if trace_encode
-              puts "      After choice: @choice.prob=#{@choice.probability}"
-              puts "      Encoding choice2=1 with prob=#{@choice2.probability}"
-            end
             range_encoder.encode_bit(@choice2, 1)
-            if trace_encode
-              puts "      After choice2: @choice2.prob=#{@choice2.probability}"
-            end
             encode_tree(range_encoder, @high,
                         length - LEN_LOW_SYMBOLS - LEN_MID_SYMBOLS,
                         NUM_LEN_HIGH_BITS)
           end
-
-          if trace_encode
-            puts "      FINAL @choice.prob=#{@choice.probability} @choice2.prob=#{@choice2.probability}"
-            puts "    [LengthCoder.encode] END"
-          end
         end
 
         # Decode a match length using SDK-compatible decoding
@@ -136,58 +94,21 @@ module Omnizip
         # @param pos_state [Integer] Position state for tree selection
         # @return [Integer] Decoded length value (before adding MATCH_LEN_MIN)
         def decode(range_decoder, pos_state)
-          trace_decode = ENV.fetch("LZMA_DEBUG_DISTANCE",
-                                   nil) && ENV.fetch("TRACE_LENGTH_CODER", nil)
-
-          if trace_decode
-            caller_loc = caller_locations(2, 1).first
-            puts "    [LengthCoder.decode] START: pos_state=#{pos_state}"
-            puts "      self.object_id=#{object_id}"
-            puts "      @choice.object_id=#{@choice.object_id} prob=#{@choice.probability}"
-            puts "      @choice2.object_id=#{@choice2.object_id} prob=#{@choice2.probability}"
-            puts "      Called from: #{caller_loc.label} at #{caller_loc.lineno}"
-          end
-
           choice_bit = range_decoder.decode_bit(@choice)
-          if trace_decode
-            puts "      Decoded choice=#{choice_bit} with prob=#{@choice.probability}"
-            puts "      After choice decode: @choice.prob=#{@choice.probability}"
-          end
 
           if choice_bit.zero?
             # Low tree
-            if trace_decode
-              puts "      Using LOW tree"
-            end
-            result = decode_tree(range_decoder, @low[pos_state],
-                                 NUM_LEN_LOW_BITS)
+            decode_tree(range_decoder, @low[pos_state],
+                        NUM_LEN_LOW_BITS)
           elsif range_decoder.decode_bit(@choice2).zero?
             # Mid tree
-            if trace_decode
-              puts "      Decoded choice2=0 with prob=#{@choice2.probability}"
-              puts "      After choice2 decode: @choice2.prob=#{@choice2.probability}"
-              puts "      Using MID tree"
-            end
-            result = LEN_LOW_SYMBOLS +
+            LEN_LOW_SYMBOLS +
               decode_tree(range_decoder, @mid[pos_state], NUM_LEN_MID_BITS)
           else
             # High tree
-            if trace_decode
-              puts "      Decoded choice2=1 with prob=#{@choice2.probability}"
-              puts "      After choice2 decode: @choice2.prob=#{@choice2.probability}"
-              puts "      Using HIGH tree"
-            end
-            result = LEN_LOW_SYMBOLS + LEN_MID_SYMBOLS +
+            LEN_LOW_SYMBOLS + LEN_MID_SYMBOLS +
               decode_tree(range_decoder, @high, NUM_LEN_HIGH_BITS)
           end
-
-          if trace_decode
-            puts "      FINAL @choice.prob=#{@choice.probability} @choice2.prob=#{@choice2.probability}"
-            puts "      Result: length_encoded=#{result}"
-            puts "    [LengthCoder.decode] END"
-          end
-
-          result
         end
 
         # Reset probability models to initial values
@@ -197,12 +118,6 @@ module Omnizip
         #
         # @return [void]
         def reset_models
-          if ENV["TRACE_RESET_MODELS"]
-            puts "    [LengthCoder.reset_models] CALLED!"
-            puts "      Before reset: @choice.prob=#{@choice.probability} @choice2.prob=#{@choice2.probability}"
-            caller_loc = caller_locations(2, 1).first
-            puts "      Called from: #{caller_loc.label} at #{caller_loc.path}:#{caller_loc.lineno}"
-          end
           @choice.reset
           @choice2.reset
 
@@ -215,9 +130,6 @@ module Omnizip
           end
 
           @high.each(&:reset)
-          if ENV["TRACE_RESET_MODELS"]
-            puts "      After reset: @choice.prob=#{@choice.probability} @choice2.prob=#{@choice2.probability}"
-          end
         end
 
         private