@woosh/meep-engine 2.53.0 → 2.55.0

package/src/engine/graphics/particles/particular/engine/emitter/serde/ParticleEmitterSerializationAdapter.js

@@ -7,12 +7,9 @@ import { ParticleEmitterFlag } from "../ParticleEmitterFlag.js";
 import { SimulationStepDefinition } from "../../simulator/SimulationStepDefinition.js";
 
 export class ParticleEmitterSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = ParticleEmitter;
-        this.version = 1;
-    }
+    klass = ParticleEmitter;
+    version = 1;
 
     /**
      *

package/src/engine/graphics/particles/particular/engine/shader/ShaderManager.js

@@ -10,7 +10,7 @@ import {
     Vector2 as ThreeVector2
 } from "three";
 import Vector2 from "../../../../../../core/geom/Vector2.js";
-import { HashMap } from "../../../../../../core/collection/HashMap.js";
+import { HashMap } from "../../../../../../core/collection/map/HashMap.js";
 import { ParticleEmitterFlag } from "../emitter/ParticleEmitterFlag.js";
 import { writeSample2DDataToDataTexture } from "../../../../texture/sampler/writeSampler2DDataToDataTexture.js";
 import { MaterialRecord } from "./MaterialRecord.js";

package/src/engine/graphics/texture/atlas/ManagedTextureAtlas.js

@@ -2,7 +2,7 @@ import { TextureAtlas } from "./TextureAtlas.js";
 import { ReferenceManager } from "../../../reference/v1/ReferenceManager.js";
 import { Sampler2D } from "../sampler/Sampler2D.js";
 import { GameAssetType } from "../../../asset/GameAssetType.js";
-import { HashMap } from "../../../../core/collection/HashMap.js";
+import { HashMap } from "../../../../core/collection/map/HashMap.js";
 import { strictEquals } from "../../../../core/function/Functions.js";
 import { computeStringHash } from "../../../../core/primitives/strings/computeStringHash.js";
 import { copy_Sampler2D_channel_data } from "../sampler/copy_Sampler2D_channel_data.js";

package/src/engine/graphics/texture/sampler/serialization/Sampler2DSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../../../ecs/storage/binary/
 import { Sampler2D } from "../Sampler2D.js";
 
 export class Sampler2DSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.version = 0;
-        this.klass = Sampler2D;
-    }
+    version = 0;
+    klass = Sampler2D;
 
     /**
      *

package/src/engine/grid/grid2transform/GridPosition2Transform.js

@@ -9,19 +9,17 @@ import { GridPosition2TransformFlags } from "./GridPosition2TransformFlags.js";
 const DEFAULT_FLAGS = GridPosition2TransformFlags.Immediate;
 
 export class GridPosition2Transform {
-    constructor() {
-        /**
-         *
-         * @type {Vector2}
-         */
-        this.offset = new Vector2(0, 0);
-
-        /**
-         *
-         * @type {GridPosition2TransformFlags|number}
-         */
-        this.flags = DEFAULT_FLAGS;
-    }
+    /**
+     *
+     * @type {Vector2}
+     */
+    offset = new Vector2(0, 0);
+
+    /**
+     *
+     * @type {GridPosition2TransformFlags|number}
+     */
+    flags = DEFAULT_FLAGS;
 
     /**
      *

package/src/engine/grid/grid2transform/GridPosition2TransformSerializationAdapter.js

@@ -2,12 +2,10 @@ import { BinaryClassSerializationAdapter } from "../../ecs/storage/binary/Binary
 import { GridPosition2Transform } from "./GridPosition2Transform.js";
 
 export class GridPosition2TransformSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = GridPosition2Transform;
-        this.version = 1;
-    }
+    klass = GridPosition2Transform;
+    version = 1;
+
 
     /**
      *

package/src/engine/grid/obstacle/GridObstacleSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../ecs/storage/binary/Binary
 import GridObstacle from "./GridObstacle.js";
 
 export class GridObstacleSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = GridObstacle;
-        this.version = 0;
-    }
+    klass = GridObstacle;
+    version = 0;
 
     /**
      *

package/src/engine/grid/position/GridPositionSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../ecs/storage/binary/Binary
 import GridPosition from "./GridPosition.js";
 
 export class GridPositionSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = GridPosition;
-        this.version = 1;
-    }
+    klass = GridPosition;
+    version = 1;
 
     /**
      *

package/src/engine/grid/transform2grid/Transform2GridPosition.js

@@ -1,19 +1,16 @@
 import { Transform2GridPositionMode } from "./Transform2GridPositionMode.js";
 
 export class Transform2GridPosition {
-    constructor() {
-        /**
-         *
-         * @type {Transform2GridPositionFlags|number}
-         */
-        this.flags = 0;
-
-        /**
-         *
-         * @type {number}
-         */
-        this.mode = Transform2GridPositionMode.Continuous;
-    }
+    /**
+     *
+     * @type {Transform2GridPositionFlags|number}
+     */
+    flags = 0;
+    /**
+     *
+     * @type {number}
+     */
+    mode= Transform2GridPositionMode.Continuous;
 
     /**
      *

package/src/engine/input/devices/KeyboardDevice.js

@@ -73,8 +73,11 @@ class KeyboardDevice {
         /*
         Only element in focus receives keyboard events, so the element supplied here must be focusable in order to be able to receive events
          */
