@colyseus/schema 4.0.14 → 4.0.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@colyseus/schema",
-  "version": "4.0.14",
+  "version": "4.0.16",
   "description": "Binary state serializer with delta encoding for games",
   "type": "module",
   "bin": {

package/src/codegen/parser.ts

@@ -77,12 +77,15 @@ function inspectNode(node: ts.Node, context: Context, decoratorName: string) {
 
         case ts.SyntaxKind.PropertySignature:
             if (currentStructure instanceof Interface) {
-                const interfaceDeclaration = node.parent;
+                const parent = node.parent;
 
-                if (
-                    currentStructure.name !== (interfaceDeclaration as ts.TypeParameterDeclaration).name.escapedText.toString()
-                ) {
-                    // skip if property if for a another interface than the one we're interested in.
+                // Only process direct children of InterfaceDeclaration, skip TypeLiterals
+                if (!ts.isInterfaceDeclaration(parent)) {
+                    break;
+                }
+
+                // Skip if property if for a another interface than the one we're interested in.
+                if (currentStructure.name !== parent.name.escapedText.toString()) {
                     break;
                 }
 

package/src/encoder/StateView.ts

@@ -261,14 +261,31 @@ export class StateView {
 
             } else {
                 // delete all "tagged" properties.
-                metadata?.[$viewFieldIndexes]?.forEach((index) =>
-                    changes[index] = OPERATION.DELETE);
+                metadata?.[$viewFieldIndexes]?.forEach((index) => {
+                    changes[index] = OPERATION.DELETE;
+
+                    // Remove child structures of @view() fields from visible set.
+                    // (They were added during view.add() via forEachChild)
+                    const value = changeTree.ref[metadata[index].name as keyof Ref];
+                    if (value?.[$changes]) {
+                        this.visible.delete(value[$changes]);
+                        this._recursiveDeleteVisibleChangeTree(value[$changes]);
+                    }
+                });
             }
 
         } else {
             // delete only tagged properties
-            metadata?.[$fieldIndexesByViewTag][tag].forEach((index) =>
-                changes[index] = OPERATION.DELETE);
+            metadata?.[$fieldIndexesByViewTag][tag].forEach((index) => {
+                changes[index] = OPERATION.DELETE;
+
+                // Remove child structures from visible set
+                const value = changeTree.ref[metadata[index].name as keyof Ref];
+                if (value?.[$changes]) {
+                    this.visible.delete(value[$changes]);
+                    this._recursiveDeleteVisibleChangeTree(value[$changes]);
+                }
+            });
         }
 
         // remove tag