fix(sync): Apply document state from create request

Prevents possible race condition when steps from first sync request got
applied before documentState from first push request (answer to
SyncStep1) arrived.

Signed-off-by: Jonas <jonas@freesources.org>

Author: mejo-

src/components/Editor.vue

@@ -96,7 +96,7 @@ import {
 import ReadonlyBar from './Menu/ReadonlyBar.vue'
 
 import { logger } from '../helpers/logger.js'
-import { getDocumentState } from '../helpers/yjs.ts'
+import { applyDocumentState, getDocumentState } from '../helpers/yjs.ts'
 import { SyncService, ERROR_TYPE, IDLE_TIMEOUT } from './../services/SyncService.ts'
 import SessionApi from '../services/SessionApi.js'
 import createSyncServiceProvider from './../services/SyncServiceProvider.js'
@@ -556,7 +556,9 @@ export default {
 		},
 
 		onLoaded({ document, documentSource, documentState }) {
-			if (!documentState) {
+			if (documentState) {
+				applyDocumentState(this.ydoc, documentState, this.syncProvider)
+			} else {
 				this.setInitialYjsState(documentSource, { isRichEditor: this.isRichEditor })
 			}
 

src/services/y-websocket.js

@@ -224,11 +224,6 @@ const setupWS = (provider) => {
 			provider.emit('status', [{
 				status: 'connected',
 			}])
-			// always send sync step 1 when connected
-			const encoder = encoding.createEncoder()
-			encoding.writeVarUint(encoder, messageSync)
-			syncProtocol.writeSyncStep1(encoder, provider.doc)
-			websocket.send(encoding.toUint8Array(encoder))
 			// broadcast local awareness state
 			if (provider.awareness.getLocalState() !== null) {
 				const encoderAwarenessState = encoding.createEncoder()