feat(lyrics-plus): optimize lyrics convert performance & improved code quality

[Lseoksee]

Changes

lyrics Convert Performance Improvement

• Introduced caching for converted lyrics and optimized the process by ensuring that only the selected Display Mode is applied instead of converting into all formats for a single language.

Unifying Convert and Translation Code

• Improved code consistency between Convert and Translation functionalities.

improved Lyrics Mode Select

• If the lyrics are not in lockMode, the last used mode for that lyrics is stored in memory.
  When the lyrics are reloaded, they will automatically switch to the saved mode.

Remove Korean-To-Korean Conversion

• Converting lyrics detected as Korean back into Korean is unnecessary. Only romaja conversion will be retained.

Provider Label Update for Spotify Official Lyrics

• Previously, Spotify's official lyrics provider was displayed as musixmatch; this has now been changed to Spotify.

Please let me know if you have any feedback on this.

New Features

• Added a function in Lyrics Config to clear cached lyrics from memory.
• Added an option to delete individual locally saved lyrics.
  • Pressing the Save Lyrics button again will remove the stored lyrics.

[Copilot]

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Files not reviewed (1)

• CustomApps/lyrics-plus/OptionsMenu.js: Evaluated as low risk

Comments suppressed due to low confidence (3)

CustomApps/lyrics-plus/Translator.js:48

• The call to injectExternals inside the setInterval seems redundant and could lead to performance issues. Consider checking if the external scripts are already loaded before calling this function.

this.injectExternals(language);

CustomApps/lyrics-plus/Translator.js:53

• The clearInterval and resolve calls should be inside the if (this.finished[lan]) block to ensure they are only executed when the condition is met.

clearInterval(interval);

CustomApps/lyrics-plus/Utils.js:12

• The element 'p1' is not a valid HTML element. It should be 'p'.

return react.createElement("p1", null, reactChildren);