-        if (!isFocusable(domElement) && domElement.getAttribute('tabindex') === null) {
-            console.error('Supplied element is not inherently focusable and does not have tabindex attribute, so it must have a tabindex attribute in order to be able receive keyboard events. Something like tabindex=0 would suffice.');
+        if (
+            !isFocusable(domElement)
+            && domElement.getAttribute('tabindex') === null
+        ) {
+            new TypeError('Supplied element is not inherently focusable and does not have tabindex attribute, so it must have a tabindex attribute in order to be able receive keyboard events. Something like tabindex=0 would suffice.');
         }
 
         /**

package/src/engine/input/devices/KeyboardDevice.spec.js

@@ -0,0 +1,12 @@
+/**
+ * @jest-environment jsdom
+ */
+
+import KeyboardDevice from "./KeyboardDevice.js";
+
+test("constructor does not throw", () => {
+    const el = document.createElement('div');
+    el.setAttribute('tabindex', "0");
+
+    new KeyboardDevice(el);
+});

package/src/engine/input/devices/PointerDevice.js

@@ -322,9 +322,13 @@ export function readPositionFromMouseEvent(result, event, source = event.target)
     result.set(x, y);
 }
 
+/**
+ * Abstracts Mouse and Touch interfaces as single "pointer" device.
+ */
 export class PointerDevice {
 
     /**
+     * Current live position of the pointer
      * @readonly
      * @type {Vector2}
      */
@@ -373,7 +377,7 @@ export class PointerDevice {
     isRunning = false;
 
     /**
-     * The MouseEvent.buttons is a 32bit field, which means we can encode up to 32 buttons
+     * The {@link MouseEvent.buttons} is a 32bit field, which means we can encode up to 32 buttons
      * @readonly
      * @type {InputDeviceSwitch[]}
      */

package/src/engine/input/ecs/components/Input.js

@@ -4,12 +4,28 @@ import { InputBinding } from "../ism/InputBinding.js";
 
 
 export class Input {
-    constructor() {
-        /**
-         *
-         * @type {List<InputBinding>}
-         */
-        this.bindings = new List();
+
+    /**
+     * @readonly
+     * @type {List<InputBinding>}
+     */
+    bindings = new List();
+
+    /**
+     *
+     * @param {Input} other
+     * @returns {boolean}
+     */
+    equals(other) {
+        return this.bindings.equals(other.bindings);
+    }
+
+    /**
+     *
+     * @return {number}
+     */
+    hash() {
+        return this.bindings.hash();
     }
 
     /**
@@ -19,10 +35,20 @@ export class Input {
      * @returns {boolean}
      */
     exists(path, event) {
-        assert.typeOf(path, 'string', "path");
-        assert.typeOf(event, 'string', "event");
+        assert.isString(path, "path");
+        assert.isString(event, "event");
+
+        const bindings = this.bindings;
+        const count = bindings.length;
+        for (let i = 0; i < count; i++) {
+            const binding = bindings.get(i);
+
+            if (binding.path === path && binding.event === event) {
+                return true;
+            }
+        }
 
-        return this.bindings.some(b => b.path === path && b.event === event);
+        return false;
     }
 
     /**
@@ -32,8 +58,8 @@ export class Input {
      * @returns {boolean}
      */
     bind(path, event) {
-        assert.typeOf(path, 'string', path);
-        assert.typeOf(event, 'string', path);
+        assert.isString(path, "path");
+        assert.isString(event, "event");
 
         if (this.exists(path, event)) {
             //binding exists

package/src/engine/input/ecs/ism/InputBinding.js

@@ -1,18 +1,18 @@
 import { assert } from "../../../../core/assert.js";
+import { computeStringHash } from "../../../../core/primitives/strings/computeStringHash.js";
 
 export class InputBinding {
-    constructor() {
-        /**
-         *
-         * @type {string}
-         */
-        this.path = "";
-        /**
-         *
-         * @type {string}
-         */
-        this.event = "";
-    }
+
+    /**
+     *
+     * @type {string}
+     */
+    path = "";
+    /**
+     *
+     * @type {string}
+     */
+    event = "";
 
     /**
      *
@@ -27,6 +27,24 @@ export class InputBinding {
         this.event = event;
     }
 
+    /**
+     *
+     * @param {InputBinding} other
+     * @returns {boolean}
+     */
+    equals(other) {
+        return this.event === other.event
+            && this.path === other.path;
+    }
+
+    /**
+     *
+     * @return {number}
+     */
+    hash() {
+        return computeStringHash(this.event) ^ computeStringHash(this.path);
+    }
+
     /**
      *
      * @param {BinaryBuffer} buffer

package/src/engine/intelligence/behavior/ecs/BehaviorComponentSerializationAdapter.js

@@ -4,18 +4,15 @@ import { SequenceBehavior } from "../composite/SequenceBehavior.js";
 import { SucceedingBehavior } from "../primitive/SucceedingBehavior.js";
 
 export class BehaviorComponentSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = BehaviorComponent;
-        this.version = 0;
+    klass = BehaviorComponent;
+    version = 0;
 
-        /**
-         *
-         * @type {BinaryObjectSerializationAdapter}
-         */
-        this.objectAdapter = null;
-    }
+    /**
+     *
+     * @type {BinaryObjectSerializationAdapter}
+     */
+    objectAdapter = null;
 
     /**
      *

package/src/engine/intelligence/behavior/ecs/WaitForEventBehaviorSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../../ecs/storage/binary/Bin
 import { WaitForEventBehavior } from "./WaitForEventBehavior.js";
 
 export class WaitForEventBehaviorSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = WaitForEventBehavior;
-        this.version = 0;
-    }
+    klass = WaitForEventBehavior;
+    version = 0;
 
     /**
      *

package/src/engine/intelligence/behavior/primitive/FailingBehaviorSerializationAdapter.js

@@ -2,12 +2,8 @@ import { BinaryClassSerializationAdapter } from "../../../ecs/storage/binary/Bin
 import { FailingBehavior } from "./FailingBehavior.js";
 
 export class FailingBehaviorSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
-
-        this.klass = FailingBehavior;
-        this.version = 0;
-    }
+    klass = FailingBehavior;
+    version = 0;
 
     /**
      *

package/src/engine/intelligence/behavior/primitive/SucceedingBehaviorSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../../ecs/storage/binary/Bin
 import { SucceedingBehavior } from "./SucceedingBehavior.js";
 
 export class SucceedingBehaviorSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = SucceedingBehavior;
-        this.version = 0;
-    }
+    klass = SucceedingBehavior;
+    version = 0;
 
     /**
      *

package/src/engine/intelligence/blackboard/BlackboardSerializationAdapter.js

@@ -4,12 +4,9 @@ import { Blackboard } from "./Blackboard.js";
 import { BlackboardValue } from "./BlackboardValue.js";
 
 export class BlackboardSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = Blackboard;
-        this.version = 0;
-    }
+    klass = Blackboard;
+    version = 0;
 
     /**
      *

package/src/engine/knowledge/database/StaticKnowledgeDatabase.js

@@ -18,22 +18,21 @@ export const StaticKnowledgeState = {
 };
 
 export class StaticKnowledgeDatabase {
-    constructor() {
 
-        /**
-         *
-         * @type {ObservedEnum<StaticKnowledgeState>}
-         */
-        this.__state = new ObservedEnum(StaticKnowledgeState.Initial, StaticKnowledgeState);
+    /**
+     *
+     * @type {ObservedEnum<StaticKnowledgeState>}
+     */
+    __state = new ObservedEnum(StaticKnowledgeState.Initial, StaticKnowledgeState);
 
 
-        /**
-         *
-         * @type {StaticKnowledgeDataTableDescriptor[]}
-         * @private
-         */
-        this.__specs = [];
-    }
+    /**
+     *
+     * @type {StaticKnowledgeDataTableDescriptor[]}
+     * @private
+     */
+    __specs = [];
+    
 
     /**
      * Reset database, drop all table data

package/src/engine/knowledge/database/StaticKnowledgeDatabase.spec.js

@@ -0,0 +1,23 @@
+import { StaticKnowledgeDatabase } from "./StaticKnowledgeDatabase.js";
+import { StaticKnowledgeDataTable } from "./StaticKnowledgeDataTable.js";
+
+test("constructor does not throw", () => {
+    new StaticKnowledgeDatabase()
+});
+
+test("reset empty", () => {
+    const database = new StaticKnowledgeDatabase();
+
+    database.reset();
+});
+
+test("add/get", () => {
+    const database = new StaticKnowledgeDatabase();
+
+    const table = new StaticKnowledgeDataTable();
+    database.add("x", "y", table)
+
+    const actual = database.getTable('x');
+
+    expect(actual).toBe(table);
+});

package/src/engine/navigation/ecs/components/PathSerializationAdapter.js

@@ -6,12 +6,9 @@ import Path from "./Path.js";
 const v0 = new Vector3();
 
 export class PathSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = Path;
-        this.version = 3;
-    }
+    klass = Path;
+    version = 3;
 
     /**
      *

package/src/engine/navigation/ecs/path_following/PathFollowerSerializationAdapter.js

@@ -2,12 +2,9 @@ import { BinaryClassSerializationAdapter } from "../../../ecs/storage/binary/Bin
 import PathFollower from "./PathFollower.js";
 
 export class PathFollowerSerializationAdapter extends BinaryClassSerializationAdapter {
-    constructor() {
-        super();
 
-        this.klass = PathFollower;
-        this.version = 3;
-    }
+    klass = PathFollower;
+    version = 3;
 
     /**
      *

package/src/engine/navigation/grid/find_path_on_grid_astar.spec.js

@@ -5,3 +5,28 @@ test("sanity check on 1x1 grid", () => {
 
     expect(path).toEqual([0]);
 });
+
+test("sanity check on 2x1 grid", () => {
+    const path = find_path_on_grid_astar(
+        [0, 0], 2, 1,
+        0, 1,
+        0, 0, 1
+    );
+
+    expect(path).toEqual([0, 1]);
+});
+
+test("sanity check on 2x2 grid", () => {
+
+    const path = find_path_on_grid_astar(
+        [
+            0, 0,
+            1, 0
+        ],
+        2, 2,
+        0, 3,
+        0, 0, 1
+    );
+
+    expect(path).toEqual([0, 1, 3]);
+});

package/src/engine/options/OptionAbstract.js

@@ -1,4 +1,10 @@
 export class OptionAbstract {
+    /**
+     *
+     * @type {OptionGroup|null}
+     */
+    parent = null;
+
     /**
      *
      * @param {string} id
@@ -11,11 +17,6 @@ export class OptionAbstract {
          */
         this.id = id;
 
-        /**
-         *
-         * @type {OptionGroup|null}
-         */
-        this.parent = null;
     }
 
     /**

package/src/engine/options/OptionGroup.js

@@ -8,20 +8,12 @@ function isOption(c) {
 }
 
 export class OptionGroup extends OptionAbstract {
+
     /**
      *
-     * @param {string} id
-     * @constructor
+     * @type {Array<OptionGroup|Option>}
      */
-    constructor(id) {
-        super(id);
-
-        /**
-         *
-         * @type {Array<OptionGroup|Option>}
-         */
-        this.children = [];
-    }
+    children = [];
 
     /**
      *
@@ -141,6 +133,7 @@ export class OptionGroup extends OptionAbstract {
      *
      * @param {string} path
      * @param {Storage} storage
+     * @returns {Promise}
      */
     attachToStorage(path, storage) {
         const group = this;
@@ -151,20 +144,6 @@ export class OptionGroup extends OptionAbstract {
             storage.load(key, resolve, reject);
         });
 
-        p
-            .then((loaded) => {
-                try {
-                    group.fromJSON(JSON.parse(loaded));
-                } catch (e) {
-                    console.log('Failed to load options')
-                }
-            })
-            .catch(console.error)
-            .finally(() => {
-
-                group.traverseOptions(op => op.on.written.add(store));
-            });
-
 
         function store() {
             const json = group.toJSON();
@@ -174,6 +153,15 @@ export class OptionGroup extends OptionAbstract {
             storage.store(key, value, noop, console.error, noop);
         }
 
+        return p
+            .then((loaded) => {
+                group.fromJSON(JSON.parse(loaded));
+            })
+            .finally(() => {
+
+                group.traverseOptions(op => op.on.written.add(store));
+            });
+
     }
 }