@voyant-travel/hono 0.118.3 → 0.118.4

package/dist/lib/request-event-bus.d.ts

@@ -10,9 +10,10 @@ import type { EventBus, OutboxEventStore } from "@voyant-travel/core";
  * With a `store` (transactional outbox), emits are also DURABLE: the
  * envelope is persisted before any handler runs, and failed deliveries
  * are retried by the drain. If the durable capture itself fails (DB
- * unreachable), the emit falls back to direct (non-durable) delivery —
- * losing durability for that one event is strictly better than losing
- * the event, and an order of magnitude better than failing the request.
+ * unreachable), the emit falls back to direct (non-durable) delivery.
+ * Once capture has succeeded, the wrapper must not redeliver directly:
+ * the stored row owns completion/retry, and direct fallback would make
+ * subscriber side effects ambiguous.
  *
  * Buses that predate the {@link EmitOptions} parameter simply ignore it
  * and keep awaiting all handlers — a safe degradation.

package/dist/lib/request-event-bus.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"request-event-bus.d.ts","sourceRoot":"","sources":["../../src/lib/request-event-bus.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAe,QAAQ,EAAiB,gBAAgB,EAAE,MAAM,qBAAqB,CAAA;AAEjG;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,qBAAqB,CACnC,GAAG,EAAE,QAAQ,EACb,QAAQ,EAAE,CAAC,CAAC,OAAO,EAAE,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,CAAC,GAAG,SAAS,EAC3D,KAAK,CAAC,EAAE,gBAAgB,GACvB,QAAQ,CA+BV"}
+{"version":3,"file":"request-event-bus.d.ts","sourceRoot":"","sources":["../../src/lib/request-event-bus.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAe,QAAQ,EAAiB,gBAAgB,EAAE,MAAM,qBAAqB,CAAA;AAEjG;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,qBAAqB,CACnC,GAAG,EAAE,QAAQ,EACb,QAAQ,EAAE,CAAC,CAAC,OAAO,EAAE,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,CAAC,GAAG,SAAS,EAC3D,KAAK,CAAC,EAAE,gBAAgB,GACvB,QAAQ,CAiDV"}

package/dist/lib/request-event-bus.js

@@ -9,9 +9,10 @@
  * With a `store` (transactional outbox), emits are also DURABLE: the
  * envelope is persisted before any handler runs, and failed deliveries
  * are retried by the drain. If the durable capture itself fails (DB
- * unreachable), the emit falls back to direct (non-durable) delivery —
- * losing durability for that one event is strictly better than losing
- * the event, and an order of magnitude better than failing the request.
+ * unreachable), the emit falls back to direct (non-durable) delivery.
+ * Once capture has succeeded, the wrapper must not redeliver directly:
+ * the stored row owns completion/retry, and direct fallback would make
+ * subscriber side effects ambiguous.
  *
  * Buses that predate the {@link EmitOptions} parameter simply ignore it
  * and keep awaiting all handlers — a safe degradation.
@@ -26,12 +27,28 @@ export function requestScopedEventBus(bus, schedule, store) {
             if (!store) {
                 return bus.emit(event, data, metadata, { ...base, ...options });
             }
+            let captureFailed = false;
+            const guardedStore = {
+                async insert(envelope) {
+                    try {
+                        return await store.insert(envelope);
+                    }
+                    catch (err) {
+                        captureFailed = true;
+                        throw err;
+                    }
+                },
+                complete: (id) => store.complete(id),
+                fail: (id, error) => store.fail(id, error),
+            };
             try {
-                return await bus.emit(event, data, metadata, { ...base, store, ...options });
+                return await bus.emit(event, data, metadata, { ...base, store: guardedStore, ...options });
             }
             catch (err) {
-                // Only the outbox insert can reject (handler errors are
-                // collected, bookkeeping failures are swallowed inside the bus).
+                if (!captureFailed) {
+                    console.error(`[events] durable delivery failed for "${event}" after outbox capture:`, err);
+                    return;
+                }
                 console.error(`[events] outbox capture failed for "${event}" — falling back to direct delivery:`, err);
                 return bus.emit(event, data, metadata, { ...base, ...options });
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@voyant-travel/hono",
-  "version": "0.118.3",
+  "version": "0.118.4",
   "license": "Apache-2.0",
   "type": "module",
   "exports": {
@@ -135,7 +135,7 @@
     "drizzle-orm": "^0.45.2",
     "hono": "^4.12.25",
     "zod": "^4.4.3",
-    "@voyant-travel/core": "^0.111.0",
+    "@voyant-travel/core": "^0.111.1",
     "@voyant-travel/db": "^0.109.4",
     "@voyant-travel/storage": "^0.106.0",
     "@voyant-travel/types": "^0.106.1",