fix: spawn shell processes outside blocking workers (#4307)

Author: sxyazi

yazi-scheduler/src/process/shell.rs

@@ -1,7 +1,7 @@
 use std::{ffi::OsString, process::Stdio};
 
 use anyhow::Result;
-use tokio::process::{Child, Command};
+use tokio::{process::{Child, Command}, task};
 use yazi_fs::Cwd;
 use yazi_macro::impl_data_any;
 use yazi_shared::url::{AsUrl, UrlBuf};
@@ -30,42 +30,40 @@ impl ShellOpt {
 }
 
 pub(crate) async fn shell(opt: ShellOpt) -> Result<Child> {
-	tokio::task::spawn_blocking(move || {
-		let cwd = Cwd::ensure(opt.cwd.as_url());
+	let (cwd, opt) =
+		task::spawn_blocking(move || (Cwd::ensure(opt.cwd.as_url()).into_owned(), opt)).await?;
 
-		#[cfg(unix)]
-		return Ok(unsafe {
-			Command::new("sh")
-				.stdin(opt.stdio())
-				.stdout(opt.stdio())
-				.stderr(opt.stdio())
-				.arg("-c")
-				.arg(opt.cmd)
-				.current_dir(cwd)
-				.kill_on_drop(!opt.orphan)
-				.pre_exec(move || {
-					if !opt.block && libc::setsid() < 0 {
-						return Err(std::io::Error::last_os_error());
-					}
-					Ok(())
-				})
-				.spawn()?
-		});
+	#[cfg(unix)]
+	return Ok(unsafe {
+		Command::new("sh")
+			.stdin(opt.stdio())
+			.stdout(opt.stdio())
+			.stderr(opt.stdio())
+			.arg("-c")
+			.arg(opt.cmd)
+			.current_dir(cwd)
+			.kill_on_drop(!opt.orphan)
+			.pre_exec(move || {
+				if !opt.block && libc::setsid() < 0 {
+					return Err(std::io::Error::last_os_error());
+				}
+				Ok(())
+			})
+			.spawn()?
+	});
 
-		#[cfg(windows)]
-		return Ok(
-			Command::new("cmd.exe")
-				.stdin(opt.stdio())
-				.stdout(opt.stdio())
-				.stderr(opt.stdio())
-				.env("=", r#""^\n\n""#)
-				.raw_arg(r#"/Q /S /D /V:OFF /E:ON /C ""#)
-				.raw_arg(opt.cmd)
-				.raw_arg(r#"""#)
-				.current_dir(cwd)
-				.kill_on_drop(!opt.orphan)
-				.spawn()?,
-		);
-	})
-	.await?
+	#[cfg(windows)]
+	return Ok(
+		Command::new("cmd.exe")
+			.stdin(opt.stdio())
+			.stdout(opt.stdio())
+			.stderr(opt.stdio())
+			.env("=", r#""^\n\n""#)
+			.raw_arg(r#"/Q /S /D /V:OFF /E:ON /C ""#)
+			.raw_arg(opt.cmd)
+			.raw_arg(r#"""#)
+			.current_dir(cwd)
+			.kill_on_drop(!opt.orphan)
+			.spawn()?,
+	);
 }