@vheins/local-memory-mcp 0.1.3 → 0.1.5

package/dist/router.test.js

@@ -0,0 +1,113 @@
+// Feature: memory-mcp-optimization, Property 11: createRouter() uses provided storage
+import { describe, it, expect, vi } from "vitest";
+import * as fc from "fast-check";
+import { createRouter } from "./router.js";
+/**
+ * Property 11: createRouter() menggunakan storage yang diberikan
+ * Validates: Requirements 10.1, 10.4
+ *
+ * For any mock SQLiteStore given to createRouter(mockDb, mockVectors),
+ * all tool operations run through the router SHALL use mockDb and not access the real DB.
+ */
+describe("createRouter() — Property 11: uses provided storage", () => {
+    function makeMockDb() {
+        return {
+            insert: vi.fn(),
+            update: vi.fn(),
+            delete: vi.fn(),
+            getById: vi.fn().mockReturnValue(null),
+            searchByRepo: vi.fn().mockReturnValue([]),
+            searchBySimilarity: vi.fn().mockReturnValue([]),
+            getRecentMemories: vi.fn().mockReturnValue([]),
+            getTotalCount: vi.fn().mockReturnValue(0),
+            getSummary: vi.fn().mockReturnValue(null),
+            upsertSummary: vi.fn(),
+            listRepos: vi.fn().mockReturnValue([]),
+            listRecent: vi.fn().mockReturnValue([]),
+            incrementHitCount: vi.fn(),
+            incrementRecallCount: vi.fn(),
+            getStats: vi.fn().mockReturnValue({ total: 0, byType: {}, unused: 0 }),
+            getAllMemoriesWithStats: vi.fn().mockReturnValue([]),
+            upsertVectorEmbedding: vi.fn(),
+            getVectorEmbedding: vi.fn().mockReturnValue(null),
+            archiveExpiredMemories: vi.fn().mockReturnValue(0),
+            logQuery: vi.fn(),
+            getRecentQueries: vi.fn().mockReturnValue([]),
+            close: vi.fn(),
+        };
+    }
+    function makeMockVectors() {
+        return {
+            upsert: vi.fn().mockResolvedValue(undefined),
+            remove: vi.fn().mockResolvedValue(undefined),
+            search: vi.fn().mockResolvedValue([]),
+        };
+    }
+    it("memory-recap calls getRecentMemories on the provided mock db", async () => {
+        const mockDb = makeMockDb();
+        const mockVectors = makeMockVectors();
+        const router = createRouter(mockDb, mockVectors);
+        await router("tools/call", {
+            name: "memory-recap",
+            arguments: { repo: "test-repo", limit: 5 },
+        });
+        expect(mockDb.getRecentMemories).toHaveBeenCalledWith("test-repo", 5, 0);
+        expect(mockDb.getTotalCount).toHaveBeenCalledWith("test-repo");
+    });
+    it("memory-search calls searchBySimilarity on the provided mock db", async () => {
+        const mockDb = makeMockDb();
+        const mockVectors = makeMockVectors();
+        const router = createRouter(mockDb, mockVectors);
+        await router("tools/call", {
+            name: "memory-search",
+            arguments: { query: "test query", repo: "test-repo", limit: 5 },
+        });
+        expect(mockDb.searchBySimilarity).toHaveBeenCalled();
+        // Verify the first argument to searchBySimilarity contains the repo
+        const callArgs = mockDb.searchBySimilarity.mock.calls[0];
+        expect(callArgs[1]).toBe("test-repo");
+    });
+    it("property: for any repo string, memory-recap always uses the injected db", async () => {
+        await fc.assert(fc.asyncProperty(fc.string({ minLength: 1, maxLength: 50 }).filter((s) => s.trim().length > 0), fc.integer({ min: 1, max: 50 }), async (repo, limit) => {
+            const mockDb = makeMockDb();
+            const mockVectors = makeMockVectors();
+            const router = createRouter(mockDb, mockVectors);
+            await router("tools/call", {
+                name: "memory-recap",
+                arguments: { repo, limit },
+            });
+            // The mock db methods must have been called (not a real DB)
+            expect(mockDb.getRecentMemories).toHaveBeenCalled();
+            expect(mockDb.getTotalCount).toHaveBeenCalled();
+        }), { numRuns: 100 });
+    });
+    it("property: for any valid store args, memory-store uses the injected db", async () => {
+        await fc.assert(fc.asyncProperty(fc.record({
+            repo: fc.string({ minLength: 1, maxLength: 30 }).filter((s) => s.trim().length > 0),
+            content: fc.string({ minLength: 10, maxLength: 200 }),
+            importance: fc.integer({ min: 1, max: 5 }),
+            type: fc.constantFrom("code_fact", "decision", "mistake", "pattern"),
+        }), async ({ repo, content, importance, type }) => {
+            const mockDb = makeMockDb();
+            const mockVectors = makeMockVectors();
+            const router = createRouter(mockDb, mockVectors);
+            await router("tools/call", {
+                name: "memory-store",
+                arguments: { type, content, importance, scope: { repo } },
+            });
+            expect(mockDb.insert).toHaveBeenCalled();
+        }), { numRuns: 100 });
+    });
+    it("different router instances use their own injected db independently", () => {
+        const mockDb1 = makeMockDb();
+        const mockDb2 = makeMockDb();
+        const mockVectors = makeMockVectors();
+        const router1 = createRouter(mockDb1, mockVectors);
+        const router2 = createRouter(mockDb2, mockVectors);
+        // Both routers are distinct functions
+        expect(router1).not.toBe(router2);
+        // Each router closes over its own db
+        // (verified by the property tests above that each mock is called independently)
+    });
+});
+//# sourceMappingURL=router.test.js.map

