cleaned all redundant comments

d7cc70256aaec88fd6a0f0499656aebf31756661

YunZLu <130174259+YunZLu@users.noreply.github.com>

Signed

Parents: d511875db99ebd441563ec5e057e6dc102dbf345

public/scripts/extensions/tts/system.js+5 -32 Viewed

     //#################//
     fetchTtsVoiceObjects() {
         if (!('speechSynthesis' in window)) {
 -            // Browser doesn't support speech synthesis
             return Promise.resolve([]);
+        }
         return new Promise((resolve) => {
 -            // Use a minimal timeout to allow the voice list to potentially populate
             setTimeout(() => {
                 let voices = speechSynthesis.getVoices();
                 if (voices.length === 0) {
 -                    // If no voices returned (e.g., Edge on first load), provide a default option
 +                    // Edge compat: Provide default when voices empty
                     console.warn('SystemTTS: getVoices() returned empty list. Providing browser default option.');
                     const defaultVoice = {
                         name: SystemTtsProvider.BROWSER_DEFAULT_VOICE_NAME,
                         voice_id: SystemTtsProvider.BROWSER_DEFAULT_VOICE_ID,
                         preview_url: false,
 -                        // Try to guess the browser's default language
                         lang: navigator.language || 'en-US',
                     };
                     resolve([defaultVoice]);
                 } else {
 -                    // If voices are available, map them as before
                     const mappedVoices = voices
                         .sort((a, b) => a.lang.localeCompare(b.lang) || a.name.localeCompare(b.name))
                         .map(x => ({ name: x.name, voice_id: x.voiceURI, preview_url: false, lang: x.lang }));
                     resolve(mappedVoices);
+                }
 -            }, 50); // Increased timeout slightly just in case it helps voice population on some browsers
 +            }, 50);
         });
+    }
+-
     previewTtsVoice(voiceId) {
         if (!('speechSynthesis' in window)) {
             throw new Error('Speech synthesis API is not supported'); // Keep Error type for consistency
+        }
         let voice = null;
 -        // Check if the requested voice is NOT the browser default
         if (voiceId !== SystemTtsProvider.BROWSER_DEFAULT_VOICE_ID) {
             const voices = speechSynthesis.getVoices();
 -            // Try to find the actual voice
             voice = voices.find(x => x.voiceURI === voiceId);
             if (!voice && voices.length > 0) {
 -                // If voices are loaded but the specific ID wasn't found, log a warning
                 console.warn(`SystemTTS Preview: Voice ID "${voiceId}" not found among available voices. Using browser default.`);
 -                // Fallback to default (voice remains null)
             } else if (!voice && voices.length === 0) {
 -                // If no voices are loaded at all, we expect to use default
                 console.warn('SystemTTS Preview: Voice list is empty. Using browser default.');
 -                // Fallback to default (voice remains null)
+            }
         } else {
             console.log('SystemTTS Preview: Using browser default voice as requested.');
 -            // Use default (voice remains null)
+        }
         speechSynthesis.cancel(); // Stop any previous speech
 -        // Use the language from the found voice if available, otherwise default to 'en-US' or browser lang for the preview text
         const langForPreview = voice ? voice.lang : (navigator.language || 'en-US');
         const text = getPreviewString(langForPreview);
         const utterance = new SpeechSynthesisUtterance(text);
 -        // Only set the voice if we found a specific one and it wasn't the default request
         if (voice) {
             utterance.voice = voice;
+        }
 -        // Otherwise, utterance.voice remains null/undefined, causing the browser to use its default
         utterance.rate = this.settings.rate || 1;
         utterance.pitch = this.settings.pitch || 1;
 -        // Add error handling for the speech itself
         utterance.onerror = (event) => {
             console.error(`SystemTTS Preview Error: ${event.error}`, event);
 -            // Potentially notify the user here
         };
         speechSynthesis.speak(utterance);

     async getVoice(voiceName) {
         if (!('speechSynthesis' in window)) {
 -            // Return a predictable null-like structure if API not supported
             return { voice_id: null, name: 'API Not Supported' };
+        }
 -        // Check if the requested name is the browser default placeholder
         if (voiceName === SystemTtsProvider.BROWSER_DEFAULT_VOICE_NAME) {
             return {
                 voice_id: SystemTtsProvider.BROWSER_DEFAULT_VOICE_ID,

             };
+        }
 -        // Attempt to get voices, might be async
 -        // Note: This relies on voices potentially being populated by now.
 -        // A more robust approach might involve re-calling fetchTtsVoiceObjects if needed,
 -        // but sticking to minimal changes based on original code structure.
         const voices = speechSynthesis.getVoices();
         if (voices.length === 0) {
 -            // If voices are still empty, we can't find any specific name
 +            console.warn('SystemTTS: Empty voice list, using default fallback');
 -            console.warn(`SystemTTS getVoice: Voice list empty, cannot find "${voiceName}". Falling back to browser default ID.`);
 -            // Return the default placeholder as a fallback in this edge case
             return {
                 voice_id: SystemTtsProvider.BROWSER_DEFAULT_VOICE_ID,
                 name: SystemTtsProvider.BROWSER_DEFAULT_VOICE_NAME,

         const match = voices.find(x => x.name == voiceName);
         if (!match) {
 -            // If voices are loaded but name not found, throw error as before
             throw new Error(`SystemTTS getVoice: TTS Voice name "${voiceName}" not found`);
+        }

             speechUtteranceChunker(utterance, {
                 chunkLength: 200,
             }, function () {
 -                //some code to execute when done
                 resolve(silence);
                 console.log('System TTS done');
             });