From 0b58b0486c52dc3cfc377c0504e4d834b2d6bd00 Mon Sep 17 00:00:00 2001
From: dave <futurechimp@users.noreply.github.com>
Date: Sun, 12 Apr 2026 00:45:47 +0000
Subject: [PATCH] fix: use Stdio::inherit for run_tests to prevent pipe
 deadlock

spawn() with piped stdout/stderr deadlocks when the test binary
produces more output than the OS pipe buffer (64KB). Switch to
Stdio::inherit so test output flows to server logs and we can
see what's happening.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 server/src/http/mcp/shell_tools.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/src/http/mcp/shell_tools.rs b/server/src/http/mcp/shell_tools.rs
index 16164bad..7d5e82e3 100644
--- a/server/src/http/mcp/shell_tools.rs
+++ b/server/src/http/mcp/shell_tools.rs
@@ -411,8 +411,8 @@ pub(super) async fn tool_run_tests(args: &Value, ctx: &AppContext) -> Result<Str
     let child = std::process::Command::new("bash")
         .arg(&script_path)
         .current_dir(&working_dir)
-        .stdout(std::process::Stdio::piped())
-        .stderr(std::process::Stdio::piped())
+        .stdout(std::process::Stdio::inherit())
+        .stderr(std::process::Stdio::inherit())
         .spawn()
         .map_err(|e| format!("Failed to spawn test script: {e}"))?;