ekms 7.12.4 → 7.12.5-beta.0

package/common/fastfood.js

@@ -30,6 +30,8 @@ const fastfood_instance = require('./fastfood.instance.json')
   double hamburger
   number 1 and 2
   number 1 2 and 3
+  combo 1 through 5
+
 */
 
 const template = {
@@ -160,7 +162,6 @@ const template = {
        { "context": [['smoothie_ingredient', 1], ['list', 0], ['smoothie_ingredient', 1], ['smoothie', 1]], ordered: true, choose: [1] },
 
        { "context": [['list', 0], ['number', 1], ['combo', 1], ['number', 1]], ordered: true, choose: [2,3] },
-       { "context": [['withModification', 0], ['modification', 1], ['list', 0], ['modification', 1]], ordered: true, choose: [2] },
 
 
        { context: [['comboNumber', 0], ['counting',0]], choose: [0] },
@@ -224,7 +225,7 @@ const template = {
         { 
           id: 'withModification',
           level: 0,
-          isA: ['preposition'],
+          before: ['preposition'],
           generatorp: ({context, gp}) => `with ${gp(context.modifications)}`,
           bridge: "{ ...next(operator), modifications: after[0], flatten: false }",
         },
@@ -250,7 +251,7 @@ const template = {
         { 
           id: 'comboNumber',
           convolution: true,
-          before: ['combo'],
+          before: ['combo', 'preposition'],
           bridge: "{ ...next(before[0]), postModifiers: append(before[0].postModifiers, ['comboNumber']), comboNumber: after[0], instance: true, flatten: true }",
         },
         { 
@@ -297,17 +298,24 @@ const template = {
             const naArray = api.getAskedForButNotAvailable()
             naArray.forEach((f) => f.paraphrase = true)
             const naContext = toContext(naArray)
-            /*
-            naContext.isResponse = true
-            naContext.marker = 'verbatim'
-            naContext.verbatim = `The following are not menu items: ${gp(naContext)}`
-            insert(naContext)
-            */
             verbatim(`The following are not menu items: ${gp(naContext)}`)
             // allow other motivation to run
             context.cascade = true
           }
         },
+        {
+          where: where(),
+          match: ({context, api}) => context.marker == 'controlEnd' && api.hasAskedForButNotAvailableModification(),
+          apply: ({context, api, gp, toContext, verbatim}) => {
+            const naArray = api.getAskedForButNotAvailableModification().map(({ item, modification }) => {
+              // return `${gp(item)} can not be modified with ${gp(modification)}.`
+              return `XXX can not be modified with ${gp(modification)}.`
+            })
+            verbatim(naArray.join(' '))
+            // allow other motivation to run
+            context.cascade = true
+          }
+        },
       ]
     },
     ({ask, api}) => {
@@ -342,17 +350,35 @@ const template = {
         {
           where: where(),
           oneShot: false,
-          matchq: (args) => askAbout(args).length > 0,
+          onNevermind: ({verbatim, ...args}) => {
+            // this is cross km boundaries from the dialogues km to this one so the api if for dialogs. 
+            // i need to get the one for fastfood here.
+            const api = args.kms.fastfood.api
+            const needsDrink = askAbout({ args, api })
+            for (const item of needsDrink) {
+              api.remove(item)
+            }
+          },
+          matchq: (args) => askAbout(args).length > 0 && args.context.marker == 'controlEnd',
           applyq: (args) => {
             args.context.cascade = true
             const needsDrink = askAbout(args)
+            // const details = args.gp({ marker: 'list', value: needsDrink.map((item) => item.food)})
             if (needsDrink.length > 1) {
               return `What drinks do you want?`
             } else {
               return `What drink do you want?`
             }
           },
-          matchr: (args) => args.isA(args.context.marker, 'drink') && askAbout(args).length > 0,
+          matchr: (args) => {
+            if (args.isA(args.context.marker, 'drink') && askAbout(args).length > 0) {
+              const needsDrink = askAbout(args)
+              if (args.api.isAvailableModification(needsDrink[0].food, { ...args.context , id: args.context.value })) {
+                return true
+              }
+            }
+            return false
+          },
           applyr: (args) => {
             // TODO check for is available for all modifications
             const needsDrink = askAbout(args)
@@ -375,6 +401,33 @@ const template = {
       ])
     },
     {
+      operators: [
+        "([change] (meal/* || drink/*) (to/1))",
+      ],
+      hierarchy: [
+        ['meal', 'toAble'],
+        ['drink', 'toAble'],
+      ],
+      bridges: [
+        { 
+          id: "change",
+          isA: ['verby'],
+          localHierarchy: [ ['thisitthat', 'meal'] ],
+          generatorp: ({context, gp}) => `change ${gp(context.from)} to ${gp(context.to)}`,
+          bridge: "{ ...next(operator), from: after[0], to: after[1].toObject }",
+          semantic: ({context, api, e}) => {
+            const state = api.state
+            const eFrom = e(context.from).evalue
+            const from = state.getIdCombo(eFrom.fromSTM ? eFrom : context.from)
+            const to = state.getIdCombo(context.to)
+            for (const item of api.items()) {
+              if (item.id == from.id) {
+                api.modify(item, { id: to.id, food: context.to })
+              }
+            }
+          }
+        },
+      ],
       priorities: [
         { context: [['combo', 0], ['number',1], ['list', 0], ['combo', 0]], ordered: true, choose: [0,1] },
         { context: [['list', 0], ['combo',0], ['number',1]], ordered: true, choose: [1,2] },
@@ -382,6 +435,41 @@ const template = {
         { context: [['mango', 0], ['passion',0], ['list', 0]], ordered: true, choose: [0,1] },
         { context: [['number', 1], ['mango_passion',1], ['list', 0]], ordered: true, choose: [0,1] },
         { context: [['mango', 0], ['mango_passion',0], ['passion',0], ['list', 0]], ordered: true, choose: [0,1,2] },
+
+        { context: [['drink', 0], ['list',0], ['combo',0], ['number', 0]], ordered: true, choose: [2,3] },
+        { context: [['drink', 1], ['list',0], ['combo',0], ['number', 0]], ordered: true, choose: [2,3] },
+        { context: [['withModification', 0], ['modification', 1], ['list', 0], ['modification', 1]], ordered: true, choose: [2] },
+        { context: [['withModification', 0], ['modification', 1], ['list', 0], ['combo', 1]], ordered: true, choose: [0] },
+        { context: [['combo', 2], ['list', 0], ['combo', 1], ['withModification', 1]], ordered: true, choose: [3] },
+      ],
+    },
+    {
+      operators: [
+        "([remove|remove,delete,drop,ditch,forget,no] (food/*))",
+        "([reset|reset,restart,clear])",
+      ],
+      bridges: [
+        {
+          id: 'remove',
+          isA: ['verby'],
+          bridge: "{ ...next(operator), remove: after[0], postModifiers: ['remove'] }",
+          semantic: ({context, api}) => {
+            const state = api.state
+            for (const item of api.items()) {
+              if (state.match(context.remove, item)) {
+                api.remove(item)
+              }
+            }
+          }
+        },
+        {
+          id: 'reset',
+          isA: ['verby'],
+          bridge: "{ ...next(operator) }",
+          semantic: ({context, api}) => {
+            api.reset()
+          }
+        },
       ],
     },
   ],
@@ -390,35 +478,66 @@ const template = {
 class API {
   initialize({ objects, config }) {
     this._objects = objects
-    this._objects.items = []
-    this._objects.notAvailable = []
+    this.reset()
   }
 
   show() {
     this._objects.show = this._objects.items
   }
 
+  toItem(item_id) {
+    if (Array.isArray(item_id)) {
+      return this._objects.items[item_id[0]].modifications[item_id[1]]
+    } else {
+      return this._objects.items[item_id]
+    }
+  }
+
+  new_item_id() {
+    const item_id = this._objects.item_id_counter
+    this._objects.item_id_counter += 1
+    return item_id
+  }
+
   // returns an item id so things can be updated if needed
   add(item) {
-    item.item_id = this._objects.items.length
+    item.item_id = this.new_item_id()
     if (!item.modifications) {
       item.modifications = []
     }
+    item.modifications.forEach((modification, index) => {
+      modification.item_id = [item.item_id, index]
+    })
     this._objects.items.push(item)
-    return item.item_id 
+  }
+
+  reset() {
+    this._objects.items = []
+    this._objects.notAvailable = []
+    this._objects.notAvailableModification = []
+    this._objects.item_id_counter = 0
   }
 
   get(item_id) {
-    return this._objects.items[item_id]
+    return this.toItem(item_id)
+  }
+
+  modify(item, changes) {
+    Object.assign(this.toItem(item.item_id), changes)
+  }
+
+  remove(item) {
+    this._objects.items = this._objects.items.filter( (i) => i.item_id !== item.item_id )
   }
 
   items() {
-    return this._objects.items
+    return [...this._objects.items]
   }
 
   addDrink(item_id, drink) {
-    this._objects.items[item_id].modifications.push(drink)
-    this._objects.items[item_id].needsDrink = false
+    const item = this.toItem(item_id)
+    item.modifications.push(drink)
+    item.needsDrink = false
   }
 
   say(response) {
@@ -430,17 +549,40 @@ class API {
     return this._objects.notAvailable.length > 0
   }
 
+  hasAskedForButNotAvailableModification(item) {
+    return this._objects.notAvailableModification.length > 0
+  }
+
   getAskedForButNotAvailable(item) {
     const na = this._objects.notAvailable
     this._objects.notAvailable = []
     return na
   }
 
+  getAskedForButNotAvailableModification(item) {
+    const na = this._objects.notAvailableModification
+    this._objects.notAvailableModification = []
+    return na
+  }
+
   addAskedForButNotAvailable(item) {
     this._objects.notAvailable.push(item)
   }
 
+  addAskedForButNotAvailableModification(item, modification) {
+    this._objects.notAvailableModification.push({item, modification})
+  }
+
+  isAvailableModification(food, modification) {
+    if (this.isAvailable(modification)) {
+      if (this.args.isA(modification.id, 'fry') || this.args.isA(modification.id, 'pop')) {
+        return true
+      }
+    }
+  }
+
   isAvailable(item) {
+    item.id = item.id || item.value
     if (item.id == 'chicken_nugget') {
       if (![4,5,6,10].includes(item.pieces)) {
         return false
@@ -468,6 +610,10 @@ class API {
       item.id = 'coca_cola'
     }
 
+    if (item.id == 'fry') {
+      item.id = 'french_fry'
+    }
+
     return [
       "hamburger",
       "cheeseburger",
@@ -558,6 +704,27 @@ class API {
     }
     return map[number]
   }
+
+  canBeCombo(id) {
+    return this.getComboNumber(id) > 0
+  }
+
+  getComboNumber(id) {
+    const combos = [
+       'single',
+       'double',
+       'triple',
+       'baconator',
+       'bacon_deluxe',
+       'spicy',
+       'homestyle',
+       'asiago_range_chicken_club',
+       'ultimate_chicken_grill',
+       'chicken_nugget',
+       'premium_cod',
+       ]
+    return combos.findIndex((e) => e == id) + 1
+  }
 }
 
 const api = new API()
@@ -567,17 +734,13 @@ class State {
     this.api = api
   }
 
-  add(food) {
-    let quantity = 1
-    if (food.quantity) {
-      quantity = food.quantity.value
-    }
+  getIdCombo(food) {
     let id, combo
     if (food.comboNumber?.marker == 'numberNumberCombo') {
       id = this.api.getCombo(food.comboNumber.comboNumber.value)
       if (!id) {
         this.api.addAskedForButNotAvailable(food)
-        return
+        return { done: true }
       }
       combo = true
     }
@@ -585,11 +748,11 @@ class State {
       id = this.api.getCombo(food.comboNumber.value)
       if (!id) {
         this.api.addAskedForButNotAvailable(food)
-        return
+        return { done: true }
       }
       combo = true
     } else if (food.marker == 'combo') {
-      id = food.type.value
+      id = food.type?.value
       combo = true
     } else {
       id = food.value
@@ -600,6 +763,32 @@ class State {
       id = 'chicken_nugget'
     }
 
+    return { id, combo }
+  }
+
+  match(pattern, item) {
+    Object.assign(pattern, this.getIdCombo(pattern))
+    if (pattern.id == item.id) {
+      return true
+    }
+    if (!pattern.id) {
+      if (pattern.combo == item.combo) {
+        return true
+      }
+    }
+  }
+
+  add(food) {
+    let quantity = 1
+    if (food.quantity) {
+      quantity = food.quantity.value
+    }
+
+    const { id, combo, done } = this.getIdCombo(food)
+    if (done) {
+      return
+    }
+
     const addSize = (item, data) => {
       if (item.size) {
         data.size = item.size.value
@@ -607,15 +796,39 @@ class State {
       return data
     }
 
+    const getAvailableChildren = (item) => {
+      // see if this is a categories of items 
+      const descendants = this.api.args.hierarchy.descendants(item.id)
+      const available = []
+      for (const descendant of descendants) {
+        if (this.api.isAvailable({ id: descendant})) {
+          available.push(descendant)
+        }
+      }
+      return available
+    }
+
     let modifications
+    const addsInsteadOfModifications = []
     if (food.modifications) {
       modifications = []
       for (const modification of propertyToArray(food.modifications.modifications)) {
         if (modification.size) {
           food.size = modification.size
         }
-        addSize(modification, { id: modification.value })
-        modifications.push(addSize(modification, { id: modification.value }))
+     
+        // if not a modification treat as top level request 
+        if (!this.api.isAvailableModification(food, { ...modification, id: modification.value })) {
+          if (this.api.isAvailable(modification)) {
+            //this.add(modification)
+            addsInsteadOfModifications.push(modification)
+          } else {
+            this.api.addAskedForButNotAvailable(modification)
+          }
+        } else {
+          addSize(modification, { id: modification.value })
+          modifications.push(addSize(modification, { id: modification.value }))
+        }
       }
     }
 
@@ -632,46 +845,82 @@ class State {
     for (let i = 0; i < quantity; ++i) {
       const item = addSize(food, { id, combo, modifications, pieces, food })
       if (!this.api.isAvailable(item)) {
-        this.api.addAskedForButNotAvailable(food)
-        return
-      }
-
-      const item_id = this.api.add(item)
-
-      if (false) {
-        // see if followup for drink is needed
-
-        const hasDrink = (item_id) => {
-          const item = this.api.get(item_id)
-          let hasDrink = false
-          for (let modification of (item.modifications || [])) {
-            if (!this.api.args.isA(modification.id, 'drink')) {
-              hasDrink = true
-              break
-            }
+        const available = []
+        for (const descendant of this.api.args.hierarchy.descendants(food.marker)) {
+          if (this.api.isAvailable({ id: descendant })) {
+            available.push(descendant)
           }
-          return hasDrink
-        }
-        const needsDrink = (item_id) => {
-          const item = this.api.get(item_id)
-          return item.needsDrink
         }
 
-        if (!hasDrink(item_id) && needsDrink(item_id)) {
+          // this sentence runs but it doesnt setup the hierarchy: 'combo 1, 2, 3, 4, 5, 6, 7, 9, 10, and 11 are combos'
+          // i made a wrong design choice when i setup the phrase 'combo 1 etc'. I should have mapped that to the 'single_combo'
+          // but instead had it be a combo with a comboNumber property. That means the language layer doesnt know about the mapping
+          // so that phrase doesnt work. if I set it up the other way that phrase would work. This is just a demo and I have other
+          // demoes to write so i am not fixin that and instead will do || is a combo
+
+        if (available.length > 0 || food.marker == 'combo') {
           this.api.args.ask([
-              {
-                where: where(),
-                matchq: ({objects}) => !hasDrink(item_id) && needsDrink(item_id),
-                applyq: () => `What drink do you want?`,
-                matchr: ({context, isA}) => isA(context.marker, 'drink'),
-                applyr: ({context, objects, api}) => {
-                  // TODO check for is available for all modifications
-                  this.api.addDrink(item_id, { id: context.value }) 
-                }
+            {
+              where: where(),
+              oneShot: true,
+              matchq: ({context}) => context.marker == 'controlEnd',
+              applyq: () => {
+                // args.context.cascade = true
+                const word = food.word
+                return `What kind of ${word}?`
               },
-            ]
-          )
+              semanticsr: [
+                // stuipid hack one because i didnt put combo fully into the NLI layer
+                {
+                  where: where(),
+                  match: ({context, callId, isA, api}) => {
+                    return api.canBeCombo(context.marker)
+                  },
+                  apply: ({context}) => {
+                    const comboNumber = {
+                      value: api.getComboNumber(context.marker)
+                    }
+                    food.comboNumber = comboNumber
+                    this.add(food)
+                  }
+                },
+                {
+                  where: where(),
+                  match: ({context, isA}) => isA(context.marker, 'number') && !context.evaluate,
+                  apply: ({context, e}) => {
+                    food.comboNumber = { value: e(context).value }
+                    this.add(Object.assign(food, context))
+                  }
+                },
+                {
+                  where: where(),
+                  match: ({context, isA}) => isA(context.marker, food.marker),
+                  apply: ({context}) => {
+                    this.add(Object.assign(food, context))
+                  }
+                },
+                {
+                  where: where(),
+                  match: ({context, isA}) => isA(context.marker, `${food.marker}_modifier`),
+                  apply: ({context}) => {
+                    const value = `${context.value}_${food.value}`
+                    this.add(Object.assign(food, { value }))
+                  }
+                },
+              ]
+            },
+          ])
+        } else {
+          this.api.addAskedForButNotAvailable(food)
         }
+        return
+      }
+
+      this.api.add(item)
+      this.api.args.mentioned(food)
+
+      for (const addIt of addsInsteadOfModifications) {
+        this.add(addIt)
       }
     }
   }
@@ -686,7 +935,7 @@ class State {
   }
 }
 
-const createConfig = () => {
+const createConfig = (additionalConfig) => {
   const config = new Config({ 
     name: 'fastfood',
     operators: [
@@ -708,7 +957,7 @@ const createConfig = () => {
       },
       {
         where: where(),
-        match: ({context, isAListable}) => isAListable(context, 'edible') && context.marker !== 'edible' && !context.same && !context.isResponse,
+        match: ({context, isAListable}) => isAListable(context, 'edible') && context.marker !== 'edible' && !context.same && !context.isResponse && !context.evaluate,
         apply: ({context, km, api, instance}) => {
           for (const element of propertyToArray(context)) {
             km('fastfood').api.state.add(element)
@@ -743,6 +992,9 @@ const createConfig = () => {
   config.initializer( ({api}) => {
     api.state = new State(api)
   })
+  if (additionalConfig) {
+    additionalConfig(config)
+  }
   config.restart_auto_rebuild()
   return config
 }
@@ -751,16 +1003,30 @@ knowledgeModule( {
     module,
     description: 'fastfood related concepts',
     createConfig,
+    acceptsAdditionalConfig: true,
     test: {
             name: './fastfood.test.json',
             contents: fastfood_tests,
             checks: {
               objects: [
                 'show', 
-                { property: 'items', filter: ['combo', 'pieces', 'size', 'item_id', 'id', 'modifications', 'needsDrink'] },
+                { 
+                  property: 'items', 
+                  filter: [
+                      'combo', 
+                      'pieces', 
+                      'size', 
+                      'item_id', 
+                      'id', 
+                      { property: 'food', filter: [ 'marker', 'value', 'text' ] }, 
+                      { property: 'modifications', filter: [ 'id', 'item_id', 'food' ] }, 
+                      'needsDrink'
+                  ],
+                },
                 'changes', 
                 'response', 
                 { property: 'notAvailable', filter: [ 'marker', 'value', 'text' ] }, 
+                { property: 'notAvailableModification', filter: [ 'marker', 'value', 'text' ] }, 
                 { property: 'quantity', filter: ['marker', 'value', 'text' ] },
                 { property: 'pieces', filter: ['marker', 'value', 'text' ] },
               ],