ofiere-openclaw-plugin 4.18.1 → 4.18.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.18.1",
+  "version": "4.18.3",
   "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

@@ -4131,7 +4131,7 @@ function generatePlanNodeId(): string {
 }
 
 function generatePlanId(): string {
-  return `plan-${Date.now()}-${Math.random().toString(36).slice(2, 7)}`;
+  return crypto.randomUUID();
 }
 
 /** Normalize agent-provided node into the PlanNode shape stored in plan_data */
@@ -4233,11 +4233,13 @@ async function handleListPlans(supabase: SupabaseClient, userId: string, params:
 async function handleGetPlan(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
   try {
     if (!params.plan_id) return err("Missing required field: plan_id");
-    const { data, error } = await supabase.from("pm_plans").select("*").eq("id", params.plan_id as string).eq("user_id", userId).single();
+    const { data, error } = await supabase.from("pm_plans").select("*").eq("id", params.plan_id as string).eq("user_id", userId);
     if (error) return err(error.message);
+    if (!data || data.length === 0) return err("Plan not found");
+    const plan = data[0];
     let parsed: any = null;
-    try { parsed = JSON.parse(data.plan_data || "{}"); } catch { parsed = {}; }
-    return ok({ plan: { ...data, plan_data: parsed } });
+    try { parsed = JSON.parse(plan.plan_data || "{}"); } catch { parsed = {}; }
+    return ok({ plan: { ...plan, plan_data: parsed } });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }
 
@@ -4267,9 +4269,22 @@ async function handleUpdatePlan(supabase: SupabaseClient, userId: string, params
     if (params.nodes !== undefined) {
       const rootNodes = Array.isArray(params.nodes) ? (params.nodes as any[]).map(normalizeNode) : [];
       updates.plan_data = buildPlanJson(params.plan_id as string, (params.name as string) || "Plan", params.description as string | undefined, rootNodes);
+    } else if (params.name !== undefined || params.description !== undefined) {
+      // Sync plan_data inner name/description without replacing nodes
+      const { data: existing } = await supabase.from("pm_plans").select("plan_data").eq("id", params.plan_id as string).eq("user_id", userId);
+      if (existing && existing.length > 0) {
+        try {
+          const pd = JSON.parse(existing[0].plan_data || "{}");
+          if (params.name !== undefined) pd.name = params.name;
+          if (params.description !== undefined) pd.description = params.description;
+          pd.updatedAt = new Date().toISOString();
+          updates.plan_data = JSON.stringify(pd, null, 2);
+        } catch { /* plan_data parse failure — skip inner sync */ }
+      }
     }
-    const { error } = await supabase.from("pm_plans").update(updates).eq("id", params.plan_id as string).eq("user_id", userId);
+    const { data, error } = await supabase.from("pm_plans").update(updates).eq("id", params.plan_id as string).eq("user_id", userId).select("id");
     if (error) return err(error.message);
+    if (!data || data.length === 0) return err("Plan not found");
     return ok({ message: `Plan updated` });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }
@@ -4277,8 +4292,9 @@ async function handleUpdatePlan(supabase: SupabaseClient, userId: string, params
 async function handleDeletePlan(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
   try {
     if (!params.plan_id) return err("Missing required field: plan_id");
-    const { error } = await supabase.from("pm_plans").delete().eq("id", params.plan_id as string).eq("user_id", userId);
+    const { data, error } = await supabase.from("pm_plans").delete().eq("id", params.plan_id as string).eq("user_id", userId).select("id");
     if (error) return err(error.message);
+    if (!data || data.length === 0) return err("Plan not found — nothing deleted");
     return ok({ message: `Plan deleted`, deleted: true });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }
@@ -4330,15 +4346,18 @@ async function handleExecutePlan(
     let folderId = (params.folder_id as string) || null;
 
     // Step 1: Create project folder
+    let folderSkipped = false;
     if (createFolder && spaceId) {
       const folderRow = {
-        id: `folder-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`,
-        user_id: userId, space_id: spaceId, name: plan.name || "Untitled Plan",
+        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,
         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;
+    } else if (createFolder && !spaceId) {
+      folderSkipped = true;
     }
 
     // Step 2: BFS — create tasks with full field mapping
@@ -4348,7 +4367,7 @@ async function handleExecutePlan(
 
     while (queue.length > 0) {
       const { node, parentTaskId } = queue.shift()!;
-      if (node.type === "task" || node.type === "milestone") {
+      if (node.type === "task") {
         const taskId = `task-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
         const now = new Date().toISOString();
         const agentId = node.agentId ? await resolveAgent(node.agentId) : null;
@@ -4446,11 +4465,12 @@ async function handleExecutePlan(
     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);
 
     return ok({
-      message: `Plan "${plan.name}" executed successfully`,
+      message: `Plan "${data.name || plan.name}" executed successfully`,
       tasks_created: tasksCreated,
       dependencies_created: depsCreated,
       folder_id: folderId,
       space_id: spaceId,
+      ...(folderSkipped ? { folder_skipped_reason: "No space_id provided — assign the plan to a space to enable folder creation" } : {}),
     });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }