@cuboapp/crdt 1.0.16 → 1.0.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cuboapp/crdt",
-  "version": "1.0.16",
+  "version": "1.0.17",
   "description": "CRDT for CUBO",
   "main": "src/index.ts",
   "repository": {

package/src/server/index.ts

@@ -597,6 +597,21 @@ export class CuboCrdtServer<M, A = {}, E extends Extract<keyof M, string> = Extr
 
     let document = this.documents.get(documentName)
     if (document) {
+      // Освежаем существующий документ строкой из БД.
+      //
+      // Раньше документ возвращался как есть, а подписка отправляла клиенту его
+      // stateAsUpdate — то есть КЭШ документа, а не только что прочитанную строку.
+      // Документ, проспавший мутацию, оставался устаревшим навсегда: даже перезагрузка
+      // страницы отдавала старое значение, потому что reconcile брал его же.
+      //
+      // Стало заметно после того, как документы без подписчиков начали удаляться:
+      // документ может быть создан, остаться без подписок, пропустить мутации и
+      // «воскреснуть» на новой подписке уже неактуальным.
+      //
+      // store: false — это не правка от клиента, а синхронизация с БД, писать обратно нечего.
+      document.write(row, { expose: 'all', store: false })
+      document.setPersistedState(row)
+
       return document
     }
 
@@ -650,11 +665,12 @@ export class CuboCrdtServer<M, A = {}, E extends Extract<keyof M, string> = Extr
 
     if (!existed) {
       this.pushDocumentAction('create', entity, document, document.stateAsUpdate)
-    } else {
-      document.write(row, { expose: 'all', store: false })
-      document.setPersistedState(row)
     }
 
+    // Для уже существующего документа освежать состояние здесь не нужно —
+    // это делает getOrCreateDocument, одинаково для всех путей (рассылка и подписка).
+    // write() сам пропускает неизменившиеся ключи, так что лишних update-эвентов нет.
+
     if (!this.subscribesByDocument.get(documentName)?.size) {
       this.checkDocumentNeedRemove(documentName)
     }