refactor(server): enhance python server architecture and stability

- Centralize utility functions and logging configuration
- Improve argument parsing and main entry point structure
- Enhance stdio transport reliability with node wrapper improvements
- Remove legacy scripts and cleanup project root
- Translate comments to English and fix minor bugs

---

refactor(server): 파이썬 서버 아키텍처 및 안정성 개선

- 유틸리티 함수 및 로깅 설정 중앙화
- 인자 파싱 및 메인 엔트리 포인트 구조 개선
- Node 래퍼 개선을 통한 stdio 전송 신뢰성 향상
- 레거시 스크립트 제거 및 프로젝트 루트 정리
- 주석 영문화 및 사소한 버그 수정

Author: choej2303

.github/workflows/python-tests.yml

@@ -22,7 +22,7 @@ jobs:
           version: "latest"
 
       - name: Set up Python
-        run: uv python install 3.10
+        run: uv python install 3.11
 
       - name: Install dependencies
         run: |

MCPForUnity/Server~/mcp_wrapper.py

@@ -20,7 +20,10 @@ dev = [
 ]
 
 [project.scripts]
-mcp-for-unity = "main:main"
+mcp-for-unity = "src.main:main"
+
+[tool.setuptools.packages.find]
+where = ["src"]
 
 [build-system]
 requires = ["setuptools>=80.9.0", "wheel>=0.45.1"]

MCPForUnity/Server~/pyproject.toml

@@ -65,13 +65,6 @@
     "httpx", 
     "urllib3", 
     "mcp.server.lowlevel.server", 
-    "uvicorn", 
-    "uvicorn.access", 
-    "uvicorn.error", 
-    "docket", 
-    "docket.worker", 
-    "fastmcp",   # <--- 🚨 범인 검거
-    "starlette"  # <--- 혹시 모를 공범
 ):
     try:
         logging.getLogger(noisy).setLevel(
@@ -412,9 +405,26 @@ def main():
         logger.info(f"Starting FastMCP with HTTP transport on {host}:{port}")
         mcp.run(transport=transport, host=host, port=port)
     else:
+        
         # Use stdio transport for traditional MCP
         logger.info("Starting FastMCP with stdio transport")
-        mcp.run(transport='stdio', show_banner=False)
+        # 🚨 [CRITICAL] In STDIO mode only, suppress related loggers.
+        # Silence Uvicorn and related loggers to prevent stdout pollution.
+        # 🚨 [CRITICAL] Prevent stdout pollution in STDIO mode
+        for name in (
+            "uvicorn", "uvicorn.error", "uvicorn.access",
+            "starlette",
+            "docket", "docket.worker",
+            "fastmcp",
+        ):
+            lg = logging.getLogger(name)
+            lg.setLevel(logging.WARNING) # ERROR if still too chatty
+            lg.propagate = False # prevent duplicate root logs
+            # If the rotating file handler was successfully created, attach it
+            if '_fh' in globals() and _fh not in lg.handlers:
+                lg.addHandler(_fh)
+
+        mcp.run(transport='stdio')
 
 
 # Run the server

MCPForUnity/Server~/src/main.py

@@ -17,20 +17,45 @@ log("Wrapper started");
 
 const serverDir = __dirname;
 
+// Construct updated PATH with common uv locations
+const localAppData = process.env.LOCALAPPDATA || "";
+const userProfile = process.env.USERPROFILE || "";
+const homeDir = process.env.HOME || process.env.USERPROFILE || "";
+
+const extraPaths =
+  process.platform === "win32"
+    ? [
+        path.join(localAppData, "Programs", "uv"), // uv standalone
+        path.join(localAppData, "uv"), // Alternative
+        path.join(userProfile, ".cargo", "bin"), // Cargo install
+        path.join(localAppData, "bin"),
+      ]
+    : [
+        path.join(homeDir, ".cargo", "bin"),
+        path.join(homeDir, ".local", "bin"),
+        "/usr/local/bin",
+        "/opt/homebrew/bin", // macOS Homebrew
+      ];
+
+const newPath =
+  extraPaths.join(path.delimiter) + path.delimiter + (process.env.PATH || "");
+
 // Use uv run with --quiet to minimize noise
-// Add --quiet to uv run commands to suppress "resolved ..." messages
 const pythonProcess = spawn(
   "uv",
   ["run", "--quiet", "src/main.py", "--transport", "stdio"],
   {
     cwd: serverDir,
     stdio: ["pipe", "pipe", "pipe"],
-    shell: true, // Needed for windows command resolution sometimes
+    shell: true,
     env: {
       ...process.env,
+      PATH: newPath, // Inject updated PATH
       PYTHONUNBUFFERED: "1",
       PYTHONIOENCODING: "utf-8",
-      HOME: process.env.USERPROFILE, // Ensure uv finds home on Windows
+      ...(process.platform === "win32" && userProfile
+        ? { HOME: userProfile }
+        : {}),
     },
   }
 );
@@ -92,9 +117,11 @@ function cleanup() {
     try {
       pythonProcess.kill();
       if (process.platform === "win32") {
-        require("child_process").execSync(
-          `taskkill /pid ${pythonProcess.pid} /T /F`
-        );
+        if (pythonProcess.pid) {
+          require("child_process").execSync(
+            `taskkill /pid ${pythonProcess.pid} /T /F`
+          );
+        }
       }
     } catch (e) {
       /* ignore */