🐛 Fix to avoid counting existing entry twice (#1995)

KATO-Hiro · KATO-Hiro · commit a820479ea68a · 2025-05-07T12:37:37.000Z
diff --git a/src/lib/clients/cache.ts b/src/lib/clients/cache.ts
@@ -70,6 +70,9 @@ export class Cache<T> {
       throw new Error('Invalid cache key');
     }
 
+    // Note: Remove existing entry first to avoid counting it twice.
+    this.cache.delete(key);
+
     if (this.cache.size >= this.maxSize) {
       const oldestKey = this.findOldestEntry();
 

Original file line number	Diff line number	Diff line change
`@@ -70,6 +70,9 @@ export class Cache<T> {`
`70`	`70`	`throw new Error('Invalid cache key');`
`71`	`71`	`}`
`72`	`72`
	`73`	`+ // Note: Remove existing entry first to avoid counting it twice.`
	`74`	`+ this.cache.delete(key);`
	`75`	`+`
`73`	`76`	`if (this.cache.size >= this.maxSize) {`
`74`	`77`	`const oldestKey = this.findOldestEntry();`
`75`	`78`