@woosh/meep-engine 2.43.48 → 2.43.50

package/core/collection/list/List.js

@@ -92,6 +92,21 @@ List.prototype.add = function (el) {
     return this;
 };
 
+/**
+ *
+ * @param {T} el
+ * @return {boolean}
+ */
+List.prototype.addUnique = function (el) {
+    if (this.contains(el)) {
+        return false;
+    }
+
+    this.add(el);
+
+    return true;
+}
+
 /**
  * Insert element at a specific position into the list
  * @param {number} index

package/core/model/node-graph/DataType.js

@@ -9,7 +9,7 @@ export class DataType {
         this.id = 0;
 
         /**
-         *
+         * Name is not guaranteed to be unique, it is intended mainly for debug purposes
          * @type {string}
          */
         this.name = "";
@@ -55,6 +55,9 @@ export class DataType {
      * @returns {DataType}
      */
     static from(id, name) {
+        assert.isNonNegativeInteger(id, 'id');
+        assert.isString(name, 'name');
+
         const r = new DataType();
 
         r.id = id;
@@ -63,3 +66,10 @@ export class DataType {
         return r;
     }
 }
+
+
+/**
+ * @readonly
+ * @type {boolean}
+ */
+DataType.prototype.isDataType = true;

package/core/model/node-graph/NodeGraph.js

@@ -243,6 +243,10 @@ export class NodeGraph {
 
         this.connections.add(connection);
 
+        // add connection links to the nodes
+        sourceNodeInstance.connections.addUnique(connection);
+        targetNodeInstance.connections.addUnique(connection);
+
         return id;
     }
 
@@ -261,6 +265,10 @@ export class NodeGraph {
 
         this.connections.removeOneOf(connection);
 
+        // remove from end-point nodes
+        connection.source.instance.connections.removeOneOf(connection);
+        connection.target.instance.connections.removeOneOf(connection);
+
         return true;
     }
 

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

@@ -110,6 +110,7 @@ export class NodeDescription {
      * @returns {NodeParameterDescription}
      */
     getParameter(id) {
+        assert.isNonNegativeInteger(id, 'id');
 
         const paramDescriptors = this.parameters;
 
@@ -133,6 +134,13 @@ export class NodeDescription {
      * @param {PortDirection} direction
      */
     createPort(type, name, direction) {
+        assert.defined(type, 'type');
+        assert.notNull(type, 'type');
+        assert.equal(type.isDataType, true, 'type.isDataType !== true');
+
+        assert.defined(direction, 'direction');
+        assert.isString(name, 'name');
+
         const port = new Port();
 
         const id = pickNewSetId(this.ports);
@@ -154,6 +162,8 @@ export class NodeDescription {
      * @returns {Port|null}
      */
     getPortById(id) {
+        assert.isNonNegativeInteger(id, 'id');
+
         for (const port of this.ports) {
             if (port.id === id) {
                 return port;
@@ -170,6 +180,7 @@ export class NodeDescription {
      * @return {Port|undefined}
      */
     getPortByName(name) {
+        assert.isString(name, 'name');
 
         for (const port of this.ports) {
             if (port.name === name) {

package/package.json

@@ -5,7 +5,7 @@
   "productName": "Meep",
   "description": "production-ready JavaScript game engine based on Entity Component System Architecture",
   "author": "Alexander Goldring",
-  "version": "2.43.48",
+  "version": "2.43.50",
   "dependencies": {
     "gl-matrix": "3.4.3",
     "fast-levenshtein": "2.0.6",