@btravstack/core 0.29.0 → 0.30.0

package/dist/index.cjs

@@ -124,14 +124,18 @@ const noObserver = () => () => {};
 * matters because the settling call sites are error paths, where a `tap` and a
 * `tapFailure` on the same chain, or a retry, is exactly the shape that fires
 * twice and doubles a failure count.
+*
+* A broken observer is dropped independently, on startup and settlement. No
+* recursive reporting is attempted: telemetry must never alter the operation
+* or prevent another observer from settling.
 */
 const observe = (observers, operation) => {
-	const settlers = observers.map((observer) => observer(operation));
+	const settlers = observers.map((observer) => (0, unthrown.fromSafeThrowable)(observer)(operation).recoverDefect(() => (0, unthrown.Ok)(noObserver())).get());
 	let done = false;
 	return (settled) => {
 		if (done) return;
 		done = true;
-		for (const settle of settlers) settle(settled);
+		for (const settle of settlers) (0, unthrown.fromSafeThrowable)(settle)(settled).recoverDefect(() => (0, unthrown.Ok)()).get();
 	};
 };
 /**
@@ -143,14 +147,21 @@ const observe = (observers, operation) => {
 *
 * `settled` replaces either default where a starter has more to say: `ok`
 * receives the value (a cache reports a hit or a miss), `failure` the
-* `FailureView` (a store reports a missing object as an ordinary answer).
+* `FailureView` (a store reports a missing object as an ordinary answer). A
+* throwing shaping hook is dropped and the ordinary settlement is used.
 */
 const observed = (observers, operation, call, settled = {}) => {
 	const settle = observe(observers, operation);
-	return call().tap((value) => settle(settled.ok?.(value) ?? { outcome: "ok" })).tapFailure((failure) => settle(settled.failure?.(failure) ?? {
-		outcome: "error",
-		cause: failure.tag === "Err" ? failure.error : failure.cause
-	}));
+	return (0, unthrown.OkAsync)().flatMap(call).tap((value) => {
+		const rendered = (0, unthrown.fromSafeThrowable)(() => settled.ok?.(value))().recoverDefect(() => (0, unthrown.Ok)(void 0)).get();
+		settle(rendered ?? { outcome: "ok" });
+	}).tapFailure((failure) => {
+		const rendered = (0, unthrown.fromSafeThrowable)(() => settled.failure?.(failure))().recoverDefect(() => (0, unthrown.Ok)(void 0)).get();
+		settle(rendered ?? {
+			outcome: "error",
+			cause: failure.tag === "Err" ? failure.error : failure.cause
+		});
+	});
 };
 //#endregion
 //#region src/clock.ts

package/dist/index.d.cts

@@ -309,6 +309,10 @@ export declare const noObserver: () => Settle;
  * matters because the settling call sites are error paths, where a `tap` and a
  * `tapFailure` on the same chain, or a retry, is exactly the shape that fires
  * twice and doubles a failure count.
