@stackwright-pro/mcp 0.2.0-alpha.100 → 0.2.0-alpha.101

package/dist/server.mjs

@@ -4121,6 +4121,67 @@ function handleValidateArtifact(input) {
   }
   return result;
 }
+function handleEmitEvent(input) {
+  const emitFn = input._emit ?? emit;
+  try {
+    const caller = input.otter ?? "foreman";
+    if (caller !== "foreman") {
+      return {
+        text: JSON.stringify({
+          emitted: false,
+          reason: `AUTHORIZATION: only the foreman otter may emit lifecycle events. Specialist otters should signal completion via their response text and the artifact write \u2014 not by emitting phase events. Caller was "${caller}". This is informational, not a failure.`
+        }),
+        isError: false
+      };
+    }
+    let emitted = false;
+    if (input.type === "phase_start") {
+      if (!input.phase) {
+        return {
+          text: JSON.stringify({ emitted: false, reason: "phase required for phase_start" }),
+          isError: false
+        };
+      }
+      emitted = emitFn({ type: "phase_start", phase: input.phase, otter: caller });
+    } else if (input.type === "phase_complete") {
+      if (!input.phase) {
+        return {
+          text: JSON.stringify({ emitted: false, reason: "phase required for phase_complete" }),
+          isError: false
+        };
+      }
+      emitted = emitFn({
+        type: "phase_complete",
+        phase: input.phase,
+        otter: caller,
+        ...input.durationSec != null ? { durationSec: input.durationSec } : {}
+      });
+    } else if (input.type === "agent_invoke_start") {
+      emitted = emitFn({
+        type: "agent_invoke_start",
+        targetOtter: input.targetOtter ?? "",
+        phase: input.phase,
+        otter: caller
+      });
+    } else if (input.type === "agent_invoke_complete") {
+      emitted = emitFn({
+        type: "agent_invoke_complete",
+        targetOtter: input.targetOtter ?? "",
+        success: input.success ?? true,
+        phase: input.phase,
+        otter: caller,
+        ...input.durationSec != null ? { durationSec: input.durationSec } : {}
+      });
+    }
+    return { text: JSON.stringify({ emitted }), isError: false };
+  } catch (err) {
+    const reason = err instanceof Error ? err.message : String(err);
+    return {
+      text: JSON.stringify({ emitted: false, reason: `telemetry error (non-fatal): ${reason}` }),
+      isError: false
+    };
+  }
+}
 function registerPipelineTools(server2) {
   const DESC = "Writes state to .stackwright/ \u2014 the filesystem is the state machine.";
   const res = (r) => ({
@@ -4272,60 +4333,18 @@ function registerPipelineTools(server2) {
   );
   server2.tool(
     "stackwright_pro_emit_event",
-    `Emit a telemetry event to .stackwright/pipeline-events.ndjson. Foreman calls this at phase boundaries and agent-invoke boundaries. Best-effort: failures are silent and never block. ${DESC}`,
+    `FOREMAN ONLY \u2014 specialist otters that call this will receive an authorization rejection (no-op, not an error). Emit a telemetry event to .stackwright/pipeline-events.ndjson. Foreman calls this at phase boundaries and agent-invoke boundaries. Best-effort: failures are silent and never block. ${DESC}`,
     {
       type: z13.enum(["phase_start", "phase_complete", "agent_invoke_start", "agent_invoke_complete"]).describe("Event type to emit"),
       phase: z13.string().optional().describe("Current phase name"),
       targetOtter: z13.string().optional().describe("For agent_invoke_* events: which otter is being invoked"),
-      success: z13.boolean().optional().describe("For agent_invoke_complete: did the invocation succeed?"),
+      success: boolCoerce(z13.boolean().optional()).describe(
+        'For agent_invoke_complete: did the invocation succeed? (boolean; string "true"/"false" also accepted for caller tolerance)'
+      ),
       otter: z13.string().optional().describe('Which otter is emitting (defaults to "foreman")'),
       durationSec: z13.number().optional().describe("Duration in seconds (for *_complete events)")
     },
-    async ({ type, phase, targetOtter, success, otter, durationSec }) => {
-      let emitted = false;
-      try {
-        if (type === "phase_start") {
-          if (!phase) {
-            return res({
-              text: JSON.stringify({ emitted: false, reason: "phase required for phase_start" }),
-              isError: false
-            });
-          }
-          emitted = emit({ type: "phase_start", phase, otter: otter ?? "foreman" });
-        } else if (type === "phase_complete") {
-          if (!phase) {
-            return res({
-              text: JSON.stringify({ emitted: false, reason: "phase required for phase_complete" }),
-              isError: false
-            });
-          }
-          emitted = emit({
-            type: "phase_complete",
-            phase,
-            otter: otter ?? "foreman",
-            ...durationSec != null ? { durationSec } : {}
-          });
-        } else if (type === "agent_invoke_start") {
-          emitted = emit({
-            type: "agent_invoke_start",
-            targetOtter: targetOtter ?? "",
-            phase,
-            otter: otter ?? "foreman"
-          });
-        } else if (type === "agent_invoke_complete") {
-          emitted = emit({
-            type: "agent_invoke_complete",
-            targetOtter: targetOtter ?? "",
-            success: success ?? true,
-            phase,
-            otter: otter ?? "foreman",
-            ...durationSec != null ? { durationSec } : {}
-          });
-        }
-      } catch {
-      }
-      return res({ text: JSON.stringify({ emitted }), isError: false });
-    }
+    async ({ type, phase, targetOtter, success, otter, durationSec }) => res(handleEmitEvent({ type, phase, targetOtter, success, otter, durationSec }))
   );
 }
 
@@ -5798,7 +5817,7 @@ var _checksums = /* @__PURE__ */ new Map([
   ],
   [
     "stackwright-services-otter.json",
-    "8997bdd9a6fd3e6e02563578a1649480bc21acf919534ae41f73e70ade7cf500"
+    "2a50ceb3fad166251d0ad8709a34a32118645713e0e34628165d16dced1d5c93"
   ]
 ]);
 Object.freeze(_checksums);
@@ -7552,7 +7571,7 @@ var package_default = {
     "test:coverage": "vitest run --coverage"
   },
   name: "@stackwright-pro/mcp",
-  version: "0.2.0-alpha.100",
+  version: "0.2.0-alpha.101",
   description: "MCP tools for Stackwright Pro - Data Explorer, Security, ISR, and Dashboard generation",
   license: "SEE LICENSE IN LICENSE",
   main: "./dist/server.js",