@woosh/meep-engine 2.81.3 → 2.82.0

package/editor/ecs/component/editors/primitive/ArrayEditor.js

@@ -1,9 +1,9 @@
-import { TypeEditor } from "../../TypeEditor.js";
 import List from "../../../../../src/core/collection/list/List.js";
+import { noop } from "../../../../../src/core/function/noop.js";
 import { FieldDescriptor } from "../../FieldDescriptor.js";
 import { FieldValueAdapter } from "../../FieldValueAdapter.js";
+import { TypeEditor } from "../../TypeEditor.js";
 import { ListEditor } from "../collection/ListEditor.js";
-import { noop } from "../../../../../src/core/function/Functions.js";
 
 export class ArrayEditor extends TypeEditor {
     inline = false;

package/editor/enableEditor.js

@@ -1,5 +1,5 @@
+import { noop } from "../src/core/function/noop.js";
 import Editor from "./Editor.js";
-import { noop } from "../src/core/function/Functions.js";
 
 /**
  *

package/editor/view/EditorView.js

@@ -3,38 +3,38 @@
  */
 
 
-import View from "../../src/view/View.js";
+import { passThrough } from "../../src/core/function/passThrough.js";
+import { SurfacePoint3 } from "../../src/core/geom/3d/SurfacePoint3.js";
+import Vector2 from "../../src/core/geom/Vector2.js";
+import Vector3 from "../../src/core/geom/Vector3.js";
+import { ProcessState } from "../../src/core/process/ProcessState.js";
+import { obtainTerrain } from "../../src/engine/ecs/terrain/util/obtainTerrain.js";
+import { Transform } from "../../src/engine/ecs/transform/Transform.js";
+import Mesh from "../../src/engine/graphics/ecs/mesh/Mesh.js";
+import { MeshEvents } from "../../src/engine/graphics/ecs/mesh/MeshEvents.js";
+import { make_ray_from_viewport_position } from "../../src/engine/graphics/make_ray_from_viewport_position.js";
+import LabelView from '../../src/view/common/LabelView.js';
+import ListView from "../../src/view/common/ListView.js";
+import VirtualListView from '../../src/view/common/VirtualListView.js';
 import dom from "../../src/view/DOM.js";
+import BottomLeftResizeHandleView from "../../src/view/elements/BottomLeftResizeHandleView.js";
 
-import EntityListView from './ecs/EntityList.js';
-import EntityEditorView from './ecs/EntityEditor.js';
+import SimpleWindow from '../../src/view/elements/SimpleWindow.js';
+import View from "../../src/view/View.js";
+import ComponentAddAction from "../actions/concrete/ComponentAddAction.js";
+import EntityCreateAction from "../actions/concrete/EntityCreateAction.js";
+import SelectionAddAction from "../actions/concrete/SelectionAddAction.js";
+import SelectionClearAction from "../actions/concrete/SelectionClearAction.js";
 import ComponentControlFactory from './ecs/ComponentControlFactory.js';
-import VirtualListView from '../../src/view/common/VirtualListView.js';
-import LabelView from '../../src/view/common/LabelView.js';
-import ToolView from './tools/ToolView.js';
+import EntityEditorView from './ecs/EntityEditor.js';
 
-import SimpleWindow from '../../src/view/elements/SimpleWindow.js';
-import ToolSettingsView from "./tools/ToolSettingsView.js";
+import EntityListView from './ecs/EntityList.js';
 import GridPickCoordinateView from "./GridPickCoordinateView.js";
-import ListView from "../../src/view/common/ListView.js";
-import ProcessView from "./process/ProcessView.js";
 import MeshLibraryView from "./library/MeshLibraryView.js";
-import BottomLeftResizeHandleView from "../../src/view/elements/BottomLeftResizeHandleView.js";
-import Vector2 from "../../src/core/geom/Vector2.js";
-import Vector3 from "../../src/core/geom/Vector3.js";
-import Mesh from "../../src/engine/graphics/ecs/mesh/Mesh.js";
-import { Transform } from "../../src/engine/ecs/transform/Transform.js";
-import SelectionClearAction from "../actions/concrete/SelectionClearAction.js";
-import EntityCreateAction from "../actions/concrete/EntityCreateAction.js";
-import ComponentAddAction from "../actions/concrete/ComponentAddAction.js";
-import SelectionAddAction from "../actions/concrete/SelectionAddAction.js";
-import { passThrough } from "../../src/core/function/Functions.js";
-import { ProcessState } from "../../src/core/process/ProcessState.js";
-import { MeshEvents } from "../../src/engine/graphics/ecs/mesh/MeshEvents.js";
-import { obtainTerrain } from "../../src/engine/ecs/terrain/util/obtainTerrain.js";
-import { SurfacePoint3 } from "../../src/core/geom/3d/SurfacePoint3.js";
-import { make_ray_from_viewport_position } from "../../src/engine/graphics/make_ray_from_viewport_position.js";
 import { makeEntityDecorators } from "./makeEntityDecorators.js";
+import ProcessView from "./process/ProcessView.js";
+import ToolSettingsView from "./tools/ToolSettingsView.js";
+import ToolView from './tools/ToolView.js';
 
 
 class ViewManager extends View {

package/editor/view/ecs/components/particles/ParticleEmitterController.js

@@ -1,20 +1,20 @@
-import DatGuiController from "../DatGuiController.js";
-import { BlendingType } from "../../../../../src/engine/graphics/texture/sampler/BlendingType.js";
-import { ParticleLayer } from "../../../../../src/engine/graphics/particles/particular/engine/emitter/ParticleLayer.js";
-import ParticleLayerController from "./ParticleLayerController.js";
+import { noop } from "../../../../../src/core/function/noop.js";
+import ObservedValue from "../../../../../src/core/model/ObservedValue.js";
 import {
     ParticleEmitterFlag
 } from "../../../../../src/engine/graphics/particles/particular/engine/emitter/ParticleEmitterFlag.js";
-import { NativeListController } from "../../../../../src/view/controller/controls/NativeListController.js";
-import EmptyView from "../../../../../src/view/elements/EmptyView.js";
-import ObservedValue from "../../../../../src/core/model/ObservedValue.js";
-import { noop } from "../../../../../src/core/function/Functions.js";
-import {
-    SimulationStepType
-} from "../../../../../src/engine/graphics/particles/particular/engine/simulator/SimulationStepType.js";
+import { ParticleLayer } from "../../../../../src/engine/graphics/particles/particular/engine/emitter/ParticleLayer.js";
 import {
     SimulationStepDefinition
 } from "../../../../../src/engine/graphics/particles/particular/engine/simulator/SimulationStepDefinition.js";
+import {
+    SimulationStepType
+} from "../../../../../src/engine/graphics/particles/particular/engine/simulator/SimulationStepType.js";
+import { BlendingType } from "../../../../../src/engine/graphics/texture/sampler/BlendingType.js";
+import { NativeListController } from "../../../../../src/view/controller/controls/NativeListController.js";
+import EmptyView from "../../../../../src/view/elements/EmptyView.js";
+import DatGuiController from "../DatGuiController.js";
+import ParticleLayerController from "./ParticleLayerController.js";
 
 
 /**

package/editor/view/node-graph/NodeView.js

@@ -1,13 +1,13 @@
-import View from "../../../src/view/View.js";
-import { PortView } from "./PortView.js";
-import EmptyView from "../../../src/view/elements/EmptyView.js";
-import LabelView from "../../../src/view/common/LabelView.js";
-import { DraggableAspect } from "../../../src/engine/ui/DraggableAspect.js";
+import { noop } from "../../../src/core/function/noop.js";
 import Vector2 from "../../../src/core/geom/Vector2.js";
-import { noop } from "../../../src/core/function/Functions.js";
 import { NodeParameterDataType } from "../../../src/core/model/node-graph/node/parameter/NodeParameterDataType.js";
-import DatGuiController from "../ecs/components/DatGuiController.js";
 import { PortDirection } from "../../../src/core/model/node-graph/node/PortDirection.js";
+import { DraggableAspect } from "../../../src/engine/ui/DraggableAspect.js";
+import LabelView from "../../../src/view/common/LabelView.js";
+import EmptyView from "../../../src/view/elements/EmptyView.js";
+import View from "../../../src/view/View.js";
+import DatGuiController from "../ecs/components/DatGuiController.js";
+import { PortView } from "./PortView.js";
 
 export class NodeView extends View {
     /**

package/package.json

@@ -5,7 +5,7 @@
   "description": "Fully featured ECS game engine written in JavaScript",
   "type": "module",
   "author": "Alexander Goldring",
-  "version": "2.81.3",
+  "version": "2.82.0",
   "main": "build/meep.module.js",
   "module": "build/meep.module.js",
   "exports": {

package/samples/terrain/editor.js

@@ -1,27 +1,28 @@
-import { EngineHarness } from "../../src/engine/EngineHarness.js";
-import EmptyView from "../../src/view/elements/EmptyView.js";
-import { EngineConfiguration } from "../../src/engine/EngineConfiguration.js";
-import { ArrayBufferLoader } from "../../src/engine/asset/loaders/ArrayBufferLoader.js";
-import { TerrainFlags } from "../../src/engine/ecs/terrain/ecs/TerrainFlags.js";
-import Signal from "../../src/core/events/signal/Signal.js";
-import { NativeListController } from "../../src/view/controller/controls/NativeListController.js";
-import { TerrainLayer } from "../../src/engine/ecs/terrain/ecs/layers/TerrainLayer.js";
-import CheckersTextureURI from "../../src/engine/graphics/texture/CheckersTextureURI.js";
-import { obtainTerrain } from "../../src/engine/ecs/terrain/util/obtainTerrain.js";
-import { Cache } from "../../src/core/cache/Cache.js";
-import { computeStringHash } from "../../src/core/primitives/strings/computeStringHash.js";
-import { noop, strictEquals } from "../../src/core/function/Functions.js";
-import { sampler2d_scale } from "../../src/engine/graphics/texture/sampler/resize/sampler2d_scale.js";
-import { Sampler2D } from "../../src/engine/graphics/texture/sampler/Sampler2D.js";
 import { BinaryDataType } from "../../src/core/binary/type/BinaryDataType.js";
 import {
     compute_typed_array_constructor_from_data_type
 } from "../../src/core/binary/type/DataType2TypedArrayConstructorMapping.js";
-import { copy_Sampler2D_channel_data } from "../../src/engine/graphics/texture/sampler/copy_Sampler2D_channel_data.js";
+import { Cache } from "../../src/core/cache/Cache.js";
+import Signal from "../../src/core/events/signal/Signal.js";
+import { noop } from "../../src/core/function/noop.js";
+import { strictEquals } from "../../src/core/function/strictEquals.js";
+import { computeStringHash } from "../../src/core/primitives/strings/computeStringHash.js";
+import { ArrayBufferLoader } from "../../src/engine/asset/loaders/ArrayBufferLoader.js";
 import { TextureAssetLoader } from "../../src/engine/asset/loaders/texture/TextureAssetLoader.js";
-import { CanvasView } from "../../src/view/elements/CanvasView.js";
-import convertSampler2D2Canvas from "../../src/engine/graphics/texture/sampler/Sampler2D2Canvas.js";
+import { TerrainLayer } from "../../src/engine/ecs/terrain/ecs/layers/TerrainLayer.js";
+import { TerrainFlags } from "../../src/engine/ecs/terrain/ecs/TerrainFlags.js";
+import { obtainTerrain } from "../../src/engine/ecs/terrain/util/obtainTerrain.js";
+import { EngineConfiguration } from "../../src/engine/EngineConfiguration.js";
+import { EngineHarness } from "../../src/engine/EngineHarness.js";
 import { load_and_set_cubemap_v0 } from "../../src/engine/graphics/load_and_set_cubemap_v0.js";
+import CheckersTextureURI from "../../src/engine/graphics/texture/CheckersTextureURI.js";
+import { copy_Sampler2D_channel_data } from "../../src/engine/graphics/texture/sampler/copy_Sampler2D_channel_data.js";
+import { sampler2d_scale } from "../../src/engine/graphics/texture/sampler/resize/sampler2d_scale.js";
+import { Sampler2D } from "../../src/engine/graphics/texture/sampler/Sampler2D.js";
+import convertSampler2D2Canvas from "../../src/engine/graphics/texture/sampler/Sampler2D2Canvas.js";
+import { NativeListController } from "../../src/view/controller/controls/NativeListController.js";
+import { CanvasView } from "../../src/view/elements/CanvasView.js";
+import EmptyView from "../../src/view/elements/EmptyView.js";
 
 const engineHarness = new EngineHarness();
 

package/src/core/cache/Cache.js

@@ -1,9 +1,11 @@
 import { assert } from "../assert.js";
+import { collectIteratorValueToArray } from "../collection/collectIteratorValueToArray.js";
 import { HashMap } from "../collection/map/HashMap.js";
 import Signal from "../events/signal/Signal.js";
-import { returnOne, returnZero, strictEquals } from "../function/Functions.js";
+import { returnOne } from "../function/returnOne.js";
+import { returnZero } from "../function/returnZero.js";
+import { strictEquals } from "../function/strictEquals.js";
 import { CacheElement } from "./CacheElement.js";
-import { collectIteratorValueToArray } from "../collection/collectIteratorValueToArray.js";
 
 // TODO validate hashes of held elements to keep them up to date. Keys are assumed to be immutable, but this assumption may be broken by users
 

package/src/core/cache/Cache.spec.js

@@ -1,6 +1,6 @@
 import { jest } from "@jest/globals";
+import { returnOne } from "../function/returnOne.js";
 import { Cache } from "./Cache.js";
-import { returnOne } from "../function/Functions.js";
 
 test("constructor doesn't throw", () => {
     new Cache({});

package/src/core/collection/array/arraySetDiff.js

@@ -1,5 +1,7 @@
-import {strictEquals} from "../../function/Functions.js";
-import {arrayIndexByEquality} from "./arrayIndexByEquality.js";
+import { assert } from "../../assert.js";
+
+import { strictEquals } from "../../function/strictEquals.js";
+import { arrayIndexByEquality } from "./arrayIndexByEquality.js";
 
 /**
  * Compute a diff between two arrays, result is a 3 way split between common items, unique items in `a` array and unique items in `b` array
@@ -10,6 +12,10 @@ import {arrayIndexByEquality} from "./arrayIndexByEquality.js";
  * @returns {{uniqueA:T[], uniqueB:T[], common:T[]}}
  */
 export function arraySetDiff(a, b, equals = strictEquals) {
+    assert.isArray(a, 'a');
+    assert.isArray(b, 'b');
+    assert.isFunction(equals, 'equals');
+
     // TODO we can do this faster if we use a comparator instead of equality and pre-sort the data
     const uniqueA = a.slice();
     const uniqueB = b.slice();

package/src/core/collection/array/computeHashArray.js

@@ -1,9 +1,14 @@
+import { assert } from "../../assert.js";
+
 /**
  * @template T
  * @param {T[]} array
  * @param {function(T):number} elementHashFunction
+ * @returns {number}
  */
 export function computeHashArray(array, elementHashFunction) {
+    assert.isArrayLike(array, 'array');
+    assert.isFunction(elementHashFunction, 'elementHashFunction');
 
     const numArguments = array.length;
     let hash = numArguments;

package/src/core/collection/array/groupArrayBy.js

@@ -1,4 +1,4 @@
-import { returnZero } from "../../function/Functions.js";
+import { returnZero } from "../../function/returnZero.js";
 import { HashMap } from "../map/HashMap.js";
 
 /**

package/src/core/collection/array/weightedRandomFromArray.spec.js

@@ -1,5 +1,7 @@
+import { passThrough } from "../../function/passThrough.js";
+import { returnOne } from "../../function/returnOne.js";
+import { returnZero } from "../../function/returnZero.js";
 import { weightedRandomFromArray } from "./weightedRandomFromArray.js";
-import { passThrough, returnOne, returnZero } from "../../function/Functions.js";
 
 test("empty array", () => {
     expect(weightedRandomFromArray([], returnZero, passThrough, null, 0))

package/src/core/collection/heap/FastBinaryHeap.spec.js

@@ -1,7 +1,8 @@
+import { passThrough } from "../../function/passThrough.js";
+import { returnZero } from "../../function/returnZero.js";
+import { randomIntegerBetween } from "../../math/random/randomIntegerBetween.js";
 import { seededRandom } from "../../math/random/seededRandom.js";
 import BinaryHeap from "./FastBinaryHeap.js";
-import { passThrough, returnZero } from "../../function/Functions.js";
-import { randomIntegerBetween } from "../../math/random/randomIntegerBetween.js";
 
 test("constructor doesn't throw", () => {
     new BinaryHeap(returnZero);

package/src/core/collection/list/List.js

@@ -5,6 +5,7 @@
 
 import { assert } from "../../assert.js";
 import Signal from "../../events/signal/Signal.js";
+import { objectsEqual } from "../../function/objectsEqual.js";
 import { invokeObjectEquals } from "../../model/object/invokeObjectEquals.js";
 import { arrayIndexByEquality } from "../array/arrayIndexByEquality.js";
 import { arraySetDiff } from "../array/arraySetDiff.js";
@@ -190,13 +191,13 @@ class List {
      * Replace the data, replacements is performed surgically, meaning that diff is computed and add/remove operations are performed on the set
      * This method is tailored to work well with visualisation as only elements that's missing from the new set is removed, and only elements that are new to are added
      * Conversely, relevant events are dispatched that can observe. This results in fewer changes required to the visualisation
-     * @param {T[]} newOutput
+     * @param {T[]} new_data
      */
-    patch(newOutput) {
+    patch(new_data) {
 
         const data = this.data;
 
-        const diff = arraySetDiff(data, newOutput, invokeObjectEquals);
+        const diff = arraySetDiff(data, new_data, objectsEqual);
 
         //resolve diff
         const removals = diff.uniqueA;
@@ -213,8 +214,8 @@ class List {
 
         //sort additions by their position in the input
         additions.sort((a, b) => {
-            const ai = this.indexOf(a);
-            const bi = this.indexOf(b);
+            const ai = new_data.indexOf(a);
+            const bi = new_data.indexOf(b);
 
             return bi - ai;
         });
@@ -225,11 +226,11 @@ class List {
             const item = additions[i];
 
             //find where the item is in the input
-            const inputIndex = this.indexOf(item);
+            const inputIndex = new_data.indexOf(item);
 
             let p = 0;
             for (let i = inputIndex - 1; i >= 0; i--) {
-                const prev = this.get(i);
+                const prev = new_data[i];
 
                 //look for previous item in the output
                 const j = this.indexOf(prev);
@@ -949,16 +950,6 @@ class List {
     }
 }
 
-/**
- *
- * @param {Object} a
- * @param {Object} b
- * @returns {boolean}
- */
-function objectsEqual(a, b) {
-    return a === b || (typeof a.equals === "function" && a.equals(b));
-}
-
 function conditionEqualsViaMethod(v) {
     return this === v || this.equals(v);
 }

package/src/core/collection/list/List.spec.js

@@ -94,6 +94,27 @@ test("'set' at the end of the list", () => {
     expect(addHandler).toHaveBeenLastCalledWith(42, 0);
 });
 
+test("'set' replace existing value", () => {
+    const list = new List();
+
+    const fn = jest.fn();
+
+    list.on.removed.add(fn);
+
+    list.set(0, "a");
+
+    expect(fn).not.toHaveBeenCalled();
+
+    list.set(0, "b");
+
+    expect(list.length).toBe(1);
+
+    expect(list.get(0)).toBe("b");
+
+    expect(fn).toHaveBeenCalledTimes(1);
+    expect(fn).toHaveBeenCalledWith("a", 0);
+});
+
 test("add increases length", () => {
     const list = new List();
 
@@ -200,3 +221,37 @@ test("map", () => {
 
     expect(list.map(a => a * 7)).toEqual([7, 14, 21]);
 });
+
+test("patch", () => {
+
+    /**
+     *
+     * @type {List<number>}
+     */
+    const list = new List();
+
+    list.patch([]);
+
+    expect(list.asArray()).toEqual([]);
+    expect(list.length).toBe(0);
+
+    list.patch([1]);
+
+    expect(list.asArray()).toEqual([1]);
+    expect(list.length).toBe(1);
+
+    list.patch([1, 2]);
+
+    expect(list.asArray()).toEqual([1, 2]);
+    expect(list.length).toBe(2);
+
+    list.patch([3, 1, 2]);
+
+    expect(list.asArray()).toEqual([3, 1, 2]);
+    expect(list.length).toBe(3);
+
+    list.patch([3, 1, 2]);
+
+    expect(list.asArray()).toEqual([3, 1, 2]);
+    expect(list.length).toBe(3);
+});

package/src/core/collection/map/HashMap.spec.js

@@ -1,5 +1,7 @@
+import { passThrough } from "../../function/passThrough.js";
+import { returnOne } from "../../function/returnOne.js";
+import { strictEquals } from "../../function/strictEquals.js";
 import { generate_next_linear_congruential_index, HashMap } from "./HashMap.js";
-import { passThrough, returnOne, strictEquals } from "../../function/Functions.js";
 
 /**
  *

package/src/core/collection/set/HashSet.spec.js

@@ -0,0 +1,23 @@
+import { strictEquals } from "../../function/strictEquals.js";
+import { computeStringHash } from "../../primitives/strings/computeStringHash.js";
+import { HashSet } from "./HashSet.js";
+
+test("constructor", () => {
+    const set = new HashSet();
+});
+
+test("add same element twice", () => {
+
+    const set = new HashSet({
+        keyHashFunction: computeStringHash,
+        keyEqualityFunction: strictEquals
+    });
+
+    set.add("a");
+
+    expect(set.size).toBe(1);
+
+    set.add("a");
+
+    expect(set.size).toBe(1);
+});

package/src/core/collection/table/RowFirstTableSpec.js

@@ -13,41 +13,43 @@ import { computeHashArray } from "../array/computeHashArray.js";
 import { isArrayEqualStrict } from "../array/isArrayEqualStrict.js";
 
 /**
+ * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DataView
  * @readonly
  * @enum {string}
  */
 export const DataType2DataViewReaders = {
-    "uint8": "getUint8",
-    "uint16": "getUint16",
-    "uint32": "getUint32",
-    "uint64": "getUint64",
-
-    "int8": "getInt8",
-    "int16": "getInt16",
-    "int32": "getInt32",
-    "int64": "getInt64",
-
-    "float32": "getFloat32",
-    "float64": "getFloat64"
+    [BinaryDataType.Uint8]: "getUint8",
+    [BinaryDataType.Uint16]: "getUint16",
+    [BinaryDataType.Uint32]: "getUint32",
+    [BinaryDataType.Uint64]: "getUint64",
+
+    [BinaryDataType.Int8]: "getInt8",
+    [BinaryDataType.Int16]: "getInt16",
+    [BinaryDataType.Int32]: "getInt32",
+    [BinaryDataType.Int64]: "getInt64",
+
+    [BinaryDataType.Float32]: "getFloat32",
+    [BinaryDataType.Float64]: "getFloat64"
 };
 
 /**
+ * @see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DataView
  * @readonly
  * @enum {string}
  */
 export const DataType2DataViewWriters = {
-    "uint8": "setUint8",
-    "uint16": "setUint16",
-    "uint32": "setUint32",
-    "uint64": "setUint64",
-
-    "int8": "setInt8",
-    "int16": "setInt16",
-    "int32": "setInt32",
-    "int64": "setInt64",
-
-    "float32": "setFloat32",
-    "float64": "setFloat64"
+    [BinaryDataType.Uint8]: "setUint8",
+    [BinaryDataType.Uint16]: "setUint16",
+    [BinaryDataType.Uint32]: "setUint32",
+    [BinaryDataType.Uint64]: "setUint64",
+
+    [BinaryDataType.Int8]: "setInt8",
+    [BinaryDataType.Int16]: "setInt16",
+    [BinaryDataType.Int32]: "setInt32",
+    [BinaryDataType.Int64]: "setInt64",
+
+    [BinaryDataType.Float32]: "setFloat32",
+    [BinaryDataType.Float64]: "setFloat64"
 };
 
 /**
@@ -67,10 +69,14 @@ function genRowReader(types, endianType = EndianType.BigEndian) {
 
         const type = types[i];
 
+        const reader_name = DataType2DataViewReaders[type];
+
+        assert.isString(reader_name, 'reader_name');
+
         const littleEndianFlag = endianType === EndianType.BigEndian ? 'false' : 'true';
 
         // See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DataView
-        lines.push(`result[${i}] = dataView.${DataType2DataViewReaders[type]}(${offset} + byteOffset, ${littleEndianFlag});`);
+        lines.push(`result[${i}] = dataView.${reader_name}(${offset} + byteOffset, ${littleEndianFlag});`);
 
         offset += DataTypeByteSizes[type];
 
@@ -100,10 +106,14 @@ function genRowWriter(types, endianType = EndianType.BigEndian) {
     for (let i = 0; i < numTypes; i++) {
         const type = types[i];
 
+        const writer_name = DataType2DataViewWriters[type];
+
+        assert.isString(writer_name, 'writer_name');
+
         const littleEndianFlag = endianType === EndianType.BigEndian ? 'false' : 'true';
 
         // See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DataView
-        lines.push(`dataView.${DataType2DataViewWriters[type]}(${offset} + byteOffset, record[${i}], ${littleEndianFlag});`);
+        lines.push(`dataView.${writer_name}(${offset} + byteOffset, record[${i}], ${littleEndianFlag});`);
 
         offset += DataTypeByteSizes[type];
     }
@@ -267,7 +277,7 @@ export class RowFirstTableSpec {
     }
 
     /**
-     *
+     * Produces an immutable spec, either creates a new spec or retrieves a cached version
      * @param {BinaryDataType[]} types
      * @param {EndianType} [endianType]
      * @returns {RowFirstTableSpec}
@@ -276,21 +286,25 @@ export class RowFirstTableSpec {
         //compute hash
         const hash = types.join('.') + ':' + endianType;
 
-        const cachedValue = cache.get(hash);
+        const cachedValue = SPEC_CACHE.get(hash);
 
         if (cachedValue !== null) {
             return cachedValue;
         }
 
         const newValue = new RowFirstTableSpec(types);
-        cache.put(hash, newValue);
+        SPEC_CACHE.put(hash, newValue);
 
         return newValue;
     }
 }
 
 
-const cache = new Cache({
+/**
+ * @readonly
+ * @type {Cache<string,RowFirstTableSpec>}
+ */
+const SPEC_CACHE = new Cache({
     keyHashFunction: computeStringHash
 });
 

package/src/core/events/signal/signal_aggregate_by_time_window.js

@@ -1,4 +1,4 @@
-import { passThrough } from "../../function/Functions.js";
+import { passThrough } from "../../function/passThrough.js";
 import Signal from "./Signal.js";
 
 /**

package/src/core/fsm/simple/SimpleStateMachine.js

@@ -1,6 +1,7 @@
 import { dispatchViaProxy, findSignalHandlerIndexByHandle } from "../../events/signal/Signal.js";
 import { SignalHandler } from "../../events/signal/SignalHandler.js";
-import { noop } from "../../function/Functions.js";
+
+import { noop } from "../../function/noop.js";
 import { SimpleStateMachineDescription } from "./SimpleStateMachineDescription.js";
 
 export class SimpleStateMachine {

package/src/core/function/chain.js

@@ -0,0 +1,19 @@
+/**
+ *
+ * @param {...function} processes
+ * @returns {function(*): *}
+ */
+export function chain(...processes) {
+    const numProcesses = processes.length;
+
+    return function chain(v) {
+        let result = v;
+
+        for (let i = 0; i < numProcesses; i++) {
+            const process = processes[i];
+            result = process(result);
+        }
+
+        return result;
+    }
+}

package/src/core/function/makeReturnValue.js

@@ -0,0 +1,8 @@
+/**
+ * @template T
+ * @param {T} v
+ * @returns {function():T}
+ */
+export function makeReturnValue(v) {
+    return () => v;
+}