ofiere-openclaw-plugin 4.56.6 → 4.56.8

package/dist/src/tools.js

@@ -849,7 +849,8 @@ async function handleCreateTask(supabase, userId, resolveAgent, params, fallback
         const effectiveAgentId = insertData.agent_id || assignee;
         if (startDate && effectiveAgentId && normalized) {
             try {
-                let { nextRunAtEpoch } = normalized;
+                const originalEpoch = normalized.nextRunAtEpoch;
+                let nextRunAtEpoch = originalEpoch;
                 const nowEpoch = Math.floor(Date.now() / 1000);
                 // Safety net: if computed time is in the past, schedule for now + 60s
                 // Fix #2 (2026-05-17): jitter past-time bump 0..5min so bulk creates
@@ -886,6 +887,21 @@ async function handleCreateTask(supabase, userId, resolveAgent, params, fallback
                         nextRunAtEpoch += recent * perSlotSecs;
                     }
                 }
+                // Fix #3 cosmetic parity (2026-05-17): when either Fix #2 jitter or
+                // Fix #3 stagger mutated nextRunAtEpoch, recompute the local-clock
+                // display fields so the dashboard UI doesn't show the ORIGINAL T0 for
+                // an event that actually fires minutes later. Dispatch uses
+                // next_run_at, but users read scheduled_date+scheduled_time.
+                let scheduledDateLocalFinal = normalized.scheduledDateLocal;
+                let scheduledTimeLocalFinal = normalized.scheduledTimeLocal;
+                if (nextRunAtEpoch !== originalEpoch) {
+                    const offsetMin = Math.round(TZ_OFFSET_HOURS * 60);
+                    const localMs = nextRunAtEpoch * 1000 + offsetMin * 60 * 1000;
+                    const localD = new Date(localMs);
+                    const padN = (n) => String(n).padStart(2, "0");
+                    scheduledDateLocalFinal = `${localD.getUTCFullYear()}-${padN(localD.getUTCMonth() + 1)}-${padN(localD.getUTCDate())}`;
+                    scheduledTimeLocalFinal = `${padN(localD.getUTCHours())}:${padN(localD.getUTCMinutes())}`;
+                }
                 await supabase.from("scheduler_events").insert({
                     id: crypto.randomUUID(),
                     user_id: userId,
@@ -894,8 +910,8 @@ async function handleCreateTask(supabase, userId, resolveAgent, params, fallback
                     subagent_id: subagentId,
                     title: params.title,
                     description: params.description || params.instructions || null,
-                    scheduled_date: normalized.scheduledDateLocal,
-                    scheduled_time: normalized.scheduledTimeLocal,
+                    scheduled_date: scheduledDateLocalFinal,
+                    scheduled_time: scheduledTimeLocalFinal,
                     duration_minutes: 30,
                     recurrence_type: params.recurrence_type || "none",
                     recurrence_interval: params.recurrence_interval || 1,
@@ -1127,7 +1143,7 @@ async function handleDeleteTask(supabase, userId, params) {
         if (!params.task_id)
             return err("Missing required field: task_id");
         const taskId = params.task_id;
-        await supabase.from("scheduler_events").delete().eq("task_id", taskId);
+        await supabase.from("scheduler_events").delete().eq("task_id", taskId).eq("user_id", userId);
         const { data: subtasks } = await supabase
             .from("tasks")
             .select("id")
@@ -1135,7 +1151,7 @@ async function handleDeleteTask(supabase, userId, params) {
             .eq("user_id", userId);
         if (subtasks && subtasks.length > 0) {
             for (const sub of subtasks) {
-                await supabase.from("scheduler_events").delete().eq("task_id", sub.id);
+                await supabase.from("scheduler_events").delete().eq("task_id", sub.id).eq("user_id", userId);
             }
             await supabase
                 .from("tasks")
@@ -1822,7 +1838,7 @@ function registerScheduleOps(api, supabase, userId) {
                     if (upd.status === "completed" || upd.status === "cancelled") {
                         upd.next_run_at = null;
                     }
-                    const { error } = await supabase.from("scheduler_events").update(upd).eq("id", params.id);
+                    const { error } = await supabase.from("scheduler_events").update(upd).eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Event updated", ok: true });
@@ -1830,7 +1846,7 @@ function registerScheduleOps(api, supabase, userId) {
                 case "delete": {
                     if (!params.id)
                         return err("Missing required: id");
-                    const { error } = await supabase.from("scheduler_events").delete().eq("id", params.id);
+                    const { error } = await supabase.from("scheduler_events").delete().eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Event deleted", ok: true });
@@ -1944,7 +1960,7 @@ function registerKnowledgeOps(api, supabase, userId) {
                             upd[k] = params[k];
                     if (Object.keys(upd).length === 0)
                         return err("No valid fields to update");
-                    const { error } = await supabase.from("knowledge_documents").update(upd).eq("id", params.id);
+                    const { error } = await supabase.from("knowledge_documents").update(upd).eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Document updated", ok: true });
@@ -1952,7 +1968,7 @@ function registerKnowledgeOps(api, supabase, userId) {
                 case "delete": {
                     if (!params.id)
                         return err("Missing required: id");
-                    const { error } = await supabase.from("knowledge_documents").delete().eq("id", params.id);
+                    const { error } = await supabase.from("knowledge_documents").delete().eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Document deleted", ok: true });
@@ -2654,13 +2670,13 @@ function registerNotifyOps(api, supabase, userId, resolveAgent) {
                 case "mark_read": {
                     if (!params.id)
                         return err("Missing required: id");
-                    const { error } = await supabase.from("notifications").update({ read: true }).eq("id", params.id);
+                    const { error } = await supabase.from("notifications").update({ read: true }).eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Notification marked as read", ok: true });
                 }
                 case "mark_all_read": {
-                    const { error } = await supabase.from("notifications").update({ read: true }).eq("read", false);
+                    const { error } = await supabase.from("notifications").update({ read: true }).eq("read", false).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "All notifications marked as read", ok: true });
@@ -2668,7 +2684,7 @@ function registerNotifyOps(api, supabase, userId, resolveAgent) {
                 case "delete": {
                     if (!params.id)
                         return err("Missing required: id");
-                    const { error } = await supabase.from("notifications").delete().eq("id", params.id);
+                    const { error } = await supabase.from("notifications").delete().eq("id", params.id).eq("user_id", userId);
                     if (error)
                         return err(error.message);
                     return ok({ message: "Notification deleted", ok: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.56.6",
+  "version": "4.56.8",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 18 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, execution plan builder, SOP management, agent brain, talent management, corporate frameworks, agent office canvas, and PM gate approvals",
   "keywords": [

package/src/tools.ts

@@ -946,7 +946,8 @@ async function handleCreateTask(
     const effectiveAgentId = (insertData.agent_id as string) || assignee;
     if (startDate && effectiveAgentId && normalized) {
       try {
-        let { nextRunAtEpoch } = normalized;
+        const originalEpoch = normalized.nextRunAtEpoch;
+        let nextRunAtEpoch = originalEpoch;
         const nowEpoch = Math.floor(Date.now() / 1000);
         // Safety net: if computed time is in the past, schedule for now + 60s
         // Fix #2 (2026-05-17): jitter past-time bump 0..5min so bulk creates
@@ -985,6 +986,22 @@ async function handleCreateTask(
           }
         }
 
+        // Fix #3 cosmetic parity (2026-05-17): when either Fix #2 jitter or
+        // Fix #3 stagger mutated nextRunAtEpoch, recompute the local-clock
+        // display fields so the dashboard UI doesn't show the ORIGINAL T0 for
+        // an event that actually fires minutes later. Dispatch uses
+        // next_run_at, but users read scheduled_date+scheduled_time.
+        let scheduledDateLocalFinal = normalized.scheduledDateLocal;
+        let scheduledTimeLocalFinal = normalized.scheduledTimeLocal;
+        if (nextRunAtEpoch !== originalEpoch) {
+          const offsetMin = Math.round(TZ_OFFSET_HOURS * 60);
+          const localMs = nextRunAtEpoch * 1000 + offsetMin * 60 * 1000;
+          const localD = new Date(localMs);
+          const padN = (n: number) => String(n).padStart(2, "0");
+          scheduledDateLocalFinal = `${localD.getUTCFullYear()}-${padN(localD.getUTCMonth() + 1)}-${padN(localD.getUTCDate())}`;
+          scheduledTimeLocalFinal = `${padN(localD.getUTCHours())}:${padN(localD.getUTCMinutes())}`;
+        }
+
         await supabase.from("scheduler_events").insert({
           id: crypto.randomUUID(),
           user_id: userId,
@@ -993,8 +1010,8 @@ async function handleCreateTask(
           subagent_id: subagentId,
           title: params.title,
           description: (params.description as string) || (params.instructions as string) || null,
-          scheduled_date: normalized.scheduledDateLocal,
-          scheduled_time: normalized.scheduledTimeLocal,
+          scheduled_date: scheduledDateLocalFinal,
+          scheduled_time: scheduledTimeLocalFinal,
           duration_minutes: 30,
           recurrence_type: (params.recurrence_type as string) || "none",
           recurrence_interval: (params.recurrence_interval as number) || 1,
@@ -1247,7 +1264,7 @@ async function handleDeleteTask(
     if (!params.task_id) return err("Missing required field: task_id");
     const taskId = params.task_id as string;
 
-    await supabase.from("scheduler_events").delete().eq("task_id", taskId);
+    await supabase.from("scheduler_events").delete().eq("task_id", taskId).eq("user_id", userId);
 
     const { data: subtasks } = await supabase
       .from("tasks")
@@ -1257,7 +1274,7 @@ async function handleDeleteTask(
 
     if (subtasks && subtasks.length > 0) {
       for (const sub of subtasks) {
-        await supabase.from("scheduler_events").delete().eq("task_id", sub.id);
+        await supabase.from("scheduler_events").delete().eq("task_id", sub.id).eq("user_id", userId);
       }
       await supabase
         .from("tasks")
@@ -1973,13 +1990,13 @@ function registerScheduleOps(
           if (upd.status === "completed" || upd.status === "cancelled") {
             upd.next_run_at = null;
           }
-          const { error } = await supabase.from("scheduler_events").update(upd).eq("id", params.id);
+          const { error } = await supabase.from("scheduler_events").update(upd).eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Event updated", ok: true });
         }
         case "delete": {
           if (!params.id) return err("Missing required: id");
-          const { error } = await supabase.from("scheduler_events").delete().eq("id", params.id);
+          const { error } = await supabase.from("scheduler_events").delete().eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Event deleted", ok: true });
         }
@@ -2090,13 +2107,13 @@ function registerKnowledgeOps(
           const upd: Record<string, any> = {};
           for (const k of allowed) if ((params as any)[k] !== undefined) upd[k] = (params as any)[k];
           if (Object.keys(upd).length === 0) return err("No valid fields to update");
-          const { error } = await supabase.from("knowledge_documents").update(upd).eq("id", params.id);
+          const { error } = await supabase.from("knowledge_documents").update(upd).eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Document updated", ok: true });
         }
         case "delete": {
           if (!params.id) return err("Missing required: id");
-          const { error } = await supabase.from("knowledge_documents").delete().eq("id", params.id);
+          const { error } = await supabase.from("knowledge_documents").delete().eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Document deleted", ok: true });
         }
@@ -2823,18 +2840,18 @@ function registerNotifyOps(
         }
         case "mark_read": {
           if (!params.id) return err("Missing required: id");
-          const { error } = await supabase.from("notifications").update({ read: true }).eq("id", params.id);
+          const { error } = await supabase.from("notifications").update({ read: true }).eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Notification marked as read", ok: true });
         }
         case "mark_all_read": {
-          const { error } = await supabase.from("notifications").update({ read: true }).eq("read", false);
+          const { error } = await supabase.from("notifications").update({ read: true }).eq("read", false).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "All notifications marked as read", ok: true });
         }
         case "delete": {
           if (!params.id) return err("Missing required: id");
-          const { error } = await supabase.from("notifications").delete().eq("id", params.id);
+          const { error } = await supabase.from("notifications").delete().eq("id", params.id).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Notification deleted", ok: true });
         }