Address PR feedback

Author: VeskeR

src/plugins/liveobjects/syncobjectspool.ts

@@ -48,6 +48,18 @@ export class SyncObjectsPool {
       }
 
       const objectState = objectMessage.object;
+
+      if (!objectState.counter && !objectState.map) {
+        // RTO5f3
+        this._client.Logger.logAction(
+          this._client.logger,
+          this._client.Logger.LOG_MAJOR,
+          'SyncObjectsPool.applyObjectSyncMessages()',
+          `received OBJECT_SYNC message with unsupported object type, expected 'counter' or 'map' to be present, skipping message; message id: ${objectMessage.id}, channel: ${this._channel.name}`,
+        );
+        continue;
+      }
+
       const objectId = objectState.objectId;
       const existingEntry = this._pool.get(objectId);
 
@@ -75,14 +87,6 @@ export class SyncObjectsPool {
         this._mergeMapSyncState(existingEntry, objectMessage);
         continue;
       }
-
-      // RTO5f2c
-      this._client.Logger.logAction(
-        this._client.logger,
-        this._client.Logger.LOG_MAJOR,
-        'SyncObjectsPool.applyObjectSyncMessages()',
-        `received OBJECT_SYNC message with unsupported object type, expected 'counter' or 'map' to be present, skipping message; message id: ${objectMessage.id}, channel: ${this._channel.name}`,
-      );
     }
   }
 
@@ -107,10 +111,8 @@ export class SyncObjectsPool {
       existingObjectState.map!.entries = {};
     }
 
-    if (newObjectState.map?.entries) {
-      // RTO5f2a2 - during partial sync, no two messages contain the same map key,
-      // so entries can be merged directly without conflict checking.
-      Object.assign(existingObjectState.map!.entries, newObjectState.map.entries);
-    }
+    // RTO5f2a2 - during partial sync, no two messages contain the same map key,
+    // so entries can be merged directly without conflict checking.
+    Object.assign(existingObjectState.map!.entries, newObjectState.map!.entries);
   }
 }