png_conform 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8bcefecf4f13700db28e89f3ec81ad93df86ef0d5774057a3366ef9c74121169
-  data.tar.gz: bfd053fc3545d3ede27936476e8fce4d483c85e227d5eb5824e9faaa4f7c8872
+  metadata.gz: f40907ca0fa307f62da38b9fecdd97fa05e6f9953315d3bc197a6c1385c1c490
+  data.tar.gz: 27891ced82861b1ff72ac09ef12fcc6801c1f28b46cb849e1a4d6100a6ad9f6c
 SHA512:
-  metadata.gz: 8b16cfe85d3e17edc7dcdd5c4a7b1b9e416fd91be3825ac758bb2f95de19015cacc3c8c3301aabbac7174dc651e180930862b026a42968959539d4e1520857b6
-  data.tar.gz: c966fa4713fc24c7a4439e41ada2382814395d7be7bbe2e782fb388bf59d3f723ee5d36db5cac431c078c02b1dbdd9f517dbfa341ee0132ffee8403208185a69
+  metadata.gz: f58ca78ca13ca9934b677118c06cd67d362b96af38863b943dc0f60b757d0685435c450261ee1107d4eebbc5aa95e494c877cf6f55faccccb0dc89fdb0dd9bfc
+  data.tar.gz: 4479d1b4a79ee5d97fc49c297297f11818443e1f7c29e612131520d7037293951488a8fc8ec6af8772d7b49f833d45973ee21817e4bcaa34f98a4f88117818ff

data/.rubocop_todo.yml

@@ -1,92 +1,63 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2026-01-19 08:52:09 UTC using RuboCop version 1.81.1.
+# on 2026-04-04 06:26:35 UTC using RuboCop version 1.86.0.
 # 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: 6
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_argument, with_fixed_indentation
-Layout/ArgumentAlignment:
-  Exclude:
-    - 'lib/png_conform/commands/check_command.rb'
-    - 'lib/png_conform/services/chunk_processor.rb'
-    - 'lib/png_conform/services/profile_manager.rb'
-    - 'lib/png_conform/services/validator_pool.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleAlignWith.
-# SupportedStylesAlignWith: either, start_of_block, start_of_line
-Layout/BlockAlignment:
-  Exclude:
-    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
-    - 'lib/png_conform/analyzers/optimization_analyzer.rb'
-    - 'lib/png_conform/services/file_signature.rb'
-    - 'lib/png_conform/services/lru_cache.rb'
-
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-Layout/BlockEndNewline:
-  Exclude:
-    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
-    - 'lib/png_conform/analyzers/optimization_analyzer.rb'
-    - 'lib/png_conform/services/file_signature.rb'
-    - 'lib/png_conform/services/lru_cache.rb'
-
 # Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
-Layout/ElseAlignment:
+Gemspec/RequireMFA:
   Exclude:
-    - 'lib/png_conform/services/validation_cache.rb'
+    - 'png_conform.gemspec'
 
 # Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyleAlignWith, Severity.
-# SupportedStylesAlignWith: keyword, variable, start_of_line
-Layout/EndAlignment:
+# Configuration parameters: AllowForAlignment, AllowBeforeTrailingComments, ForceEqualSignAlignment.
+Layout/ExtraSpacing:
   Exclude:
-    - 'lib/png_conform/services/validation_cache.rb'
+    - 'spec/png_conform/validators/ancillary/bkgd_validator_spec.rb'
 
-# Offense count: 2
+# Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: AllowMultipleStyles, EnforcedHashRocketStyle, EnforcedColonStyle, EnforcedLastArgumentHashStyle.
-# SupportedHashRocketStyles: key, separator, table
-# SupportedColonStyles: key, separator, table
-# SupportedLastArgumentHashStyles: always_inspect, always_ignore, ignore_implicit, ignore_explicit
-Layout/HashAlignment:
+# Configuration parameters: EnforcedStyle.
+# SupportedStyles: normal, indented_internal_methods
+Layout/IndentationConsistency:
   Exclude:
-    - 'lib/png_conform/services/validator_pool.rb'
+    - 'png_conform.gemspec'
 
-# Offense count: 10
+# Offense count: 6
 # This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: Width, AllowedPatterns.
-Layout/IndentationWidth:
+# Configuration parameters: EnforcedStyle, IndentationWidth.
+# SupportedStyles: aligned, indented
+Layout/LineEndStringConcatenationIndentation:
   Exclude:
