feat: improve process cleanup and test configuration

- Add proper process cleanup in shell_executor
- Update pytest-asyncio configuration

Author: Yoshihiro Takahara

src/mcp_shell_server/shell_executor.py

@@ -386,6 +386,7 @@ async def execute(
         timeout: Optional[int] = None,
     ) -> Dict[str, Any]:
         start_time = time.time()
+        process = None  # Initialize process variable
 
         try:
             # Validate directory if specified
@@ -590,6 +591,11 @@ async def communicate_with_timeout():
                 "stderr": str(e),
                 "execution_time": time.time() - start_time,
             }
+        finally:
+            # Ensure process is terminated
+            if process and process.returncode is None:
+                process.kill()
+                await process.wait()
 
     async def _execute_pipeline(
         self,
@@ -692,5 +698,10 @@ async def _execute_pipeline(
             }
 
         finally:
+            # Ensure all processes are terminated
+            for process in processes:
+                if process.returncode is None:
+                    process.kill()
+                    await process.wait()
             if isinstance(last_stdout, IO):
                 last_stdout.close()

tests/conftest.py

@@ -1,10 +1,8 @@
 import os
 
-
 # Configure pytest-asyncio
 def pytest_configure(config):
     """Configure pytest-asyncio defaults"""
-    config.option.asyncio_mode = "strict"
     # Enable command execution for tests
     os.environ["ALLOW_COMMANDS"] = "1"
     # Add allowed commands for tests