ofiere-openclaw-plugin 4.24.0 → 4.26.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ofiere-openclaw-plugin",
-  "version": "4.24.0",
+  "version": "4.26.0",
   "type": "module",
   "description": "OpenClaw plugin for Ofiere PM - 13 meta-tools covering tasks, agents, projects, scheduling, knowledge, workflows, notifications, memory, prompts, constellation, space file management, execution plan builder, and SOP management",
   "keywords": ["openclaw", "ofiere", "project-management", "agents", "plugin"],

package/src/prompt.ts

@@ -62,14 +62,20 @@ const TOOL_DOCS: Record<string, string> = {
     - Edge handles: condition edges use sourceHandle "condition-true"/"condition-false". Loop edges use "loop_body"/"done"
     - Variables: Use {{prev.nodeId.outputText}} for prior outputs, {{variables.key}} for stored variables`,
 
-  OFIERE_NOTIFY_OPS: `- **OFIERE_NOTIFY_OPS** — Notifications & Channel Reports (action: "list", "mark_read", "mark_all_read", "delete", "send_report", "schedule_report", "list_schedules", "delete_schedule")
+  OFIERE_NOTIFY_OPS: `- **OFIERE_NOTIFY_OPS** — Notifications & Channel Reports (action: "list", "mark_read", "mark_all_read", "delete", "send_report", "get_task_detail", "schedule_report", "list_schedules", "delete_schedule")
     - list: Recent notifications. unread_only=true for unread only
     - mark_read: Mark one notification read by ID
     - mark_all_read: Mark all as read
-    - send_report: Send a PM progress report to YOUR connected channels (Telegram, Discord, Slack, etc.)
+    - send_report: Send a numbered PM progress report to YOUR connected channels (Telegram, Discord, Slack, etc.)
       - Required: scope_type (space|folder|project|task|all), agent_id (YOUR name, e.g. "thalia")
       - Optional: scope_id, channel_types[] (filter to specific channels), include_completed
       - The report is automatically generated from current PM data and sent through YOUR channel bindings ONLY
+      - Tasks are numbered globally (1, 2, 3...) across Active/Pending/Done sections. Recurring tasks that executed recently appear in BOTH Done (today's run) and Pending (next cycle)
+    - get_task_detail: Get the FULL result/content of a task by its report number
+      - Required: task_number (1-indexed number from the report)
+      - Optional: scope_type (default all), scope_id
+      - Returns: complete task description (where execution results live), custom_fields, status, tags, and all metadata
+      - Use when user asks "details on number X", "what's task 2 result", "show me #3", etc.
     - schedule_report: Create a recurring/scheduled report
       - Required: scope_type, recurrence_type (hourly|daily|weekly|monthly), agent_id (YOUR name)
       - Optional: scope_id, scope_label, channel_types[], recurrence_time (HH:MM UTC, default 09:00), recurrence_interval, recurrence_days_of_week (e.g. "mon,wed,fri"), include_completed
@@ -202,7 +208,8 @@ ${toolDocs}
 - When task instructions or system prompts contain file references like @[filename](file:FILE_ID), use OFIERE_FILE_OPS action:"read_text_file" file_id:"FILE_ID" to read the file content. Do NOT ask the user for the file — retrieve it yourself.
 - Use OFIERE_FILE_OPS to create output files (reports, data, configs) in the Space Files explorer. Prefer create_text_file for text-based outputs.
 - To save task output as a file, call OFIERE_FILE_OPS action:"create_text_file" with the space_id from the task context.
-- CHANNEL REPORTS: When the user asks you to "send a report", "send progress", "update me on Telegram/Discord/Slack/WhatsApp", use OFIERE_NOTIFY_OPS action:"send_report" with agent_id set to YOUR name. ALWAYS include agent_id — without it the report will fail. The report is generated from live PM data and sent through YOUR connected channels ONLY — not other agents' channels.
+- CHANNEL REPORTS: When the user asks you to "send a report", "send progress", "update me on Telegram/Discord/Slack/WhatsApp", use OFIERE_NOTIFY_OPS action:"send_report" with agent_id set to YOUR name. ALWAYS include agent_id — without it the report will fail. The report is generated from live PM data and sent through YOUR connected channels ONLY — not other agents' channels. Tasks are numbered (1, 2, 3...) for easy reference.
+- TASK DRILL-DOWN: When the user asks "details on number X", "what's task X result", "show me #X", or any variation referencing a numbered task from a report, use OFIERE_NOTIFY_OPS action:"get_task_detail" with task_number set to the number they mentioned. The description field contains the actual execution result (e.g. scan findings, analysis output). Present the description content as the primary response — that IS the task result.
 - To set up recurring reports (e.g. "send me a daily report at 9am"), use OFIERE_NOTIFY_OPS action:"schedule_report" with scope_type, recurrence_type, and agent_id set to YOUR name.
 - If a report is too long for the channel's message limit, save the full report as a markdown file using OFIERE_FILE_OPS action:"create_text_file" and send a summary to the channel instead.
 - PLANNING WORKFLOW: Use OFIERE_PLAN_OPS to build complex multi-step execution flows BEFORE creating individual tasks. Build the plan → let the user review in the Planning Tab → execute when approved.
@@ -215,6 +222,13 @@ ${toolDocs}
 - When creating SOPs for department chiefs (Thalia=CMO, Ivy=COO, Daisy=CTO-Intel, Celia=CTO-Eng), tailor content to their domain expertise.
 - Prerequisites should be actionable checklist items. Success criteria should be measurable outcomes.
 - After creating an SOP, suggest the agent set it to "active" status when ready for execution.
+- PLANNING GATE: Before creating ANY task with 3+ execution steps, a due_date, or when the user describes a multi-phase project, ALWAYS ask: "Should I create a Plan first so you can review the structure before I create individual tasks?" If the user says yes, use OFIERE_PLAN_OPS. If they say no or it's a simple one-shot task, proceed directly with OFIERE_TASK_OPS.
+- TASK PLACEMENT — SOP-AWARE ROUTING: When creating a task, the system auto-assigns a PM space. But for FOLDER placement, follow this protocol:
+  1. If the user explicitly provides space_id or folder_id, use those directly.
+  2. If not, and you have active SOPs (loaded via 🔴 COMPLEX assessment), check if any SOP defines an operating structure or folder routing (e.g. "place marketing tasks in Marketing/Campaigns"). Follow the SOP's structure.
+  3. If no SOP guidance exists, check the Space Files tab for an operating map/structure document using OFIERE_FILE_OPS action:"list_files". If found, read it with "read_text_file" and follow its routing rules.
+  4. If no routing guidance exists at all, ask the user: "I can place this task in your default space root, or create a new folder/project for it. Which do you prefer?"
+  5. Do NOT blindly dump all SOPs to check routing — smart-select only SOPs whose title/department matches the task domain.
 
 ## SOP PROTOCOL — Adaptive Complexity Assessment
 

package/src/tools.ts

@@ -441,6 +441,61 @@ async function handleCreateTask(
       cf.assignees = [{ id: assignee, type: "agent" }];
     }
 
+    // ── Intelligent space_id resolution ──────────────────────────────────
+    // Priority: explicit param > agent's default_space_id > first existing space > auto-create
+    let resolvedSpaceId = (params.space_id as string) || null;
+    let resolvedFolderId = (params.folder_id as string) || null;
+    let spaceAutoCreated = false;
+
+    if (!resolvedSpaceId) {
+      try {
+        // 1. Check agent's configured default_space_id
+        if (assignee) {
+          const { data: agentRow } = await supabase
+            .from("agents")
+            .select("default_space_id")
+            .eq("id", assignee)
+            .eq("user_id", userId)
+            .single();
+          if (agentRow?.default_space_id) {
+            resolvedSpaceId = agentRow.default_space_id;
+          }
+        }
+
+        // 2. Fallback: use the first existing space for this user
+        if (!resolvedSpaceId) {
+          const { data: existingSpaces } = await supabase
+            .from("pm_spaces")
+            .select("id")
+            .eq("user_id", userId)
+            .order("created_at", { ascending: true })
+            .limit(1);
+          if (existingSpaces && existingSpaces.length > 0) {
+            resolvedSpaceId = existingSpaces[0].id;
+          }
+        }
+
+        // 3. Nuclear fallback: auto-create a space
+        if (!resolvedSpaceId) {
+          const newSpaceId = crypto.randomUUID();
+          const { error: spaceErr } = await supabase.from("pm_spaces").insert({
+            id: newSpaceId,
+            user_id: userId,
+            name: "Operations",
+            icon: "🏢",
+            icon_color: "#FF6D29",
+            sort_order: 0,
+          });
+          if (!spaceErr) {
+            resolvedSpaceId = newSpaceId;
+            spaceAutoCreated = true;
+          }
+        }
+      } catch {
+        // Non-fatal: task will still be created, just without space context
+      }
+    }
+
     const insertData: Record<string, unknown> = {
       id,
       user_id: userId,
@@ -450,8 +505,8 @@ async function handleCreateTask(
       assignee_type: "agent",
       status: (params.status as string) || "PENDING",
       priority: params.priority !== undefined ? params.priority : 1,
-      space_id: (params.space_id as string) || null,
-      folder_id: (params.folder_id as string) || null,
+      space_id: resolvedSpaceId,
+      folder_id: resolvedFolderId,
       start_date: (params.start_date as string) || null,
       due_date: (params.due_date as string) || (params.start_date as string) || null,
       tags: (params.tags as string[]) || [],
@@ -578,6 +633,17 @@ async function handleCreateTask(
       id,
       message: `Task "${params.title}" created and assigned to ${assignee || "no one"}${extrasStr}`,
       task: insertData,
+      spacePlacement: resolvedSpaceId
+        ? {
+            space_id: resolvedSpaceId,
+            auto_created: spaceAutoCreated,
+            note: spaceAutoCreated
+              ? 'A new "Operations" space was auto-created because no PM spaces existed.'
+              : resolvedFolderId
+                ? `Placed in folder ${resolvedFolderId}`
+                : "Placed in space root. To organize, check your SOPs/operating structure for folder routing, or specify folder_id.",
+          }
+        : undefined,
       scheduledExecution: didSchedule ? `Will auto-execute on ${startDate}` : undefined,
       recurrence: recurrenceInfo,
     });
@@ -2156,9 +2222,13 @@ function registerNotifyOps(
       `- "mark_read": Mark one as read. Required: id\n` +
       `- "mark_all_read": Mark all as read\n` +
       `- "delete": Delete a notification. Required: id\n` +
-      `- "send_report": Send a PM progress report to YOUR connected channels (Telegram, Discord, etc.)\n` +
+      `- "send_report": Send a numbered PM progress report to YOUR connected channels (Telegram, Discord, etc.)\n` +
       `    Required: scope_type (space|folder|project|task|all), agent_id (YOUR name e.g. "thalia")\n` +
       `    Optional: scope_id, channel_types[] (filter specific channels), include_completed (default true)\n` +
+      `    Tasks are numbered globally (1, 2, 3...) across Active/Pending/Done sections for easy reference\n` +
+      `- "get_task_detail": Get the FULL result/content of a task by its report number. Use when user asks "details on number X"\n` +
+      `    Required: task_number (1-indexed from the report). Optional: scope_type, scope_id (defaults to all)\n` +
+      `    Returns the complete task description, execution results, status, and metadata\n` +
       `- "schedule_report": Create a recurring/scheduled report\n` +
       `    Required: scope_type, recurrence_type (hourly|daily|weekly|monthly), agent_id (YOUR name)\n` +
       `    Optional: scope_id, scope_label, channel_types[], recurrence_time (HH:MM UTC, default 09:00),\n` +
@@ -2169,10 +2239,11 @@ function registerNotifyOps(
       type: "object",
       required: ["action"],
       properties: {
-        action: { type: "string", enum: ["list", "mark_read", "mark_all_read", "delete", "send_report", "schedule_report", "list_schedules", "delete_schedule"] },
+        action: { type: "string", enum: ["list", "mark_read", "mark_all_read", "delete", "send_report", "get_task_detail", "schedule_report", "list_schedules", "delete_schedule"] },
         agent_id: { type: "string", description: "Your agent name or ID (required for send_report/schedule_report)" },
         id: { type: "string", description: "Notification ID" },
         schedule_id: { type: "string", description: "Schedule ID (for delete_schedule)" },
+        task_number: { type: "number", description: "1-indexed task number from the report (for get_task_detail)" },
         unread_only: { type: "boolean", description: "Only show unread" },
         limit: { type: "number", description: "Max results (default 50)" },
         scope_type: { type: "string", enum: ["space", "folder", "project", "task", "all"], description: "Report scope" },
@@ -2257,6 +2328,69 @@ function registerNotifyOps(
           });
         }
 
+        // ── GET TASK DETAIL — Drill into task result by report number ──
+        case "get_task_detail": {
+          const taskNumber = params.task_number as number;
+          if (!taskNumber || taskNumber < 1) return err("Missing required: task_number (1-indexed number from the report)");
+          const detailScopeType = (params.scope_type as string) || "all";
+          const detailScopeId = params.scope_id as string | undefined;
+
+          // Re-run the same query the report uses to get consistent ordering
+          let dq = supabase.from("tasks").select("id, title, description, status, priority, progress, completed_at, due_date, updated_at, custom_fields, tags, agent_id")
+            .eq("user_id", userId).is("parent_task_id", null);
+          if (detailScopeType === "space" && detailScopeId) dq = dq.eq("space_id", detailScopeId);
+          else if (detailScopeType === "folder" && detailScopeId) dq = dq.eq("folder_id", detailScopeId);
+          else if (detailScopeType === "project" && detailScopeId) dq = dq.eq("project_id", detailScopeId);
+          else if (detailScopeType === "task" && detailScopeId) dq = dq.eq("id", detailScopeId);
+
+          const { data: detailTasks } = await dq.order("priority", { ascending: false }).limit(40);
+          const dtAll = detailTasks || [];
+
+          // Build same ordered list as report: Active → Pending → Done
+          const dtInProgress = dtAll.filter((t: any) => t.status === "IN_PROGRESS" || t.status === "RUNNING");
+          const dtPending = dtAll.filter((t: any) => t.status === "PENDING" || t.status === "NEW");
+          const dtCompleted = dtAll.filter((t: any) => t.status === "DONE");
+          const dtRecentThreshold = 24 * 60 * 60 * 1000;
+          const dtNowMs = Date.now();
+          const dtRecentlyExecuted = dtPending.filter((t: any) =>
+            t.description && t.description.length > 50 &&
+            t.updated_at && (dtNowMs - new Date(t.updated_at).getTime()) < dtRecentThreshold
+          );
+
+          // Same ordering as report formatter
+          const dtOrdered: any[] = [];
+          const dtSeen = new Set<string>();
+          const dtAdd = (t: any) => { if (!dtSeen.has(t.id)) { dtSeen.add(t.id); dtOrdered.push(t); } };
+          for (const t of dtInProgress.slice(0, 8)) dtAdd(t);
+          for (const t of dtPending.slice(0, 6)) dtAdd(t);
+          for (const t of dtRecentlyExecuted) dtAdd(t);
+          for (const t of dtCompleted.slice(0, 5)) dtAdd(t);
+
+          if (taskNumber > dtOrdered.length) {
+            return err(`Task #${taskNumber} not found. The report has ${dtOrdered.length} numbered task(s). Valid range: 1-${dtOrdered.length}.`);
+          }
+
+          const target = dtOrdered[taskNumber - 1];
+          const isRecurring = dtRecentlyExecuted.some((r: any) => r.id === target.id);
+
+          return ok({
+            task_number: taskNumber,
+            task_id: target.id,
+            title: target.title,
+            status: target.status,
+            priority: target.priority === 3 ? "CRITICAL" : target.priority === 2 ? "HIGH" : target.priority === 1 ? "MEDIUM" : "LOW",
+            progress: target.progress,
+            is_recurring_today: isRecurring,
+            description: target.description || "(no description)",
+            custom_fields: target.custom_fields || {},
+            tags: target.tags || [],
+            agent_id: target.agent_id,
+            due_date: target.due_date,
+            completed_at: target.completed_at,
+            updated_at: target.updated_at,
+          });
+        }
+
         // ── SCHEDULE REPORT — Create recurring report ──
         case "schedule_report": {
           const scopeType = (params.scope_type as string) || "all";
@@ -2319,7 +2453,7 @@ function registerNotifyOps(
         }
 
         default:
-          return err(`Unknown action "${action}". Valid: list, mark_read, mark_all_read, delete, send_report, schedule_report, list_schedules, delete_schedule`);
+          return err(`Unknown action "${action}". Valid: list, mark_read, mark_all_read, delete, send_report, get_task_detail, schedule_report, list_schedules, delete_schedule`);
       }
     },
   });