-    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
     - 'lib/png_conform/analyzers/optimization_analyzer.rb'
-    - 'lib/png_conform/services/file_signature.rb'
-    - 'lib/png_conform/services/lru_cache.rb'
-    - 'lib/png_conform/services/validation_cache.rb'
 
-# Offense count: 267
+# Offense count: 263
 # 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: 3
+# Offense count: 1
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: AllowInHeredoc.
 Layout/TrailingWhitespace:
   Exclude:
-    - 'lib/png_conform/commands/check_command.rb'
-    - 'lib/png_conform/services/profile_manager.rb'
-    - 'lib/png_conform/services/validator_pool.rb'
+    - 'spec/png_conform/validators/ancillary/bkgd_validator_spec.rb'
+
+# Offense count: 14
+# This cop supports safe autocorrection (--autocorrect).
+Lint/AmbiguousOperatorPrecedence:
+  Exclude:
+    - 'benchmarks/lib/benchmark_runner.rb'
+    - 'benchmarks/lib/report_generator.rb'
+    - 'lib/png_conform/services/result_builder.rb'
+    - 'lib/png_conform/services/zlib_validator.rb'
+    - 'lib/png_conform/validators/critical/ihdr_validator.rb'
 
 # Offense count: 1
 Lint/BinaryOperatorWithIdenticalOperands:
@@ -102,6 +73,12 @@ Lint/DuplicateBranch:
     - 'lib/png_conform/validators/ancillary/sbit_validator.rb'
     - 'spec/pngsuite/helpers/semantic_validator.rb'
 
+# Offense count: 1
+# Configuration parameters: AllowComments, AllowEmptyLambdas.
+Lint/EmptyBlock:
+  Exclude:
+    - 'spec/png_conform/validators/ancillary/bkgd_validator_spec.rb'
+
 # Offense count: 4
 # Configuration parameters: AllowedParentClasses.
 Lint/MissingSuper:
@@ -142,6 +119,13 @@ Metrics/ParameterLists:
 Metrics/PerceivedComplexity:
   Enabled: false
 
+# Offense count: 132
+# Configuration parameters: Mode, AllowedMethods, AllowedPatterns, AllowBangMethods, WaywardPredicates.
+# AllowedMethods: call
+# WaywardPredicates: infinite?, nonzero?
+Naming/PredicateMethod:
+  Enabled: false
+
 # Offense count: 1
 # Configuration parameters: EnforcedStyle, CheckMethodNames, CheckSymbols, AllowedIdentifiers, AllowedPatterns.
 # SupportedStyles: snake_case, normalcase, non_integer
@@ -156,6 +140,13 @@ Performance/CollectionLiteralInLoop:
   Exclude:
     - 'examples/advanced_usage.rb'
 
+# Offense count: 4
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Performance/MapCompact:
+  Exclude:
+    - 'benchmarks/lib/metrics_collector.rb'
+    - 'lib/png_conform/services/file_signature.rb'
+
 # Offense count: 70
 # Configuration parameters: Prefixes, AllowedPatterns.
 # Prefixes: when, with, without
@@ -187,51 +178,81 @@ RSpec/MultipleMemoizedHelpers:
   Max: 6
 
 # Offense count: 2
-# Configuration parameters: CustomTransform, IgnoreMethods, IgnoreMetadata.
+# This cop supports unsafe autocorrection (--autocorrect-all).
+RSpec/Output:
+  Exclude:
+    - 'spec/png_conform/validators/ancillary/bkgd_validator_spec.rb'
+
+# Offense count: 2
+# Configuration parameters: CustomTransform, IgnoreMethods, IgnoreMetadata, InflectorPath, EnforcedInflector.
+# SupportedInflectors: default, active_support
 RSpec/SpecFilePathFormat:
   Exclude:
     - 'spec/png_conform/bindata/chunk_structure_spec.rb'
     - 'spec/png_conform/bindata/png_file_spec.rb'
 
-# Offense count: 45
+# Offense count: 44
 # Configuration parameters: IgnoreNameless, IgnoreSymbolicNames.
 RSpec/VerifiedDoubles:
   Enabled: false
 
