@agent-native/core 0.98.5 → 0.98.7

package/dist/client/session-replay.js

@@ -120,6 +120,7 @@ function getState() {
         g[SESSION_REPLAY_STATE_KEY] = {
             active: false,
             startPromise: null,
+            startGeneration: 0,
             replayId: null,
             startedAtMs: null,
             sequence: 0,
@@ -152,6 +153,7 @@ function getState() {
     state.pendingFlushReason ??= null;
     state.pendingFlushWaiters ??= [];
     state.awaitingFullSnapshot ??= false;
+    state.startGeneration ??= 0;
     return state;
 }
 // The replay session record (replayId + sequence counter) lives in
@@ -656,12 +658,14 @@ const REPLAY_RESOURCE_TAGS = new Set([
     "audio",
     "track",
     "input",
+    "object",
     "link",
 ]);
 const NO_REPLAY_RESOURCE_ATTRIBUTES = new Set();
 const REPLAY_SRC_ATTRIBUTES = new Set(["src"]);
 const REPLAY_SRCSET_ATTRIBUTES = new Set(["src", "srcset"]);
 const REPLAY_VIDEO_ATTRIBUTES = new Set(["src", "poster"]);
+const REPLAY_OBJECT_ATTRIBUTES = new Set(["data"]);
 const REPLAY_HREF_ATTRIBUTES = new Set(["href"]);
 function replayAttributeString(attributes, key) {
     return typeof attributes[key] === "string"
@@ -692,6 +696,8 @@ function replayPreservedResourceAttributes(node) {
         case "audio":
         case "track":
             return REPLAY_SRC_ATTRIBUTES;
+        case "object":
+            return REPLAY_OBJECT_ATTRIBUTES;
         case "input":
             return node.type === "image"
                 ? REPLAY_SRC_ATTRIBUTES
@@ -983,12 +989,12 @@ class ReplayUploadHttpError extends Error {
         this.status = status;
     }
 }
-/** 4xx statuses where retrying the exact same batch can never succeed --
- * e.g. a 409 chunk-sequence/checksum conflict, or a 400 the server will
- * reject again. 408 (timeout) and 429 (rate limit) are excluded because a
- * later retry can plausibly succeed. */
+/** 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, and stopping forever on one of those would silently
+ * black out the rest of a long-lived SPA session. */
 function isDefinitiveReplayUploadClientError(status) {
-    return status >= 400 && status < 500 && status !== 408 && status !== 429;
+    return status === 400 || status === 409 || status === 413 || status === 422;
 }
 async function sendReplayUpload(options, body, callbacks = {}) {
     if (isCrossOriginReplayEndpoint(options.endpoint)) {
@@ -1382,6 +1388,21 @@ export async function flushSessionReplay(reason = "manual") {
  * on internal option shapes continuing to round-trip through the public API.
  */
 async function restartSessionReplayAfterConflict(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.
+    if (state.startPromise)
+        return state.startPromise;
+    const startGeneration = ++state.startGeneration;
+    let startPromise;
+    startPromise = restartSessionReplayAfterConflictInternal(state, options, sessionId, startGeneration).finally(() => {
+        if (state.startPromise === startPromise)
+            state.startPromise = null;
+    });
+    state.startPromise = startPromise;
+    return startPromise;
+}
+async function restartSessionReplayAfterConflictInternal(state, options, sessionId, startGeneration) {
     if (options.shouldStart && !options.shouldStart()) {
         return { started: false, reason: "disabled", sessionId, sampled: true };
     }
@@ -1402,7 +1423,7 @@ async function restartSessionReplayAfterConflict(state, options, sessionId) {
             sampled: true,
         };
     }
-    return startSessionReplayRecorder(state, options, sessionId, true, initialProperties);
+    return startSessionReplayRecorder(state, options, sessionId, true, initialProperties, startGeneration);
 }
 function installUrlMonitor(state) {
     if (!state.options || state.restoreUrlMonitor)
@@ -2178,8 +2199,12 @@ export async function startSessionReplay(options = {}) {
     }
     if (state.startPromise)
         return state.startPromise;
+    // This is a new caller-initiated recording episode. A prior episode's
+    // conflict-loop guard must not prevent this one from recovering once.
+    state.automaticConflictRestartAttempted = false;
+    const startGeneration = ++state.startGeneration;
     let startPromise;
-    startPromise = startSessionReplayRecorder(state, normalized, sessionId, sampled, initialProperties).finally(() => {
+    startPromise = startSessionReplayRecorder(state, normalized, sessionId, sampled, initialProperties, startGeneration).finally(() => {
         if (state.startPromise === startPromise) {
             state.startPromise = null;
         }
@@ -2187,7 +2212,7 @@ export async function startSessionReplay(options = {}) {
     state.startPromise = startPromise;
     return startPromise;
 }
-async function startSessionReplayRecorder(state, normalized, sessionId, sampled, initialProperties) {
+async function startSessionReplayRecorder(state, normalized, sessionId, sampled, initialProperties, startGeneration) {
     if (state.active && state.replayId) {
         return {
             started: true,
@@ -2204,6 +2229,9 @@ async function startSessionReplayRecorder(state, normalized, sessionId, sampled,
     catch {
         return { started: false, reason: "import-failed", sessionId, sampled };
     }
+    if (state.startGeneration !== startGeneration) {
+        return { started: false, reason: "disabled", sessionId, sampled };
+    }
     if (normalized.shouldStart && !normalized.shouldStart()) {
         return { started: false, reason: "disabled", sessionId, sampled };
     }
@@ -2232,6 +2260,19 @@ async function startSessionReplayRecorder(state, normalized, sessionId, sampled,
             };
         }
     }
+    // stopSessionReplay may be called while the duplicate-tab probe is waiting.
+    // Recheck both cancellation and the caller's live eligibility before rrweb
+    // is activated so a deferred start cannot escape a route/auth teardown.
+    if (state.startGeneration !== startGeneration ||
+        (normalized.shouldStart && !normalized.shouldStart())) {
+        try {
+            replayChannel?.close();
+        }
+        catch {
+            // best-effort cleanup
+        }
+        return { started: false, reason: "disabled", sessionId, sampled };
+    }
     state.options = normalized;
     state.replayId = replaySession.replayId;
     state.startedAtMs = replaySession.startedAtMs;
@@ -2328,6 +2369,10 @@ async function startSessionReplayRecorder(state, normalized, sessionId, sampled,
 }
 export async function stopSessionReplay(reason = "manual") {
     const state = getState();
+    // Invalidate an import/probe that has not set `active` yet. Without this,
+    // 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;
     if (!state.active)
         return;
     // Restore console/fetch/XHR before tearing down the recorder: the restore