cross-tab-worker-databus 0.20.88 → 0.20.89

package/dist/cjs/index.cjs

@@ -1659,8 +1659,9 @@ var DataBusTraceReporter = class {
   /** Start the periodic metrics flush interval. No-op when mode is 'events'
    * (no metrics to emit), when disabled, or when already running. */
   start() {
-    if (!this.enabled || this.intervalHandle || this.mode === TRACE_MODE.EVENTS) return;
+    if (!this.enabled) return;
     this.stopped = false;
+    if (this.intervalHandle || this.mode === TRACE_MODE.EVENTS) return;
     this.intervalStartedAt = this.now();
     this.intervalHandle = setInterval(() => this.flush(), this.metricsIntervalMs);
   }
@@ -1673,6 +1674,7 @@ var DataBusTraceReporter = class {
   }
   stop() {
     this.stopped = true;
+    this.pendingEvents = [];
     this.pause();
   }
   /** Synchronous sink state for diagnostics: whether delivery is async and how
@@ -1683,7 +1685,7 @@ var DataBusTraceReporter = class {
   }
   /** Record an instantaneous trace event (lifecycle, status, error, etc.). */
   event(event) {
-    if (!this.enabled || this.mode === TRACE_MODE.METRICS) return;
+    if (!this.enabled || this.stopped || this.mode === TRACE_MODE.METRICS) return;
     this.emit({ ...event, timestamp: this.now() });
   }
   /** Synchronous snapshot of the current metrics window without resetting it.
@@ -1766,7 +1768,7 @@ var DataBusTraceReporter = class {
    * Extracted so the four record / flush methods share one guard expression
    * instead of repeating `!this.enabled || this.mode === 'events'` at each. */
   get metricsActive() {
-    return this.enabled && this.mode !== TRACE_MODE.EVENTS;
+    return this.enabled && !this.stopped && this.mode !== TRACE_MODE.EVENTS;
   }
   /** Emit the accumulated metrics snapshot if the interval is active. */
   flush() {
@@ -2068,6 +2070,8 @@ var ReplayManager = class {
    * or stopped bus does not keep hammering the store) and stop the sweep. */
   suspend() {
     this.retryGeneration += 1;
+    this.pendingReplayPersistence = [];
+    this.retentionCutoff = null;
     this.stop();
   }
   /** Drop all in-memory buffers (used on full teardown). */
@@ -2166,19 +2170,20 @@ var ReplayManager = class {
       this.retentionCutoff = cutoff;
     }
     if (this.retentionCleanup) return;
+    const generation = this.retryGeneration;
     this.retentionCleanup = (async () => {
-      while (this.retentionCutoff !== null) {
+      while (this.retentionCutoff !== null && generation === this.retryGeneration) {
         const nextCutoff = this.retentionCutoff;
         this.retentionCutoff = null;
         try {
           await this.persistence.clearBefore(nextCutoff);
         } catch (error) {
-          this.onPersistenceError(error);
+          if (generation === this.retryGeneration) this.onPersistenceError(error);
         }
       }
     })().finally(() => {
       this.retentionCleanup = null;
-      if (this.retentionCutoff !== null) {
+      if (this.retentionCutoff !== null && generation === this.retryGeneration) {
         this.scheduleRetentionCleanup(this.retentionCutoff);
       }
     });
@@ -2195,7 +2200,9 @@ var ReplayManager = class {
       attempt += 1;
       try {
         if (generation !== this.retryGeneration) throw new PersistenceRetryCancelledError();
-        return await operation();
+        const result = await operation();
+        if (generation !== this.retryGeneration) throw new PersistenceRetryCancelledError();
+        return result;
       } catch (error) {
         if (error instanceof PersistenceRetryCancelledError || generation !== this.retryGeneration) {
           throw new PersistenceRetryCancelledError();
@@ -2329,7 +2336,7 @@ var DedupManager = class {
 };
 
 // src/core/version.ts
-var SDK_VERSION = true ? "0.20.88" : "";
+var SDK_VERSION = true ? "0.20.89" : "";
 
 // src/core/data-bus.ts
 var DEFAULT_REPLAY_MAX_PER_TOPIC = 100;
@@ -2564,8 +2571,8 @@ var CrossTabDataBus = class {
     this.suspended = false;
     this.activeConfig = config;
     this.resetFailureState();
-    this.trace.event({ type: TRACE_EVENT_TYPE.LIFECYCLE, action: TRACE_LIFECYCLE_ACTION.START });
     this.trace.start();
+    this.trace.event({ type: TRACE_EVENT_TYPE.LIFECYCLE, action: TRACE_LIFECYCLE_ACTION.START });
     this.startDedupSweep();
     this.replayManager.start();
     this.updateStatus(WORKER_STATUS.CONNECTING);
@@ -2925,7 +2932,7 @@ var CrossTabDataBus = class {
   getHealthSummary() {
     const transport = this.transport;
     const transportDown = this.status !== WORKER_STATUS.CONNECTED;
-    const state = !this.started ? HEALTH_STATE.STOPPED : this.suspended ? HEALTH_STATE.SUSPENDED : transportDown ? this.recoveryExhausted ? HEALTH_STATE.DEGRADED : this.status === WORKER_STATUS.CONNECTING && this.recoveryAttempt === 0 ? HEALTH_STATE.STARTING : HEALTH_STATE.RECOVERING : HEALTH_STATE.HEALTHY;
+    const state = !this.started || this.stopping ? HEALTH_STATE.STOPPED : this.suspended ? HEALTH_STATE.SUSPENDED : transportDown ? this.recoveryExhausted ? HEALTH_STATE.DEGRADED : this.status === WORKER_STATUS.CONNECTING && this.recoveryAttempt === 0 ? HEALTH_STATE.STARTING : HEALTH_STATE.RECOVERING : HEALTH_STATE.HEALTHY;
     return {
       healthy: state === HEALTH_STATE.HEALTHY,
       state,
@@ -2996,7 +3003,7 @@ var CrossTabDataBus = class {
       this.canceledQueuedStartToken = this.queuedStartToken;
       this.queuedStart = null;
     }
-    if (this.stopPromise) return this.stopPromise;
+    if (this.stopPromise && this.stopping) return this.stopPromise;
     if (!this.started && !this.startPromise && !this.pendingStop && !this.transportReady) {
       return Promise.resolve();
     }
@@ -3240,8 +3247,8 @@ var CrossTabDataBus = class {
    * pageshow path and explicit start() must run this so an explicit resume
    * cannot leave trace metrics and periodic cleanup timers permanently off. */
   resumeSuspendedResources() {
-    this.trace.event({ type: TRACE_EVENT_TYPE.LIFECYCLE, action: TRACE_LIFECYCLE_ACTION.RESUME });
     this.trace.start();
+    this.trace.event({ type: TRACE_EVENT_TYPE.LIFECYCLE, action: TRACE_LIFECYCLE_ACTION.RESUME });
     this.startDedupSweep();
     this.replayManager.start();
   }
@@ -3936,10 +3943,16 @@ var WebSocketTransport = class {
    * server cannot crash the message path. */
   async handleMessage(raw) {
     if (typeof Blob !== "undefined" && raw instanceof Blob) {
+      const socket = this.socket;
+      const handlers = this.handlers;
       try {
-        await this.handleMessage(await raw.arrayBuffer());
+        const buffer = await raw.arrayBuffer();
+        if (this.socket !== socket || this.handlers !== handlers || !this.socketActive) return;
+        await this.handleMessage(buffer);
       } catch (error) {
-        this.handlers?.onError(error);
+        if (this.socket === socket && this.handlers === handlers && this.socketActive) {
+          this.handlers?.onError(error);
+        }
       }
       return;
     }