@yaag/runtime 0.8.2 → 0.8.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaag/runtime",
-  "version": "0.8.2",
+  "version": "0.8.3",
   "publishConfig": {
     "access": "public"
   },

package/src/cassette/run-checkpoint.ts

@@ -30,7 +30,8 @@ export interface RunCheckpointOptions {
  * What one publication attempt did. `published` covers both a written artifact
  * and a Run that keeps none. `displaced` carries the error that must become the
  * Run's outcome. `lost` reports a Checkpoint the Run asked for and did not get,
- * while a primary program error keeps the outcome.
+ * while the Run keeps its own outcome — a completed result, or a primary
+ * program error (ADR-0021 amendment).
  */
 export type RunCheckpointResult =
   | { readonly kind: "published" }
@@ -66,18 +67,21 @@ export async function publishRunCheckpoint(
       writeRecordingDiagnostic(error);
       return { kind: "lost", message: String(error) };
     }
-    if (options.outcome === "stopped") {
-      return {
-        kind: "displaced",
-        error: new Error(`run stopped but is not restorable: ${String(error)}`, { cause: error }),
-      };
+    // A completed Run keeps its outcome and its result: the work is done, and
+    // no restorability promise is broken. The lost Cassette rides
+    // run_end.checkpointLost and the stderr diagnostic instead of displacing
+    // the outcome (ADR-0021 amendment).
+    if (options.outcome === "completed") {
+      const message = `run completed but its Cassette could not be written: ${String(error)}`;
+      writeRecordingDiagnostic(message);
+      return { kind: "lost", message };
     }
-    // A requested checkpoint silently going missing is worse than rejecting an
-    // otherwise successful Run (ADR-0021). The two facts stay readable: the
-    // program completed, and the Cassette did not land.
+    // A pausing or stopped Run promises restorability, and a Checkpoint that
+    // did not land makes that acknowledgement a lie, so the failure takes the
+    // outcome (ADR-0021: a flush failure is a hard failure there).
     return {
       kind: "displaced",
-      error: new Error(`run completed but its Cassette could not be written: ${String(error)}`, {
+      error: new Error(`run ${options.outcome} but is not restorable: ${String(error)}`, {
         cause: error,
       }),
     };