@sailfish-ai/recorder 1.10.5 → 1.10.7

package/dist/inAppReportIssueModal/state.js

@@ -1,9 +1,12 @@
 import { STORAGE_KEYS } from "./types";
+import { HAS_LOCAL_STORAGE } from "../runtimeEnv";
 // Helper function to load user preferences from localStorage
 function loadUserPreferences() {
     return {
-        createIssue: localStorage.getItem(STORAGE_KEYS.CREATE_ISSUE) === "true",
-        createEngTicket: localStorage.getItem(STORAGE_KEYS.CREATE_ENG_TICKET) === "true",
+        createIssue: HAS_LOCAL_STORAGE &&
+            localStorage.getItem(STORAGE_KEYS.CREATE_ISSUE) === "true",
+        createEngTicket: HAS_LOCAL_STORAGE &&
+            localStorage.getItem(STORAGE_KEYS.CREATE_ENG_TICKET) === "true",
     };
 }
 // Helper function to get initial state

package/dist/index.js

@@ -8,11 +8,12 @@ import { readGitSha } from "./env";
 import { initializeErrorInterceptor } from "./errorInterceptor";
 import { fetchCaptureSettings, fetchFunctionSpanTrackingEnabled, sendDomainsToNotPropagateHeaderTo, startRecordingSession, } from "./graphql";
 import { sendMapUuidIfAvailable } from "./mapUuid";
-import { getUrlAndStoredUuids, initializeConsolePlugin, initializeDomContentEvents, initializeRecording, } from "./recording";
+import { getUrlAndStoredUuids, initializeConsolePlugin, initializeDomContentEvents, initializeRecording, invalidateUrlCache, } from "./recording";
 import { HAS_DOCUMENT, HAS_LOCAL_STORAGE, HAS_SESSION_STORAGE, HAS_WINDOW, } from "./runtimeEnv";
 import { ensureSessionListeners, getOrSetSessionId } from "./session";
 import { withAppUrlMetadata } from "./utils";
 import { clearStaleFuncSpanState, getFuncSpanHeader, isFunctionSpanTrackingEnabled, restoreFuncSpanState, sendEvent, sendMessage, } from "./websocket";
+import { yieldToMain } from "./scheduler";
 const DEBUG = readDebugFlag(); // A wrapper around fetch that suppresses connection refused errors
 // Regex cache for matchUrlWithWildcard() — avoids recompiling on every network request
 const _regexCache = new Map();
