updated README

.gitignore

@@ -18,4 +18,3 @@ dist/
 *.ps
 *.toc
 *.gz
-index.html

README.md

@@ -4,7 +4,7 @@ A pure JavaScript implementation of the ITA project's CEStore - called CENode. C
 
 Please visit the project's [home page](http://cenode.io) for more information and for documentation.
 
-**We recommend beginners check out the [Getting Started Guide](https://github.com/willwebberley/CENode/wiki/Getting-Started-Guide) before continuing.**
+See also the [Getting Started Tutorial](https://github.com/flyingsparx/CENode/blob/master/docs/getting_started.md).
 
 ## Getting started
 
@@ -12,7 +12,7 @@ CENode can be imported into your Node apps or run in a browser. Either way, you
 
 Then add CENode to your project using NPM:
 ```
-npm install cenode
+npm install
 ```
 
 If using CENode in a webpage, then include it (and models, if necessary) in script tags:
@@ -33,13 +33,13 @@ const CEModels = require('cenode/models'); // if requred
 const node = new CENode(CEModels.core);
 ```
 
-See the [Wiki](https://github.com/willwebberley/CENode/wiki) for further guides and the API reference.
+See the [Wiki](https://github.com/flyingsparx/CENode/wiki) for further guides and the API reference.
 
 ## Testing
 
 Clone the repository
 ```
-git clone git@github.com:willwebberley/CENode.git
+git clone git@github.com:flyingsparx/CENode.git
 ```
 
 Install the necessary dev dependencies.
@@ -54,7 +54,7 @@ npm test
 
 ## More Information
 
-See the CENode [Wiki](https://github.com/willwebberley/CENode/wiki) for more information, guides, and the API reference.
+See the CENode [Wiki](https://github.com/flyingsparx/CENode/wiki) for more information, guides, and the API reference.
 
 
 ## Licence

models/core.js

@@ -18,7 +18,6 @@ module.exports = [
   "conceptualise a ~ policy ~ P that is an entity and has the value V as ~ enabled ~ and has the agent A as ~ target ~",
   "conceptualise a ~ tell policy ~ P that is a policy",
   "conceptualise an ~ ask policy ~ P that is a policy",
-  "conceptualise a ~ gist policy ~ P that is a policy",
   "conceptualise a ~ listen policy ~ P that is a policy",
   "conceptualise a ~ listen onbehalfof policy ~ P that is a policy",
   "conceptualise a ~ forwardall policy ~ P that is a policy and has the timestamp T as ~ start time ~ and has the value V as ~ all agents ~",

package.json

@@ -1,13 +1,13 @@
 {
   "name": "cenode",
-  "version": "3.0.12",
+  "version": "3.0.2",
   "description": "A pure JavaScript implementation of the ITA project's CEStore - called CENode. CENode is able to understand the basic sentence types parsed by the CEStore, such as conceptualising and instance creation and modification.",
   "homepage": "http://cenode.io",
   "license": "Apache-2.0",
   "author": "Will Webberley & Alun Preece",
   "repository": {
     "type": "git",
-    "url": "https://github.com/willwebberley/CENode"
+    "url": "https://github.com/flyingsparx/CENode"
   },
   "files": [
     "src",

src/CEAgent.js

@@ -43,9 +43,9 @@ class CEAgent {
 
   getInstance() {
     const instances = this.node.getInstances('agent');
-    for (const instance of instances) {
-      if (instance.name.toLowerCase() === name.toLowerCase()) {
-        return instance;
+    for (let i = 0; i < instances.length; i += 1) {
+      if (instances[i].name.toLowerCase() === name.toLowerCase()) {
+        return instances[i];
       }
     }
     return null;

src/CEConcept.js

@@ -19,14 +19,6 @@
 class CEConcept {
 
   constructor(node, name, source) {
-    if (!name) {
-      return;
-    }
-    for (const concept of node.concepts) {
-      if (concept.name.toLowerCase() === name.toLowerCase()) {
-        return;
-      }
-    }
     this.name = name;
     this.source = source;
     this.id = node.newConceptId();
@@ -38,101 +30,99 @@ class CEConcept {
     node.concepts.push(this);
     this.node.conceptDict[this.id] = this;
 
-    if (isNaN(name[0])) {
-      const concept = this;
-      Object.defineProperty(node.concepts, name.toLowerCase().replace(/ /g, '_'), {
-        get() {
-          return concept;
-        },
-        configurable: true,
-      });
-    }
+    const concept = this;
+    Object.defineProperty(node.concepts, name.toLowerCase().replace(/ /g, '_'), {
+      get() {
+        return concept;
+      },
+      configurable: true,
+    });
   }
 
   get instances() {
-    const array = [];
-    for (const instance of this.node.instances) {
-      if (instance.concept.id === this.id) {
-        array.push(instance);
+    const instances = [];
+    for (let i = 0; i < this.node.instances.length; i += 1) {
+      if (this.node.instances[i].type.id === this.id) {
+        instances.push(this.node.instances[i]);
       }
     }
-    return array;
+    return instances;
   }
 
   get allInstances() {
     const allConcepts = this.descendants.concat(this);
-    const array = [];
-    for (const instance of this.node.instances) {
-      for (const concept of allConcepts) {
-        if (instance.concept.id === concept.id) {
-          array.push(instance);
+    const instances = [];
+    for (let i = 0; i < this.node.instances.length; i += 1) {
+      for (let j = 0; j < allConcepts.length; j += 1) {
+        if (this.node.instances[i].type.id === allConcepts[j].id) {
+          instances.push(this.node.instances[i]);
         }
       }
     }
-    return array;
+    return instances;
   }
 
   get parents() {
-    const array = [];
-    for (const id of this.parentIds) {
-      array.push(this.node.getConceptById(id));
+    const p = [];
+    for (let i = 0; i < this.parentIds.length; i += 1) {
+      p.push(this.node.getConceptById(this.parentIds[i]));
     }
-    return array;
+    return p;
   }
 
   get ancestors() {
-    const array = [];
+    const parents = [];
     const stack = [];
-    for (const parent of this.parents) {
-      stack.push(parent);
+    for (let i = 0; i < this.parentIds.length; i += 1) {
+      stack.push(this.parents[i]);
     }
     while (stack.length > 0) {
       const current = stack.pop();
-      array.push(current);
-      for (const parent of current.parents) {
-        stack.push(parent);
+      parents.push(current);
+      for (let i = 0; i < current.parents.length; i += 1) {
+        stack.push(current.parents[i]);
       }
     }
-    return array;
+    return parents;
   }
 
   get children() {
-    const array = [];
-    for (const concept of this.node.concepts) {
-      for (const parent of concept.parents) {
-        if (parent.id === this.id) {
-          array.push(concept);
+    const children = [];
+    for (let i = 0; i < this.node.concepts.length; i += 1) {
+      for (let j = 0; j < this.node.concepts[i].parents.length; j += 1) {
+        if (this.node.concepts[i].parents[j].id === this.id) {
+          children.push(this.node.concepts[i]);
         }
       }
     }
-    return array;
+    return children;
   }
 
   get descendants() {
-    const array = [];
+    const children = [];
     const stack = [];
-    for (const child of this.children) {
-      stack.push(child);
+    for (let i = 0; i < this.children.length; i += 1) {
+      stack.push(this.children[i]);
     }
     while (stack.length > 0) {
       const current = stack.pop();
-      array.push(current);
+      children.push(current);
       const currentChildren = current.children;
       if (currentChildren) {
-        for (const child of currentChildren) {
-          stack.push(child);
+        for (let i = 0; i < currentChildren.length; i += 1) {
+          stack.push(currentChildren[i]);
         }
       }
     }
-    return array;
+    return children;
   }
 
   get relationships() {
     const rels = [];
-    for (const id of this.relationshipIds) {
+    for (let i = 0; i < this.relationshipIds.length; i += 1) {
       const relationship = {};
-      relationship.label = id.label;
-      relationship.concept = this.node.getConceptById(id.target);
+      relationship.label = this.relationshipIds[i].label;
+      relationship.concept = this.node.getConceptById(this.relationshipIds[i].target);
       rels.push(relationship);
     }
     return rels;
@@ -140,61 +130,60 @@ class CEConcept {
 
   get values() {
     const vals = [];
-    for (const val of this.valueIds) {
+    for (let i = 0; i < this.valueIds.length; i += 1) {
       const value = {};
-      value.label = val.label;
-      value.concept = val.type && this.node.getConceptById(val.type);
+      value.label = this.valueIds[i].label;
+      if (this.valueIds[i].conceptId === 0) {
+        value.concept = this.valueIds[i].typeName;
+      } else {
+        value.concept = this.node.getConceptById(this.valueIds[i].type);
+      }
       vals.push(value);
     }
     return vals;
   }
 
-  getCE(isModification) {
-    let ce = '';
-    if (isModification) {
-      ce += `conceptualise the ${this.name} ${this.name.charAt(0).toUpperCase()}`;
-    } else {
-      ce += `conceptualise a ~ ${this.name} ~ ${this.name.charAt(0).toUpperCase()}`;
-    }
-    if (!isModification && (this.parentIds.length > 0 || this.valueIds.length > 0 || this.relationshipIds.length > 0)) {
+  get ce() {
+    let ce = `conceptualise a ~ ${this.name} ~ ${this.name.charAt(0).toUpperCase()}`;
+    if (this.parentIds.length > 0 || this.valueIds.length > 0 || this.relationshipIds.length > 0) {
       ce += ' that';
     }
     if (this.parentIds.length > 0) {
-      for (let i = 0; i < this.parents.length; i += 1) {
-        ce += ` is a ${this.parents[i].name}`;
-        if (i < this.parents.length - 1) { ce += ' and'; }
+      for (let i = 0; i < this.parentIds.length; i += 1) {
+        ce += ` is a ${this.parentIds[i].name}`;
+        if (i < this.parentIds.length - 1) { ce += ' and'; }
       }
     }
-    const facts = [];
+    let facts = [];
     const alph = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O'];
     for (let i = 0; i < this.valueIds.length; i += 1) {
       if (this.valueIds[i].type === 0) {
-        facts.push(`has the value ${alph[i]} as ~ ${this.valueIds[i].label} ~`);
+        facts.push(`has the value ${alph[i]} as ${this.valueIds[i].label}`);
       } else {
         const valType = this.node.getConceptById(this.valueIds[i].type);
-        facts.push(`has the ${valType.name} ${valType.name.charAt(0).toUpperCase()} as ~ ${this.valueIds[i].label} ~`);
+        facts.push(`has the ${valType.name} ${valType.name.charAt(0).toUpperCase()} as ${this.valueIds[i].label}`);
       }
     }
-    for (let i = 0; i < this.relationshipIds.length; i += 1) {
-      const relType = this.node.getConceptById(this.relationshipIds[i].target);
-      facts.push(`~ ${this.relationshipIds[i].label} ~ the ${relType.name} ${alph[i]}`);
-    }
     if (facts.length > 0) {
       if (this.parentIds.length > 0) { ce += ' and'; }
       ce += ` ${facts.join(' and ')}`;
     }
     ce += '.';
+    if (this.relationshipIds.length > 0) {
+      facts = [];
+      ce += `\nconceptualise the ${this.name} ${this.name.charAt(0).toUpperCase()}`;
+      for (let i = 0; i < this.relationshipIds.length; i += 1) {
+        const relType = this.node.getConceptById(this.relationshipIds[i].target);
+        facts.push(`~ ${this.relationshipIds[i].label} ~ the ${relType.name} ${alph[i]}`);
+      }
+      if (facts.length > 0) {
+        if (this.parentIds.length > 0 || this.valueIds.length > 0) { ce += ' and'; }
+        ce += ` ${facts.join(' and ')}.`;
+      }
+    }
     return ce;
   }
 
-  get creationCE() {
-    return `conceptualise a ~ ${this.name} ~ ${this.name.charAt(0).toUpperCase()}`;
-  }
-
-  get ce() {
-    return this.getCE();
-  }
-
   get gist() {
     let gist = '';
     if (this.parentIds.length > 0) { gist += `A ${this.name}`; }
@@ -230,14 +219,12 @@ class CEConcept {
     value.label = label;
     value.type = typeof type === 'number' ? type : type.id;
     this.valueIds.push(value);
-    if (isNaN(label[0])) {
-      Object.defineProperty(this, label.toLowerCase().replace(/ /g, '_'), {
-        get() {
-          return type === 0 ? 'value' : type;
-        },
-        configurable: true,
-      });
-    }
+    Object.defineProperty(this, label.toLowerCase().replace(/ /g, '_'), {
+      get() {
+        return type === 0 ? 'value' : type;
+      },
+      configurable: true,
+    });
   }
 
   addRelationship(label, target, source) {
@@ -246,14 +233,12 @@ class CEConcept {
     relationship.label = label;
     relationship.target = target.id;
     this.relationshipIds.push(relationship);
-    if (isNaN(label[0])) {
-      Object.defineProperty(this, label.toLowerCase().replace(/ /g, '_'), {
-        get() {
-          return target;
-        },
-        configurable: true,
-      });
-    }
+    Object.defineProperty(this, label.toLowerCase().replace(/ /g, '_'), {
+      get() {
+        return target;
+      },
+      configurable: true,
+    });
   }
 
   addParent(parentConcept) {
@@ -263,8 +248,8 @@ class CEConcept {
   }
 
   addSynonym(synonym) {
-    for (const currentSynonym of this.synonyms) {
-      if (currentSynonym.toLowerCase() === synonym.toLowerCase()) {
+    for (let i = 0; i < this.synonyms.length; i += 1) {
+      if (this.synonyms[i].toLowerCase() === synonym.toLowerCase()) {
         return;
       }
     }

src/CEInstance.js

@@ -19,14 +19,10 @@
 class CEInstance {
 
   constructor(node, type, name, source) {
-    if (!type || !name) {
+    if (!type) {
       return;
     }
-    for (const instance of node.instances) {
-      if (instance.name.toLowerCase() === name.toLowerCase() && type.id === instance.concept.id) {
-        return;
-      }
-    }
+
     this.node = node;
     this.name = name;
     this.source = source;
@@ -42,28 +38,26 @@ class CEInstance {
     node.instances.push(this);
     this.node.instanceDict[this.id] = this;
 
-    if (isNaN(name[0])) {
-      const instance = this;
-      const helperName = name.toLowerCase().replace(/ /g, '_').replace(/'/g, '');
-      Object.defineProperty(node.instances, helperName, {
-        get() {
-          return instance;
-        },
-        configurable: true,
-      });
-      Object.defineProperty(type, helperName, {
-        get() {
-          return instance;
-        },
-        configurable: true,
-      });
-    }
+    const instance = this;
+    Object.defineProperty(node.instances, name.toLowerCase().replace(/ /g, '_').replace(/'/g, ''), {
+      get() {
+        return instance;
+      },
+      configurable: true,
+    });
+
+    Object.defineProperty(type, name.toLowerCase(), {
+      get() {
+        return instance;
+      },
+      configurable: true,
+    });
   }
 
   get type() {
-    for (const concept of this.node.concepts) {
-      if (concept.id === this.concept.id) {
-        return concept;
+    for (let i = 0; i < this.node.concepts.length; i += 1) {
+      if (this.node.concepts[i].id === this.concept.id) {
+        return this.node.concepts[i];
       }
     }
     return null;
@@ -71,11 +65,11 @@ class CEInstance {
 
   get relationships() {
     const rels = [];
-    for (const id of this.relationshipIds) {
+    for (let i = 0; i < this.relationshipIds.length; i += 1) {
       const relationship = {};
-      relationship.label = id.label;
-      relationship.source = id.source;
-      relationship.instance = this.node.getInstanceById(id.targetId);
+      relationship.label = this.relationshipIds[i].label;
+      relationship.source = this.relationshipIds[i].source;
+      relationship.instance = this.node.getInstanceById(this.relationshipIds[i].targetId);
       rels.push(relationship);
     }
     return rels;
@@ -83,14 +77,14 @@ class CEInstance {
 
   get values() {
     const vals = [];
-    for (const id of this.valueIds) {
+    for (let i = 0; i < this.valueIds.length; i += 1) {
       const value = {};
-      value.label = id.label;
-      value.source = id.source;
-      if (id.conceptId === 0) {
-        value.instance = id.typeName;
+      value.label = this.valueIds[i].label;
+      value.source = this.valueIds[i].source;
+      if (this.valueIds[i].conceptId === 0) {
+        value.instance = this.valueIds[i].typeName;
       } else {
-        value.instance = this.node.getInstanceById(id.conceptId);
+        value.instance = this.node.getInstanceById(this.valueIds[i].conceptId);
       }
       vals.push(value);
     }
@@ -102,19 +96,15 @@ class CEInstance {
   }
 
   getPossibleProperties() {
-    let ancestorInstances = this.concept.ancestors;
+    const ancestorInstances = this.concept.ancestors;
     ancestorInstances.push(this.concept);
-    for (const subConcept of this.subConcepts) {
-      ancestorInstances.push(subConcept);
-      ancestorInstances = ancestorInstances.concat(subConcept.ancestors);
-    }
     const properties = { values: [], relationships: [] };
-    for (const ancestor of ancestorInstances) {
-      for (const value of ancestor.values) {
-        properties.values.push(value.label.toLowerCase());
+    for (let i = 0; i < ancestorInstances.length; i += 1) {
+      for (let j = 0; j < ancestorInstances[i].values.length; j += 1) {
+        properties.values.push(ancestorInstances[i].values[j].label.toLowerCase());
       }
-      for (const relationship of ancestor.relationships) {
-        properties.relationships.push(relationship.label.toLowerCase());
+      for (let j = 0; j < ancestorInstances[i].relationships.length; j += 1) {
+        properties.relationships.push(ancestorInstances[i].relationships[j].label.toLowerCase());
       }
     }
     return properties;
@@ -133,7 +123,7 @@ class CEInstance {
       this.valueIds.push(value);
       const valueNameField = label.toLowerCase().replace(/ /g, '_');
 
-      if (this.reservedFields.indexOf(valueNameField) === -1 && isNaN(valueNameField[0])) {
+      if (this.reservedFields.indexOf(valueNameField) === -1) {
         Object.defineProperty(this, valueNameField, {
           get() {
             return value.conceptId === 0 ? value.typeName : this.node.getInstanceById(value.conceptId);
@@ -145,9 +135,9 @@ class CEInstance {
           Object.defineProperty(this, `${valueNameField}s`, {
             get() {
               const instances = [];
-              for (const id of this.valueIds) {
-                if (id.label.toLowerCase().replace(/ /g, '_') === valueNameField) {
-                  instances.push(id.conceptId === 0 ? id.typeName : this.node.getInstanceById(id.conceptId));
+              for (let i = 0; i < this.valueIds.length; i += 1) {
+                if (this.valueIds[i].label.toLowerCase().replace(/ /g, '_') === valueNameField) {
+                  instances.push(this.valueIds[i].conceptId === 0 ? this.valueIds[i].typeName : this.node.getInstanceById(this.valueIds[i].conceptId));
                 }
               }
               return instances;
@@ -172,7 +162,7 @@ class CEInstance {
       this.relationshipIds.push(relationship);
       const relNameField = label.toLowerCase().replace(/ /g, '_');
 
-      if (this.reservedFields.indexOf(relNameField) === -1 && isNaN(relNameField[0])) {
+      if (this.reservedFields.indexOf(relNameField) === -1) {
         Object.defineProperty(this, relNameField, {
           get() {
             return this.node.getInstanceById(relationship.targetId);
@@ -184,9 +174,9 @@ class CEInstance {
           Object.defineProperty(this, `${relNameField}s`, {
             get() {
               const instances = [];
-              for (const id of this.relationshipIds) {
-                if (id.label.toLowerCase().replace(/ /g, '_') === relNameField) {
-                  instances.push(this.node.getInstanceById(id.targetId));
+              for (let i = 0; i < this.relationshipIds.length; i += 1) {
+                if (this.relationshipIds[i].label.toLowerCase().replace(/ /g, '_') === relNameField) {
+                  instances.push(this.node.getInstanceById(this.relationshipIds[i].targetId));
                 }
               }
               return instances;
@@ -205,19 +195,17 @@ class CEInstance {
     if (!synonym || !synonym.length) {
       return null;
     }
-    for (const checkSynonym of this.synonyms) {
-      if (checkSynonym.toLowerCase() === synonym.toLowerCase()) {
+    for (let i = 0; i < this.synonyms.length; i += 1) {
+      if (this.synonyms[i].toLowerCase() === synonym.toLowerCase()) {
         return null;
       }
     }
     this.synonyms.push(synonym);
-    if (isNaN(synonym[0])) {
-      Object.defineProperty(this, synonym.toLowerCase().replace(/ /g, '_'), {
-        get() {
-          return this;
-        },
-      });
-    }
+    Object.defineProperty(this, synonym.toLowerCase().replace(/ /g, '_'), {
+      get() {
+        return this;
+      },
+    });
     return null;
   }
 
@@ -262,79 +250,69 @@ class CEInstance {
     return onlyOne ? null : properties;
   }
 
-  getCE(isModification) {
+  get ce() {
     const concept = this.concept;
     if (!concept) { return ''; }
-
-    let ce = '';
-    if (isModification) {
-      ce += `the ${concept.name} '${this.name}'`;
-    } else {
-      ce += `there is a ${concept.name} named '${this.name}'`;
-    }
+    let ce = `there is a ${concept.name} named '${this.name}'`;
     const facts = [];
     for (const subConcept of this.subConcepts) {
       facts.push(`is a ${subConcept.name}`);
     }
-    for (const id of this.valueIds) {
-      if (id.conceptId === 0) {
-        facts.push(`has '${id.typeName.replace(/'/g, "\\'")}' as ${id.label}`);
+    for (let i = 0; i < this.valueIds.length; i += 1) {
+      const value = this.valueIds[i];
+      if (value.conceptId === 0) {
+        facts.push(`has '${value.typeName.replace(/'/g, "\\'")}' as ${value.label}`);
       } else {
-        const valueInstance = this.node.getInstanceById(id.conceptId);
+        const valueInstance = this.node.getInstanceById(value.conceptId);
         const valueConcept = valueInstance.type;
-        facts.push(`has the ${valueConcept.name} '${valueInstance.name}' as ${id.label}`);
+        facts.push(`has the ${valueConcept.name} '${valueInstance.name}' as ${value.label}`);
       }
     }
