ofiere-openclaw-plugin 4.38.0 → 4.39.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.38.0",
+  "version": "4.39.1",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 16 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, execution plan builder, SOP management, agent brain, talent management, and corporate frameworks",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],
@@ -16,6 +16,10 @@
     "openclaw.plugin.json",
     "README.md"
   ],
+  "scripts": {
+    "typecheck": "tsc --noEmit",
+    "build": "tsc --noEmit"
+  },
   "openclaw": {
     "extensions": [
       "./index.ts"
@@ -32,5 +36,8 @@
   "dependencies": {
     "@supabase/supabase-js": "^2.98.0",
     "zod": "^3.25.11"
+  },
+  "devDependencies": {
+    "typescript": "^5.6.0"
   }
 }

package/src/attachments.ts

@@ -277,27 +277,16 @@ function isSystemName(name: string): boolean {
   return s === "" || s === "ofiere" || s === "openclaw" || s === "system" || s === "plugin" || s === "gateway" || s.includes("plugin");
 }
 
-async function buildAttachmentBlock(args: {
+// Render a block from explicit doc-id lists. Used by both the dispatch-params
+// path (when workflow/task-dispatcher hands ids in via ctx) and the conversation
+// fallback path (when no explicit ids are present).
+async function renderBlockForIds(args: {
   supabase: SupabaseClient;
   userId: string;
-  agentId: string;
+  sopIds: string[];
+  fwIds: string[];
 }): Promise<string> {
-  const { supabase, userId, agentId } = args;
-
-  // Find the most recently active conversation for this agent. The dashboard
-  // bumps `updated_at` whenever a message is sent or attachments change, so
-  // this gives us the right run target most of the time.
-  const { data: convRow } = await supabase
-    .from("conversations")
-    .select("id, attached_sop_ids, attached_framework_ids")
-    .eq("user_id", userId)
-    .eq("agent_id", agentId)
-    .order("updated_at", { ascending: false })
-    .limit(1)
-    .maybeSingle();
-
-  const sopIds: string[] = (convRow?.attached_sop_ids as string[] | null) || [];
-  const fwIds: string[] = (convRow?.attached_framework_ids as string[] | null) || [];
+  const { supabase, userId, sopIds, fwIds } = args;
   if (!sopIds.length && !fwIds.length) return "";
 
   const [sopsRes, fwsRes] = await Promise.all([
@@ -329,6 +318,57 @@ async function buildAttachmentBlock(args: {
   return renderAttachmentBlock({ sops, frameworks });
 }
 
+// Pluck attachment ids from the `before_prompt_build` ctx. Exact path varies by
+// the dispatch surface that fires the prompt build, so we probe several known
+// locations. Workflow executor + task-dispatcher edge function both stash ids
+// under `metadata` on their request frames.
+function readDispatchAttachmentIds(ctx: any): { sopIds: string[]; fwIds: string[] } {
+  const candidates: Array<Record<string, unknown> | undefined> = [
+    ctx?.metadata,
+    ctx?.params?.metadata,
+    ctx?.payload?.metadata,
+    ctx?.request?.metadata,
+    ctx?.options?.metadata,
+    ctx, // last resort: top level
+  ];
+  for (const c of candidates) {
+    if (!c || typeof c !== "object") continue;
+    const sop = (c as any).attached_sop_ids;
+    const fw = (c as any).attached_framework_ids;
+    if (Array.isArray(sop) || Array.isArray(fw)) {
+      return {
+        sopIds: Array.isArray(sop) ? sop.filter((x: unknown): x is string => typeof x === "string") : [],
+        fwIds: Array.isArray(fw) ? fw.filter((x: unknown): x is string => typeof x === "string") : [],
+      };
+    }
+  }
+  return { sopIds: [], fwIds: [] };
+}
+
+async function buildAttachmentBlock(args: {
+  supabase: SupabaseClient;
+  userId: string;
+  agentId: string;
+}): Promise<string> {
+  const { supabase, userId, agentId } = args;
+
+  // Find the most recently active conversation for this agent. The dashboard
+  // bumps `updated_at` whenever a message is sent or attachments change, so
+  // this gives us the right run target most of the time.
+  const { data: convRow } = await supabase
+    .from("conversations")
+    .select("id, attached_sop_ids, attached_framework_ids")
+    .eq("user_id", userId)
+    .eq("agent_id", agentId)
+    .order("updated_at", { ascending: false })
+    .limit(1)
+    .maybeSingle();
+
+  const sopIds: string[] = (convRow?.attached_sop_ids as string[] | null) || [];
+  const fwIds: string[] = (convRow?.attached_framework_ids as string[] | null) || [];
+  return renderBlockForIds({ supabase, userId, sopIds, fwIds });
+}
+
 export function registerAttachmentContextHook(args: {
   api: any;
   supabase: SupabaseClient;
@@ -351,7 +391,26 @@ export function registerAttachmentContextHook(args: {
         }
         if (!resolvedAgentId) return;
 
-        const cacheKey = resolvedAgentId;
+        // Dispatch-params path: workflow executor + task-dispatcher edge function
+        // can stash explicit `attached_sop_ids` / `attached_framework_ids` on the
+        // chat.send frame's metadata. When present, prefer them over the
+        // most-recent-conversation lookup and bypass cache (per-dispatch ids).
+        const dispatchIds = readDispatchAttachmentIds(ctx);
+        if (dispatchIds.sopIds.length || dispatchIds.fwIds.length) {
+          api.logger?.debug?.(
+            `[ofiere-attach] dispatch ids agent=${resolvedAgentId} sops=${dispatchIds.sopIds.length} fws=${dispatchIds.fwIds.length}`,
+          );
+          const block = await renderBlockForIds({
+            supabase, userId,
+            sopIds: dispatchIds.sopIds,
+            fwIds: dispatchIds.fwIds,
+          });
+          return block ? { appendSystemContext: block } : undefined;
+        }
+
+        // Multi-tenant: a single plugin process can serve multiple users — key
+        // by (userId, agentId) so User B never sees User A's cached block.
+        const cacheKey = `${userId}::${resolvedAgentId}`;
         const cached = attachmentCache.get(cacheKey);
         if (cached) {
           const age = Date.now() - cached.at;

package/src/tools.ts

@@ -5624,7 +5624,9 @@ function registerBrainOps(
           // Touch access_count + last_accessed_at for FadeMem
           if (data && data.length > 0) {
             const ids = data.map((m: any) => m.id);
-            supabase.rpc("touch_memories", { memory_ids: ids }).then(() => {}).catch(() => {});
+            // PostgrestFilterBuilder is PromiseLike, not a full Promise — use the
+            // 2-arg form of .then() instead of .catch() to silence rejection.
+            supabase.rpc("touch_memories", { memory_ids: ids }).then(() => {}, () => {});
           }
 
           return ok({ memories: data || [], count: (data || []).length, query: queryText, search_mode: "fulltext" });
@@ -6242,7 +6244,7 @@ function registerBrainExtractionHook(
 
         // ── Fast Stream: Write L1_focus + L2_episode in parallel ──
         const rawContent = `User: ${lastUser}\nAssistant: ${lastAssistant}`;
-        const immediateWrites: Promise<any>[] = [];
+        const immediateWrites: PromiseLike<any>[] = [];
 
         if (rawContent.length > 50) {
           immediateWrites.push(supabase.from("agent_memories").insert({

package/src/types/openclaw.d.ts

@@ -0,0 +1,8 @@
+// Ambient declaration for the OpenClaw plugin SDK. The gateway resolves this
+// module at runtime; npm has no published types. Keep loose — the plugin only
+// uses a small surface (logger, on, registerTool, registerCommand, etc.).
+
+declare module "openclaw/plugin-sdk" {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  export type OpenClawPluginApi = any;
+}