ofiere-openclaw-plugin 4.18.0 → 4.18.2

package/package.json

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

@@ -4101,7 +4101,7 @@ function registerPlanOps(
               constraints: { type: "array", items: { type: "object", properties: { label: { type: "string" }, type: { type: "string" } }, required: ["label"] } },
               system_prompt: { type: "string" },
               parallel: { type: "boolean" },
-              children: { type: "array", description: "Nested child nodes (recursive)" },
+              children: { type: "array", description: "Nested child nodes (recursive)", items: { type: "object", properties: { title: { type: "string" }, type: { type: "string", enum: ["task", "gate", "milestone"] }, description: { type: "string" }, agent_id: { type: "string" }, priority: { type: "number" }, status: { type: "string" }, start_date: { type: "string" }, due_date: { type: "string" }, parallel: { type: "boolean" } }, required: ["title"] } },
             },
             required: ["title"],
           },
@@ -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)); }
 }
 
@@ -4268,8 +4270,9 @@ async function handleUpdatePlan(supabase: SupabaseClient, userId: string, params
       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);
     }
-    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 +4280,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)); }
 }
@@ -4332,7 +4336,7 @@ async function handleExecutePlan(
     // Step 1: Create project folder
     if (createFolder && spaceId) {
       const folderRow = {
-        id: `folder-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`,
+        id: crypto.randomUUID(),
         user_id: userId, space_id: spaceId, name: plan.name || "Untitled Plan",
         type: "project", parent_folder_id: folderId, sort_order: 0,
         created_at: new Date().toISOString(), updated_at: new Date().toISOString(),