@aiquants/daily-report 0.3.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aiquants/daily-report",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Reusable daily-report feature package: shared types/schemas, React (Router v7) UI with virtual scrolling + optimistic updates + SSE sync, and a drizzle (mssql) server layer with DI ports (auth, user resolution, redis, external sources).",
   "sideEffects": false,
   "main": "dist/index.js",
@@ -75,8 +75,8 @@
     "react-dom": ">=18",
     "react-router": "^7.0.0",
     "zod": ">=3.25.0 <5.0.0",
-    "@aiquants/virtualscroll": "^1.18.3",
-    "@aiquants/swipe-overlay": "^1.2.3"
+    "@aiquants/swipe-overlay": "^1.2.3",
+    "@aiquants/virtualscroll": "^1.18.3"
   },
   "peerDependenciesMeta": {
     "drizzle-orm": {

package/src/server/cache.spec.ts

@@ -0,0 +1,47 @@
+import { describe, expect, it, vi } from "vitest"
+import { createEpochStore, SqlResultCache } from "./cache"
+
+describe("SqlResultCache Memory Cleanup", () => {
+    it("should clean up expired cache entries when map size exceeds 500", async () => {
+        // Mock epoch store
+        const epochStore = createEpochStore(undefined)
+        vi.spyOn(epochStore, "getEpoch").mockResolvedValue(0)
+
+        const cache = new SqlResultCache({ defaultTtlMs: 1000 }, epochStore)
+
+        // 1. 期限切れのキャッシュを 500 件作成 (expireAt を過去に設定)
+        // 直接プライベートフィールド buckets に流し込むことでモック状態を作る
+        const buckets = (cache as any).buckets
+        for (let i = 0; i < 505; i++) {
+            buckets.set(`expired-key-${i}`, {
+                records: [i],
+                expireAt: Date.now() - 1000, // 過去時間
+                epoch: 0,
+            })
+        }
+
+        // 2. 有効なキャッシュを 10 件作成 (expireAt を未来に設定)
+        for (let i = 0; i < 10; i++) {
+            buckets.set(`valid-key-${i}`, {
+                records: [i],
+                expireAt: Date.now() + 100000, // 未来時間
+                epoch: 0,
+            })
+        }
+
+        expect(buckets.size).toBe(515)
+
+        // 3. getOrFetch を実行して 500 件超えの閾値トリガーを引く
+        const fetcher = vi.fn().mockResolvedValue(["new-value"])
+        await cache.getOrFetch({
+            cacheKey: "trigger-key",
+            fetcher,
+        })
+
+        // 4. 検証: 期限切れの 505 件のデータが cleanExpired で削除され、有効な 10 件 + 新規の 1 件のみ残っていること
+        expect(buckets.size).toBe(11) // 10 (valid) + 1 (trigger-key)
+        expect(buckets.has("trigger-key")).toBe(true)
+        expect(buckets.has("valid-key-0")).toBe(true)
+        expect(buckets.has("expired-key-0")).toBe(false)
+    })
+})

package/src/server/cache.ts

@@ -123,6 +123,10 @@ export class SqlResultCache {
                 // キャッシュ保存時に現在の epoch を記録
                 const epoch = opts.epochKey ? await this.epochStore.getEpoch(opts.epochKey) : 0
                 this.buckets.set(cacheKey, { records, expireAt, epoch })
+                // 蓄積防止のため、サイズが一定値を超えたら期限切れキャッシュを一括クリーンアップ
+                if (this.buckets.size > 500) {
+                    this.cleanExpired()
+                }
             }
             return records
         })()
@@ -162,4 +166,18 @@ export class SqlResultCache {
             }
         }
     }
+
+    /**
+     * Cleans up all expired cache buckets to prevent memory accumulation.
+     * メモリー蓄積を防ぐため、期限切れのキャッシュバケットをすべてクリーンアップする処理。
+     */
+    private cleanExpired(): void {
+        const now = Date.now()
+        // 期限切れのキーを削除
+        for (const [key, bucket] of this.buckets.entries()) {
+            if (bucket.expireAt <= now) {
+                this.buckets.delete(key)
+            }
+        }
+    }
 }