ofiere-openclaw-plugin 4.1.0 → 4.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.1.0",
+  "version": "4.3.0",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 10 meta-tools with 13-action workflow mastery covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, and constellation agent architecture",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/tools.ts

@@ -1334,17 +1334,24 @@ function registerWorkflowOps(
         return { wf: data, error: null };
       }
 
-      // Helper: save updated graph back to DB
+      // Helper: save updated graph back to DB with verification re-read
       async function saveGraph(wfId: string, nodes: any[], edges: any[]) {
-        const { data, error } = await supabase
+        const { error } = await supabase
           .from("workflows")
           .update({ nodes, edges, updated_at: new Date().toISOString() })
           .eq("id", wfId)
+          .eq("user_id", userId);
+        if (error) return { wf: null, error: error.message };
+        // Verification re-read — guarantees we return the actual persisted state,
+        // not a potentially stale response from the update's connection.
+        const { data: verified, error: readErr } = await supabase
+          .from("workflows")
+          .select("*")
+          .eq("id", wfId)
           .eq("user_id", userId)
-          .select()
           .single();
-        if (error) return { wf: null, error: error.message };
-        return { wf: data, error: null };
+        if (readErr) return { wf: null, error: readErr.message };
+        return { wf: verified, error: null };
       }
 
       switch (action) {
@@ -1358,9 +1365,12 @@ function registerWorkflowOps(
         case "get": {
           const wfId = (params.id || params.workflow_id) as string;
           if (!wfId) return err("Missing required: id");
-          const { data, error } = await supabase.from("workflows").select("*").eq("id", wfId).eq("user_id", userId).single();
-          if (error) return err(error.message);
-          return ok({ workflow: data });
+          // Route through serialization queue so reads wait for pending mutations
+          return sequentialWorkflowOp(wfId, async () => {
+            const { data, error } = await supabase.from("workflows").select("*").eq("id", wfId).eq("user_id", userId).single();
+            if (error) return err(error.message);
+            return ok({ workflow: data });
+          });
         }
         case "create": {
           if (!params.name) return err("Missing required: name");
@@ -1389,8 +1399,37 @@ function registerWorkflowOps(
             finalNodes.unshift(triggerNode);
           }
 
-          // Build edges
-          let finalEdges = ((params.edges as any[]) || []).map((e: any, i: number) => normalizeEdge(e, i));
+          // Build edges — remap IDs from pre-normalization to post-normalization
+          const idRemap = new Map<string, string>();
+          rawNodes.forEach((raw, i) => {
+            if (raw.id && finalNodes[hasTrigger ? i : i + 1]?.id !== raw.id) {
+              idRemap.set(raw.id, finalNodes[hasTrigger ? i : i + 1].id);
+            }
+          });
+
+          let finalEdges: any[];
+          const suppliedEdges = (params.edges as any[]) || [];
+
+          if (suppliedEdges.length > 0) {
+            // Remap source/target IDs in user-supplied edges
+            finalEdges = suppliedEdges.map((e: any, i: number) => {
+              const remapped = {
+                ...e,
+                source: idRemap.get(e.source) || e.source,
+                target: idRemap.get(e.target) || e.target,
+              };
+              return normalizeEdge(remapped, i);
+            });
+          } else {
+            // No edges supplied — auto-chain all nodes in order
+            finalEdges = [];
+            for (let i = 0; i < finalNodes.length - 1; i++) {
+              finalEdges.push(normalizeEdge({
+                source: finalNodes[i].id,
+                target: finalNodes[i + 1].id,
+              }, i));
+            }
+          }
 
           // Auto-wire trigger to first non-trigger node if no edge connects from trigger
           if (!hasTrigger && finalNodes.length > 1) {
@@ -1398,11 +1437,10 @@ function registerWorkflowOps(
             const firstStepId = finalNodes[1].id;
             const triggerHasEdge = finalEdges.some((e: any) => e.source === triggerId);
             if (!triggerHasEdge) {
-              finalEdges.unshift({
-                id: `edge-trigger-${Date.now()}`,
+              finalEdges.unshift(normalizeEdge({
                 source: triggerId,
                 target: firstStepId,
-              });
+              }, finalEdges.length));
             }
           }
 
@@ -1453,7 +1491,7 @@ function registerWorkflowOps(
           if (!wfId) return err("Missing required: workflow_id");
           const { data, error } = await supabase.from("workflow_runs").select("*")
             .eq("workflow_id", wfId)
-            .order("created_at", { ascending: false })
+            .order("started_at", { ascending: false })
             .limit((params.limit as number) || 20);
           if (error) return err(error.message);
           return ok({ runs: data || [], count: (data || []).length });