better folder opening + subs embedded in the video directly

Author: pricisTrail

src-tauri/.gitignore

@@ -5,3 +5,5 @@
 # Generated by Tauri
 # will have schema files for capabilities auto-completion
 /gen/schemas
+
+README.md

src-tauri/src/lib.rs

@@ -441,6 +441,7 @@ async fn start_download(
         ffmpeg_path,
         "--merge-output-format".to_string(),
         "mp4".to_string(),
+        "--embed-thumbnail".to_string(),
         "--no-keep-fragments".to_string(),
         "-P".to_string(),
         home_path,
@@ -489,6 +490,9 @@ async fn start_download(
         args.push("--write-subs".to_string());
         args.push("--write-auto-sub".to_string());
         args.push("--embed-subs".to_string());
+        // Remove subtitle sidecars after embedding to avoid .vtt file clutter.
+        args.push("--compat-options".to_string());
+        args.push("no-keep-subs".to_string());
         args.push("--sub-langs".to_string());
         // Limit subtitle downloads to English variants to avoid fetching dozens of auto-translated tracks.
         args.push("en.*,en,-live_chat".to_string());
@@ -984,14 +988,45 @@ async fn update_ytdlp(app: AppHandle) -> Result<String, String> {
     Ok(new_version)
 }
 
+#[tauri::command]
+async fn open_folder(path: String) -> Result<(), String> {
+    #[cfg(target_os = "windows")]
+    {
+        // Normalize: replace forward slashes with backslashes and strip any trailing separator
+        let normalized = path.replace('/', "\\");
+        let normalized = normalized.trim_end_matches('\\');
+        tokio::process::Command::new("explorer.exe")
+            .arg(normalized)
+            .spawn()
+            .map_err(|e| e.to_string())?;
+        return Ok(());
+    }
+    #[cfg(target_os = "macos")]
+    {
+        tokio::process::Command::new("open")
+            .arg(&path)
+            .spawn()
+            .map_err(|e| e.to_string())?;
+        return Ok(());
+    }
+    #[cfg(target_os = "linux")]
+    {
+        tokio::process::Command::new("xdg-open")
+            .arg(&path)
+            .spawn()
+            .map_err(|e| e.to_string())?;
+        return Ok(());
+    }
+}
+
 #[cfg_attr(mobile, tauri::mobile_entry_point)]
 pub fn run() {
     tauri::Builder::default()
         .plugin(tauri_plugin_shell::init())
         .plugin(tauri_plugin_dialog::init())
         .plugin(tauri_plugin_fs::init())
         .plugin(tauri_plugin_opener::init())
-        .invoke_handler(tauri::generate_handler![start_download, fetch_formats, fetch_playlist_info, cancel_download, check_ytdlp_update, update_ytdlp])
+        .invoke_handler(tauri::generate_handler![start_download, fetch_formats, fetch_playlist_info, cancel_download, check_ytdlp_update, update_ytdlp, open_folder])
         .run(tauri::generate_context!())
         .expect("error while running tauri application");
 }

src/App.tsx

@@ -5,7 +5,6 @@ import { twMerge } from "tailwind-merge";
 import { invoke } from "@tauri-apps/api/core";
 import { listen } from "@tauri-apps/api/event";
 import { open } from "@tauri-apps/plugin-dialog";
-import { revealItemInDir } from "@tauri-apps/plugin-opener";
 import { downloadDir } from "@tauri-apps/api/path";
 
 function cn(...inputs: ClassValue[]) {
@@ -448,12 +447,10 @@ export default function App() {
     setHistory(prev => prev.filter(h => h.id !== id));
   };
 
-  const openFolder = async (path: string) => {
-    try {
-      await revealItemInDir(path);
-    } catch (error) {
+  const openFolder = (path: string) => {
+    invoke('open_folder', { path }).catch(error => {
       console.error('Failed to open folder:', error);
-    }
+    });
   };
 
   // Handle TXT file import