plain schema({}) declaration: do not auto-initialize child Schema types

Author: endel

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@colyseus/schema",
-  "version": "3.0.70",
+  "version": "3.0.71",
   "description": "Binary state serializer with delta encoding for games",
   "bin": {
     "schema-codegen": "bin/schema-codegen",

src/annotations.ts

@@ -607,7 +607,6 @@ export function schema<
         } else if (typeof (value) === "function") {
             if (Schema.is(value)) {
                 // Direct Schema type: Type → new Type()
-                defaultValues[fieldName] = new value();
                 fields[fieldName] = getNormalizedType(value);
             } else {
                 methods[fieldName] = value;

test/BackwardsForwardsCompatibility.test.ts

@@ -55,6 +55,7 @@ describe("backwards/forwards compatibility", () => {
             x: 10,
             y: 10,
             name: "Forward",
+            // @ts-ignore
             arrayOfStrings: new ArraySchema("one"),
         }));
 

test/Definition.test.ts

@@ -656,6 +656,27 @@ describe("Definition Tests", () => {
 
         });
 
+        it("should not auto-initialize Schema instances", () => {
+            const Base = schema({
+                value: 'string',
+                initialize(value: { something: string }) {
+                    this.value = value.something;
+                },
+            });
+
+            const Child = schema({
+                world: 'string',
+                random: Base,
+                initialize(world: string) {
+                    this.world = world;
+                },
+            });
+
+            assert.doesNotThrow(() => {
+                new Child('hello');
+            });
+        });
+
     });
 
 });

test/Schema.test.ts

@@ -344,6 +344,7 @@ describe("Type: Schema", () => {
 
             assert.throws(() => {
                 class Player extends Entity {
+                    // @ts-ignore
                     @type("string") id: string;
                 }
             }, /Duplicate 'id' definition on 'Player'/);

test/StateView.test.ts

@@ -2520,7 +2520,7 @@ describe("StateView", () => {
             entity.components.push(new TagComponent());
             assert.strictEqual(entity.components[1][$changes].isFiltered, true);
 
-            entity.components.push(new ListComponent().assign({list: new ArraySchema("one", "two")}));
+            entity.components.push(new ListComponent().assign({ list: new ArraySchema("one", "two") }));
             assert.strictEqual(entity.components[2][$changes].isFiltered, true);
 
             entity.component = new Component();