@woosh/meep-engine 2.47.38 → 2.47.40

package/package.json

@@ -5,7 +5,7 @@
   "description": "Fully featured ECS game engine written in JavaScript",
   "type": "module",
   "author": "Alexander Goldring",
-  "version": "2.47.38",
+  "version": "2.47.40",
   "main": "build/meep.module.js",
   "module": "build/meep.module.js",
   "exports": {
@@ -52,6 +52,7 @@
     "@rollup/plugin-strip": "3.0.2",
     "@types/three": "^0.135.0",
     "babel-jest": "26.6.3",
+    "jest": "26.6.3",
     "rollup": "3.16.0"
   },
   "keywords": [

package/src/core/model/node-graph/json/deserializeNodeGraphFromJSON.js

@@ -58,19 +58,35 @@ export function deserializeNodeGraphFromJSON({
     for (let i = 0; i < j_descriptions.length; i++) {
         const jDescription = j_descriptions[i];
 
-        const nodeDescription = abstractJSONDeserializer(jDescription, deserializers);
+        const loaded_node = abstractJSONDeserializer(jDescription, deserializers);
 
         if (
-            typeof nodeDescription !== "object"
-            || nodeDescription.isNodeDescription !== true
+            typeof loaded_node !== "object"
+            || loaded_node.isNodeDescription !== true
         ) {
             throw new Error(`Deserialized object was expected to be a NodeDescription, instead got something else. Source JSON[${i}]: ${jDescription}`);
         }
 
-        descriptions[i] = nodeDescription;
+        const existing_node = node_registry.getNode(loaded_node.id);
+
+        if (existing_node === undefined) {
+
+            descriptions[i] = loaded_node;
+
+            // make sure to register the node
+            node_registry.addNode(loaded_node);
+
+        } else {
+
+            if (!existing_node.equals(loaded_node)) {
+                throw new Error(`Loaded node does not match the one in the registry. Loaded: ${loaded_node}, Existing: ${existing_node}`);
+            }
+
+            // re-use node from registry
+            descriptions[i] = existing_node;
+
+        }
 
-        // make sure to register the node
-        node_registry.addNode(nodeDescription);
     }
 
     // parse nodes

package/src/core/model/node-graph/node/NodeDescription.js

@@ -5,6 +5,7 @@ import { Port } from "./Port.js";
 import { PortDirection } from "./PortDirection.js";
 import Signal from "../../../events/signal/Signal.js";
 import { invokeObjectToJSON } from "../../object/invokeObjectToJSON.js";
+import { isArrayEqual } from "../../../collection/array/isArrayEqual.js";
 
 
 /**
@@ -88,7 +89,7 @@ export class NodeDescription {
      * This gives us an opportunity to add/change ports based on current state of a specific node instance
      * @param {NodeInstance} instance
      */
-    configureNode(instance){
+    configureNode(instance) {
         // override in subclasses as necessary
     }
 
@@ -326,6 +327,22 @@ export class NodeDescription {
         return this.ports
     }
 
+    /**
+     *
+     * @param {NodeDescription} other
+     * @returns {boolean}
+     */
+    equals(other) {
+        return this.id === other.id
+            && this.name === other.name
+            && isArrayEqual(this.ports, other.ports)
+            ;
+    }
+
+    toString() {
+        return `NodeDescription[id=${this.id}, name='${this.name}']`;
+    }
+
     toJSON() {
         return {
             id: this.id,

package/src/core/model/node-graph/node/NodeRegistry.js

@@ -49,7 +49,7 @@ export class NodeRegistry {
 
         if (existing_node !== undefined) {
 
-            if (existing_node !== node) {
+            if (existing_node !== node && !existing_node.equals(node)) {
                 // node with the same ID exists and is not the same node
                 throw new Error(`A different node with the same ID ${node.id} is already registered. IDs of all nodes in the registry must be unique`);
             }