+ *
+ * A broken observer is dropped independently, on startup and settlement. No
+ * recursive reporting is attempted: telemetry must never alter the operation
+ * or prevent another observer from settling.
  */
 export declare const observe: (observers: readonly ((operation: Operation) => Settle)[], operation: Operation) => Settle;
 /**
@@ -320,7 +324,8 @@ export declare const observe: (observers: readonly ((operation: Operation) => Se
  *
  * `settled` replaces either default where a starter has more to say: `ok`
  * receives the value (a cache reports a hit or a miss), `failure` the
- * `FailureView` (a store reports a missing object as an ordinary answer).
+ * `FailureView` (a store reports a missing object as an ordinary answer). A
+ * throwing shaping hook is dropped and the ordinary settlement is used.
  */
 export declare const observed: <T, E>(observers: readonly ((operation: Operation) => Settle)[], operation: Operation, call: () => AsyncResult<T, E>, settled?: {
   readonly ok?: (value: T) => Settled;

package/dist/index.d.mts

@@ -309,6 +309,10 @@ export declare const noObserver: () => Settle;
  * matters because the settling call sites are error paths, where a `tap` and a
  * `tapFailure` on the same chain, or a retry, is exactly the shape that fires
  * twice and doubles a failure count.
+ *
+ * A broken observer is dropped independently, on startup and settlement. No
+ * recursive reporting is attempted: telemetry must never alter the operation
+ * or prevent another observer from settling.
  */
 export declare const observe: (observers: readonly ((operation: Operation) => Settle)[], operation: Operation) => Settle;
 /**
@@ -320,7 +324,8 @@ export declare const observe: (observers: readonly ((operation: Operation) => Se
  *
  * `settled` replaces either default where a starter has more to say: `ok`
  * receives the value (a cache reports a hit or a miss), `failure` the
- * `FailureView` (a store reports a missing object as an ordinary answer).
+ * `FailureView` (a store reports a missing object as an ordinary answer). A
+ * throwing shaping hook is dropped and the ordinary settlement is used.
  */
 export declare const observed: <T, E>(observers: readonly ((operation: Operation) => Settle)[], operation: Operation, call: () => AsyncResult<T, E>, settled?: {
   readonly ok?: (value: T) => Settled;

package/dist/index.mjs

@@ -1,5 +1,5 @@
 import { Module, Port, Provider } from "@btravstack/di";
-import { Err, Ok, OkAsync, P, TaggedError, allAsync, fromSafePromise } from "unthrown";
+import { Err, Ok, OkAsync, P, TaggedError, allAsync, fromSafePromise, fromSafeThrowable } from "unthrown";
 import { setTimeout as setTimeout$1 } from "node:timers/promises";
 import { Config, ConfigInvalid, Env } from "@btravstack/config";
 import { createServer } from "node:http";
@@ -123,14 +123,18 @@ const noObserver = () => () => {};
 * matters because the settling call sites are error paths, where a `tap` and a
 * `tapFailure` on the same chain, or a retry, is exactly the shape that fires
 * twice and doubles a failure count.
+*
+* A broken observer is dropped independently, on startup and settlement. No
+* recursive reporting is attempted: telemetry must never alter the operation
+* or prevent another observer from settling.
 */
 const observe = (observers, operation) => {
-	const settlers = observers.map((observer) => observer(operation));
+	const settlers = observers.map((observer) => fromSafeThrowable(observer)(operation).recoverDefect(() => Ok(noObserver())).get());
 	let done = false;
 	return (settled) => {
 		if (done) return;
 		done = true;
-		for (const settle of settlers) settle(settled);
+		for (const settle of settlers) fromSafeThrowable(settle)(settled).recoverDefect(() => Ok()).get();
 	};
 };
 /**
@@ -142,14 +146,21 @@ const observe = (observers, operation) => {
 *
 * `settled` replaces either default where a starter has more to say: `ok`
 * receives the value (a cache reports a hit or a miss), `failure` the
-* `FailureView` (a store reports a missing object as an ordinary answer).
+* `FailureView` (a store reports a missing object as an ordinary answer). A
+* throwing shaping hook is dropped and the ordinary settlement is used.
 */
 const observed = (observers, operation, call, settled = {}) => {
 	const settle = observe(observers, operation);
-	return call().tap((value) => settle(settled.ok?.(value) ?? { outcome: "ok" })).tapFailure((failure) => settle(settled.failure?.(failure) ?? {
-		outcome: "error",
-		cause: failure.tag === "Err" ? failure.error : failure.cause
-	}));
+	return OkAsync().flatMap(call).tap((value) => {
+		const rendered = fromSafeThrowable(() => settled.ok?.(value))().recoverDefect(() => Ok(void 0)).get();
+		settle(rendered ?? { outcome: "ok" });
+	}).tapFailure((failure) => {
+		const rendered = fromSafeThrowable(() => settled.failure?.(failure))().recoverDefect(() => Ok(void 0)).get();
+		settle(rendered ?? {
+			outcome: "error",
+			cause: failure.tag === "Err" ? failure.error : failure.cause
+		});
+	});
 };
 //#endregion
 //#region src/clock.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@btravstack/core",
-  "version": "0.29.0",
+  "version": "0.30.0",
   "description": "The application kernel: boot a di module into a running process, and stop it without losing work",
   "keywords": [
     "application",
@@ -45,8 +45,8 @@
     "./package.json": "./package.json"
   },
   "devDependencies": {
-    "@btravstack/config": "0.29.0",
-    "@btravstack/di": "0.29.0",
+    "@btravstack/config": "0.30.0",
+    "@btravstack/di": "0.30.0",
     "@btravstack/tsconfig": "0.4.0",
     "@stryker-mutator/core": "10.0.0",
     "@stryker-mutator/vitest-runner": "10.0.0",
@@ -60,8 +60,8 @@
     "vitest": "5.0.3"
   },
   "peerDependencies": {
-    "@btravstack/config": "^0.29.0",
-    "@btravstack/di": "^0.29.0",
+    "@btravstack/config": "^0.30.0",
+    "@btravstack/di": "^0.30.0",
     "unthrown": "^5.0.0"
   },
   "engines": {