cryptique-sdk 1.2.4 → 1.2.6

package/lib/cjs/index.js

@@ -4302,31 +4302,6 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
 
   };
 
-  // ============================================================================
-  // SECTION 13: API COMMUNICATION
-  // ============================================================================
-  // PURPOSE: Unified API client for all backend communication. Handles sending
-  //          session data, custom events, and UTM events.
-  //
-  // KEY PRINCIPLES:
-  // 1. Unified API client for all endpoints
-  // 2. Transforms data using DataTransformer before sending
-  // 3. Handles retries and errors gracefully
-  // 4. Uses sendBeacon for beforeunload (last resort)
-  // 5. Uses fetch with keepalive for regular sends
-  // 6. Comprehensive error handling and logging
-  //
-  // BENEFITS:
-  // - Single place to handle all API communication
-  // - Consistent error handling
-  // - Retry logic for reliability
-  // - Proper use of sendBeacon vs fetch
-  // ============================================================================
-
-  // Throttle network error logs (e.g. "Failed to fetch" from ad blockers) - log at most once per 30s
-  let _lastNetworkErrorLog = 0;
-  const _NETWORK_ERROR_THROTTLE_MS = 30000;
-
   /**
    * APIClient - Unified API communication
    * 
@@ -4502,17 +4477,24 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
         }
         
         // Handle AbortError gracefully (timeout or page unload)
-        if (error.name === 'AbortError' || error.message?.includes('aborted') || error.message?.includes('signal is aborted')) {
+        if (
+          error.name === 'AbortError' ||
+          error.message?.includes('aborted') ||
+          error.message?.includes('signal is aborted')
+        ) {
           // Don't log as error - this is expected behavior for timeouts or page unloads
           // Don't retry aborted requests - they were intentionally cancelled
           return; // Silently return, don't throw
         }
 
-        // Throttle "Failed to fetch" logs (common with ad blockers, CORS, network issues)
-        const isNetworkError = error.name === 'TypeError' && (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
-        const now = Date.now();
-        if (isNetworkError && (now - _lastNetworkErrorLog) < _NETWORK_ERROR_THROTTLE_MS) ; else {
-          if (isNetworkError) _lastNetworkErrorLog = now;
+        // Treat common network issues (ad blockers, CORS, offline, etc.) as silent failures.
+        // We still retry according to the "retries" option, but we do not log these to the console.
+        const isNetworkError =
+          error.name === 'TypeError' &&
+          (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
+
+        // Only log unexpected / non‑network errors
+        if (!isNetworkError) {
           console.error('❌ Error sending data:', error);
         }
         
@@ -4522,7 +4504,11 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
           return this.send(endpoint, data, { ...options, retries: retries - 1 });
         }
         
-        throw error;
+        // For non‑network errors that made it this far, surface to callers
+        if (!isNetworkError) {
+          throw error;
+        }
+        // For network errors, fail silently after retries are exhausted
       }
     },
 
@@ -5339,57 +5325,16 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
     },
 
     /**
-     * NetworkTracker - Tracks network errors (fetch and XMLHttpRequest)
+     * NetworkTracker - Tracks network errors (XHR only)
+     *
+     * NOTE:
+     * We intentionally do NOT wrap window.fetch here, to avoid confusing
+     * browser DevTools attribution for third‑party requests. All SDK
+     * fetch-based calls to backend.cryptique.io are already error-handled
+     * inside APIClient.send(), so fetch wrapping is redundant.
      */
     startNetworkTracking() {
       try {
-        // Track fetch errors - only intercept calls to our own backend
-        // Use _nativeFetch as the base so the wrapper always delegates to the
-        // true browser fetch, regardless of what other libraries have patched.
-        window.fetch = function(...args) {
-          const startTime = Date.now();
-          const url = typeof args[0] === 'string' ? args[0] : (args[0]?.url || '');
-
-          // Pass third-party requests through completely untouched — no interception,
-          // no involvement of our SDK in their call stack or error handling
-          if (!url.includes('backend.cryptique.io')) {
-            return _nativeFetch.apply(this, args);
-          }
-
-          return _nativeFetch.apply(this, args).then(response => {
-            // Only track error status codes (4xx, 5xx)
-            if (response.status >= 400) {
-              const networkErrorData = {
-                type: 'api_error',
-                url: url,
-                method: 'GET',
-                status: response.status,
-                error: `HTTP ${response.status} Error`,
-                duration: Date.now() - startTime,
-                path: window.location.pathname
-              };
-              
-              InteractionManager.add('networkEvents', networkErrorData);
-            }
-            
-            return response;
-          }).catch(error => {
-            // Track actual network errors
-            const networkErrorData = {
-              type: 'api_error',
-              url: url,
-              method: 'GET',
-              status: 0,
-              error: error.message || 'Network Error',
-              duration: Date.now() - startTime,
-              path: window.location.pathname
-            };
-            
-            InteractionManager.add('networkEvents', networkErrorData);
-            throw error;
-          });
-        };
-
         // Track XMLHttpRequest errors - only intercept calls to our own backend
         const originalXHROpen = XMLHttpRequest.prototype.open;
         const originalXHRSend = XMLHttpRequest.prototype.send;

package/lib/esm/index.js

@@ -4300,31 +4300,6 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
 
   };
 
-  // ============================================================================
-  // SECTION 13: API COMMUNICATION
-  // ============================================================================
-  // PURPOSE: Unified API client for all backend communication. Handles sending
-  //          session data, custom events, and UTM events.
-  //
-  // KEY PRINCIPLES:
-  // 1. Unified API client for all endpoints
-  // 2. Transforms data using DataTransformer before sending
-  // 3. Handles retries and errors gracefully
-  // 4. Uses sendBeacon for beforeunload (last resort)
-  // 5. Uses fetch with keepalive for regular sends
-  // 6. Comprehensive error handling and logging
-  //
-  // BENEFITS:
-  // - Single place to handle all API communication
-  // - Consistent error handling
-  // - Retry logic for reliability
-  // - Proper use of sendBeacon vs fetch
-  // ============================================================================
-
-  // Throttle network error logs (e.g. "Failed to fetch" from ad blockers) - log at most once per 30s
-  let _lastNetworkErrorLog = 0;
-  const _NETWORK_ERROR_THROTTLE_MS = 30000;
-
   /**
    * APIClient - Unified API communication
    * 
@@ -4500,17 +4475,24 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
         }
         
         // Handle AbortError gracefully (timeout or page unload)
-        if (error.name === 'AbortError' || error.message?.includes('aborted') || error.message?.includes('signal is aborted')) {
+        if (
+          error.name === 'AbortError' ||
+          error.message?.includes('aborted') ||
+          error.message?.includes('signal is aborted')
+        ) {
           // Don't log as error - this is expected behavior for timeouts or page unloads
           // Don't retry aborted requests - they were intentionally cancelled
           return; // Silently return, don't throw
         }
 
-        // Throttle "Failed to fetch" logs (common with ad blockers, CORS, network issues)
-        const isNetworkError = error.name === 'TypeError' && (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
-        const now = Date.now();
-        if (isNetworkError && (now - _lastNetworkErrorLog) < _NETWORK_ERROR_THROTTLE_MS) ; else {
-          if (isNetworkError) _lastNetworkErrorLog = now;
+        // Treat common network issues (ad blockers, CORS, offline, etc.) as silent failures.
+        // We still retry according to the "retries" option, but we do not log these to the console.
+        const isNetworkError =
+          error.name === 'TypeError' &&
+          (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
+
+        // Only log unexpected / non‑network errors
+        if (!isNetworkError) {
           console.error('❌ Error sending data:', error);
         }
         
@@ -4520,7 +4502,11 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
           return this.send(endpoint, data, { ...options, retries: retries - 1 });
         }
         
-        throw error;
+        // For non‑network errors that made it this far, surface to callers
+        if (!isNetworkError) {
+          throw error;
+        }
+        // For network errors, fail silently after retries are exhausted
       }
     },
 
@@ -5337,57 +5323,16 @@ if (window.Cryptique && window.Cryptique.initialized) ; else {
     },
 
     /**
-     * NetworkTracker - Tracks network errors (fetch and XMLHttpRequest)
+     * NetworkTracker - Tracks network errors (XHR only)
+     *
+     * NOTE:
+     * We intentionally do NOT wrap window.fetch here, to avoid confusing
+     * browser DevTools attribution for third‑party requests. All SDK
+     * fetch-based calls to backend.cryptique.io are already error-handled
+     * inside APIClient.send(), so fetch wrapping is redundant.
      */
     startNetworkTracking() {
       try {
-        // Track fetch errors - only intercept calls to our own backend
-        // Use _nativeFetch as the base so the wrapper always delegates to the
-        // true browser fetch, regardless of what other libraries have patched.
-        window.fetch = function(...args) {
-          const startTime = Date.now();
-          const url = typeof args[0] === 'string' ? args[0] : (args[0]?.url || '');
-
-          // Pass third-party requests through completely untouched — no interception,
-          // no involvement of our SDK in their call stack or error handling
-          if (!url.includes('backend.cryptique.io')) {
-            return _nativeFetch.apply(this, args);
-          }
-
-          return _nativeFetch.apply(this, args).then(response => {
-            // Only track error status codes (4xx, 5xx)
-            if (response.status >= 400) {
-              const networkErrorData = {
-                type: 'api_error',
-                url: url,
-                method: 'GET',
-                status: response.status,
-                error: `HTTP ${response.status} Error`,
-                duration: Date.now() - startTime,
-                path: window.location.pathname
-              };
-              
-              InteractionManager.add('networkEvents', networkErrorData);
-            }
-            
-            return response;
-          }).catch(error => {
-            // Track actual network errors
-            const networkErrorData = {
-              type: 'api_error',
-              url: url,
-              method: 'GET',
-              status: 0,
-              error: error.message || 'Network Error',
-              duration: Date.now() - startTime,
-              path: window.location.pathname
-            };
-            
-            InteractionManager.add('networkEvents', networkErrorData);
-            throw error;
-          });
-        };
-
         // Track XMLHttpRequest errors - only intercept calls to our own backend
         const originalXHROpen = XMLHttpRequest.prototype.open;
         const originalXHRSend = XMLHttpRequest.prototype.send;

package/lib/umd/index.js

@@ -4306,31 +4306,6 @@
 
     };
 
-    // ============================================================================
-    // SECTION 13: API COMMUNICATION
-    // ============================================================================
-    // PURPOSE: Unified API client for all backend communication. Handles sending
-    //          session data, custom events, and UTM events.
-    //
-    // KEY PRINCIPLES:
-    // 1. Unified API client for all endpoints
-    // 2. Transforms data using DataTransformer before sending
-    // 3. Handles retries and errors gracefully
-    // 4. Uses sendBeacon for beforeunload (last resort)
-    // 5. Uses fetch with keepalive for regular sends
-    // 6. Comprehensive error handling and logging
-    //
-    // BENEFITS:
-    // - Single place to handle all API communication
-    // - Consistent error handling
-    // - Retry logic for reliability
-    // - Proper use of sendBeacon vs fetch
-    // ============================================================================
-
-    // Throttle network error logs (e.g. "Failed to fetch" from ad blockers) - log at most once per 30s
-    let _lastNetworkErrorLog = 0;
-    const _NETWORK_ERROR_THROTTLE_MS = 30000;
-
     /**
      * APIClient - Unified API communication
      * 
@@ -4506,17 +4481,24 @@
           }
           
           // Handle AbortError gracefully (timeout or page unload)
-          if (error.name === 'AbortError' || error.message?.includes('aborted') || error.message?.includes('signal is aborted')) {
+          if (
+            error.name === 'AbortError' ||
+            error.message?.includes('aborted') ||
+            error.message?.includes('signal is aborted')
+          ) {
             // Don't log as error - this is expected behavior for timeouts or page unloads
             // Don't retry aborted requests - they were intentionally cancelled
             return; // Silently return, don't throw
           }
 
-          // Throttle "Failed to fetch" logs (common with ad blockers, CORS, network issues)
-          const isNetworkError = error.name === 'TypeError' && (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
-          const now = Date.now();
-          if (isNetworkError && (now - _lastNetworkErrorLog) < _NETWORK_ERROR_THROTTLE_MS) ; else {
-            if (isNetworkError) _lastNetworkErrorLog = now;
+          // Treat common network issues (ad blockers, CORS, offline, etc.) as silent failures.
+          // We still retry according to the "retries" option, but we do not log these to the console.
+          const isNetworkError =
+            error.name === 'TypeError' &&
+            (error.message === 'Failed to fetch' || error.message?.includes('fetch'));
+
+          // Only log unexpected / non‑network errors
+          if (!isNetworkError) {
             console.error('❌ Error sending data:', error);
           }
           
@@ -4526,7 +4508,11 @@
             return this.send(endpoint, data, { ...options, retries: retries - 1 });
           }
           
-          throw error;
+          // For non‑network errors that made it this far, surface to callers
+          if (!isNetworkError) {
+            throw error;
+          }
+          // For network errors, fail silently after retries are exhausted
         }
       },
 
@@ -5343,57 +5329,16 @@
       },
 
       /**
-       * NetworkTracker - Tracks network errors (fetch and XMLHttpRequest)
+       * NetworkTracker - Tracks network errors (XHR only)
+       *
+       * NOTE:
+       * We intentionally do NOT wrap window.fetch here, to avoid confusing
+       * browser DevTools attribution for third‑party requests. All SDK
+       * fetch-based calls to backend.cryptique.io are already error-handled
+       * inside APIClient.send(), so fetch wrapping is redundant.
        */
       startNetworkTracking() {
         try {
-          // Track fetch errors - only intercept calls to our own backend
-          // Use _nativeFetch as the base so the wrapper always delegates to the
-          // true browser fetch, regardless of what other libraries have patched.
-          window.fetch = function(...args) {
-            const startTime = Date.now();
-            const url = typeof args[0] === 'string' ? args[0] : (args[0]?.url || '');
-
-            // Pass third-party requests through completely untouched — no interception,
-            // no involvement of our SDK in their call stack or error handling
-            if (!url.includes('backend.cryptique.io')) {
-              return _nativeFetch.apply(this, args);
-            }
-
-            return _nativeFetch.apply(this, args).then(response => {
-              // Only track error status codes (4xx, 5xx)
-              if (response.status >= 400) {
-                const networkErrorData = {
-                  type: 'api_error',
-                  url: url,
-                  method: 'GET',
-                  status: response.status,
-                  error: `HTTP ${response.status} Error`,
-                  duration: Date.now() - startTime,
-                  path: window.location.pathname
-                };
-                
-                InteractionManager.add('networkEvents', networkErrorData);
-              }
-              
-              return response;
-            }).catch(error => {
-              // Track actual network errors
-              const networkErrorData = {
-                type: 'api_error',
-                url: url,
-                method: 'GET',
-                status: 0,
-                error: error.message || 'Network Error',
-                duration: Date.now() - startTime,
-                path: window.location.pathname
-              };
-              
-              InteractionManager.add('networkEvents', networkErrorData);
-              throw error;
-            });
-          };
-
           // Track XMLHttpRequest errors - only intercept calls to our own backend
           const originalXHROpen = XMLHttpRequest.prototype.open;
           const originalXHRSend = XMLHttpRequest.prototype.send;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cryptique-sdk",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "type": "module",
   "description": "Cryptique Analytics SDK - Comprehensive web analytics and user tracking for modern web applications",
   "main": "lib/cjs/index.js",