fix: prevent quotes in file(1) arguments from being stripped under MSYS2 (#3364)

Author: sxyazi

CHANGELOG.md

@@ -73,6 +73,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/):
 - Force Git checkout for plugin cache repositories ([#3169])
 - Check compatibility when reusing previewer bytecode cache ([#3190])
 - Disable kitty keyboard protocol on Windows due to `crossterm` inability to handle it ([#3250])
+- Prevent quotes in file(1) arguments from being stripped under MSYS2 ([#3364])
 - Expose `ya` CLI in the Snap build ([#2904])
 - Fallback to `PollWatcher` for file changes watching on NetBSD ([#2941])
 - Generate unique image IDs for Kgp to tolerate tmux ([#3038])
@@ -1540,3 +1541,4 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/):
 [#3313]: https://github.com/sxyazi/yazi/pull/3313
 [#3317]: https://github.com/sxyazi/yazi/pull/3317
 [#3360]: https://github.com/sxyazi/yazi/pull/3360
+[#3364]: https://github.com/sxyazi/yazi/pull/3364

Cargo.lock

@@ -35,7 +35,7 @@ hashbrown           = { version = "0.16.1", features = [ "serde" ] }
 indexmap            = { version = "2.12.1", features = [ "serde" ] }
 libc                = "0.2.177"
 lru                 = "0.16.2"
-mlua                = { version = "0.11.4", features = [ "anyhow", "async", "error-send", "lua54", "macros", "serde" ] }
+mlua                = { version = "0.11.5", features = [ "anyhow", "async", "error-send", "lua54", "macros", "serde" ] }
 objc2               = "0.6.3"
 ordered-float       = { version = "5.1.0", features = [ "serde" ] }
 parking_lot         = "0.12.5"

Cargo.toml

@@ -15,6 +15,29 @@ local function match_mimetype(line)
 	end
 end
 
+local function spawn_file1(paths)
+	local bin = os.getenv("YAZI_FILE_ONE") or "file"
+	local windows = ya.target_family() == "windows"
+
+	local cmd = Command(bin):arg({ "-bL", "--mime-type" }):stdout(Command.PIPED)
+	if windows then
+		cmd:arg({ "-f", "-" }):stdin(Command.PIPED)
+	else
+		cmd:arg("--"):arg(paths)
+	end
+
+	local child, err = cmd:spawn()
+	if not child then
+		return nil, Err("Failed to start `%s`, error: %s", bin, err)
+	elseif windows then
+		child:write_all(table.concat(paths, "\n"))
+		child:flush()
+		ya.drop(child:take_stdin())
+	end
+
+	return child
+end
+
 function M:fetch(job)
 	local urls, paths = {}, {}
 	for i, file in ipairs(job.files) do
@@ -25,10 +48,9 @@ function M:fetch(job)
 		end
 	end
 
-	local cmd = os.getenv("YAZI_FILE_ONE") or "file"
-	local child, err = Command(cmd):arg({ "-bL", "--mime-type", "--" }):arg(paths):stdout(Command.PIPED):spawn()
+	local child, err = spawn_file1(paths)
 	if not child then
-		return true, Err("Failed to start `%s`, error: %s", cmd, err)
+		return true, err
 	end
 
 	local updates, last = {}, ya.time()

yazi-plugin/preset/plugins/mime-local.lua

@@ -1,4 +1,7 @@
+use std::any::TypeId;
+
 use mlua::{AnyUserData, ExternalError, Function, Lua};
+use tokio::process::{ChildStderr, ChildStdin, ChildStdout};
 use yazi_binding::{Id, Permit, PermitRef, deprecate};
 use yazi_proxy::{AppProxy, HIDER};
 
@@ -15,6 +18,19 @@ impl Utils {
 		})
 	}
 
+	pub(super) fn drop(lua: &Lua) -> mlua::Result<Function> {
+		lua.create_function(|_, ud: AnyUserData| {
+			match ud.type_id() {
+				Some(t) if t == TypeId::of::<ChildStdin>() => {}
+				Some(t) if t == TypeId::of::<ChildStdout>() => {}
+				Some(t) if t == TypeId::of::<ChildStderr>() => {}
+				Some(t) => Err(format!("Cannot drop userdata of type {t:?}").into_lua_err())?,
+				None => Err("Cannot drop scoped userdata".into_lua_err())?,
+			};
+			ud.destroy()
+		})
+	}
+
 	pub(super) fn hide(lua: &Lua) -> mlua::Result<Function> {
 		lua.create_async_function(|lua, ()| async move {
 			deprecate!(lua, "`ya.hide()` is deprecated, use `ui.hide()` instead, in your {}\nSee #2939 for more details: https://github.com/sxyazi/yazi/pull/2939");

yazi-plugin/src/utils/app.rs

@@ -10,6 +10,7 @@ pub fn compose(
 		match key {
 			// App
 			b"id" => Utils::id(lua)?,
+			b"drop" => Utils::drop(lua)?,
 			b"hide" => Utils::hide(lua)?,
 
 			// Cache