desktop: Only attach to parent console on Windows if stdout is not a pipe or a regular file

Author: SuchAFuriousDeath

desktop/Cargo.toml

@@ -60,7 +60,7 @@ async-task = "4.7.1"
 ashpd = "0.11.0"
 
 [target.'cfg(windows)'.dependencies]
-windows-sys = { version = "0.61.2", features = ["Win32_System_Console"] }
+windows-sys = { version = "0.61.2", features = ["Win32_System_Console", "Win32_Storage_FileSystem"] }
 
 [build-dependencies]
 embed-resource = "3"

desktop/src/main.rs

@@ -61,10 +61,32 @@ fn init() {
     // When linked with the windows subsystem windows won't automatically attach
     // to the console of the parent process, so we do it explicitly. This fails
     // silently if the parent has no console.
+    //
+    // However, if stdout/stderr are already redirected (e.g., `ruffle.exe > file.txt`),
+    // we should NOT attach to the console as that would bypass the redirection.
+    // See: https://github.com/ruffle-rs/ruffle/issues/9145
     #[cfg(windows)]
     unsafe {
-        use windows_sys::Win32::System::Console::{ATTACH_PARENT_PROCESS, AttachConsole};
-        AttachConsole(ATTACH_PARENT_PROCESS);
+        use windows_sys::Win32::Storage::FileSystem::{
+            FILE_TYPE_DISK, FILE_TYPE_PIPE, GetFileType,
+        };
+        use windows_sys::Win32::System::Console::{
+            ATTACH_PARENT_PROCESS, AttachConsole, GetStdHandle, STD_OUTPUT_HANDLE,
+        };
+
+        // Check if stdout is already redirected to a file or pipe
+        let stdout_handle = GetStdHandle(STD_OUTPUT_HANDLE);
+        let file_type = GetFileType(stdout_handle);
+
+        match file_type {
+            // If output is redirected to a file or pipe, don't attach to console
+            // as that would bypass the redirection
+            FILE_TYPE_DISK | FILE_TYPE_PIPE => {}
+            // Otherwise, attach to parent console for interactive use
+            _ => {
+                AttachConsole(ATTACH_PARENT_PROCESS);
+            }
+        }
     }
 
     let prev_hook = std::panic::take_hook();