ofiere-openclaw-plugin 4.18.3 → 4.18.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.18.3",
+  "version": "4.18.5",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 12 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, and execution plan builder",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/tools.ts

@@ -4347,15 +4347,20 @@ async function handleExecutePlan(
 
     // Step 1: Create project folder
     let folderSkipped = false;
+    let folderError: string | null = null;
     if (createFolder && spaceId) {
       const folderRow = {
         id: crypto.randomUUID(),
         user_id: userId, space_id: spaceId, name: data.name || plan.name || "Untitled Plan",
-        type: "project", parent_folder_id: folderId, sort_order: 0,
+        folder_type: "project", parent_folder_id: folderId, sort_order: 0,
         created_at: new Date().toISOString(), updated_at: new Date().toISOString(),
       };
       const { error: folderErr } = await supabase.from("pm_folders").insert(folderRow);
-      if (!folderErr) folderId = folderRow.id;
+      if (!folderErr) {
+        folderId = folderRow.id;
+      } else {
+        folderError = folderErr.message;
+      }
     } else if (createFolder && !spaceId) {
       folderSkipped = true;
     }
@@ -4428,39 +4433,72 @@ async function handleExecutePlan(
       }
     }
 
-    // Step 3: Create dependencies (sequential chain)
+    // Step 3: Create dependencies — sibling chaining + parent→child
     const depsQueue = [...rootNodes];
     let depsCreated = 0;
+
+    // Helper: resolve a node to its nearest task ID (skip gates/milestones)
+    function resolveTaskId(n: any): string | undefined {
+      return idMap.get(n.id);
+    }
+
+    // Helper: insert a dependency row into pm_dependencies
+    async function insertDep(predId: string, succId: string): Promise<boolean> {
+      const { error } = await supabase.from("pm_dependencies").insert({
+        id: `dep-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`,
+        user_id: userId, predecessor_id: predId, successor_id: succId,
+        dependency_type: "finish_to_start", lag_days: 0,
+      });
+      return !error;
+    }
+
     while (depsQueue.length > 0) {
       const node = depsQueue.shift()!;
-      const nodeRealId = idMap.get(node.id);
+      const nodeRealId = resolveTaskId(node);
+
+      // Chain siblings: sequential children of this node get chained left-to-right
+      if (Array.isArray(node.children) && node.children.length > 1 && !node.parallel) {
+        const taskChildren: string[] = [];
+        for (const child of node.children) {
+          const cid = resolveTaskId(child);
+          if (cid) taskChildren.push(cid);
+        }
+        for (let i = 1; i < taskChildren.length; i++) {
+          if (await insertDep(taskChildren[i - 1], taskChildren[i])) depsCreated++;
+        }
+      }
+
+      // Parent → first child link (sequential only)
       if (nodeRealId && !node.parallel && node.children?.length > 0) {
-        const firstChildId = idMap.get(node.children[0].id);
+        const firstChildId = resolveTaskId(node.children[0]);
         if (firstChildId) {
-          await supabase.from("task_dependencies").insert({ user_id: userId, predecessor_id: nodeRealId, successor_id: firstChildId, dependency_type: "finish_to_start", lag_days: 0 });
-          depsCreated++;
-        }
-        for (let i = 1; i < node.children.length; i++) {
-          const prevId = idMap.get(node.children[i - 1].id);
-          const currId = idMap.get(node.children[i].id);
-          if (prevId && currId) {
-            await supabase.from("task_dependencies").insert({ user_id: userId, predecessor_id: prevId, successor_id: currId, dependency_type: "finish_to_start", lag_days: 0 });
-            depsCreated++;
-          }
+          if (await insertDep(nodeRealId, firstChildId)) depsCreated++;
         }
       }
+
+      // Parallel fan-out: parent → each child
       if (nodeRealId && node.parallel && node.children?.length > 0) {
         for (const child of node.children) {
-          const childId = idMap.get(child.id);
+          const childId = resolveTaskId(child);
           if (childId) {
-            await supabase.from("task_dependencies").insert({ user_id: userId, predecessor_id: nodeRealId, successor_id: childId, dependency_type: "finish_to_start", lag_days: 0 });
-            depsCreated++;
+            if (await insertDep(nodeRealId, childId)) depsCreated++;
           }
         }
       }
+
       if (Array.isArray(node.children)) depsQueue.push(...node.children);
     }
 
+    // Chain root-level siblings (sequential root nodes)
+    const rootTaskIds: string[] = [];
+    for (const rn of rootNodes) {
+      const rid = resolveTaskId(rn);
+      if (rid) rootTaskIds.push(rid);
+    }
+    for (let i = 1; i < rootTaskIds.length; i++) {
+      if (await insertDep(rootTaskIds[i - 1], rootTaskIds[i])) depsCreated++;
+    }
+
     // Mark plan as deployed
     await supabase.from("pm_plans").update({ is_deployed: true, deployed_at: new Date().toISOString(), updated_at: new Date().toISOString() }).eq("id", params.plan_id as string).eq("user_id", userId);
 
@@ -4471,6 +4509,7 @@ async function handleExecutePlan(
       folder_id: folderId,
       space_id: spaceId,
       ...(folderSkipped ? { folder_skipped_reason: "No space_id provided — assign the plan to a space to enable folder creation" } : {}),
+      ...(folderError ? { folder_error: folderError } : {}),
     });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }