Use anyio.move_on_after for process group termination timeout

Replace manual deadline checking with anyio.move_on_after context manager
for cleaner and more idiomatic timeout handling in Unix process group
termination.

Author: felixweinberger

src/mcp/client/stdio/__init__.py

@@ -265,14 +265,14 @@ async def _terminate_process_tree(process: Process | FallbackProcess, timeout: f
             pgid = os.getpgid(pid)
             os.killpg(pgid, signal.SIGTERM)
 
-            deadline = anyio.current_time() + timeout
-            while anyio.current_time() < deadline:
-                try:
-                    # Check if process group still exists (signal 0 = check only)
-                    os.killpg(pgid, 0)
-                    await anyio.sleep(0.1)
-                except ProcessLookupError:
-                    return
+            with anyio.move_on_after(timeout):
+                while True:
+                    try:
+                        # Check if process group still exists (signal 0 = check only)
+                        os.killpg(pgid, 0)
+                        await anyio.sleep(0.1)
+                    except ProcessLookupError:
+                        return
 
             try:
                 os.killpg(pgid, signal.SIGKILL)

src/mcp/client/stdio/win32.py

@@ -227,8 +227,7 @@ async def _create_windows_fallback_process(
             cwd=cwd,
             bufsize=0,
         )
-    process = FallbackProcess(popen_obj)
-    return process
+    return FallbackProcess(popen_obj)
 
 
 def _create_job_object() -> int | None: