ofiere-openclaw-plugin 4.32.0 → 4.33.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.32.0",
+  "version": "4.33.0",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 14 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, execution plan builder, SOP management, and agent brain (memory + self-improvement)",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/prompt.ts

@@ -159,7 +159,7 @@ ${toolIndex}
 - CONSTELLATION: ALWAYS read_blueprint before creating/editing agents. ALWAYS confirm before delete.
 - WORKFLOWS: ALWAYS "get" before modifying. Use "insert_node_between" for mid-flow additions.
 - CHANNEL REPORTS: ALWAYS include agent_id (YOUR name) in send_report. Use get_task_detail for drill-down.
-- PLANNING GATE: Before creating tasks with 3+ steps or due_dates, ask if user wants a Plan first.
+- **PLANNING GATE (MANDATORY):** Before creating ANY task with 3+ execution steps, goals, constraints, OR a due_date, you MUST ask the user: "Would you like me to plan this first, or create the task directly?" If the user chooses to plan, use OFIERE_PLAN_OPS to create a visual plan first, then execute it to generate the real tasks. Do NOT skip this step.
 - File refs @[name](file:ID): Use OFIERE_FILE_OPS read_text_file to retrieve — don't ask user.
 - Task approvals (OFIERE_TASK_OPS) ≠ workflow gate approvals (human_approval nodes).
 

package/src/tools.ts

@@ -198,7 +198,7 @@ function registerTaskOps(
       `Actions:\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` +
+      `- "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. ⚠️ PLANNING GATE: If the task has 3+ execution steps, goals, or constraints, FIRST ask the user "Plan first or create directly?" If they choose to plan, use OFIERE_PLAN_OPS instead.\n` +
       `- "update": Update a task. Required: task_id. Optional: all create fields + progress\n` +
       `- "delete": Delete task + subtasks. Required: task_id\n` +
       `- "add_approval": Request approval on a task. Required: task_id, approver_name. Optional: approver_type (human|agent, auto-detected), due_date, comment\n` +
@@ -241,7 +241,7 @@ function registerTaskOps(
         folder_id: { type: "string", description: "PM Folder ID" },
         start_date: { type: "string", description: "Start date (ISO 8601). Required for scheduled/recurring tasks." },
         due_date: { type: "string", description: "Due date (ISO 8601)" },
-        scheduled_time: { type: "string", description: "Time to execute in HH:MM format (UTC). If omitted, extracted from start_date or defaults to now+60s." },
+        scheduled_time: { type: "string", description: "Time to execute in HH:MM format (user's LOCAL time, e.g. 10:00 for 10 AM WIB). The system converts to UTC automatically." },
         recurrence_type: {
           type: "string",
           description: "How often the task recurs. 'none' for one-shot.",
@@ -542,8 +542,12 @@ async function handleCreateTask(
     // ── Auto-create scheduler event if task has a start_date ──────────────
     // This bridges the plugin → scheduler so the pg_cron task-dispatcher
     // Edge Function picks up the task at the right time.
+    //
+    // TIMEZONE: scheduled_time is treated as the user's LOCAL time (default WIB, UTC+7).
+    // We convert to UTC epoch for next_run_at so the edge function fires correctly.
     const startDate = params.start_date as string | undefined;
     const effectiveAgentId = (insertData.agent_id as string) || assignee;
+    const WIB_OFFSET_HOURS = 7; // Asia/Jakarta = UTC+7
     if (startDate && effectiveAgentId) {
       try {
         // Parse start_date robustly — it can be:
@@ -563,23 +567,41 @@ async function handleCreateTask(
           const hasTimeInfo = /[T ]\d{2}:\d{2}/.test(startDate);
 
           if (explicitScheduledTime) {
-            // Agent explicitly passed a scheduled_time — use date from start_date + explicit time
+            // Agent explicitly passed a scheduled_time — treat as WIB local time
             const dateStr = parsedDate.toISOString().split("T")[0]; // YYYY-MM-DD
-            const dt = new Date(`${dateStr}T${explicitScheduledTime}:00Z`);
+            const [localH, localM] = explicitScheduledTime.split(":").map(Number);
+            const utcH = localH - WIB_OFFSET_HOURS;
+            const dt = new Date(`${dateStr}T00:00:00Z`);
+            dt.setUTCHours(utcH, localM, 0, 0);
             nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
-            scheduledTimeFinal = explicitScheduledTime;
+            scheduledTimeFinal = explicitScheduledTime; // Store as user's local time
             scheduledDateFinal = dateStr;
           } else if (hasTimeInfo) {
-            // start_date already contains time — use it directly
-            nextRunAtEpoch = Math.floor(parsedDate.getTime() / 1000);
-            scheduledTimeFinal = `${String(parsedDate.getUTCHours()).padStart(2, "0")}:${String(parsedDate.getUTCMinutes()).padStart(2, "0")}`;
-            scheduledDateFinal = parsedDate.toISOString().split("T")[0];
+            // start_date already contains time — assume it's in the user's local timezone (WIB)
+            // Extract the local hour:minute from the string, NOT from UTC parsing
+            const timeMatch = startDate.match(/(\d{2}):(\d{2})/);
+            if (timeMatch) {
+              const localH = parseInt(timeMatch[1], 10);
+              const localM = parseInt(timeMatch[2], 10);
+              const dateStr = parsedDate.toISOString().split("T")[0];
+              const dt = new Date(`${dateStr}T00:00:00Z`);
+              dt.setUTCHours(localH - WIB_OFFSET_HOURS, localM, 0, 0);
+              nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
+              scheduledTimeFinal = `${String(localH).padStart(2, "0")}:${String(localM).padStart(2, "0")}`;
+              scheduledDateFinal = dateStr;
+            } else {
+              // Can't extract time — fall back to UTC parsing
+              nextRunAtEpoch = Math.floor(parsedDate.getTime() / 1000);
+              scheduledTimeFinal = `${String(parsedDate.getUTCHours()).padStart(2, "0")}:${String(parsedDate.getUTCMinutes()).padStart(2, "0")}`;
+              scheduledDateFinal = parsedDate.toISOString().split("T")[0];
+            }
           } else {
-            // Date only, no time — default to 09:00 UTC
+            // Date only, no time — default to 09:00 WIB (= 02:00 UTC)
             const dateStr = parsedDate.toISOString().split("T")[0];
-            const dt = new Date(`${dateStr}T09:00:00Z`);
+            const dt = new Date(`${dateStr}T00:00:00Z`);
+            dt.setUTCHours(9 - WIB_OFFSET_HOURS, 0, 0, 0); // 09:00 WIB = 02:00 UTC
             nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
-            scheduledTimeFinal = "09:00";
+            scheduledTimeFinal = "09:00"; // Stored as WIB local time
             scheduledDateFinal = dateStr;
           }
         } else {
@@ -611,6 +633,7 @@ async function handleCreateTask(
           next_run_at: nextRunAtEpoch,
           run_count: 0,
           priority: params.priority !== undefined ? params.priority : 1,
+          timezone: "Asia/Jakarta",
         });
       } catch (schedErr) {
         // Non-fatal: task was created, just the scheduler event failed