@devskin/browser-sdk 1.0.2 → 1.0.4

package/dist/devskin.esm.js

@@ -637,11 +637,16 @@ class ErrorCollector {
             console[level] = (...args) => {
                 // Call original
                 original.apply(console, args);
-                // Add breadcrumb
+                // Add breadcrumb (skip DevSkin internal logs to avoid infinite loop)
                 if (level === 'warn' || level === 'error') {
+                    const message = args.map((arg) => String(arg)).join(' ');
+                    // Skip DevSkin internal messages
+                    if (message.startsWith('[DevSkin]')) {
+                        return;
+                    }
                     this.addBreadcrumb({
                         category: 'console',
-                        message: args.map((arg) => String(arg)).join(' '),
+                        message: message,
                         level: level === 'warn' ? 'warning' : 'error',
                         data: { arguments: args },
                     });
@@ -4978,6 +4983,7 @@ class RRWebRecorder {
         this.events = [];
         this.onEventsReady = null;
         this.flushInterval = null;
+        this.hasFullSnapshot = false;
         this.sessionId = sessionId;
         this.config = config;
         this.onEventsReady = onEventsReady;
@@ -4995,9 +5001,15 @@ class RRWebRecorder {
             console.log('[RRWeb] Starting session recording:', this.sessionId);
             this.stopFn = record({
                 emit: (event) => {
+                    // Check if this is a FullSnapshot (type 2)
+                    if (event.type === 2) {
+                        this.hasFullSnapshot = true;
+                        console.log('[RRWeb] FullSnapshot captured! Recording is ready.');
+                    }
                     this.events.push(event);
-                    // Flush events periodically to avoid memory buildup
-                    if (this.events.length >= 50) {
+                    // Only flush if we have the FullSnapshot
+                    // This ensures the first batch always contains the full DOM
+                    if (this.hasFullSnapshot && this.events.length >= 50) {
                         this.flush();
                     }
                 },
@@ -5037,8 +5049,9 @@ class RRWebRecorder {
                 recordCrossOriginIframes: false, // Security: don't record cross-origin iframes
             });
             // Set up periodic flush (every 10 seconds)
+            // Only flush if we have FullSnapshot to ensure first batch is complete
             this.flushInterval = window.setInterval(() => {
-                if (this.events.length > 0) {
+                if (this.hasFullSnapshot && this.events.length > 0) {
                     this.flush();
                 }
             }, 10000);
@@ -5058,8 +5071,11 @@ class RRWebRecorder {
             clearInterval(this.flushInterval);
             this.flushInterval = null;
         }
-        // Flush remaining events
+        // Flush remaining events (even without FullSnapshot, to not lose data)
         if (this.events.length > 0) {
+            if (!this.hasFullSnapshot) {
+                console.warn('[RRWeb] Flushing events without FullSnapshot - recording may not replay correctly');
+            }
             this.flush();
         }
     }
@@ -5124,6 +5140,13 @@ class Transport {
         this.enqueue('performance', metric);
     }
     sendRecordingEvents(sessionId, events) {
+        // DEBUG: Log event types being sent
+        const eventTypes = events.reduce((acc, e) => {
+            acc[e.type] = (acc[e.type] || 0) + 1;
+            return acc;
+        }, {});
+        console.log(`[DevSkin SDK] Sending ${events.length} recording events:`, eventTypes);
+        console.log(`[DevSkin SDK] First 3 events:`, events.slice(0, 3).map(e => ({ type: e.type, timestamp: e.timestamp })));
         // Recording events can be large, send immediately
         this.sendToBackend('/v1/rum/recordings', {
             session_id: sessionId,