fix(sync): send first update without initial document state

Process initial document state as incoming message.
This way it will be tracked as received from the remote server by y-websocket
and the update send to the server will exclude it.

Also use the sync services `opened` event inside WebsocketPolyfill
rather than mixing async `syncService.open().then` and event handling.
This allows us to emit `opened` and then `sync` from `syncService.open()`.

Signed-off-by: Max <max@nextcloud.com>

Author: max-nextcloud

src/services/SyncService.ts

@@ -268,10 +268,7 @@ class SyncService {
 				this.#outbox.clearSentData(sendable)
 				const { steps, documentState } = response.data
 				if (documentState) {
-					const documentStateStep = documentStateToStep(documentState)
-					this.bus.emit('sync', {
-						steps: [documentStateStep],
-					})
+					this._emitDocumentStateStep(documentState)
 				}
 				this.pushError = 0
 				this.sending = false

src/services/WebSocketPolyfill.ts

@@ -25,6 +25,7 @@ export default function initWebSocketPolyfill(syncService: SyncService, fileId:
 		onopen?: () => void
 		#notifyPushBus
 		#onSync
+		#onOpened
 		#processingVersion = 0
 
 		constructor(url: string) {
@@ -41,7 +42,6 @@ export default function initWebSocketPolyfill(syncService: SyncService, fileId:
 					})
 				}
 			}
-
 			syncService.bus.on('sync', this.#onSync)
 
 			syncService.open({ fileId, initialSession }).then((_data) => {