refactor: Improve npm command logging and settings initialization

Author: CoderGamester

Editor/UnityBridge/McpUnityEditorWindow.cs

@@ -143,16 +143,6 @@ private void DrawServerTab()
 
             EditorGUILayout.Space();
 
-            // NPM Executable Path
-            string newNpmPath = EditorGUILayout.TextField(new GUIContent("NPM Executable Path", "Optional: Full path to the npm executable (e.g., /Users/user/.asdf/shims/npm or C:\\path\\to\\npm.cmd). If not set, 'npm' from the system PATH will be used."), settings.NpmExecutablePath);
-            if (newNpmPath != settings.NpmExecutablePath)
-            {
-                settings.NpmExecutablePath = newNpmPath;
-                settings.SaveSettings();
-            }
-            
-            EditorGUILayout.Space();
-            
             // Server control buttons
             EditorGUILayout.BeginHorizontal();
 
@@ -210,6 +200,16 @@ private void DrawServerTab()
             }
 
             EditorGUILayout.EndVertical();
+
+            // NPM Executable Path
+            string newNpmPath = EditorGUILayout.TextField(new GUIContent("NPM Executable Path", "Optional: Full path to the npm executable (e.g., /Users/user/.asdf/shims/npm or C:\\path\\to\\npm.cmd). If not set, 'npm' from the system PATH will be used."), settings.NpmExecutablePath);
+            if (newNpmPath != settings.NpmExecutablePath)
+            {
+                settings.NpmExecutablePath = newNpmPath;
+                settings.SaveSettings();
+            }
+            
+            EditorGUILayout.Space();
 
             // MCP Config generation section
             EditorGUILayout.Space();

Editor/UnityBridge/McpUnitySettings.cs

@@ -74,6 +74,11 @@ public void LoadSettings()
                     string json = File.ReadAllText(SettingsPath);
                     JsonUtility.FromJsonOverwrite(json, this);
                 }
+                else
+                {
+                    // Create default settings file on the first time initialization
+                    SaveSettings();
+                }
             }
             catch (Exception ex)
             {

Editor/Utils/McpUtils.cs

@@ -354,24 +354,18 @@ public static void RunNpmCommand(string arguments, string workingDirectory)
 
                     if (process.ExitCode == 0)
                     {
-                        Debug.Log($"[MCP Unity] npm {arguments} completed successfully in {workingDirectory}.");
+                        Debug.Log($"[MCP Unity] npm {arguments} completed successfully in {workingDirectory}.\n{output}");
                     }
                     else
                     {
-                        Debug.LogError($"[MCP Unity] npm {arguments} failed in {workingDirectory}. Exit Code: {process.ExitCode}");
+                        Debug.LogError($"[MCP Unity] npm {arguments} failed in {workingDirectory}. Exit Code: {process.ExitCode}. Error: {error}");
                     }
-                    
-                    if (!string.IsNullOrEmpty(output)) Debug.Log($"[MCP Unity] Output:\n{output}");
-                    if (!string.IsNullOrEmpty(error)) Debug.LogError($"[MCP Unity] Error:\n{error}");
                 }
             }
-            catch (System.ComponentModel.Win32Exception ex) // Catch specific exception for "file not found"
-            {
-                Debug.LogError($"[MCP Unity] Failed to run npm command '{shellCommand} {shellArguments}'. Ensure Node.js and npm are installed and in your system's PATH. Error: {ex.Message}");
-            }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Exception while running npm command '{shellCommand} {shellArguments}' in {workingDirectory}: {ex.Message}\nDetails: {ex.StackTrace}");
+                // Use commandToLog here
+                Debug.LogError($"[MCP Unity] Exception while running npm {arguments} in {workingDirectory}. Error: {ex.Message}");
             }
         }
     }