bug #391 [Store][Cache] Fix setup method (Guikingone)

OskarStark · OskarStark · commit d7b27c3c380a · 2025-09-01T11:15:19.000+02:00
This PR was merged into the main branch. Discussion ---------- [Store][Cache] Fix `setup` method | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | Docs? | no | Issues | None | License | MIT Hi 👋🏻 Here's the `CacheStore` fix as discussed in #335. Commits ------- 394e3f6 ref(store): CacheStore
diff --git a/src/store/src/Bridge/Local/CacheStore.php b/src/store/src/Bridge/Local/CacheStore.php
@@ -43,7 +43,11 @@ public function setup(array $options = []): void
             throw new InvalidArgumentException('No supported options.');
         }
 
-        $this->cache->clear();
+        if ($this->cache->hasItem($this->cacheKey)) {
+            return;
+        }
+
+        $this->cache->get($this->cacheKey, static fn (): array => []);
     }
 
     public function add(VectorDocument ...$documents): void

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,11 @@ public function setup(array $options = []): void`
`43`	`43`	`throw new InvalidArgumentException('No supported options.');`
`44`	`44`	`}`
`45`	`45`
`46`		`- $this->cache->clear();`
	`46`	`+ if ($this->cache->hasItem($this->cacheKey)) {`
	`47`	`+ return;`
	`48`	`+ }`
	`49`	`+`
	`50`	`+ $this->cache->get($this->cacheKey, static fn (): array => []);`
`47`	`51`	`}`
`48`	`52`
`49`	`53`	`public function add(VectorDocument ...$documents): void`