-# Offense count: 4
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle, ProceduralMethods, FunctionalMethods, AllowedMethods, AllowedPatterns, AllowBracesOnProceduralOneLiners, BracesRequiredMethods.
-# SupportedStyles: line_count_based, semantic, braces_for_chaining, always_braces
-# ProceduralMethods: benchmark, bm, bmbm, create, each_with_object, measure, new, realtime, tap, with_object
-# FunctionalMethods: let, let!, subject, watch
-# AllowedMethods: lambda, proc, it
-Style/BlockDelimiters:
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/BitwisePredicate:
   Exclude:
-    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
-    - 'lib/png_conform/analyzers/optimization_analyzer.rb'
-    - 'lib/png_conform/services/file_signature.rb'
-    - 'lib/png_conform/services/lru_cache.rb'
+    - 'lib/png_conform/bindata/chunk_structure.rb'
+    - 'lib/png_conform/services/chunk_processor.rb'
 
-# Offense count: 41
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/ComparableBetween:
+  Exclude:
+    - 'lib/png_conform/validators/ancillary/trns_validator.rb'
+    - 'lib/png_conform/validators/base_validator.rb'
+
+# Offense count: 1
+Style/FileOpen:
+  Exclude:
+    - 'lib/png_conform/container.rb'
+
+# Offense count: 36
 # This cop supports safe autocorrection (--autocorrect).
 # Configuration parameters: MaxUnannotatedPlaceholdersAllowed, Mode, AllowedMethods, AllowedPatterns.
 # SupportedStyles: annotated, template, unannotated
 Style/FormatStringToken:
   EnforcedStyle: unannotated
 
-# Offense count: 1
-Style/MissingRespondToMissing:
+# Offense count: 4
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/MapToSet:
   Exclude:
-    - 'lib/png_conform/cli.rb'
+    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
 
 # Offense count: 1
+# This cop supports unsafe autocorrection (--autocorrect-all).
+Style/RedundantFormat:
+  Exclude:
+    - 'benchmarks/lib/report_generator.rb'
+
+# Offense count: 2
+# This cop supports unsafe autocorrection (--autocorrect-all).
+# Configuration parameters: AllowedMethods, AllowedPatterns.
+Style/ReturnNilInPredicateMethodDefinition:
+  Exclude:
+    - 'lib/png_conform/analyzers/comparison_analyzer.rb'
+    - 'spec/pngsuite/helpers/pngcheck_parser.rb'
+
+# Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
-Style/MultilineTernaryOperator:
+# Configuration parameters: EnforcedStyle, ConsistentQuotesInMultiline.
+# SupportedStyles: single_quotes, double_quotes
+Style/StringLiterals:
   Exclude:
-    - 'lib/png_conform/services/validation_cache.rb'
+    - 'png_conform.gemspec'
 
 # Offense count: 2
 # This cop supports safe autocorrection (--autocorrect).
-Style/RedundantParentheses:
+Style/SuperArguments:
   Exclude:
-    - 'lib/png_conform/services/validation_cache.rb'
+    - 'lib/png_conform/models/chunk_info.rb'
+    - 'lib/png_conform/models/validation_result.rb'

data/benchmarks/lib/benchmark_runner.rb

@@ -165,7 +165,7 @@ class BenchmarkRunner
     pct = (current.to_f / total * 100).round(1)
     bar_width = 40
     filled = (bar_width * current / total).to_i
-    bar = "=" * filled + " " * (bar_width - filled)
+    bar = ("=" * filled) + (" " * (bar_width - filled))
 
     print "\r  [#{bar}] #{pct}% (#{current}/#{total}) #{label}     "
     $stdout.flush

data/benchmarks/lib/metrics_collector.rb

@@ -46,8 +46,8 @@ class MetricsCollector
     tool_runs = runs_for_tool(tool)
     return nil if tool_runs.empty?
 
-    execution_times = tool_runs.map { |r| r[:execution_time] }.compact
-    memory_values = tool_runs.map { |r| r[:peak_memory] }.compact
+    execution_times = tool_runs.filter_map { |r| r[:execution_time] }
+    memory_values = tool_runs.filter_map { |r| r[:peak_memory] }
 
     {
       tool: tool,
@@ -118,7 +118,7 @@ class MetricsCollector
       total_runs: @runs.size,
       tools: tools,
       files_tested: @runs.map { |r| r[:file] }.uniq.size,
-      tool_statistics: tools.map { |tool| calculate_statistics(tool) }.compact,
+      tool_statistics: tools.filter_map { |tool| calculate_statistics(tool) },
     }
   end
 

data/benchmarks/lib/report_generator.rb

