ofiere-openclaw-plugin 4.56.8 → 4.56.10

package/src/sop-render.ts

@@ -1,216 +1,216 @@
-// src/sop-render.ts — Plugin-side markdown renderer for SOPs and Frameworks.
-// Mirrors dashboard/lib/sopRender.ts. Output is what gets prepended/appended to
-// the system prompt during `before_prompt_build`.
-
-interface SOPCheckItem { text?: string; checked?: boolean; }
-interface SOPStep {
-  name?: string; action?: string; owner?: string; output?: string;
-  decision_logic?: string; failure_state?: string; fallback_action?: string;
-  estimated_duration?: string;
-}
-interface SOPEscalationRule { trigger?: string; escalateTo?: string; priority?: string; }
-interface SOPPrerequisites {
-  conditions?: SOPCheckItem[]; required_tools?: string[];
-  required_permissions?: string[]; safety_warnings?: string[];
-}
-export interface SOPData {
-  title?: string;
-  purpose?: string; scope?: string; applicability?: string;
-  prerequisites?: SOPPrerequisites;
-  procedure_steps?: SOPStep[];
-  expected_outputs?: string[];
-  acceptance_criteria?: SOPCheckItem[];
-  escalation_rules?: SOPEscalationRule[];
-  rollback_procedure?: string;
-  notes?: string;
-}
-
-interface FrameworkObjective {
-  objective?: string; target?: string; measurement?: string; frequency?: string;
-}
-interface FrameworkEscalation {
-  trigger?: string; escalate_to?: string; priority?: string; response_sla?: string;
-}
-export interface FrameworkData {
-  title?: string;
-  mission?: string; vision?: string;
-  core_principles?: string[];
-  decision_authority?: string; budget_constraints?: string;
-  resource_limits?: string; tool_stack?: string[];
-  strategic_objectives?: FrameworkObjective[];
-  risk_appetite?: string; compliance_requirements?: string[];
-  escalation_matrix?: FrameworkEscalation[];
-  team_composition?: string; integration_points?: string[];
-  review_frequency?: string; last_reviewed?: string; next_review?: string;
-  notes?: string;
-}
-
-function bullet(items: (string | undefined | null)[]): string {
-  const cleaned = items.map((s) => (s ?? "").toString().trim()).filter(Boolean);
-  return cleaned.length ? cleaned.map((s) => `- ${s}`).join("\n") : "";
-}
-
-function section(label: string, body: string): string {
-  const trimmed = body.trim();
-  return trimmed ? `### ${label}\n${trimmed}` : "";
-}
-
-function joinSections(parts: string[]): string {
-  return parts.filter(Boolean).join("\n\n");
-}
-
-function safeParse<T = unknown>(s: string): T | null {
-  try { return JSON.parse(s) as T; } catch { return null; }
-}
-
-export function renderSopMarkdown(input: string | SOPData | null | undefined): string {
-  if (input == null) return "";
-  const data: SOPData = typeof input === "string"
-    ? (safeParse<SOPData>(input) || {})
-    : input;
-
-  const parts: string[] = [];
-  if (data.purpose) parts.push(section("Purpose", data.purpose));
-  if (data.scope) parts.push(section("Scope", data.scope));
-  if (data.applicability) parts.push(section("Applicability", data.applicability));
-
-  const prereq = data.prerequisites;
-  if (prereq) {
-    const pieces: string[] = [];
-    const cond = (prereq.conditions || []).map((c) => c?.text).filter(Boolean) as string[];
-    if (cond.length) pieces.push(`**Conditions:**\n${bullet(cond)}`);
-    if (prereq.required_tools?.length) pieces.push(`**Required tools:**\n${bullet(prereq.required_tools)}`);
-    if (prereq.required_permissions?.length) pieces.push(`**Required permissions:**\n${bullet(prereq.required_permissions)}`);
-    if (prereq.safety_warnings?.length) pieces.push(`**Safety warnings:**\n${bullet(prereq.safety_warnings)}`);
-    if (pieces.length) parts.push(section("Prerequisites", pieces.join("\n\n")));
-  }
-
-  if (data.procedure_steps?.length) {
-    const steps = data.procedure_steps.map((step, i) => {
-      const lines: string[] = [`${i + 1}. **${step.name || `Step ${i + 1}`}**`];
-      if (step.action) lines.push(`   - Action: ${step.action}`);
-      if (step.owner) lines.push(`   - Owner: ${step.owner}`);
-      if (step.output) lines.push(`   - Output: ${step.output}`);
-      if (step.decision_logic) lines.push(`   - Decision logic: ${step.decision_logic}`);
-      if (step.failure_state) lines.push(`   - Failure state: ${step.failure_state}`);
-      if (step.fallback_action) lines.push(`   - Fallback: ${step.fallback_action}`);
-      if (step.estimated_duration) lines.push(`   - ETA: ${step.estimated_duration}`);
-      return lines.join("\n");
-    }).join("\n");
-    parts.push(section("Procedure", steps));
-  }
-
-  if (data.expected_outputs?.length) parts.push(section("Expected Outputs", bullet(data.expected_outputs)));
-  if (data.acceptance_criteria?.length) {
-    parts.push(section("Acceptance Criteria", bullet(data.acceptance_criteria.map((c) => c?.text || ""))));
-  }
-
-  if (data.escalation_rules?.length) {
-    const rules = data.escalation_rules
-      .filter((r) => r.trigger || r.escalateTo)
-      .map((r) => `- [${r.priority || "P2"}] ${r.trigger || "(unspecified trigger)"} → ${r.escalateTo || "(unspecified)"}`)
-      .join("\n");
-    if (rules) parts.push(section("Escalation", rules));
-  }
-
-  if (data.rollback_procedure) parts.push(section("Rollback", data.rollback_procedure));
-  if (data.notes) parts.push(section("Notes", data.notes));
-
-  return joinSections(parts);
-}
-
-export function renderFrameworkMarkdown(input: string | FrameworkData | null | undefined): string {
-  if (input == null) return "";
-  const data: FrameworkData = typeof input === "string"
-    ? (safeParse<FrameworkData>(input) || {})
-    : input;
-
-  const parts: string[] = [];
-  if (data.mission) parts.push(section("Mission", data.mission));
-  if (data.vision) parts.push(section("Vision", data.vision));
-  if (data.core_principles?.length) parts.push(section("Core Principles", bullet(data.core_principles)));
-
-  const authParts: string[] = [];
-  if (data.decision_authority) authParts.push(`**Decision authority:** ${data.decision_authority}`);
-  if (data.budget_constraints) authParts.push(`**Budget constraints:** ${data.budget_constraints}`);
-  if (data.resource_limits) authParts.push(`**Resource limits:** ${data.resource_limits}`);
-  if (data.tool_stack?.length) authParts.push(`**Tool stack:**\n${bullet(data.tool_stack)}`);
-  if (authParts.length) parts.push(section("Authority & Constraints", authParts.join("\n\n")));
-
-  if (data.strategic_objectives?.length) {
-    const rows = data.strategic_objectives
-      .filter((o) => o.objective || o.target)
-      .map((o) => {
-        const bits = [
-          o.objective ? `**${o.objective}**` : "",
-          o.target ? `target: ${o.target}` : "",
-          o.measurement ? `measure: ${o.measurement}` : "",
-          o.frequency ? `cadence: ${o.frequency}` : "",
-        ].filter(Boolean);
-        return `- ${bits.join(" · ")}`;
-      })
-      .join("\n");
-    if (rows) parts.push(section("Strategic Objectives (KPIs)", rows));
-  }
-
-  const riskParts: string[] = [];
-  if (data.risk_appetite) riskParts.push(`**Risk appetite:** ${data.risk_appetite}`);
-  if (data.compliance_requirements?.length) riskParts.push(`**Compliance:**\n${bullet(data.compliance_requirements)}`);
-  if (riskParts.length) parts.push(section("Risk & Compliance", riskParts.join("\n\n")));
-
-  if (data.escalation_matrix?.length) {
-    const rows = data.escalation_matrix
-      .filter((e) => e.trigger || e.escalate_to)
-      .map((e) => {
-        const sla = e.response_sla ? ` (SLA ${e.response_sla})` : "";
-        return `- [${e.priority || "P2"}] ${e.trigger || "(unspecified)"} → ${e.escalate_to || "(unspecified)"}${sla}`;
-      })
-      .join("\n");
-    if (rows) parts.push(section("Escalation Matrix", rows));
-  }
-
-  const teamParts: string[] = [];
-  if (data.team_composition) teamParts.push(`**Team:** ${data.team_composition}`);
-  if (data.integration_points?.length) teamParts.push(`**Integration points:**\n${bullet(data.integration_points)}`);
-  if (teamParts.length) parts.push(section("Team & Integrations", teamParts.join("\n\n")));
-
-  if (data.review_frequency || data.next_review) {
-    const lines = [
-      data.review_frequency ? `**Review cadence:** ${data.review_frequency}` : "",
-      data.last_reviewed ? `**Last reviewed:** ${data.last_reviewed}` : "",
-      data.next_review ? `**Next review:** ${data.next_review}` : "",
-    ].filter(Boolean);
-    if (lines.length) parts.push(section("Review Cycle", lines.join("\n")));
-  }
-
-  if (data.notes) parts.push(section("Notes", data.notes));
-
-  return joinSections(parts);
-}
-
-export function renderAttachmentBlock(args: {
-  sops?: Array<{ title: string; content: string }>;
-  frameworks?: Array<{ title: string; content: string }>;
-}): string {
-  const blocks: string[] = [];
-
-  if (args.frameworks?.length) {
-    const sections = args.frameworks.map((fw) => {
-      const md = renderFrameworkMarkdown(fw.content);
-      if (!md) return `## ${fw.title}\n_(empty)_`;
-      return `## ${fw.title}\n${md}`;
-    }).join("\n\n");
-    blocks.push(`[FRAMEWORKS — ATTACHED]\nThese frameworks are active for this run. Orient strategic decisions toward them.\n\n${sections}`);
-  }
-
-  if (args.sops?.length) {
-    const sections = args.sops.map((sop) => {
-      const md = renderSopMarkdown(sop.content);
-      if (!md) return `## ${sop.title}\n_(empty)_`;
-      return `## ${sop.title}\n${md}`;
-    }).join("\n\n");
-    blocks.push(`[SOPS — ATTACHED]\nThese SOPs are active for this run. Follow the procedure steps and escalation rules.\n\n${sections}`);
-  }
-
-  return blocks.join("\n\n");
-}
+// src/sop-render.ts — Plugin-side markdown renderer for SOPs and Frameworks.
+// Mirrors dashboard/lib/sopRender.ts. Output is what gets prepended/appended to
+// the system prompt during `before_prompt_build`.
+
+interface SOPCheckItem { text?: string; checked?: boolean; }
+interface SOPStep {
+  name?: string; action?: string; owner?: string; output?: string;
+  decision_logic?: string; failure_state?: string; fallback_action?: string;
+  estimated_duration?: string;
+}
+interface SOPEscalationRule { trigger?: string; escalateTo?: string; priority?: string; }
+interface SOPPrerequisites {
+  conditions?: SOPCheckItem[]; required_tools?: string[];
+  required_permissions?: string[]; safety_warnings?: string[];
+}
+export interface SOPData {
+  title?: string;
+  purpose?: string; scope?: string; applicability?: string;
+  prerequisites?: SOPPrerequisites;
+  procedure_steps?: SOPStep[];
+  expected_outputs?: string[];
+  acceptance_criteria?: SOPCheckItem[];
+  escalation_rules?: SOPEscalationRule[];
+  rollback_procedure?: string;
+  notes?: string;
+}
+
+interface FrameworkObjective {
+  objective?: string; target?: string; measurement?: string; frequency?: string;
+}
+interface FrameworkEscalation {
+  trigger?: string; escalate_to?: string; priority?: string; response_sla?: string;
+}
+export interface FrameworkData {
+  title?: string;
+  mission?: string; vision?: string;
+  core_principles?: string[];
+  decision_authority?: string; budget_constraints?: string;
+  resource_limits?: string; tool_stack?: string[];
+  strategic_objectives?: FrameworkObjective[];
+  risk_appetite?: string; compliance_requirements?: string[];
+  escalation_matrix?: FrameworkEscalation[];
+  team_composition?: string; integration_points?: string[];
+  review_frequency?: string; last_reviewed?: string; next_review?: string;
+  notes?: string;
+}
+
+function bullet(items: (string | undefined | null)[]): string {
+  const cleaned = items.map((s) => (s ?? "").toString().trim()).filter(Boolean);
+  return cleaned.length ? cleaned.map((s) => `- ${s}`).join("\n") : "";
+}
+
+function section(label: string, body: string): string {
+  const trimmed = body.trim();
+  return trimmed ? `### ${label}\n${trimmed}` : "";
+}
+
+function joinSections(parts: string[]): string {
+  return parts.filter(Boolean).join("\n\n");
+}
+
+function safeParse<T = unknown>(s: string): T | null {
+  try { return JSON.parse(s) as T; } catch { return null; }
+}
+
+export function renderSopMarkdown(input: string | SOPData | null | undefined): string {
+  if (input == null) return "";
+  const data: SOPData = typeof input === "string"
+    ? (safeParse<SOPData>(input) || {})
+    : input;
+
+  const parts: string[] = [];
+  if (data.purpose) parts.push(section("Purpose", data.purpose));
+  if (data.scope) parts.push(section("Scope", data.scope));
+  if (data.applicability) parts.push(section("Applicability", data.applicability));
+
+  const prereq = data.prerequisites;
+  if (prereq) {
+    const pieces: string[] = [];
+    const cond = (prereq.conditions || []).map((c) => c?.text).filter(Boolean) as string[];
+    if (cond.length) pieces.push(`**Conditions:**\n${bullet(cond)}`);
+    if (prereq.required_tools?.length) pieces.push(`**Required tools:**\n${bullet(prereq.required_tools)}`);
+    if (prereq.required_permissions?.length) pieces.push(`**Required permissions:**\n${bullet(prereq.required_permissions)}`);
+    if (prereq.safety_warnings?.length) pieces.push(`**Safety warnings:**\n${bullet(prereq.safety_warnings)}`);
+    if (pieces.length) parts.push(section("Prerequisites", pieces.join("\n\n")));
+  }
+
+  if (data.procedure_steps?.length) {
+    const steps = data.procedure_steps.map((step, i) => {
+      const lines: string[] = [`${i + 1}. **${step.name || `Step ${i + 1}`}**`];
+      if (step.action) lines.push(`   - Action: ${step.action}`);
+      if (step.owner) lines.push(`   - Owner: ${step.owner}`);
+      if (step.output) lines.push(`   - Output: ${step.output}`);
+      if (step.decision_logic) lines.push(`   - Decision logic: ${step.decision_logic}`);
+      if (step.failure_state) lines.push(`   - Failure state: ${step.failure_state}`);
+      if (step.fallback_action) lines.push(`   - Fallback: ${step.fallback_action}`);
+      if (step.estimated_duration) lines.push(`   - ETA: ${step.estimated_duration}`);
+      return lines.join("\n");
+    }).join("\n");
+    parts.push(section("Procedure", steps));
+  }
+
+  if (data.expected_outputs?.length) parts.push(section("Expected Outputs", bullet(data.expected_outputs)));
+  if (data.acceptance_criteria?.length) {
+    parts.push(section("Acceptance Criteria", bullet(data.acceptance_criteria.map((c) => c?.text || ""))));
+  }
+
+  if (data.escalation_rules?.length) {
+    const rules = data.escalation_rules
+      .filter((r) => r.trigger || r.escalateTo)
+      .map((r) => `- [${r.priority || "P2"}] ${r.trigger || "(unspecified trigger)"} → ${r.escalateTo || "(unspecified)"}`)
+      .join("\n");
+    if (rules) parts.push(section("Escalation", rules));
+  }
+
+  if (data.rollback_procedure) parts.push(section("Rollback", data.rollback_procedure));
+  if (data.notes) parts.push(section("Notes", data.notes));
+
+  return joinSections(parts);
+}
+
+export function renderFrameworkMarkdown(input: string | FrameworkData | null | undefined): string {
+  if (input == null) return "";
+  const data: FrameworkData = typeof input === "string"
+    ? (safeParse<FrameworkData>(input) || {})
+    : input;
+
+  const parts: string[] = [];
+  if (data.mission) parts.push(section("Mission", data.mission));
+  if (data.vision) parts.push(section("Vision", data.vision));
+  if (data.core_principles?.length) parts.push(section("Core Principles", bullet(data.core_principles)));
+
+  const authParts: string[] = [];
+  if (data.decision_authority) authParts.push(`**Decision authority:** ${data.decision_authority}`);
+  if (data.budget_constraints) authParts.push(`**Budget constraints:** ${data.budget_constraints}`);
+  if (data.resource_limits) authParts.push(`**Resource limits:** ${data.resource_limits}`);
+  if (data.tool_stack?.length) authParts.push(`**Tool stack:**\n${bullet(data.tool_stack)}`);
+  if (authParts.length) parts.push(section("Authority & Constraints", authParts.join("\n\n")));
+
+  if (data.strategic_objectives?.length) {
+    const rows = data.strategic_objectives
+      .filter((o) => o.objective || o.target)
+      .map((o) => {
+        const bits = [
+          o.objective ? `**${o.objective}**` : "",
+          o.target ? `target: ${o.target}` : "",
+          o.measurement ? `measure: ${o.measurement}` : "",
+          o.frequency ? `cadence: ${o.frequency}` : "",
+        ].filter(Boolean);
+        return `- ${bits.join(" · ")}`;
+      })
+      .join("\n");
+    if (rows) parts.push(section("Strategic Objectives (KPIs)", rows));
+  }
+
+  const riskParts: string[] = [];
+  if (data.risk_appetite) riskParts.push(`**Risk appetite:** ${data.risk_appetite}`);
+  if (data.compliance_requirements?.length) riskParts.push(`**Compliance:**\n${bullet(data.compliance_requirements)}`);
+  if (riskParts.length) parts.push(section("Risk & Compliance", riskParts.join("\n\n")));
+
+  if (data.escalation_matrix?.length) {
+    const rows = data.escalation_matrix
+      .filter((e) => e.trigger || e.escalate_to)
+      .map((e) => {
+        const sla = e.response_sla ? ` (SLA ${e.response_sla})` : "";
+        return `- [${e.priority || "P2"}] ${e.trigger || "(unspecified)"} → ${e.escalate_to || "(unspecified)"}${sla}`;
+      })
+      .join("\n");
+    if (rows) parts.push(section("Escalation Matrix", rows));
+  }
+
+  const teamParts: string[] = [];
+  if (data.team_composition) teamParts.push(`**Team:** ${data.team_composition}`);
+  if (data.integration_points?.length) teamParts.push(`**Integration points:**\n${bullet(data.integration_points)}`);
+  if (teamParts.length) parts.push(section("Team & Integrations", teamParts.join("\n\n")));
+
+  if (data.review_frequency || data.next_review) {
+    const lines = [
+      data.review_frequency ? `**Review cadence:** ${data.review_frequency}` : "",
+      data.last_reviewed ? `**Last reviewed:** ${data.last_reviewed}` : "",
+      data.next_review ? `**Next review:** ${data.next_review}` : "",
+    ].filter(Boolean);
+    if (lines.length) parts.push(section("Review Cycle", lines.join("\n")));
+  }
+
+  if (data.notes) parts.push(section("Notes", data.notes));
+
+  return joinSections(parts);
+}
+
+export function renderAttachmentBlock(args: {
+  sops?: Array<{ title: string; content: string }>;
+  frameworks?: Array<{ title: string; content: string }>;
+}): string {
+  const blocks: string[] = [];
+
+  if (args.frameworks?.length) {
+    const sections = args.frameworks.map((fw) => {
+      const md = renderFrameworkMarkdown(fw.content);
+      if (!md) return `## ${fw.title}\n_(empty)_`;
+      return `## ${fw.title}\n${md}`;
+    }).join("\n\n");
+    blocks.push(`[FRAMEWORKS — ATTACHED]\nThese frameworks are active for this run. Orient strategic decisions toward them.\n\n${sections}`);
+  }
+
+  if (args.sops?.length) {
+    const sections = args.sops.map((sop) => {
+      const md = renderSopMarkdown(sop.content);
+      if (!md) return `## ${sop.title}\n_(empty)_`;
+      return `## ${sop.title}\n${md}`;
+    }).join("\n\n");
+    blocks.push(`[SOPS — ATTACHED]\nThese SOPs are active for this run. Follow the procedure steps and escalation rules.\n\n${sections}`);
+  }
+
+  return blocks.join("\n\n");
+}

