canon 0.3.68 → 0.3.69

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 590320aa865898c6342d5c96566e6313e678a6d6f31a4d1e9463a50c9f434052
-  data.tar.gz: 6daacd998c4d52ae54096bc91329cd48686d33b7942970313891f8d69edf2141
+  metadata.gz: 4f23fa758538acdbe352d58250d880b04e905191ce3c44b30528580cee06f224
+  data.tar.gz: c33b27f2839764f6d91fded3b3d7cb2042dc7e91d1d7a630037bcda0ce645c37
 SHA512:
-  metadata.gz: 5d9fe69844ee374debb42557852bdcfd3d8cd3cadb30d15f01c2e2b932ad7412339107996bb2c88f04669951b88752856c56b812d2fd4ecbd3f993390b682efe
-  data.tar.gz: bfee82458261d1b20f78d7745da3cf620ce9beff2a993c01edcf6ff1d22f2538e8ddf7c45f21237e64f41a0218d0a25432dffcf0ea6cbcff3443cd416d4a677b
+  metadata.gz: 6137233af91995ea5956d35fa154d2c206557e30c7fd9d1ced455667560f05810a034c4a1c407e4635fdbaefe8ad9929ad5b28cdf4cf5118990d83865611c4a3
+  data.tar.gz: 16a631574442c56fc87e5a48851643630ba9342d7e06f4b0b938f28a24792d5b414884099ec7c767fedb067971cc6ab82cf48871d55a1d1551f9cc5d7886e868

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/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canon
 version: !ruby/object:Gem::Version
-  version: 0.3.68
+  version: 0.3.69
 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