@@ -148,6 +149,7 @@ function trackDomainChangesOnce() {
             const prevPageVisitUUID = sessionStorage.getItem("pageVisitUUID");
             sessionStorage.setItem("pageVisitUUID", pageVisitUUID);
             sessionStorage.setItem("prevPageVisitUUID", prevPageVisitUUID);
+            invalidateUrlCache();
             const timestamp = Date.now();
             sendMessage({
                 type: "routeChange",
@@ -242,6 +244,7 @@ function handleVisibilityChange() {
     // Store visibility state in sessionStorage
     sessionStorage.setItem("tabVisibilityChanged", timestamp.toString());
     sessionStorage.setItem("tabVisibilityState", visibilityState);
+    invalidateUrlCache();
 }
 function clearPageVisitDataFromSessionStorage() {
     if (!HAS_SESSION_STORAGE)
@@ -250,6 +253,7 @@ function clearPageVisitDataFromSessionStorage() {
     sessionStorage.removeItem("prevPageVisitUUID");
     sessionStorage.removeItem("tabVisibilityChanged");
     sessionStorage.removeItem("tabVisibilityState");
+    invalidateUrlCache();
 }
 // Visibility/beforeunload listeners are deferred — see _ensureModuleSideEffects()
 // One-time deferred side effects that were previously at module top-level.
@@ -1013,14 +1017,9 @@ function getMapUuidFromWindow() {
 // Note - we do NOT send serviceIdentifier because
 // it would be 1 serviceIdentifier per frontend user session,
 // which is very wasteful
-export async function startRecording({ apiKey, backendApi = "https://api-service.sailfishqa.com", domainsToPropagateHeaderTo = [], domainsToNotPropagateHeaderTo = [], serviceVersion, serviceIdentifier, gitSha, serviceAdditionalMetadata, enableIpTracking, captureStreamingResponseBody = true, captureResponseBodyMaxMb = 10, captureStreamPrefixKb = 64, captureStreamTimeoutMs = 10000, enableFiberTracking = false, deferRecordingStart = true, useWsWorker = true, }) {
-    // Initialize deferred module-level side effects (one-time)
-    _ensureModuleSideEffects();
-    const effectiveGitSha = gitSha ?? readGitSha();
-    const effectiveServiceIdentifier = serviceIdentifier ?? "";
-    const effectiveServiceVersion = serviceVersion ?? "";
-    const effectiveLibrary = "JS/TS"; // from Recorder.LIBRARY_CHOICES
-    const effectiveMapUuid = getMapUuidFromWindow();
+export async function startRecording({ apiKey, backendApi = "https://api-service.sailfishqa.com", domainsToPropagateHeaderTo = [], domainsToNotPropagateHeaderTo = [], serviceVersion, serviceIdentifier, gitSha, serviceAdditionalMetadata, enableIpTracking, captureStreamingResponseBody = true, captureResponseBodyMaxMb = 10, captureStreamPrefixKb = 64, captureStreamTimeoutMs = 10000, enableFiberTracking = false, deferRecording, deferRecordingStart, chunkSnapshot, useWsWorker = true, }) {
+    // deferRecording takes precedence; fall back to deprecated deferRecordingStart; default true
+    const effectiveDeferRecording = deferRecording ?? deferRecordingStart ?? true;
     const sessionId = getOrSetSessionId();
     const g = (window.__sailfish_recorder ||= {});
     g.sessionId = sessionId;
@@ -1035,32 +1034,60 @@ export async function startRecording({ apiKey, backendApi = "https://api-service
         trackDomainChangesOnce(); // ensure single watcher alive
         return;
     }
-    // One-time installs (per window)
+    // ── Phase 1: Critical interceptor installs ───────────────────────────────
+    // These interceptors MUST be installed before app code makes network
+    // requests or logs to the console. Yield between each setup to ensure
+    // no single task exceeds 50ms (especially on mobile with CPU throttle).
+    const bodyCaptureConfig = {
+        captureStreamingResponseBody,
+        captureResponseBodyMaxMb,
+        captureStreamPrefixKb,
+        captureStreamTimeoutMs,
+    };
+    // 1a. XHR interceptor
+    if (!g.xhrPatched) {
+        setupXMLHttpRequestInterceptor(domainsToNotPropagateHeaderTo, bodyCaptureConfig);
+        g.xhrPatched = true;
+    }
+    await yieldToMain();
+    // 1b. Fetch interceptor
+    if (!g.fetchPatched) {
+        setupFetchInterceptor(domainsToNotPropagateHeaderTo, bodyCaptureConfig);
+        g.fetchPatched = true;
+    }
+    await yieldToMain();
+    // 1c. DOM content events
     if (!g.domEventsInit) {
         initializeDomContentEvents(sessionId);
         g.domEventsInit = true;
     }
+    await yieldToMain();
+    // 1d. Console plugin (fire-and-forget dynamic import)
     if (!g.consoleInit) {
         initializeConsolePlugin(DEFAULT_CONSOLE_RECORDING_SETTINGS, sessionId);
         g.consoleInit = true;
     }
+    await yieldToMain();
+    // 1e. Error interceptor
     if (!g.errorInit) {
         initializeErrorInterceptor();
         g.errorInit = true;
     }
+    await yieldToMain();
+    // ── Phase 2: Deferred async work ────────────────────────────────────────
+    // Everything else — GraphQL calls, WebSocket init, device info, domain
+    // reporting, visibility listeners, funcspan state restore, etc.
+    // None of this needs to happen before app code runs.
+    // Deferred module-level side effects (funcspan restore, visibility listeners, etc.)
+    _ensureModuleSideEffects();
     storeCredentialsAndConnection({ apiKey, backendApi });
-    trackDomainChangesOnce();
-    sessionStorage.setItem("sailfishApiKey", apiKey);
-    sessionStorage.setItem("sailfishBackendApi", backendApi);
-    // Function span tracking state is loaded from localStorage on module init (websocket.tsx)
-    // Validate localStorage state with backend if tracking appears to be enabled but WebSocket not connected
+    // Function span tracking state validation (fire-and-forget)
     if (isFunctionSpanTrackingEnabled() && (!g.ws || g.ws.readyState !== 1)) {
         fetchFunctionSpanTrackingEnabled(apiKey, backendApi)
             .then((funcSpanResponse) => {
             const isEnabled = funcSpanResponse.data?.isFunctionSpanTrackingEnabledFromApiKey ??
                 false;
             if (!isEnabled) {
-                // Backend says tracking is NOT active, clear stale localStorage data
                 clearStaleFuncSpanState();
                 if (DEBUG) {
                     console.log("[Sailfish] Cleared stale function span tracking state - backend validation shows tracking is not active");
@@ -1076,7 +1103,6 @@ export async function startRecording({ apiKey, backendApi = "https://api-service
             if (DEBUG) {
                 console.warn("[Sailfish] Failed to validate function span tracking status with backend:", error);
             }
-            // On error, keep the localStorage state and let WebSocket correct it when it connects
         });
     }
     if (!g.sentDoNotPropagateOnce) {
@@ -1086,28 +1112,19 @@ export async function startRecording({ apiKey, backendApi = "https://api-service
         ], backendApi).catch((error) => console.error("Failed to send domains to not propagate header to:", error));
         g.sentDoNotPropagateOnce = true;
     }
-    // Network body capture config
-    const bodyCaptureConfig = {
-        captureStreamingResponseBody,
-        captureResponseBodyMaxMb,
-        captureStreamPrefixKb,
-        captureStreamTimeoutMs,
-    };
-    // Patch XHR/fetch once per window
-    if (!g.xhrPatched) {
-        setupXMLHttpRequestInterceptor(domainsToNotPropagateHeaderTo, bodyCaptureConfig);
-        g.xhrPatched = true;
-    }
-    if (!g.fetchPatched) {
-        setupFetchInterceptor(domainsToNotPropagateHeaderTo, bodyCaptureConfig);
-        g.fetchPatched = true;
-    }
     gatherAndCacheDeviceInfo();
     if (enableIpTracking) {
         fetchAndSendIp(sessionId);
     }
     try {
+        const effectiveGitSha = gitSha ?? readGitSha();
+        const effectiveServiceIdentifier = serviceIdentifier ?? "";
+        const effectiveServiceVersion = serviceVersion ?? "";
+        const effectiveLibrary = "JS/TS";
+        const effectiveMapUuid = getMapUuidFromWindow();
         const metadataWithAppUrl = withAppUrlMetadata(serviceAdditionalMetadata);
+        // Yield before GraphQL calls
+        await yieldToMain();
         // Fire both GraphQL calls in parallel — they are independent
         const [captureSettingsResponse, sessionResponse] = await Promise.all([
             fetchCaptureSettings(apiKey, backendApi),
@@ -1123,12 +1140,15 @@ export async function startRecording({ apiKey, backendApi = "https://api-service
             return;
         }
         if (sessionResponse.data?.startRecordingSession) {
-            // Extract env value from serviceAdditionalMetadata
             const envValue = serviceAdditionalMetadata?.env ||
                 serviceAdditionalMetadata?.environment;
-            const websocket = await initializeRecording(captureSettings, backendApi, apiKey, sessionId, envValue, deferRecordingStart, useWsWorker);
+            // Yield before WebSocket initialization
+            await yieldToMain();
+            const websocket = await initializeRecording(captureSettings, backendApi, apiKey, sessionId, envValue, effectiveDeferRecording, useWsWorker, chunkSnapshot ?? false);
             g.ws = websocket;
             g.initialized = true;
+            // Route watcher and mapUuid run after WS is connected
+            trackDomainChangesOnce();
             if (!g.sentMapUuidOnce) {
                 sendMapUuidIfAvailable(serviceIdentifier, serviceVersion);
                 g.sentMapUuidOnce = true;