@stackwright-pro/mcp 0.2.0-alpha.111 → 0.2.0-alpha.112

package/dist/server.mjs

@@ -3498,11 +3498,29 @@ var PHASE_TO_OTTER2 = {
   geo: "stackwright-pro-geo-otter",
   qa: "stackwright-pro-qa-otter"
 };
+var PhaseStatusSchema = z14.object({
+  status: z14.enum(["pending", "running", "completed", "skipped", "failed"]).default("pending"),
+  questionsCollected: z14.boolean().default(false),
+  answered: z14.boolean().default(false),
+  executed: z14.boolean().default(false),
+  artifactWritten: z14.boolean().default(false),
+  retryCount: z14.number().default(0),
+  inFlight: z14.boolean().default(false)
+});
+var PipelineStateSchema = z14.object({
+  version: z14.literal("1.0"),
+  currentPhase: z14.string(),
+  status: z14.enum(["setup", "questions", "execution", "done"]),
+  phases: z14.record(z14.string(), PhaseStatusSchema),
+  startedAt: z14.string(),
+  updatedAt: z14.string()
+});
 function isValidPhase2(phase) {
   return PHASE_ORDER.includes(phase);
 }
 function defaultPhaseStatus() {
   return {
+    status: "pending",
     questionsCollected: false,
     answered: false,
     executed: false,
@@ -3533,10 +3551,9 @@ function readState(cwd) {
   const p = statePath(cwd);
   if (!existsSync7(p)) return createDefaultState();
   const raw = JSON.parse(safeReadSync(p));
-  if (typeof raw !== "object" || raw === null || raw.version !== "1.0") {
-    return createDefaultState();
-  }
-  return raw;
+  const result = PipelineStateSchema.safeParse(raw);
+  if (!result.success) return createDefaultState();
+  return result.data;
 }
 function safeWriteSync(filePath, content) {
   if (existsSync7(filePath)) {
@@ -3625,13 +3642,16 @@ function handleSetPipelineState(input) {
       isError: true
     };
   }
-  const VALID_FIELDS = [
-    "questionsCollected",
-    "answered",
-    "executed",
-    "artifactWritten",
-    "inFlight"
-  ];
+  if (input.field === "artifactWritten") {
+    return {
+      text: JSON.stringify({
+        error: true,
+        message: "Cannot set 'artifactWritten' directly. Use stackwright_pro_validate_artifact instead \u2014 it verifies the file exists on disk before flagging."
+      }),
+      isError: true
+    };
+  }
+  const VALID_FIELDS = ["questionsCollected", "answered", "executed", "inFlight"];
   if (input.field && !VALID_FIELDS.includes(input.field)) {
     return {
       text: JSON.stringify({
@@ -3652,6 +3672,15 @@ function handleSetPipelineState(input) {
           isError: true
         };
       }
+      if (update.field === "artifactWritten") {
+        return {
+          text: JSON.stringify({
+            error: true,
+            message: "Cannot set 'artifactWritten' directly. Use stackwright_pro_validate_artifact instead \u2014 it verifies the file exists on disk before flagging."
+          }),
+          isError: true
+        };
+      }
       if (!VALID_FIELDS.includes(update.field)) {
         return {
           text: JSON.stringify({
@@ -3677,6 +3706,9 @@ function handleSetPipelineState(input) {
         if (input.field && input.value !== void 0) {
           phaseState[input.field] = input.value;
         }
+        if (input.phaseStatus !== void 0) {
+          phaseState.status = input.phaseStatus;
+        }
         if (input.incrementRetry) {
           phaseState.retryCount += 1;
         }
@@ -3703,16 +3735,6 @@ function handleSetPipelineState(input) {
         },
         { cwd }
       );
-      if (input.field === "artifactWritten" && input.value === true && input.phase) {
-        emit({ type: "phase_complete", phase: input.phase }, { cwd });
-      }
-      if (input.updates) {
-        for (const update of input.updates) {
-          if (update.field === "artifactWritten" && update.value === true) {
-            emit({ type: "phase_complete", phase: update.phase }, { cwd });
-          }
-        }
-      }
     } catch {
     }
     return { text: JSON.stringify(state), isError: false };
@@ -4791,6 +4813,7 @@ function _validateArtifactInner(input) {
       if (!state.phases[phase]) state.phases[phase] = defaultPhaseStatus();
       const ps = state.phases[phase];
       ps.artifactWritten = true;
+      ps.status = "completed";
     });
     const topKeys = Object.keys(artifact).slice(0, 5).join(", ");
     const result = {
@@ -4818,6 +4841,7 @@ function handleValidateArtifact(input) {
     const parsed = JSON.parse(result.text);
     if (parsed.valid === true && parsed.artifactPath) {
       emit({ type: "file_write", path: parsed.artifactPath, phase }, { cwd });
+      emit({ type: "phase_complete", phase }, { cwd });
     }
     emit(
       {
@@ -4918,12 +4942,19 @@ function registerPipelineTools(server2) {
     `Atomic read\u2192modify\u2192write pipeline state. ${DESC}`,
     {
       phase: z14.string().optional().describe('Phase to update, e.g. "designer"'),
-      field: z14.enum(["questionsCollected", "answered", "executed", "artifactWritten", "inFlight"]).optional().describe("Boolean field to set"),
+      field: z14.enum(["questionsCollected", "answered", "executed", "inFlight"]).optional().describe(
+        "Boolean field to set. NOTE: artifactWritten is intentionally absent \u2014 use stackwright_pro_validate_artifact instead (it verifies the file exists on disk before flagging)."
+      ),
       // inFlight: set true BEFORE specialist invoke (dataflow mode, swp-ioc7); false after.
       value: boolCoerce(z14.boolean().optional()).describe(
         'Value for the field \u2014 must be a JSON boolean (true/false), NOT the string "true"/"false"'
       ),
-      status: z14.enum(["setup", "questions", "execution", "done"]).optional().describe("Top-level status override"),
+      // phaseStatus: per-phase execution enum — DISTINCT from top-level `status` (pipeline flow).
+      // Named `phaseStatus` to avoid ambiguity. Sets PhaseStatus.status on the given phase.
+      phaseStatus: z14.enum(["pending", "running", "completed", "skipped", "failed"]).optional().describe(
+        "Per-phase execution status (pending/running/completed/skipped/failed). Distinct from top-level `status` (pipeline flow). Use phaseStatus: 'skipped' when a specialist otter is unregistered."
+      ),
+      status: z14.enum(["setup", "questions", "execution", "done"]).optional().describe("Top-level pipeline flow status override \u2014 distinct from phaseStatus"),
       incrementRetry: boolCoerce(z14.boolean().optional()).describe(
         "Bump retryCount by 1 \u2014 must be a JSON boolean"
       ),
@@ -4935,8 +4966,8 @@ function registerPipelineTools(server2) {
               "questionsCollected",
               "answered",
               "executed",
-              "artifactWritten",
               "inFlight"
+              // artifactWritten intentionally absent — use validate_artifact (swp-og9c)
             ]),
             value: z14.boolean()
           })
@@ -4948,6 +4979,7 @@ function registerPipelineTools(server2) {
         ...args.phase != null ? { phase: args.phase } : {},
         ...args.field != null ? { field: args.field } : {},
         ...args.value != null ? { value: args.value } : {},
+        ...args.phaseStatus != null ? { phaseStatus: args.phaseStatus } : {},
         ...args.status != null ? { status: args.status } : {},
         ...args.incrementRetry != null ? { incrementRetry: args.incrementRetry } : {},
         ...args.updates != null ? { updates: args.updates } : {}
@@ -6552,7 +6584,7 @@ var _checksums = /* @__PURE__ */ new Map([
   ],
   [
     "stackwright-pro-foreman-otter.json",
-    "88c4101154d44a7c5470ded13562402a39323ee570ed3a90a73b8d85d7fd2817"
+    "65aa3273274b2b1859d432c3d1166c4a73971861036bf18c35acd8a5357fcf26"
   ],
   [
     "stackwright-pro-form-wizard-otter.json",
@@ -8662,7 +8694,7 @@ var package_default = {
     "test:coverage": "vitest run --coverage"
   },
   name: "@stackwright-pro/mcp",
-  version: "0.2.0-alpha.111",
+  version: "0.2.0-alpha.112",
   description: "MCP tools for Stackwright Pro - Data Explorer, Security, ISR, and Dashboard generation",
   license: "SEE LICENSE IN LICENSE",
   main: "./dist/server.js",