All critical review feedback from CodeRabbit has been addressed! 🎉

Completed Items:

Supply Chain Fix: Restored the default Git URL in AssetPathUtility.cs to the official repository (CoplayDev).

i18n (Internationalization): Localized core Korean comments in main.py into English.

Stability: Implemented a safe fallback mechanism in ConfigJsonBuilder.cs for when Node path override fails.

Robustness: Added PID validation checks in wrapper.js to prevent errors.

Refactoring: Removed unnecessary dummy code from McpSettingsSection.cs and MCPForUnityEditorWindow.cs.

Author: choej2303

MCPForUnity/Editor/Helpers/AssetPathUtility.cs

@@ -268,10 +268,9 @@ public static string GetMcpServerGitUrl()
                 return gitUrlOverride;
             }
 
-            // Return hardcoded Git URL as requested by the user who prefers to push changes and use the clean URL
-            // instead of local file paths.
-            // Note: This means local changes MUST be pushed to GitHub to take effect in the server!
-            return "git+https://github.com/choej2303/unity-mcp-gg.git@main#subdirectory=MCPForUnity/Server~";
+            // Return official repository URL as default
+            // Users can override via EditorPrefs.GitUrlOverride for development/testing
+            return "git+https://github.com/CoplayDev/unity-mcp.git@main#subdirectory=MCPForUnity/Server~";
 
             /* 
             // Local path logic disabled for clean config aesthetics

MCPForUnity/Editor/Helpers/ConfigJsonBuilder.cs

@@ -110,9 +110,16 @@ private static void PopulateUnityNode(JObject unity, string uvPath, McpClient cl
                     // We assume 'node' is in PATH unless overridden.
                     string nodeCommand = "node";
                     string nodeOverride = EditorPrefs.GetString(EditorPrefKeys.NodePathOverride, "");
-                    if (!string.IsNullOrEmpty(nodeOverride) && File.Exists(nodeOverride))
+                    if (!string.IsNullOrEmpty(nodeOverride))
                     {
-                        nodeCommand = nodeOverride;
+                        if (File.Exists(nodeOverride))
+                        {
+                            nodeCommand = nodeOverride;
+                        }
+                        else
+                        {
+                            McpLog.Warn($"Node override path not found: {nodeOverride}, falling back to 'node'");
+                        }
                     }
 
                     unity["command"] = nodeCommand;

MCPForUnity/Editor/Windows/Components/Settings/McpSettingsSection.cs

@@ -88,11 +88,7 @@ private void RegisterCallbacks()
             });
         }
 
-        public void UpdatePathOverrides()
-        {
-            // Now handled in Setup Section or Client Config
-            // Kept empty method for compatibility with existing calls
-        }
+
 
         private void UpdateVersionLabel()
         {

MCPForUnity/Editor/Windows/MCPForUnityEditorWindow.cs

@@ -383,8 +383,6 @@ private void RefreshAllData()
                 _ = connectionSection?.VerifyBridgeConnectionAsync();
             }
 
-            settingsSection?.UpdatePathOverrides();
-            settingsSection?.UpdatePathOverrides();
             clientConfigSection?.RefreshSelectedClient();
             // Removed toolsSection?.Refresh() - tools are now loaded lazily when tab is opened
         }

MCPForUnity/Server~/src/main.py

@@ -408,9 +408,9 @@ def main():
 
         # Use stdio transport for traditional MCP
         logger.info("Starting FastMCP with stdio transport")
-        # 🚨 [핵심] STDIO 모드일 때만 관련 로거를 CRITICAL로 낮춥니다.
-        # Uvicorn 및 관련 로거들의 입을 막아 stdout 오염을 방지합니다.
-        # 🚨 [핵심] STDIO 모드일 때 stdout 오염 방지
+        # 🚨 [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",

MCPForUnity/Server~/wrapper.js

@@ -117,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 */