ofiere-openclaw-plugin 4.30.0 → 4.31.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.30.0",
+  "version": "4.31.0",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 14 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, execution plan builder, SOP management, and agent brain (memory + self-improvement)",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/agent-resolver.ts

@@ -35,12 +35,12 @@ export async function resolveAgentId(
     return cache.get(cacheKey)!;
   }
 
-  // 2. Look up by name (case-insensitive)
+  // 2. Look up by name OR codename in a single query (v4.30.0 optimization)
   const { data: existing } = await supabase
     .from("agents")
     .select("id")
     .eq("user_id", userId)
-    .ilike("name", accountId)
+    .or(`name.ilike.${accountId},codename.ilike.${accountId}`)
     .limit(1)
     .single();
 
@@ -49,21 +49,7 @@ export async function resolveAgentId(
     return existing.id;
   }
 
-  // 3. Also try matching by codename
-  const { data: byCodename } = await supabase
-    .from("agents")
-    .select("id")
-    .eq("user_id", userId)
-    .ilike("codename", accountId)
-    .limit(1)
-    .single();
-
-  if (byCodename?.id) {
-    cache.set(cacheKey, byCodename.id);
-    return byCodename.id;
-  }
-
-  // 4. Auto-register a new agent
+  // 3. Auto-register a new agent
   const newId = `agent-${accountId.toLowerCase()}-${Date.now()}`;
   const { data: created } = await supabase
     .from("agents")

package/src/tools.ts

@@ -113,10 +113,16 @@ const SYSTEM_NAME_BLOCKLIST = new Set([
   "ofiere pm plugin", "ofiere-openclaw-plugin",
 ]);
 
+// Module-scoped brain context cache (v4.30.0)
+// Shared between registerBrainContextHook and BRAIN_OPS save_memory
+// so explicit saves invalidate the cache for immediate consistency.
+const brainCache = new Map<string, { text: string; at: number }>();
+
 function isSystemName(name: string): boolean {
   return SYSTEM_NAME_BLOCKLIST.has(name.toLowerCase().trim());
 }
 
+
 function createAgentResolver(
   api: any,
   supabase: SupabaseClient,
@@ -5213,6 +5219,10 @@ function registerBrainOps(
           }).select("id, tier, importance, decay_score, created_at").single();
 
           if (error) return err(error.message);
+
+          // Invalidate brain context cache so next prompt build sees the new memory
+          if (agentId) brainCache.delete(agentId);
+
           return ok({ message: `Memory saved to ${tier}`, memory: data });
         }
 
