ofiere-openclaw-plugin 4.48.0 → 4.49.0

package/package.json

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

@@ -6562,10 +6562,20 @@ function registerBrainExtractionHook(
 ): void {
   try {
     api.on("agent_end", async (event: any, ctx: any) => {
+      // Cycle 7b BUGSHOOT-4 — entry log is load-bearing: 4 prior cycles spent
+      // chasing downstream symptoms because no log existed to prove the hook
+      // even fired. If you're debugging staff-dispatch persistence and DO NOT
+      // see this line in the gateway log, the hook never ran — investigate
+      // hook registration / agent scoping before anything else.
+      const _entrySessionKey = ctx?.sessionKey || event?.sessionKey || event?.context?.sessionKey || "(none)";
+      api.logger.info?.(`[ofiere-brain] agent_end fired (sessionKey=${_entrySessionKey})`);
       try {
         // Extract messages from event — agent_end provides the conversation
         const messages: any[] = event?.messages || event?.context?.messages || [];
-        if (!messages || messages.length < 2) return;
+        if (!messages || messages.length < 2) {
+          api.logger.debug?.(`[ofiere-brain] agent_end early-exit: messages.length=${messages?.length ?? 0} (<2)`);
+          return;
+        }
 
         // Find last user + last assistant message
         let lastUser = "";
@@ -6589,8 +6599,15 @@ function registerBrainExtractionHook(
           if (lastUser && lastAssistant) break;
         }
 
-        // Skip trivial exchanges
-        if (lastUser.length < 20 || lastAssistant.length < 30) return;
+        // Cycle 7b BUGSHOOT-4 — REMOVED early-return on short messages here.
+        // The prior gate (`if lastUser<20 || lastAssistant<30 return`) sat
+        // BEFORE the staff_report / task_dispatch_log / conversation_messages
+        // writes, which silently killed every staff dispatch whose output was
+        // shorter than 30 chars (e.g. smoke tests outputting "SMOKE-OK" = 8).
+        // The trivial-skip is now applied just before the brain L1/L2/L3/L4
+        // extraction below — those are noise filters, not bookkeeping.
+        // Bookkeeping (dispatch_log completion, conversation messages,
+        // staff_report) MUST run regardless of message length.
 
         // ── Agent identity resolution (FIX) ──────────────────────────
         // Priority: ctx.agentId (from PluginHookAgentContext, the CORRECT source)
@@ -6776,6 +6793,16 @@ function registerBrainExtractionHook(
           }
         }
 
+        // Cycle 7b BUGSHOOT-4 — trivial-skip moved here from line ~6593.
+        // Brain L1/L2/L3/L4 extraction skips trivial chit-chat to reduce
+        // memory noise. This MUST sit AFTER staff_report + task_dispatch_log
+        // bookkeeping above, otherwise short-output staff dispatches silently
+        // fail to persist (root cause of cycles 7b BUGSHOOT 1-4).
+        if (lastUser.length < 20 || lastAssistant.length < 30) {
+          api.logger.debug?.(`[ofiere-brain] skip brain extraction (trivial: user=${lastUser.length}c assistant=${lastAssistant.length}c)`);
+          return;
+        }
+
         // ── Fast Stream: Write L1_focus + L2_episode in parallel ──
         const rawContent = `User: ${lastUser}\nAssistant: ${lastAssistant}`;
         const immediateWrites: PromiseLike<any>[] = [];