ofiere-openclaw-plugin 4.7.0 → 4.8.0

package/package.json

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

@@ -190,7 +190,8 @@ function registerTaskOps(
     description:
       `Manage tasks in the Ofiere PM dashboard. All task operations go through this tool.\n\n` +
       `Actions:\n` +
-      `- "list": List/filter tasks. Optional: status, agent_id, space_id, folder_id, limit\n` +
+      `- "list": List/filter tasks. Optional: status, agent_id, space_id, folder_id, task_id, limit\n` +
+      `- "get": Get a single task by ID. Required: task_id\n` +
       `- "create": Create a task. Required: title. Optional: agent_id, description, status, priority, space_id, folder_id, start_date, due_date, tags, instructions, execution_plan, goals, constraints, system_prompt, recurrence_type, recurrence_interval, scheduled_time\n` +
       `- "update": Update a task. Required: task_id. Optional: all create fields + progress\n` +
       `- "delete": Delete task + subtasks. Required: task_id\n\n` +
@@ -212,6 +213,10 @@ function registerTaskOps(
         title: { type: "string", description: "Task title (required for create)" },
         description: { type: "string", description: "Task description" },
         instructions: { type: "string", description: "Detailed instructions for the agent executing this task" },
+        task_id: {
+          type: "string",
+          description: "Task ID — use with 'list' to filter or 'get' to fetch a single task",
+        },
         agent_id: {
           type: "string",
           description: "Agent name or ID. Your name to self-assign, 'none' for unassigned.",
@@ -284,6 +289,9 @@ function registerTaskOps(
       switch (action) {
         case "list":
           return handleListTasks(supabase, userId, params);
+        case "get":
+          if (!params.task_id) return err("Missing required field: task_id");
+          return handleListTasks(supabase, userId, { ...params, limit: 1 });
         case "create":
           return handleCreateTask(supabase, userId, resolveAgent, params);
         case "update":
@@ -292,7 +300,7 @@ function registerTaskOps(
           return handleDeleteTask(supabase, userId, params);
         default:
           return err(
-            `Unknown action "${action}". Valid actions: list, create, update, delete`,
+            `Unknown action "${action}". Valid actions: list, get, create, update, delete`,
           );
       }
     },
@@ -316,6 +324,7 @@ async function handleListTasks(
       .eq("user_id", userId)
       .order("updated_at", { ascending: false });
 
+    if (params.task_id) query = query.eq("id", params.task_id as string);
     if (params.space_id) query = query.eq("space_id", params.space_id as string);
     if (params.folder_id) query = query.eq("folder_id", params.folder_id as string);
     if (params.agent_id) query = query.eq("agent_id", params.agent_id as string);
@@ -600,33 +609,45 @@ async function handleUpdateTask(
       const mergedCf = { ...existingCf };
 
       if (params.execution_plan !== undefined) {
-        mergedCf.execution_plan = Array.isArray(params.execution_plan)
+        const existingSteps = (existingCf.execution_plan || []) as any[];
+        const newSteps = Array.isArray(params.execution_plan)
           ? (params.execution_plan as any[]).map((step: any, i: number) => ({
               id: step.id || `step-${Date.now()}-${i}`,
               text: typeof step === "string" ? step : step.text || String(step),
-              order: i,
+              order: existingSteps.length + i,
             }))
           : [];
+        // Merge: concat existing + new, dedup by id
+        const stepIds = new Set(existingSteps.map((s: any) => s.id));
+        mergedCf.execution_plan = [...existingSteps, ...newSteps.filter((s: any) => !stepIds.has(s.id))];
       }
 
       if (params.goals !== undefined) {
-        mergedCf.goals = Array.isArray(params.goals)
+        const existingGoals = (existingCf.goals || []) as any[];
+        const newGoals = Array.isArray(params.goals)
           ? (params.goals as any[]).map((g: any, i: number) => ({
               id: g.id || `goal-${Date.now()}-${i}`,
               type: g.type || "custom",
               label: typeof g === "string" ? g : g.label || String(g),
             }))
           : [];
+        // Merge: concat existing + new, dedup by id
+        const goalIds = new Set(existingGoals.map((g: any) => g.id));
+        mergedCf.goals = [...existingGoals, ...newGoals.filter((g: any) => !goalIds.has(g.id))];
       }
 
       if (params.constraints !== undefined) {
-        mergedCf.constraints = Array.isArray(params.constraints)
+        const existingConstraints = (existingCf.constraints || []) as any[];
+        const newConstraints = Array.isArray(params.constraints)
           ? (params.constraints as any[]).map((c: any, i: number) => ({
               id: c.id || `cstr-${Date.now()}-${i}`,
               type: c.type || "custom",
               label: typeof c === "string" ? c : c.label || String(c),
             }))
           : [];
+        // Merge: concat existing + new, dedup by id
+        const cstrIds = new Set(existingConstraints.map((c: any) => c.id));
+        mergedCf.constraints = [...existingConstraints, ...newConstraints.filter((c: any) => !cstrIds.has(c.id))];
       }
 
       if (params.system_prompt !== undefined) mergedCf.system_prompt = params.system_prompt;
@@ -753,15 +774,6 @@ async function handleListAgents(
   fallbackAgentId: string,
 ): Promise<ToolResult> {
   try {
-    // Resolve calling agent's ID for the "your_agent_id" hint
-    const callerName = getCallingAgentName(api);
-    let yourAgentId = fallbackAgentId || "";
-    if (callerName && !yourAgentId) {
-      try {
-        yourAgentId = await resolveAgentId(callerName, userId, supabase);
-      } catch { /* ignore */ }
-    }
-
     const { data, error } = await supabase
       .from("agents")
       .select("id, name, codename, role, status")
@@ -769,10 +781,40 @@ async function handleListAgents(
       .order("name");
 
     if (error) return err(error.message);
+
+    // BUG 1 fix: resolve your_agent_id to the agent NAME (e.g. "celia")
+    // so callers can use it directly for task assignment.
+    // Previous behavior returned an internal UUID/auto-generated ID that
+    // didn't match any assignable agent in the list.
+    let yourAgentName: string | null = null;
+
+    // Try fallbackAgentId first (env var OFIERE_AGENT_ID)
+    if (fallbackAgentId) {
+      // If fallback is a UUID, find the matching agent name
+      const match = (data || []).find((a: any) => a.id === fallbackAgentId);
+      yourAgentName = match?.name || null;
+    }
+
+    // Try runtime detection
+    if (!yourAgentName) {
+      const callerName = getCallingAgentName(api);
+      if (callerName && !isSystemName(callerName)) {
+        // Check if callerName matches any agent name (case-insensitive)
+        const match = (data || []).find(
+          (a: any) => a.name?.toLowerCase() === callerName.toLowerCase() ||
+                      a.codename?.toLowerCase() === callerName.toLowerCase()
+        );
+        yourAgentName = match?.name || null;
+      }
+    }
+
     return ok({
       agents: data || [],
       count: (data || []).length,
-      your_agent_id: yourAgentId,
+      your_agent_id: yourAgentName,
+      hint: !yourAgentName
+        ? "Could not auto-detect your agent. Use an agent name from the list above (e.g. 'celia') when creating tasks."
+        : undefined,
     });
   } catch (e) {
     return err(e instanceof Error ? e.message : String(e));