@tiledesk/tiledesk-tybot-connector 2.0.8-rc1 → 2.0.8-rc3

package/tiledeskChatbotPlugs/directives/DirReplyV2.js

@@ -20,7 +20,6 @@ class DirReplyV2 {
     this.requestId = context.requestId;
     this.token = context.token;
     this.tdcache = context.tdcache;
-    this.log = context.log;
     this.intentDir = new DirIntent(context);
     this.chatbot = context.chatbot;
     this.reply = context.reply;
@@ -31,8 +30,7 @@ class DirReplyV2 {
       projectId: this.context.projectId,
       token: this.context.token,
       APIURL: this.API_ENDPOINT,
-      APIKEY: "___",
-      log: this.log
+      APIKEY: "___"
     });
   }
 
@@ -61,11 +59,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;
@@ -74,11 +71,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);
 
@@ -197,7 +192,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/DirSendEmail.js

@@ -15,15 +15,13 @@ class DirSendEmail {
     this.context = context;
     this.tdcache = context.tdcache;
     this.requestId = context.requestId;
-    this.log = context.log;    
 
     this.API_ENDPOINT = context.API_ENDPOINT;
     this.tdClient = new TiledeskClient({
       projectId: this.context.projectId,
       token: this.context.token,
       APIURL: this.API_ENDPOINT,
-      APIKEY: "___",
-      log: this.log
+      APIKEY: "___"
     });
   }
 

package/tiledeskChatbotPlugs/directives/DirSendWhatsapp.js

@@ -19,7 +19,6 @@ class DirSendWhatsapp {
     this.requestId = this.context.requestId;
     this.intentDir = new DirIntent(context);
     this.API_ENDPOINT = this.context.API_ENDPOINT;
-    this.log = context.log;
   }
 
   execute(directive, callback) {

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

@@ -79,7 +79,6 @@ class DirSetAttributeV2 {
         }
         this.context = context;
         this.tdcache = context.tdcache;
-        this.log = context.log;
     }
 
     execute(directive, callback) {
@@ -199,9 +198,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;
@@ -17,8 +16,7 @@ class DirSetConversationTags {
       projectId: this.context.projectId,
       token: this.context.token,
       APIURL: this.API_ENDPOINT,
-      APIKEY: "___",
-      log: this.log
+      APIKEY: "___"
     });
   }
 

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/DirWait.js

@@ -12,7 +12,6 @@ class DirWait {
     this.chatbot = context.chatbot;
     this.tdcache = context.tdcache;
     this.requestId = context.requestId;
-    this.log = context.log;
   }
 
   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

@@ -16,7 +16,6 @@ class DirWebRequestV2 {
     this.requestId = context.requestId;
     this.chatbot = context.chatbot;
     this.intentDir = new DirIntent(context);
-    this.log = context.log;
   }
 
   execute(directive, callback) {
@@ -302,21 +301,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/DirWebResponse.js

@@ -3,6 +3,7 @@ const { TiledeskChatbot } = require('../../engine/TiledeskChatbot');
 const { TiledeskChatbotUtil } = require('../../utils/TiledeskChatbotUtil');
 const winston = require('../../utils/winston');
 let axios = require('axios');
+const { Logger } = require('../../Logger');
 
 class DirWebResponse {
 
@@ -15,7 +16,7 @@ class DirWebResponse {
     this.requestId = context.requestId;
     this.token = context.token;
     this.tdcache = context.tdcache;
-    this.log = context.log;
+    this.logger = new Logger({ request_id: this.requestId, dev: this.context.supportRequest.draft });
   }
 
   execute(directive, callback) {
@@ -29,8 +30,9 @@ class DirWebResponse {
       callback();
       return;
     }
+    this.logger.info("Executing WebResponse action ", directive.action)
     this.go(action, () => {
-        // return stop true?
+      this.logger.info("WebResponse action terminated")
         callback();
     });
   }
@@ -59,6 +61,8 @@ class DirWebResponse {
       payload: json
     }
 
+    this.logger.debug("WebResponse payload: ", webResponse);
+
     const topic = `/webhooks/${this.requestId}`;
     
     try {

package/tiledeskChatbotPlugs/directives/DirWhatsappByAttribute.js

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

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")
         }
     }
 
@@ -496,7 +496,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`
 
@@ -567,12 +567,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);
                     }
@@ -581,10 +581,10 @@ class TiledeskChatbotUtil {
                     }
                 }
                 let currentLeadName = await chatbot.getParameter(TiledeskChatbotConst.REQ_LEAD_USERFULLNAME_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.fullname && !currentLeadName) {
                     // worth saving
-                    if (chatbot.log) { winston.debug("(TiledeskChatbotUtil) worth saving email"); }
+                    winston.debug("(TiledeskChatbotUtil) worth saving email");
                     try {
                         await chatbot.addParameter(TiledeskChatbotConst.REQ_LEAD_USERFULLNAME_KEY, message.request.lead.fullname);
                     }
@@ -661,10 +661,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);
@@ -698,12 +698,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);
                 });
             }
@@ -716,20 +716,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)