refactor: update variable checker and waiting for promises

Author: jorenrui

lib/state.js

@@ -79,12 +79,14 @@ export class State {
 
   addVariable(variable, entityID) {
     const variables = this.variables.get(variable) || []
+    if (variables.includes(entityID)) return
     this.variables.set(variable, [...variables, entityID])
   }
 
   addEntityVariable(groupID, variable, entityID) {
     const key = `${groupID}.${variable}`
     const variables = this.entityVariables.get(key) || []
+    if (variables.includes(entityID)) return
     this.entityVariables.set(key, [...variables, entityID])
   }
 
@@ -205,10 +207,14 @@ export class State {
   }
 
   evaluate() {
-    Array.from(this.entities.values()).forEach(async (entity) => {
-      entity.attributes.evaluate()
+    const promises = []
+
+    Array.from(this.entities.values()).forEach((entity) => {
+      promises.push(entity.attributes.evaluate())
     })
 
+    await Promise.all(promises)
+    
     this.attachVariableHelpers(Array.from(this.variables.keys()))
     this.shouldUpdate = true
   }