Added toggle to enable and disable the logs into the Unity console. This cleans up the console for the user's interest

CoderGamester · CoderGamester · commit 2d40387a5276 · 2025-04-14T18:30:54.000+03:00
Added new Logger to control console logs inside MCP package
diff --git a/Editor/Services/ConsoleLogsService.cs b/Editor/Services/ConsoleLogsService.cs
@@ -64,8 +64,6 @@ public void StopListening()
             // Unity 2022.3 implementation using reflection
             EditorApplication.update -= CheckConsoleClearViaReflection;
 #endif
-            
-            Debug.Log("[MCP Unity] Console logs service shutdown");
         }
         
         /// <summary>
diff --git a/Editor/Tools/AddAssetToSceneTool.cs b/Editor/Tools/AddAssetToSceneTool.cs
@@ -3,6 +3,7 @@
 using UnityEditor;
 using Newtonsoft.Json.Linq;
 using McpUnity.Unity;
+using McpUnity.Utils;
 
 namespace McpUnity.Tools
 {
@@ -113,7 +114,7 @@ public override JObject Execute(JObject parameters)
                     }
                     else
                     {
-                        Debug.LogWarning($"[MCP Unity] Parent object not found, asset will be created at the root of the scene");
+                        McpLogger.LogWarning($"Parent object not found, asset will be created at the root of the scene");
                     }
                 }
                 
@@ -130,7 +131,7 @@ public override JObject Execute(JObject parameters)
             }
             
             // Log the action
-            Debug.Log($"[MCP Unity] Added asset '{asset.name}' to scene from path '{assetPath}'");
+            McpLogger.LogInfo($"Added asset '{asset.name}' to scene from path '{assetPath}'");
             
             // Create the response
             return new JObject
diff --git a/Editor/Tools/AddPackageTool.cs b/Editor/Tools/AddPackageTool.cs
@@ -9,6 +9,7 @@
 using System.Threading.Tasks;
 using McpUnity.Tools;
 using McpUnity.Unity;
