fix(nbstore): check before save empty update (#10304)

packages/common/nbstore/src/sync/doc/peer.ts

@@ -437,6 +437,7 @@ export class DocSyncPeer {
           data.length > 0 ? await this.mergeUpdates(data) : new Uint8Array();
 
         throwIfAborted(signal);
+        if (!isEmptyUpdate(update)) {
           await this.local.pushDocUpdate(
             {
               docId,
@@ -444,6 +445,7 @@ export class DocSyncPeer {
             },
             this.uniqueId
           );
+        }
         throwIfAborted(signal);
 
         await this.syncMetadata.setPeerPulledRemoteClock(this.peerId, {

packages/frontend/core/src/modules/userspace/entities/user-db-engine.ts

@@ -1,5 +1,5 @@
-import { IndexedDBDocStorage } from '@affine/nbstore/idb';
+import { IndexedDBDocStorage, IndexedDBSyncStorage } from '@affine/nbstore/idb';
-import { SqliteDocStorage } from '@affine/nbstore/sqlite';
+import { SqliteDocStorage, SqliteSyncStorage } from '@affine/nbstore/sqlite';
 import type { StoreClient } from '@affine/nbstore/worker/client';
 import { Entity } from '@toeverything/infra';
 
@@ -16,6 +16,10 @@ export class UserDBEngine extends Entity<{
     BUILD_CONFIG.isElectron || BUILD_CONFIG.isIOS
       ? SqliteDocStorage
       : IndexedDBDocStorage;
+  SyncStorageType =
+    BUILD_CONFIG.isElectron || BUILD_CONFIG.isIOS
+      ? SqliteSyncStorage
+      : IndexedDBSyncStorage;
 
   canGracefulStop() {
     // TODO(@eyhn): Implement this
@@ -40,6 +44,14 @@ export class UserDBEngine extends Entity<{
               type: 'userspace',
             },
           },
+          sync: {
+            name: this.SyncStorageType.identifier,
+            opts: {
+              id: `${serverService.server.id}:` + this.userId,
+              type: 'userspace',
+              flavour: serverService.server.id,
+            },
+          },
         },
         remotes: {
           cloud: {