@@ -42,9 +42,9 @@ class ReportGenerator
     return "No benchmark data available.\n" if tools.empty?
 
     output = []
-    output << "=" * 80
+    output << ("=" * 80)
     output << "PNG Validation Tool Benchmark Comparison"
-    output << "=" * 80
+    output << ("=" * 80)
     output << ""
     output << "Configuration:"
     output << "  Files tested:     #{summary[:files_tested]} PNG files"
@@ -62,9 +62,9 @@ class ReportGenerator
     # Performance comparison (if we have 2 tools)
     if tools.size == 2
       comparison = metrics_collector.compare_tools(tools[0], tools[1])
-      output << "-" * 80
+      output << ("-" * 80)
       output << "PERFORMANCE SUMMARY"
-      output << "-" * 80
+      output << ("-" * 80)
       output << ""
       output << format_comparison_table(comparison)
       output << ""
@@ -73,9 +73,9 @@ class ReportGenerator
     end
 
     # Detailed statistics per tool
-    output << "-" * 80
+    output << ("-" * 80)
     output << "DETAILED STATISTICS"
-    output << "-" * 80
+    output << ("-" * 80)
     output << ""
 
     tools.each do |tool|
@@ -176,9 +176,8 @@ class ReportGenerator
   # Format comparison table for text output.
   def format_comparison_table(comparison)
     lines = []
-    lines << sprintf("%-15s %12s %12s %12s %8s",
-                     "Tool", "Avg Time", "Files/sec", "Peak Memory", "Winner")
-    lines << "-" * 80
+    lines << "Tool                Avg Time    Files/sec  Peak Memory   Winner"
+    lines << ("-" * 80)
 
     [comparison[:tool1], comparison[:tool2]].each do |tool|
       stats = comparison[:stats][tool]

data/lib/png_conform/analyzers/comparison_analyzer.rb

@@ -175,19 +175,19 @@ module PngConform
       end
 
       def new_errors
-        errors1 = @result1.errors.map(&:message).to_set
-        errors2 = @result2.errors.map(&:message).to_set
+        errors1 = @result1.errors.to_set(&:message)
+        errors2 = @result2.errors.to_set(&:message)
         (errors2 - errors1).to_a
       end
 
       def resolved_errors
-        errors1 = @result1.errors.map(&:message).to_set
-        errors2 = @result2.errors.map(&:message).to_set
+        errors1 = @result1.errors.to_set(&:message)
+        errors2 = @result2.errors.to_set(&:message)
         (errors1 - errors2).to_a
       end
 
       def compression_improved?
-        return nil unless @result1.compression_ratio && @result2.compression_ratio
+        return false unless @result1.compression_ratio && @result2.compression_ratio
 
         @result2.compression_ratio > @result1.compression_ratio
       end

data/lib/png_conform/analyzers/optimization_analyzer.rb

@@ -42,7 +42,7 @@ module PngConform
           priority: :medium,
           savings_bytes: savings,
           description: "Remove #{unnecessary.count} unnecessary chunks " \
-                      "(#{unnecessary.map(&:type).join(', ')})",
+                       "(#{unnecessary.map(&:type).join(', ')})",
           chunks: unnecessary.map(&:type),
         }
       end
@@ -65,7 +65,7 @@ module PngConform
             priority: :high,
             savings_bytes: estimated_savings,
             description: "Convert from 16-bit to 8-bit depth " \
-                        "(estimated ~#{@config.optimization_percentages[:bit_depth_reduction]}% file size reduction)",
+                         "(estimated ~#{@config.optimization_percentages[:bit_depth_reduction]}% file size reduction)",
             current: "16-bit",
             recommended: "8-bit",
           }
@@ -87,7 +87,7 @@ module PngConform
             @config.optimization_percentages[:palette_conversion] / 100.0
           ).to_i,
           description: "Consider converting to palette mode if using limited colors " \
-                      "(potential ~#{@config.optimization_percentages[:palette_conversion]}% reduction)",
+                       "(potential ~#{@config.optimization_percentages[:palette_conversion]}% reduction)",
           current: "RGB (Truecolor)",
           recommended: "Indexed (Palette)",
         }
@@ -109,7 +109,7 @@ module PngConform
           priority: :low,
           savings_bytes: savings,
           description: "Remove interlacing for smaller file size " \
