foobara-lru-cache 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 752fea9f8a8c09190e16fea3e710e2d860dbaad38b0c3bbc55aac543a4d36842
-  data.tar.gz: 35413dc39e7b39b9ab4553cbef65f857800f8c2cfa0a383737d8b4142cad4ec4
+  metadata.gz: ad9e286058d8549c8afada45a9f4f648f0b5e74966e55ae306912f7b70ef0f9e
+  data.tar.gz: f90e2f9c733371f785d7a11b788f1289f33ad011256b28cefb986ecf73216645
 SHA512:
-  metadata.gz: 3565ef92e71359a98c1dab733dd13032110fbc64906525419d6c999001ea7804a7fdcb1ec3fd0aff167dd42ad58005f3bb2bae9d9bcc4cb2fcd6eb5b4fedeb16
-  data.tar.gz: 3360bf084aa44837a4718185e65674398dadee3e19c15317f5e861507fc636d807cd33a2e9f1cd1c858af631db15b4f98be35f7df89ecc32e8fb22cab6cf5a95
+  metadata.gz: c0bc7897734a9685fc710092f1b208694c567dbdfabdfac87357484011f67cbee0ed36edb9563614914965266faeb6af322ac0095c2c653cfb8f76902bae513e
+  data.tar.gz: 6ef2c8384a6a9a978fcb314e54ff64ced52e6e7c06073d4beb9faf41e2959fdfb565ae7bcf302c127d6d9d27809e9f71fd13e45fa8e6a5fd8e42a269fd3aa87e

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [0.0.2] - 2025-02-28
+
+- Fix bug that corrupts the cache
+- Make the cached method thread-safe
+- Add #reset!
+
 ## [0.0.1] - 2025-02-27
 
 - Release as a gem

data/lib/foobara/lru_cache.rb

@@ -21,42 +21,64 @@ module Foobara
     end
 
     def cached(key)
-      if @key_to_node.key?(key)
-        node = @key_to_node[key]
-        move_node_to_front(node)
-        node.value
-      else
-        value = yield
-        node = Node.new(key, value)
-        @key_to_node[key] = node
-        prepend_node(node)
-        value
+      mutex.synchronize do
+        if @key_to_node.key?(key)
+          node = @key_to_node[key]
+          move_node_to_front(node)
+          return node.value
+        end
+      end
+
+      value = yield
+      mutex.synchronize do
+        if @key_to_node.key?(key)
+          node = @key_to_node[key]
+          move_node_to_front(node)
+        else
+          node = Node.new(key, value)
+          @key_to_node[key] = node
+          prepend_node(node)
+        end
       end
+
+      value
     end
 
     def key?(key)
       @key_to_node.key?(key)
     end
 
+    def reset!
+      mutex.synchronize do
+        @size = 0
+        @key_to_node.clear
+        @head = @tail = nil
+      end
+    end
+
     private
 
+    def mutex
+      @mutex ||= Mutex.new
+    end
+
     def move_node_to_front(node)
       return if node == @head
 
-      if node == @tail
-        @tail = node.prev
-      end
-
       prev_node = node.prev
+      next_node = node.next
 
-      node.prev = nil
-      prev_node.next = node.next
-
-      if node.next
-        node.next.prev = prev_node
+      if node == @tail
+        @tail = prev_node
+        @tail.next = nil
+      else
+        prev_node.next = next_node
+        next_node.prev = prev_node
       end
 
+      @head.prev = node
       node.next = @head
+      node.prev = nil
       @head = node
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara-lru-cache
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Miles Georgi