Use shell redirection to temp files instead of pipes

Pipes have buffer limits (~64KB) that can cause deadlocks when
reading stdout/stderr sequentially. Using shell redirection to
temp files avoids this entirely - no pipes, no buffer issues.

The shell handles argument passing via "$@" which avoids escaping.

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: diogot

Sources/XCResultParser/XCResultTool.swift

@@ -10,34 +10,43 @@ actor XCResultTool {
 
     /// Execute xcresulttool and return the JSON data
     private func execute(arguments: [String]) async throws -> Data {
-        let process = Process()
-        process.executableURL = URL(fileURLWithPath: "/usr/bin/xcrun")
-        process.arguments = ["xcresulttool"] + arguments + ["--path", path, "--compact"]
+        let fileManager = FileManager.default
+        let tempDir = fileManager.temporaryDirectory
+        let uuid = UUID().uuidString
+        let stdoutFile = tempDir.appendingPathComponent("xcresulttool-\(uuid).stdout")
+        let stderrFile = tempDir.appendingPathComponent("xcresulttool-\(uuid).stderr")
+
+        defer {
+            try? fileManager.removeItem(at: stdoutFile)
+            try? fileManager.removeItem(at: stderrFile)
+        }
 
-        let stdout = Pipe()
-        let stderr = Pipe()
-        process.standardOutput = stdout
-        process.standardError = stderr
+        let process = Process()
+        process.executableURL = URL(fileURLWithPath: "/bin/sh")
+        process.arguments = [
+            "-c",
+            "xcrun \"$@\" > \"\(stdoutFile.path)\" 2> \"\(stderrFile.path)\"",
+            "--",
+            "xcresulttool"
+        ] + arguments + ["--path", path, "--compact"]
 
         do {
             try process.run()
         } catch {
             throw XCResultParserError.xcresulttoolNotFound
         }
 
-        // Read output BEFORE waiting to avoid pipe buffer deadlock
-        // (if output exceeds ~64KB, the process blocks waiting to write)
-        let outputData = stdout.fileHandleForReading.readDataToEndOfFile()
-        let errorData = stderr.fileHandleForReading.readDataToEndOfFile()
+        process.waitUntilExit()
 
-        // Wait for process termination on a detached task to avoid:
-        // 1. Blocking the cooperative thread pool (waitUntilExit blocks)
-        // 2. Race condition with terminationHandler (process may have exited before handler was registered)
-        await Task.detached {
-            process.waitUntilExit()
-        }.value
+        let outputData: Data
+        do {
+            outputData = try Data(contentsOf: stdoutFile)
+        } catch {
+            throw XCResultParserError.xcresulttoolFailed("Failed to read stdout: \(error.localizedDescription)")
+        }
 
         if process.terminationStatus != 0 {
+            let errorData = (try? Data(contentsOf: stderrFile)) ?? Data()
             let errorMessage = String(data: errorData, encoding: .utf8) ?? "Unknown error"
             throw XCResultParserError.xcresulttoolFailed(errorMessage.trimmingCharacters(in: .whitespacesAndNewlines))
         }