@agent-native/core 0.163.1 → 0.163.2

package/dist/client/session-replay.js

@@ -136,6 +136,10 @@ function getState() {
             flushing: false,
             pendingFlushReason: null,
             pendingFlushWaiters: [],
+            pendingReplayUpload: null,
+            pendingReplayStart: null,
+            pendingReplayRecovery: null,
+            bfcacheRestored: false,
             awaitingFullSnapshot: false,
             stopRecorder: null,
             restoreUrlMonitor: null,
@@ -157,6 +161,10 @@ function getState() {
     state.restoreIframeBridge ??= null;
     state.pendingFlushReason ??= null;
     state.pendingFlushWaiters ??= [];
+    state.pendingReplayUpload ??= null;
+    state.pendingReplayStart ??= null;
+    state.pendingReplayRecovery ??= null;
+    state.bfcacheRestored ??= false;
     state.awaitingFullSnapshot ??= false;
     state.startGeneration ??= 0;
     state.replayLinkBaseUrl ??= null;
@@ -839,6 +847,17 @@ function enqueueReplayEvent(state, event) {
     if (!state.options)
         return;
     const eventType = typeof event.type === "number" ? event.type : null;
+    if (state.pendingReplayUpload) {
+        // A timed-out request has an uncertain server outcome. Keep at most the
+        // newest FullSnapshot as a bounded placeholder until the old request
+        // settles; never let an uncertain upload turn into a retry backlog.
+        if (eventType !== RRWEB_FULL_SNAPSHOT_EVENT_TYPE)
+            return;
+        state.queue = [];
+        state.queuedBytes = 0;
+        state.retryBatches = [];
+        state.awaitingFullSnapshot = false;
+    }
     if (state.awaitingFullSnapshot) {
         if (eventType !== RRWEB_FULL_SNAPSHOT_EVENT_TYPE)
             return;
@@ -1031,6 +1050,19 @@ class ReplayUploadHttpError extends Error {
         this.status = status;
     }
 }
+/**
+ * The timeout won before the transport settled, so retrying the batch would
+ * race the original request. The caller fences this request and waits for its
+ * late outcome before allowing another upload for the replay sequence.
+ */
+class ReplayUploadInFlightTimeoutError extends Error {
+    request;
+    constructor(request) {
+        super("Session replay upload timed out while still in flight");
+        this.name = "ReplayUploadInFlightTimeoutError";
+        this.request = request;
+    }
+}
 /** 4xx statuses where retrying the exact same batch can never succeed.
  * Keep this deliberately narrow: 401/403/404 can be temporary during auth or
  * deploy transitions, so they get a small retry budget before the rejected
@@ -1052,45 +1084,76 @@ function startReplayUploadTimeout() {
     // settles so completed uploads do not leave timeout work behind.
     const controller = typeof AbortController === "undefined" ? undefined : new AbortController();
     let rejectTimeout;
+    let timedOut = false;
+    const timeoutError = new Error("Session replay upload timed out");
     const promise = new Promise((_, reject) => {
         rejectTimeout = reject;
     });
     const timer = setTimeout(() => {
+        timedOut = true;
         controller?.abort();
-        rejectTimeout(new Error("Session replay upload timed out"));
+        rejectTimeout(timeoutError);
     }, REPLAY_UPLOAD_TIMEOUT_MS);
     return {
         signal: controller?.signal,
         promise,
+        didTimeout: () => timedOut,
         done: () => clearTimeout(timer),
     };
 }
 function isTransientReplayUploadClientError(status) {
     return status === 401 || status === 403 || status === 404;
 }
+function awaitReplayUploadRequest(timeout, createRequest) {
+    let request;
+    try {
+        request = createRequest();
+    }
+    catch (error) {
+        // fetch can throw before awaitReplayUpload gets a chance to enter its
+        // finally block. Clear the timer here so a synchronous request failure
+        // cannot leave an orphaned timeout or rejected timeout promise behind.
+        timeout.done();
+        throw error;
+    }
+    return awaitReplayUpload(request, timeout);
+}
+async function awaitReplayUpload(request, timeout) {
+    const checkedRequest = request.then((response) => {
+        if (!response.ok) {
+            throw new ReplayUploadHttpError(response.status);
+        }
+    });
+    try {
+        await Promise.race([checkedRequest, timeout.promise]);
+    }
+    catch (error) {
+        // An abort signal only cancels this client's observation of fetch. A
+        // keepalive request may already have reached the server, so every timeout
+        // is uncertain until the original promise settles. Fence the old replay
+        // identity instead of retrying its sequence.
+        if (timeout.didTimeout()) {
+            throw new ReplayUploadInFlightTimeoutError(checkedRequest);
+        }
+        throw error;
+    }
+    finally {
+        timeout.done();
+    }
+}
 async function sendReplayUpload(options, body, callbacks = {}) {
     if (isCrossOriginReplayEndpoint(options.endpoint)) {
         const canUseKeepalive = canUseReplayKeepalive(body);
         if (canUseKeepalive)
             callbacks.beforeKeepaliveUpload?.();
         const timeout = startReplayUploadTimeout();
-        let response;
-        try {
-            const request = fetch(options.endpoint, {
-                method: "POST",
-                body,
-                keepalive: canUseKeepalive,
-                headers: { "Content-Type": "text/plain;charset=UTF-8" },
-                signal: timeout.signal,
-            });
-            response = await Promise.race([request, timeout.promise]);
-        }
-        finally {
-            timeout.done();
-        }
-        if (!response.ok) {
-            throw new ReplayUploadHttpError(response.status);
-        }
+        await awaitReplayUploadRequest(timeout, () => fetch(options.endpoint, {
+            method: "POST",
+            body,
+            keepalive: canUseKeepalive,
+            headers: { "Content-Type": "text/plain;charset=UTF-8" },
+            signal: timeout.signal,
+        }));
         return;
     }
     const upload = await buildReplayUploadBody(body);
@@ -1098,42 +1161,39 @@ async function sendReplayUpload(options, body, callbacks = {}) {
     if (canUseKeepalive)
         callbacks.beforeKeepaliveUpload?.();
     const timeout = startReplayUploadTimeout();
-    let response;
-    try {
-        const request = fetch(options.endpoint, {
-            method: "POST",
-            body: upload.body,
-            keepalive: canUseKeepalive,
-            headers: {
-                ...upload.headers,
-                "X-Agent-Native-Analytics-Key": options.publicKey,
-            },
-            signal: timeout.signal,
-        });
-        response = await Promise.race([request, timeout.promise]);
-    }
-    finally {
-        timeout.done();
-    }
-    if (!response.ok) {
-        throw new ReplayUploadHttpError(response.status);
-    }
+    await awaitReplayUploadRequest(timeout, () => fetch(options.endpoint, {
+        method: "POST",
+        body: upload.body,
+        keepalive: canUseKeepalive,
+        headers: {
+            ...upload.headers,
+            "X-Agent-Native-Analytics-Key": options.publicKey,
+        },
+        signal: timeout.signal,
+    }));
 }
 function isFinalFlushReason(reason) {
     return [
         "auth-cleared",
         "manual",
         "pagehide",
+        "pagehide-persisted",
         "beforeunload",
         "url-blocked",
         "max-duration",
     ].includes(reason);
 }
+function isTerminalReplayFlushReason(reason) {
+    return reason === "pagehide" || reason === "beforeunload";
+}
 function flushReasonPriority(reason) {
     // Unload reasons must retain their keepalive sequence reservation even when
     // another final request (for example, an explicit manual stop) is coalesced.
-    if (reason === "pagehide" || reason === "beforeunload")
+    if (reason === "pagehide" ||
+        reason === "pagehide-persisted" ||
+        reason === "beforeunload") {
         return 3;
+    }
     if (isFinalFlushReason(reason))
         return 2;
     if (reason === "visibility-hidden")
@@ -1149,6 +1209,7 @@ function mergePendingFlushReason(current, requested) {
 }
 function shouldReserveSequenceBeforeKeepalive(reason) {
     return (reason === "pagehide" ||
+        reason === "pagehide-persisted" ||
         reason === "beforeunload" ||
         reason === "visibility-hidden");
 }
@@ -1169,6 +1230,8 @@ function flushQueuedReplayIfNeeded(state) {
     const options = state.options;
     if (!options)
         return;
+    if (state.pendingReplayUpload)
+        return;
     if (state.retryBatches.length > 0)
         return;
     if (!shouldFlushQueuedReplay(state))
@@ -1269,6 +1332,105 @@ function quarantinePendingReplayUntilFullSnapshot(state) {
 function restoreReplayEvents(state, events) {
     state.retryBatches.unshift(events);
 }
+function resumeAfterPendingReplayUpload(state, pending) {
+    if (state.pendingReplayUpload !== pending && !state.pendingReplayRecovery) {
+        return;
+    }
+    void recoverAfterPendingReplayUpload(state, pending);
+}
+function recoverAfterPendingReplayUpload(state, pending) {
+    if (state.pendingReplayRecovery)
+        return state.pendingReplayRecovery;
+    if (state.pendingReplayUpload !== pending)
+        return Promise.resolve();
+    let recovery;
+    recovery = recoverAfterPendingReplayUploadInternal(state, pending).finally(() => {
+        if (state.pendingReplayRecovery === recovery) {
+            state.pendingReplayRecovery = null;
+        }
+        for (const resolve of state.pendingFlushWaiters.splice(0))
+            resolve();
+    });
+    state.pendingReplayRecovery = recovery;
+    return recovery;
+}
+async function recoverAfterPendingReplayUploadInternal(state, pending) {
+    state.bfcacheRestored = false;
+    try {
+        if (state.replayId !== pending.payload.replayId) {
+            state.pendingReplayUpload = null;
+            state.pendingReplayStart = null;
+            return;
+        }
+        const wasActive = state.active;
+        const suppressRestart = isTerminalReplayFlushReason(pending.reason) ||
+            isTerminalReplayFlushReason(state.pendingFlushReason ?? "");
+        // Never reuse the replay identity after a timeout. The server may have
+        // accepted the old request even when this client observed an abort, so a
+        // later FullSnapshot under the old identity could conflict at the same
+        // sequence. Restarting rrweb also emits a fresh Meta + FullSnapshot pair.
+        state.queue = [];
+        state.queuedBytes = 0;
+        state.retryBatches = [];
+        state.awaitingFullSnapshot = false;
+        removeStoredReplaySession(pending.payload.replayId);
+        let stopCancelledRecovery = false;
+        if (wasActive) {
+            // stopSessionReplay increments startGeneration synchronously. If a
+            // caller explicitly stops during its teardown await, that extra
+            // generation change must cancel the implicit timeout restart rather
+            // than allowing the captured old recorder options to start again.
+            const expectedStopGeneration = state.startGeneration + 1;
+            await stopSessionReplay("upload-timeout");
+            stopCancelledRecovery =
+                state.startGeneration !== expectedStopGeneration &&
+                    !state.pendingReplayStart;
+        }
+        // Teardown yields while it flushes the final capture restoration. A caller
+        // can request a different replay configuration during that gap, so select
+        // the pending request only after teardown has finished instead of restarting
+        // with the stale request captured above.
+        const restartRequest = suppressRestart || stopCancelledRecovery
+            ? null
+            : (state.pendingReplayStart ??
+                (wasActive && state.options
+                    ? { options: state.options, sessionId: pending.payload.sessionId }
+                    : null));
+        // Capture restoration during stop can emit one last event. Keep it bounded
+        // and discard it before the fresh recorder emits its own Meta + FullSnapshot.
+        state.queue = [];
+        state.queuedBytes = 0;
+        state.retryBatches = [];
+        state.awaitingFullSnapshot = false;
+        state.pendingFlushReason = null;
+        state.pendingReplayUpload = null;
+        state.pendingReplayStart = null;
+        if (!restartRequest || state.active)
+            return;
+        await restartSessionReplayWithFreshIdentity(state, restartRequest.options, restartRequest.sessionId);
+    }
+    finally {
+        // The wrapper owns clearing state.pendingReplayRecovery after every
+        // recovery caller observes the same settled promise.
+    }
+}
+function fenceTimedOutReplayUpload(state, pending) {
+    void pending.request
+        .then(() => undefined, (error) => {
+        // The uncertain batch is intentionally not requeued. Once the old
+        // promise settles, recovery rotates to a fresh replay identity instead
+        // of guessing whether this error means the server rejected the write.
+        const previousInternal = replayCaptureInternal;
+        replayCaptureInternal = true;
+        try {
+            console.warn("[session-replay] timed-out upload settled without a retry; restarting the replay", error);
+        }
+        finally {
+            replayCaptureInternal = previousInternal;
+        }
+    })
+        .finally(() => resumeAfterPendingReplayUpload(state, pending));
+}
 function advanceReplaySequence(state, payload) {
     if (state.replayId !== payload.replayId)
         return;
@@ -1287,18 +1449,29 @@ export async function flushSessionReplay(reason = "manual") {
     const state = getState();
     if (!state.options)
         return;
+    if (state.pendingReplayUpload) {
+        // The fence blocks another upload, but it must not turn teardown into an
+        // unbounded wait when the original transport never settles.
+        state.pendingFlushReason = mergePendingFlushReason(state.pendingFlushReason, reason);
+        return;
+    }
     if (state.flushing) {
         state.pendingFlushReason = mergePendingFlushReason(state.pendingFlushReason, reason);
         return new Promise((resolve) => {
             state.pendingFlushWaiters.push(resolve);
         });
     }
-    if (!hasPendingReplayBatch(state))
+    if (!hasPendingReplayBatch(state)) {
+        if (reason === "pagehide-persisted")
+            state.bfcacheRestored = false;
         return;
+    }
     const events = state.retryBatches.shift() ?? takeQueuedReplayBatch(state);
     const payload = buildReplayBody(state, reason, events);
     if (!payload || !state.options) {
         restoreReplayEvents(state, events);
+        if (reason === "pagehide-persisted")
+            state.bfcacheRestored = false;
         return;
     }
     state.flushing = true;
@@ -1306,6 +1479,8 @@ export async function flushSessionReplay(reason = "manual") {
     let reservedSequence = false;
     let splitRejectedBatch = false;
     let droppedOversizedBatch = false;
+    let fencedTimedOutUpload = false;
+    let isDefinitiveClientError = false;
     let definitiveClientErrorStatus = null;
     try {
         await sendReplayUpload(state.options, payload.body, {
@@ -1323,63 +1498,90 @@ export async function flushSessionReplay(reason = "manual") {
         uploaded = true;
     }
     catch (error) {
-        if (reservedSequence)
-            rollbackReplaySequenceReservation(state, payload);
-        // A definitive 4xx (e.g. a 409 chunk-sequence/checksum conflict) can
-        // never succeed by retrying the exact same batch -- requeuing it would
-        // just spin forever, blocking every later batch behind it (flushes are
-        // FIFO via `retryBatches`). Drop it and move on instead.
-        const rejectedStatus = error instanceof ReplayUploadHttpError ? error.status : null;
-        const splitBatch = rejectedStatus === 413 ? splitReplayBatch(events) : null;
-        const isUnsplittableOversizedBatch = rejectedStatus === 413 && splitBatch === null;
-        const isTransientClientError = rejectedStatus !== null &&
-            isTransientReplayUploadClientError(rejectedStatus);
-        if (isTransientClientError) {
-            state.transientClientErrorFailures += 1;
-        }
-        else {
-            state.transientClientErrorFailures = 0;
-        }
-        const exhaustedTransientClientRetries = isTransientClientError &&
-            state.transientClientErrorFailures >=
-                MAX_TRANSIENT_REPLAY_CLIENT_FAILURES;
-        const isDefinitiveClientError = error instanceof ReplayUploadHttpError &&
-            (isDefinitiveReplayUploadClientError(error.status) ||
-                exhaustedTransientClientRetries) &&
-            !splitBatch &&
-            !isUnsplittableOversizedBatch;
-        if (splitBatch) {
-            // A server or platform can enforce a stricter decompressed-body limit
-            // than the recorder's configured queue cap. Bisect in FIFO order and
-            // retry both halves at the same sequence; only successful halves advance
-            // it, so no event is duplicated or skipped.
-            state.retryBatches.unshift(...splitBatch);
-            splitRejectedBatch = true;
-        }
-        else if (isUnsplittableOversizedBatch) {
-            // This one event cannot be made any smaller. Drop only the rejected
-            // singleton and keep later retry halves/live batches in FIFO order. If
-            // it carried DOM structure, quarantine dependent mutations until a new
-            // FullSnapshot can safely re-anchor rrweb's node mirror.
-            droppedOversizedBatch = true;
-            if (replayBatchNeedsDomReset(events)) {
-                quarantinePendingReplayUntilFullSnapshot(state);
-            }
-        }
-        else if (isDefinitiveClientError) {
-            // Continuing after a checksum/sequence conflict would reuse the same
-            // rejected sequence forever. More importantly, advancing past it would
-            // append mutations to a replay whose DOM stream may belong to another
-            // tab. End this recorder and clear its persisted identity so the next
-            // start creates a clean replay instead of producing corrupt playback.
+        if (error instanceof ReplayUploadInFlightTimeoutError) {
+            const pending = {
+                request: error.request,
+                payload,
+                reason,
+            };
+            state.pendingReplayUpload = pending;
+            // The timed-out batch is deliberately fenced rather than requeued:
+            // without transport cancellation, a late success could commit it after
+            // a retry and create a duplicate sequence. Drop the accumulated live
+            // backlog and wait for a new FullSnapshot to re-anchor safely.
+            state.retryBatches = [];
             state.queue = [];
             state.queuedBytes = 0;
-            state.retryBatches = [];
-            removeStoredReplaySession(payload.replayId);
-            definitiveClientErrorStatus = error.status;
+            state.awaitingFullSnapshot = true;
+            fencedTimedOutUpload = true;
+            fenceTimedOutReplayUpload(state, pending);
+            if (state.bfcacheRestored) {
+                // If pageshow happened before this timeout fired, there was no
+                // pending upload for the pageshow handler to recover. Connect this
+                // post-restore timeout directly to the same fresh-identity path.
+                void recoverAfterPendingReplayUpload(state, pending);
+            }
         }
         else {
-            restoreReplayEvents(state, events);
+            if (reservedSequence)
+                rollbackReplaySequenceReservation(state, payload);
+            // A definitive 4xx (e.g. a 409 chunk-sequence/checksum conflict) can
+            // never succeed by retrying the exact same batch - requeuing it would
+            // just spin forever, blocking every later batch behind it (flushes are
+            // FIFO via `retryBatches`). Drop it and move on instead.
+            const rejectedStatus = error instanceof ReplayUploadHttpError ? error.status : null;
+            const splitBatch = rejectedStatus === 413 ? splitReplayBatch(events) : null;
+            const isUnsplittableOversizedBatch = rejectedStatus === 413 && splitBatch === null;
+            const isTransientClientError = rejectedStatus !== null &&
+                isTransientReplayUploadClientError(rejectedStatus);
+            if (isTransientClientError) {
+                state.transientClientErrorFailures += 1;
+            }
+            else {
+                state.transientClientErrorFailures = 0;
+            }
+            const exhaustedTransientClientRetries = isTransientClientError &&
+                state.transientClientErrorFailures >=
+                    MAX_TRANSIENT_REPLAY_CLIENT_FAILURES;
+            isDefinitiveClientError =
+                error instanceof ReplayUploadHttpError &&
+                    (isDefinitiveReplayUploadClientError(error.status) ||
+                        exhaustedTransientClientRetries) &&
+                    !splitBatch &&
+                    !isUnsplittableOversizedBatch;
+            if (splitBatch) {
+                // A server or platform can enforce a stricter decompressed-body limit
+                // than the recorder's configured queue cap. Bisect in FIFO order and
+                // retry both halves at the same sequence; only successful halves advance
+                // it, so no event is duplicated or skipped.
+                state.retryBatches.unshift(...splitBatch);
+                splitRejectedBatch = true;
+            }
+            else if (isUnsplittableOversizedBatch) {
+                // This one event cannot be made any smaller. Drop only the rejected
+                // singleton and keep later retry halves/live batches in FIFO order. If
+                // it carried DOM structure, quarantine dependent mutations until a new
+                // FullSnapshot can safely re-anchor rrweb's node mirror.
+                droppedOversizedBatch = true;
+                if (replayBatchNeedsDomReset(events)) {
+                    quarantinePendingReplayUntilFullSnapshot(state);
+                }
+            }
+            else if (isDefinitiveClientError) {
+                // Continuing after a checksum/sequence conflict would reuse the same
+                // rejected sequence forever. More importantly, advancing past it would
+                // append mutations to a replay whose DOM stream may belong to another
+                // tab. End this recorder and clear its persisted identity so the next
+                // start creates a clean replay instead of producing corrupt playback.
+                state.queue = [];
+                state.queuedBytes = 0;
+                state.retryBatches = [];
+                removeStoredReplaySession(payload.replayId);
+                definitiveClientErrorStatus = rejectedStatus;
+            }
+            else {
+                restoreReplayEvents(state, events);
+            }
         }
         // Guard the recorder's own warning so console capture never records it
         // (a captured warning would enqueue an event and retrigger a flush).
@@ -1395,6 +1597,9 @@ export async function flushSessionReplay(reason = "manual") {
             else if (isDefinitiveClientError) {
                 console.warn(`[session-replay] dropping upload (HTTP ${error.status})`, error);
             }
+            else if (fencedTimedOutUpload) {
+                console.warn("[session-replay] upload timed out; waiting for it to settle before restarting");
+            }
             else {
                 console.warn("[session-replay] upload failed", error);
             }
@@ -1406,6 +1611,11 @@ export async function flushSessionReplay(reason = "manual") {
     finally {
         state.flushing = false;
     }
+    if (reason === "pagehide-persisted" && !fencedTimedOutUpload) {
+        state.bfcacheRestored = false;
+    }
+    if (fencedTimedOutUpload)
+        return;
     const coalescedReason = state.pendingFlushReason;
     const coalescedWaiters = coalescedReason
         ? state.pendingFlushWaiters.splice(0)
@@ -1457,7 +1667,7 @@ export async function flushSessionReplay(reason = "manual") {
         // configuration/input errors and must not create a restart loop.
         let restartResult = null;
         if (shouldRestartAfterConflict && rejectedOptions) {
-            restartResult = await restartSessionReplayAfterConflict(state, rejectedOptions, payload.sessionId);
+            restartResult = await restartSessionReplayWithFreshIdentity(state, rejectedOptions, payload.sessionId);
         }
         // Rare recovery-path telemetry lets Analytics owners quantify conflicts
         // without recording the rejected replay id, URL, or any captured content.
@@ -1479,7 +1689,7 @@ export async function flushSessionReplay(reason = "manual") {
         resolve();
 }
 /**
- * Restart a recorder whose replay identity was rejected without routing the
+ * Restart a recorder under a fresh replay identity without routing the
  * already-normalized options back through the public start API.
  *
  * The original recording already passed whole-session sampling. Re-entering
@@ -1490,7 +1700,7 @@ export async function flushSessionReplay(reason = "manual") {
  * reuse the exact normalized console/network capture caps instead of relying
  * on internal option shapes continuing to round-trip through the public API.
  */
-async function restartSessionReplayAfterConflict(state, options, sessionId) {
+async function restartSessionReplayWithFreshIdentity(state, options, sessionId) {
     // Share the public-start mutex. A consumer may call start in the await gap
     // after the rejected recorder stops; both paths must converge on one rrweb
     // instance rather than racing two fresh identities.
@@ -1498,14 +1708,14 @@ async function restartSessionReplayAfterConflict(state, options, sessionId) {
         return state.startPromise;
     const startGeneration = ++state.startGeneration;
     let startPromise;
-    startPromise = restartSessionReplayAfterConflictInternal(state, options, sessionId, startGeneration).finally(() => {
+    startPromise = restartSessionReplayWithFreshIdentityInternal(state, options, sessionId, startGeneration).finally(() => {
         if (state.startPromise === startPromise)
             state.startPromise = null;
     });
     state.startPromise = startPromise;
     return startPromise;
 }
-async function restartSessionReplayAfterConflictInternal(state, options, sessionId, startGeneration) {
+async function restartSessionReplayWithFreshIdentityInternal(state, options, sessionId, startGeneration) {
     if (options.shouldStart && !options.shouldStart()) {
         return { started: false, reason: "disabled", sessionId, sampled: true };
     }
@@ -1565,14 +1775,31 @@ function installLifecycleListeners(state) {
             void flushSessionReplay("visibility-hidden");
         }
     };
-    const flushOnUnload = () => {
-        void flushSessionReplay("pagehide");
+    const flushOnUnload = (event) => {
+        // pagehide also fires when the document enters BFCache. That page is
+        // expected to resume capture on pageshow, so keep the timeout fence's
+        // restart behavior for persisted pagehide while retaining terminal
+        // unload semantics for an actual document teardown.
+        state.bfcacheRestored = false;
+        void flushSessionReplay(event.persisted ? "pagehide-persisted" : "pagehide");
+    };
+    const resumeFromBfcache = (event) => {
+        if (!event.persisted)
+            return;
+        state.bfcacheRestored = true;
+        const pending = state.pendingReplayUpload;
+        if (pending)
+            void recoverAfterPendingReplayUpload(state, pending);
+        else if (!state.flushing)
+            state.bfcacheRestored = false;
     };
     document.addEventListener("visibilitychange", flushOnHidden);
     window.addEventListener("pagehide", flushOnUnload);
+    window.addEventListener("pageshow", resumeFromBfcache);
     state.removeLifecycleListeners = () => {
         document.removeEventListener("visibilitychange", flushOnHidden);
         window.removeEventListener("pagehide", flushOnUnload);
+        window.removeEventListener("pageshow", resumeFromBfcache);
         state.removeLifecycleListeners = null;
     };
 }
@@ -2291,6 +2518,18 @@ export async function startSessionReplay(options = {}) {
         return { started: false, reason: "url-blocked", sessionId, sampled };
     }
     const state = getState();
+    if (state.pendingReplayUpload) {
+        state.pendingReplayStart = { options: normalized, sessionId };
+        // Do not start a new recorder episode while the previous episode's
+        // timed-out upload still owns an uncertain server outcome.
+        return {
+            started: false,
+            reason: "already-active",
+            replayId: state.replayId ?? undefined,
+            sessionId,
+            sampled,
+        };
+    }
     if (state.active && state.replayId) {
         return {
             started: true,
@@ -2392,6 +2631,8 @@ async function startSessionReplayRecorder(state, normalized, sessionId, sampled,
     state.queue = [];
     state.queuedBytes = 0;
     state.retryBatches = [];
+    state.pendingReplayUpload = null;
+    state.pendingReplayStart = null;
     state.pendingFlushReason = null;
     for (const resolve of state.pendingFlushWaiters.splice(0))
         resolve();
@@ -2485,6 +2726,14 @@ export async function stopSessionReplay(reason = "manual") {
     // stop during the duplicated-tab claim window was a no-op and rrweb started
     // after the caller believed recording had been disabled.
     state.startGeneration += 1;
+    // An explicit stop cancels a caller-requested restart that was queued behind
+    // an uncertain upload. Recovery uses its private reason to preserve that
+    // request while it tears down the old recorder.
+    if (reason !== "upload-timeout")
+        state.pendingReplayStart = null;
+    if (reason !== "pagehide-persisted") {
+        state.bfcacheRestored = false;
+    }
     if (!state.active)
         return;
     // Restore console/fetch/XHR before tearing down the recorder: the restore

package/dist/collab/struct-routes.d.ts

@@ -13,8 +13,8 @@
  * Body: { json: any, fieldName?: string, type?: "map"|"array", requestSource?: string }
  */
 export declare const postCollabJson: import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
-    ok?: undefined;
     error: string;
+    ok?: undefined;
 } | {
     error?: undefined;
     ok: boolean;

package/dist/file-upload/actions/upload-image.d.ts

@@ -17,11 +17,11 @@ declare const _default: import("../../action.js").ActionDefinition<{
     id?: undefined;
     provider?: undefined;
 } | {
-    error?: undefined;
     configured?: undefined;
     connectPath?: undefined;
     url: string;
     id: string;
     provider: string;
+    error?: undefined;
 }>;
 export default _default;

package/dist/secrets/routes.d.ts

@@ -34,37 +34,37 @@ export declare function createListSecretsHandler(): import("h3").EventHandlerWit
 /** POST /_agent-native/secrets/:key — write a secret. */
 export declare function createWriteSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
     error: string;
-    ok?: undefined;
     status?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     status: string;
+    error?: undefined;
 } | {
-    ok?: undefined;
     error: string;
     removed?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     removed: boolean;
+    error?: undefined;
 }>>;
 /**
  * POST /_agent-native/secrets/:key/test — validate an optional candidate value
  * or the current stored value without changing anything.
  */
 export declare function createTestSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
-    ok?: undefined;
     error: string;
     note?: undefined;
+    ok?: undefined;
 } | {
-    error?: undefined;
     ok: boolean;
     note?: undefined;
-} | {
     error?: undefined;
+} | {
     ok: boolean;
     note: string;
+    error?: undefined;
 } | {
     note?: undefined;
     ok: boolean;
@@ -95,11 +95,11 @@ export interface AdHocSecretPayload {
 export declare function createAdHocSecretHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<AdHocSecretPayload[] | {
     error: string;
 } | {
-    error?: undefined;
     ok: boolean;
     key: string;
-} | {
     error?: undefined;
+} | {
     ok: boolean;
     removed: boolean;
+    error?: undefined;
 }>>;

package/dist/server/agent-engine-api-key-route.d.ts

@@ -27,10 +27,10 @@ export declare function resolveAgentEngineApiKeyWriteTarget(event: H3Event, scop
 export declare function createAgentEngineApiKeyHandler(): import("h3").EventHandlerWithFetch<import("h3").EventHandlerRequest, Promise<{
     error: any;
 } | {
-    error?: undefined;
     ok: boolean;
     key: string;
     baseUrlKey?: string;
     scope: AgentEngineApiKeyScope;
+    error?: undefined;
 }>>;
 export {};

package/dist/server/transcribe-voice.d.ts

@@ -20,6 +20,6 @@ export declare function createTranscribeVoiceHandler(): import("h3").EventHandle
     error: string;
     text?: undefined;
 } | {
-    error?: undefined;
     text: string;
+    error?: undefined;
 }>>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-native/core",
-  "version": "0.163.1",
+  "version": "0.163.2",
   "description": "Framework for agent-native application development — where AI agents and UI share SQL state, actions, and context",
   "homepage": "https://github.com/BuilderIO/agent-native#readme",
   "bugs": {