ofiere-openclaw-plugin 4.33.0 → 4.34.0

package/package.json

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

@@ -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",
@@ -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" },
@@ -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");
@@ -547,7 +566,20 @@ async function handleCreateTask(
     // 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
+
+    // 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:
@@ -567,17 +599,19 @@ async function handleCreateTask(
           const hasTimeInfo = /[T ]\d{2}:\d{2}/.test(startDate);
 
           if (explicitScheduledTime) {
-            // Agent explicitly passed a scheduled_time — treat as WIB local time
+            // Agent explicitly passed a scheduled_time — treat as user’s local time
             const dateStr = parsedDate.toISOString().split("T")[0]; // YYYY-MM-DD
             const [localH, localM] = explicitScheduledTime.split(":").map(Number);
-            const utcH = localH - WIB_OFFSET_HOURS;
+            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; // 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 — assume it's in the user's local timezone (WIB)
+            // 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) {
@@ -585,10 +619,12 @@ async function handleCreateTask(
               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);
+              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);
@@ -596,13 +632,15 @@ async function handleCreateTask(
               scheduledDateFinal = parsedDate.toISOString().split("T")[0];
             }
           } else {
-            // Date only, no time — default to 09:00 WIB (= 02: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}T00:00:00Z`);
-            dt.setUTCHours(9 - WIB_OFFSET_HOURS, 0, 0, 0); // 09:00 WIB = 02:00 UTC
+            dt.setUTCHours(9 - TZ_OFFSET_HOURS, 0, 0, 0); // 09:00 local = UTC - offset
             nextRunAtEpoch = Math.floor(dt.getTime() / 1000);
-            scheduledTimeFinal = "09:00"; // Stored as WIB local time
+            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
@@ -633,7 +671,7 @@ async function handleCreateTask(
           next_run_at: nextRunAtEpoch,
           run_count: 0,
           priority: params.priority !== undefined ? params.priority : 1,
-          timezone: "Asia/Jakarta",
+          timezone: timezone,
         });
       } catch (schedErr) {
         // Non-fatal: task was created, just the scheduler event failed
@@ -5611,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;
 }