ofiere-openclaw-plugin 4.32.0 → 4.34.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.32.0",
+  "version": "4.34.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/config.ts

@@ -7,6 +7,7 @@ export const OfiereConfigSchema = z.object({
   serviceRoleKey: z.string().default(""),
   userId: z.string().default(""),
   agentId: z.string().default(""),
+  timezone: z.string().default("Asia/Jakarta"),
 });
 
 // Cache the first successful config so per-agent re-registrations don't lose it
@@ -52,12 +53,20 @@ export function parseOfiereConfig(value: unknown): OfiereConfig {
     _cachedConfig?.agentId ||
     "";
 
+  const timezone =
+    (typeof configObj?.timezone === "string" && configObj.timezone.trim()) ||
+    (typeof raw.timezone === "string" && raw.timezone.trim()) ||
+    process.env.OFIERE_TIMEZONE ||
+    _cachedConfig?.timezone ||
+    "Asia/Jakarta";
+
   const parsed = OfiereConfigSchema.parse({
     ...raw,
     supabaseUrl,
     serviceRoleKey,
     userId,
     agentId,
+    timezone,
   });
 
   // Cache if this parse yielded a complete config

package/src/prompt.ts

@@ -146,6 +146,11 @@ Full parameter docs are in each tool's description — call the tool to see deta
 
 ${toolIndex}
 
+## ⛔ PLANNING GATE (NON-NEGOTIABLE)
+- Before creating ANY task with 3+ total execution steps, goals, or constraints, 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.
+- NEVER skip this step. The server will block creation of complex tasks — use action: "create_force" only if the user explicitly confirms bypass.
+
 ## Rules
 - ALWAYS pass agent_id with your own name when creating tasks (e.g. agent_id: "ivy").
 - ${assignRule}
@@ -159,7 +164,6 @@ ${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.
 - 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

@@ -189,6 +189,7 @@ function registerTaskOps(
   supabase: SupabaseClient,
   userId: string,
   resolveAgent: (id?: string) => Promise<string | null>,
+  timezone: string,
 ): void {
   api.registerTool({
     name: "OFIERE_TASK_OPS",
@@ -198,7 +199,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` +
@@ -217,7 +218,7 @@ function registerTaskOps(
         action: {
           type: "string",
           description: "The operation to perform",
-          enum: ["list", "get", "create", "update", "delete", "add_approval", "list_approvals", "resolve_approval"],
+          enum: ["list", "get", "create", "create_force", "update", "delete", "add_approval", "list_approvals", "resolve_approval"],
         },
         title: { type: "string", description: "Task title (required for create)" },
         description: { type: "string", description: "Task description" },
@@ -241,7 +242,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.",
@@ -314,8 +315,25 @@ function registerTaskOps(
         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 "create": {
+          // ── PLANNING GATE — server-side enforcement ──
+          // Trigger only on steps/goals/constraints count >= 3
+          const execSteps = Array.isArray(params.execution_plan) ? (params.execution_plan as any[]).length : 0;
+          const goalCount = Array.isArray(params.goals) ? (params.goals as any[]).length : 0;
+          const constraintCount = Array.isArray(params.constraints) ? (params.constraints as any[]).length : 0;
+          const complexityCount = execSteps + goalCount + constraintCount;
+          if (complexityCount >= 3) {
+            return ok({
+              blocked: true,
+              message: `⚠️ PLANNING GATE: This task has ${complexityCount} execution steps/goals/constraints (${execSteps} steps, ${goalCount} goals, ${constraintCount} constraints). Planning Mode is mandatory for complex tasks. Ask the user: "Would you like me to plan this first using OFIERE_PLAN_OPS, or force-create with action: 'create_force'?"`,
+              hint: "To bypass: re-call with action 'create_force'. To plan first: use OFIERE_PLAN_OPS.",
+            });
+          }
+          return handleCreateTask(supabase, userId, resolveAgent, params, timezone);
+        }
+        case "create_force":
+          // Bypasses planning gate — agent explicitly chose to skip planning
+          return handleCreateTask(supabase, userId, resolveAgent, params, timezone);
         case "update":
           return handleUpdateTask(supabase, userId, params);
         case "delete":
@@ -328,7 +346,7 @@ function registerTaskOps(
           return handleResolveApproval(supabase, userId, params);
         default:
           return err(
-            `Unknown action "${action}". Valid actions: list, get, create, update, delete, add_approval, list_approvals, resolve_approval`,
+            `Unknown action "${action}". Valid actions: list, get, create, create_force, update, delete, add_approval, list_approvals, resolve_approval`,
           );
       }
     },
@@ -397,6 +415,7 @@ async function handleCreateTask(
   userId: string,
   resolveAgent: (id?: string) => Promise<string | null>,
   params: Record<string, unknown>,
+  timezone: string = "Asia/Jakarta",
 ): Promise<ToolResult> {
   try {
     if (!params.title) return err("Missing required field: title");
@@ -542,8 +561,25 @@ 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;
+
+    // Resolve timezone offset from IANA timezone string (configurable per user)
+    const getTimezoneOffsetHours = (tz: string): number => {
+      try {
+        const now = new Date();
+        const utcStr = now.toLocaleString('en-US', { timeZone: 'UTC' });
+        const tzStr = now.toLocaleString('en-US', { timeZone: tz });
+        const diffMs = new Date(tzStr).getTime() - new Date(utcStr).getTime();
+        return Math.round(diffMs / 3600000);
+      } catch {
+        return 7; // Fallback to WIB (UTC+7)
+      }
+    };
+    const TZ_OFFSET_HOURS = getTimezoneOffsetHours(timezone);
     if (startDate && effectiveAgentId) {
       try {
         // Parse start_date robustly — it can be:
@@ -563,24 +599,48 @@ 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 user’s 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 - TZ_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;
+            // Normalize start_date to ISO UTC for consistent downstream parsing
+            insertData.start_date = dt.toISOString();
           } 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
+            // 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 - TZ_OFFSET_HOURS, localM, 0, 0);
+              nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
+              scheduledTimeFinal = `${String(localH).padStart(2, "0")}:${String(localM).padStart(2, "0")}`;
+              scheduledDateFinal = dateStr;
+              // Normalize start_date to ISO UTC
+              insertData.start_date = dt.toISOString();
+            } 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 user local time
             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 - TZ_OFFSET_HOURS, 0, 0, 0); // 09:00 local = UTC - offset
             nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
-            scheduledTimeFinal = "09:00";
+            scheduledTimeFinal = "09:00"; // Stored as user's local time
             scheduledDateFinal = dateStr;
+            // Normalize start_date to ISO UTC
+            insertData.start_date = dt.toISOString();
           }
         } else {
           // Unparseable date — fallback to now + 60s
@@ -611,6 +671,7 @@ async function handleCreateTask(
           next_run_at: nextRunAtEpoch,
           run_count: 0,
           priority: params.priority !== undefined ? params.priority : 1,
+          timezone: timezone,
         });
       } catch (schedErr) {
         // Non-fatal: task was created, just the scheduler event failed
@@ -5588,7 +5649,7 @@ export function registerTools(
   const resolveAgent = createAgentResolver(api, supabase, userId, fallbackAgentId);
 
   // ── Register each domain meta-tool ──
-  registerTaskOps(api, supabase, userId, resolveAgent);       // 1
+  registerTaskOps(api, supabase, userId, resolveAgent, config.timezone);       // 1
   registerAgentOps(api, supabase, userId, fallbackAgentId);   // 2
   registerProjectOps(api, supabase, userId);                  // 3
   registerScheduleOps(api, supabase, userId);                 // 4

package/src/types.ts

@@ -5,4 +5,6 @@ export interface OfiereConfig {
   userId: string;
   /** Optional — if not set, agent identity is resolved at runtime from OpenClaw context */
   agentId: string;
+  /** IANA timezone string for the user (e.g. 'Asia/Jakarta', 'America/New_York'). Default: 'Asia/Jakarta' */
+  timezone: string;
 }