fix: delete shred slots when there are too many

Author: asuzuki-jumptrading

src/features/Overview/ShredsProgression/atoms.ts

@@ -118,36 +118,50 @@ export function createLiveShredsAtoms() {
 
           if (!prev || !slotRange) return;
 
-          const nowDelta = new Date().getTime() - prev.referenceTs;
-
-          let shouldDeleteSlot = false;
-          for (
-            let slotNumber = slotRange.max;
-            slotNumber >= slotRange.min;
-            slotNumber--
-          ) {
-            const slot = prev.slots[slotNumber];
-            if (!slot) continue;
-
-            if (
-              !shouldDeleteSlot &&
-              slot.completionTsDelta != null &&
-              nowDelta - slot.completionTsDelta > xRangeMs + delayMs
+          if (slotRange.max - slotRange.min > 50) {
+            // only keep 50 slots
+            for (
+              let slotNumber = slotRange.min;
+              slotNumber <= slotRange.max - 50;
+              slotNumber++
             ) {
-              // once we find a slot that is complete and far enough in the past, delete all slot numbers less it
-              shouldDeleteSlot = true;
-            }
+              const slot = prev.slots[slotNumber];
+              if (!slot) continue;
 
-            if (shouldDeleteSlot) {
               delete prev.slots[slotNumber];
             }
+          } else {
+            // delete slots outside visible range
+            const nowDelta = new Date().getTime() - prev.referenceTs;
+
+            let shouldDeleteSlot = false;
+            for (
+              let slotNumber = slotRange.max;
+              slotNumber >= slotRange.min;
+              slotNumber--
+            ) {
+              const slot = prev.slots[slotNumber];
+              if (!slot) continue;
+
+              if (
+                !shouldDeleteSlot &&
+                slot.completionTsDelta != null &&
+                nowDelta - slot.completionTsDelta > xRangeMs + delayMs
+              ) {
+                // once we find a slot that is complete and far enough in the past, delete all slot numbers less it
+                shouldDeleteSlot = true;
+              }
+
+              if (shouldDeleteSlot) {
+                delete prev.slots[slotNumber];
+              }
+            }
           }
 
+          // update range to reflect remaining slots
           const remainingSlotNumbers = Object.keys(prev.slots).map(
             (slotNumber) => parseInt(slotNumber),
           );
-
-          // update range to reflect remaining slots
           set(_slotRangeAtom, (prevRange) => {
             if (!prevRange || !remainingSlotNumbers.length) {
               return;