Fix stderr event loss due to missing WaitForExit in DisposeProcess (#1280)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: stephentoub <[email protected]>

Author: Copilot

src/ModelContextProtocol.Core/Client/StdioClientTransport.cs

@@ -227,6 +227,18 @@ internal static void DisposeProcess(
                     // and Node.js does not kill its children when it exits properly.
                     process.KillTree(shutdownTimeout);
                 }
+
+                // Ensure all redirected stderr/stdout events have been dispatched
+                // before disposing. Only the no-arg WaitForExit() guarantees this;
+                // WaitForExit(int) (as used by KillTree) does not.
+                // This should not hang: either the process already exited on its own
+                // (no child processes holding handles), or KillTree killed the entire
+                // process tree. If it does take too long, the test infrastructure's
+                // own timeout will catch it.
+                if (!processRunning && HasExited(process))
+                {
+                    process.WaitForExit();
+                }
             }
             finally
             {