openclaw-memory-alibaba-local 1.0.9 → 1.0.11

package/index.ts

@@ -843,47 +843,60 @@ async function extractWorldImageWithLLM(
       }>;
     };
     const list = Array.isArray(parsed.actions) ? parsed.actions : [];
-    const resultMap = new Map<number, WorldImageAction[]>();
+    // Per-index: accumulate deletes separately from the final action (insert/update/skip)
+    const deleteMap = new Map<number, WorldImageAction[]>();  // delete actions per index
+    const finalMap = new Map<number, WorldImageAction>();     // final action per index (insert/update/skip)
     // Default: insert for each index
     for (let i = 0; i < newItems.length; i++) {
-      resultMap.set(i, [{
+      finalMap.set(i, {
         action: "insert" as const,
         text: newItems[i]!.text,
         importance: newItems[i]!.importance,
-      }]);
+      });
     }
 
     for (const a of list) {
       const idx = typeof a.index === "number" ? a.index : -1;
       if (idx < 0 || idx >= newItems.length) continue;
 
-      if (a.action === "skip") {
-        resultMap.set(idx, [{ action: "skip" }]);
-      } else if (a.action === "delete" && typeof a.memoryId === "string" && existingIdSet.has(a.memoryId)) {
-        const existing = resultMap.get(idx) ?? [];
-        resultMap.set(idx, [{ action: "delete", memoryId: a.memoryId }, ...existing.filter(x => x.action !== "skip")]);
+      if (a.action === "delete" && typeof a.memoryId === "string" && existingIdSet.has(a.memoryId)) {
+        // Accumulate deletes — they don't replace the final action
+        const dels = deleteMap.get(idx) ?? [];
+        dels.push({ action: "delete", memoryId: a.memoryId });
+        deleteMap.set(idx, dels);
+      } else if (a.action === "skip") {
+        finalMap.set(idx, { action: "skip" });
       } else if (a.action === "update" && typeof a.memoryId === "string" && existingIdSet.has(a.memoryId)) {
         const text = typeof a.text === "string" ? a.text.trim() : "";
         if (text.length < 10) continue;
-        resultMap.set(idx, [{
+        finalMap.set(idx, {
           action: "update",
           memoryId: a.memoryId,
           text,
           importance: clampImportance(a.importance ?? newItems[idx]!.importance),
-        }]);
+        });
       } else if (a.action === "insert") {
         const text = typeof a.text === "string" ? a.text.trim() : "";
         const finalText = text.length >= 10 ? text : newItems[idx]!.text;
-        resultMap.set(idx, [{
+        finalMap.set(idx, {
           action: "insert",
           text: finalText,
           importance: clampImportance(a.importance ?? newItems[idx]!.importance),
-        }]);
+        });
       }
     }
+    // Assemble: deletes first, then final action per index
     const result: WorldImageAction[] = [];
     for (let i = 0; i < newItems.length; i++) {
-      result.push(...(resultMap.get(i) ?? []));
+      const dels = deleteMap.get(i);
+      const final = finalMap.get(i);
+      // If SKIP + deletes for same index: honor SKIP (don't delete), per user rule
+      if (final?.action === "skip") {
+        result.push(final);
+      } else {
+        if (dels) result.push(...dels);
+        if (final) result.push(final);
+      }
     }
     return result;
   } catch (err: unknown) {
@@ -1337,6 +1350,7 @@ async function captureUserMemoryFromInboundTexts(
   
     // 4. Execute actions
     let insertCount = 0;
+    let deleteCount = 0;
     for (const action of worldActions) {
       if (action.action === "skip") continue;
   
@@ -1344,6 +1358,7 @@ async function captureUserMemoryFromInboundTexts(
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
           await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
+          deleteCount++;
         }
         continue;
       }
@@ -1352,6 +1367,7 @@ async function captureUserMemoryFromInboundTexts(
         const hit = existingCandidates.find((c) => c.entry.id === action.memoryId);
         if (hit) {
           await deleteSimilarLogicalMemory(db, agentId, hit.entry.sessionId, hit);
+          deleteCount++;
         }
         const { vectors } = await backend.encodeForStorage(action.text);
         const rows = buildChunkRows(
@@ -1373,7 +1389,8 @@ async function captureUserMemoryFromInboundTexts(
         insertCount++;
       }
     }
-    console.log(`[openclaw-memory-alibaba-local] worldImageExtraction done: ${worldActions.length} actions, ${insertCount} stored`);
+
+    console.log(`[openclaw-memory-alibaba-local] worldImageExtraction done: ${worldActions.length} actions, ${insertCount} stored, ${deleteCount} deleted`);
   
     // 5. World fact LRU GC: every 10 insertions
     if (insertCount > 0) {
@@ -1597,13 +1614,11 @@ function buildChunkRows(
 async function deleteSimilarLogicalMemory(
   db: MemoryDB,
   agentId: string,
-  sessionId: string | null | undefined,
+  _sessionId: string | null | undefined,
   hit: MemorySearchResult,
 ): Promise<void> {
-  // 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(() => {});
+  console.log(`[openclaw-memory-alibaba-local] deleteSimilarLogicalMemory id=${hit.entry.id} agentId=${agentId} text=${hit.entry.text.slice(0, 60)}`);
+  await db.delete(agentId, hit.entry.id);
 }
 
 async function storeOneCaptureItem(
@@ -2177,19 +2192,22 @@ const memoryPlugin = {
         const isUserTrigger = trigger === "user";
 
         // Skip memory extraction for system/bootstrap sessions (/new, /reset).
-        // Check the last user message for known bootstrap patterns.
+        // Check ANY user message for known bootstrap patterns (not just last).
         if (isUserTrigger) {
-          const lastUserMsg = [...event.messages].reverse().find(
-            (m: any) => m && typeof m === "object" && (m as Record<string, unknown>).role === "user",
-          ) as Record<string, unknown> | undefined;
-          const lastUserText = typeof lastUserMsg?.content === "string" ? lastUserMsg.content.toLowerCase() : "";
-          if (
-            lastUserText.includes("a new session was started") ||
-            lastUserText.includes("session startup sequence") ||
-            lastUserText.includes("/new or /reset") ||
-            lastUserText.startsWith("system:") ||
-            lastUserText.startsWith("run your session")
-          ) {
+          const isBootstrapSession = event.messages.some((m: any) => {
+            if (!m || typeof m !== "object") return false;
+            const text = typeof (m as Record<string, unknown>).content === "string"
+              ? ((m as Record<string, unknown>).content as string).toLowerCase()
+              : "";
+            return (
+              text.includes("a new session was started") ||
+              text.includes("session startup sequence") ||
+              text.includes("/new or /reset") ||
+              text.startsWith("system:") ||
+              text.startsWith("run your session")
+            );
+          });
+          if (isBootstrapSession) {
             console.log("[openclaw-memory-alibaba-local] agent_end skip capture (system/bootstrap session)");
             return;
           }

package/package.json

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

package/prompts.ts

@@ -267,7 +267,7 @@ Only INSERT or UPDATE information that captures a concrete, verifiable fact or e
 - Purely conversational filler with no informational content
 
 # Reconciliation Principles
-1. **Prefer the richer version**: When a batch item and a Store item describe the same topic, keep whichever has the most information. If the batch adds new details, UPDATE.
+1. **Prefer the richer version**: When a batch item and a Store item describe the same topic, keep whichever has the most information. If the batch is richer, DELETE the old Store item and INSERT the batch item. If they are roughly equal, UPDATE to merge details.
 2. **Preserve temporal markers**: Keep [as of ...] or [date] prefixes — world facts are time-sensitive.
 3. **High cohesion**: Only merge entries about the exact same event or fact. Different events stay separate even if related.
 4. **Contradiction = replace**: If a batch item directly contradicts a Store item (e.g. different outcome), DELETE the old item and INSERT the new one.
@@ -275,8 +275,8 @@ Only INSERT or UPDATE information that captures a concrete, verifiable fact or e
 # Actions (one per batch index)
 - **INSERT**: New world fact not in Store.
 - **UPDATE** (memoryId): Store item covers the same event/fact; merge to produce the richer version.
-- **SKIP**: Already fully covered by Store, or not a concrete fact.
-- **DELETE** (memoryId): Batch item contradicts a Store item. Delete old; INSERT new.
+- **SKIP**: Already fully covered by a Store item (old is sufficient). Do NOT combine SKIP with DELETE for the same index — if you skip the new item, you are keeping the old one.
+- **DELETE** (memoryId) + **INSERT**: Batch item supersedes or contradicts a Store item. You MUST delete the old item AND insert the replacement. A single index may list one or more DELETE actions followed by one INSERT.
 
 # Output
 Reply with ONLY a JSON object:
@@ -284,9 +284,11 @@ Reply with ONLY a JSON object:
   {"index":0,"action":"insert","text":"[2026-04-01] Project X launched v2.0","importance":0.5},
   {"index":1,"action":"skip"},
   {"index":2,"action":"update","memoryId":"uuid","text":"[2026-03-28] Company Y acquired Z for $2B, deal finalized","importance":0.5},
-  {"index":3,"action":"delete","memoryId":"uuid"}
+  {"index":3,"action":"delete","memoryId":"uuid1"},
+  {"index":3,"action":"delete","memoryId":"uuid2"},
+  {"index":3,"action":"insert","text":"[2026-04-02] Replacement fact","importance":0.5}
 ]}
-Every batch index must appear. A single index may produce BOTH delete + insert.
+Every batch index must appear exactly once as the primary action. When replacing, list DELETE(s) first then INSERT for that index.
 `;
 
 export function buildWorldImageExtractionPrompt(