npm - @ryanfw/prompt-orchestration-pipeline - Versions diffs - 1.3.2 → 1.3.3 - Mend

@ryanfw/prompt-orchestration-pipeline 1.3.2 → 1.3.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (46) hide show

package/docs/http-api.md +66 -0
package/package.json +1 -1
package/src/api/__tests__/index.test.ts +311 -3
package/src/api/index.ts +94 -45
package/src/config/__tests__/statuses.test.ts +41 -0
package/src/config/statuses.ts +20 -1
package/src/core/__tests__/config.test.ts +31 -1
package/src/core/__tests__/job-concurrency.test.ts +60 -0
package/src/core/__tests__/job-view.test.ts +859 -0
package/src/core/__tests__/orchestrator.test.ts +242 -0
package/src/core/__tests__/runner-liveness.test.ts +865 -0
package/src/core/__tests__/single-derivation.test.ts +159 -0
package/src/core/config.ts +19 -0
package/src/core/job-concurrency.ts +6 -1
package/src/core/job-view.ts +329 -0
package/src/core/orchestrator.ts +78 -6
package/src/core/runner-liveness.ts +276 -0
package/src/core/status-writer.ts +24 -5
package/src/ui/client/__tests__/job-adapter.test.ts +78 -0
package/src/ui/client/adapters/job-adapter.ts +10 -0
package/src/ui/client/types.ts +2 -0
package/src/ui/components/JobTable.tsx +29 -12
package/src/ui/components/__tests__/JobTable.test.tsx +54 -1
package/src/ui/components/types.ts +2 -0
package/src/ui/dist/assets/{index--RH3sAt3.js → index-L6cvsCAx.js} +25 -12
package/src/ui/dist/assets/{index--RH3sAt3.js.map → index-L6cvsCAx.js.map} +1 -1
package/src/ui/dist/index.html +1 -1
package/src/ui/embedded-assets.js +6 -6
package/src/ui/server/__tests__/config-bridge.test.ts +9 -1
package/src/ui/server/__tests__/job-control-endpoints.test.ts +57 -1
package/src/ui/server/__tests__/job-endpoints.test.ts +115 -1
package/src/ui/server/__tests__/sse-enhancer.test.ts +31 -0
package/src/ui/server/config-bridge.ts +3 -4
package/src/ui/server/endpoints/__tests__/meta-endpoint.test.ts +1 -0
package/src/ui/server/endpoints/gate-endpoints.ts +2 -6
package/src/ui/server/endpoints/job-control-endpoints.ts +7 -68
package/src/ui/server/endpoints/job-endpoints.ts +6 -0
package/src/ui/server/endpoints/meta-endpoint.ts +1 -1
package/src/ui/state/__tests__/snapshot.test.ts +51 -0
package/src/ui/state/__tests__/types.test.ts +98 -5
package/src/ui/state/snapshot.ts +1 -1
package/src/ui/state/transformers/__tests__/list-transformer.test.ts +43 -2
package/src/ui/state/transformers/__tests__/status-transformer.test.ts +208 -22
package/src/ui/state/transformers/list-transformer.ts +7 -3
package/src/ui/state/transformers/status-transformer.ts +36 -170
package/src/ui/state/types.ts +9 -47

package/src/ui/dist/index.html CHANGED Viewed

@@ -11,7 +11,7 @@
     />
     <title>Prompt Pipeline Dashboard</title>
     <link rel="icon" type="image/svg+xml" href="/favicon.svg" />
-    <script type="module" crossorigin src="/assets/index--RH3sAt3.js"></script>
+    <script type="module" crossorigin src="/assets/index-L6cvsCAx.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/style-CSSKuMOe.css">
   </head>
   <body>

package/src/ui/embedded-assets.js CHANGED Viewed

@@ -1,12 +1,12 @@
 // Auto-generated by scripts/generate-embedded-assets.js — do not edit
 import _asset0 from "./dist/index.html" with { type: "file" };
