ofiere-openclaw-plugin 4.39.2 → 4.40.1

package/package.json

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

@@ -13,7 +13,7 @@
 
 const TOOL_SUMMARIES: Record<string, string> = {
   OFIERE_TASK_OPS: "Manage tasks: list, create, update, delete, approvals",
-  OFIERE_AGENT_OPS: "Query agents: list all with IDs, names, roles",
+  OFIERE_AGENT_OPS: "Query + manage agents and their staff subagents",
   OFIERE_PROJECT_OPS: "PM hierarchy: spaces, folders, dependencies",
   OFIERE_SCHEDULE_OPS: "Calendar: list, create, update, delete events",
   OFIERE_KNOWLEDGE_OPS: "Knowledge library: search, list, create, update, delete",
@@ -48,7 +48,12 @@ Actions: "list", "create", "update", "delete", "add_approval", "list_approvals",
 - list_approvals: List approvals. Optional: task_id, approval_status (pending|approved|rejected)
 - resolve_approval: Approve/reject. Requires: approval_id, approval_status. Optional: comment`,
 
-  OFIERE_AGENT_OPS: `Query agents. Action: "list" — see all agents with IDs, names, roles for task assignment.`,
+  OFIERE_AGENT_OPS: `Query + manage agents and their staff subagents.
+Actions: "list", "list_subagents", "create_subagent", "delete_subagent"
+- list: All top-level agents (chiefs / native + OpenClaw) with IDs, names, roles. Use for task assignment lookup.
+- list_subagents: Staff under a chief. Required: chief_agent_id.
+- create_subagent: Add a staff subagent under a chief (max 5 per chief). Required: chief_agent_id, name. Optional: role (default "Staff"), codename, color_hex.
+- delete_subagent: Remove a staff subagent. Required: subagent_id.`,
 
   OFIERE_PROJECT_OPS: `Manage PM hierarchy.
 Actions: "list_spaces", "create_space", "update_space", "delete_space", "list_folders", "create_folder", "update_folder", "delete_folder", "list_dependencies", "add_dependency", "remove_dependency"

package/src/tools.ts

@@ -1088,9 +1088,12 @@ function registerAgentOps(
     name: "OFIERE_AGENT_OPS",
     label: "Ofiere Agent Operations",
     description:
-      `Query agents in the Ofiere PM system.\n\n` +
+      `Query and manage agents + their staff subagents in the Ofiere PM system.\n\n` +
       `Actions:\n` +
-      `- "list": List all available agents with their IDs, names, roles, and status. Use this to find the correct agent_id for task assignment.`,
+      `- "list": List all top-level agents (chiefs / native + OpenClaw) with their IDs, names, roles, and status. Use this to find the correct agent_id for task assignment.\n` +
+      `- "list_subagents": List staff subagents under a chief. Required: chief_agent_id.\n` +
+      `- "create_subagent": Create a staff subagent under a chief (max 5 per chief). Required: chief_agent_id, name. Optional: role (default "Staff"), codename, color_hex (default "#64748b").\n` +
+      `- "delete_subagent": Remove a staff subagent. Required: subagent_id.`,
     parameters: {
       type: "object",
       required: ["action"],
@@ -1098,8 +1101,14 @@ function registerAgentOps(
         action: {
           type: "string",
           description: "The operation to perform",
-          enum: ["list"],
+          enum: ["list", "list_subagents", "create_subagent", "delete_subagent"],
         },
+        chief_agent_id: { type: "string", description: "Chief agent ID (required for list_subagents, create_subagent)" },
+        subagent_id: { type: "string", description: "Subagent ID (required for delete_subagent)" },
+        name: { type: "string", description: "Subagent display name (required for create_subagent)" },
+        role: { type: "string", description: "Subagent role label, e.g. 'Staff', 'Analyst'. Defaults to 'Staff'." },
+        codename: { type: "string", description: "Optional subagent codename" },
+        color_hex: { type: "string", description: "Optional subagent UI color, default '#64748b'" },
       },
     },
     async execute(_id: string, params: Record<string, unknown>) {
@@ -1108,8 +1117,14 @@ function registerAgentOps(
       switch (action) {
         case "list":
           return handleListAgents(api, supabase, userId, fallbackAgentId);
+        case "list_subagents":
+          return handleListSubagents(supabase, userId, params);
+        case "create_subagent":
+          return handleCreateSubagent(supabase, userId, params);
+        case "delete_subagent":
+          return handleDeleteSubagent(supabase, userId, params);
         default:
-          return err(`Unknown action "${action}". Valid actions: list`);
+          return err(`Unknown action "${action}". Valid actions: list, list_subagents, create_subagent, delete_subagent`);
       }
     },
   });
@@ -4847,7 +4862,7 @@ function registerSOPOps(
       `- "get": Get full SOP details. Required: sop_id\n` +
       `- "update": Modify SOP. Required: sop_id. Optional: title, sop_data, status, department\n` +
       `- "delete": Remove SOP. Required: sop_id\n` +
-      `- "list_subagents": List subagents for a chief. Required: chief_agent_id\n` +
+      `- "list_subagents": List subagents for a chief. Required: chief_agent_id. (Provided here for convenience when scoping SOP authoring; team management lives in OFIERE_AGENT_OPS.)\n` +
       `- "apply_template": Create SOP from template. Required: agent_id, template_id. Optional: title, department\n` +
       `- "propose_attach": Propose attaching SOPs to a run target (conversation/task/scheduler_event). Returns token cost + confirmation_token. Required: target_kind, target_id, doc_ids[]. The user MUST be asked to approve before commit.\n` +
       `- "commit_attach": Commit a proposed attachment. Required: target_kind, target_id, doc_ids[], confirmation_token (from propose_attach). Only call AFTER user approves the token cost.\n` +
@@ -4931,7 +4946,7 @@ function registerSOPOps(
         case "get": return handleSOPGet(supabase, userId, params);
         case "update": return handleSOPUpdate(supabase, userId, params);
         case "delete": return handleSOPDelete(supabase, userId, params);
-        case "list_subagents": return handleSOPListSubagents(supabase, userId, params);
+        case "list_subagents": return handleListSubagents(supabase, userId, params);
         case "apply_template": return handleSOPApplyTemplate(supabase, userId, resolveAgent, params);
         case "propose_attach": return handleProposeAttach({ supabase, userId, docKind: "sop", params });
         case "commit_attach": return handleCommitAttach({ supabase, userId, docKind: "sop", params });
@@ -5176,7 +5191,12 @@ async function handleSOPDelete(supabase: SupabaseClient, userId: string, params:
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }
 
-async function handleSOPListSubagents(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
+// ── Shared subagent handlers (used by OFIERE_AGENT_OPS; SOP_OPS keeps a
+//    convenience wrapper for `list_subagents` only). ──────────────────────────
+
+const MAX_SUBAGENTS_PER_CHIEF = 5;
+
+async function handleListSubagents(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
   try {
     if (!params.chief_agent_id) return err("Missing required field: chief_agent_id");
     const { data, error } = await supabase
@@ -5186,7 +5206,57 @@ async function handleSOPListSubagents(supabase: SupabaseClient, userId: string,
       .eq("chief_agent_id", params.chief_agent_id as string)
       .order("created_at", { ascending: true });
     if (error) return err(error.message);
-    return ok({ subagents: data || [], count: (data || []).length, max_allowed: 5 });
+    return ok({ subagents: data || [], count: (data || []).length, max_allowed: MAX_SUBAGENTS_PER_CHIEF });
+  } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
+}
+
+async function handleCreateSubagent(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
+  try {
+    const chiefAgentId = params.chief_agent_id as string | undefined;
+    const name = params.name as string | undefined;
+    if (!chiefAgentId) return err("Missing required field: chief_agent_id");
+    if (!name) return err("Missing required field: name");
+
+    // Enforce per-chief cap (mirrors POST /api/sop/subagents in dashboard)
+    const { count, error: countErr } = await supabase
+      .from("agent_subagents")
+      .select("id", { count: "exact", head: true })
+      .eq("user_id", userId)
+      .eq("chief_agent_id", chiefAgentId);
+    if (countErr) return err(countErr.message);
+    if ((count ?? 0) >= MAX_SUBAGENTS_PER_CHIEF) {
+      return err(`Maximum ${MAX_SUBAGENTS_PER_CHIEF} subagents per department chief`);
+    }
+
+    const role = (params.role as string) || "Staff";
+    const codename = (params.codename as string) ?? null;
+    const colorHex = (params.color_hex as string) || "#64748b";
+
+    const { data, error } = await supabase
+      .from("agent_subagents")
+      .insert({ user_id: userId, chief_agent_id: chiefAgentId, name, role, codename, color_hex: colorHex })
+      .select("id, user_id, chief_agent_id, name, role, codename, color_hex, created_at")
+      .single();
+    if (error) return err(error.message);
+    return ok({ subagent: data, message: `Subagent "${name}" created under chief ${chiefAgentId}` });
+  } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
+}
+
+async function handleDeleteSubagent(supabase: SupabaseClient, userId: string, params: Record<string, unknown>): Promise<ToolResult> {
+  try {
+    const subagentId = params.subagent_id as string | undefined;
+    if (!subagentId) return err("Missing required field: subagent_id");
+
+    const { data, error } = await supabase
+      .from("agent_subagents")
+      .delete()
+      .eq("id", subagentId)
+      .eq("user_id", userId)
+      .select("id, name")
+      .maybeSingle();
+    if (error) return err(error.message);
+    if (!data) return err("Subagent not found — nothing deleted");
+    return ok({ message: `Subagent "${data.name}" deleted`, deleted: true });
   } catch (e) { return err(e instanceof Error ? e.message : String(e)); }
 }