package/src/supabase.ts

@@ -1,13 +1,13 @@
-import { createClient, type SupabaseClient } from "@supabase/supabase-js";
-
-let _client: SupabaseClient | null = null;
-
-export function getSupabase(supabaseUrl: string, serviceRoleKey: string): SupabaseClient {
-  if (_client) return _client;
-
-  _client = createClient(supabaseUrl, serviceRoleKey, {
-    auth: { persistSession: false },
-  });
-
-  return _client;
-}
+import { createClient, type SupabaseClient } from "@supabase/supabase-js";
+
+let _client: SupabaseClient | null = null;
+
+export function getSupabase(supabaseUrl: string, serviceRoleKey: string): SupabaseClient {
+  if (_client) return _client;
+
+  _client = createClient(supabaseUrl, serviceRoleKey, {
+    auth: { persistSession: false },
+  });
+
+  return _client;
+}

package/src/tools.ts

@@ -69,6 +69,20 @@ function err(message: string): ToolResult {
   };
 }
 
+/**
+ * D1 fix (2026-05-23): true when an LLM-supplied value is an "empty echo" —
+ * a field the model parroted back from a prior read with no real new content
+ * (null/undefined, empty or whitespace-only string, or empty array). The task
+ * update path is LLM-only; without this guard an echoed blank overwrites a
+ * real column. Mirrors isEmptyEcho in dashboard/lib/ofie/tool-executor.ts.
+ */
+function isEmptyEcho(v: unknown): boolean {
+  if (v === null || v === undefined) return true;
+  if (typeof v === "string") return v.trim() === "";
+  if (Array.isArray(v)) return v.length === 0;
+  return false;
+}
+
 // ─── Subagent ↔ chief invariant ──────────────────────────────────────────────
 // Mirrors dashboard/lib/subagentValidation.ts. Used by TASK_OPS + SCHEDULE_OPS
 // when a chief delegates work to one of their staff via tool call.
