@colyseus/schema 3.0.72 → 3.0.73

package/build/esm/index.mjs

@@ -3522,7 +3522,15 @@ function schema(fieldsAndMethods, name, inherits = Schema) {
                     // Collection: Set → new SetSchema()
                     defaultValues[fieldName] = new SetSchema();
                 }
-                else if (value['type'] !== undefined && Schema.is(value['type'])) ;
+                else if (value['type'] !== undefined && Schema.is(value['type'])) {
+                    // Direct Schema type: Type → new Type()
+                    if (!value['type'].prototype.initialize || value['type'].prototype.initialize.length === 0) {
+                        // only auto-initialize Schema instances if:
+                        // - they don't have an initialize method
+                        // - or initialize method doesn't accept any parameters
+                        defaultValues[fieldName] = new value['type']();
+                    }
+                }
             }
             else {
                 defaultValues[fieldName] = value['default'];
@@ -3531,8 +3539,12 @@ function schema(fieldsAndMethods, name, inherits = Schema) {
         else if (typeof (value) === "function") {
             if (Schema.is(value)) {
                 // Direct Schema type: Type → new Type()
-                // TODO: should we auto-initialize Schema instances, in case their initialize() method is not defined?
-                // defaultValues[fieldName] = new value();
+                if (!value.prototype.initialize || value.prototype.initialize.length === 0) {
+                    // only auto-initialize Schema instances if:
+                    // - they don't have an initialize method
+                    // - or initialize method doesn't accept any parameters
+                    defaultValues[fieldName] = new value();
+                }
                 fields[fieldName] = getNormalizedType(value);
             }
             else {