@noetaris/harness 0.7.0 → 0.8.0

package/README.md

@@ -255,7 +255,7 @@ throws on an adapter event never fails a step. Every hook is optional.
 | Hook | Fires |
 |---|---|
 | `onRunStart(ctx)` | once when a run starts |
-| `onRunEnd(ctx, { signal, durationMs })` | once when a run settles — completed, stopped (`$stopped`), paused, or rejected by a thrown error (`$error`) |
+| `onRunEnd(ctx, { signal, durationMs, error? })` | once when a run settles — completed, stopped (`$stopped`), paused, or rejected by a thrown error (`$error`). `error` is set when the run ends with `$error` because of an error no `onError` step handled (paused) or a rejection; a route that returns `'$error'` itself gives no `error`, so test for `error`, not the signal |
 | `onStepStart(ctx)` | before each step |
 | `onStepEnd(ctx, { durationMs })` | after a step's `run` succeeds, before its `route` |
 | `onStepError(ctx, { error, durationMs })` | when a step's `run` throws |
@@ -424,6 +424,7 @@ See [LIMITATIONS.md](LIMITATIONS.md) for known limitations.
 - [`@noetaris/harness-openai`](https://github.com/noetaris-lab/harness-openai) — OpenAI adapter
 - [`@noetaris/harness-google`](https://github.com/noetaris-lab/harness-google) — Google Gemini adapter
 - [`@noetaris/harness-otel`](https://github.com/noetaris-lab/harness-otel) — OpenTelemetry observer bridge
+- [`@noetaris/harness-otel-genai`](https://github.com/noetaris-lab/harness-otel-genai) — GenAI semantic-conventions extension for the bridge
 
 ## License
 

package/dist/index.d.ts

@@ -247,12 +247,19 @@ interface Observer {
     onRunStart?: (ctx: RunContext) => void;
     /**
      * Called once when a run settles — completed, stopped, paused, or rejected by a thrown error
-     * (a graph-definition error or a throwing callback). On a rejection `signal` is `'$error'` and
-     * the run rejects right after, with the original error.
+     * (a graph-definition error or a throwing callback).
+     *
+     * When the run ends because of an error, `signal` is `'$error'` and `error` holds it: the error
+     * a step's `run` or `route` threw, or a failed fork's branch error, when no `onError` step
+     * handled it (the run resolves paused); or the original thrown value when the run rejects (it
+     * rejects right after). `error` is absent on every other ending. A route may also return
+     * `'$error'` as an ordinary signal; that ending has no `error` key, so test for `error`, not
+     * for the signal.
      */
     onRunEnd?: (ctx: RunContext, event: {
         signal: string;
         durationMs: number;
+        error?: unknown;
     }) => void;
     /** Called immediately before each step's `run` function is invoked. */
     onStepStart?: (ctx: StepContext) => void;

package/dist/index.js

@@ -725,9 +725,10 @@ async function runLoop(graph, state, ctx, schema, shouldStop, startCursor, callb
   };
   safeInvoke("onRunStart", () => obs?.onRunStart?.(runCtx), onObserverError);
   let runEnded = false;
-  const endRun = (signal) => {
+  const endRun = (signal, failure) => {
     runEnded = true;
-    safeInvoke("onRunEnd", () => obs?.onRunEnd?.(runCtx, { signal, durationMs: Date.now() - runStart }), onObserverError);
+    const event = { signal, durationMs: Date.now() - runStart, ...failure !== void 0 ? { error: failure.error } : {} };
+    safeInvoke("onRunEnd", () => obs?.onRunEnd?.(runCtx, event), onObserverError);
   };
   const callCountRef = { current: 0 };
   const stepCtxRef = { current: null };
@@ -854,7 +855,7 @@ async function runLoop(graph, state, ctx, schema, shouldStop, startCursor, callb
         } catch (e) {
           state.$error = e instanceof Error ? e : new Error(String(e));
           settle({ outcome: "error", error: state.$error, next: { kind: "pause" } });
-          endRun("$error");
+          endRun("$error", { error: state.$error });
           return { state, signal: "$error", cursor, paused: true };
         }
         const routed = runSucceeded || step.run === void 0 ? { outcome: "ok", signal } : { outcome: "error", signal, error: state.$error };
@@ -878,7 +879,7 @@ async function runLoop(graph, state, ctx, schema, shouldStop, startCursor, callb
           cursor = graph.onError;
         } else {
           settle({ outcome: "error", error: state.$error, next: { kind: "pause" } });
-          endRun("$error");
+          endRun("$error", { error: state.$error });
           return { state, signal: "$error", cursor, paused: true };
         }
       } else {
@@ -893,7 +894,7 @@ async function runLoop(graph, state, ctx, schema, shouldStop, startCursor, callb
       }
     }
   } catch (e) {
-    if (!runEnded) endRun("$error");
+    if (!runEnded) endRun("$error", { error: e });
     throw e;
   }
 }
@@ -1036,7 +1037,7 @@ async function handleFork(forkNode, priorBranches, state, ctx, schema, shouldSto
       return { done: false, nextCursor: graph.onError };
     }
     fireSettled(obs, stepCtx, stepStart, { outcome: "error", error: state.$error, next: { kind: "pause" } }, onObserverError);
-    endRun("$error");
+    endRun("$error", { error: state.$error });
     return { done: true, result: { state, signal: "$error", cursor: forkNode.name, paused: true } };
   }
   state.$error = null;