-                      "(~#{@config.optimization_percentages[:interlace_removal]}% reduction, but slower initial display)",
+                       "(~#{@config.optimization_percentages[:interlace_removal]}% reduction, but slower initial display)",
           current: "Adam7 interlaced",
           recommended: "Non-interlaced",
         }
@@ -129,7 +129,7 @@ module PngConform
           priority: :low,
           savings_bytes: total_text_size,
           description: "#{text_chunks.count} text chunks using #{total_text_size} bytes " \
-                      "(consider removing non-essential metadata)",
+                       "(consider removing non-essential metadata)",
           chunks: text_chunks.map(&:type),
         }
       end
@@ -151,7 +151,7 @@ module PngConform
           priority: :medium,
           savings_bytes: total_metadata,
           description: "Metadata comprises #{(total_metadata.to_f / file_size * 100).round(1)}% " \
-                      "of file size (#{total_metadata} bytes)",
+                       "of file size (#{total_metadata} bytes)",
           recommendation: "Review if all metadata is necessary",
         }
       end

data/lib/png_conform/bindata/chunk_structure.rb

@@ -88,7 +88,7 @@ module PngConform
       #
       # @return [Boolean] true if critical chunk
       def critical?
-        (chunk_type[0].ord & 0x20).zero?
+        chunk_type[0].ord.nobits?(0x20)
       end
 
       # Check if this is an ancillary chunk

data/lib/png_conform/cli.rb

@@ -117,12 +117,5 @@ module PngConform
     def self.exit_on_failure?
       true
     end
-
-    # Provide helpful error message for unknown commands
-    def method_missing(method, *_args)
-      puts "Error: Unknown command '#{method}'"
-      puts "Run 'png_conform help' for usage information"
-      exit(1)
-    end
   end
 end

data/lib/png_conform/commands/check_command.rb

@@ -39,14 +39,15 @@ module PngConform
         if files.empty?
           puts "Error: No files specified"
           puts "Usage: png_conform check [OPTIONS] FILES"
-          exit(1)
+          raise PngConform::NoFilesSpecifiedError.new
         end
 
         # Check if profile exists (if specified)
         if options[:profile] && !Services::ProfileManager.profile_exists?(options[:profile])
           puts "Error: Unknown profile '#{options[:profile]}'"
           puts "Available profiles: #{Services::ProfileManager.available_profiles.join(', ')}"
-          exit(1)
+          raise PngConform::UnknownProfileError.new(options[:profile],
+                                                    Services::ProfileManager.available_profiles)
         end
 
         # Check for conflicting options

data/lib/png_conform/models/chunk_info.rb

@@ -22,7 +22,7 @@ module PngConform
       def initialize(attributes = {})
         @data = attributes.delete(:data)
         @crc = attributes.delete(:crc)
-        super(attributes)
+        super
       end
 
       # Format offset as hex for display

data/lib/png_conform/models/validation_result.rb

@@ -24,7 +24,7 @@ module PngConform
 
       # Initialize with hash map for fast lookups
       def initialize(*args)
-        super(*args)
+        super
         @chunks_by_type_map = {}
         rebuild_chunks_map
       end

data/lib/png_conform/services/chunk_processor.rb

@@ -172,7 +172,7 @@ module PngConform
         first_byte = chunk_type.bytes[0]
 
         # Bit 5 (0x20) of first byte indicates ancillary vs critical
-        if (first_byte & 0x20).zero?
+        if first_byte.nobits?(0x20)
           # Critical chunk - must be recognized
           @context.add_error(
             chunk_type: chunk_type,

data/lib/png_conform/services/file_signature.rb

@@ -103,9 +103,9 @@ module PngConform
           chunk_count: @result.chunks.count,
           chunk_types: @result.chunks.map(&:type).sort,
           chunk_sizes: @result.chunks.map(&:length).sort,
-          crcs: @result.chunks.map do |c|
+          crcs: @result.chunks.filter_map do |c|
             c.crc_actual || c.crc_expected
-          end.compact,
+          end,
         }
       end
 

data/lib/png_conform/services/result_builder.rb

@@ -225,7 +225,7 @@ module PngConform
 
           # Calculate percentage: (compressed/original - 1) * 100
           # Negative means compression, positive means expansion
-          ((compressed_size.to_f / original_size - 1) * 100).round(1)
+          (((compressed_size.to_f / original_size) - 1) * 100).round(1)
         rescue StandardError
           # If decompression fails, we can't calculate ratio
           0.0

