@colyseus/schema 3.0.19 → 3.0.20

package/build/cjs/index.js

@@ -3313,7 +3313,7 @@ class CollectionSchema {
     static [(_a$1 = $encoder, _b$1 = $decoder, $filter)](ref, index, view) {
         return (!view ||
             typeof (ref[$childType]) === "string" ||
-            view.items.has(ref[$getByIndex](index)[$changes]));
+            view.items.has((ref[$getByIndex](index) ?? ref.deletedItems[index])[$changes]));
     }
     static is(type) {
         return type['collection'] !== undefined;
@@ -3321,6 +3321,7 @@ class CollectionSchema {
     constructor(initialValues) {
         this.$items = new Map();
         this.$indexes = new Map();
+        this.deletedItems = {};
         this.$refId = 0;
         this[$changes] = new ChangeTree(this);
         this[$changes].indexes = {};
@@ -3370,7 +3371,7 @@ class CollectionSchema {
         if (index === undefined) {
             return false;
         }
-        this[$changes].delete(index);
+        this.deletedItems[index] = this[$changes].delete(index);
         this.$indexes.delete(index);
         return this.$items.delete(index);
     }
@@ -3415,6 +3416,9 @@ class CollectionSchema {
         this.$items.delete(key);
         this.$indexes.delete(index);
     }
+    [$onEncodeEnd]() {
+        this.deletedItems = {};
+    }
     toArray() {
         return Array.from(this.$items.values());
     }
@@ -3469,7 +3473,7 @@ class SetSchema {
     static [(_a = $encoder, _b = $decoder, $filter)](ref, index, view) {
         return (!view ||
             typeof (ref[$childType]) === "string" ||
-            view.items.has(ref[$getByIndex](index)[$changes]));
+            view.items.has((ref[$getByIndex](index) ?? ref.deletedItems[index])[$changes]));
     }
     static is(type) {
         return type['set'] !== undefined;
@@ -3477,6 +3481,7 @@ class SetSchema {
     constructor(initialValues) {
         this.$items = new Map();
         this.$indexes = new Map();
+        this.deletedItems = {};
         this.$refId = 0;
         this[$changes] = new ChangeTree(this);
         this[$changes].indexes = {};
@@ -3526,7 +3531,7 @@ class SetSchema {
         if (index === undefined) {
             return false;
         }
-        this[$changes].delete(index);
+        this.deletedItems[index] = this[$changes].delete(index);
         this.$indexes.delete(index);
         return this.$items.delete(index);
     }
@@ -3583,6 +3588,9 @@ class SetSchema {
         this.$items.delete(key);
         this.$indexes.delete(index);
     }
+    [$onEncodeEnd]() {
+        this.deletedItems = {};
+    }
     toArray() {
         return Array.from(this.$items.values());
     }
@@ -3787,7 +3795,7 @@ class Encoder {
                     view.invisible.add(changeTree);
                     continue; // skip this change tree
                 }
-                else if (view.invisible.has(changeTree)) {
+                else {
                     view.invisible.delete(changeTree); // remove from invisible list
                 }
             }
@@ -4743,10 +4751,11 @@ class StateView {
                 } // skip "undefined" indexes
                 const op = changeTree.indexedOperations[index] ?? exports.OPERATION.ADD;
                 const tagAtIndex = metadata?.[index].tag;
-                if ((isInvisible || // if "invisible", include all
-                    tagAtIndex === undefined || // "all change" with no tag
-                    tagAtIndex === tag // tagged property
-                ) &&
+                if (!changeTree.isNew && // new structures will be added as part of .encode() call, no need to force it to .encodeView()
+                    (isInvisible || // if "invisible", include all
+                        tagAtIndex === undefined || // "all change" with no tag
+                        tagAtIndex === tag // tagged property
+                    ) &&
                     op !== exports.OPERATION.DELETE) {
                     changes[index] = op;
                 }
@@ -4775,10 +4784,8 @@ class StateView {
             if (parentChangeTree && (parentChangeTree.filteredChanges !== undefined)) {
                 this.addParentOf(changeTree, tag);
             }
-            // parent is already available, no need to add it!
-            if (!this.invisible.has(changeTree)) {
-                return;
-            }
+            // // parent is already available, no need to add it!
+            // if (!this.invisible.has(changeTree)) { return; }
         }
         // add parent's tag properties
         if (changeTree.getChange(parentIndex) !== exports.OPERATION.DELETE) {