-    for (const id of this.relationshipIds) {
-      const relationshipInstance = this.node.getInstanceById(id.targetId);
+    for (let i = 0; i < this.relationshipIds.length; i += 1) {
+      const relationship = this.relationshipIds[i];
+      const relationshipInstance = this.node.getInstanceById(relationship.targetId);
       const relationshipConcept = relationshipInstance.type;
-      facts.push(`${id.label} the ${relationshipConcept.name} '${relationshipInstance.name}'`);
+      facts.push(`${relationship.label} the ${relationshipConcept.name} '${relationshipInstance.name}'`);
     }
-    if (facts.length > 0) { ce += `${!isModification && ' that'} ${facts.join(' and ')}`; }
+    if (facts.length > 0) { ce += ` that ${facts.join(' and ')}`; }
     return `${ce}.`;
   }
 
-  get creationCE() {
-    return `there is a ${this.concept && this.concept.name} named '${this.name}'`;
-  }
-
-  get ce() {
-    return this.getCE();
-  }
-
   get gist() {
     const vowels = ['a', 'e', 'i', 'o', 'u'];
     const concept = this.concept;
     if (!concept) { return ''; }
     let gist = `${this.name} is`;
-    if (vowels.indexOf(concept.name.toLowerCase()[0]) > -1) { gist += ` an ${concept.name}`; } else { gist += ` a ${concept.name}`; }
-    for (const subConcept of this.subConcepts) {
-      gist += ` and ${vowels.indexOf(subConcept.name.toLowerCase()[0]) > -1 ? 'an' : 'a'} ${subConcept.name}`;
+    if (vowels.indexOf(concept.name.toLowerCase()[0]) > -1) { gist += ` an ${concept.name}.`; } else { gist += ` a ${concept.name}`; }
+    for (let i = 0; i < this.subConcepts.length; i += 1) {
+      gist += ` and a ${this.subConcepts[i].name}`;
     }
     gist += '.';
 
     const facts = {};
     let factFound = false;
-    for (const id of this.valueIds) {
+    for (let i = 0; i < this.valueIds.length; i += 1) {
       factFound = true;
+      const value = this.valueIds[i];
       let fact = '';
-      if (id.conceptId === 0) {
-        fact = `has '${id.typeName.replace(/'/g, "\\'")}' as ${id.label}`;
+      if (value.conceptId === 0) {
+        fact = `has '${value.typeName.replace(/'/g, "\\'")}' as ${value.label}`;
       } else {
-        const valueInstance = this.node.getInstanceById(id.conceptId);
+        const valueInstance = this.node.getInstanceById(value.conceptId);
         const valueConcept = valueInstance.type;
-        fact = `has the ${valueConcept.name} '${valueInstance.name}' as ${id.label}`;
+        fact = `has the ${valueConcept.name} '${valueInstance.name}' as ${value.label}`;
       }
       if (!(fact in facts)) {
         facts[fact] = 0;
       }
       facts[fact] += 1;
     }
-    for (const id of this.relationshipIds) {
+    for (let i = 0; i < this.relationshipIds.length; i += 1) {
       factFound = true;
-      const relationshipInstance = this.node.getInstanceById(id.targetId);
+      const relationship = this.relationshipIds[i];
+      const relationshipInstance = this.node.getInstanceById(relationship.targetId);
       const relationshipConcept = relationshipInstance.type;
-      const fact = `${id.label} the ${relationshipConcept.name} '${relationshipInstance.name}'`;
+      const fact = `${relationship.label} the ${relationshipConcept.name} '${relationshipInstance.name}'`;
       if (!(fact in facts)) {
         facts[fact] = 0;
       }

src/CENode.js

@@ -46,13 +46,13 @@ class CENode {
 
   getConceptByName(name) {
     if (!name) { return null; }
-    for (const concept of this.concepts) {
-      if (concept.name.toLowerCase() === name.toLowerCase()) {
-        return concept;
+    for (let i = 0; i < this.concepts.length; i += 1) {
+      if (this.concepts[i].name.toLowerCase() === name.toLowerCase()) {
+        return this.concepts[i];
       }
-      for (const synonym of concept.synonyms) {
-        if (synonym.toLowerCase() === name.toLowerCase()) {
-          return concept;
+      for (let j = 0; j < this.concepts[i].synonyms.length; j += 1) {
+        if (this.concepts[i].synonyms[j].toLowerCase() === name.toLowerCase()) {
+          return this.concepts[i];
         }
       }
     }
@@ -63,18 +63,15 @@ class CENode {
     return this.instanceDict[id];
   }
 
-  getInstanceByName(name, concept) {
+  getInstanceByName(name) {
     if (!name) { return null; }
-    const possibleConcepts = concept ? [concept.id].concat(concept.ancestors.map(ancestor => ancestor.id)).concat(concept.descendants.map(descendant => descendant.id)) : [];
-    for (const instance of this.instances) {
-      if (instance && (concept ? possibleConcepts.indexOf(instance.concept.id) > -1 : true)) {
-        if (instance.name.toLowerCase() === name.toLowerCase()) {
-          return instance;
-        }
-        for (const synonym of instance.synonyms) {
-          if (synonym.toLowerCase() === name.toLowerCase()) {
-            return instance;
-          }
+    for (let i = 0; i < this.instances.length; i += 1) {
+      if (this.instances[i].name.toLowerCase() === name.toLowerCase()) {
+        return this.instances[i];
+      }
+      for (let j = 0; j < this.instances[i].synonyms.length; j += 1) {
+        if (this.instances[i].synonyms[j].toLowerCase() === name.toLowerCase()) {
+          return this.instances[i];
         }
       }
     }
@@ -98,17 +95,15 @@ class CENode {
    * Returns: [obj{instance}]
    */
   getInstances(conceptType, recurse) {
-    const instanceList = [];
+    let instanceList = [];
     if (!conceptType) {
-      for (const instance of this.instances) {
-        instanceList.push(instance);
-      }
+      instanceList = this.instances;
     } else if (conceptType && !recurse) {
       const concept = this.getConceptByName(conceptType);
       if (concept) {
-        for (const instance of this.instances) {
-          if (instance && instance.concept.id === concept.id) {
-            instanceList.push(instance);
+        for (let i = 0; i < this.instances.length; i += 1) {
+          if (this.instances[i].type.id === concept.id) {
+            instanceList.push(this.instances[i]);
           }
         }
       }
@@ -117,10 +112,10 @@ class CENode {
       if (concept) {
         const descendants = concept.descendants.concat(concept);
         const childrenIds = [];
-        for (const descendant of descendants) { childrenIds.push(descendant.id); }
-        for (const instance of this.instances) {
-          if (instance && childrenIds.indexOf(instance.concept.id) > -1) {
-            instanceList.push(instance);
+        for (let i = 0; i < descendants.length; i += 1) { childrenIds.push(descendants[i].id); }
+        for (let i = 0; i < this.instances.length; i += 1) {
+          if (childrenIds.indexOf(this.instances[i].type.id) > -1) {
+            instanceList.push(this.instances[i]);
           }
         }
       }
@@ -137,12 +132,12 @@ class CENode {
    */
   addSentence(sentence, source) {
     const ceResult = this.addCE(sentence, false, source);
-    if (!ceResult.error) {
+    if (ceResult.success) {
       return ceResult;
     }
 
     const questionResult = this.askQuestion(sentence);
-    if (!questionResult.error) {
+    if (questionResult.success) {
       return questionResult;
     }
 
@@ -156,8 +151,8 @@ class CENode {
    */
   addSentences(sentences, source) {
     const responses = [];
-    for (const sentence of sentences) {
-      responses.push(this.addSentence(sentence, source));
+    for (let i = 0; i < sentences.length; i += 1) {
+      responses.push(this.addSentence(sentences[i], source));
     }
     return responses;
   }
@@ -170,7 +165,13 @@ class CENode {
    * Returns: {success: bool, type: str, data: str}
    */
   addCE(sentence, source) {
-    return this.ceParser.parse(sentence.trim().replace('{now}', new Date().getTime()).replace('{uid}', this.newCardId()), source);
+    const success = this.ceParser.parse(sentence.trim().replace('{now}', new Date().getTime()).replace('{uid}', this.newCardId()), source);
+    return {
+      success: success[0],
+      type: 'gist',
+      data: success[1],
+      result: success[2] || undefined,
+    };
   }
 
   /*
@@ -180,7 +181,12 @@ class CENode {
    * Returns: {success: bool, type: str, data: str}
    */
   askQuestion(sentence) {
-    return this.questionParser.parse(sentence);
+    const success = this.questionParser.parse(sentence);
+    return {
+      success: success[0],
+      type: success[0] ? 'gist' : undefined,
+      data: success[0] ? success[1] : undefined,
+    };
   }
 
   /*
@@ -190,7 +196,11 @@ class CENode {
    * Returns: {type: str, data: str}
    */
   addNL(sentence) {
-    return this.nlParser.parse(sentence);
+    const success = this.nlParser.parse(sentence);
+    return {
+      type: success[0] ? 'confirm' : 'gist',
+      data: success[1],
+    };
   }
 
   /*
@@ -200,10 +210,8 @@ class CENode {
    */
   loadModel(sentences) {
     const responses = [];
-    if (sentences && sentences.length) {
-      for (const sentence of sentences) {
-        responses.push(this.addCE(sentence));
-      }
+    for (let i = 0; i < sentences.length; i += 1) {
+      responses.push(this.addCE(sentences[i]));
     }
     return responses;
   }
@@ -242,8 +250,8 @@ class CENode {
     this.lastInstanceId = this.instances.length;
     this.lastConceptId = this.concepts.length;
     this.lastCardId = 0;
-    for (const model of models) {
-      this.loadModel(model);
+    for (let i = 0; i < models.length; i += 1) {
+      this.loadModel(models[i]);
     }
   }
 }

src/CEParser.js

@@ -30,14 +30,6 @@ const quotes = {
 
 class CEParser {
 
-  static error(message, concerns) {
-    return { error: true, response: { message, type: 'gist', concerns } };
-  }
-
-  static success(message, concerns) {
-    return { error: false, response: { message, type: 'gist', concerns } };
-  }
-
   /*
    * Submit CE to be processed by node.
    * This may result in
@@ -50,22 +42,17 @@ class CEParser {
    * Returns: [bool, str] (bool = success, str = error or parsed string)
    */
   parse(input, source) {
-    try {
-      // Whitespace -> single space:
-      const t = input.replace(/\s+/g, ' ').replace(/\.+$/, '').trim();
-      if (t.match(/^conceptualise an?/i)) {
-        return this.newConcept(t, source);
-      } else if (t.match(/^conceptualise the ([a-zA-Z0-9 ]*) ([A-Z0-9]+) (?:has|is|~)/i)) {
-        return this.modifyConcept(t, source);
-      } else if (t.match(/^there is an? ([a-zA-Z0-9 ]*) named/i)) {
-        return this.newInstance(t, source);
-      } else if (t.match(/^the ([a-zA-Z0-9 ]*)/i)) {
-        return this.modifyInstance(t, source);
-      }
-      return CEParser.error('Input not a valid CE sentence.');
-    } catch (err) {
-      return CEParser.error(`There was a problem parsing the CE. ${err}.`);
+    const t = input.replace(/\s+/g, ' ').replace(/\.+$/, '').trim(); // Whitespace -> single space
+    if (t.match(/^conceptualise an?/i)) {
+      return this.newConcept(t, source);
+    } else if (t.match(/^conceptualise the/i)) {
+      return this.modifyConcept(t, source);
+    } else if (t.match(/^there is an? ([a-zA-Z0-9 ]*) named/i)) {
+      return this.newInstance(t, source);
+    } else if (t.match(/^the ([a-zA-Z0-9 ]*)/i)) {
+      return this.modifyInstance(t, source);
     }
+    return [false, null];
   }
 
   newConcept(t, source) {
@@ -74,7 +61,7 @@ class CEParser {
     const storedConcept = this.node.getConceptByName(conceptName);
     let concept = null;
     if (storedConcept) {
-      return CEParser.error('This concept already exists');
+      return [false, 'This concept already exists.'];
     }
     concept = new CEConcept(this.node, conceptName, source);
 
@@ -83,19 +70,16 @@ class CEParser {
     for (const fact of facts) {
       this.processConceptFact(concept, fact, source);
     }
-    return CEParser.success(t, concept);
+    return [true, t, concept];
   }
 
   modifyConcept(t, source) {
     const conceptInfo = t.match(/^conceptualise the ([a-zA-Z0-9 ]*) ([A-Z0-9]+) (?:has|is|~)/);
-    if (!conceptInfo) {
-      return CEParser.error('Unable to parse sentence');
-    }
     const conceptName = conceptInfo[1];
     const conceptVar = conceptInfo[2];
     const concept = this.node.getConceptByName(conceptName);
     if (!concept) {
-      return CEParser.error(`Concept ${conceptInfo[1]} not known.`);
+      return [false, `Concept ${conceptInfo[1]} not known.`];
     }
 
     const remainderRegex = new RegExp(`^conceptualise the ${conceptName} ${conceptVar}`, 'i');
@@ -104,7 +88,7 @@ class CEParser {
     for (const fact of facts) {
       this.processConceptFact(concept, fact, source);
     }
-    return CEParser.success(t, concept);
+    return [true, t, concept];
   }
 
   processConceptFact(concept, fact, source) {
@@ -148,19 +132,19 @@ class CEParser {
     let names = t.match(/^there is an? ([a-zA-Z0-9 ]*) named '([^'\\]*(?:\\.[^'\\]*)*)'/i);
     if (!names) {
       names = t.match(/^there is an? ([a-zA-Z0-9 ]*) named ([a-zA-Z0-9_]*)/i);
-      if (!names) { return CEParser.error('Unable to determine name of instance.'); }
+      if (!names) { return [false, 'Unable to determine name of instance.']; }
     }
     const conceptName = names[1];
     const instanceName = names[2].replace(/\\/g, '');
     const concept = this.node.getConceptByName(conceptName);
-    const currentInstance = this.node.getInstanceByName(instanceName, concept);
+    const currentInstance = this.node.getInstanceByName(instanceName);
+    const instance = new CEInstance(this.node, concept, instanceName, source);
     if (!concept) {
-      return CEParser.error(`Instance type unknown: ${conceptName}`);
+      return [false, `Instance type unknown: ${conceptName}`];
     }
     if (currentInstance && currentInstance.type.id === concept.id) {
-      return CEParser.error('There is already an instance of this type with this name.', currentInstance);
+      return [true, 'There is already an instance of this type with this name.', currentInstance];
     }
-    const instance = new CEInstance(this.node, concept, instanceName, source);
     instance.sentences.push(t);
 
     const remainder = t.replace(/^there is an? (?:[a-zA-Z0-9 ]*) named (?:[a-zA-Z0-9_]*|'[a-zA-Z0-9_ ]*') that/, '');
@@ -168,47 +152,42 @@ class CEParser {
     for (const fact of facts) {
       this.processInstanceFact(instance, fact, source);
     }
-    return CEParser.success(t, instance);
+    return [true, t, instance];
   }
 
   modifyInstance(t, source) {
     let concept;
     let instance;
-    let instanceName;
-    if (t.match(/^the ([a-zA-Z0-9 ]*)/i)) {
+    if (t.match(/^the ([a-zA-Z0-9 ]*) '([^'\\]*(?:\\.[^'\\]*)*)'/i)) {
+      const names = t.match(/^the ([a-zA-Z0-9 ]*) '([^'\\]*(?:\\.[^'\\]*)*)'/i);
+      if (names) {
+        concept = this.node.getConceptByName(names[1]);
+        instance = this.node.getInstanceByName(names[2].replace(/\\/g, ''));
+      }
+    }
+    if (!instance && t.match(/^the ([a-zA-Z0-9 ]*)/i)) {
       const names = t.match(/^the ([a-zA-Z0-9 ]*)/i);
       const nameTokens = names[1].split(' ');
-      for (const conceptCheck of this.node.concepts) {
-        if (names[1].toLowerCase().indexOf(conceptCheck.name.toLowerCase()) === 0) {
-          concept = conceptCheck;
-          instanceName = nameTokens[concept.name.split(' ').length];
-          instance = this.node.getInstanceByName(instanceName, concept);
+      for (let i = 0; i < this.node.concepts.length; i += 1) {
+        if (names[1].toLowerCase().indexOf(this.node.concepts[i].name.toLowerCase()) === 0) {
+          concept = this.node.concepts[i];
+          const instanceName = nameTokens[concept.name.split(' ').length].toLowerCase();
+          instance = concept[instanceName];
           break;
         }
       }
     }
-    if (!instance && t.match(/^the ([a-zA-Z0-9 ]*) '([^'\\]*(?:\\.[^'\\]*)*)'/i)) {
-      const names = t.match(/^the ([a-zA-Z0-9 ]*) '([^'\\]*(?:\\.[^'\\]*)*)'/i);
-      if (names) {
-        concept = this.node.getConceptByName(names[1]);
-        instanceName = names[2].replace(/\\/g, '');
-        instance = this.node.getInstanceByName(instanceName, concept);
-      }
-    }
     if (!concept || !instance) {
-      return CEParser.error(`Unknown concept/instance combination in: ${t}`);
+      return [false, `Unknown concept/instance combination in: ${t}`];
     }
     instance.sentences.push(t);
-    const tokens = t.split(' ');
-    tokens.splice(0, 1 + concept.name.split(' ').length + instanceName.split(' ').length);
-    const remainder = tokens.join(' ');
+
+    const remainder = t.replace(`'${instance.name}'`, instance.name).replace(`the ${concept.name} ${instance.name}`.trim(), '');
     const facts = remainder.replace(/\band\b/g, '+').match(/(?:'(?:\\.|[^'])*'|[^+])+/g);
-    if (facts) {
-      for (const fact of facts) {
-        this.processInstanceFact(instance, fact, source);
-      }
+    for (const fact of facts) {
+      this.processInstanceFact(instance, fact, source);
     }
-    return CEParser.success(t, instance);
+    return [true, t, instance];
   }
 
   processInstanceFact(instance, fact, source) {
@@ -220,14 +199,11 @@ class CEParser {
       const relConceptName = match[2];
       const relInstanceName = match[3].replace(/'/g, '');
       const relConcept = this.node.getConceptByName(relConceptName);
-
-      if (relConcept) {
-        let relInstance = this.node.getInstanceByName(relInstanceName, relConcept);
-        if (!relInstance) {
-          relInstance = new CEInstance(this.node, relConcept, relInstanceName, source);
-        }
-        instance.addRelationship(label, relInstance, true, source);
+      let relInstance = this.node.getInstanceByName(relInstanceName);
+      if (!relInstance) {
+        relInstance = new CEInstance(this.node, relConcept, relInstanceName, source);
       }
+      instance.addRelationship(label, relInstance, true, source);
     }
     if (input.match(/^has ([a-zA-Z0-9]*|'[^'\\]*(?:\\.[^'\\]*)*') as ([a-zA-Z0-9 ]*)/)) {
       const re = /^has ([a-zA-Z0-9]*|'[^'\\]*(?:\\.[^'\\]*)*') as ([a-zA-Z0-9 ]*)/;
@@ -243,13 +219,11 @@ class CEParser {
       const valInstanceName = match[2].replace(/'/g, '');
       const label = match[3];
       const valConcept = this.node.getConceptByName(valConceptName);
-      if (valConcept) {
-        let valInstance = this.node.getInstanceByName(valInstanceName, valConcept);
-        if (!valInstance) {
-          valInstance = new CEInstance(this.node, valConcept, valInstanceName, source);
-        }
-        instance.addValue(label, valInstance, true, source);
+      let valInstance = this.node.getInstanceByName(valInstanceName);
+      if (!valInstance) {
+        valInstance = new CEInstance(this.node, valConcept, valInstanceName, source);
       }
+      instance.addValue(label, valInstance, true, source);
     }
     if (input.match(/(?:is| )?an? ([a-zA-Z0-9 ]*)/g)) {
       const re = /(?:is| )?an? ([a-zA-Z0-9 ]*)/g;

src/CEServer.js

@@ -20,277 +20,141 @@ const http = require('http');
 const CENode = require('./CENode.js');
 const CEModels = require('../models/index.js');
 
-class CEServer {
+const POST_SENTENCES_ENDPOINT = '/sentences';
+const GET_CARDS_ENDPOINT = '/cards';
 
-  constructor(name, port, models) {
-    this.port = port;
-    this.node = new CENode();
-    if (models) {
-      for (const model of models) {
-        this.node.loadModel(CEModels[model]);
+function postSentences(node, request, response) {
+  let body = '';
+  request.on('data', (chunk) => { body += chunk; });
+  request.on('end', () => {
+    body = decodeURIComponent(body.replace('sentence=', '').replace(/\+/g, ' '));
+    const sentences = body.split(/\\n|\n/);
+    const responses = node.addSentences(sentences);
+    response.write(responses.map(resp => resp.data).join('\n'));
+    response.end();
+  });
+}
+
+function getCards(node, request, response, ignoresInput) {
+  const url = decodeURIComponent(request.url);
+  const agentRegex = url.match(/agent=(.*)/);
+  const ignores = ignoresInput || [];
+  let agentStr = null;
+  let agents = [];
+  if (agentRegex) { agentStr = agentRegex[1]; }
+  if (agentStr) {
+    agents = agentStr.toLowerCase().split(',');
+  }
+  const cards = node.getInstances('card', true);
+  let s = '';
+  for (let i = 0; i < cards.length; i += 1) {
+    if (ignores.indexOf(cards[i].name) === -1) {
+      if (!agents || agents.length === 0) {
+        s += `${cards[i].ce}\n`;
+      } else {
+        const tos = cards[i].is_tos;
+        if (tos) {
+          for (let j = 0; j < tos.length; j += 1) {
+            for (let k = 0; k < agents.length; k += 1) {
+              if (tos[j].name.toLowerCase() === agents[k]) {
+                s += `${cards[i].ce}\n`;
+                break;
+              }
+            }
+          }
+        }
       }
     }
-    this.node.attachAgent();
-    this.node.agent.setName(name);
-    this.handlers = {
-      GET: {
-        '/cards': (request, response) => {
-          const agentRegex = decodeURIComponent(request.url).match(/agent=(.*)/);
-          const agentStr = agentRegex ? agentRegex[1] : null;
-          const agents = (agentStr && agentStr.toLowerCase().split(',')) || [];
-          let s = '';
-          for (const card of this.node.getInstances('card', true)) {
-            for (const to of card.is_tos) {
-              for (const agent of agents) {
-                if (to.name.toLowerCase() === agent) {
-                  s += `${card.ce}\n`;
-                  break;
-                }
-              }
-            }
-          }
-          response.writeHead(200, { 'Content-Type': 'text/ce' });
-          response.end(s);
-        },
-        '/concepts': (request, response) => {
-          const concepts = [];
-          for (const concept of this.node.concepts) {
-            concepts.push({
-              name: concept.name,
-              id: concept.id,
-            });
-          }
-          response.writeHead(200, { 'Content-Type': 'application/json' });
-          response.end(JSON.stringify(concepts));
-        },
-        '/concept': (request, response) => {
-          const idRegex = decodeURIComponent(request.url).match(/id=(.*)/);
-          const id = idRegex ? idRegex[1] : null;
-          const concept = this.node.getConceptById(id);
-          if (concept) {
-            const body = { name: concept.name, ce: concept.ce, parents: [], children: [], instances: [], values: [], relationships: [] };
-            for (const parent of concept.parents) {
-              body.parents.push({
-                name: parent.name,
-                id: parent.id,
-              });
-            }
-            for (const child of concept.children) {
-              body.children.push({
-                name: child.name,
-                id: child.id,
-              });
-            }
-            for (const instance of concept.instances) {
-              body.instances.push({
-                name: instance.name,
-                id: instance.id,
-              });
-            }
-            for (const value of concept.values) {
-              const valueName = value.concept && value.concept.name;
-              const valueId = value.concept && value.concept.id;
-              body.values.push({ label: value.label, targetName: valueName, targetId: valueId });
-            }
-            for (const relationship of concept.relationships) {
-              body.relationships.push({ label: relationship.label, targetName: relationship.concept.name, targetId: relationship.concept.id });
-            }
-            response.writeHead(200, { 'Content-Type': 'application/json' });
-            return response.end(JSON.stringify(body));
-          }
-          response.writeHead(404);
-          return response.end('Concept not found');
-        },
-        '/instances': (request, response) => {
-          const instances = [];
-          for (const instance of this.node.instances) {
-            instances.push({
-              name: instance.name,
-              id: instance.id,
-              conceptName: instance.concept.name,
-              conceptId: instance.concept.id,
-            });
-          }
-          response.writeHead(200, { 'Content-Type': 'application/json' });
-          response.end(JSON.stringify(instances));
-        },
-        '/instance': (request, response) => {
-          const idRegex = decodeURIComponent(request.url).match(/id=(.*)/);
-          const nameRegex = decodeURIComponent(request.url).match(/name=(.*)/);
-          const idQuery = idRegex ? idRegex[1] : null;
-          const nameQuery = nameRegex ? nameRegex[1] : null;
-          let instance;
-          if (idQuery) {
-            instance = this.node.getInstanceById(idQuery);
-          } else if (nameQuery) {
-            instance = this.node.getInstanceByName(nameQuery);
-          }
-          if (instance) {
-            const body = {
-              name: instance.name,
-              conceptName: instance.concept.name,
-              conceptId: instance.concept.id,
-              ce: instance.ce,
-              gist: instance.gist,
-              synonyms: instance.synonyms,
-              subConcepts: [],
-              values: [],
-              relationships: [],
-            };
-            for (const concept of instance.subConcepts) {
-              body.subConcepts.push({ name: concept.name, id: concept.id });
-            }
-            for (const value of instance.values) {
-              const valueName = value.instance.name || value.instance;
-              const valueId = value.instance.id;
-              const conceptName = value.instance.concept && value.instance.concept.name;
-              const conceptId = value.instance.concept && value.instance.concept.id;
-              body.values.push({ label: value.label, targetName: valueName, targetId: valueId, targetConceptName: conceptName, targetConceptId: conceptId });
-            }
-            for (const relationship of instance.relationships) {
-              body.relationships.push({ label: relationship.label, targetName: relationship.instance.name, targetId: relationship.instance.id, targetConceptName: relationship.instance.concept.name, targetConceptId: relationship.instance.concept.id });
-            }
-            response.writeHead(200, { 'Content-Type': 'application/json' });
-            return response.end(JSON.stringify(body));
-          }
-          response.writeHead(404);
-          return response.end('Unable to find the instance.');
-        },
-        '/info': (request, response) => {
-          const body = { recentInstances: [], recentConcepts: [], instanceCount: this.node.instances.length, conceptCount: this.node.concepts.length };
-          const recentInstances = this.node.instances.slice(this.node.instances.length >= 10 ? this.node.instances.length - 10 : 0);
-          for (const instance of recentInstances) {
-            body.recentInstances.push({
-              name: instance.name,
-              id: instance.id,
-              conceptName: instance.concept.name,
-              conceptId: instance.concept.id,
-            });
-          }
-          for (const concept of this.node.concepts) {
-            body.recentConcepts.push({
-              name: concept.name,
-              id: concept.id,
-            });
-          }
-          response.writeHead(200, { 'Content-Type': 'application/json' });
-          response.end(JSON.stringify(body));
-        },
-        '/model': (request, response) => {
-          let body = '';
-          for (const concept of this.node.concepts) { body += `${concept.creationCE}\n`; }
-          for (const concept of this.node.concepts) { body += `${concept.getCE(true)}\n`; }
-          for (const instance of this.node.instances) { body += `${instance.creationCE}\n`; }
-          for (const instance of this.node.instances) { body += `${instance.getCE(true)}\n`; }
-          response.writeHead(200, { 'Content-Type': 'text/ce', 'Content-Disposition': `attachment; filename="${this.node.agent.name}.ce"` });
-          response.end(body);
-        },
-      },
-      POST: {
-        '/cards': (request, response) => {
-          let body = '';
-          request.on('data', (chunk) => { body += chunk; });
-          request.on('end', () => {
-            const ignores = body.split(/\\n|\n/);
-            const agentRegex = decodeURIComponent(request.url).match(/agent=(.*)/);
-            const agentStr = agentRegex ? agentRegex[1] : null;
-            const agents = (agentStr && agentStr.toLowerCase().split(',')) || [];
-            let s = '';
-            for (const card of this.node.getInstances('card', true)) {
-              if (ignores.indexOf(card.name) === -1) {
-                if (agents.length === 0) {
-                  s += `${card.ce}\n`;
-                } else if (card.is_tos) {
-                  for (const to of card.is_tos) {
-                    for (const agent of agents) {
-                      if (to.name.toLowerCase() === agent) {
-                        s += `${card.ce}\n`;
-                        break;
-                      }
-                    }
-                  }
-                }
-              }
-            }
-            response.writeHead(200, { 'Content-Type': 'text/ce' });
-            response.end(s);
-          });
-        },
-        '/sentences': (request, response) => {
-          let body = '';
-          request.on('data', (chunk) => { body += chunk; });
-          request.on('end', () => {
-            body = decodeURIComponent(body.replace('sentence=', '').replace(/\+/g, ' '));
-            const sentences = body.split(/\\n|\n/);
-            const responses = this.node.addSentences(sentences);
-            response.writeHead(200, { 'Content-Type': 'text/ce' });
-            response.end(responses.map(resp => resp.data).join('\n'));
-          });
-        },
-      },
-      PUT: {
-        '/reset': (request, response) => {
-          this.node.resetAll();
-          response.writeHead(204);
-          response.end();
-        },
-        '/agent/name': (request, response) => {
-          let body = '';
-          request.on('data', (chunk) => { body += chunk; });
-          request.on('end', () => {
-            body = decodeURIComponent(body.replace('name=', '').replace(/\+/g, ' '));
-            this.node.agent.setName(body);
-            response.writeHead(302, { Location: '/' });
-            response.end();
-          });
-        },
-      },
-    };
+  }
+  response.write(s);
+  response.end();
+}
+
+function startServer() {
+  const node = new CENode(CEModels.core, CEModels.server);
+  node.attachAgent();
+  let port = 5555;
+  if (process.argv.length > 3) {
+    port = process.argv[3];
+  }
+  if (process.argv.length > 2) {
+    node.agent.setName(process.argv[2]);
   }
 
-  start() {
-    this.server = http.createServer((request, response) => {
-      response.setHeader('Access-Control-Allow-Origin', '*');
-      if (request.method in this.handlers) {
-        const path = request.url.indexOf('?') > 1 ? request.url.slice(0, request.url.indexOf('?')) : request.url;
-        if (path in this.handlers[request.method]) {
-          try {
-            this.handlers[request.method][path](request, response);
-          } catch (err) {
-            response.writeHead(500);
-            response.end(`500: ${err}.`);
+  http.createServer((request, response) => {
+    response.setHeader('Access-Control-Allow-Origin', '*');
+
+    if (request.method === 'GET') {
+      if (request.url === '/') {
+        const ins = node.instances;
+        const con = node.concepts;
+        let s = '<html><head><title>CENode Management</title></head><body><h1>CENode Server Admin Interface</h1>';
+        s += '<div style="width:48%;float:left;"><h2>Conceptual model</h2>';
+        s += '<p>Add CE sentences to the node:</p><form action="/ui/sentences" enctype="application/x-www-form-urlencoded" method="POST"><textarea name="sentence" style="width:95%;height:100px;"></textarea><br /><br /><input type="submit" /></form></div>';
+        s += `<div style="width:48%;float:left;"><h2>Node settings</h2><p>Update local agent name:</p><form method="POST" action="/agent-name"><input type="text" name="name" value="${node.agent.name}" /><input type="submit" /></form>`;
+        s += '<p>Other options:</p><button onclick="window.location=\'/reset\';">Empty model</button>';
+        s += '</div><div style="clear:both;"></div>';
+        s += '<div style="display:inline-block;width:45%;float:left;"><h2>Concepts</h2>';
+        for (const concept of con) {
+          s += concept.name;
+          if (concept.parents.length) {
+            s += ` (${concept.parents[0].name})`;
           }
-        } else {
-          response.writeHead(404);
-          response.end(`404: Resource not found for method ${request.method}.`);
+          s += '<br>';
         }
-      } else if (request.method === 'OPTIONS') {
-        response.setHeader('Access-Control-Allow-Headers', 'X-Requested-With, Content-Type');
-        response.setHeader('Access-Control-Allow-Methods', 'GET, POST, PUT, DELETE');
-        response.writeHead(200);
+        s += '</div><div style="display:inline-block;width:45%;float:right;"><h2>Instances</h2>';
+        for (const instance of ins) {
+          s += `${instance.name} (${instance.type.name})<br>`;
+        }
+        s += '</div><body></html>';
+        response.writeHead(200, { 'Content-Type': 'text/html' });
+        response.end(s);
+      } else if (request.url.indexOf(GET_CARDS_ENDPOINT) === 0) {
+        response.writeHead(200, { 'Content-Type': 'text/ce' });
+        getCards(node, request, response);
+      } else if (request.url === '/reset') {
+        node.resetAll();
+        response.writeHead(302, { Location: '/' });
         response.end();
       } else {
-        response.writeHead(405);
-        response.end('405: Method not allowed on this server.');
+        response.writeHead(404);
+        response.end('404: Resource not found for method GET.');
       }
-    });
-    this.server.listen(this.port);
-    this.server.on('error', () => { this.node = undefined; });
-  }
-
-  stop() {
-    if (this.server) {
-      delete this.node;
-      this.server.close();
+    } else if (request.method === 'POST') {
+      if (request.url.indexOf(GET_CARDS_ENDPOINT) === 0) {
+        let body = '';
+        request.on('data', (chunk) => { body += chunk; });
+        request.on('end', () => {
+          const ignores = body.split(/\\n|\n/);
+          response.writeHead(200, { 'Content-Type': 'text/ce' });
+          getCards(node, request, response, ignores);
+        });
+      } else if (request.url === POST_SENTENCES_ENDPOINT) {
+        response.writeHead(200, { 'Content-Type': 'text/ce' });
+        postSentences(node, request, response);
+      } else if (request.url === '/ui/sentences') {
+        response.writeHead(302, { Location: '/' });
+        postSentences(node, request, response);
+      } else if (request.url === '/agent-name') {
+        let body = '';
+        request.on('data', (chunk) => { body += chunk; });
+        request.on('end', () => {
+          body = decodeURIComponent(body.replace('name=', '').replace(/\+/g, ' '));
+          node.agent.setName(body);
+          response.writeHead(302, { Location: '/' });
+          response.end();
+        });
+      } else {
+        response.writeHead(404);
+        response.end('404: Resource not found for method POST.');
+      }
+    } else {
+      response.writeHead(405);
+      response.end('405: Method not allowed on this server.');
     }
-  }
+  }).listen(port || 5555);
 }
 
 if (require.main === module) {
-  const name = process.argv.length > 2 ? process.argv[2] : 'Moira';
-  const port = process.argv.length > 3 ? process.argv[3] : 5555;
-  const models = process.argv.slice(4);
-  new CEServer(name, port, models).start();
+  startServer();
 }
-
-module.exports = CEServer;

src/CardHandler.js

@@ -38,9 +38,9 @@ class CardHandler {
           // Prepare the response 'tell card' and add this back to the node
           let urls;
           let c;
-          if (data.response.message) {
-            urls = data.response.message.match(/(https?:\/\/[a-zA-Z0-9./\-+_&=?!%]*)/gi);
-            c = `there is a ${data.response.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${data.response.message.replace(/'/g, "\\'")}' as content`;
+          if (data.data) {
+            urls = data.data.match(/(https?:\/\/[a-zA-Z0-9./\-+_&=?!%]*)/gi);
+            c = `there is a ${data.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${data.data.replace(/'/g, "\\'")}' as content`;
           } else {
             c = `there is a gist card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has 'Sorry; your question was not understood.' as content`;
           }
@@ -63,12 +63,12 @@ class CardHandler {
         // Add the CE sentence to the node
         const data = this.node.addCE(card.content, card.is_from && card.is_from.name);
 
-        if (data.error && card.is_from) {
+        if (!data.success && card.is_from) {
           // If unsuccessful, write an error back
           return this.node.addSentence(`there is a gist card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and is to the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has 'Sorry. Your input was not understood.' as content and is in reply to the card '${card.name}'.`);
         }
 
-        if (!data.error) {
+        if (data.success === true) {
           // Add sentence to any active tell policy queues
           for (const policy of this.node.getInstances('tell policy')) {
             if (policy.enabled === 'true' && policy.target && policy.target.name) {
@@ -86,12 +86,12 @@ class CardHandler {
               const ack = policy.acknowledgement;
               if (policy.target.name.toLowerCase() === card.is_from.name.toLowerCase()) {
                 let c;
-                if (ack === 'basic') { c = 'OK.'; } else if (data.response.type === 'tell') {
-                  c = `OK. I added this to my knowledge base: ${data.response.message}`;
-                } else if (data.response.type === 'ask' || data.response.type === 'confirm' || data.response.type === 'gist') {
-                  c = data.response.message;
+                if (ack === 'basic') { c = 'OK.'; } else if (data.type === 'tell') {
+                  c = `OK. I added this to my knowledge base: ${data.data}`;
+                } else if (data.type === 'ask' || data.type === 'confirm' || data.type === 'gist') {
+                  c = data.data;
                 }
-                return this.node.addSentence(`there is a ${data.response.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and is to the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${c.replace(/'/g, "\\'")}' as content and is in reply to the card '${card.name}'.`);
+                return this.node.addSentence(`there is a ${data.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and is to the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${c.replace(/'/g, "\\'")}' as content and is in reply to the card '${card.name}'.`);
               }
             }
           }
@@ -102,34 +102,23 @@ class CardHandler {
       'nl card': (card) => {
         let data = this.node.addCE(card.content, card.is_from && card.is_from.name);
         // If valid CE, then replicate the nl card as a tell card ('autoconfirm')
-        if (!data.error) {
+        if (data.success) {
           return this.node.addSentence(`there is a tell card named 'msg_{uid}' that is from the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and is to the agent '${this.agent.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${card.content.replace(/'/g, "\\'")}' as content.`);
         }
         data = this.node.askQuestion(card.content);
         // If question was success replicate as ask card ('autoask')
-        if (!data.error) {
+        if (data.success) {
           return this.node.addSentence(`there is an ask card named 'msg_{uid}' that is from the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and is to the agent '${this.agent.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${card.content.replace(/'/g, "\\'")}' as content.`);
         }
         // If question not understood then place the response to the NL card in a new response
         data = this.node.addNL(card.content);
-        return this.node.addSentence(`there is a ${data.response.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and is to the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${data.response.message.replace(/'/g, "\\'")}' as content and is in reply to the card '${card.name}'.`);
-      },
-
-      'gist card': (card) => {
-        // Add sentence to any active gist policy queues
-        for (const policy of this.node.getInstances('gist policy')) {
-          if (policy.enabled === 'true' && policy.target && policy.target.name) {
-            const targetName = policy.target.name;
-            if (!(targetName in this.agent.policyHandler.unsentGistCards)) { this.agent.policyHandler.unsentGistCards[targetName] = []; }
-            this.agent.policyHandler.unsentGistCards[targetName].push(card);
-          }
-        }
+        return this.node.addSentence(`there is a ${data.type} card named 'msg_{uid}' that is from the agent '${this.agent.name.replace(/'/g, "\\'")}' and is to the ${card.is_from.type.name} '${card.is_from.name.replace(/'/g, "\\'")}' and has the timestamp '{now}' as timestamp and has '${data.data.replace(/'/g, "\\'")}' as content and is in reply to the card '${card.name}'.`);
       },
     };
   }
 
   handle(card) {
-    if (card.type.name in this.handlers && card.is_tos && card.content && this.agent.handledCards.indexOf(card.name) === -1) {
+    if (card.is_tos && card.content && this.agent.handledCards.indexOf(card.name) === -1) {
       // Determine whether or not to read or ignore this card:
       for (const to of card.is_tos) {
         if (to.name.toLowerCase() === this.agent.name.toLowerCase()) {

src/NLParser.js

@@ -18,14 +18,6 @@
 
 class NLParser {
 
-  static error(message) {
-    return { error: true, response: { message, type: 'gist' } };
-  }
-
-  static success(message) {
-    return { error: false, response: { message, type: 'confirm' } };
-  }
-
   /*
    * Submit natural language to be processed by node.
    * This results in
@@ -130,7 +122,7 @@ class NLParser {
         }
       }
       if (facts.length > 0) {
-        return NLParser.success(ce + facts.join(' and '));
+        return [true, ce + facts.join(' and ')];
       }
     }
 
@@ -148,12 +140,12 @@ class NLParser {
           }
         }
         if (newInstanceName && newInstanceName.length) {
-          return NLParser.success(`there is a ${this.node.concepts[i].name} named '${newInstanceName.trim()}'`);
+          return [true, `there is a ${this.node.concepts[i].name} named '${newInstanceName.trim()}'`];
         }
-        return NLParser.success(`there is a ${this.node.concepts[i].name} named '${this.node.concepts[i].name} ${this.node.instances.length}${1}'`);
+        return [true, `there is a ${this.node.concepts[i].name} named '${this.node.concepts[i].name} ${this.node.instances.length}${1}'`];
       }
     }
-    return NLParser.error(`Un-parseable input: ${t}`);
+    return [false, `Un-parseable input: ${t}`];
   }
 
   /*

src/PolicyHandler.js

@@ -84,9 +84,8 @@ class PolicyHandler {
   constructor(agent) {
     this.agent = agent;
     this.node = agent.node;
-    this.unsentTellCards = {};
-    this.unsentAskCards = {};
-    this.unsentGistCards = {};
+    this.unsentTellCards = [];
+    this.unsentAskCards = [];
     this.lastSuccessfulRequest = 0;
     this.handlers = {
 
@@ -94,9 +93,6 @@ class PolicyHandler {
         // For each tell policy in place, send all currently-untold cards to each target
         // multiple cards to be sent to one target line-separated
         if (policy.target && policy.target.name && policy.target.address) {
-          if (!(policy.target.name in this.unsentTellCards)) {
-            this.unsentTellCards[policy.target.name] = [];
-          }
           let data = '';
           for (const card of this.unsentTellCards[policy.target.name]) {
             if (card.is_tos && card.is_from.name.toLowerCase() !== policy.target.name.toLowerCase()) { // Don't send back a card sent from target agent
@@ -124,9 +120,6 @@ class PolicyHandler {
         // For each ask policy in place send all currently-untold cards to each target
         // multiple cards to be sent to one target are line-separated
         if (policy.target && policy.target.name) {
-          if (!(policy.target.name in this.unsentAskCards)) {
-            this.unsentAskCards[policy.target.name] = [];
-          }
           let data = '';
           for (const card of this.unsentAskCards[policy.target.name]) {
             const froms = card.is_froms;
@@ -160,36 +153,6 @@ class PolicyHandler {
         }
       },
 
-      'gist policy': (policy) => {
-        // For each gist policy in place, send all currently-untold cards to each target
-        // multiple cards to be sent to one target line-separated
-        if (policy.target && policy.target.name && policy.target.address) {
-          if (!(policy.target.name in this.unsentGistCards)) {
-            this.unsentGistCards[policy.target.name] = [];
-          }
-          let data = '';
-          for (const card of this.unsentGistCards[policy.target.name]) {
-            if (card.is_tos && card.is_from.name.toLowerCase() !== policy.target.name.toLowerCase()) { // Don't send back a card sent from target agent
-              // Make sure target is not already a recipient
-              let inCard = false;
-              for (const to of card.is_tos) {
-                if (to.id === policy.target.id) { inCard = true; break; }
-              }
-              if (!inCard) {
-                card.addRelationship('is to', policy.target);
-              }
-              data += `${card.ce}\n`;
-            }
-          }
-          if (data.length) {
-            net.makeRequest('POST', policy.target.address, POST_SENTENCES_ENDPOINT, data, () => {
-              this.lastSuccessfulRequest = new Date().getTime();
-              this.unsentGistCards[policy.target.name] = [];
-            });
-          }
-        }
-      },
-
       'listen policy': (policy) => {
         // Make request to target to get cards addressed to THIS agent
         if (policy.target && policy.target.address) {
@@ -212,32 +175,30 @@ class PolicyHandler {
         if (policy.start_time) {
           const startTime = policy.start_time;
           for (const card of cards) {
-            if (card.timestamp && card.is_froms.length) {
-              let toAgent = false;
-              const tos = card.is_tos;
-              const from = card.is_froms[0];
-              const cardTimestamp = card.timestamp.name;
-              if (tos && parseInt(cardTimestamp, 10) > parseInt(startTime.name, 10)) {
-                for (const to of tos) {
-                  if (to.name === this.agent.name) { // If card sent to THIS agent
-                    toAgent = true;
-                    break;
-                  }
+            let toAgent = false;
+            const tos = card.is_tos;
+            const from = card.is_froms[0];
+            const cardTimestamp = card.timestamp.name;
+            if (tos && parseInt(cardTimestamp, 10) > parseInt(startTime, 10)) {
+              for (const to of tos) {
+                if (to.name === this.agent.name) { // If card sent to THIS agent
+                  toAgent = true;
+                  break;
                 }
-                if (toAgent) {
-                  // Add each other agent as a recipient (if they aren't already)
-                  for (const agentCheck of agents) {
-                    let agentIsRecipient = false;
-                    for (const to of tos) {
-                      if (to.name.toLowerCase() === agentCheck.name.toLowerCase()) {
-                        agentIsRecipient = true;
-                        break;
-                      }
-                    }
-                    if (!agentIsRecipient && agentCheck.name.toLowerCase() !== this.agent.name.toLowerCase() && agentCheck.name.toLowerCase() !== from.name.toLowerCase()) {
-                      card.addRelationship('is to', agentCheck);
+              }
+              if (toAgent) {
+                // Add each other agent as a recipient (if they aren't already)
+                for (const agentCheck of agents) {
+                  let agentIsRecipient = false;
+                  for (const to of tos) {
+                    if (to.name.toLowerCase() === agentCheck.name.toLowerCase()) {
+                      agentIsRecipient = true;
+                      break;
                     }
                   }
+                  if (!agentIsRecipient && agentCheck.name.toLowerCase() !== this.agent.name.toLowerCase() && agentCheck.name.toLowerCase() !== from.name.toLowerCase()) {
+                    card.addRelationship('is to', agentCheck);
+                  }
                 }
               }
             }
@@ -248,7 +209,7 @@ class PolicyHandler {
   }
 
   handle(policy) {
-    if (policy.enabled === 'true' && policy.type.name in this.handlers) {
+    if (policy.enabled === 'true') {
       this.handlers[policy.type.name](policy);
     }
   }

src/QuestionParser.js

@@ -18,14 +18,6 @@
 
 class QuestionParser {
 
-  static error(message) {
-    return { error: true, response: { message, type: 'gist' } };
-  }
-
-  static success(message) {
-    return { error: false, response: { message, type: 'gist' } };
-  }
-
   /*
    * Submit a who/what/where question to be processed by node.
    * This may result in
@@ -36,25 +28,21 @@ class QuestionParser {
    * Returns: [bool, str] (bool = success, str = error or response)
    */
   parse(t) {
-    try {
-      const input = t.trim();
-      if (t.match(/^where (is|are)/i)) {
-        return this.whereIs(input);
-      } else if (t.match(/^(\bwho\b|\bwhat\b) is(?: \bin?\b | \bon\b | \bat\b)/i)) {
-        return this.whatIsIn(input);
-      } else if (t.match(/^(\bwho\b|\bwhat\b) (?:is|are)/i)) {
-        return this.whatIs(input);
-      } else if (t.match(/^(\bwho\b|\bwhat\b) does/i)) {
-        return this.whatDoes(input);
-      } else if (t.match(/^(\bwho\b|\bwhat\b)/i)) {
-        return this.whatRelationship(input);
-      } else if (t.match(/^list (\ball\b|\binstances\b)/i)) {
-        return this.listInstances(input);
-      }
-      return QuestionParser.error('Input is not a valid question');
-    } catch (err) {
-      return QuestionParser.error(`There was a problem with the question. ${err}.`);
+    const input = t.trim();
+    if (t.match(/^where (is|are)/i)) {
+      return this.whereIs(input);
+    } else if (t.match(/^(\bwho\b|\bwhat\b) is(?: \bin?\b | \bon\b | \bat\b)/i)) {
+      return this.whatIsIn(input);
+    } else if (t.match(/^(\bwho\b|\bwhat\b) (?:is|are)/i)) {
+      return this.whatIs(input);
+    } else if (t.match(/^(\bwho\b|\bwhat\b) does/i)) {
+      return this.whatDoes(input);
+    } else if (t.match(/^(\bwho\b|\bwhat\b)/i)) {
+      return this.whatRelationship(input);
+    } else if (t.match(/^list (\ball\b|\binstances\b)/i)) {
+      return this.listInstances(input);
     }
+    return [false, null];
   }
 
   whereIs(t) {
@@ -62,7 +50,8 @@ class QuestionParser {
     const instance = this.node.getInstanceByName(thing);
     let message;
     if (!instance) {
-      return QuestionParser.success(`I don't know what ${thing} is.`);
+      message = `I don't know what ${thing} is.`;
+      return [true, message];
     }
     const locatableInstances = this.node.getInstances('location', true);
     const locatableIds = [];
@@ -91,7 +80,8 @@ class QuestionParser {
       }
     }
     if (!placeFound) {
-      return QuestionParser.success(`I don't know where ${instance.name} is.`);
+      message = `I don't know where ${instance.name} is.`;
+      return [true, message];
     }
     message = instance.name;
     for (const place in places) {
@@ -103,7 +93,7 @@ class QuestionParser {
         message += ' and';
       }
     }
-    return QuestionParser.success(`${message.substring(0, message.length - 4)}.`);
+    return [true, `${message.substring(0, message.length - 4)}.`];
   }
 
   whatIsIn(t) {
@@ -116,7 +106,7 @@ class QuestionParser {
       }
     }
     if (!instance) {
-      return QuestionParser.success(`${thing} is not an instance of type location.`);
+      return [true, `${thing} is not an instance of type location.`];
     }
     const things = {};
     let thingFound = false;
@@ -143,7 +133,7 @@ class QuestionParser {
       }
     }
     if (!thingFound) {
-      return QuestionParser.success(`I don't know what is located in/on/at the ${instance.type.name} ${instance.name}.`);
+      return [true, `I don't know what is located in/on/at the ${instance.type.name} ${instance.name}.`];
     }
 
     let message = '';
@@ -154,7 +144,7 @@ class QuestionParser {
       }
       message += ' and';
     }
-    return QuestionParser.success(`${message.substring(0, message.length - 4)}.`);
+    return [true, `${message.substring(0, message.length - 4)}.`];
   }
 
   whatIs(input) {
@@ -166,7 +156,7 @@ class QuestionParser {
     if (name) {
       instance = this.node.getInstanceByName(name[2]);
       if (instance) {
-        return QuestionParser.success(instance.gist);
+        return [true, instance.gist];
       }
     }
 
@@ -197,46 +187,44 @@ class QuestionParser {
           }
         }
         if (possibilities.length > 0) {
-          return QuestionParser.success(`'${name}' ${possibilities.join(' and ')}.`);
+          return [true, `'${name}' ${possibilities.join(' and ')}.`];
         }
 
         // If nothing found, do fuzzy search
         const searchReturn = this.fuzzySearch(t);
         let fuzzyGist = 'I know about ';
         let fuzzyFound = false;
-        if (searchReturn) {
-          for (const key in searchReturn) {
-            if (searchReturn[key].length > 1) {
-              for (let i = 0; i < searchReturn[key].length; i += 1) {
-                if (i === 0) {
-                  if (!fuzzyFound) {
-                    fuzzyGist += `the ${key}s '${searchReturn[key][i]}'`;
-                  } else {
-                    fuzzyGist += `The ${key}s '${searchReturn[key][i]}'`;
-                  }
+        for (const key in searchReturn) {
+          if (searchReturn[key].length > 1) {
+            for (let i = 0; i < searchReturn[key].length; i += 1) {
+              if (i === 0) {
+                if (!fuzzyFound) {
+                  fuzzyGist += `the ${key}s '${searchReturn[key][i]}'`;
                 } else {
-                  fuzzyGist += `, '${searchReturn[key][i]}'`;
-                }
-                if (i === searchReturn[key].length - 1) {
-                  fuzzyGist += '. ';
+                  fuzzyGist += `The ${key}s '${searchReturn[key][i]}'`;
                 }
+              } else {
+                fuzzyGist += `, '${searchReturn[key][i]}'`;
+              }
+              if (i === searchReturn[key].length - 1) {
+                fuzzyGist += '. ';
               }
-            } else if (!fuzzyFound) {
-              fuzzyGist += `the ${key} '${searchReturn[key][0]}'. `;
-            } else {
-              fuzzyGist += `The ${key} '${searchReturn[key][0]}'. `;
             }
-            fuzzyFound = true;
+          } else if (!fuzzyFound) {
+            fuzzyGist += `the ${key} '${searchReturn[key][0]}'. `;
+          } else {
+            fuzzyGist += `The ${key} '${searchReturn[key][0]}'. `;
           }
+          fuzzyFound = true;
         }
         if (fuzzyFound) {
-          return QuestionParser.success(fuzzyGist);
+          return [true, fuzzyGist];
         }
-        return QuestionParser.success('I don\'t know who or what that is.');
+        return [true, 'I don\'t know who or what that is.'];
       }
-      return QuestionParser.success(concept.gist);
+      return [true, concept.gist];
     }
-    return QuestionParser.success(instance.gist);
+    return [true, instance.gist];
   }
 
   whatDoes(t) {
@@ -268,21 +256,20 @@ class QuestionParser {
           property = instance.property(fixedPropertyName);
         }
         if (property) {
-          return QuestionParser.success(`${instance.name} ${fixedPropertyName} the ${property.type.name} ${property.name}.`);
+          return [true, `${instance.name} ${fixedPropertyName} the ${property.type.name} ${property.name}.`];
         }
-        return QuestionParser.success(`Sorry - I don't know that property about the ${instance.type.name} ${instance.name}.`);
+        return [true, `Sorry - I don't know that property about the ${instance.type.name} ${instance.name}.`];
       }
     } catch (err) {
-      return QuestionParser.success('Sorry - I can\'t work out what you\'re asking.');
+      return [false, 'Sorry - I can\'t work out what you\'re asking.'];
     }
-    return QuestionParser.success('Sorry - I can\'t work out what you\'re asking about.');
+    return null;
   }
 
   whatRelationship(t) {
     const data = t.match(/^(\bwho\b|\bwhat\b) ([a-zA-Z0-9_ ]*)/i);
     const body = data[2].replace(/\ban\b/gi, '').replace(/\bthe\b/gi, '').replace(/\ba\b/gi, '');
     const tokens = body.split(' ');
-    const uniqueResponses = new Set([]);
     let instance;
     for (let i = 0; i < tokens.length; i += 1) {
       const testString = tokens.slice(tokens.length - (i + 1), tokens.length).join(' ').trim();
@@ -296,7 +283,6 @@ class QuestionParser {
         break;
       }
     }
-
     if (instance) {
       const propertyName = tokens.splice(0, tokens.length - instance.name.split(' ').length).join(' ').trim();
       for (let i = 0; i < this.node.instances.length; i += 1) {
@@ -318,16 +304,12 @@ class QuestionParser {
           property = subject.property(fixedPropertyName);
         }
         if (property && property.name === instance.name) {
-          uniqueResponses.add(`${subject.name} ${fixedPropertyName} the ${property.type.name} ${property.name}.`);
-        }
-        const responsesArray = Array.from(uniqueResponses);
-        if (responsesArray.length > 0 && i === this.node.instances.length - 1) {
-          return QuestionParser.success(responsesArray.join(' '));
+          return [true, `${subject.name} ${fixedPropertyName} the ${property.type.name} ${property.name}.`];
         }
       }
-      return QuestionParser.success(`Sorry - I don't know that property about the ${instance.type.name} ${instance.name}.`);
+      return [true, `Sorry - I don't know that property about the ${instance.type.name} ${instance.name}.`];
     }
-    return QuestionParser.success('Sorry - I don\'t know the instance you\'re referring to.');
+    return null;
   }
 
   listInstances(t) {
@@ -342,20 +324,21 @@ class QuestionParser {
       ins = this.node.getInstances(con, true);
       s = `All instances of type '${con}':`;
     } else if (t.toLowerCase() === 'list instances') {
-      ins = this.node.getInstances();
+      ins = this.node.instances;
       s = 'All instances:';
     }
     if (ins.length === 0) {
-      return QuestionParser.success('I could not find any instances matching your query.');
+      return [true, 'I could not find any instances matching your query.'];
     }
     const names = [];
     for (let i = 0; i < ins.length; i += 1) {
       names.push(ins[i].name);
     }
-    return QuestionParser.success(`${s} ${names.join(', ')}`);
+    return [true, `${s} ${names.join(', ')}`];
   }
 
   /*
+   *
    * Search the knowledge base for an instance name similar to the one asked about.
    */
   fuzzySearch(sentence) {
@@ -365,10 +348,12 @@ class QuestionParser {
     let instancesFiltered = [];
 
     if (searchFor.indexOf(' ')) {
+      // if theres spaces then split
       multipleSearch = searchFor.split(' ');
     }
 
     if (multipleSearch) {
+      // loop through to create return string
       for (let x = 0; x < multipleSearch.length; x += 1) {
         const instancesFilteredTemp = instances.filter((input) => {
           if (input.name.toUpperCase().includes(multipleSearch[x].toUpperCase())) {
@@ -379,6 +364,7 @@ class QuestionParser {
         instancesFiltered = instancesFiltered.concat(instancesFilteredTemp);
       }
     } else {
+      // single search term
       instancesFiltered = instances.filter((input) => {
         if (input.name.toUpperCase().includes(searchFor.toUpperCase())) {
           return input;

src/RuleEngine.js

@@ -64,15 +64,16 @@ class RuleEngine {
     if (typeof objectInstance === 'string') {
       return;
     }
-    for (const ruleInstance of this.node.getInstances('rule')) {
-      const rule = RuleEngine.parseRule(ruleInstance.instruction);
+    const rules = this.node.getInstances('rule');
+    for (let i = 0; i < rules.length; i += 1) {
+      const rule = RuleEngine.parseRule(rules[i].instruction);
       if (!rule) { return; }
       if (rule.if.concept === subjectInstance.type.name) {
         if ((propertyType === 'relationship' && rule.if.relationship) || (propertyType === 'value' && rule.if.value)) {
-          const ancestorConcepts = objectInstance.concept.ancestors;
-          ancestorConcepts.push(objectInstance.concept);
-          for (const ancestorConcept of ancestorConcepts) {
-            if (ancestorConcept.name.toLowerCase() === rule.if[propertyType].type.toLowerCase()) {
+          const ancestorConcepts = objectInstance.type.ancestors;
+          ancestorConcepts.push(objectInstance.type);
+          for (let j = 0; j < ancestorConcepts.length; j += 1) {
+            if (ancestorConcepts[j].name.toLowerCase() === rule.if[propertyType].type.toLowerCase()) {
               if (rule.then.relationship && rule.then.relationship.type === subjectInstance.type.name) {
                 objectInstance.addRelationship(rule.then.relationship.label, subjectInstance, false, source);
               } else if (rule.then.value && rule.then.value.type === subjectInstance.type.name) {

test/CEParser.js

@@ -44,22 +44,6 @@ describe('CEParser', function() {
       expect(node.concepts.plant.relationships.length).to.be(1);
       expect(node.concepts.plant.relationships[0].label).to.be('grows into');
     });
-    it('prevent multi-conceptualisation', () => {
-      node.addCE('conceptualise a ~ river ~ R');
-      node.addCE('conceptualise a ~ river ~ R');
-      let counter = 0;
-      for (const concept of node.concepts) {
-        if (concept.name === 'river'){
-          counter += 1;
-        }
-      }
-      expect(counter).to.equal(1);
-    });
-    it('ensure concepts can be addressed by synonyms', () => {
-      node.addCE('conceptualise a ~ seat ~ that ~ is expressed by ~ chair and has the value V as ~ height ~');
-      node.addCE('there is a chair named chair1 that has 43cm as height');
-      expect(node.instances.chair1.height).to.equal('43cm');
-    });
   });   
   
   
@@ -106,50 +90,6 @@ describe('CEParser', function() {
       expect(node.instances.jane.subConcepts[0].name).to.be('barrister');
       expect(node.instances.jane.subConcepts[1].name).to.be('londoner');
     });
-    
-    it('prevent multi-instantiation', () => {
-      node.addCE('there is a person named Francesca');
-      node.addCE('there is a person named Francesca');
-      let counter = 0;
-      for (const instance of node.instances) {
-        if (instance.name === 'Francesca'){
-          counter += 1;
-        }
-      }
-      expect(counter).to.equal(1);
-    });
-    it('ensure instance CE is correct', () => {
-      node.addCE('there is an entity named Hagrid');
-      const hagrid = node.instances.hagrid;
-      expect(hagrid.ce).to.equal('there is a entity named \'Hagrid\'.');
-      node.addCE('the entity Hagrid is a person');
-      expect(hagrid.ce).to.equal('there is a entity named \'Hagrid\' that is a person.');
-    });
-    
-    it('ensure instances can be addressed by synonyms', () => {
-      node.addCE('conceptualise an ~ engineer ~ E');
-      node.addCE('there is a person named William that is expressed by Will');
-      node.addCE('the person Will is an engineer');
-      expect(node.instances.william.subConcepts[0].name).to.be('engineer');
-    });
-    it('ensure instances inherit properties from subConcepts', () => {
-      node.addCE('conceptualise a ~ borough ~ B');
-      node.addCE('conceptualise the londoner L ~ lives in ~ the borough B');
-      node.addCE('conceptualise the barrister B has the value V as ~ speciality ~');
-      node.addCE('there is a person named Amy that is a londoner and is a barrister');
-      node.addCE('the person Amy lives in the borough Chelsea and has \'family law\' as speciality');
-      expect(node.instances.amy.lives_in.name).to.be('Chelsea');
-      expect(node.instances.amy.speciality).to.be('family law');
-    });
-    it('ensure strings with a mix of quoted and unquoted names/values are parsed', () => {
-      node.addCE('there is a londoner named Ella that lives in the borough \'Kensington and Chelsea\'');
-      node.addCE('there is a londoner named \'Betty Hughes\' that lives in the borough Camden');
-      node.addCE('there is a londoner named Sally');
-      node.addCE('the londoner Sally lives in the borough \'Kensington and Chelsea\'');
-      expect(node.instances.ella.lives_in.name).to.be('Kensington and Chelsea');
-      expect(node.instances.betty_hughes.lives_in.name).to.be('Camden');
-      expect(node.instances.sally.lives_in.name).to.be('Kensington and Chelsea');
-    });
   });
 
   describe('Specific Examples', function() {
@@ -178,12 +118,11 @@ describe('CEParser', function() {
       expect(node.instances.fred.works_for.name).to.be('IBM');
     });
 
-    it('the person Fred works for the company IBM and is married to the person Jane and has 53 as age and has the city Cardiff as address.', function() {
+    it('the person Fred works for the company IBM and is married to the person Jane and has 53 as age.', function() {
       node.addCE('conceptualise a ~ company ~ C');
-      node.addCE('conceptualise a ~ city ~ C');
-      node.addCE('conceptualise a ~ person ~ P that ~ works for ~ the company C and ~ is married to ~ the person Q and has the value V as ~ age ~ and has the city C as ~ address ~');
+      node.addCE('conceptualise a ~ person ~ P that ~ works for ~ the company C and ~ is married to ~ the person Q and has the value V as ~ age ~');
       node.addCE('there is a person named Fred');
-      node.addCE('the person Fred works for the company IBM and is married to the person Jane and has 53 as age and has the city Cardiff as address.');
+      node.addCE('the person Fred works for the company IBM and is married to the person Jane and has 53 as age.');
       expect(node.instances.fred.works_for.name).to.be('IBM');
       expect(node.instances.fred.is_married_to.name).to.be('Jane');
       expect(node.instances.fred.age).to.be('53');
@@ -219,5 +158,7 @@ describe('CEParser', function() {
       expect(node.concepts.farmer.parents[1].name).to.be('land owner');
     });
   });
+
+
 });
 

test/QuestionParser.js

@@ -1,40 +0,0 @@
-const CENode = require('../src/CENode.js');
-const CEModels = require('../models/index.js');
-const expect = require('expect.js');
-const myName = 'User'
-const PLANETS_MODEL = [
-  "there is a rule named 'r1' that has 'if the planet C ~ orbits ~ the star D then the star D ~ is orbited by ~ the planet C' as instruction.",
-  "there is a rule named 'r2' that has 'if the planet C ~ is orbited by ~ the moon D then the moon D ~ orbits ~ the planet C' as instruction.",
-  "conceptualise a ~ celestial body ~ C.",
-  "conceptualise the celestial body C ~ orbits ~ the celestial body D and ~ is orbited by ~ the celestial body E.",
-  "conceptualise a ~ planet ~ P that is a celestial body and is an imageable thing.",
-  "conceptualise a ~ star ~ S that is a celestial body.",
-  "there is a star named sun.",
-  "there is a planet named Venus that orbits the star 'sun' and has 'media/Venus.jpg' as image.",
-  "there is a planet named Mercury that orbits the star 'sun' and has 'media/Mercury.jpg' as image."
-]
-
-let node;
-describe('CEQuestionParser', function() {
-  describe('What relation questions', function () {
-    this.timeout(2050);
-    before(function() {
-      node = new CENode(CEModels.core, PLANETS_MODEL);
-      node.attachAgent();
-      node.agent.setName('agent1');
-    });
-    it('returns the correct number of responses', (done) => {
-      const message = 'what orbits the sun?';
-      const askCard = "there is a nl card named '{uid}' that is to the agent 'agent1' and is from the individual '" + myName + "' and has the timestamp '{now}' as timestamp and has '" + message.replace(/'/g, "\\'")+"' as content.";
-      
-      node.addSentence(askCard);
-      
-      setTimeout(function() {
-        const cards = node.concepts.card.allInstances;
-        const card = cards[cards.length - 1];
-        expect(card.content).to.equal('Venus orbits the star sun. Mercury orbits the star sun.');
-        done();
-      }, 2000);
-    });
-  });
-});