-import _asset1 from "./dist/assets/style-CSSKuMOe.css" with { type: "file" };
-import _asset2 from "./dist/assets/index--RH3sAt3.js" with { type: "file" };
-import _asset3 from "./dist/assets/index--RH3sAt3.js.map" with { type: "file" };
+import _asset1 from "./dist/assets/index-L6cvsCAx.js.map" with { type: "file" };
+import _asset2 from "./dist/assets/style-CSSKuMOe.css" with { type: "file" };
+import _asset3 from "./dist/assets/index-L6cvsCAx.js" with { type: "file" };
 export const embeddedAssets = {
   "/index.html": { path: _asset0, mime: "text/html" },
-  "/assets/style-CSSKuMOe.css": { path: _asset1, mime: "text/css" },
-  "/assets/index--RH3sAt3.js": { path: _asset2, mime: "application/javascript" },
-  "/assets/index--RH3sAt3.js.map": { path: _asset3, mime: "application/json" }
+  "/assets/index-L6cvsCAx.js.map": { path: _asset1, mime: "application/json" },
+  "/assets/style-CSSKuMOe.css": { path: _asset2, mime: "text/css" },
+  "/assets/index-L6cvsCAx.js": { path: _asset3, mime: "application/javascript" }
 };

package/src/ui/server/__tests__/config-bridge.test.ts CHANGED Viewed

