@dcl/ecs 7.27.1-34418200091.commit-06bc595 → 7.27.1-34480514651.commit-2778e4b

package/dist/engine/input.js

@@ -57,22 +57,22 @@ export function createInputSystem(engine) {
             const arrayCommands = Array.from(commands);
             for (let i = arrayCommands.length - 1; i >= 0; i--) {
                 const command = arrayCommands[i];
+                // Commands are kept sorted by timestamp, so this one and every one
+                // below it belongs to a frame that was already accounted for. Stopping
+                // on the button state instead would end this entity's whole loop over
+                // a button another entity had already reported this frame.
+                if (command.timestamp <= globalState.previousFrameMaxTimestamp) {
+                    break;
+                }
                 if (command.timestamp > maxTimestamp) {
                     maxTimestamp = command.timestamp;
                 }
-                if (command.timestamp > globalState.previousFrameMaxTimestamp) {
-                    globalState.thisFrameCommands.push(command);
-                }
+                globalState.thisFrameCommands.push(command);
                 if (command.state === 0 /* PointerEventType.PET_UP */ || command.state === 1 /* PointerEventType.PET_DOWN */) {
                     const prevCommand = globalState.buttonState.get(command.button);
                     if (!prevCommand || command.timestamp > prevCommand.timestamp) {
                         globalState.buttonState.set(command.button, command);
                     }
-                    else {
-                        // since we are iterating a descending array, we can early finish the
-                        // loop
-                        break;
-                    }
                 }
             }
         }

package/dist/schemas/OneOf.js

@@ -1,3 +1,5 @@
+/** Reserved case index for a value whose case was never selected. */
+const NO_CASE = 0;
 export const IOneOf = (specs) => {
     const specKeys = Object.keys(specs);
     const keyToIndex = specKeys.reduce((dict, key, index) => {
@@ -10,12 +12,22 @@ export const IOneOf = (specs) => {
     }, {});
     return {
         serialize({ $case, value }, builder) {
+            // `create()` selects no case, so a field the scene never set arrives here
+            // with an undefined `$case`. Index 0 is free, cases start at 1.
+            if ($case === undefined) {
+                builder.writeUint8(NO_CASE);
+                return;
+            }
             const _value = keyToIndex[$case.toString()] + 1;
             builder.writeUint8(_value);
             specs[$case].serialize(value, builder);
         },
         deserialize(reader) {
-            const $case = specKeys[reader.readInt8() - 1];
+            const caseIndex = reader.readInt8();
+            if (caseIndex === NO_CASE) {
+                return {};
+            }
+            const $case = specKeys[caseIndex - 1];
             const value = specs[$case].deserialize(reader);
             return { $case, value };
         },

package/dist-cjs/engine/input.js

@@ -83,22 +83,22 @@ function createInputSystem(engine) {
             const arrayCommands = Array.from(commands);
             for (let i = arrayCommands.length - 1; i >= 0; i--) {
                 const command = arrayCommands[i];
+                // Commands are kept sorted by timestamp, so this one and every one
+                // below it belongs to a frame that was already accounted for. Stopping
+                // on the button state instead would end this entity's whole loop over
+                // a button another entity had already reported this frame.
+                if (command.timestamp <= globalState.previousFrameMaxTimestamp) {
+                    break;
+                }
                 if (command.timestamp > maxTimestamp) {
                     maxTimestamp = command.timestamp;
                 }
-                if (command.timestamp > globalState.previousFrameMaxTimestamp) {
-                    globalState.thisFrameCommands.push(command);
-                }
+                globalState.thisFrameCommands.push(command);
                 if (command.state === 0 /* PointerEventType.PET_UP */ || command.state === 1 /* PointerEventType.PET_DOWN */) {
                     const prevCommand = globalState.buttonState.get(command.button);
                     if (!prevCommand || command.timestamp > prevCommand.timestamp) {
                         globalState.buttonState.set(command.button, command);
                     }
-                    else {
-                        // since we are iterating a descending array, we can early finish the
-                        // loop
-                        break;
-                    }
                 }
             }
         }

package/dist-cjs/schemas/OneOf.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.IOneOf = void 0;
+/** Reserved case index for a value whose case was never selected. */
+const NO_CASE = 0;
 const IOneOf = (specs) => {
     const specKeys = Object.keys(specs);
     const keyToIndex = specKeys.reduce((dict, key, index) => {
@@ -13,12 +15,22 @@ const IOneOf = (specs) => {
     }, {});
     return {
         serialize({ $case, value }, builder) {
+            // `create()` selects no case, so a field the scene never set arrives here
+            // with an undefined `$case`. Index 0 is free, cases start at 1.
+            if ($case === undefined) {
+                builder.writeUint8(NO_CASE);
+                return;
+            }
             const _value = keyToIndex[$case.toString()] + 1;
             builder.writeUint8(_value);
             specs[$case].serialize(value, builder);
         },
         deserialize(reader) {
-            const $case = specKeys[reader.readInt8() - 1];
+            const caseIndex = reader.readInt8();
+            if (caseIndex === NO_CASE) {
+                return {};
+            }
+            const $case = specKeys[caseIndex - 1];
             const value = specs[$case].deserialize(reader);
             return { $case, value };
         },

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@dcl/ecs",
   "description": "Decentraland ECS",
-  "version": "7.27.1-34418200091.commit-06bc595",
+  "version": "7.27.1-34480514651.commit-2778e4b",
   "author": "DCL",
   "bugs": "https://github.com/decentraland/ecs/issues",
   "files": [
@@ -34,5 +34,5 @@
   "dependencies": {},
   "types": "./dist/index.d.ts",
   "typings": "./dist/index.d.ts",
-  "commit": "06bc5956460407d608a0f7bc96b74fffd415ad2a"
+  "commit": "2778e4bb891f1626a98380a8c906e5e793b74024"
 }