@@ -2336,7 +2470,7 @@ async function generatePMReportDirect(
   includeCompleted: boolean,
 ): Promise<{ report: string; scopeLabel: string; taskCount: number }> {
   let scopeLabel = "All Projects";
-  let query = supabase.from("tasks").select("id, title, status, priority, progress, completed_at, due_date")
+  let query = supabase.from("tasks").select("id, title, status, priority, progress, completed_at, due_date, description, updated_at")
     .eq("user_id", userId).is("parent_task_id", null);
 
   if (scopeType === "space" && scopeId) {
@@ -2367,24 +2501,53 @@ async function generatePMReportDirect(
     .limit(1).single();
   if (agentRow) agentName = agentRow.name || agentRow.codename || agentId;
 
+  // Detect recently-executed recurring tasks: PENDING but updated within 24h with content
+  const recentThreshold = 24 * 60 * 60 * 1000;
+  const nowMs = Date.now();
+  const isRecentlyExecuted = (t: any) =>
+    (t.status === "PENDING" || t.status === "NEW") &&
+    t.description && t.description.length > 50 &&
+    t.updated_at && (nowMs - new Date(t.updated_at).getTime()) < recentThreshold;
+
+  const recentlyExecuted = all.filter(isRecentlyExecuted);
   const completed = all.filter((t: any) => t.status === "DONE");
   const inProgress = all.filter((t: any) => t.status === "IN_PROGRESS" || t.status === "RUNNING");
   const pending = all.filter((t: any) => t.status === "PENDING" || t.status === "NEW");
   const failed = all.filter((t: any) => t.status === "FAILED");
 
+  // Build ordered task list for consistent numbering (same order as report sections)
+  const orderedTaskIds: string[] = [];
+  const taskNumMap = new Map<string, number>();
+  const assignNum = (t: any) => {
+    if (!taskNumMap.has(t.id)) {
+      orderedTaskIds.push(t.id);
+      taskNumMap.set(t.id, orderedTaskIds.length);
+    }
+    return taskNumMap.get(t.id)!;
+  };
+
+  // Pre-assign numbers in display order: Active → Pending → Recently Executed → Done
+  for (const t of inProgress.slice(0, 8)) assignNum(t);
+  for (const t of pending.slice(0, 6)) assignNum(t);
+  for (const t of recentlyExecuted) assignNum(t);
+  for (const t of completed.slice(0, 5)) assignNum(t);
+
+  const doneCount = completed.length + recentlyExecuted.length;
+
   const lines: string[] = [];
   lines.push(`📊 Project Report — ${scopeLabel}`);
   lines.push(`Agent: ${agentName}`);
   lines.push("");
   lines.push("━━━ Summary ━━━");
-  lines.push(`✅ ${completed.length}  |  🔄 ${inProgress.length}  |  ⏳ ${pending.length}  |  ❌ ${failed.length}`);
+  lines.push(`✅ ${doneCount}  |  🔄 ${inProgress.length}  |  ⏳ ${pending.length}  |  ❌ ${failed.length}`);
 
   if (inProgress.length > 0) {
     lines.push("");
     lines.push("━━━ Active ━━━");
     for (const t of inProgress.slice(0, 8)) {
+      const n = taskNumMap.get(t.id)!;
       const pl = t.priority === 3 ? "CRIT" : t.priority === 2 ? "HIGH" : t.priority === 1 ? "MED" : "LOW";
-      lines.push(`🔄 [${pl}] ${t.title}${t.progress ? ` — ${t.progress}%` : ""}`);
+      lines.push(`${n}. 🔄 [${pl}] ${t.title}${t.progress ? ` — ${t.progress}%` : ""}`);
     }
     if (inProgress.length > 8) lines.push(`... +${inProgress.length - 8} more`);
   }
@@ -2393,26 +2556,41 @@ async function generatePMReportDirect(
     lines.push("");
     lines.push("━━━ Pending ━━━");
     for (const t of pending.slice(0, 6)) {
+      const n = taskNumMap.get(t.id)!;
       const pl = t.priority === 3 ? "CRIT" : t.priority === 2 ? "HIGH" : t.priority === 1 ? "MED" : "LOW";
-      lines.push(`⏳ [${pl}] ${t.title}`);
+      const suffix = isRecentlyExecuted(t) ? " (next cycle)" : "";
+      lines.push(`${n}. ⏳ [${pl}] ${t.title}${suffix}`);
     }
     if (pending.length > 6) lines.push(`... +${pending.length - 6} more`);
   }
 
-  if (includeCompleted && completed.length > 0) {
+  // Done section: recently-executed recurring tasks + regular completed
+  const doneItems: { task: any; isRecurring: boolean }[] = [
+    ...recentlyExecuted.map((t: any) => ({ task: t, isRecurring: true })),
+    ...(includeCompleted ? completed.filter((t: any) => t.completed_at)
+      .sort((a: any, b: any) => new Date(b.completed_at).getTime() - new Date(a.completed_at).getTime())
+      .slice(0, 5).map((t: any) => ({ task: t, isRecurring: false })) : []),
+  ];
+
+  if (doneItems.length > 0) {
     lines.push("");
     lines.push("━━━ Done ━━━");
-    const recent = completed.filter((t: any) => t.completed_at)
-      .sort((a: any, b: any) => new Date(b.completed_at).getTime() - new Date(a.completed_at).getTime())
-      .slice(0, 5);
-    for (const t of recent) {
-      const diff = Date.now() - new Date(t.completed_at).getTime();
-      const mins = Math.floor(diff / 60000);
-      const ago = mins < 60 ? `${mins}m ago` : mins < 1440 ? `${Math.floor(mins / 60)}h ago` : `${Math.floor(mins / 1440)}d ago`;
-      lines.push(`✅ ${t.title} (${ago})`);
+    for (const { task: t, isRecurring } of doneItems) {
+      const n = assignNum(t);
+      if (isRecurring) {
+        lines.push(`${n}. ✅ ${t.title} (today's run)`);
+      } else {
+        const diff = Date.now() - new Date(t.completed_at).getTime();
+        const mins = Math.floor(diff / 60000);
+        const ago = mins < 60 ? `${mins}m ago` : mins < 1440 ? `${Math.floor(mins / 60)}h ago` : `${Math.floor(mins / 1440)}d ago`;
+        lines.push(`${n}. ✅ ${t.title} (${ago})`);
+      }
     }
   }
 
+  lines.push("");
+  lines.push(`💡 Reply "details on #N" for full task result`);
+
   const now = new Date();
   lines.push("");
   lines.push(`Generated: ${now.toISOString().replace("T", " ").slice(0, 16)} UTC`);