package/dist/storage/sqlite.d.ts

@@ -0,0 +1,95 @@
+import { MemoryEntry } from "../types.js";
+export declare class SQLiteStore {
+    private db;
+    constructor(dbPath?: string);
+    private migrate;
+    insert(entry: MemoryEntry): void;
+    update(id: string, updates: {
+        title?: string;
+        content?: string;
+        importance?: number;
+    }): void;
+    searchByRepo(repo: string, options?: {
+        types?: string[];
+        minImportance?: number;
+        limit?: number;
+    }): MemoryEntry[];
+    getById(id: string): MemoryEntry | null;
+    getByIdWithStats(id: string): (MemoryEntry & {
+        recall_rate: number;
+    }) | null;
+    listRecent(limit?: number): Array<{
+        id: string;
+        type: string;
+        repo: string;
+    }>;
+    getSummary(repo: string): {
+        summary: string;
+        updated_at: string;
+    } | null;
+    upsertSummary(repo: string, summary: string): void;
+    delete(id: string): void;
+    listRepos(): string[];
+    listRepoNavigation(): Array<{
+        repo: string;
+        memory_count: number;
+        last_updated_at: string | null;
+    }>;
+    incrementHitCount(id: string): void;
+    incrementRecallCount(id: string): void;
+    getStats(repo?: string): {
+        total: number;
+        byType: Record<string, number>;
+        unused: number;
+    };
+    getRecentMemories(repo: string, limit: number, offset?: number): MemoryEntry[];
+    getTotalCount(repo: string): number;
+    searchBySimilarity(query: string, repo: string, limit?: number): Array<MemoryEntry & {
+        similarity: number;
+    }>;
+    archiveExpiredMemories(): number;
+    getAllMemoriesWithStats(repo?: string): Array<MemoryEntry & {
+        hit_count: number;
+        recall_count: number;
+        recall_rate: number;
+        last_used_at: string | null;
+    }>;
+    listMemoriesForDashboard(options: {
+        repo: string;
+        type?: string;
+        search?: string;
+        minImportance?: number;
+        maxImportance?: number;
+        sortBy?: string;
+        sortOrder?: "asc" | "desc";
+        limit?: number;
+        offset?: number;
+    }): {
+        items: Array<MemoryEntry & {
+            hit_count: number;
+            recall_count: number;
+            recall_rate: number;
+            last_used_at: string | null;
+        }>;
+        total: number;
+    };
+    upsertVectorEmbedding(memoryId: string, vector: number[] | string[]): void;
+    getVectorEmbedding(memoryId: string): number[] | string[] | null;
+    close(): void;
+    private computeVector;
+    private cosineSimilarity;
+    private rowToMemoryEntry;
+    logAction(action: 'search' | 'read' | 'write' | 'update' | 'delete', repo: string, options?: {
+        query?: string;
+        memoryId?: string;
+        resultCount?: number;
+    }): void;
+    getRecentActions(repo?: string, limit?: number): Array<{
+        action: string;
+        query?: string;
+        memory_id?: string;
+        result_count?: number;
+        created_at: string;
+    }>;
+}
+//# sourceMappingURL=sqlite.d.ts.map

