@eleboucher/pi-memini 0.5.7 → 0.5.9

package/dist/index.js

@@ -229,8 +229,12 @@ function createClient(cfg, warn) {
                 signal: AbortSignal.timeout(cfg.timeout_ms),
             });
             if (!res.ok) {
-                if (cfg.fallback_on_error)
+                if (cfg.fallback_on_error) {
+                    // Degrade but never silently: a swallowed 401/500 on a capture or
+                    // recall looks like "memory isn't working" with nothing to debug.
+                    warn(`memini ${method} ${path} failed: ${res.status}`);
                     return null;
+                }
                 const text = await res.text().catch(() => "");
                 throw new Error(`memini ${method} ${path} failed: ${res.status} ${text}`);
             }
@@ -348,6 +352,28 @@ export default function meminiExtension(pi) {
     // Assistant message ids already captured, so a re-fired agent_end never writes
     // a duplicate turn.
     const captured = new Set();
+    // Memory ids each session has already been shown (mirrors the openclaw
+    // plugin): the recall injection is a persistent context message, so
+    // re-injecting an unchanged match every turn stacks identical blocks in the
+    // prompt. Bounded so long-lived hosts can't grow the map without limit.
+    const injectedBySession = new Map();
+    const MAX_TRACKED_SESSIONS = 200;
+    const rememberInjected = (session, ids) => {
+        let seen = injectedBySession.get(session);
+        if (!seen) {
+            seen = new Set();
+            injectedBySession.set(session, seen);
+            while (injectedBySession.size > MAX_TRACKED_SESSIONS) {
+                const oldest = injectedBySession.keys().next().value;
+                if (oldest === undefined)
+                    break;
+                injectedBySession.delete(oldest);
+            }
+        }
+        for (const id of ids)
+            if (id)
+                seen.add(id);
+    };
     // Recall before the turn: search for the user's prompt and inject the matches
     // as a persistent context message. Buffer the prompt for capture at agent_end.
     pi.on("before_agent_start", async (event, ctx) => {
@@ -366,7 +392,14 @@ export default function meminiExtension(pi) {
             body.min_score = cfg.recall_min_score;
         const result = await client.postJson("/v1/search", body);
         const floor = cfg.recall_min_score > 0 ? cfg.recall_min_score : 0;
-        const rawHits = Array.isArray(result?.results) ? result.results : [];
+        let rawHits = Array.isArray(result?.results) ? result.results : [];
+        // Suppress memories this session has already been shown — the injected
+        // message persists in context, so a repeat adds nothing but noise.
+        if (sid) {
+            const seen = injectedBySession.get(sid);
+            if (seen?.size)
+                rawHits = rawHits.filter((r) => !seen.has(r?.memory?.id));
+        }
         const filtered = floor > 0
             ? rawHits.filter((r) => (typeof r?.score === "number" ? r.score : 0) >= floor)
             : rawHits;
@@ -376,6 +409,9 @@ export default function meminiExtension(pi) {
         const fit = fitByTokens(hits, cfg.recall_max_tokens);
         if (fit.items.length === 0)
             return;
+        if (sid) {
+            rememberInjected(sid, filtered.map((r) => r?.memory?.id).filter(Boolean));
+        }
         const lines = [
             "Relevant long-term memory from memini (background context — prefer " +
                 "current workspace state and the user's instructions):",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@eleboucher/pi-memini",
-  "version": "0.5.7",
+  "version": "0.5.9",
   "description": "Shared cross-session memory for the Pi coding agent, backed by a memini service.",
   "files": [
     "dist"