@tpsdev-ai/pi-flair 0.17.0 → 0.19.0

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,+BAA+B,CAAC;AAEpF,OAAO,EAA2B,KAAK,iBAAiB,EAAwB,MAAM,yBAAyB,CAAC;AAIhH,UAAU,YAAa,SAAQ,iBAAiB;IAC9C,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAkFD;;;;GAIG;AACH,wBAAgB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAoBpE;AAQD,MAAM,CAAC,OAAO,WAAW,EAAE,EAAE,YAAY,QAqNxC;AAID,wBAAsB,eAAe,CACnC,MAAM,EAAE,YAAY,EACpB,SAAS,SAAO,GACf,OAAO,CAAC,MAAM,CAAC,CAIjB"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AAEH,OAAO,KAAK,EAAE,YAAY,EAAoB,MAAM,+BAA+B,CAAC;AAEpF,OAAO,EAA2B,KAAK,iBAAiB,EAAwB,MAAM,yBAAyB,CAAC;AAIhH,UAAU,YAAa,SAAQ,iBAAiB;IAC9C,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,WAAW,CAAC,EAAE,OAAO,CAAC;CACvB;AAkFD;;;;GAIG;AACH,wBAAgB,aAAa,CAAC,GAAG,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAoBpE;AAQD,MAAM,CAAC,OAAO,WAAW,EAAE,EAAE,YAAY,QAkNxC;AAID,wBAAsB,eAAe,CACnC,MAAM,EAAE,YAAY,EACpB,SAAS,SAAO,GACf,OAAO,CAAC,MAAM,CAAC,CAIjB"}

package/dist/index.js

@@ -196,35 +196,33 @@ export default function (pi) {
                     dedup: true,
                     dedupThreshold: 0.95,
                 });
-                // Dedup signal: use the explicit `deduped` flag that flair-client
-                // sets on the returned record. The previous prefix-match check
-                // (`!result.id.startsWith(`${agentId}-`)`) was unreliable — when
-                // a dedup hit returned an existing memory from the SAME agent,
-                // both IDs start with the same prefix and the check silently
-                // returned the success path, dropping the new content. Same fix
-                // class as #358 (P0 silent data loss) applied to flair-mcp;
-                // pi-flair was missed. See flair#449 (filed by a Claude Code
-                // session that hit this exact failure).
-                const wasDeduped = result.deduped === true;
-                if (wasDeduped) {
-                    return {
-                        content: [{
-                                type: "text",
-                                text: `Similar memory already exists (id: ${result.id}): ${result.content?.slice(0, 200)}\n(no new entry written)`,
-                            }],
-                        details: { deduplicated: true, mergedWith: result.id },
-                    };
-                }
+                // Dedup signal: the server's conservative gate NEVER suppresses a
+                // write (memory-integrity fix, flair#526) — `result.deduplicated`
+                // is a collision SIGNAL, not a "was this dropped" flag. The content
+                // is always written; when flagged, `result.matchedId` names the
+                // similar existing memory. (Historical note: this tool used to check
+                // the now-removed `result.deduped` flag from a client-side gate that
+                // DID suppress the write, and told the user "no new entry written" —
+                // that was the #449/#526 silent-data-loss failure mode. Both the
+                // suppression and the misleading text are gone.)
+                const wasDeduplicated = result.deduplicated === true;
+                const matchedId = result.matchedId;
                 const preview = content.length > 120 ? content.slice(0, 120) + "..." : content;
                 const tagStr = params.tags?.length ? params.tags.join(", ") : "none";
-                const text = [
+                const lines = [
                     `Memory stored (id: ${result.id})`,
                     `Preview: ${preview}`,
                     `Size: ${content.length} chars`,
                     `Tags: ${tagStr}`,
                     `Durability: ${durability}`,
-                ].join("\n");
-                return { content: [{ type: "text", text }], details: { deduplicated: false, id: result.id } };
+                ];
+                if (wasDeduplicated && matchedId) {
+                    lines.push(`Note: similar to existing memory (id: ${matchedId}) — both are kept.`);
+                }
+                return {
+                    content: [{ type: "text", text: lines.join("\n") }],
+                    details: { deduplicated: wasDeduplicated, id: result.id, written: true, ...(wasDeduplicated ? { matchedId } : {}) },
+                };
             }
             catch (err) {
                 return errorResult(err, flair.url);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tpsdev-ai/pi-flair",
-  "version": "0.17.0",
+  "version": "0.19.0",
   "description": "Flair memory extension for pi — persistent memory access from within pi sessions",
   "type": "module",
   "main": "dist/index.js",
@@ -21,7 +21,7 @@
     "node": ">=18"
   },
   "dependencies": {
-    "@tpsdev-ai/flair-client": "0.17.0",
+    "@tpsdev-ai/flair-client": "0.19.0",
     "@sinclair/typebox": "0.34.48"
   },
   "license": "Apache-2.0",