package/dist/storage/sqlite.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"sqlite.d.ts","sourceRoot":"","sources":["../../src/storage/sqlite.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,WAAW,EAAe,MAAM,aAAa,CAAC;AAMvD,qBAAa,WAAW;IACtB,OAAO,CAAC,EAAE,CAAoB;gBAElB,MAAM,CAAC,EAAE,MAAM;IAK3B,OAAO,CAAC,OAAO;IA0Gf,MAAM,CAAC,KAAK,EAAE,WAAW,GAAG,IAAI;IA2BhC,MAAM,CAAC,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,CAAC,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI;IAyC5F,YAAY,CACV,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE;QACP,KAAK,CAAC,EAAE,MAAM,EAAE,CAAC;QACjB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,KAAK,CAAC,EAAE,MAAM,CAAC;KACX,GACL,WAAW,EAAE;IA6BhB,OAAO,CAAC,EAAE,EAAE,MAAM,GAAG,WAAW,GAAG,IAAI;IAOvC,gBAAgB,CAAC,EAAE,EAAE,MAAM,GAAG,CAAC,WAAW,GAAG;QAAE,WAAW,EAAE,MAAM,CAAA;KAAE,CAAC,GAAG,IAAI;IAgB5E,UAAU,CAAC,KAAK,GAAE,MAAW,GAAG,KAAK,CAAC;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IAUjF,UAAU,CAAC,IAAI,EAAE,MAAM,GAAG;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI;IAKxE,aAAa,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,IAAI;IAWlD,MAAM,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAMxB,SAAS,IAAI,MAAM,EAAE;IAMrB,kBAAkB,IAAI,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,YAAY,EAAE,MAAM,CAAC;QAAC,eAAe,EAAE,MAAM,GAAG,IAAI,CAAA;KAAE,CAAC;IAcnG,iBAAiB,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAUnC,oBAAoB,CAAC,EAAE,EAAE,MAAM,GAAG,IAAI;IAUtC,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG;QACvB,KAAK,EAAE,MAAM,CAAC;QACd,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAC/B,MAAM,EAAE,MAAM,CAAC;KAChB;IAkCD,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,GAAE,MAAU,GAAG,WAAW,EAAE;IAYjF,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM;IAOnC,kBAAkB,CAChB,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,KAAK,GAAE,MAAW,GACjB,KAAK,CAAC,WAAW,GAAG;QAAE,UAAU,EAAE,MAAM,CAAA;KAAE,CAAC;IA6C9C,sBAAsB,IAAI,MAAM;IA4BhC,uBAAuB,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,KAAK,CAC3C,WAAW,GAAG;QACZ,SAAS,EAAE,MAAM,CAAC;QAClB,YAAY,EAAE,MAAM,CAAC;QACrB,WAAW,EAAE,MAAM,CAAC;QACpB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;KAC7B,CACF;IAuBD,wBAAwB,CAAC,OAAO,EAAE;QAChC,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,CAAC,EAAE,MAAM,CAAC;QACd,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,aAAa,CAAC,EAAE,MAAM,CAAC;QACvB,MAAM,CAAC,EAAE,MAAM,CAAC;QAChB,SAAS,CAAC,EAAE,KAAK,GAAG,MAAM,CAAC;QAC3B,KAAK,CAAC,EAAE,MAAM,CAAC;QACf,MAAM,CAAC,EAAE,MAAM,CAAC;KACjB,GAAG;QACF,KAAK,EAAE,KAAK,CACV,WAAW,GAAG;YACZ,SAAS,EAAE,MAAM,CAAC;YAClB,YAAY,EAAE,MAAM,CAAC;YACrB,WAAW,EAAE,MAAM,CAAC;YACpB,YAAY,EAAE,MAAM,GAAG,IAAI,CAAC;SAC7B,CACF,CAAC;QACF,KAAK,EAAE,MAAM,CAAC;KACf;IAqED,qBAAqB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,GAAG,IAAI;IAW1E,kBAAkB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,EAAE,GAAG,MAAM,EAAE,GAAG,IAAI;IAWhE,KAAK,IAAI,IAAI;IAKb,OAAO,CAAC,aAAa;IASrB,OAAO,CAAC,gBAAgB;IAwBxB,OAAO,CAAC,gBAAgB;IAqBxB,SAAS,CAAC,MAAM,EAAE,QAAQ,GAAG,MAAM,GAAG,OAAO,GAAG,QAAQ,GAAG,QAAQ,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE;QAAE,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE;IAexJ,gBAAgB,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,KAAK,SAAK,GAAG,KAAK,CAAC;QAAE,MAAM,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAC;QAAC,SAAS,CAAC,EAAE,MAAM,CAAC;QAAC,YAAY,CAAC,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,MAAM,CAAA;KAAE,CAAC;CAgBtJ"}