@tiledesk/tiledesk-tybot-connector 0.4.2 → 0.5.0-rc2

package/tiledeskChatbotPlugs/directives/DirAddTags.js

@@ -9,6 +9,7 @@ const req = require("express/lib/request");
 const { update } = require("../../models/faq");
 const { TiledeskClient } = require("@tiledesk/tiledesk-client");
 require('dotenv').config();
+const winston = require('../../utils/winston');
 
 class DirAddTags {
 
@@ -33,13 +34,13 @@ class DirAddTags {
   }
 
   execute(directive, callback) {
-    if (this.log) { console.log("AddTags directive: ", directive); }
+    winston.verbose("Execute AddTags directive");
     let action;
     if (directive.action) {
       action = directive.action;
     }
     else {
-      console.error("Incorrect directive: ", JSON.stringify(directive));
+      winston.warn("Incorrect directive: ", directive);
       callback();
       return;
     }
@@ -49,9 +50,9 @@ class DirAddTags {
   }
 
   async go(action, callback) {
-    if (this.log) { console.log("(DirAddTags) action:", JSON.stringify(action)); }
+    winston.debug("(DirAddTags) Action: ", action);
     if (!this.tdcache) {
-      console.error("Error: (DirAddTags) tdcache is mandatory");
+      winston.error("(DirAddTags) Error: tdcache is mandatory");
       callback();
       return;
     }
@@ -64,7 +65,7 @@ class DirAddTags {
     pushToList = action.pushToList
 
     if (!action.tags || action.tags === '') {
-      console.error("Error: (DirAddTags) tags attribute is mandatory")
+      winston.error("(DirAddTags) Error: tags attribute is mandatory")
       await this.chatbot.addParameter("flowError", "Add tags Error: tags attribute is mandatory");
       callback();
       return;
@@ -78,18 +79,7 @@ class DirAddTags {
 
     const filler = new Filler();
     const filled_tags = filler.fill(action.tags, requestVariables);
-  
-    if (this.log) {
-      console.log("(DirAddTags) tags: ", filled_tags);
-    }
-
-    // let request = await this.tdClient.getRequestById(this.requestId);
-    // if (this.log) { console.log('(DirAddTags) request detail: ', request) }
-    // if(!request){
-    //   if (this.log) { console.log("(DirAddTags) - request not found for request_id:", this.requestId); }
-    //   callback();
-    //   return;
-    // }
+    winston.debug("(DirAddTags) filled_tags: ", filled_tags);
 
     /** use case: CONVERSATION */
     if(target === 'request'){
@@ -106,7 +96,7 @@ class DirAddTags {
         })
       }
 
-      if (this.log) { console.log('(DirAddTags) UPDATE request with newTags', newTags) }
+      winston.debug('(DirAddTags) UPDATE request with newTags', newTags)
       let updatedRequest = await this.updateRequestWithTags(newTags)
       if(!updatedRequest){
         callback();
@@ -120,9 +110,9 @@ class DirAddTags {
       let newTags = filled_tags.split(',').filter(tag => tag !== '').map(el => el.trim())
 
       let request = await this.tdClient.getRequestById(this.requestId);
-      if (this.log) { console.log('(DirAddTags) request detail: ', request) }
+      winston.debug('(DirAddTags) request detail: ', request)
       if(!request){
-        if (this.log) { console.log("(DirAddTags) - request not found for request_id:", this.requestId); }
+        winston.debug("(DirAddTags) - request not found for request_id: " + this.requestId);
         callback();
         return;
       }
@@ -137,7 +127,7 @@ class DirAddTags {
         })
       }
 
-      if (this.log) {  console.log('(DirAddTags) UPDATE lead with newTags', newTags) }
+      winston.debug('(DirAddTags) UPDATE lead with newTags ', newTags)
       let updatedLead = await this.updateLeadWithTags(request.lead._id, newTags)
       if(!updatedLead){
         callback();
@@ -165,71 +155,7 @@ class DirAddTags {
 
   }
 
-  async #executeCondition(result, trueIntent, trueIntentAttributes, falseIntent, falseIntentAttributes, callback) {
-    let trueIntentDirective = null;
-    if (trueIntent) {
-      trueIntentDirective = DirIntent.intentDirectiveFor(trueIntent, trueIntentAttributes);
-    }
-    let falseIntentDirective = null;
-    if (falseIntent) {
-      falseIntentDirective = DirIntent.intentDirectiveFor(falseIntent, falseIntentAttributes);
-    }
-    if (result === true) {
-      if (trueIntentDirective) {
-        this.intentDir.execute(trueIntentDirective, () => {
-          if (callback) {
-            callback();
-          }
-        })
-      }
-      else {
-        if (this.log) { console.log("No trueIntentDirective specified"); }
-        if (callback) {
-          callback();
-        }
-      }
-    }
-    else {
-      if (falseIntentDirective) {
-        this.intentDir.execute(falseIntentDirective, () => {
-          if (callback) {
-            callback();
-          }
-        });
-      }
-      else {
-        if (this.log) { console.log("No falseIntentDirective specified"); }
-        if (callback) {
-          callback();
-        }
-      }
-    }
-  }
-
-  async #assignAttributes(action, answer) {
-    if (this.log) {
-      console.log("assignAttributes action:", action)
-      console.log("assignAttributes answer:", answer)
-    }
-    if (this.context.tdcache) {
-      if (action.assignReplyTo && answer) {
-        await TiledeskChatbot.addParameterStatic(this.context.tdcache, this.context.requestId, action.assignReplyTo, answer);
-      }
-      // Debug log
-      if (this.log) {
-        const all_parameters = await TiledeskChatbot.allParametersStatic(this.context.tdcache, this.context.requestId);
-        for (const [key, value] of Object.entries(all_parameters)) {
-          if (this.log) { console.log("(DirAddTags) request parameter:", key, "value:", value, "type:", typeof value) }
-        }
-      }
-    }
-  }
-
   #myrequest(options, callback) {
-    if (this.log) {
-      console.log("API URL:", options.url);
-      console.log("** Options:", JSON.stringify(options));
-    }
     let axios_options = {
       url: options.url,
       method: options.method,
@@ -239,9 +165,6 @@ class DirAddTags {
     if (options.json !== null) {
       axios_options.data = options.json
     }
-    if (this.log) {
-      console.log("axios_options:", JSON.stringify(axios_options));
-    }
     if (options.url.startsWith("https:")) {
       const httpsAgent = new https.Agent({
         rejectUnauthorized: false,
@@ -250,10 +173,6 @@ class DirAddTags {
     }
     axios(axios_options)
       .then((res) => {
-        if (this.log) {
-          console.log("Response for url:", options.url);
-          console.log("Response headers:\n", JSON.stringify(res.headers));
-        }
         if (res && res.status == 200 && res.data) {
           if (callback) {
             callback(null, res.data);
@@ -289,7 +208,7 @@ class DirAddTags {
       this.#myrequest(
         HTTPREQUEST, async (err, resbody) => {
           if (err) {
-            console.error("(httprequest) DirAddTags add tags to list err: ", err);
+            winston.error("(httprequest) DirAddTags add tags to list err: ", err);
             resolve(true)
           } else {
             if (resbody) {
@@ -309,9 +228,7 @@ class DirAddTags {
       let json = []
       let filteredTags = tags.map((tag) => ({tag: tag, color: '#f0806f'}))
       json.push(...filteredTags)
-      if (this.log) {
-        console.log('(httprequest) DirAddTags updateRequestWithTags tags--> ', json)
-      }
+      winston.debug('(httprequest) DirAddTags updateRequestWithTags tags: ', json)
       const HTTPREQUEST = {
         url: this.API_ENDPOINT + "/" + this.context.projectId + "/requests/" + this.requestId + '/tag',
         headers: {
@@ -325,7 +242,7 @@ class DirAddTags {
       this.#myrequest(
         HTTPREQUEST, async (err, resbody) => {
           if (err) {
-            console.error("(httprequest) DirAddTags patch request with new tags err: ", err);
+            winston.error("(httprequest) DirAddTags patch request with new tags err: ", err);
             resolve(true)
           } else {
             if (resbody) {
@@ -354,7 +271,7 @@ class DirAddTags {
       this.#myrequest(
         HTTPREQUEST, async (err, resbody) => {
           if (err) {
-            console.error("(httprequest) DirAddTags put lead with new tags err: ", err);
+            winston.error("(httprequest) DirAddTags put lead with new tags err: ", err);
             resolve(true)
           } else {
             if (resbody) {

package/tiledeskChatbotPlugs/directives/DirAiPrompt.js

@@ -94,8 +94,8 @@ class DirAiPrompt {
       }
     }
 
-    const llm_endpoint = process.env.KB_ENDPOINT_QA;
-    if (this.log) { console.log("DirAiPrompt llm_endpoint ", llm_endpoint); }
+    const AI_endpoint = process.env.AI_ENDPOINT
+    if (this.log) { console.log("DirAiPrompt AI_endpoint ", AI_endpoint); }
 
     let key = await this.getKeyFromIntegrations(action.llm);
 
@@ -130,7 +130,7 @@ class DirAiPrompt {
     if (this.log) { console.log("DirAiPrompt json: ", json) }
 
     const HTTPREQUEST = {
-      url: llm_endpoint + '/ask',
+      url: AI_endpoint + '/ask',
       headers: {
         'Content-Type': 'application/json'
       },

package/tiledeskChatbotPlugs/directives/DirAskGPT.js

@@ -4,6 +4,7 @@ const { Filler } = require('../Filler');
 let https = require("https");
 const { DirIntent } = require("./DirIntent");
 require('dotenv').config();
+const winston = require('../../utils/winston')
 
 class DirAskGPT {
 
@@ -20,13 +21,13 @@ class DirAskGPT {
   }
 
   execute(directive, callback) {
-    if (this.log) { console.log("AskGPT directive: ", directive); }
+    winston.verbose("Execute AskGPT directive");
     let action;
     if (directive.action) {
       action = directive.action;
     }
     else {
-      console.error("Incorrect directive: ", JSON.stringify(directive));
+      winston.warn("DirAskGPT Incorrect directive: ", directive);
       callback();
       return;
     }
@@ -36,9 +37,9 @@ class DirAskGPT {
   }
 
   async go(action, callback) {
-    if (this.log) { console.log("DirAskGPT action:", JSON.stringify(action)); }
+    winston.debug("(DirAskGPT) Action: ", action);
     if (!this.tdcache) {
-      console.error("Error: DirAskGPT tdcache is mandatory");
+      winston.error("Error: DirAskGPT tdcache is mandatory");
       callback();
       return;
     }
@@ -49,19 +50,17 @@ class DirAskGPT {
     let trueIntentAttributes = action.trueIntentAttributes;
     let falseIntentAttributes = action.falseIntentAttributes;
 
-    if (this.log) {
-      console.log("DirAskGPT trueIntent", trueIntent)
-      console.log("DirAskGPT falseIntent", falseIntent)
-      console.log("DirAskGPT trueIntentAttributes", trueIntentAttributes)
-      console.log("DirAskGPT falseIntentAttributes", falseIntentAttributes)
-    }
+    winston.debug("(DirAskGPT) trueIntent " + trueIntent)
+    winston.debug("(DirAskGPT) falseIntent " + falseIntent)
+    winston.debug("(DirAskGPT) trueIntentAttributes " + trueIntentAttributes)
+    winston.debug("(DirAskGPT) falseIntentAttributes " + falseIntentAttributes)
 
     // default values
     let answer = "No answers";
     let source = null;
 
     if (!action.question || action.question === '') {
-      console.error("Error: DirAskGPT question attribute is mandatory. Executing condition false...");
+      winston.error("(DirAskGPT) Error: question attribute is mandatory. Executing condition false...");
       await this.#assignAttributes(action, answer, source);
       if (falseIntent) {
         await this.#executeCondition(false, trueIntent, trueIntentAttributes, falseIntent, falseIntentAttributes);
@@ -71,7 +70,7 @@ class DirAskGPT {
     }
 
     if (!action.kbid) {
-      console.error("Error: DirAskGPT kbid attribute is mandatory. Executing condition false...");
+      winston.error("(DirAskGPT) Error: kbid attribute is mandatory. Executing condition false...");
       await this.#assignAttributes(action, answer, source);
       if (falseIntent) {
         await this.#executeCondition(false, trueIntent, trueIntentAttributes, falseIntent, falseIntentAttributes)
@@ -90,22 +89,22 @@ class DirAskGPT {
     const filled_question = filler.fill(action.question, requestVariables);
 
     const kb_endpoint = process.env.KB_ENDPOINT;
-    if (this.log) { console.log("DirAskGPT KbEndpoint URL: ", kb_endpoint); }
+    winston.verbose("DirAskGPT KbEndpoint URL: ", kb_endpoint);
 
     let key = await this.getKeyFromIntegrations();
     if (!key) {
-      if (this.log) { console.log("DirAskGPT - Key not found in Integrations. Searching in kb settings..."); }
+      winston.debug("(DirAskGPT) - Key not found in Integrations. Searching in kb settings...");
       key = await this.getKeyFromKbSettings();
     }
 
     if (!key) {
-      if (this.log) { console.log("DirAskGPT - Retrieve public gptkey")}
+      winston.debug("(DirAskGPT) - Retrieve public gptkey")
       key = process.env.GPTKEY;
       publicKey = true;
     }
 
     if (!key) {
-      console.error("Error: DirAskGPT gptkey is mandatory");
+      winston.error("(DirAskGPT) Error: gptkey is mandatory");
       await this.#assignAttributes(action, answer);
       if (falseIntent) {
         await this.#executeCondition(false, trueIntent, trueIntentAttributes, falseIntent, falseIntentAttributes);
@@ -119,7 +118,7 @@ class DirAskGPT {
     if (publicKey === true) {
       let keep_going = await this.checkQuoteAvailability();
       if (keep_going === false) {
-        if (this.log) { console.log("DirAskGPT - Quota exceeded for tokens. Skip the action")}
+        winston.debug("(DirAskGPT) - Quota exceeded for tokens. Skip the action")
         callback();
         return;
       }
@@ -130,26 +129,25 @@ class DirAskGPT {
       kbid: action.kbid,
       gptkey: key
     };
-    if (this.log) { console.log("DirAskGPT json:", json); }
+    winston.debug("(DirAskGPT)DirAskGPT json:", json); 
 
     const HTTPREQUEST = {
       url: kb_endpoint + "/qa",
       json: json,
       method: "POST"
     }
-    if (this.log) { console.log("DirAskGPT HTTPREQUEST", HTTPREQUEST); }
+    winston.debug("(DirAskGPT) HttpRequest", HTTPREQUEST); 
     
     this.#myrequest(
       HTTPREQUEST, async (err, resbody) => {
-        if (this.log && err) {
-          console.log("DirAskGPT error: ", err);
-        }
-        if (this.log) { console.log("DirAskGPT resbody:", resbody); }
+
+        winston.debug("(DirAskGPT) resbody:", resbody); 
         let answer = resbody.answer;
         let source = resbody.source_url;
         await this.#assignAttributes(action, answer, source);
-
+        
         if (err) {
+          winston.error("(DirAskGPT) error: ", err);
           if (callback) {
             if (falseIntent) {
               await this.#executeCondition(false, trueIntent, trueIntentAttributes, falseIntent, falseIntentAttributes);
@@ -205,7 +203,7 @@ class DirAskGPT {
         })
       }
       else {
-        if (this.log) { console.log("No trueIntentDirective specified"); }
+        winston.debug("(DirAskGPT) No trueIntentDirective specified");
         if (callback) {
           callback();
         }
@@ -220,7 +218,7 @@ class DirAskGPT {
         });
       }
       else {
-        if (this.log) { console.log("No falseIntentDirective specified"); }
+        winston.debug("(DirAskGPT) No falseIntentDirective specified");
         if (callback) {
           callback();
         }
@@ -229,36 +227,20 @@ class DirAskGPT {
   }
 
   async #assignAttributes(action, answer, source) {
-    if (this.log) {
-      console.log("assignAttributes action:", action)
-      console.log("assignAttributes answer:", answer)
-      console.log("assignAttributes source:", source)
-    }
+    winston.debug("(DirAskGPT) assignAttributes action:", action)
+    winston.debug("(DirAskGPT) assignAttributes answer:", answer)
+    winston.debug("(DirAskGPT) assignAttributes source:", source)
     if (this.context.tdcache) {
       if (action.assignReplyTo && answer) {
         await TiledeskChatbot.addParameterStatic(this.context.tdcache, this.context.requestId, action.assignReplyTo, answer);
       }
-      // console.log("--> action.assignSourceTo: ", action.assignSourceTo)
-      // console.log("--> source: ", source)
       if (action.assignSourceTo && source) {
-        // console.log("--> source: ", source)
         await TiledeskChatbot.addParameterStatic(this.context.tdcache, this.context.requestId, action.assignSourceTo, source);
       }
-      // Debug log
-      if (this.log) {
-        const all_parameters = await TiledeskChatbot.allParametersStatic(this.context.tdcache, this.context.requestId);
-        for (const [key, value] of Object.entries(all_parameters)) {
-          if (this.log) { console.log("(askgpt) request parameter:", key, "value:", value, "type:", typeof value) }
-        }
-      }
     }
   }
 
   #myrequest(options, callback) {
-    if (this.log) {
-      console.log("API URL:", options.url);
-      console.log("** Options:", JSON.stringify(options));
-    }
     let axios_options = {
       url: options.url,
       method: options.method,
@@ -268,9 +250,6 @@ class DirAskGPT {
     if (options.json !== null) {
       axios_options.data = options.json
     }
-    if (this.log) {
-      console.log("axios_options:", JSON.stringify(axios_options));
-    }
     if (options.url.startsWith("https:")) {
       const httpsAgent = new https.Agent({
         rejectUnauthorized: false,
@@ -279,10 +258,6 @@ class DirAskGPT {
     }
     axios(axios_options)
       .then((res) => {
-        if (this.log) {
-          console.log("Response for url:", options.url);
-          console.log("Response headers:\n", JSON.stringify(res.headers));
-        }
         if (res && res.status == 200 && res.data) {
           if (callback) {
             callback(null, res.data);
@@ -295,7 +270,7 @@ class DirAskGPT {
         }
       })
       .catch((error) => {
-        console.error("(DirAskGPT) Axios error: ", JSON.stringify(error));
+        winston.error("(DirAskGPT) Axios error: ", error.response.data);
         if (callback) {
           callback(error, null);
         }
@@ -313,12 +288,12 @@ class DirAskGPT {
         },
         method: "GET"
       }
-      if (this.log) { console.log("DirAskGPT INTEGRATIONS_HTTPREQUEST ", INTEGRATIONS_HTTPREQUEST) }
+      winston.debug("(DirAskGPT) Integrations HttpRequest ", INTEGRATIONS_HTTPREQUEST)
 
       this.#myrequest(
         INTEGRATIONS_HTTPREQUEST, async (err, integration) => {
           if (err) {
-            if (this.log) { console.error("DirAskGPT Get integrations error ", err); }
+            if (this.log) { winston.error("DirAskGPT Get integrations error ", err); }
             resolve(null);
           } else {
 
@@ -345,12 +320,12 @@ class DirAskGPT {
         },
         method: "GET"
       }
-      if (this.log) { console.log("DirAskGPT KB_HTTPREQUEST", KB_HTTPREQUEST); }
+      winston.debug("(DirAskGPT) KB HttpRequest ", KB_HTTPREQUEST);
 
       this.#myrequest(
         KB_HTTPREQUEST, async (err, resbody) => {
           if (err) {
-            if (this.log) { console.error("DirAskGPT Get kb settings error ", err); }
+            winston.error("DirAskGPT Get kb settings error ", err?.response?.data);
             resolve(null);
           } else {
             if (!resbody.gptkey) {