@tiledesk/tiledesk-tybot-connector 2.0.12-rc5 → 2.0.12-rc6

package/tiledeskChatbotPlugs/directives/DirIfOpenHours_OLD.js

@@ -1,6 +1,7 @@
 const { TiledeskClient } = require('@tiledesk/tiledesk-client');
 const { DirIntent } = require('./DirIntent');
 const ms = require('minimist-string');
+const winston = require('../../utils/winston');
 
 class DirIfOpenHours {
 
@@ -26,9 +27,6 @@ class DirIfOpenHours {
       let params;
       params = this.parseParams(directive.parameter);
       if (!params.trueIntent && !params.falseIntent) {
-        if (this.log) {
-          console.log("missing both params.trueIntent & params.falseIntent");
-        }
         callback();
         return;
       }
@@ -60,14 +58,14 @@ class DirIfOpenHours {
     if (falseIntent && falseIntent.trim() === "") {
       falseIntent = null;
     }
-    if (this.log) {console.log("condition action:", action);}
+    winston.verbose("(DirIfOpenHours) Action:", action);
     if (!trueIntent && !falseIntent) {
-      if (this.log) {console.log("Invalid condition, no intents specified");}
-      callback();
+      winston.error("(DirIfOpenHours) Error: missing both action.trueIntent & action.falseIntent");
+            callback();
       return;
     }
     this.tdClient.openNow((err, result) => {
-      if (this.log) {console.log("openNow():", result);}
+      winston.error("(DirIfOpenHours) openNow():", result)
       if (err) {
         console.error("*** DirIfOpenHours Error:", err);
         callback();
@@ -75,7 +73,7 @@ class DirIfOpenHours {
       else if (result && result.isopen) {
         if (trueIntent) {
           let intentDirective = DirIntent.intentDirectiveFor(trueIntent);
-          if (this.log) {console.log("DirIfOpenHours (openHours) => trueIntent");}
+          winston.debug("(DirIfOpenHours) (openHours) => trueIntent ");
           this.intentDir.execute(intentDirective, () => {
             callback(stopOnConditionMet);
           });
@@ -87,7 +85,7 @@ class DirIfOpenHours {
       }
       else if (falseIntent) {
         let intentDirective = DirIntent.intentDirectiveFor(falseIntent);
-        if (this.log) {console.log("DirIfOpenHours (openHours) => falseIntent", falseIntent);}
+        winston.debug("(DirIfOpenHours) (openHours) => falseIntent ");
         this.intentDir.execute(intentDirective, () => {
           callback(stopOnConditionMet);
         });

package/tiledeskChatbotPlugs/directives/DirIntent.js

@@ -16,7 +16,6 @@ class DirIntent {
     this.TILEBOT_ENDPOINT = context.TILEBOT_ENDPOINT;
     this.supportRequest = context.supportRequest;
     this.token = context.token;
-    this.log = context.log;
   }
 
   execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirLockIntent.js

@@ -12,7 +12,6 @@ class DirLockIntent {
       throw new Error('tdcache (TdCache) object is mandatory.');
     }
     this.tdcache = this.context.tdcache;
-    this.log = context.log;
   }
 
   async execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirMake.js

@@ -204,21 +204,7 @@ class DirMake {
 
       })
       .catch((err) => {
-        if (this.log) {
-          // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - START
-          let cache = [];
-          let error_log = JSON.stringify(err, function (key, value) { // try to use a separate function
-            if (typeof value === 'object' && value != null) {
-              if (cache.indexOf(value) !== -1) {
-                return;
-              }
-              cache.push(value);
-            }
-            return value;
-          });
-          winston.error("(DirMake) An error occurred: ", error_log);
-          // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - END;
-        }
+        // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - END;
         if (callback) {
           let status = 1000;
           let cache = [];

package/tiledeskChatbotPlugs/directives/DirMessage.js

@@ -16,7 +16,6 @@ class DirMessage {
     this.requestId = context.requestId;
     this.tdcache = this.context.tdcache;
     this.token = context.token;
-    this.log = this.context.log;
     this.supportRequest = this.context.supportRequest
 
     this.tdClient = new TiledeskClient({ projectId: this.context.projectId, token: this.context.token, APIURL: this.API_ENDPOINT, APIKEY: "___", log: this.log });

package/tiledeskChatbotPlugs/directives/DirMessageToBot.js

@@ -15,7 +15,6 @@ class DirMessageToBot {
     this.TILEBOT_ENDPOINT = context.TILEBOT_ENDPOINT;
     this.supportRequest = context.supportRequest;
     this.token = context.token;
-    this.log = context.log;
   }
 
   execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirMoveToAgent.js

@@ -1,4 +1,3 @@
-// const { TiledeskClient } = require('@tiledesk/tiledesk-client');
 const { Directives } = require('./Directives');
 const { TiledeskChatbot } = require('../../engine/TiledeskChatbot');
 const { TiledeskChatbotConst } = require('../../engine/TiledeskChatbotConst');

package/tiledeskChatbotPlugs/directives/DirMoveToUnassigned.js

@@ -1,4 +1,3 @@
-// const { TiledeskClient } = require('@tiledesk/tiledesk-client');
 const { Directives } = require('./Directives');
 const { TiledeskChatbot } = require('../../engine/TiledeskChatbot');
 const { TiledeskChatbotConst } = require('../../engine/TiledeskChatbotConst');

package/tiledeskChatbotPlugs/directives/DirReply.js

@@ -84,7 +84,7 @@ class DirReply {
             let command = commands[i];
             if (command.type === 'message' && command.message && command.message.text) {
               command.message.text = filler.fill(command.message.text, requestAttributes);
-              TiledeskChatbotUtil.fillCommandAttachments(command, requestAttributes, this.log);
+              TiledeskChatbotUtil.fillCommandAttachments(command, requestAttributes);
               winston.debug("DirReply command filled: " + command.message.text);
             }
             if (command.type === 'settings' && command.settings) {

package/tiledeskChatbotPlugs/directives/DirReplyV2.js

@@ -21,6 +21,7 @@ class DirReplyV2 {
     this.requestId = context.requestId;
     this.token = context.token;
     this.tdcache = context.tdcache;
+    this.intentDir = new DirIntent(context);
     this.chatbot = context.chatbot;
     this.reply = context.reply;
     this.originalMessage = context.message;
@@ -59,11 +60,10 @@ class DirReplyV2 {
     const message = action;
 
     let current; // debug only
-    if (this.log) {
-      if (message.attributes.commands[1].message.text) {
-        current = message.attributes.commands[1].message.text
-      }
+    if (message.attributes.commands[1].message.text) {
+      current = message.attributes.commands[1].message.text
     }
+    
     let must_stop = false;
     // fill
     let requestAttributes = null;
@@ -72,11 +72,9 @@ class DirReplyV2 {
       await TiledeskChatbot.allParametersStatic(
         this.tdcache, this.requestId
       );
-      if (this.log) {
-        for (const [key, value] of Object.entries(requestAttributes)) {
-          const value_type = typeof value;
-        }
-      }
+      // for (const [key, value] of Object.entries(requestAttributes)) {
+      //   const value_type = typeof value;
+      // }
 
       TiledeskChatbotUtil.replaceJSONButtons(message, requestAttributes);
 
@@ -195,7 +193,7 @@ class DirReplyV2 {
             let command = commands[i];
             if (command.type === 'message' && command.message && command.message.text) {
               command.message.text = filler.fill(command.message.text, requestAttributes);
-              TiledeskChatbotUtil.fillCommandAttachments(command, requestAttributes, this.log);
+              TiledeskChatbotUtil.fillCommandAttachments(command, requestAttributes);
             }
           }
         }

package/tiledeskChatbotPlugs/directives/DirSetAttribute.js

@@ -77,7 +77,6 @@ class DirSetAttribute {
             throw new Error('context object is mandatory.');
         }
         this.context = context;
-        this.log = context.log;
     }
 
     execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirSetAttributeV2.js

@@ -205,9 +205,7 @@ class DirSetAttributeV2 {
         httpUtils.request(
             HTTPREQUEST, async (err, resbody) => {
                 if (err) {
-                    if (this.log) {
-                        winston.error("(DirSetAttributeV2) persistOnTiledesk() error: ", err);
-                    }
+                    winston.error("(DirSetAttributeV2) persistOnTiledesk() error: ", err);
                 } else {
                     winston.debug("(DirSetAttributeV2) Attributes saved: ", resbody);
                 }

package/tiledeskChatbotPlugs/directives/DirSetConversationTags.js

@@ -9,7 +9,6 @@ class DirSetConversationTags {
       throw new Error('context object is mandatory.');
     }
     this.context = context;
-    this.log = context.log;
     this.requestId = context.requestId;
     this.API_ENDPOINT = context.API_ENDPOINT;
     

package/tiledeskChatbotPlugs/directives/DirUnlockIntent.js

@@ -11,7 +11,6 @@ class DirUnlockIntent {
       throw new Error('tdcache (TdCache) object is mandatory.');
     }
     this.tdcache = context.tdcache;
-    this.log = context.log;
   }
 
   async execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirWebRequest.js

@@ -13,7 +13,6 @@ class DirWebRequest {
     this.context = context;
     this.tdcache = context.tdcache;
     this.requestId = context.requestId;
-    this.log = context.log;
   }
 
   execute(directive, callback) {

package/tiledeskChatbotPlugs/directives/DirWebRequestV2.js

@@ -317,21 +317,7 @@ class DirWebRequestV2 {
           }
         })
         .catch((err) => {
-          if (this.log) {
-            // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - START
-            let cache = [];
-            let error_log = JSON.stringify(err, function (key, value) { // try to use a separate function
-              if (typeof value === 'object' && value != null) {
-                if (cache.indexOf(value) !== -1) {
-                  return;
-                }
-                cache.push(value);
-              }
-              return value;
-            });
-            winston.error("(DirWebRequestv2) An error occurred: ", error_log);
-            // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - END
-          }
+          // FIX THE STRINGIFY OF CIRCULAR STRUCTURE BUG - END
           if (callback) {
             let status = 1000;
             let cache = [];

package/tiledeskChatbotPlugs/directives/Directives.js

@@ -60,6 +60,7 @@ class Directives {
   static ADD_TAGS = 'add_tags'
   static WEB_RESPONSE = "web_response";
   static FLOW_LOG = "flow_log";
+  static ADD_KB_CONTENT = "add_kb_content";
 
   // static WHEN_ONLINE_MOVE_TO_AGENT = "whenonlinemovetoagent"; // DEPRECATED?
   // static WHEN_OFFLINE_HOURS = "whenofflinehours"; // DEPRECATED // adds a message on top of the original message when offline hours opts: --replace

package/utils/TiledeskChatbotUtil.js

@@ -252,24 +252,24 @@ class TiledeskChatbotUtil {
         }
     }
 
-    static fillCommandAttachments(command, variables, log) {
-        if (log) { winston.debug("(TiledeskChatbotUtils) Filling command button: ", command) }
+    static fillCommandAttachments(command, variables) {
+        winston.debug("(TiledeskChatbotUtils) Filling command button: ", command)
         if (command.message && command.message.attributes && command.message.attributes.attachment && command.message.attributes.attachment.buttons && command.message.attributes.attachment.buttons.length > 0) {
             let buttons = command.message.attributes.attachment.buttons;
             const filler = new Filler();
             buttons.forEach(button => {
                 if (button.link) {
                     button.link = filler.fill(button.link, variables);
-                    if (log) { winston.debug("(TiledeskChatbotUtils) button.link filled: " + button.link) }
+                    winston.debug("(TiledeskChatbotUtils) button.link filled: " + button.link)
                 }
                 if (button.value) {
                     button.value = filler.fill(button.value, variables);
-                    if (log) { winston.debug("(TiledeskChatbotUtils) button.value filled: " + button.value) }
+                    winston.debug("(TiledeskChatbotUtils) button.value filled: " + button.value)
                 }
             });
         }
         else {
-            if (log) { winston.debug("(TiledeskChatbotUtils) No attachments to fill in command") }
+            winston.debug("(TiledeskChatbotUtils) No attachments to fill in command")
         }
     }
 
@@ -520,7 +520,7 @@ class TiledeskChatbotUtil {
     static async updateRequestAttributes(chatbot, chatbotToken, message, projectId, requestId) {
         // update request context
         try {
-            if (chatbot.log) { winston.debug("Updating request variables. Message:", message); }
+            winston.debug("Updating request variables. Message:", message);
             const messageId = message._id;
             const chat_url = `https://panel.tiledesk.com/v3/dashboard/#/project/${projectId}/wsrequest/${requestId}/messages`
 
@@ -591,12 +591,12 @@ class TiledeskChatbotUtil {
             }
             
             if (message && message.request && message.request.lead) {
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Lead found with email: " + message.request.lead.email + " and lead.fullname " + message.request.lead.fullname); }
+                winston.debug("(TiledeskChatbotUtil) Lead found with email: " + message.request.lead.email + " and lead.fullname " + message.request.lead.fullname);
                 let currentLeadEmail = await chatbot.getParameter(TiledeskChatbotConst.REQ_LEAD_EMAIL_KEY);
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) You lead email from attributes: " + currentLeadEmail); }
+                winston.debug("(TiledeskChatbotUtil) You lead email from attributes: " + currentLeadEmail);
                 if (message.request.lead.email && !currentLeadEmail) {
                     // worth saving
-                    if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) worth saving email"); }
+                    winston.debug("(TiledeskChatbotUtil) worth saving email");
                     try {
                         await chatbot.addParameter(TiledeskChatbotConst.REQ_LEAD_EMAIL_KEY, message.request.lead.email);
                     }
@@ -685,10 +685,10 @@ class TiledeskChatbotUtil {
                     message.attributes = {}
                 }
                 message.attributes.payload = { ...message.attributes.payload, ...message.request.attributes.payload }
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Forced Set message.attributes.payload ", message.attributes.payload); }
+                winston.debug("(TiledeskChatbotUtil) Forced Set message.attributes.payload ", message.attributes.payload); 
             }
             if (message.attributes) {
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Ok message.attributes ", message.attributes); }
+                winston.debug("(TiledeskChatbotUtil) Ok message.attributes ", message.attributes);
 
                 await chatbot.addParameter(TiledeskChatbotConst.REQ_END_USER_ID_KEY, message.attributes.requester_id);
                 await chatbot.addParameter(TiledeskChatbotConst.REQ_END_USER_IP_ADDRESS_KEY, message.attributes.ipAddress);
@@ -722,12 +722,12 @@ class TiledeskChatbotUtil {
             
             
             const _bot = chatbot.bot; // aka FaqKB
-            if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Adding Globals to context: ", _bot); }
+            winston.debug("(TiledeskChatbotUtil) Adding Globals to context: ", _bot); 
             
             if (_bot.attributes && _bot.attributes.globals) {
-                if (chatbot.log) { winston.error("(TiledeskChatbotUtil) Got Globals: ", _bot.attributes.globals); }
+                winston.error("(TiledeskChatbotUtil) Got Globals: ", _bot.attributes.globals);
                 _bot.attributes.globals.forEach(async (global_var) => {
-                    if (chatbot.log) { winston.error("(TiledeskChatbotUtil) Adding global: " + global_var.key + " value: " + global_var.value); }
+                    winston.error("(TiledeskChatbotUtil) Adding global: " + global_var.key + " value: " + global_var.value);
                     await chatbot.addParameter(global_var.key, global_var.value);
                 });
             }
@@ -740,20 +740,6 @@ class TiledeskChatbotUtil {
             //     }
             // );
             
-            if (chatbot.log) {
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) requestId: " + requestId); }
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Key: " + TiledeskChatbotConst.REQ_PROJECT_ID_KEY); }
-
-                let proj_ = await chatbot.getParameter(TiledeskChatbotConst.REQ_PROJECT_ID_KEY);
-                if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) request parameter proj_: " + proj_); }
-
-                const all_parameters = await chatbot.allParameters();
-                for (const [key, value] of Object.entries(all_parameters)) {
-                    const value_type = typeof value;
-                    if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) Request Attribute: " + key + " value: " + value + " type: " + value_type); }
-                    
-                }
-            }
         } catch(error) {
             winston.error("(TiledeskChatbotUtil) updateRequestAttributes Error: ", error);
             process.exit(1)