@colyseus/schema 2.0.10 → 2.0.12

package/build/cjs/index.js

@@ -472,6 +472,14 @@ var ArraySchema = /** @class */ (function () {
     };
     ArraySchema.prototype.setAt = function (index, value) {
         var _a, _b;
+        if (value === undefined || value === null) {
+            console.error("ArraySchema items cannot be null nor undefined; Use `deleteAt(index)` instead.");
+            return;
+        }
+        // skip if the value is the same as cached.
+        if (this.$items.get(index) === value) {
+            return;
+        }
         if (value['$changes'] !== undefined) {
             value['$changes'].setParent(this, this.$changes.root, index);
         }
@@ -939,6 +947,9 @@ var MapSchema = /** @class */ (function () {
         if (value === undefined || value === null) {
             throw new Error("MapSchema#set('".concat(key, "', ").concat(value, "): trying to set ").concat(value, " value on '").concat(key, "'."));
         }
+        // Force "key" as string
+        // See: https://github.com/colyseus/colyseus/issues/561#issuecomment-1646733468
+        key = key.toString();
         // get "index" for this value.
         var hasIndex = typeof (this.$changes.indexes[key]) !== "undefined";
         var index = (hasIndex)
@@ -959,6 +970,11 @@ var MapSchema = /** @class */ (function () {
             this.$changes.indexes[key] = index;
             this.$indexes.set(index, key);
         }
+        else if (!isRef &&
+            this.$items.get(key) === value) {
+            // if value is the same, avoid re-encoding it.
+            return;
+        }
         else if (isRef && // if is schema, force ADD operation if value differ from previous one.
             this.$items.get(key) !== value) {
             operation = exports.OPERATION.ADD;