fix(memory): reuse full qmd manager for status

neeravmakwana · neeravmakwana · commit 08a0bac627db · 2026-03-24T09:01:00.000-04:00
diff --git a/src/memory/search-manager.test.ts b/src/memory/search-manager.test.ts
@@ -216,6 +216,20 @@ describe("getMemorySearchManager caching", () => {
     expect(second.manager).toBe(first.manager);
   });
 
+  it("reuses cached full qmd manager for status-only requests", async () => {
+    const agentId = "status-reuses-full-agent";
+    const cfg = createQmdCfg(agentId);
+
+    const full = await getMemorySearchManager({ cfg, agentId });
+    const status = await getMemorySearchManager({ cfg, agentId, purpose: "status" });
+
+    requireManager(full);
+    requireManager(status);
+    expect(status.manager).toBe(full.manager);
+    // eslint-disable-next-line @typescript-eslint/unbound-method
+    expect(createQmdManagerMock).toHaveBeenCalledTimes(1);
+  });
+
   it("does not evict a newer cached wrapper when closing an older failed wrapper", async () => {
     const retryAgentId = "retry-agent-close";
     const {
diff --git a/src/memory/search-manager.ts b/src/memory/search-manager.ts
@@ -52,6 +52,15 @@ export async function getMemorySearchManager(params: {
     if (cached) {
       return { manager: cached };
     }
+    if (statusOnly) {
+      // Reuse an existing full manager for status requests to avoid duplicate
+      // QMD managers and extra sqlite handles for the same agent/config.
+      const fullCached = QMD_MANAGER_CACHE.get(`${baseCacheKey}:full`);
+      if (fullCached) {
+        QMD_MANAGER_CACHE.set(cacheKey, fullCached);
+        return { manager: fullCached };
+      }
+    }
     try {
       const { QmdMemoryManager } = await import("./qmd-manager.js");
       const primary = await QmdMemoryManager.create({