@@ -3,6 +3,7 @@ import { describe, expect, it } from "vitest";
 import {
   Constants,
   createErrorResponse,
+  determineJobStatus,
   getStatusPriority,
   validateJobId,
   validateTaskState,
@@ -25,7 +26,7 @@ describe("config-bridge", () => {
   it("returns status priorities", () => {
     expect(getStatusPriority("running")).toBe(4);
-    expect(getStatusPriority("error")).toBe(3);
+    expect(getStatusPriority("failed")).toBe(3);
     expect(getStatusPriority("pending")).toBe(2);
     expect(getStatusPriority("complete")).toBe(1);
     expect(getStatusPriority("unknown")).toBe(0);
@@ -38,4 +39,11 @@ describe("config-bridge", () => {
       message: "missing",
     });
   });
+  it("determines the canonical job status from task states", () => {
+    expect(determineJobStatus({ a: { state: "failed" } })).toBe("failed");
+    expect(determineJobStatus({ a: { state: "done" }, b: { state: "done" } })).toBe("complete");
+    expect(determineJobStatus({ a: { state: "running" } })).toBe("running");
+    expect(determineJobStatus({ a: { state: "pending" } })).toBe("pending");
+  });
 });

package/src/ui/server/__tests__/job-control-endpoints.test.ts CHANGED Viewed

@@ -7,11 +7,11 @@ import { afterEach, describe, expect, it, vi } from "vitest";
 import { initPATHS, resetPATHS } from "../config-bridge-node";
 import {
-  classifyStaleRunningStatus,
   handleJobRestart,
   handleJobStop,
   handleTaskStart,
 } from "../endpoints/job-control-endpoints";
+import { classifyStaleRunningStatus } from "../../../core/runner-liveness";
 import { resetConfig } from "../../../core/config";
 import {
   getConcurrencyRuntimePaths,
@@ -122,6 +122,7 @@ afterEach(async () => {
   vi.restoreAllMocks();
   resetPATHS();
   delete process.env["PO_MAX_RUNNING_JOBS"];
+  delete process.env["PO_STALE_RUNNING_GRACE_MS"];
   resetConfig();
   for (const proc of childProcs.splice(0)) {
     try { proc.kill(); } catch {}
@@ -566,6 +567,33 @@ describe("handleJobRestart", () => {
     expect(body["code"]).toBe("job_running");
   });
+  it("uses configured staleRunningGraceMs when deciding whether restart can recover a running task", async () => {
+    const root = await makeTempRoot();
+    initPATHS(root);
+    process.env["PO_STALE_RUNNING_GRACE_MS"] = "60000";
+    resetConfig();
+    vi.spyOn(Date, "now").mockReturnValue(Date.parse("2026-04-01T10:00:45.000Z"));
+    await setupJob(root, "restart-configured-grace", {
+      id: "restart-configured-grace",
+      state: "running",
+      current: "research",
+      currentStage: "prompt",
+      lastUpdated: "2026-04-01T10:00:00.000Z",
+      tasks: {
+        research: { state: "running", currentStage: "prompt" },
+      },
+      files: { artifacts: [], logs: [], tmp: [] },
+    }, 999999);
+    const req = new Request("http://localhost/api/jobs/restart-configured-grace/restart", { method: "POST" });
+    const res = await handleJobRestart(req, "restart-configured-grace", root);
+    const body = await res.json() as Record<string, unknown>;
+    expect(res.status).toBe(409);
+    expect(body["code"]).toBe("job_running");
+  });
   it("proceeds when task-derived running status is stale and the runner is gone", async () => {
     const root = await makeTempRoot();
     initPATHS(root);
@@ -939,6 +967,34 @@ describe("handleTaskStart", () => {
     expect(body["code"]).toBe("job_running");
   });
+  it("uses configured staleRunningGraceMs when deciding whether task start can recover a running task", async () => {
+    const root = await makeTempRoot();
+    initPATHS(root);
+    process.env["PO_STALE_RUNNING_GRACE_MS"] = "60000";
+    resetConfig();
+    vi.spyOn(Date, "now").mockReturnValue(Date.parse("2026-04-01T10:00:45.000Z"));
+    await setupJob(root, "task-start-configured-grace", {
+      id: "task-start-configured-grace",
+      state: "running",
+      current: "research",
+      currentStage: "prompt",
+      lastUpdated: "2026-04-01T10:00:00.000Z",
+      tasks: {
+        research: { state: "running", currentStage: "prompt" },
+        analysis: { state: "pending", currentStage: null },
+      },
+      files: { artifacts: [], logs: [], tmp: [] },
+    }, 999999);
+    const req = new Request("http://localhost/api/jobs/task-start-configured-grace/tasks/analysis/start", { method: "POST" });
+    const res = await handleTaskStart(req, "task-start-configured-grace", "analysis", root);
+    const body = await res.json() as Record<string, unknown>;
+    expect(res.status).toBe(409);
+    expect(body["code"]).toBe("job_running");
+  });
   it("returns 202 when stale-running status is the only blocker for an eligible pending task", async () => {
     const root = await makeTempRoot();
     initPATHS(root);

package/src/ui/server/__tests__/job-endpoints.test.ts CHANGED Viewed

@@ -5,7 +5,7 @@ import path from "node:path";
 import { afterEach, beforeEach, describe, expect, it } from "vitest";
 import { resetConfig } from "../../../core/config";
-import { handleJobDetail } from "../endpoints/job-endpoints";
+import { handleJobDetail, handleJobList } from "../endpoints/job-endpoints";
 import { initPATHS, resetPATHS } from "../config-bridge-node";
 const tempRoots: string[] = [];
@@ -122,3 +122,117 @@ describe("job endpoints pipeline config loading", () => {
     expect(body.data.pipelineConfig?.["tasks"]).toEqual([{ name: "shared-task" }]);
   });
 });
+describe("handleJobDetail corrupt status", () => {
+  it("returns 409 STATUS_CORRUPT with the corrupt tasks-status.json path", async () => {
+    const root = await makeTempRoot();
+    process.env["PO_ROOT"] = root;
+    resetConfig();
+    initPATHS(root);
+    await writePipelineRegistry(root);
+    const jobDir = await writeJob(root, "current", "job-corrupt");
+    await writeFile(path.join(jobDir, "tasks-status.json"), "{ not valid json");
+    const res = await handleJobDetail("job-corrupt");
+    const body = (await res.json()) as { code: string; path?: string };
+    expect(res.status).toBe(409);
+    expect(body.code).toBe("STATUS_CORRUPT");
+    expect(body.path).toBe(path.join(jobDir, "tasks-status.json"));
+  });
+});
+describe("canonical status in HTTP responses", () => {
+  it("handleJobDetail returns canonical status:'failed' for a failed-state fixture", async () => {
+    const root = await makeTempRoot();
+    process.env["PO_ROOT"] = root;
+    resetConfig();
+    initPATHS(root);
+    await writePipelineRegistry(root);
+    const jobDir = await writeJob(root, "current", "job-failed");
+    await writeFile(
+      path.join(jobDir, "tasks-status.json"),
+      JSON.stringify({
+        id: "job-failed",
+        name: "job-failed",
+        pipeline: "demo",
+        createdAt: "2026-06-12T12:00:00.000Z",
+        state: "failed",
+        tasks: { a: { state: "failed" } },
+      }),
+    );
+    const res = await handleJobDetail("job-failed");
+    const body = (await res.json()) as { data: { status: string } };
+    expect(res.status).toBe(200);
+    expect(body.data.status).toBe("failed");
+  });
+  it("handleJobDetail returns canonical status:'complete' for a complete-state fixture", async () => {
+    const root = await makeTempRoot();
+    process.env["PO_ROOT"] = root;
+    resetConfig();
+    initPATHS(root);
+    await writePipelineRegistry(root);
+    const jobDir = await writeJob(root, "current", "job-complete");
+    await writeFile(
+      path.join(jobDir, "tasks-status.json"),
+      JSON.stringify({
+        id: "job-complete",
+        name: "job-complete",
+        pipeline: "demo",
+        createdAt: "2026-06-12T12:00:00.000Z",
+        state: "done",
+        tasks: { a: { state: "done" } },
+      }),
+    );
+    const res = await handleJobDetail("job-complete");
+    const body = (await res.json()) as { data: { status: string } };
+    expect(res.status).toBe(200);
+    expect(body.data.status).toBe("complete");
+  });
+  it("handleJobList returns canonical status for every job in the listing", async () => {
+    const root = await makeTempRoot();
+    process.env["PO_ROOT"] = root;
+    resetConfig();
+    initPATHS(root);
+    await writePipelineRegistry(root);
+    const failedDir = await writeJob(root, "current", "job-list-failed");
+    await writeFile(
+      path.join(failedDir, "tasks-status.json"),
+      JSON.stringify({
+        id: "job-list-failed",
+        name: "job-list-failed",
+        pipeline: "demo",
+        createdAt: "2026-06-12T12:00:00.000Z",
+        state: "failed",
+        tasks: { a: { state: "failed" } },
+      }),
+    );
+    const completeDir = await writeJob(root, "complete", "job-list-complete");
+    await writeFile(
+      path.join(completeDir, "tasks-status.json"),
+      JSON.stringify({
+        id: "job-list-complete",
+        name: "job-list-complete",
+        pipeline: "demo",
+        createdAt: "2026-06-12T12:00:00.000Z",
+        state: "done",
+        tasks: { a: { state: "done" } },
+      }),
+    );
+    const res = await handleJobList();
+    const body = (await res.json()) as { data: Array<{ jobId: string; status: string }> };
+    expect(res.status).toBe(200);
+    expect(body.data).toHaveLength(2);
+    const byId = new Map(body.data.map((job) => [job.jobId, job.status]));
+    expect(byId.get("job-list-failed")).toBe("failed");
+    expect(byId.get("job-list-complete")).toBe("complete");
+  });
+});

package/src/ui/server/__tests__/sse-enhancer.test.ts CHANGED Viewed

@@ -53,4 +53,35 @@ describe("sse-enhancer", () => {
     enhancer.cleanup();
     expect(enhancer.getPendingCount()).toBe(0);
   });
+  it("broadcasts canonical failed status when the persisted state is 'failed'", async () => {
+    const readJobFn = vi.fn(async () => ({
+      ok: true as const,
+      jobId: "j1",
+      location: "current",
+      path: "/tmp/j1/tasks-status.json",
+      data: {
+        title: "Job 1",
+        state: "failed",
+        createdAt: "2024-01-01T00:00:00.000Z",
+        updatedAt: "2024-01-01T00:00:00.000Z",
+        tasks: { a: { state: "failed", files: {} } },
+      },
+    }));
+    const broadcast = vi.fn();
+    const enhancer = createSSEEnhancer({
+      readJobFn,
+      sseRegistry: { addClient() {}, removeClient() {}, broadcast, getClientCount() { return 0; }, closeAll() {} },
+      debounceMs: 100,
+    });
+    enhancer.handleJobChange({ jobId: "j1" });
+    vi.advanceTimersByTime(100);
+    await Promise.resolve();
+    expect(broadcast).toHaveBeenCalledTimes(1);
+    const [, payload] = broadcast.mock.calls[0] as [string, { status: string }];
+    expect(broadcast.mock.calls[0]?.[0]).toBe("job:created");
+    expect(payload.status).toBe("failed");
+  });
 });

package/src/ui/server/config-bridge.ts CHANGED Viewed

@@ -16,7 +16,7 @@ export const Constants = {
   JOB_ID_REGEX: /^[A-Za-z0-9-_]+$/,
   TASK_STATES: Object.freeze(Object.values(TaskState)),
   JOB_LOCATIONS: Object.freeze(Object.values(JobLocation)),
-  STATUS_ORDER: Object.freeze(["running", "error", "pending", "complete"]),
+  STATUS_ORDER: Object.freeze(["running", "failed", "pending", "complete"]),
   FILE_LIMITS: Object.freeze({ MAX_FILE_SIZE: 5 * 1024 * 1024 }),
   RETRY_CONFIG: Object.freeze({ MAX_ATTEMPTS: 3, DELAY_MS: 10 }),
   SSE_CONFIG: Object.freeze({ DEBOUNCE_MS: 200 }),
@@ -32,7 +32,7 @@ export const Constants = {
 const STATUS_PRIORITY = new Map<string, number>([
   ["running", 4],
-  ["error", 3],
+  ["failed", 3],
   ["pending", 2],
   ["complete", 1],
 ]);
@@ -53,8 +53,7 @@ export function determineJobStatus(tasks: Record<string, { state: string }>): st
   const normalizedTasks = Object.values(tasks).map((task) => ({
     state: task.state,
   }));
-  const status = deriveJobStatusFromTasks(normalizedTasks);
-  return status === "failed" ? "error" : status;
+  return deriveJobStatusFromTasks(normalizedTasks);
 }
 export function createErrorResponse(

package/src/ui/server/endpoints/__tests__/meta-endpoint.test.ts CHANGED Viewed

@@ -11,6 +11,7 @@ describe("handleMeta", () => {
     const body = await res.json() as { ok: boolean; data: Record<string, unknown> };
     expect(body.ok).toBe(true);
     expect(body.data["protocolVersion"]).toBe(PROTOCOL_VERSION);
+    expect(body.data["protocolVersion"]).toBe(2);
   });
   it("serves the exact package.json version (drift guard)", async () => {

package/src/ui/server/endpoints/gate-endpoints.ts CHANGED Viewed

@@ -9,12 +9,8 @@ import { releaseJobSlot } from "../../../core/job-concurrency";
 import { appendRunEvent } from "../../../core/run-events";
 import { readJobStatus, writeJobStatus, StatusCorruptError } from "../../../core/status-writer";
 import { assertStatusParseable, statusCorruptResponse } from "../utils/status-corruption";
-import {
-  acquireConcurrencySlot,
-  isProcessAlive,
-  readRunnerPid,
-  spawnWithSlot,
-} from "./job-control-endpoints";
+import { acquireConcurrencySlot, spawnWithSlot } from "./job-control-endpoints";
+import { isProcessAlive, readRunnerPid } from "../../../core/runner-liveness";
 type GateAction = "approve" | "reject";

package/src/ui/server/endpoints/job-control-endpoints.ts CHANGED Viewed

@@ -5,7 +5,6 @@ import { createErrorResponse } from "../config-bridge";
 import { Constants } from "../config-bridge-node";
 import { sendJson } from "../utils/http-utils";
 import { getJobDirectoryPath, getPipelineDataDir } from "../../../config/paths";
-import { deriveJobStatusFromTasks } from "../../../config/statuses";
 import { getOrchestratorConfig, getPipelineConfig } from "../../../core/config";
 import {
   releaseJobSlot,
@@ -13,6 +12,11 @@ import {
   updateJobSlotPid,
   type JobSlotLease,
 } from "../../../core/job-concurrency";
+import {
+  classifyStaleRunningStatus,
+  isProcessAlive,
+  readRunnerPid,
+} from "../../../core/runner-liveness";
 import {
   readJobStatus,
   resetJobFromTask,
@@ -29,19 +33,6 @@ import {
 import { readFileWithRetry } from "../file-reader";
 const RUNNER_PATH = path.resolve(import.meta.dir, "../../../core/pipeline-runner.ts");
-const STALE_RUNNING_GRACE_MS = 30_000;
-interface StaleRunningInput {
-  status: StatusSnapshot;
-  pid: number | null;
-  pidAlive: boolean;
-  nowMs: number;
-  staleAfterMs: number;
-}
-type StaleRunningDecision =
-  | { stale: false; reason: "not_running" | "live_pid" | "fresh_status" | "missing_last_updated" }
-  | { stale: true; runningTaskId: string | null; ageMs: number };
 const restartingJobs = new Set<string>();
 const stoppingJobs = new Set<string>();
@@ -174,25 +165,6 @@ export async function spawnWithSlot(
   }
 }
-export async function readRunnerPid(jobDir: string): Promise<number | null> {
-  try {
-    const content = await Bun.file(path.join(jobDir, "runner.pid")).text();
-    const pid = parseInt(content.trim(), 10);
-    return Number.isNaN(pid) ? null : pid;
-  } catch {
-    return null;
-  }
-}
-export function isProcessAlive(pid: number): boolean {
-  try {
-    process.kill(pid, 0);
-    return true;
-  } catch {
-    return false;
-  }
-}
 const KILL_GRACE_MS = 1500;
 const KILL_POLL_MS = 100;
@@ -208,39 +180,6 @@ function isDependencySatisfied(state: unknown): boolean {
   return state === "done";
 }
-function getTaskDerivedStatus(snapshot: StatusSnapshot): string {
-  return deriveJobStatusFromTasks(
-    Object.values(snapshot.tasks).map((task) => ({ state: task.state })),
-  );
-}
-function findRunningTaskId(snapshot: StatusSnapshot): string | null {
-  if (snapshot.current && snapshot.tasks[snapshot.current]?.state === "running") {
-    return snapshot.current;
-  }
-  return Object.keys(snapshot.tasks).find((taskId) => snapshot.tasks[taskId]?.state === "running") ?? null;
-}
-export function classifyStaleRunningStatus(input: StaleRunningInput): StaleRunningDecision {
-  if (getTaskDerivedStatus(input.status) !== "running") {
-    return { stale: false, reason: "not_running" };
-  }
-  if (input.pid !== null && input.pidAlive) {
-    return { stale: false, reason: "live_pid" };
-  }
-  const updatedMs = Date.parse(input.status.lastUpdated);
-  if (!Number.isFinite(updatedMs)) {
-    // No live PID and no parseable timestamp: the runner crashed before writing a
-    // valid lastUpdated. There is no evidence of liveness, so treat as stale.
-    return { stale: true, runningTaskId: findRunningTaskId(input.status), ageMs: 0 };
-  }
-  const ageMs = input.nowMs - updatedMs;
-  if (ageMs < input.staleAfterMs) {
-    return { stale: false, reason: "fresh_status" };
-  }
-  return { stale: true, runningTaskId: findRunningTaskId(input.status), ageMs };
-}
 function isTaskLikelyInProgress(task: Record<string, unknown>): boolean {
   const state = task["state"];
   const nonTerminal = isNonTerminalTaskState(state);
@@ -552,7 +491,7 @@ export async function handleJobRestart(
         pid,
         pidAlive,
         nowMs: Date.now(),
-        staleAfterMs: STALE_RUNNING_GRACE_MS,
+        staleAfterMs: getOrchestratorConfig().staleRunningGraceMs,
       });
       // Note: "live_pid" can never be returned here because pidAlive is guaranteed false
       // by the early-return guard above; only "not_running", "fresh_status", or stale=true
@@ -764,7 +703,7 @@ export async function handleTaskStart(
       pid,
       pidAlive,
       nowMs: Date.now(),
-      staleAfterMs: STALE_RUNNING_GRACE_MS,
+      staleAfterMs: getOrchestratorConfig().staleRunningGraceMs,
     });
     if (!staleDecision.stale && staleDecision.reason !== "not_running") {
       return sendJson(409, createErrorResponse("job_running", "Job is currently running (task-level running)"));

package/src/ui/server/endpoints/job-endpoints.ts CHANGED Viewed

@@ -60,6 +60,12 @@ export async function handleJobDetail(jobId: string): Promise<Response> {
   const result = await readJob(jobId);
   if (!result.ok) {
+    if (result.code === Constants.ERROR_CODES.INVALID_JSON) {
+      return sendJson(
+        409,
+        createErrorResponse(Constants.ERROR_CODES.STATUS_CORRUPT, result.message, result.path),
+      );
+    }
     const status = result.code === Constants.ERROR_CODES.JOB_NOT_FOUND ? 404 : 400;
     return sendJson(status, result);
   }

package/src/ui/server/endpoints/meta-endpoint.ts CHANGED Viewed

@@ -1,7 +1,7 @@
 import { sendJson } from "../utils/http-utils";
 import pkg from "../../../../package.json";
-export const PROTOCOL_VERSION = 1;
+export const PROTOCOL_VERSION = 2;
 export function handleMeta(): Response {
   return sendJson(200, {

package/src/ui/state/__tests__/snapshot.test.ts CHANGED Viewed

@@ -1,6 +1,7 @@
 import { describe, expect, it, vi } from "vitest";
 import { buildSnapshotFromFilesystem, composeStateSnapshot } from "../snapshot";
+import type { CanonicalJob, JobReadResult } from "../types";
 describe("snapshot", () => {
   it("composes default snapshots and extracts ids from variant fields", () => {
@@ -81,4 +82,54 @@ describe("snapshot", () => {
       }),
     ).rejects.toThrow(/unavailable/);
   });
+  it("sorts snapshot jobs with failed (former error) priority first", async () => {
+    const statusByJobId: Record<string, CanonicalJob["status"]> = {
+      "job-failed": "failed",
+      "job-running": "running",
+      "job-complete": "complete",
+      "job-pending": "pending",
+    };
+    const transformMultipleJobs = vi.fn((results: JobReadResult[]): CanonicalJob[] =>
+      results.map((result) => ({
+        id: result.jobId,
+        jobId: result.jobId,
+        name: result.jobId,
+        title: result.jobId,
+        status: statusByJobId[result.jobId] ?? "pending",
+        progress: 0,
+        createdAt: "2024-01-01T00:00:00.000Z",
+        updatedAt: "2024-01-01T00:00:00.000Z",
+        location: result.location,
+        tasks: {},
+        files: {},
+        costs: {},
+      })),
+    );
+    const snapshot = await buildSnapshotFromFilesystem({
+      listAllJobs: () => ({
+        current: ["job-pending", "job-running", "job-failed", "job-complete"],
+        complete: [],
+      }),
+      readJob: async (jobId, location) => ({ ok: true, jobId, location, data: {} }),
+      transformMultipleJobs,
+      now: () => new Date("2024-03-01T00:00:00.000Z"),
+    });
+    expect(transformMultipleJobs).toHaveBeenCalledTimes(1);
+    expect(snapshot.jobs.map((job) => job.jobId)).toEqual([
+      "job-failed",
+      "job-running",
+      "job-complete",
+      "job-pending",
+    ]);
+    expect(snapshot.jobs.map((job) => job.status)).toEqual([
+      "failed",
+      "running",
+      "complete",
+      "pending",
+    ]);
+    expect(snapshot.jobs[0]?.jobId).toBe("job-failed");
+  });
 });