@colyseus/schema 3.0.44 → 3.0.45

package/build/cjs/index.js

@@ -2010,7 +2010,7 @@ class ArraySchema {
                             }
                             if (previousValue !== undefined) {
                                 // remove root reference from previous value
-                                previousValue[$changes].root?.remove(previousValue[$changes], obj);
+                                previousValue[$changes].root?.remove(previousValue[$changes]);
                             }
                         }
                         else {
@@ -2167,7 +2167,7 @@ class ArraySchema {
         const changeTree = this[$changes];
         // remove children references
         changeTree.forEachChild((childChangeTree, _) => {
-            changeTree.root?.remove(childChangeTree, this);
+            changeTree.root?.remove(childChangeTree);
         });
         changeTree.discard(true);
         changeTree.operation(exports.OPERATION.CLEAR);
@@ -2721,7 +2721,7 @@ class MapSchema {
                 operation = exports.OPERATION.DELETE_AND_ADD;
                 // remove reference from previous value
                 if (previousValue !== undefined) {
-                    previousValue[$changes].root?.remove(previousValue[$changes], this);
+                    previousValue[$changes].root?.remove(previousValue[$changes]);
                 }
             }
         }
@@ -2758,7 +2758,7 @@ class MapSchema {
         changeTree.indexes = {};
         // remove children references
         changeTree.forEachChild((childChangeTree, _) => {
-            changeTree.root?.remove(childChangeTree, this);
+            changeTree.root?.remove(childChangeTree);
         });
         // clear previous indexes
         this.$indexes.clear();
@@ -3600,7 +3600,7 @@ class CollectionSchema {
         changeTree.indexes = {};
         // remove children references
         changeTree.forEachChild((childChangeTree, _) => {
-            changeTree.root?.remove(childChangeTree, this);
+            changeTree.root?.remove(childChangeTree);
         });
         // clear previous indexes
         this.$indexes.clear();
@@ -3922,7 +3922,7 @@ class Root {
         // console.log("ADD", { refId: changeTree.refId, refCount: this.refCount[changeTree.refId] });
         return isNewChangeTree;
     }
-    remove(changeTree, parent) {
+    remove(changeTree) {
         const refCount = (this.refCount[changeTree.refId]) - 1;
         // console.log("REMOVE", { refId: changeTree.refId, refCount });
         if (refCount <= 0) {
@@ -3943,7 +3943,7 @@ class Root {
                     if ((child.parentChain === undefined || // no parent, remove it
                         (child.parentChain && this.refCount[child.refId] > 1) // parent is still in use, but has more than one reference, remove it
                     )) {
-                        this.remove(child, changeTree.ref);
+                        this.remove(child);
                     }
                     else if (child.parentChain) {
                         // re-assigning a child of the same root, move it to the end
@@ -4424,7 +4424,11 @@ class Decoder {
                 // Trying to access a reference that haven't been decoded yet.
                 //
                 if (!nextRef) {
-                    throw new Error(`"refId" not found: ${nextRefId}`);
+                    // throw new Error(`"refId" not found: ${nextRefId}`);
+                    console.error(`"refId" not found: ${nextRefId}`, { previousRef: ref, previousRefId: this.currentRefId });
+                    console.warn("Please report this to the developers. All refIds =>");
+                    console.warn(Schema.debugRefIdsFromDecoder(this));
+                    this.skipCurrentStructure(bytes, it, totalBytes);
                 }
                 else {
                     ref = nextRef;