ofiere-openclaw-plugin 4.37.1 → 4.37.2

package/package.json

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

@@ -5244,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",
@@ -5278,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>,
@@ -5296,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 });
     }
@@ -5363,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)