fix(sync): ensure queue stays the same array

Change the content of `queue` with `queue.splice`
rather than setting `queue` to another array.

Signed-off-by: Max <[email protected]>

src/services/WebSocketPolyfill.js

@@ -83,21 +83,20 @@ export default function initWebSocketPolyfill(syncService, fileId, initialSessio
 			// data.forEach(logStep)
 
 			queue.push(...data)
-			let outbox = []
+			let outbox
 			return syncService.sendSteps(() => {
-				outbox = [...queue]
 				const data = {
 					steps: this.#steps,
 					awareness: this.#awareness,
 					version: this.#version,
 				}
-				queue = []
+				outbox = queue.splice(0, queue.length)
 				logger.debug('sending steps ', data)
 				return data
 			})?.catch(err => {
 				logger.error(err)
-				// try to send the steps again
-				queue = [...outbox, ...queue]
+				// Prefix the queue with the steps in outbox to send them again
+				queue.splice(0, 0, ...outbox)
 			})
 		}
 
@@ -126,18 +125,18 @@ export default function initWebSocketPolyfill(syncService, fileId, initialSessio
 			if (queue.length) {
 				let outbox = []
 				return syncService.sendStepsNow(() => {
-					outbox = [...queue]
 					const data = {
 						steps: this.#steps,
 						awareness: this.#awareness,
 						version: this.#version,
 					}
-					queue = []
+					outbox = queue.splice(0, queue.length)
 					logger.debug('sending final steps ', data)
 					return data
 				})?.catch(err => {
 					logger.error(err)
-					queue = [...outbox, ...queue]
+					// Prefix the queue with the steps in outbox to send them again
+					queue.splice(0, 0, ...outbox)
 				})
 			}
 		}