+using McpUnity.Utils;
 
 namespace McpUnity.Tools
 {
@@ -124,7 +125,7 @@ private AddRequest AddFromRegistry(JObject parameters, TaskCompletionSource<JObj
                 packageIdentifier = $"{packageName}@{version}";
             }
             
-            Debug.Log($"[MCP Unity] Adding package from registry: {packageIdentifier}");
+            McpLogger.LogInfo($"Adding package from registry: {packageIdentifier}");
             
             try
             {
@@ -188,7 +189,7 @@ private AddRequest AddFromGitHub(JObject parameters, TaskCompletionSource<JObjec
                 }
             }
             
-            Debug.Log($"[MCP Unity] Adding package from GitHub: {packageUrl}");
+            McpLogger.LogInfo($"Adding package from GitHub: {packageUrl}");
             
             try
             {
@@ -225,7 +226,7 @@ private AddRequest AddFromDisk(JObject parameters, TaskCompletionSource<JObject>
             // Format as file URL
             string packageUrl = $"file:{path}";
             
-            Debug.Log($"[MCP Unity] Adding package from disk: {packageUrl}");
+            McpLogger.LogInfo($"Adding package from disk: {packageUrl}");
             
             try
             {
@@ -289,7 +290,7 @@ private void ProcessCompletedOperation(PackageOperation operation)
         {
             if (operation.CompletionSource == null)
             {
-                Debug.LogError("[MCP Unity] TaskCompletionSource is null when processing completed operation");
+                McpLogger.LogError("TaskCompletionSource is null when processing completed operation");
                 return;
             }
             
@@ -321,6 +322,8 @@ private void ProcessCompletedOperation(PackageOperation operation)
                         ["message"] = $"Package operation completed successfully, but no package information was returned."
                     });
                 }
+                
+                McpLogger.LogInfo($"Added package {result.displayName} ({result.name}) version {result.version}");
             }
             else if (operation.Request.Status == StatusCode.Failure)
             {
diff --git a/Editor/Tools/MenuItemTool.cs b/Editor/Tools/MenuItemTool.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Threading.Tasks;
 using McpUnity.Unity;
+using McpUnity.Utils;
 using UnityEngine;
 using UnityEditor;
 using Newtonsoft.Json.Linq;
@@ -35,7 +36,7 @@ public override JObject Execute(JObject parameters)
             }
                 
             // Log the execution
-            Debug.Log($"[MCP Unity] Executing menu item: {menuPath}");
+            McpLogger.LogInfo($"[MCP Unity] Executing menu item: {menuPath}");
                 
             // Execute the menu item
             bool success = EditorApplication.ExecuteMenuItem(menuPath);
diff --git a/Editor/Tools/RunTestsTool.cs b/Editor/Tools/RunTestsTool.cs
@@ -7,6 +7,7 @@
 using System.Collections.Generic;
 using UnityEditor.TestTools.TestRunner.Api;
 using McpUnity.Services;
+using McpUnity.Utils;
 
 namespace McpUnity.Tools
 {
@@ -81,7 +82,7 @@ public override void ExecuteAsync(JObject parameters, TaskCompletionSource<JObje
             }
             
             // Log the execution
-            Debug.Log($"[MCP Unity] Running tests: Mode={testMode}, Filter={testFilter}");
+            McpLogger.LogInfo($"[MCP Unity] Running tests: Mode={testMode}, Filter={testFilter}");
             
             // Reset state
             _isRunning = true;
@@ -101,7 +102,7 @@ public override void ExecuteAsync(JObject parameters, TaskCompletionSource<JObje
         // Called when a test run starts
         public void RunStarted(ITestAdaptor testsToRun)
         {
-            Debug.Log($"[MCP Unity] Test run started: {testsToRun.Name}");
+            McpLogger.LogInfo($"[MCP Unity] Test run started: {testsToRun.Name}");
         }
         
         // Called when a test runs
@@ -122,13 +123,13 @@ public void TestFinished(ITestResultAdaptor result)
                 Duration = result.Duration
             });
             
-            Debug.Log($"[MCP Unity] Test finished: {result.Test.Name} - {result.ResultState}");
+            McpLogger.LogInfo($"[MCP Unity] Test finished: {result.Test.Name} - {result.ResultState}");
         }
         
         // Called when a test run completes
         public void RunFinished(ITestResultAdaptor result)
         {
-            Debug.Log($"[MCP Unity] Test run completed: {result.Test.Name} - {result.ResultState}");
+            McpLogger.LogInfo($"[MCP Unity] {_testResults.Count} tests completed in {result.Duration}: {result.Test.Name} - {result.ResultState}");
             
             _isRunning = false;
             
@@ -170,7 +171,7 @@ public void RunFinished(ITestResultAdaptor result)
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Failed to set test results: {ex.Message}");
+                McpLogger.LogError($"[MCP Unity] Failed to set test results: {ex.Message}");
                 _testRunCompletionSource?.TrySetException(ex);
             }
             finally
diff --git a/Editor/Tools/SelectGameObjectTool.cs b/Editor/Tools/SelectGameObjectTool.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Threading.Tasks;
 using McpUnity.Unity;
+using McpUnity.Utils;
 using UnityEngine;
 using UnityEditor;
 using Newtonsoft.Json.Linq;
@@ -53,7 +54,7 @@ public override JObject Execute(JObject parameters)
             EditorGUIUtility.PingObject(Selection.activeGameObject);
             
             // Log the selection
-            Debug.Log($"[MCP Unity] Selected GameObject: " +
+            McpLogger.LogInfo($"[MCP Unity] Selected GameObject: " +
                 (instanceId.HasValue ? $"Instance ID {instanceId.Value}" : $"Path '{objectPath}'"));
             
             // Create the response
diff --git a/Editor/Tools/UpdateComponentTool.cs b/Editor/Tools/UpdateComponentTool.cs
@@ -1,6 +1,7 @@
 using System;
 using System.Reflection;
 using McpUnity.Unity;
+using McpUnity.Utils;
 using UnityEngine;
 using UnityEditor;
 using Newtonsoft.Json.Linq;
@@ -77,7 +78,7 @@ public override JObject Execute(JObject parameters)
                 );
             }
             
-            Debug.Log($"[MCP Unity] Updating component '{componentName}' on GameObject '{gameObject.name}' (found by {identifier})");
+            McpLogger.LogInfo($"[MCP Unity] Updating component '{componentName}' on GameObject '{gameObject.name}' (found by {identifier})");
             
             // Try to find the component by name
             Component component = gameObject.GetComponent(componentName);