data/lib/png_conform/services/zlib_validator.rb

@@ -148,7 +148,7 @@ module PngConform
                             end
 
         # Bytes per pixel (rounded up)
-        ((samples_per_pixel * bit_depth + 7) / 8.0).ceil
+        (((samples_per_pixel * bit_depth) + 7) / 8.0).ceil
       end
 
       # Calculate expected decompressed data size
@@ -158,7 +158,7 @@ module PngConform
 
         if interlace_method.zero?
           # Non-interlaced: (scanline_width + 1 filter byte) * height
-          scanline_width = ((width * bit_depth * samples_for_color_type(color_type) + 7) / 8.0).ceil
+          scanline_width = (((width * bit_depth * samples_for_color_type(color_type)) + 7) / 8.0).ceil
           (scanline_width + 1) * height
         else
           # Adam7 interlaced: calculate for all 7 passes

data/lib/png_conform/validators/ancillary/trns_validator.rb

@@ -115,7 +115,7 @@ module PngConform
               return false
             end
 
-            return true if length >= 1 && length <= 256
+            return true if length.between?(1, 256)
 
             add_error("invalid tRNS length for indexed-color " \
                       "(#{length}, must be 1-256)")

data/lib/png_conform/validators/base_validator.rb

@@ -82,7 +82,7 @@ module PngConform
       # @param name [String] Name of the value for error message
       # @return [Boolean] True if value is in range
       def check_range(value, min, max, name)
-        return true if value >= min && value <= max
+        return true if value.between?(min, max)
 
         add_error("invalid #{name} (#{value}, must be #{min}-#{max})")
         false

data/lib/png_conform/validators/critical/ihdr_validator.rb

@@ -90,12 +90,12 @@ module PngConform
             valid = false
           end
 
-          if width > 2**31 - 1
+          if width > (2**31) - 1
             add_warning("image width (#{width}) exceeds maximum " \
                         "recommended value")
           end
 
-          if height > 2**31 - 1
+          if height > (2**31) - 1
             add_warning("image height (#{height}) exceeds maximum " \
                         "recommended value")
           end

data/lib/png_conform/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PngConform
-  VERSION = "0.1.3"
+  VERSION = "0.1.4"
 end

data/lib/png_conform.rb

@@ -19,6 +19,44 @@ module PngConform
 
   class ParseError < Error; end
 
+  # CLI argument errors - inherit from Thor::Error for Thor integration
+  class CliArgumentError < Thor::Error; end
+
+  class NoFilesSpecifiedError < CliArgumentError
+    def message
+      "No files specified. Usage: png_conform check [OPTIONS] FILES"
+    end
+  end
+
+  class UnknownProfileError < CliArgumentError
+    attr_reader :profile_name, :available_profiles
+
+    def initialize(profile_name, available_profiles)
+      @profile_name = profile_name
+      @available_profiles = available_profiles
+      super(build_message)
+    end
+
+    def message
+      "Unknown profile '#{profile_name}'. Available profiles: #{available_profiles.join(', ')}"
+    end
+
+    private
+
+    def build_message
+      "Unknown profile '#{profile_name}'. Available profiles: #{available_profiles.join(', ')}"
+    end
+  end
+
+  class UnknownCommandError < Thor::Error
+    attr_reader :command_name
+
+    def initialize(command_name)
+      @command_name = command_name
+      super("Unknown command '#{command_name}'. Run 'png_conform help' for usage information.")
+    end
+  end
+
   # Load BinData structures (Phase 2 - implemented)
   require_relative "png_conform/bindata/chunk_structure"
   require_relative "png_conform/bindata/png_file"

data/png_conform.gemspec

@@ -25,6 +25,7 @@ Gem::Specification.new do |spec|
   spec.metadata["homepage_uri"] = spec.homepage
   spec.metadata["source_code_uri"] = "https://github.com/claricle/png_conform"
   spec.metadata["changelog_uri"] = "https://github.com/claricle/png_conform"
+  spec.metadata["rubygems_mfa_required"] = "true"
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: png_conform
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-01-20 00:00:00.000000000 Z
+date: 2026-04-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bindata
@@ -232,6 +232,7 @@ metadata:
   homepage_uri: https://github.com/claricle/png_conform
   source_code_uri: https://github.com/claricle/png_conform
   changelog_uri: https://github.com/claricle/png_conform
+  rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []
 require_paths: