fix setting initial config

lib/tasks/cognigy/index.js

@@ -99,21 +99,19 @@ class Cognigy extends Task {
     await super.exec(cs);
 
     const opts = {
-      session: {
-        synthesizer: this.data.synthesizer || {
-          vendor: 'default',
-          language: 'default',
-          voice: 'default'
-        },
-        recognizer: this.data.recognizer || {
-          vendor: 'default',
-          language: 'default'
-        },
-        bargein: this.data.bargein || {},
-        bot: this.data.bot || {},
-        user: this.data.user || {},
-        dtmf: this.data.dtmf || {}
-      }
+      synthesizer: this.data.synthesizer || {
+        vendor: 'default',
+        language: 'default',
+        voice: 'default'
+      },
+      recognizer: this.data.recognizer || {
+        vendor: 'default',
+        language: 'default'
+      },
+      bargein: this.data.bargein || {},
+      bot: this.data.bot || {},
+      user: this.data.user || {},
+      dtmf: this.data.dtmf || {}
     };
     this.config = new SpeechConfig({logger: this.logger, ep, opts});
     this.ep = ep;

lib/tasks/cognigy/speech-config.js

@@ -7,9 +7,6 @@ const stripNulls = (obj) => {
   return obj;
 };
 
-
-const defaultSpeechConfig
-
 class SpeechConfig extends Emitter {
   constructor({logger, ep, opts = {}}) {
     super();
@@ -19,16 +16,16 @@ class SpeechConfig extends Emitter {
     this.update(opts);
   }
 
-  _mergeConfig(changedConfig = {}){
+  _mergeConfig(changedConfig = {}) {
     const merged = lodash.merge(
-        {},
-        this.sessionConfig,
-        changedConfig
-      );
-      merged.bargein.enable = changedConfig.bargein?.enable?.length === 0 ? [] : changedConfig.bargein?.enable;
-      this.logger.debug({merged, sessionConfig: this.sessionConfig, changedConfig}, 'merged config');
-      // should we override hints with empty array or leave it as it is once saved?
-      // merged.recognizer.hints = changedConfig.recognizer?.hints
+      {},
+      this.sessionConfig,
+      changedConfig
+    );
+    merged.bargein.enable = changedConfig.bargein?.enable?.length === 0 ? [] : changedConfig.bargein?.enable;
+    this.logger.debug({merged, sessionConfig: this.sessionConfig, changedConfig}, 'merged config');
+    // should we override hints with empty array or leave it as it is once saved?
+    // merged.recognizer.hints = changedConfig.recognizer?.hints
   }
 
   update(session) {

lib/tasks/gather.js

@@ -130,10 +130,10 @@ class TaskGather extends Task {
       else startListening(cs, ep);
 
       if (this.input.includes('speech') && this.listenDuringPrompt) {
-          await this._initSpeech(cs, ep);
-          this._startTranscribing(ep);
-          updateSpeechCredentialLastUsed(this.sttCredentials.speech_credential_sid)
-            .catch(() => {/*already logged error */});
+        await this._initSpeech(cs, ep);
+        this._startTranscribing(ep);
+        updateSpeechCredentialLastUsed(this.sttCredentials.speech_credential_sid)
+          .catch(() => {/*already logged error */});
       }
 
       if (this.input.includes('digits') || this.dtmfBargein) {
@@ -318,7 +318,9 @@ class TaskGather extends Task {
     }
     if (evt.is_final) this._resolve('speech', evt);
     else {
-      if (evt.stability > 0.70 && this.bargein && evt.alternatives[0].transcript.split(' ').length >= this.minBargeinWordCount) {
+      if (evt.stability > 0.70 &&
+        this.bargein &&
+        evt.alternatives[0].transcript.split(' ').length >= this.minBargeinWordCount) {
         this.logger.debug('Gather:_onTranscription - killing audio due to bargein');
         this._killAudio(cs);
         this._resolve('speech', evt);