canon 0.3.68 → 0.3.70

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 590320aa865898c6342d5c96566e6313e678a6d6f31a4d1e9463a50c9f434052
-  data.tar.gz: 6daacd998c4d52ae54096bc91329cd48686d33b7942970313891f8d69edf2141
+  metadata.gz: 5a8ff57b9e99197b824e486af4f029cd005cce5e263cfa7433a3e7351d356d55
+  data.tar.gz: 9737c271ceeaa5be0358f56f86891e02cd5c34c099721b6ba3deaaf107937256
 SHA512:
-  metadata.gz: 5d9fe69844ee374debb42557852bdcfd3d8cd3cadb30d15f01c2e2b932ad7412339107996bb2c88f04669951b88752856c56b812d2fd4ecbd3f993390b682efe
-  data.tar.gz: bfee82458261d1b20f78d7745da3cf620ce9beff2a993c01edcf6ff1d22f2538e8ddf7c45f21237e64f41a0218d0a25432dffcf0ea6cbcff3443cd416d4a677b
+  metadata.gz: d2f9294a68dc860dc315331b083b62e3661c42c69b1a44ac2984b0f29f39c998b6d90464972d04bdb763d370cc1668e8245e19a0db02eee4d1907946ab529ae1
+  data.tar.gz: d36159fe358cef45d398c7deb0137354dbfa5f78e45850adc28cdda30f821561797340dc0f58da1c51284d6c619f5403de52f2a7b7f1549bdcaf56ec02631c6d

data/lib/canon/cache.rb

@@ -24,7 +24,14 @@ module Canon
       # Maximum number of entries per cache category
       MAX_CACHE_SIZE = 100
 
-      # Fetch a value from cache, or compute and cache it
+      LOCK = Mutex.new
+
+      # Fetch a value from cache, or compute and cache it.
+      #
+      # The lock covers the cache operations only — never the compute
+      # block — so concurrent parses stay parallel; duplicate computes
+      # for the same key may race (last write wins), which any
+      # check-then-compute cache accepts.
       #
       # @param category [Symbol] Cache category (:document_parse, :format_detect, etc.)
       # @param key [String] Cache key
@@ -33,24 +40,24 @@ module Canon
       def fetch(category, key)
         cache = cache_for(category)
 
-        # Check if key exists
-        if cache.key?(key)
-          # Update access time for LRU
-          cache[key][:accessed] = Time.now
-          return cache[key][:value]
+        LOCK.synchronize do
+          if cache.key?(key)
+            @clock = (@clock || 0) + 1
+            cache[key][:accessed] = @clock
+            return cache[key][:value]
+          end
         end
 
-        # Compute and cache the value
         value = yield
 
-        # Evict oldest entry if cache is full
-        if cache.size >= MAX_CACHE_SIZE
-          oldest_key = cache.min_by { |_, v| v[:accessed] }&.first
-          cache.delete(oldest_key) if oldest_key
+        LOCK.synchronize do
+          if cache.size >= MAX_CACHE_SIZE
+            oldest_key = cache.min_by { |_, v| v[:accessed] }&.first
+            cache.delete(oldest_key) if oldest_key
+          end
+          @clock = (@clock || 0) + 1
+          cache[key] = { value: value, accessed: @clock }
         end
-
-        @clock = (@clock || 0) + 1
-        cache[key] = { value: value, accessed: @clock }
         value
       end
 
@@ -58,8 +65,10 @@ module Canon
       #
       # Useful for tests or when memory needs to be freed
       def clear_all
-        @caches&.each_value(&:clear)
-        @caches = nil
+        LOCK.synchronize do
+          @caches&.each_value(&:clear)
+          @caches = nil
+        end
       end
 
       # Clear a specific cache category

data/lib/canon/comparison/xml_comparator.rb

@@ -202,16 +202,10 @@ module Canon
         def verbose_report_proven_empty?(n1, n2, opts)
           return true unless opts[:verbose]
 
-          # Identical input parses identically — differences and
-          # attribute order coincide by construction. Recover errors
-          # ride the fingerprint's diag check when the engine exposes
-          # them (libleptris 1.9.205+, #1200); otherwise the one SAX
-          # scan covers the parse-error banner (issue #130).
-          if n1 == n2
-            return true if Xml::DigestGate.recover_diags_available?
-
-            return !Xml::Sax.probe(n1).saw_error?
-          end
+          # Identical input parses identically, and certify has
+          # already declined any recover-error document through the
+          # wrapper's parse_diagnostics — the banner cannot be lost.
+          return true if n1 == n2
 
           left = Xml::Sax.probe(n1)
           return false if left.saw_error?

data/lib/canon/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Canon
-  VERSION = "0.3.68"
+  VERSION = "0.3.70"
 end

data/lib/canon/xml/digest_gate.rb

@@ -21,26 +21,13 @@ module Canon
     module DigestGate
       module_function
 
-      # libleptris 1.9.205+ records duplicate-attribute recover
-      # events on the parsed document (#1200): read through the
-      # FFI diag counter. Evaluated lazily at call time — a load-time
-      # constant would freeze before leptris loads (defined? never
-      # resolves pending autoloads). When the adapter is not leptris
-      # (or the binding predates the surface), the verbose lane
-      # falls back to the SAX probe for its error scan.
-      def recover_diags_available?
-        defined?(::Leptris::XML::FFI) &&
-          ::Leptris::XML::FFI.respond_to?(:leptris_document_parse_diag_count)
-      rescue StandardError
-        false
-      end
-
+      # moxml 0.5.76+ (moxml#271) surfaces the engine's recover
+      # diagnostics — duplicate-attribute events and the like (#1200)
+      # — as Document#parse_diagnostics. The direct-leptris-FFI
+      # reach-in this replaced was the gate's one documented seam
+      # exception; the wrapper ends it.
       def recover_diags?(doc)
-        return false unless recover_diags_available?
-
-        native = doc.native
-        native = native.c_ptr if native.respond_to?(:c_ptr)
-        ::Leptris::XML::FFI.leptris_document_parse_diag_count(native).positive?
+        !doc.parse_diagnostics.empty?
       end
 
       def available?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.3.68
+  version: 0.3.70
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-09-22 00:00:00.000000000 Z
+date: 2026-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diff-lcs
@@ -44,28 +44,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.9.222
+        version: 1.9.228
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.9.222
+        version: 1.9.228
 - !ruby/object:Gem::Dependency
   name: moxml
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.73
+        version: 0.5.78
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.73
+        version: 0.5.78
 - !ruby/object:Gem::Dependency
   name: paint
   requirement: !ruby/object:Gem::Requirement