muse-crew 0.14.6 → 0.14.8

package/workflows/upgrade.js

@@ -141,6 +141,13 @@ async function telemetryFlush() {
 }
 async function telemetryEnd(status) {
   if (!telemetryRunId) return;
+  // Room #26 blocker 37: record-run-end rejects any status outside the
+  // schema CHECK — and this function swallowed that throw as "non-fatal",
+  // leaving the run row 'running' forever. An invalid status is a
+  // programmer error: fail loud. Transport errors stay non-fatal.
+  if (!["completed", "failed", "parked", "timed_out"].includes(status)) {
+    throw new Error("telemetryEnd: invalid status " + JSON.stringify(status) + " — must be completed|failed|parked|timed_out");
+  }
   // Flush any buffered events before recording the end.
   await telemetryFlush();
   try {
@@ -495,7 +502,12 @@ while (i < STEPS.length) {
     );
     if (!claimResult.claimed) {
       log("Standing down — task " + taskId + " was already claimed by another run");
-      await telemetryEnd("duplicate");
+      // Room #26 blocker 37: telemetryEnd only accepts schema-valid
+      // statuses ("duplicate" violated the CHECK and silently left the run
+      // row 'running' forever). The stand-down is a clean exit — the run
+      // completed without doing work; the duplicate verdict itself stays in
+      // the workflow's returned status and this log line.
+      await telemetryEnd("completed");
       return { status: "duplicate", task_id: taskId, reason: "task already claimed by another run" };
     }
     activeSessionId = claimResult.session_id;