ofiere-openclaw-plugin 4.37.0 → 4.37.2

package/package.json

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

@@ -107,7 +107,8 @@ Actions: "list", "get", "create", "update", "delete", "add_nodes", "execute"
 
   OFIERE_SOP_OPS: `Standard Operating Procedures for department chiefs.
 Actions: "list_templates", "create", "list", "get", "update", "delete", "list_subagents", "apply_template"
-- sop_data: { title, objective, scope, prerequisites[], steps[], deliverables[], escalationRules[], successCriteria[], notes }
+- sop_data: { title, purpose, scope, applicability, prerequisites:{ conditions, required_tools, required_permissions, safety_warnings }, procedure_steps[], expected_outputs[], escalation_rules[], acceptance_criteria[], rollback_procedure, notes }
+- Legacy field names still accepted (objective→purpose, steps→procedure_steps, deliverables→expected_outputs, escalationRules→escalation_rules, successCriteria→acceptance_criteria) — prefer new names.
 - See SOP PROTOCOL section for when to load vs skip`,
 
   OFIERE_BRAIN_OPS: `Agent memory, knowledge graph, self-improvement (TMT/MAGMA).

package/src/tools.ts

@@ -4879,7 +4879,27 @@ function registerSOPOps(
             purpose: { type: "string", description: "Why this procedure exists (also accepts 'objective')" },
             scope: { type: "string" },
             applicability: { type: "string" },
-            prerequisites: { type: "object", description: "{ conditions: [{text,checked}], required_tools: [string], required_permissions: [string], safety_warnings: [string] }" },
+            prerequisites: {
+              type: "object",
+              description: "Preconditions for executing this SOP. All sub-fields optional.",
+              properties: {
+                conditions: {
+                  type: "array",
+                  description: "Checklist items that must be true before starting.",
+                  items: {
+                    type: "object",
+                    properties: {
+                      text: { type: "string" },
+                      checked: { type: "boolean" },
+                    },
+                    required: ["text"],
+                  },
+                },
+                required_tools: { type: "array", items: { type: "string" } },
+                required_permissions: { type: "array", items: { type: "string" } },
+                safety_warnings: { type: "array", items: { type: "string" } },
+              },
+            },
             procedure_steps: { type: "array", description: "Also accepts 'steps'", items: { type: "object", properties: { id: { type: "string" }, name: { type: "string" }, action: { type: "string" }, owner: { type: "string" }, output: { type: "string" }, decision_logic: { type: "string" }, failure_state: { type: "string" }, fallback_action: { type: "string" }, estimated_duration: { type: "string" } }, required: ["name", "action"] } },
             expected_outputs: { type: "array", description: "Also accepts 'deliverables'", items: { type: "string" } },
             escalation_rules: { type: "array", description: "Also accepts 'escalationRules'", items: { type: "object", properties: { trigger: { type: "string" }, escalateTo: { type: "string" }, priority: { type: "string", enum: ["P1", "P2", "P3"] } }, required: ["trigger", "escalateTo"] } },
@@ -5224,9 +5244,11 @@ function registerFrameworkOps(
       `\\"strategic_objectives\\":[{\\"objective\\":\\"Grow MRR\\",\\"target\\":\\"$100k\\",\\"measurement\\":\\"Revenue dashboard\\"}],` +
       `\\"risk_appetite\\":\\"moderate\\",\\"escalation_matrix\\":[{\\"trigger\\":\\"Budget >80%\\",\\"escalate_to\\":\\"CEO\\",\\"priority\\":\\"P1\\"}],` +
       `\\"notes\\":\\"Quarterly review\\"}"\n\n` +
