Merge pull request #70 from zhigang1992/fix/removeAll

Fix: Strategy based removal on insert operates on all items rather than just inserted items

Author: mergesort

Sources/Boutique/Store.swift

@@ -335,7 +335,7 @@ internal extension Store {
         var currentItems = self.items
 
         if let strategy = existingItemsStrategy {
-            var removedItems = [item]
+            var removedItems = currentItems
             try await self.removeItemsFromStorageEngine(&removedItems, withStrategy: strategy)
             // If we remove this one it will error
             self.removeItemsFromMemory(&currentItems, withStrategy: strategy, identifier: cacheIdentifier)
@@ -360,7 +360,7 @@ internal extension Store {
 
         if let strategy = existingItemsStrategy {
             // Remove items from disk and memory based on the cache invalidation strategy
-            var removedItems = items
+            var removedItems = currentItems
             try await self.removeItemsFromStorageEngine(&removedItems, withStrategy: strategy)
             // This one is fine to remove... but why?
             // Is it the way we construct the items in the ordered dictionary?