Remove redundant HasExited check in DisposeProcess else branch

After line 222 recomputes processRunning as `processRunning && !HasExited(process)`,
the else branch already implies HasExited was true (when processRunning was
originally true). The redundant check is unnecessary.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: Copilot

src/ModelContextProtocol.Core/Client/StdioClientTransport.cs

@@ -227,7 +227,7 @@ internal static void DisposeProcess(
                     // and Node.js does not kill its children when it exits properly.
                     process.KillTree(shutdownTimeout);
                 }
-                else if (HasExited(process))
+                else
                 {
                     // The process has already exited. Call WaitForExit() (no arguments)
                     // to ensure all redirected stderr/stdout events have been dispatched