@glimt/record 0.0.33 → 0.0.35

package/dist/record.js

@@ -9622,44 +9622,45 @@ class IframeManager {
     }
   }
 }
-function makeid$1(length = 8) {
-  var result2 = "";
-  var characters = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
-  var charactersLength = characters.length;
-  for (var i2 = 0; i2 < length; i2++) {
-    result2 += characters.charAt(Math.floor(Math.random() * charactersLength));
-  }
-  return result2;
-}
 const _StormSnapshotManager = class _StormSnapshotManager2 {
   constructor() {
-    __publicField(this, "id", makeid$1());
     __publicField(this, "lastFullSnapshot", -1);
     __publicField(this, "intervalBetweenSnapshots", 150);
+    __publicField(this, "debounceTimeout", null);
+    __publicField(this, "debounceTime", 10);
     if (_StormSnapshotManager2.instance) {
       return _StormSnapshotManager2.instance;
     }
     _StormSnapshotManager2.instance = this;
   }
-  requestFullSnapshot(bufferId) {
-    if (!takeFullSnapshot) return;
+  //we're debouncing here because of how mutation buffers work.
+  //multiple observers create their own mutation buffer, and
+  //each buffer will handle mutations storms, so multiple buffers
+  //will probably request a full snapshot at (basically) the same time.
+  //we want to ensure all buffers have requested a full snapshot
+  //(so we can be sure that all mutations have been made)
+  //before we actually take a full snapshot.
+  //also, we want a low debounceTime, bc if theres multiple, distinctive mutation storms,
+  //in a somewhat quick succession, we want to record activity between them
+  //not just one full snapshot after all the storms:
+  //[mutation storm] [full snapshot] [mutation storm] [full snapshot]
+  //NOT
+  //[mutation storm] [timeout] [mutation storm] [full snapshot] (in this case, we probably miss all events which fired during the timeout)
+  requestFullSnapshot() {
+    if (this.debounceTimeout) {
+      clearTimeout(this.debounceTimeout);
+    }
+    this.debounceTimeout = setTimeout(() => {
+      this.debounceTimeout = null;
+      this.takeSnapshot();
+    }, this.debounceTime);
+  }
+  takeSnapshot() {
     if (Date.now() - this.lastFullSnapshot < this.intervalBetweenSnapshots) {
-      console.log(
-        "requestFullSnapshot: too soon",
-        "storm snapshot mng id:",
-        this.id,
-        "bufferId:",
-        bufferId
-      );
+      console.log("StormSnapshotManager, takeSnapshot: too soon");
       return;
     }
-    console.log(
-      "taking full snapshot",
-      "storm snapshot mng id:",
-      this.id,
-      "bufferId:",
-      bufferId
-    );
+    console.log("StormSnapshotManager, takeSnapshot: taking full snapshot");
     takeFullSnapshot();
     this.lastFullSnapshot = Date.now();
   }
@@ -9795,11 +9796,14 @@ class MutationBuffer {
     __publicField(this, "unattachedDoc");
     __publicField(this, "bufId", makeid());
     __publicField(this, "stormBatches", []);
-    __publicField(this, "stormInfo");
+    __publicField(this, "stormInfo", null);
     __publicField(this, "stormSettings", {
-      batchSize: 300,
-      timeout: 50,
-      mutationLimit: 1500
+      batchSize: 150,
+      //was 300
+      timeout: 30,
+      //was 50
+      mutationLimit: 800
+      //was 1500
     });
     __publicField(this, "handleStormMutations", (muts) => {
       const time = Date.now();
@@ -9817,12 +9821,6 @@ class MutationBuffer {
         };
       }
       this.stormInfo.totalMutations += muts.length;
-      console.log(
-        "current storm mutations",
-        this.stormInfo.totalMutations,
-        "buffer id:",
-        this.bufId
-      );
       if (this.stormInfo.totalMutations >= this.stormSettings.mutationLimit) {
         this.stormInfo.stormExceededLimit = true;
         this.stormBatches = [];
@@ -9832,9 +9830,14 @@ class MutationBuffer {
           mutations: muts
         });
       }
+      clearTimeout(this.stormInfo.timeout);
       if (muts.length < this.stormSettings.batchSize) {
-        clearTimeout(this.stormInfo.timeout);
         this.handleStormFinish();
+      } else {
+        this.stormInfo.timeout = setTimeout(
+          this.handleStormFinish,
+          this.stormSettings.timeout
+        );
       }
     });
     __publicField(this, "handleStormFinish", () => {
@@ -9863,7 +9866,7 @@ class MutationBuffer {
         this.processInternalMutations(muts, true);
       } else {
         this.stormBatches = [];
-        stormSnapshotManager.requestFullSnapshot(this.bufId);
+        stormSnapshotManager.requestFullSnapshot();
       }
     });
     __publicField(this, "processInternalMutations", (muts, overrideStorm = false) => {
@@ -9871,20 +9874,9 @@ class MutationBuffer {
         this.handleStormMutations(muts);
         return;
       }
-      const start = performance.now();
       for (const mut of muts) {
         this.processMutation(mut);
       }
-      console.log(
-        muts.length,
-        "mutations processed in",
-        performance.now() - start,
-        "ms",
-        "overrideStorm",
-        overrideStorm,
-        "buffer id:",
-        this.bufId
-      );
       this.emit();
     });
     __publicField(this, "processMutations", (mutations) => {