@@ -97,7 +98,7 @@ public override JObject Execute(JObject parameters)
                 
                 component = Undo.AddComponent(gameObject, componentType);
                 wasAdded = true;
-                Debug.Log($"[MCP Unity] Added component '{componentName}' to GameObject '{gameObject.name}'");
+                McpLogger.LogInfo($"[MCP Unity] Added component '{componentName}' to GameObject '{gameObject.name}'");
             }
             
             // Update component fields
@@ -272,7 +273,7 @@ private bool UpdateComponentData(Component component, JObject componentData)
                 }
                 else
                 {
-                    Debug.LogWarning($"[MCP Unity] Field '{fieldName}' not found on component '{componentType.Name}'");
+                    McpLogger.LogWarning($"Field '{fieldName}' not found on component '{componentType.Name}'");
                 }
             }
             
@@ -402,7 +403,7 @@ private object ConvertJTokenToValue(JToken token, Type targetType)
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Error converting value to type {targetType.Name}: {ex.Message}");
+                McpLogger.LogError($"[MCP Unity] Error converting value to type {targetType.Name}: {ex.Message}");
                 return null;
             }
         }
diff --git a/Editor/UnityBridge/McpUnityEditorWindow.cs b/Editor/UnityBridge/McpUnityEditorWindow.cs
@@ -117,6 +117,16 @@ private void DrawServerTab()
             
             EditorGUILayout.Space();
             
+            // Enable info logs toggle
+            bool enableInfoLogs = EditorGUILayout.Toggle(new GUIContent("Enable Info Logs", "Show informational logs in the Unity console"), settings.EnableInfoLogs);
+            if (enableInfoLogs != settings.EnableInfoLogs)
+            {
+                settings.EnableInfoLogs = enableInfoLogs;
+                settings.SaveSettings();
+            }
+            
+            EditorGUILayout.Space();
+            
             // Server control buttons
             EditorGUILayout.BeginHorizontal();
             
@@ -209,12 +219,6 @@ private void DrawServerTab()
             
             EditorGUILayout.Space();
             
-            //EditorGUILayout.BeginVertical(_connectedClientBoxStyle);
-            //EditorGUILayout.LabelField("Cursor IDE configuration is done automatically. Restart your Cursor IDE to apply the changes.", EditorStyles.boldLabel);
-            //EditorGUILayout.EndVertical();
-            
-            //EditorGUILayout.Space();
-            
             if (GUILayout.Button("Configure Claude Desktop", GUILayout.Height(30)))
             {
                 bool added = McpConfigUtils.AddToClaudeDesktopConfig(_tabsIndentationJson);
diff --git a/Editor/UnityBridge/McpUnityServer.cs b/Editor/UnityBridge/McpUnityServer.cs
@@ -9,6 +9,7 @@
 using McpUnity.Tools;
 using McpUnity.Resources;
 using McpUnity.Services;
+using McpUnity.Utils;
 using WebSocketSharp;
 using WebSocketSharp.Server;
 
@@ -76,7 +77,7 @@ private McpUnityServer()
             RegisterResources();
             RegisterTools();
             
-            Debug.Log($"[MCP Unity] Created WebSocket server on port {McpUnitySettings.Instance.Port}");
+            McpLogger.LogInfo($"Created WebSocket server on port {McpUnitySettings.Instance.Port}");
 
             if (McpUnitySettings.Instance.AutoStartServer || Application.internetReachability != NetworkReachability.NotReachable)
             {
@@ -101,11 +102,11 @@ public void StartServer()
                 // Start the server
                 _webSocketServer.Start();
                 
-                Debug.Log("[MCP Unity] WebSocket server started");
+                McpLogger.LogInfo("WebSocket server started");
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Failed to start WebSocket server: {ex.Message}");
+                McpLogger.LogError($"Failed to start WebSocket server: {ex.Message}");
             }
         }
         
@@ -120,11 +121,11 @@ public void StopServer()
             {
                 _webSocketServer?.Stop();
                 
-                Debug.Log("[MCP Unity] WebSocket server stopped");
+                McpLogger.LogInfo("WebSocket server stopped");
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Error stopping WebSocket server: {ex.Message}");
+                McpLogger.LogError($"Error stopping WebSocket server: {ex.Message}");
             }
         }
         