@@ -1095,8 +1109,16 @@ async function handleUpdateTask(
     // error instead of nulling the column. Coerce empty/whitespace → null here
     // to match the dashboard native executor (see tool-executor.ts handleTaskOps).
     const TIMESTAMP_FIELDS = ["start_date", "due_date"];
+    // D1 parity fix (2026-05-23): this update path is LLM-only. A completing
+    // agent routinely echoes the whole task object back with blank values for
+    // fields it did not change; without this guard those blanks overwrite real
+    // columns. An empty echo on a structural field means "leave unchanged".
+    // status/priority/progress are excluded (0 is valid); start_date/due_date
+    // keep the Fix #8 empty-string -> null coercion below.
+    const STRUCTURE_FIELDS = ["title", "description", "agent_id", "tags"];
     for (const f of fields) {
       if (params[f] !== undefined) {
+        if (STRUCTURE_FIELDS.includes(f) && isEmptyEcho(params[f])) continue;
         if (TIMESTAMP_FIELDS.includes(f)) {
           const v = params[f];
           updates[f] = (typeof v === "string" && v.trim() === "") ? null : v;
@@ -1147,12 +1169,15 @@ async function handleUpdateTask(
       }
     }
 
-    // If task is being marked DONE or FAILED, auto-complete any linked scheduler events
+    // If task is being marked DONE or FAILED, auto-complete any linked scheduler events.
+    // A-4 plugin parity (2026-05-18) — scope by user_id so cross-tenant task_id
+    // can't trigger foreign scheduler flip via plugin tool path.
     if (params.status === "DONE" || params.status === "FAILED") {
       try {
         await supabase
           .from("scheduler_events")
           .update({ status: "completed", next_run_at: null, updated_at: new Date().toISOString() })
+          .eq("user_id", userId)
           .eq("task_id", params.task_id as string);
       } catch (_schedErr) {
         // Non-fatal: task update should still proceed
@@ -2464,7 +2489,9 @@ function registerWorkflowOps(
         case "delete": {
           const wfId = (params.id || params.workflow_id) as string;
           if (!wfId) return err("Missing required: id");
-          await supabase.from("workflow_runs").delete().eq("workflow_id", wfId);
+          // A-4 plugin parity (2026-05-18) — scope cascade so cross-tenant
+          // workflow_id can't trigger foreign workflow_runs wipe via plugin.
+          await supabase.from("workflow_runs").delete().eq("user_id", userId).eq("workflow_id", wfId);
           const { error } = await supabase.from("workflows").delete().eq("id", wfId).eq("user_id", userId);
           if (error) return err(error.message);
           return ok({ message: "Workflow and associated runs deleted", ok: true });
@@ -4931,7 +4958,7 @@ function registerPlanOps(
       `- "delete": Remove plan. Required: plan_id\n` +
       `- "add_nodes": Add nodes to plan. Required: plan_id, nodes[]. Optional: parent_node_id\n` +
       `- "execute": Deploy plan to real PM tasks. Required: plan_id. Optional: create_folder, create_scheduler, space_id, folder_id\n\n` +
-      `Node structure: { type, title, description?, agent_id?, priority?, status?, start_date?, due_date?, tags?, execution_steps?[{text}], goals?[{label,type?}], constraints?[{label,type?}], system_prompt?, children?[], parallel? }`,
+      `Node structure: { type, title, description?, agent_id?, priority?, status?, start_date?, due_date?, tags?, execution_steps?[{text}], goals?[{label,type?}], constraints?[{label,type?}], system_prompt?, gate_condition?, children?[], parallel? }`,
     parameters: {
       type: "object",
       required: ["action"],
@@ -4964,8 +4991,13 @@ function registerPlanOps(
               goals: { type: "array", items: { type: "object", properties: { label: { type: "string" }, type: { type: "string" } }, required: ["label"] } },
               constraints: { type: "array", items: { type: "object", properties: { label: { type: "string" }, type: { type: "string" } }, required: ["label"] } },
               system_prompt: { type: "string" },
+              gate_condition: {
+                type: "string",
+                enum: ["manual", "all_predecessors_complete", "any_predecessor_complete"],
+                description: 'For type:"gate" nodes only. manual = human approval required; all_predecessors_complete = AND-join (default for gates); any_predecessor_complete = OR-join.',
+              },
               parallel: { type: "boolean" },
-              children: { type: "array", description: "Nested child nodes (recursive)", items: { type: "object", properties: { title: { type: "string" }, type: { type: "string", enum: ["task", "gate", "milestone"] }, description: { type: "string" }, agent_id: { type: "string" }, priority: { type: "number" }, status: { type: "string" }, start_date: { type: "string" }, due_date: { type: "string" }, parallel: { type: "boolean" } }, required: ["title"] } },
+              children: { type: "array", description: "Nested child nodes (recursive)", items: { type: "object", properties: { title: { type: "string" }, type: { type: "string", enum: ["task", "gate", "milestone"] }, description: { type: "string" }, agent_id: { type: "string" }, priority: { type: "number" }, status: { type: "string" }, start_date: { type: "string" }, due_date: { type: "string" }, parallel: { type: "boolean" }, gate_condition: { type: "string", enum: ["manual", "all_predecessors_complete", "any_predecessor_complete"] } }, required: ["title"] } },
             },
             required: ["title"],
           },
@@ -4998,6 +5030,17 @@ function generatePlanId(): string {
   return crypto.randomUUID();
 }
 
+const VALID_GATE_CONDITIONS = ["manual", "all_predecessors_complete", "any_predecessor_complete"] as const;
+
+/** Validate a plan node's gate_condition. Invalid/absent values fall back to the
+ *  existing default — gates get "all_predecessors_complete", non-gates get nothing. */
+function normalizeGateCondition(v: unknown, nodeType: unknown): string | undefined {
+  if (typeof v === "string" && (VALID_GATE_CONDITIONS as readonly string[]).includes(v)) {
+    return v;
+  }
+  return nodeType === "gate" ? "all_predecessors_complete" : undefined;
+}
+
 /** Normalize agent-provided node into the PlanNode shape stored in plan_data */
 function normalizeNode(raw: any): any {
   const id = raw.id || generatePlanNodeId();
@@ -5024,7 +5067,7 @@ function normalizeNode(raw: any): any {
       ? raw.constraints.map((c: any, i: number) => ({ id: `cstr-${Date.now()}-${i}`, type: c.type || "custom", label: typeof c === "string" ? c : c.label || String(c) }))
       : undefined,
     systemPrompt: raw.system_prompt || raw.systemPrompt || undefined,
-    gateCondition: raw.gate_condition || raw.gateCondition || (raw.type === "gate" ? "all_predecessors_complete" : undefined),
+    gateCondition: normalizeGateCondition(raw.gate_condition || raw.gateCondition, raw.type),
     children,
     parallel: raw.parallel || false,
     collapsed: false,

package/src/types/openclaw.d.ts

@@ -1,8 +1,8 @@
-// Ambient declaration for the OpenClaw plugin SDK. The gateway resolves this
-// module at runtime; npm has no published types. Keep loose — the plugin only
-// uses a small surface (logger, on, registerTool, registerCommand, etc.).
-
-declare module "openclaw/plugin-sdk" {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  export type OpenClawPluginApi = any;
-}
+// Ambient declaration for the OpenClaw plugin SDK. The gateway resolves this
+// module at runtime; npm has no published types. Keep loose — the plugin only
+// uses a small surface (logger, on, registerTool, registerCommand, etc.).
+
+declare module "openclaw/plugin-sdk" {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  export type OpenClawPluginApi = any;
+}

package/src/types.ts

@@ -1,10 +1,10 @@
-export interface OfiereConfig {
-  enabled: boolean;
-  supabaseUrl: string;
-  serviceRoleKey: string;
-  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;
-}
+export interface OfiereConfig {
+  enabled: boolean;
+  supabaseUrl: string;
+  serviceRoleKey: string;
+  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;
+}