@@ -5690,10 +5700,12 @@ function registerBrainExtractionHook(
 
         api.logger.debug?.(`[ofiere-brain] agent_end identity: ctx.agentId=${ctx?.agentId || "(none)"} event.agentId=${event?.agentId || "(none)"} resolved=${resolvedAgentId}`);
 
-        // ── Fast Stream: Write L1_focus ──
+        // ── Fast Stream: Write L1_focus + L2_episode in parallel ──
         const rawContent = `User: ${lastUser}\nAssistant: ${lastAssistant}`;
+        const immediateWrites: Promise<any>[] = [];
+
         if (rawContent.length > 50) {
-          await supabase.from("agent_memories").insert({
+          immediateWrites.push(supabase.from("agent_memories").insert({
             user_id: userId,
             agent_id: resolvedAgentId,
             tier: "L1_focus",
@@ -5702,16 +5714,14 @@ function registerBrainExtractionHook(
             importance: 3,
             context_key: sessionKey ? `conversation:${sessionKey}` : null,
             expires_at: new Date(Date.now() + 48 * 60 * 60 * 1000).toISOString(),
-          });
+          }));
         }
 
-        // ── Lightweight L2 episode summary ──
         if (lastUser.length + lastAssistant.length > 200) {
           const userSummary = lastUser.split(/[.!?]\s/)[0];
           const assistSummary = lastAssistant.split(/[.!?]\s/)[0];
           const episodeSummary = `User asked: "${userSummary}". Agent responded: "${assistSummary}"`;
-
-          await supabase.from("agent_memories").insert({
+          immediateWrites.push(supabase.from("agent_memories").insert({
             user_id: userId,
             agent_id: resolvedAgentId,
             tier: "L2_episode",
@@ -5719,70 +5729,86 @@ function registerBrainExtractionHook(
             source: "auto",
             importance: 4,
             context_key: sessionKey ? `episode:${sessionKey}` : null,
-          });
+          }));
         }
 
-        // ── Rule-based L3/L4 extraction from user message ──
-        // Extract factual statements → L3_pattern
+        // Fire L1+L2 writes in parallel (no await needed between them)
+        await Promise.all(immediateWrites);
+
+        // ── Batched L3/L4 extraction (v4.30.0 optimization) ──
+        // Collect all candidates first, dedup in one parallel batch, insert in one batch.
         const factPatterns = [
           /(?:my name is|i'm called|call me)\s+(\w+)/gi,
           /(?:i (?:work|am working) (?:at|for|with))\s+(.+?)(?:\.|,|$)/gi,
           /(?:i (?:like|love|prefer|enjoy|hate|dislike))\s+(.+?)(?:\.|,|$)/gi,
           /(?:i (?:am|'m))\s+(?:a|an)\s+(.+?)(?:\.|,|$)/gi,
         ];
+        const rulePatterns = [
+          /(?:always|never|make sure|don't|do not|please always)\s+(.+?)(?:\.|!|$)/gi,
+          /(?:remember to|keep in mind|note that)\s+(.+?)(?:\.|!|$)/gi,
+        ];
+
+        // Phase 1: Collect all regex candidates
+        type MemCandidate = { tier: string; content: string; contextKey: string; importance: number };
+        const candidates: MemCandidate[] = [];
 
         for (const pattern of factPatterns) {
-          const matches = lastUser.matchAll(pattern);
-          for (const match of matches) {
+          for (const match of lastUser.matchAll(pattern)) {
             const fact = match[0].trim();
-            const contextKey = `fact:${fact.slice(0, 50).toLowerCase().replace(/\s+/g, "_")}`;
-            // Dedup: check if context_key already exists
-            const { data: existing } = await supabase.from("agent_memories")
-              .select("id").eq("user_id", userId).eq("agent_id", resolvedAgentId)
-              .eq("context_key", contextKey).is("superseded_by", null).limit(1);
-            if (existing && existing.length > 0) continue;
-
-            await supabase.from("agent_memories").insert({
-              user_id: userId,
-              agent_id: resolvedAgentId,
+            candidates.push({
               tier: "L3_pattern",
               content: fact,
-              source: "auto",
+              contextKey: `fact:${fact.slice(0, 50).toLowerCase().replace(/\s+/g, "_")}`,
               importance: 6,
-              context_key: contextKey,
             });
           }
         }
-
-        // Extract directives → L4_rule
-        const rulePatterns = [
-          /(?:always|never|make sure|don't|do not|please always)\s+(.+?)(?:\.|!|$)/gi,
-          /(?:remember to|keep in mind|note that)\s+(.+?)(?:\.|!|$)/gi,
-        ];
-
         for (const pattern of rulePatterns) {
-          const matches = lastUser.matchAll(pattern);
-          for (const match of matches) {
+          for (const match of lastUser.matchAll(pattern)) {
             const rule = match[0].trim();
             if (rule.length < 10) continue;
-            const contextKey = `rule:${rule.slice(0, 50).toLowerCase().replace(/\s+/g, "_")}`;
-            const { data: existing } = await supabase.from("agent_memories")
-              .select("id").eq("user_id", userId).eq("agent_id", resolvedAgentId)
-              .eq("context_key", contextKey).is("superseded_by", null).limit(1);
-            if (existing && existing.length > 0) continue;
-
-            await supabase.from("agent_memories").insert({
-              user_id: userId,
-              agent_id: resolvedAgentId,
+            candidates.push({
               tier: "L4_rule",
               content: rule,
-              source: "auto",
+              contextKey: `rule:${rule.slice(0, 50).toLowerCase().replace(/\s+/g, "_")}`,
               importance: 7,
-              context_key: contextKey,
             });
           }
         }
 
+        // Phase 2: Parallel dedup check (1 query per candidate, all at once)
+        if (candidates.length > 0) {
+          const dedupResults = await Promise.all(
+            candidates.map(c =>
+              supabase.from("agent_memories")
+                .select("id")
+                .eq("user_id", userId)
+                .eq("agent_id", resolvedAgentId)
+                .eq("context_key", c.contextKey)
+                .is("superseded_by", null)
+                .limit(1)
+            )
+          );
+
+          // Phase 3: Batch insert only new candidates
+          const toInsert = candidates
+            .filter((_, i) => !dedupResults[i].data?.length)
+            .map(c => ({
+              user_id: userId,
+              agent_id: resolvedAgentId,
+              tier: c.tier,
+              content: c.content,
+              source: "auto" as const,
+              importance: c.importance,
+              context_key: c.contextKey,
+              decay_score: 1.0,
+            }));
+
+          if (toInsert.length > 0) {
+            await supabase.from("agent_memories").insert(toInsert);
+          }
+        }
+
         api.logger.debug?.(`[ofiere-brain] Extracted memory for agent=${resolvedAgentId} session=${sessionKey || "none"}`);
       } catch (e) {
         // Silent — brain extraction must never block chat
@@ -5815,11 +5841,9 @@ function registerBrainContextHook(
   userId: string,
   fallbackAgentId: string,
 ): void {
-  // Cache to avoid querying Supabase on every single prompt build.
-  // Key: resolved agent UUID. Value: { brainContext, loadedAt }.
-  // Cache TTL: 60 seconds — brain context refreshes at most once per minute.
-  const brainCache = new Map<string, { text: string; at: number }>();
-  const CACHE_TTL_MS = 60_000;
+  // Uses module-scoped brainCache (declared at top of file) for cross-function
+  // invalidation. save_memory calls brainCache.delete(agentId) for immediate consistency.
+  const CACHE_TTL_MS = 300_000;
 
   try {
     api.on("before_prompt_build", async (_event: any, ctx: any) => {
@@ -5846,38 +5870,26 @@ function registerBrainContextHook(
         }
 
         // ── Query brain memories for THIS specific agent ──
+        // v4.30.0: Single combined query instead of 3 separate tier queries.
         const now = new Date().toISOString();
 
-        const [l1Res, l4Res, l5Res] = await Promise.all([
-          supabase.from("agent_memories")
-            .select("content, importance")
-            .eq("user_id", userId)
-            .eq("agent_id", resolvedAgentId)
-            .eq("tier", "L1_focus")
-            .gt("decay_score", 0.1)
-            .or(`expires_at.is.null,expires_at.gt.${now}`)
-            .order("importance", { ascending: false })
-            .limit(5),
-          supabase.from("agent_memories")
-            .select("content, importance")
-            .eq("user_id", userId)
-            .eq("agent_id", resolvedAgentId)
-            .eq("tier", "L4_rule")
-            .is("superseded_by", null)
-            .order("importance", { ascending: false })
-            .limit(10),
-          supabase.from("agent_memories")
-            .select("content")
-            .eq("user_id", userId)
-            .eq("agent_id", resolvedAgentId)
-            .eq("tier", "L5_persona")
-            .order("importance", { ascending: false })
-            .limit(3),
-        ]);
-
-        const l1 = l1Res.data || [];
-        const l4 = l4Res.data || [];
-        const l5 = l5Res.data || [];
+        const { data: allBrain } = await supabase
+          .from("agent_memories")
+          .select("content, importance, tier")
+          .eq("user_id", userId)
+          .eq("agent_id", resolvedAgentId)
+          .in("tier", ["L1_focus", "L4_rule", "L5_persona"])
+          .gt("decay_score", 0.1)
+          .is("superseded_by", null)
+          .or(`expires_at.is.null,expires_at.gt.${now}`)
+          .order("importance", { ascending: false })
+          .limit(18); // 5 L1 + 10 L4 + 3 L5
+
+        // Client-side split by tier
+        const all = allBrain || [];
+        const l1 = all.filter((m: any) => m.tier === "L1_focus").slice(0, 5);
+        const l4 = all.filter((m: any) => m.tier === "L4_rule").slice(0, 10);
+        const l5 = all.filter((m: any) => m.tier === "L5_persona").slice(0, 3);
 
         if (l1.length === 0 && l4.length === 0 && l5.length === 0) {
           brainCache.set(resolvedAgentId, { text: "", at: Date.now() });