Merge pull request #1431 from valadaptive/eslint-fixes-3

ESLint fixes part 3 - more significant changes

public/scripts/extensions/tts/coqui.js

@@ -4,17 +4,15 @@ TODO:
  - Delete useless call
 */
 
-import { doExtrasFetch, extension_settings, getApiUrl, getContext, modules, ModuleWorkerWrapper } from "../../extensions.js"
+import { doExtrasFetch, extension_settings, getApiUrl, modules } from "../../extensions.js"
 import { callPopup } from "../../../script.js"
 import { initVoiceMap } from "./index.js"
 
 export { CoquiTtsProvider }
 
 const DEBUG_PREFIX = "<Coqui TTS module> ";
-const UPDATE_INTERVAL = 1000;
 
 let inApiCall = false;
-let voiceIdList = []; // Updated with module worker
 let coquiApiModels = {}; // Initialized only once
 let coquiApiModelsFull = {}; // Initialized only once
 let coquiLocalModels = []; // Initialized only once
@@ -495,7 +493,7 @@ class CoquiTtsProvider {
                 .append('<option value="none">Select language</option>')
                 .val('none');
 
-            for (var i = 0; i < model_settings["languages"].length; i++) {
+            for (let i = 0; i < model_settings["languages"].length; i++) {
                 const language_label = JSON.stringify(model_settings["languages"][i]).replaceAll("\"", "");
                 $("#coqui_api_model_settings_language").append(new Option(language_label, i));
             }
@@ -514,7 +512,7 @@ class CoquiTtsProvider {
                 .append('<option value="none">Select speaker</option>')
                 .val('none');
 
-            for (var i = 0; i < model_settings["speakers"].length; i++) {
+            for (let i = 0; i < model_settings["speakers"].length; i++) {
                 const speaker_label = JSON.stringify(model_settings["speakers"][i]).replaceAll("\"", "");
                 $("#coqui_api_model_settings_speaker").append(new Option(speaker_label, i));
             }

public/scripts/extensions/tts/index.js

@@ -17,10 +17,7 @@ const UPDATE_INTERVAL = 1000
 
 let voiceMapEntries = []
 let voiceMap = {} // {charName:voiceid, charName2:voiceid2}
-let audioControl
 let storedvalue = false;
-let lastCharacterId = null
-let lastGroupId = null
 let lastChatId = null
 let lastMessageHash = null
 
@@ -314,8 +311,6 @@ let currentAudioJob
 let audioPaused = false
 let audioQueueProcessorReady = true
 
-let lastAudioPosition = 0
-
 async function playAudioData(audioBlob) {
     // Since current audio job can be cancelled, don't playback if it is null
     if (currentAudioJob == null) {
@@ -407,14 +402,12 @@ function addAudioControl() {
             TTS Playback
         </div>`)
     $('#ttsExtensionMenuItem').attr('title', 'TTS play/pause').on('click', onAudioControlClicked)
-    audioControl = document.getElementById('tts_media_control')
     updateUiAudioPlayState()
 }
 
 function completeCurrentAudioJob() {
     audioQueueProcessorReady = true
     currentAudioJob = null
-    lastAudioPosition = 0
     talkingAnimation(false) //stop lip animation
     // updateUiPlayState();
 }
@@ -463,8 +456,6 @@ function completeTtsJob() {
 
 function saveLastValues() {
     const context = getContext()
-    lastGroupId = context.groupId
-    lastCharacterId = context.characterId
     lastChatId = context.chatId
     lastMessageHash = getStringHash(
         (context.chat.length && context.chat[context.chat.length - 1].mes) ?? ''

public/scripts/extensions/tts/system.js

@@ -191,7 +191,7 @@ class SystemTtsProvider {
         const voice = speechSynthesis.getVoices().find(x => x.voiceURI === voiceId);
 
         if (!voice) {
-            throw `TTS Voice name ${voiceName} not found`
+            throw `TTS Voice id ${voiceId} not found`
         }
 
         speechSynthesis.cancel();

public/scripts/extensions/tts/xtts.js

@@ -133,8 +133,7 @@ class XTTSTtsProvider {
 
     // Perform a simple readiness check by trying to fetch voiceIds
     async checkReady() {
-
-        const response = await this.fetchTtsVoiceObjects()
+        await this.fetchTtsVoiceObjects()
     }
 
     async onRefreshClick() {