openclaw-memory-alibaba-local 1.0.7 → 1.0.9

package/index.ts

@@ -1298,12 +1298,27 @@ async function captureUserMemoryFromInboundTexts(
       embeddingResults.push({ item: { ...item, text: truncated }, vectors });
     }
   
-    // 2. Recall top-3 similar existing world_facts per item, merge & dedup (cap ~10)
-    const allVectors = embeddingResults.flatMap((r) => r.vectors);
+    // 2. Per-item recall: for each event item recall top-3 similar existing world_facts, then dedup
     const recallMinScore = Math.max(0.5, cfg.similarityThresholdUserMemory - 0.35);
-    const existingCandidates = allVectors.length > 0
-      ? await db.searchMerged(agentId, allVectors, 3, recallMinScore, [WORLD_FACT])
-      : [];
+    const PER_ITEM_RECALL = 3;
+    const candidateMap = new Map<string, MemorySearchResult>();
+    for (const er of embeddingResults) {
+      const perItemHits = er.vectors.length > 0
+        ? await db.searchMerged(agentId, er.vectors, PER_ITEM_RECALL, recallMinScore, [WORLD_FACT])
+        : [];
+      for (const h of perItemHits) {
+        const key = `${String(h.entry.category)}\0${h.entry.text}`;
+        const prev = candidateMap.get(key);
+        if (!prev || h.score > prev.score) {
+          candidateMap.set(key, h);
+        }
+      }
+    }
+    const existingCandidates = [...candidateMap.values()].sort((a, b) => b.score - a.score);
+    console.log(`[openclaw-memory-alibaba-local] worldImageExtraction recall: ${embeddingResults.length} items × ${PER_ITEM_RECALL} = raw ${embeddingResults.length * PER_ITEM_RECALL}, deduped ${existingCandidates.length}, minScore=${recallMinScore}`);
+    if (existingCandidates.length > 0) {
+      console.log(`[openclaw-memory-alibaba-local] worldImageExtraction recall found ${existingCandidates.length} candidates: ${existingCandidates.map((c) => `[${c.score.toFixed(3)}] ${c.entry.text.slice(0, 60)}`).join(" | ")}`);
+    }
   
     // 3. LLM CRUD decision
     console.log(`[openclaw-memory-alibaba-local] worldImageExtraction input: ${eventItems.length} event items, ${existingCandidates.length} existing candidates`);
@@ -1328,7 +1343,7 @@ async function captureUserMemoryFromInboundTexts(
       if (action.action === "delete") {
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
-          await deleteSimilarLogicalMemory(db, agentId, sessionKey, hit);
+          await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
         }
         continue;
       }
@@ -1336,7 +1351,7 @@ async function captureUserMemoryFromInboundTexts(
       if (action.action === "update") {
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
-          await deleteSimilarLogicalMemory(db, agentId, sessionKey, hit);
+          await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
         }
         const { vectors } = await backend.encodeForStorage(action.text);
         const rows = buildChunkRows(
@@ -1400,9 +1415,13 @@ async function captureUserMemoryFromInboundTexts(
     // 2. Recall top-10 similar existing memories for ALL new extractions (agentId global, USER_MEMORY scope)
     const allVectors = embeddingResults.flatMap((r) => r.vectors);
     const recallMinScore = Math.max(0.5, cfg.similarityThresholdUserMemory - 0.35);
+    console.log(`[openclaw-memory-alibaba-local] userImageExtraction recall: ${allVectors.length} query vectors, minScore=${recallMinScore}`);
     const existingCandidates = allVectors.length > 0
       ? await db.searchMerged(agentId, allVectors, 10, recallMinScore, [...USER_MEMORY_CATEGORIES])
       : [];
+    if (existingCandidates.length > 0) {
+      console.log(`[openclaw-memory-alibaba-local] userImageExtraction recall found ${existingCandidates.length} candidates: ${existingCandidates.map((c) => `[${c.score.toFixed(3)}] ${c.entry.text.slice(0, 60)}`).join(" | ")}`);
+    }
 
     // 3. Call user image extraction LLM
     console.log(`[openclaw-memory-alibaba-local] userImageExtraction input: ${userItems.length} user items (${eventItems.length} event items bypassed), ${existingCandidates.length} existing candidates`);
@@ -1428,7 +1447,7 @@ async function captureUserMemoryFromInboundTexts(
       if (action.action === "delete") {
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
-          await deleteSimilarLogicalMemory(db, agentId, sessionKey, hit);
+          await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
         }
         continue;
       }
@@ -1436,7 +1455,7 @@ async function captureUserMemoryFromInboundTexts(
       if (action.action === "update") {
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
-          await deleteSimilarLogicalMemory(db, agentId, sessionKey, hit);
+          await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
         }
         const { vectors } = await backend.encodeForStorage(action.text);
         const rows = buildChunkRows(
@@ -1581,10 +1600,10 @@ async function deleteSimilarLogicalMemory(
   sessionId: string | null | undefined,
   hit: MemorySearchResult,
 ): Promise<void> {
-  const n = await db.deleteByAgentSessionCategoryText(agentId, sessionId, hit.entry.category, hit.entry.text);
-  if (n === 0) {
-    await db.delete(agentId, hit.entry.id);
-  }
+  // Try text-match delete first (covers all chunks), then ID-based fallback.
+  // deleteByAgentSessionCategoryText returns optimistic 1, so always also try by ID.
+  await db.deleteByAgentSessionCategoryText(agentId, sessionId, hit.entry.category, hit.entry.text);
+  await db.delete(agentId, hit.entry.id).catch(() => {});
 }
 
 async function storeOneCaptureItem(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openclaw-memory-alibaba-local",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "OpenClaw memory plugin: local LanceDB + DashScope-compatible embeddings",
   "type": "module",
   "engines": {