@tpsdev-ai/openclaw-flair 0.3.0 → 0.4.1

package/index.ts

@@ -87,13 +87,19 @@ async function syncWorkspaceToFlair(
 
 // ─── Auto-capture helpers ─────────────────────────────────────────────────────
 
+// Auto-capture triggers — conservative patterns that indicate genuinely
+// important context, not casual conversation. The LLM has memory_store
+// for explicit saves; auto-capture is a safety net for things it misses.
 const CAPTURE_TRIGGERS = [
-  /\b(remember|note that|important:|keep in mind|don't forget)\b/i,
-  /\b(preference|prefer|always|never|my name is|i am|i'm)\b/i,
-  /\b(decided|agreed|confirmed|finalized)\b/i,
+  /\b(remember this|note for future|important lesson|key decision|for the record)\b/i,
+  /\b(my name is|call me|i go by)\b/i,
+  /\b(we decided|final decision|agreed to|commitment:)\b/i,
 ];
 
+const MIN_CAPTURE_LENGTH = 30; // skip very short messages
+
 function shouldCapture(text: string): boolean {
+  if (text.length < MIN_CAPTURE_LENGTH) return false;
   return CAPTURE_TRIGGERS.some((re) => re.test(text));
 }
 
@@ -118,8 +124,8 @@ export default {
     const fallbackAgentId = resolveAgentId();
 
     function getClient(agentId?: string): FlairClient {
-      const id = agentId || cfg.agentId || fallbackAgentId;
-      if (!id || id === "auto") throw new Error("no agentId available — set agentId in plugin config, FLAIR_AGENT_ID env var, or ensure OpenClaw provides it via session context");
+      const id = agentId || (cfg.agentId && cfg.agentId !== "auto" ? cfg.agentId : null) || currentAgentId || fallbackAgentId;
+      if (!id || id === "auto") throw new Error("no agentId available — set agentId in plugin config, FLAIR_AGENT_ID env var, or ensure OpenClaw provides it via session context (before_agent_start)");
       let client = clientPool.get(id);
       if (!client) {
         client = new FlairClient({
@@ -144,14 +150,27 @@ export default {
       api.logger.info("openclaw-flair: auto mode — agentId will be resolved from session context");
     }
     const maxRecall = cfg.maxRecallResults ?? DEFAULT_MAX_RECALL;
-    const autoCapture = cfg.autoCapture ?? true;
+    const autoCapture = cfg.autoCapture ?? false; // opt-in — trust the LLM to use memory_store
     const autoRecall = cfg.autoRecall ?? true;
 
-    let currentAgentId: string | undefined = isAutoMode ? fallbackAgentId ?? undefined : cfg.agentId;
+    // Per-session agentId — resolved from session context at runtime.
+    // In auto mode, each session gets its own agentId from before_agent_start.
+    // With explicit config, all sessions use the configured agentId.
+    let currentAgentId: string | undefined = isAutoMode ? (fallbackAgentId ?? undefined) : cfg.agentId;
+    const configuredAgentId = cfg.agentId && cfg.agentId !== "auto" ? cfg.agentId : null;
 
     api.on("before_agent_start", async (event: any, ctx: any) => {
       const eventAgentId = ctx?.agentId || (event as any).agentId;
-      if (eventAgentId) currentAgentId = eventAgentId;
+      if (!eventAgentId) return;
+
+      if (isAutoMode) {
+        // Auto mode: always adopt the session's agentId — each session is its own agent
+        currentAgentId = eventAgentId;
+        api.logger.info(`openclaw-flair: session agentId="${eventAgentId}"`);
+      } else if (eventAgentId === configuredAgentId) {
+        // Explicit mode: only accept matching agentId
+        currentAgentId = eventAgentId;
+      }
 
       if (eventAgentId) {
         try {
@@ -242,15 +261,37 @@ export default {
           try {
             const client = getCurrentClient();
             const memId = `${client.agentId}-${Date.now()}`;
-            await client.memory.write(text, {
+            // If superseding an old memory, archive it
+            if (supersedes) {
+              try {
+                const old = await client.memory.get(supersedes);
+                if (old) {
+                  await client.request("PUT", `/Memory/${supersedes}`, {
+                    ...old,
+                    archived: true,
+                    archivedAt: new Date().toISOString(),
+                    supersededBy: memId,
+                  });
+                }
+              } catch {
+                // Old memory not found — continue with the write
+              }
+            }
+
+            const result = await client.memory.write(text, {
               id: memId,
               tags,
               durability: durability as any,
               type: type as any,
+              dedup: !supersedes, // skip dedup when explicitly superseding
+              dedupThreshold: 0.7,
             });
+            const wasDeduped = result.id !== memId;
             return {
-              content: [{ type: "text", text: `Memory stored (id: ${memId})` }],
-              details: { id: memId },
+              content: [{ type: "text", text: wasDeduped
+                ? `Similar memory already exists (id: ${result.id}): ${result.content?.slice(0, 200)}`
+                : `Memory stored (id: ${memId})` }],
+              details: { id: result.id, deduplicated: wasDeduped },
             };
           } catch (err: any) {
             api.logger.warn(`openclaw-flair: store failed: ${err.message}`);

package/key-resolver.ts

@@ -2,36 +2,22 @@
  * OpenClaw-specific identity resolution.
  *
  * Key path and private key loading are now handled by @tpsdev-ai/flair-client.
- * This file only contains resolveAgentId() which reads OpenClaw config — something
- * the generic flair-client shouldn't know about.
+ * This file only contains resolveAgentId() which reads environment variables —
+ * something the generic flair-client shouldn't know about.
  */
 
-import { existsSync, readFileSync } from "node:fs";
-import { homedir } from "node:os";
-import { resolve } from "node:path";
-
 /**
- * Resolve agent ID when not explicitly configured.
- * Priority: FLAIR_AGENT_ID env > OpenClaw config file > null
+ * Resolve agent ID from environment when not explicitly configured.
+ *
+ * IMPORTANT: Does NOT read from openclaw.json agents list. On a multi-agent
+ * gateway, the first agent in the list is NOT necessarily the current session's
+ * agent. The plugin must get the real agentId from the session context via
+ * before_agent_start, not from config guessing.
+ *
+ * Priority: FLAIR_AGENT_ID env > null (let session context provide it)
  */
 export function resolveAgentId(): string | null {
-  // 1. Explicit env var
   const envId = process.env.FLAIR_AGENT_ID;
   if (envId) return envId;
-
-  // 2. Read from OpenClaw config — first agent name
-  try {
-    const configPath = resolve(homedir(), ".openclaw", "openclaw.json");
-    if (!existsSync(configPath)) return null;
-    const config = JSON.parse(readFileSync(configPath, "utf-8"));
-    const agents = config?.agents?.list;
-    if (Array.isArray(agents) && agents.length > 0) {
-      const name = agents[0]?.name;
-      if (typeof name === "string" && name) return name.toLowerCase();
-    }
-  } catch {
-    // Config unreadable — fall through
-  }
-
   return null;
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tpsdev-ai/openclaw-flair",
-  "version": "0.3.0",
-  "description": "OpenClaw memory plugin for Flair — agent identity and semantic memory",
+  "version": "0.4.1",
+  "description": "OpenClaw memory plugin for Flair \u2014 agent identity and semantic memory",
   "type": "module",
   "main": "index.ts",
   "exports": {
@@ -30,7 +30,9 @@
   },
   "homepage": "https://github.com/tpsdev-ai/flair/tree/main/plugins/openclaw-flair",
   "openclaw": {
-    "extensions": ["./index.ts"]
+    "extensions": [
+      "./index.ts"
+    ]
   },
   "peerDependencies": {
     "openclaw": ">=2026.3.7"
@@ -40,6 +42,6 @@
   },
   "dependencies": {
     "@sinclair/typebox": "^0.34.0",
-    "@tpsdev-ai/flair-client": "^0.1.0"
+    "@tpsdev-ai/flair-client": "^0.2.0"
   }
-}
+}