-      `Supported content fields: mission, vision, core_principles, decision_authority, budget_constraints, ` +
-      `resource_limits, tool_stack, strategic_objectives, risk_appetite, compliance_requirements, ` +
-      `escalation_matrix, team_composition, integration_points, review_frequency, notes — or any custom fields.\n\n` +
+      `Field types (must match — wrong shapes are coerced):\n` +
+      `- string: mission, vision, decision_authority, budget_constraints, resource_limits, risk_appetite, team_composition, review_frequency, last_reviewed, next_review, notes\n` +
+      `- string[]: core_principles, tool_stack, compliance_requirements, integration_points, tags\n` +
+      `- object[]: strategic_objectives [{objective,target,measurement,frequency}], escalation_matrix [{trigger,escalate_to,priority,response_sla}]\n` +
+      `- IMPORTANT: team_composition is a single STRING (a paragraph or newline-separated list), NOT an array. If you have multiple roles, join with newlines inside one string.\n\n` +
       `Status values: "draft", "active", "under_review", "archived"`,
     parameters: {
       type: "object",
@@ -5258,6 +5280,33 @@ function registerFrameworkOps(
 
 // ── Framework action handlers ────────────────────────────────────────────────
 
+// Coerce field values to the shape the dashboard editor expects. Without this,
+// LLMs frequently emit team_composition / core_principles etc. as the "wrong"
+// container shape and the editor crashes on .trim() / .map() / .length.
+const FW_STRING_FIELDS = ["mission", "vision", "decision_authority", "budget_constraints", "resource_limits", "risk_appetite", "team_composition", "review_frequency", "last_reviewed", "next_review", "rollback_procedure", "notes", "framework_id", "version", "category"] as const;
+const FW_STRING_ARRAY_FIELDS = ["core_principles", "tool_stack", "compliance_requirements", "integration_points", "tags"] as const;
+
+function fwToString(v: unknown): string {
+  if (typeof v === "string") return v;
+  if (v == null) return "";
+  if (Array.isArray(v)) return v.map((x) => (typeof x === "string" ? x : JSON.stringify(x))).join("\n");
+  if (typeof v === "object") { try { return JSON.stringify(v); } catch { return ""; } }
+  return String(v);
+}
+
+function fwToStringArray(v: unknown): string[] {
+  if (Array.isArray(v)) return v.map((x) => (typeof x === "string" ? x : JSON.stringify(x)));
+  if (typeof v === "string" && v.trim()) return v.split(/\r?\n/).map((s) => s.trim()).filter(Boolean);
+  return [];
+}
+
+function normalizeFrameworkContent(raw: Record<string, unknown>): Record<string, unknown> {
+  const out: Record<string, unknown> = { ...raw };
+  for (const k of FW_STRING_FIELDS) if (k in out) out[k] = fwToString(out[k]);
+  for (const k of FW_STRING_ARRAY_FIELDS) if (k in out) out[k] = fwToStringArray(out[k]);
+  return out;
+}
+
 async function handleFWCreate(
   supabase: SupabaseClient, userId: string,
   resolveAgent: (id?: string) => Promise<string | null>,
@@ -5276,13 +5325,13 @@ async function handleFWCreate(
       // Already a JSON string — merge with title
       try {
         const parsed = JSON.parse(params.content);
-        content = JSON.stringify({ title, ...parsed });
+        content = JSON.stringify(normalizeFrameworkContent({ title, ...parsed }));
       } catch {
         // Not valid JSON — wrap as notes
-        content = JSON.stringify({ title, notes: params.content });
+        content = JSON.stringify(normalizeFrameworkContent({ title, notes: params.content }));
       }
     } else if (params.framework_data && typeof params.framework_data === "object") {
-      content = JSON.stringify({ title, ...(params.framework_data as Record<string, unknown>) });
+      content = JSON.stringify(normalizeFrameworkContent({ title, ...(params.framework_data as Record<string, unknown>) }));
     } else {
       content = JSON.stringify({ title });
     }
@@ -5343,13 +5392,13 @@ async function handleFWUpdate(supabase: SupabaseClient, userId: string, params:
     if (typeof params.content === "string" && params.content.trim()) {
       try {
         const parsed = JSON.parse(params.content);
-        updates.content = JSON.stringify({ title: parsed.title || (params.title as string) || "", ...parsed });
+        updates.content = JSON.stringify(normalizeFrameworkContent({ title: parsed.title || (params.title as string) || "", ...parsed }));
       } catch {
-        updates.content = JSON.stringify({ title: (params.title as string) || "", notes: params.content });
+        updates.content = JSON.stringify(normalizeFrameworkContent({ title: (params.title as string) || "", notes: params.content }));
       }
     } else if (params.framework_data && typeof params.framework_data === "object") {
       const raw = params.framework_data as Record<string, unknown>;
-      updates.content = JSON.stringify({ title: (raw.title as string) || (params.title as string) || "", ...raw });
+      updates.content = JSON.stringify(normalizeFrameworkContent({ title: (raw.title as string) || (params.title as string) || "", ...raw }));
     }
 
     const { data, error } = await supabase.from("frameworks").update(updates)