@colyseus/schema 4.0.3 → 4.0.4

package/build/index.js

@@ -5126,32 +5126,6 @@
         decoder.triggerChanges = callback;
     }
 
-    /**
-     * State Callbacks handler
-     *
-     * Usage:
-     * ```ts
-     * const $ = Callbacks.get(decoder);
-     *
-     * // Listen to property changes
-     * $.listen("currentTurn", (currentValue, previousValue) => { ... });
-     *
-     * // Listen to collection additions
-     * $.onAdd("entities", (sessionId, entity) => {
-     *     // Nested property listening
-     *     $.listen(entity, "hp", (currentHp, previousHp) => { ... });
-     * });
-     *
-     * // Listen to collection removals
-     * $.onRemove("entities", (sessionId, entity) => { ... });
-     *
-     * // Listen to any property change on an instance
-     * $.onChange(entity, () => { ... });
-     *
-     * // Bind properties to another object
-     * $.bindTo(player, playerVisual);
-     * ```
-     */
     class StateCallbackStrategy {
         decoder;
         uniqueRefIds = new Set();
@@ -5190,7 +5164,7 @@
                 immediate = immediate && this.isTriggering === false;
                 if (operation === exports.OPERATION.ADD && immediate) {
                     collection.forEach((value, key) => {
-                        handler(key, value);
+                        handler(value, key);
                     });
                 }
                 return this.addCallback(collection[$refId], operation, handler);
@@ -5346,11 +5320,11 @@
                         // FIXME: `previousValue` should always be available.
                         //
                         if (change.previousValue !== undefined) {
-                            // trigger onRemove (key, value)
+                            // trigger onRemove (value, key)
                             const deleteCallbacks = $callbacks[exports.OPERATION.DELETE];
                             if (deleteCallbacks) {
                                 for (let j = deleteCallbacks.length - 1; j >= 0; j--) {
-                                    deleteCallbacks[j](dynamicIndex, change.previousValue);
+                                    deleteCallbacks[j](change.previousValue, dynamicIndex);
                                 }
                             }
                         }
@@ -5360,7 +5334,7 @@
                             if (addCallbacks) {
                                 this.isTriggering = true;
                                 for (let j = addCallbacks.length - 1; j >= 0; j--) {
-                                    addCallbacks[j](dynamicIndex, change.value);
+                                    addCallbacks[j](change.value, dynamicIndex);
                                 }
                                 this.isTriggering = false;
                             }
@@ -5368,12 +5342,12 @@
                     }
                     else if ((change.op & exports.OPERATION.ADD) === exports.OPERATION.ADD &&
                         change.previousValue !== change.value) {
-                        // trigger onAdd (key, value)
+                        // trigger onAdd (value, key)
                         const addCallbacks = $callbacks[exports.OPERATION.ADD];
                         if (addCallbacks) {
                             this.isTriggering = true;
                             for (let j = addCallbacks.length - 1; j >= 0; j--) {
-                                addCallbacks[j](dynamicIndex, change.value);
+                                addCallbacks[j](change.value, dynamicIndex);
                             }
                             this.isTriggering = false;
                         }
@@ -5407,13 +5381,13 @@
          * callbacks.listen("currentTurn", (currentValue, previousValue) => { ... });
          *
          * // Listen to collection additions
-         * callbacks.onAdd("entities", (sessionId, entity) => {
+         * callbacks.onAdd("entities", (entity, sessionId) => {
          *     // Nested property listening
          *     callbacks.listen(entity, "hp", (currentHp, previousHp) => { ... });
          * });
          *
          * // Listen to collection removals
-         * callbacks.onRemove("entities", (sessionId, entity) => { ... });
+         * callbacks.onRemove("entities", (entity, sessionId) => { ... });
          *
          * // Listen to any property change on an instance
          * callbacks.onChange(entity, () => { ... });

package/build/index.mjs

@@ -5120,32 +5120,6 @@ function getRawChangesCallback(decoder, callback) {
     decoder.triggerChanges = callback;
 }
 
-/**
- * State Callbacks handler
- *
- * Usage:
- * ```ts
- * const $ = Callbacks.get(decoder);
- *
- * // Listen to property changes
- * $.listen("currentTurn", (currentValue, previousValue) => { ... });
- *
- * // Listen to collection additions
- * $.onAdd("entities", (sessionId, entity) => {
- *     // Nested property listening
- *     $.listen(entity, "hp", (currentHp, previousHp) => { ... });
- * });
- *
- * // Listen to collection removals
- * $.onRemove("entities", (sessionId, entity) => { ... });
- *
- * // Listen to any property change on an instance
- * $.onChange(entity, () => { ... });
- *
- * // Bind properties to another object
- * $.bindTo(player, playerVisual);
- * ```
- */
 class StateCallbackStrategy {
     decoder;
     uniqueRefIds = new Set();
@@ -5184,7 +5158,7 @@ class StateCallbackStrategy {
             immediate = immediate && this.isTriggering === false;
             if (operation === OPERATION.ADD && immediate) {
                 collection.forEach((value, key) => {
-                    handler(key, value);
+                    handler(value, key);
                 });
             }
             return this.addCallback(collection[$refId], operation, handler);
@@ -5340,11 +5314,11 @@ class StateCallbackStrategy {
                     // FIXME: `previousValue` should always be available.
                     //
                     if (change.previousValue !== undefined) {
-                        // trigger onRemove (key, value)
+                        // trigger onRemove (value, key)
                         const deleteCallbacks = $callbacks[OPERATION.DELETE];
                         if (deleteCallbacks) {
                             for (let j = deleteCallbacks.length - 1; j >= 0; j--) {
-                                deleteCallbacks[j](dynamicIndex, change.previousValue);
+                                deleteCallbacks[j](change.previousValue, dynamicIndex);
                             }
                         }
                     }
@@ -5354,7 +5328,7 @@ class StateCallbackStrategy {
                         if (addCallbacks) {
                             this.isTriggering = true;
                             for (let j = addCallbacks.length - 1; j >= 0; j--) {
-                                addCallbacks[j](dynamicIndex, change.value);
+                                addCallbacks[j](change.value, dynamicIndex);
                             }
                             this.isTriggering = false;
                         }
@@ -5362,12 +5336,12 @@ class StateCallbackStrategy {
                 }
                 else if ((change.op & OPERATION.ADD) === OPERATION.ADD &&
                     change.previousValue !== change.value) {
-                    // trigger onAdd (key, value)
+                    // trigger onAdd (value, key)
                     const addCallbacks = $callbacks[OPERATION.ADD];
                     if (addCallbacks) {
                         this.isTriggering = true;
                         for (let j = addCallbacks.length - 1; j >= 0; j--) {
-                            addCallbacks[j](dynamicIndex, change.value);
+                            addCallbacks[j](change.value, dynamicIndex);
                         }
                         this.isTriggering = false;
                     }
@@ -5401,13 +5375,13 @@ const Callbacks = {
      * callbacks.listen("currentTurn", (currentValue, previousValue) => { ... });
      *
      * // Listen to collection additions
-     * callbacks.onAdd("entities", (sessionId, entity) => {
+     * callbacks.onAdd("entities", (entity, sessionId) => {
      *     // Nested property listening
      *     callbacks.listen(entity, "hp", (currentHp, previousHp) => { ... });
      * });
      *
      * // Listen to collection removals
-     * callbacks.onRemove("entities", (sessionId, entity) => { ... });
+     * callbacks.onRemove("entities", (entity, sessionId) => { ... });
      *
      * // Listen to any property change on an instance
      * callbacks.onChange(entity, () => { ... });