diff --git a/Editor/UnityBridge/McpUnitySettings.cs b/Editor/UnityBridge/McpUnitySettings.cs
@@ -24,6 +24,9 @@ public class McpUnitySettings
         
         [Tooltip("Whether to automatically start the MCP server when Unity opens")]
         public bool AutoStartServer = true;
+        
+        [Tooltip("Whether to show info logs in the Unity console")]
+        public bool EnableInfoLogs = true;
 
         /// <summary>
         /// Singleton instance of settings
@@ -72,6 +75,7 @@ public void LoadSettings()
             }
             catch (Exception ex)
             {
+                // Can't use LoggerService here as it depends on settings
                 Debug.LogError($"[MCP Unity] Failed to load settings: {ex.Message}");
             }
         }
@@ -93,6 +97,7 @@ public void SaveSettings()
             }
             catch (Exception ex)
             {
+                // Can't use LoggerService here as it might create circular dependency
                 Debug.LogError($"[MCP Unity] Failed to save settings: {ex.Message}");
             }
         }
diff --git a/Editor/UnityBridge/McpUnitySocketHandler.cs b/Editor/UnityBridge/McpUnitySocketHandler.cs
@@ -11,6 +11,7 @@
 using Unity.EditorCoroutines.Editor;
 using System.Collections;
 using System.Collections.Specialized;
+using McpUnity.Utils;
 
 namespace McpUnity.Unity
 {
@@ -54,7 +55,7 @@ protected override async void OnMessage(MessageEventArgs e)
         {
             try
             {
-                Debug.Log($"[MCP Unity] WebSocket message received: {e.Data}");
+                McpLogger.LogInfo($"WebSocket message received: {e.Data}");
                 
                 var requestJson = JObject.Parse(e.Data);
                 var method = requestJson["method"]?.ToString();
@@ -87,14 +88,14 @@ protected override async void OnMessage(MessageEventArgs e)
                 JObject jsonRpcResponse = CreateResponse(requestId, responseJson);
                 string responseStr = jsonRpcResponse.ToString(Formatting.None);
                 
-                Debug.Log($"[MCP Unity] WebSocket message response: {responseStr}");
+                McpLogger.LogInfo($"WebSocket message response: {responseStr}");
                 
                 // Send the response back to the client
                 Send(responseStr);
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Error processing message: {ex.Message}");
+                McpLogger.LogError($"Error processing message: {ex.Message}");
                 
                 Send(CreateErrorResponse($"Internal server error: {ex.Message}", "internal_error").ToString(Formatting.None));
             }
@@ -116,7 +117,7 @@ protected override void OnOpen()
                 _server.Clients.Add(ID, clientName);
             }
             
-            Debug.Log($"[MCP Unity] WebSocket client '{clientName}' connected");
+            McpLogger.LogInfo($"WebSocket client '{clientName}' connected");
         }
         
         /// <summary>
@@ -129,15 +130,15 @@ protected override void OnClose(CloseEventArgs e)
             // Remove the client from the server
             _server.Clients.Remove(ID);
             
-            Debug.Log($"[MCP Unity] WebSocket client '{clientName}' disconnected: {e.Reason}");
+            McpLogger.LogInfo($"WebSocket client '{clientName}' disconnected: {e.Reason}");
         }
         
         /// <summary>
         /// Handle WebSocket errors
         /// </summary>
         protected override void OnError(ErrorEventArgs e)
         {
-            Debug.LogError($"[MCP Unity] WebSocket error: {e.Message}");
+            McpLogger.LogError($"WebSocket error: {e.Message}");
         }
         
         /// <summary>
