ofiere-openclaw-plugin 4.52.0 → 4.53.0

package/dist/src/tools.js

@@ -233,7 +233,7 @@ function registerTaskOps(api, supabase, userId, resolveAgent, timezone) {
             `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, subagent_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` +
+            `- "create": Create a task in DRAFT mode. Required: title. Optional: agent_id, subagent_id, description, status, priority, space_id, folder_id, start_date, due_date, dispatch_immediately, tags, instructions, execution_plan, goals, constraints, system_prompt, recurrence_type, recurrence_interval, scheduled_time. ⚠️ EXECUTION SEMANTICS: omit start_date → task sits in agent's task list as draft (does NOT run). Set start_date to schedule. Pass dispatch_immediately:true to run now. ⚠️ 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 + subagent_id\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` +
@@ -279,8 +279,12 @@ function registerTaskOps(api, supabase, userId, resolveAgent, timezone) {
                 progress: { type: "number", description: "Progress percentage 0-100 (update only)" },
                 space_id: { type: "string", description: "PM Space ID" },
                 folder_id: { type: "string", description: "PM Folder ID" },
-                start_date: { type: "string", description: "Start date (ISO 8601). Required for scheduled/recurring tasks." },
+                start_date: { type: "string", description: "Start date (ISO 8601). Required for scheduled/recurring tasks. Omit to create as draft." },
                 due_date: { type: "string", description: "Due date (ISO 8601)" },
+                dispatch_immediately: {
+                    type: "boolean",
+                    description: "When true and start_date is not provided, sets start_date to NOW so the task dispatches on the next cron tick. Default: false (task is created as draft and waits for an explicit start_date).",
+                },
                 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",
@@ -666,15 +670,17 @@ async function handleCreateTask(supabase, userId, resolveAgent, params, fallback
         // which left the DB row with the unconverted value and the response with
         // the converted one — confusing and unsafe for the scheduler.
         //
-        // Cycle 12 fix: when delegating to a staff (subagent_id present) without
-        // an explicit start_date, default to now + 5s so the scheduler_event auto-
-        // create branch below kicks in. Without this, the task sat PENDING forever
-        // because processDirectTasks' date-string filter also skips it (separate
-        // bug, fixed in task-dispatcher v21). The 5s buffer ensures the safety
-        // net at line ~805 doesn't trip and shift the run to +60s.
+        // Cycle 13 (BUG 3+4): create is a DRAFT operation, NOT a dispatch trigger.
+        // No start_date → rawStartDate stays null → no scheduler_event auto-insert
+        // → task sits in agent's task list as draft. To execute, caller must pass
+        // an explicit start_date OR set dispatch_immediately:true (which we then
+        // resolve to "now" ISO here). Cycle 12's silent +5s default for staff
+        // delegation is REMOVED — it turned create into an unsafe execution
+        // trigger and bypassed the chief's approval flow.
         const explicitStartDate = params.start_date || null;
+        const dispatchNow = params.dispatch_immediately === true;
         const rawStartDate = explicitStartDate
-            ?? (params.subagent_id ? new Date(Date.now() + 5_000).toISOString() : null);
+            ?? (dispatchNow ? new Date().toISOString() : null);
         const normalized = rawStartDate
             ? normalizeStartDate(rawStartDate, params.scheduled_time, TZ_OFFSET_HOURS)
             : null;
@@ -807,8 +813,18 @@ async function handleUpdateTask(supabase, userId, params) {
             if (params[f] !== undefined)
                 updates[f] = params[f];
         }
-        if (params.status === "DONE")
+        // Cycle 13 (BUG 2): keep status and completed_at consistent. Setting status
+        // to DONE stamps completed_at + progress=100. Setting status to anything
+        // other than DONE clears completed_at so the row never carries the
+        // "PENDING + completed_at" contradiction the dashboard reported.
+        if (params.status === "DONE") {
             updates.completed_at = new Date().toISOString();
+            if (params.progress === undefined)
+                updates.progress = 100;
+        }
+        else if (typeof params.status === "string") {
+            updates.completed_at = null;
+        }
         // ── Subagent + chief invariant ───────────────────────────────────────
         // Mirrors dashboard PATCH: chief change without explicit subagent_id
         // clears any existing subagent assignment; subagent_id alone validates
@@ -2771,11 +2787,68 @@ async function generatePMReportDirect(supabase, userId, scopeType, scopeId, agen
     }
     lines.push("");
     lines.push(`💡 Reply "details on #N" for full task result`);
+    // Cycle 13 (BUG 1): render the "Generated" stamp in the user's IANA TZ
+    // (from profiles.timezone) instead of UTC. The plugin can't import
+    // dashboard helpers, so this uses Intl.DateTimeFormat directly.
+    const userTz = await loadUserTimezoneForPlugin(supabase, userId);
     const now = new Date();
     lines.push("");
-    lines.push(`Generated: ${now.toISOString().replace("T", " ").slice(0, 16)} UTC`);
+    lines.push(`Generated: ${formatInUserTzInline(now, userTz)}`);
     return { report: lines.join("\n"), scopeLabel, taskCount: all.length };
 }
+// ─── Plugin-local TZ helpers (Cycle 13, BUG 1) ──────────────────────────────
+// Mirror dashboard/lib/tz.ts; kept inline because the plugin is a separate
+// npm package loaded directly by the OpenClaw gateway and cannot import from
+// the dashboard.
+const PLUGIN_FALLBACK_TZ = "Asia/Jakarta";
+async function loadUserTimezoneForPlugin(supabase, userId) {
+    if (!userId)
+        return PLUGIN_FALLBACK_TZ;
+    try {
+        const { data } = await supabase
+            .from("profiles")
+            .select("timezone")
+            .eq("id", userId)
+            .maybeSingle();
+        const tz = data?.timezone?.trim();
+        return tz && isValidIanaZoneInline(tz) ? tz : PLUGIN_FALLBACK_TZ;
+    }
+    catch {
+        return PLUGIN_FALLBACK_TZ;
+    }
+}
+function formatInUserTzInline(date, timezone) {
+    if (Number.isNaN(date.getTime()))
+        return "";
+    const tz = isValidIanaZoneInline(timezone) ? timezone : PLUGIN_FALLBACK_TZ;
+    try {
+        const fmt = new Intl.DateTimeFormat("en-US", {
+            timeZone: tz,
+            year: "numeric",
+            month: "short",
+            day: "numeric",
+            hour: "2-digit",
+            minute: "2-digit",
+            hour12: false,
+            timeZoneName: "short",
+        });
+        return fmt.format(date).replace(", ", " ");
+    }
+    catch {
+        return date.toISOString();
+    }
+}
+function isValidIanaZoneInline(tz) {
+    if (!tz)
+        return false;
+    try {
+        new Intl.DateTimeFormat("en-US", { timeZone: tz }).format(new Date());
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
 async function dispatchReportDirect(supabase, userId, agentId, report, channelTypes) {
     // Query agent's active channel bindings
     let query = supabase.from("channel_bindings").select("channel_type, channel_account_id, agent_id, channel_config, thread_id")
@@ -6590,6 +6663,52 @@ function registerBrainExtractionHook(api, supabase, userId, fallbackAgentId) {
                                 catch (logErr) {
                                     api.logger.debug?.(`[ofiere-staff-report] dispatch_log update failed: ${logErr instanceof Error ? logErr.message : String(logErr)}`);
                                 }
+                                // Cycle 13 (BUG 2): flip the tasks row so the dashboard reflects
+                                // execution outcome. Without this, tasks stay at PENDING/IN_PROGRESS
+                                // forever and the proof-of-execution string lives only in
+                                // task_dispatch_log.response_preview (invisible to PM views).
+                                // Recurring tasks: stash output but leave status untouched so the
+                                // next cron tick can re-fire. One-shot: mark DONE, set progress=100,
+                                // stamp completed_at, persist proof in custom_fields.last_output.
+                                try {
+                                    const { data: schedRow } = await supabase
+                                        .from("scheduler_events")
+                                        .select("recurrence_type")
+                                        .eq("task_id", taskId)
+                                        .maybeSingle();
+                                    const isRecurring = !!(schedRow?.recurrence_type && schedRow.recurrence_type !== "none");
+                                    const { data: curTask } = await supabase
+                                        .from("tasks")
+                                        .select("custom_fields, status")
+                                        .eq("id", taskId)
+                                        .eq("user_id", userId)
+                                        .maybeSingle();
+                                    const cf = (curTask?.custom_fields ?? {});
+                                    cf.last_output = excerpt;
+                                    cf.last_completed_session = sessionKey ?? null;
+                                    cf.last_run_at = new Date().toISOString();
+                                    if (isRecurring) {
+                                        await supabase.from("tasks")
+                                            .update({ custom_fields: cf, updated_at: new Date().toISOString() })
+                                            .eq("id", taskId)
+                                            .eq("user_id", userId);
+                                    }
+                                    else if (curTask?.status !== "FAILED") {
+                                        await supabase.from("tasks")
+                                            .update({
+                                            status: "DONE",
+                                            completed_at: new Date().toISOString(),
+                                            progress: 100,
+                                            custom_fields: cf,
+                                            updated_at: new Date().toISOString(),
+                                        })
+                                            .eq("id", taskId)
+                                            .eq("user_id", userId);
+                                    }
+                                }
+                                catch (tErr) {
+                                    api.logger.debug?.(`[ofiere-staff-report] tasks update failed: ${tErr instanceof Error ? tErr.message : String(tErr)}`);
+                                }
                             }
                             // BUG 6 fix (BUGSHOOT-2): mirror the assistant turn into the
                             // dashboard's hidden conversation_messages so the dashboard can
@@ -6673,6 +6792,50 @@ function registerBrainExtractionHook(api, supabase, userId, fallbackAgentId) {
                     catch (logErr) {
                         api.logger.debug?.(`[ofiere-brain] dispatch_log update failed: ${logErr instanceof Error ? logErr.message : String(logErr)}`);
                     }
+                    // Cycle 13 (BUG 2): flip the tasks row for chief-direct (non-staff)
+                    // dispatches too — same logic as the staff path above. Recurring
+                    // tasks keep their status; one-shots flip to DONE with the proof
+                    // string stored in custom_fields.last_output.
+                    try {
+                        const excerpt = lastAssistant.slice(0, 1500);
+                        const { data: schedRow } = await supabase
+                            .from("scheduler_events")
+                            .select("recurrence_type")
+                            .eq("task_id", dispatchTaskIdFromCtx)
+                            .maybeSingle();
+                        const isRecurring = !!(schedRow?.recurrence_type && schedRow.recurrence_type !== "none");
+                        const { data: curTask } = await supabase
+                            .from("tasks")
+                            .select("custom_fields, status")
+                            .eq("id", dispatchTaskIdFromCtx)
+                            .eq("user_id", userId)
+                            .maybeSingle();
+                        const cf = (curTask?.custom_fields ?? {});
+                        cf.last_output = excerpt;
+                        cf.last_completed_session = sessionKey ?? null;
+                        cf.last_run_at = new Date().toISOString();
+                        if (isRecurring) {
+                            await supabase.from("tasks")
+                                .update({ custom_fields: cf, updated_at: new Date().toISOString() })
+                                .eq("id", dispatchTaskIdFromCtx)
+                                .eq("user_id", userId);
+                        }
+                        else if (curTask?.status !== "FAILED") {
+                            await supabase.from("tasks")
+                                .update({
+                                status: "DONE",
+                                completed_at: new Date().toISOString(),
+                                progress: 100,
+                                custom_fields: cf,
+                                updated_at: new Date().toISOString(),
+                            })
+                                .eq("id", dispatchTaskIdFromCtx)
+                                .eq("user_id", userId);
+                        }
+                    }
+                    catch (tErr) {
+                        api.logger.debug?.(`[ofiere-brain] tasks update failed: ${tErr instanceof Error ? tErr.message : String(tErr)}`);
+                    }
                 }
                 // Cycle 7b BUGSHOOT-4 — trivial-skip moved here from line ~6593.
                 // Brain L1/L2/L3/L4 extraction skips trivial chit-chat to reduce

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.52.0",
+  "version": "4.53.0",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 16 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, and corporate frameworks",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/tools.ts

@@ -271,7 +271,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, subagent_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` +
+      `- "create": Create a task in DRAFT mode. Required: title. Optional: agent_id, subagent_id, description, status, priority, space_id, folder_id, start_date, due_date, dispatch_immediately, tags, instructions, execution_plan, goals, constraints, system_prompt, recurrence_type, recurrence_interval, scheduled_time. ⚠️ EXECUTION SEMANTICS: omit start_date → task sits in agent's task list as draft (does NOT run). Set start_date to schedule. Pass dispatch_immediately:true to run now. ⚠️ 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 + subagent_id\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` +
@@ -317,8 +317,12 @@ function registerTaskOps(
         progress: { type: "number", description: "Progress percentage 0-100 (update only)" },
         space_id: { type: "string", description: "PM Space ID" },
         folder_id: { type: "string", description: "PM Folder ID" },
-        start_date: { type: "string", description: "Start date (ISO 8601). Required for scheduled/recurring tasks." },
+        start_date: { type: "string", description: "Start date (ISO 8601). Required for scheduled/recurring tasks. Omit to create as draft." },
         due_date: { type: "string", description: "Due date (ISO 8601)" },
+        dispatch_immediately: {
+          type: "boolean",
+          description: "When true and start_date is not provided, sets start_date to NOW so the task dispatches on the next cron tick. Default: false (task is created as draft and waits for an explicit start_date).",
+        },
         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",
@@ -749,15 +753,17 @@ async function handleCreateTask(
     // which left the DB row with the unconverted value and the response with
     // the converted one — confusing and unsafe for the scheduler.
     //
-    // Cycle 12 fix: when delegating to a staff (subagent_id present) without
-    // an explicit start_date, default to now + 5s so the scheduler_event auto-
-    // create branch below kicks in. Without this, the task sat PENDING forever
-    // because processDirectTasks' date-string filter also skips it (separate
-    // bug, fixed in task-dispatcher v21). The 5s buffer ensures the safety
-    // net at line ~805 doesn't trip and shift the run to +60s.
+    // Cycle 13 (BUG 3+4): create is a DRAFT operation, NOT a dispatch trigger.
+    // No start_date → rawStartDate stays null → no scheduler_event auto-insert
+    // → task sits in agent's task list as draft. To execute, caller must pass
+    // an explicit start_date OR set dispatch_immediately:true (which we then
+    // resolve to "now" ISO here). Cycle 12's silent +5s default for staff
+    // delegation is REMOVED — it turned create into an unsafe execution
+    // trigger and bypassed the chief's approval flow.
     const explicitStartDate = (params.start_date as string) || null;
+    const dispatchNow = params.dispatch_immediately === true;
     const rawStartDate = explicitStartDate
-      ?? (params.subagent_id ? new Date(Date.now() + 5_000).toISOString() : null);
+      ?? (dispatchNow ? new Date().toISOString() : null);
     const normalized = rawStartDate
       ? normalizeStartDate(rawStartDate, params.scheduled_time as string | undefined, TZ_OFFSET_HOURS)
       : null;
@@ -896,7 +902,16 @@ async function handleUpdateTask(
     for (const f of fields) {
       if (params[f] !== undefined) updates[f] = params[f];
     }
-    if (params.status === "DONE") updates.completed_at = new Date().toISOString();
+    // Cycle 13 (BUG 2): keep status and completed_at consistent. Setting status
+    // to DONE stamps completed_at + progress=100. Setting status to anything
+    // other than DONE clears completed_at so the row never carries the
+    // "PENDING + completed_at" contradiction the dashboard reported.
+    if (params.status === "DONE") {
+      updates.completed_at = new Date().toISOString();
+      if (params.progress === undefined) updates.progress = 100;
+    } else if (typeof params.status === "string") {
+      updates.completed_at = null;
+    }
 
     // ── Subagent + chief invariant ───────────────────────────────────────
     // Mirrors dashboard PATCH: chief change without explicit subagent_id
@@ -2941,13 +2956,72 @@ async function generatePMReportDirect(
   lines.push("");
   lines.push(`💡 Reply "details on #N" for full task result`);
 
+  // Cycle 13 (BUG 1): render the "Generated" stamp in the user's IANA TZ
+  // (from profiles.timezone) instead of UTC. The plugin can't import
+  // dashboard helpers, so this uses Intl.DateTimeFormat directly.
+  const userTz = await loadUserTimezoneForPlugin(supabase, userId);
   const now = new Date();
   lines.push("");
-  lines.push(`Generated: ${now.toISOString().replace("T", " ").slice(0, 16)} UTC`);
+  lines.push(`Generated: ${formatInUserTzInline(now, userTz)}`);
 
   return { report: lines.join("\n"), scopeLabel, taskCount: all.length };
 }
 
+// ─── Plugin-local TZ helpers (Cycle 13, BUG 1) ──────────────────────────────
+// Mirror dashboard/lib/tz.ts; kept inline because the plugin is a separate
+// npm package loaded directly by the OpenClaw gateway and cannot import from
+// the dashboard.
+
+const PLUGIN_FALLBACK_TZ = "Asia/Jakarta";
+
+async function loadUserTimezoneForPlugin(
+  supabase: SupabaseClient,
+  userId: string,
+): Promise<string> {
+  if (!userId) return PLUGIN_FALLBACK_TZ;
+  try {
+    const { data } = await supabase
+      .from("profiles")
+      .select("timezone")
+      .eq("id", userId)
+      .maybeSingle();
+    const tz = ((data as any)?.timezone as string | undefined)?.trim();
+    return tz && isValidIanaZoneInline(tz) ? tz : PLUGIN_FALLBACK_TZ;
+  } catch {
+    return PLUGIN_FALLBACK_TZ;
+  }
+}
+
+function formatInUserTzInline(date: Date, timezone: string): string {
+  if (Number.isNaN(date.getTime())) return "";
+  const tz = isValidIanaZoneInline(timezone) ? timezone : PLUGIN_FALLBACK_TZ;
+  try {
+    const fmt = new Intl.DateTimeFormat("en-US", {
+      timeZone: tz,
+      year: "numeric",
+      month: "short",
+      day: "numeric",
+      hour: "2-digit",
+      minute: "2-digit",
+      hour12: false,
+      timeZoneName: "short",
+    });
+    return fmt.format(date).replace(", ", " ");
+  } catch {
+    return date.toISOString();
+  }
+}
+
+function isValidIanaZoneInline(tz: string | undefined | null): boolean {
+  if (!tz) return false;
+  try {
+    new Intl.DateTimeFormat("en-US", { timeZone: tz }).format(new Date());
+    return true;
+  } catch {
+    return false;
+  }
+}
+
 async function dispatchReportDirect(
   supabase: SupabaseClient,
   userId: string,
@@ -6752,6 +6826,53 @@ function registerBrainExtractionHook(
                 } catch (logErr) {
                   api.logger.debug?.(`[ofiere-staff-report] dispatch_log update failed: ${logErr instanceof Error ? logErr.message : String(logErr)}`);
                 }
+
+                // Cycle 13 (BUG 2): flip the tasks row so the dashboard reflects
+                // execution outcome. Without this, tasks stay at PENDING/IN_PROGRESS
+                // forever and the proof-of-execution string lives only in
+                // task_dispatch_log.response_preview (invisible to PM views).
+                // Recurring tasks: stash output but leave status untouched so the
+                // next cron tick can re-fire. One-shot: mark DONE, set progress=100,
+                // stamp completed_at, persist proof in custom_fields.last_output.
+                try {
+                  const { data: schedRow } = await supabase
+                    .from("scheduler_events")
+                    .select("recurrence_type")
+                    .eq("task_id", taskId)
+                    .maybeSingle();
+                  const isRecurring = !!(schedRow?.recurrence_type && schedRow.recurrence_type !== "none");
+
+                  const { data: curTask } = await supabase
+                    .from("tasks")
+                    .select("custom_fields, status")
+                    .eq("id", taskId)
+                    .eq("user_id", userId)
+                    .maybeSingle();
+                  const cf = ((curTask?.custom_fields as Record<string, any>) ?? {}) as Record<string, any>;
+                  cf.last_output = excerpt;
+                  cf.last_completed_session = sessionKey ?? null;
+                  cf.last_run_at = new Date().toISOString();
+
+                  if (isRecurring) {
+                    await supabase.from("tasks")
+                      .update({ custom_fields: cf, updated_at: new Date().toISOString() })
+                      .eq("id", taskId)
+                      .eq("user_id", userId);
+                  } else if (curTask?.status !== "FAILED") {
+                    await supabase.from("tasks")
+                      .update({
+                        status: "DONE",
+                        completed_at: new Date().toISOString(),
+                        progress: 100,
+                        custom_fields: cf,
+                        updated_at: new Date().toISOString(),
+                      })
+                      .eq("id", taskId)
+                      .eq("user_id", userId);
+                  }
+                } catch (tErr) {
+                  api.logger.debug?.(`[ofiere-staff-report] tasks update failed: ${tErr instanceof Error ? tErr.message : String(tErr)}`);
+                }
               }
 
               // BUG 6 fix (BUGSHOOT-2): mirror the assistant turn into the
@@ -6834,6 +6955,51 @@ function registerBrainExtractionHook(
           } catch (logErr) {
             api.logger.debug?.(`[ofiere-brain] dispatch_log update failed: ${logErr instanceof Error ? logErr.message : String(logErr)}`);
           }
+
+          // Cycle 13 (BUG 2): flip the tasks row for chief-direct (non-staff)
+          // dispatches too — same logic as the staff path above. Recurring
+          // tasks keep their status; one-shots flip to DONE with the proof
+          // string stored in custom_fields.last_output.
+          try {
+            const excerpt = lastAssistant.slice(0, 1500);
+            const { data: schedRow } = await supabase
+              .from("scheduler_events")
+              .select("recurrence_type")
+              .eq("task_id", dispatchTaskIdFromCtx)
+              .maybeSingle();
+            const isRecurring = !!(schedRow?.recurrence_type && schedRow.recurrence_type !== "none");
+
+            const { data: curTask } = await supabase
+              .from("tasks")
+              .select("custom_fields, status")
+              .eq("id", dispatchTaskIdFromCtx)
+              .eq("user_id", userId)
+              .maybeSingle();
+            const cf = ((curTask?.custom_fields as Record<string, any>) ?? {}) as Record<string, any>;
+            cf.last_output = excerpt;
+            cf.last_completed_session = sessionKey ?? null;
+            cf.last_run_at = new Date().toISOString();
+
+            if (isRecurring) {
+              await supabase.from("tasks")
+                .update({ custom_fields: cf, updated_at: new Date().toISOString() })
+                .eq("id", dispatchTaskIdFromCtx)
+                .eq("user_id", userId);
+            } else if (curTask?.status !== "FAILED") {
+              await supabase.from("tasks")
+                .update({
+                  status: "DONE",
+                  completed_at: new Date().toISOString(),
+                  progress: 100,
+                  custom_fields: cf,
+                  updated_at: new Date().toISOString(),
+                })
+                .eq("id", dispatchTaskIdFromCtx)
+                .eq("user_id", userId);
+            }
+          } catch (tErr) {
+            api.logger.debug?.(`[ofiere-brain] tasks update failed: ${tErr instanceof Error ? tErr.message : String(tErr)}`);
+          }
         }
 
         // Cycle 7b BUGSHOOT-4 — trivial-skip moved here from line ~6593.