@@ -159,7 +160,7 @@ private IEnumerator ExecuteTool(McpToolBase tool, JObject parameters, TaskComple
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Error executing tool {tool.Name}: {ex.Message}");
+                McpLogger.LogError($"Error executing tool {tool.Name}: {ex.Message}");
                 tcs.SetResult(CreateErrorResponse(
                     $"Failed to execute tool {tool.Name}: {ex.Message}",
                     "tool_execution_error"
@@ -181,7 +182,7 @@ private IEnumerator FetchResourceCoroutine(McpResourceBase resource, JObject par
             }
             catch (Exception ex)
             {
-                Debug.LogError($"[MCP Unity] Error fetching resource {resource.Name}: {ex.Message}");
+                McpLogger.LogError($"Error fetching resource {resource.Name}: {ex.Message}");
                 tcs.SetResult(CreateErrorResponse(
                     $"Failed to fetch resource {resource.Name}: {ex.Message}",
                     "resource_fetch_error"
diff --git a/Editor/Utils/Logger.cs b/Editor/Utils/Logger.cs
diff --git a/Editor/Utils/Logger.cs.meta b/Editor/Utils/Logger.cs.meta

Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,6 @@ public void StopListening()`
`64`	`64`	`// Unity 2022.3 implementation using reflection`
`65`	`65`	`EditorApplication.update -= CheckConsoleClearViaReflection;`
`66`	`66`	`#endif`
`67`		`-`
`68`		`- Debug.Log("[MCP Unity] Console logs service shutdown");`
`69`	`67`	`}`
`70`	`68`
`71`	`69`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`using UnityEditor;`
`4`	`4`	`using Newtonsoft.Json.Linq;`
`5`	`5`	`using McpUnity.Unity;`
	`6`	`+using McpUnity.Utils;`
`6`	`7`
`7`	`8`	`namespace McpUnity.Tools`
`8`	`9`	`{`
`@@ -113,7 +114,7 @@ public override JObject Execute(JObject parameters)`
`113`	`114`	`}`
`114`	`115`	`else`
`115`	`116`	`{`
`116`		`- Debug.LogWarning($"[MCP Unity] Parent object not found, asset will be created at the root of the scene");`
	`117`	`+ McpLogger.LogWarning($"Parent object not found, asset will be created at the root of the scene");`
`117`	`118`	`}`
`118`	`119`	`}`
`119`	`120`
`@@ -130,7 +131,7 @@ public override JObject Execute(JObject parameters)`
`130`	`131`	`}`
`131`	`132`
`132`	`133`	`// Log the action`
`133`		`- Debug.Log($"[MCP Unity] Added asset '{asset.name}' to scene from path '{assetPath}'");`
	`134`	`+ McpLogger.LogInfo($"Added asset '{asset.name}' to scene from path '{assetPath}'");`
`134`	`135`
`135`	`136`	`// Create the response`
`136`	`137`	`return new JObject`
Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,7 @@`
`9`	`9`	`using System.Threading.Tasks;`
`10`	`10`	`using McpUnity.Tools;`
`11`	`11`	`using McpUnity.Unity;`
	`12`	`+using McpUnity.Utils;`
`12`	`13`
`13`	`14`	`namespace McpUnity.Tools`
`14`	`15`	`{`
`@@ -124,7 +125,7 @@ private AddRequest AddFromRegistry(JObject parameters, TaskCompletionSource<JObj`
`124`	`125`	`packageIdentifier = $"{packageName}@{version}";`
`125`	`126`	`}`
`126`	`127`
`127`		`- Debug.Log($"[MCP Unity] Adding package from registry: {packageIdentifier}");`
	`128`	`+ McpLogger.LogInfo($"Adding package from registry: {packageIdentifier}");`
`128`	`129`
`129`	`130`	`try`
`130`	`131`	`{`
`@@ -188,7 +189,7 @@ private AddRequest AddFromGitHub(JObject parameters, TaskCompletionSource<JObjec`
`188`	`189`	`}`
`189`	`190`	`}`
`190`	`191`
`191`		`- Debug.Log($"[MCP Unity] Adding package from GitHub: {packageUrl}");`
	`192`	`+ McpLogger.LogInfo($"Adding package from GitHub: {packageUrl}");`
`192`	`193`
`193`	`194`	`try`
`194`	`195`	`{`
`@@ -225,7 +226,7 @@ private AddRequest AddFromDisk(JObject parameters, TaskCompletionSource<JObject>`
`225`	`226`	`// Format as file URL`
`226`	`227`	`string packageUrl = $"file:{path}";`
`227`	`228`
`228`		`- Debug.Log($"[MCP Unity] Adding package from disk: {packageUrl}");`
	`229`	`+ McpLogger.LogInfo($"Adding package from disk: {packageUrl}");`
`229`	`230`
`230`	`231`	`try`
`231`	`232`	`{`
`@@ -289,7 +290,7 @@ private void ProcessCompletedOperation(PackageOperation operation)`
`289`	`290`	`{`
`290`	`291`	`if (operation.CompletionSource == null)`
`291`	`292`	`{`
`292`		`- Debug.LogError("[MCP Unity] TaskCompletionSource is null when processing completed operation");`
	`293`	`+ McpLogger.LogError("TaskCompletionSource is null when processing completed operation");`
`293`	`294`	`return;`
`294`	`295`	`}`
`295`	`296`
`@@ -321,6 +322,8 @@ private void ProcessCompletedOperation(PackageOperation operation)`
`321`	`322`	`["message"] = $"Package operation completed successfully, but no package information was returned."`
`322`	`323`	`});`
`323`	`324`	`}`
	`325`	`+`
	`326`	`+ McpLogger.LogInfo($"Added package {result.displayName} ({result.name}) version {result.version}");`
`324`	`327`	`}`
`325`	`328`	`else if (operation.Request.Status == StatusCode.Failure)`
`326`	`329`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,7 @@`
`7`	`7`	`using System.Collections.Generic;`
`8`	`8`	`using UnityEditor.TestTools.TestRunner.Api;`
`9`	`9`	`using McpUnity.Services;`
	`10`	`+using McpUnity.Utils;`
`10`	`11`
`11`	`12`	`namespace McpUnity.Tools`
`12`	`13`	`{`
`@@ -81,7 +82,7 @@ public override void ExecuteAsync(JObject parameters, TaskCompletionSource<JObje`
`81`	`82`	`}`
`82`	`83`
`83`	`84`	`// Log the execution`
`84`		`- Debug.Log($"[MCP Unity] Running tests: Mode={testMode}, Filter={testFilter}");`
	`85`	`+ McpLogger.LogInfo($"[MCP Unity] Running tests: Mode={testMode}, Filter={testFilter}");`
`85`	`86`
`86`	`87`	`// Reset state`
`87`	`88`	`_isRunning = true;`
`@@ -101,7 +102,7 @@ public override void ExecuteAsync(JObject parameters, TaskCompletionSource<JObje`
`101`	`102`	`// Called when a test run starts`
`102`	`103`	`public void RunStarted(ITestAdaptor testsToRun)`
`103`	`104`	`{`
`104`		`- Debug.Log($"[MCP Unity] Test run started: {testsToRun.Name}");`
	`105`	`+ McpLogger.LogInfo($"[MCP Unity] Test run started: {testsToRun.Name}");`
`105`	`106`	`}`
`106`	`107`
`107`	`108`	`// Called when a test runs`
`@@ -122,13 +123,13 @@ public void TestFinished(ITestResultAdaptor result)`
`122`	`123`	`Duration = result.Duration`
`123`	`124`	`});`
`124`	`125`
`125`		`- Debug.Log($"[MCP Unity] Test finished: {result.Test.Name} - {result.ResultState}");`
	`126`	`+ McpLogger.LogInfo($"[MCP Unity] Test finished: {result.Test.Name} - {result.ResultState}");`
`126`	`127`	`}`
`127`	`128`
`128`	`129`	`// Called when a test run completes`
`129`	`130`	`public void RunFinished(ITestResultAdaptor result)`
`130`	`131`	`{`
`131`		`- Debug.Log($"[MCP Unity] Test run completed: {result.Test.Name} - {result.ResultState}");`
	`132`	`+ McpLogger.LogInfo($"[MCP Unity] {_testResults.Count} tests completed in {result.Duration}: {result.Test.Name} - {result.ResultState}");`
`132`	`133`
`133`	`134`	`_isRunning = false;`
`134`	`135`
`@@ -170,7 +171,7 @@ public void RunFinished(ITestResultAdaptor result)`
`170`	`171`	`}`
`171`	`172`	`catch (Exception ex)`
`172`	`173`	`{`
`173`		`- Debug.LogError($"[MCP Unity] Failed to set test results: {ex.Message}");`
	`174`	`+ McpLogger.LogError($"[MCP Unity] Failed to set test results: {ex.Message}");`
`174`	`175`	`_testRunCompletionSource?.TrySetException(ex);`
`175`	`176`	`}`
`176`	`177`	`finally`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`using System;`
`2`	`2`	`using System.Reflection;`
`3`	`3`	`using McpUnity.Unity;`
	`4`	`+using McpUnity.Utils;`
`4`	`5`	`using UnityEngine;`
`5`	`6`	`using UnityEditor;`
`6`	`7`	`using Newtonsoft.Json.Linq;`
`@@ -77,7 +78,7 @@ public override JObject Execute(JObject parameters)`
`77`	`78`	`);`
`78`	`79`	`}`
`79`	`80`
`80`		`- Debug.Log($"[MCP Unity] Updating component '{componentName}' on GameObject '{gameObject.name}' (found by {identifier})");`
	`81`	`+ McpLogger.LogInfo($"[MCP Unity] Updating component '{componentName}' on GameObject '{gameObject.name}' (found by {identifier})");`
`81`	`82`
`82`	`83`	`// Try to find the component by name`
`83`	`84`	`Component component = gameObject.GetComponent(componentName);`
`@@ -97,7 +98,7 @@ public override JObject Execute(JObject parameters)`
`97`	`98`
`98`	`99`	`component = Undo.AddComponent(gameObject, componentType);`
`99`	`100`	`wasAdded = true;`
`100`		`- Debug.Log($"[MCP Unity] Added component '{componentName}' to GameObject '{gameObject.name}'");`
	`101`	`+ McpLogger.LogInfo($"[MCP Unity] Added component '{componentName}' to GameObject '{gameObject.name}'");`
`101`	`102`	`}`
`102`	`103`
`103`	`104`	`// Update component fields`
`@@ -272,7 +273,7 @@ private bool UpdateComponentData(Component component, JObject componentData)`
`272`	`273`	`}`
`273`	`274`	`else`
`274`	`275`	`{`
`275`		`- Debug.LogWarning($"[MCP Unity] Field '{fieldName}' not found on component '{componentType.Name}'");`
	`276`	`+ McpLogger.LogWarning($"Field '{fieldName}' not found on component '{componentType.Name}'");`
`276`	`277`	`}`
`277`	`278`	`}`
`278`	`279`
`@@ -402,7 +403,7 @@ private object ConvertJTokenToValue(JToken token, Type targetType)`
`402`	`403`	`}`
`403`	`404`	`catch (Exception ex)`
`404`	`405`	`{`
`405`		`- Debug.LogError($"[MCP Unity] Error converting value to type {targetType.Name}: {ex.Message}");`
	`406`	`+ McpLogger.LogError($"[MCP Unity] Error converting value to type {targetType.Name}: {ex.Message}");`
`406`	`407`	`return null;`
`407`	`408`	`}`
`408`	`409`	`}`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,9 @@ public class McpUnitySettings`
`24`	`24`
`25`	`25`	`[Tooltip("Whether to automatically start the MCP server when Unity opens")]`
`26`	`26`	`public bool AutoStartServer = true;`
	`27`	`+`
	`28`	`+ [Tooltip("Whether to show info logs in the Unity console")]`
	`29`	`+ public bool EnableInfoLogs = true;`
`27`	`30`
`28`	`31`	`/// <summary>`
`29`	`32`	`/// Singleton instance of settings`
`@@ -72,6 +75,7 @@ public void LoadSettings()`
`72`	`75`	`}`
`73`	`76`	`catch (Exception ex)`
`74`	`77`	`{`
	`78`	`+ // Can't use LoggerService here as it depends on settings`
`75`	`79`	`Debug.LogError($"[MCP Unity] Failed to load settings: {ex.Message}");`
`76`	`80`	`}`
`77`	`81`	`}`
`@@ -93,6 +97,7 @@ public void SaveSettings()`
`93`	`97`	`}`
`94`	`98`	`catch (Exception ex)`
`95`	`99`	`{`
	`100`	`+ // Can't use LoggerService here as it might create circular dependency`
`96`	`101`	`Debug.LogError($"[MCP Unity] Failed to save settings: {ex.Message}");`
`97`	`102`	`}`
`98`	`103`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@`
`11`	`11`	`using Unity.EditorCoroutines.Editor;`
`12`	`12`	`using System.Collections;`
`13`	`13`	`using System.Collections.Specialized;`
	`14`	`+using McpUnity.Utils;`
`14`	`15`
`15`	`16`	`namespace McpUnity.Unity`
`16`	`17`	`{`
`@@ -54,7 +55,7 @@ protected override async void OnMessage(MessageEventArgs e)`
`54`	`55`	`{`
`55`	`56`	`try`
`56`	`57`	`{`
`57`		`- Debug.Log($"[MCP Unity] WebSocket message received: {e.Data}");`
	`58`	`+ McpLogger.LogInfo($"WebSocket message received: {e.Data}");`
`58`	`59`
`59`	`60`	`var requestJson = JObject.Parse(e.Data);`
`60`	`61`	`var method = requestJson["method"]?.ToString();`
`@@ -87,14 +88,14 @@ protected override async void OnMessage(MessageEventArgs e)`
`87`	`88`	`JObject jsonRpcResponse = CreateResponse(requestId, responseJson);`
`88`	`89`	`string responseStr = jsonRpcResponse.ToString(Formatting.None);`
`89`	`90`
`90`		`- Debug.Log($"[MCP Unity] WebSocket message response: {responseStr}");`
	`91`	`+ McpLogger.LogInfo($"WebSocket message response: {responseStr}");`
`91`	`92`
`92`	`93`	`// Send the response back to the client`
`93`	`94`	`Send(responseStr);`
`94`	`95`	`}`
`95`	`96`	`catch (Exception ex)`
`96`	`97`	`{`
`97`		`- Debug.LogError($"[MCP Unity] Error processing message: {ex.Message}");`
	`98`	`+ McpLogger.LogError($"Error processing message: {ex.Message}");`
`98`	`99`
`99`	`100`	`Send(CreateErrorResponse($"Internal server error: {ex.Message}", "internal_error").ToString(Formatting.None));`
`100`	`101`	`}`
`@@ -116,7 +117,7 @@ protected override void OnOpen()`
`116`	`117`	`_server.Clients.Add(ID, clientName);`
`117`	`118`	`}`
`118`	`119`
`119`		`- Debug.Log($"[MCP Unity] WebSocket client '{clientName}' connected");`
	`120`	`+ McpLogger.LogInfo($"WebSocket client '{clientName}' connected");`
`120`	`121`	`}`
`121`	`122`
`122`	`123`	`/// <summary>`
`@@ -129,15 +130,15 @@ protected override void OnClose(CloseEventArgs e)`
`129`	`130`	`// Remove the client from the server`
`130`	`131`	`_server.Clients.Remove(ID);`
`131`	`132`
`132`		`- Debug.Log($"[MCP Unity] WebSocket client '{clientName}' disconnected: {e.Reason}");`
	`133`	`+ McpLogger.LogInfo($"WebSocket client '{clientName}' disconnected: {e.Reason}");`
`133`	`134`	`}`
`134`	`135`
`135`	`136`	`/// <summary>`
`136`	`137`	`/// Handle WebSocket errors`
`137`	`138`	`/// </summary>`
`138`	`139`	`protected override void OnError(ErrorEventArgs e)`
`139`	`140`	`{`
`140`		`- Debug.LogError($"[MCP Unity] WebSocket error: {e.Message}");`
	`141`	`+ McpLogger.LogError($"WebSocket error: {e.Message}");`
`141`	`142`	`}`
`142`	`143`
`143`	`144`	`/// <summary>`
`@@ -159,7 +160,7 @@ private IEnumerator ExecuteTool(McpToolBase tool, JObject parameters, TaskComple`
`159`	`160`	`}`
`160`	`161`	`catch (Exception ex)`
`161`	`162`	`{`
`162`		`- Debug.LogError($"[MCP Unity] Error executing tool {tool.Name}: {ex.Message}");`
	`163`	`+ McpLogger.LogError($"Error executing tool {tool.Name}: {ex.Message}");`
`163`	`164`	`tcs.SetResult(CreateErrorResponse(`
`164`	`165`	`$"Failed to execute tool {tool.Name}: {ex.Message}",`
`165`	`166`	`"tool_execution_error"`
`@@ -181,7 +182,7 @@ private IEnumerator FetchResourceCoroutine(McpResourceBase resource, JObject par`
`181`	`182`	`}`
`182`	`183`	`catch (Exception ex)`
`183`	`184`	`{`
`184`		`- Debug.LogError($"[MCP Unity] Error fetching resource {resource.Name}: {ex.Message}");`
	`185`	`+ McpLogger.LogError($"Error fetching resource {resource.Name}: {ex.Message}");`
`185`	`186`	`tcs.SetResult(CreateErrorResponse(`
`186`	`187`	`$"Failed to fetch resource {resource.Name}: {ex.Message}",`
`187`	`188`	`"resource_fetch_error"`