refactor: update console logging and resource handling for more accurate output

- Rename getConsoleLogResource to getConsoleLogsResource for consistency
chore: remove the update pop up when changing the MCP port

Author: CoderGamester

.gitignore

@@ -114,6 +114,7 @@ GvhProjectSettings.xml
 **/package-lock.json
 **/package-lock.json.meta
 .codeiumignore
+.windsurf
 Server/log.txt
 Server/log.txt.meta
 log.txt

Editor/Resources/GetConsoleLogsResource.cs

@@ -13,27 +13,36 @@ public class GetConsoleLogsResource : McpResourceBase
         public GetConsoleLogsResource(IConsoleLogsService consoleLogsService)
         {
             Name = "get_console_logs";
-            Description = "Retrieves all logs from the Unity console";
-            Uri = "unity://logs";
+            Description = "Retrieves logs from the Unity console, optionally filtered by type (error, warning, info)";
+            Uri = "unity://logs/{logType}";
 
             _consoleLogsService = consoleLogsService;
         }
 
         /// <summary>
-        /// Fetch all logs from the Unity console
+        /// Fetch logs from the Unity console, optionally filtered by type
         /// </summary>
-        /// <param name="parameters">Resource parameters as a JObject (not used)</param>
+        /// <param name="parameters">Resource parameters as a JObject (may include 'logType')</param>
         /// <returns>A JObject containing the list of logs</returns>
         public override JObject Fetch(JObject parameters)
         {
-            // Get logs from the service
-            JArray logsArray = _consoleLogsService.GetAllLogsAsJson();
-            
+            string logType = null;
+            if (parameters != null && parameters.ContainsKey("logType") && parameters["logType"] != null)
+            {
+                logType = parameters["logType"].ToString()?.ToLowerInvariant();
+                if (string.IsNullOrWhiteSpace(logType))
+                {
+                    logType = null;
+                }
+            }
+
+            JArray logsArray = _consoleLogsService.GetAllLogsAsJson(logType);
+
             // Create the response
             return new JObject
             {
                 ["success"] = true,
-                ["message"] = $"Retrieved {logsArray.Count} log entries",
+                ["message"] = $"Retrieved {logsArray.Count} log entries" + (logType != null ? $" of type '{logType}'" : ""),
                 ["logs"] = logsArray
             };
         }

Editor/Services/ConsoleLogsService.cs

@@ -38,7 +38,7 @@ public ConsoleLogsService()
         public void StartListening()
         {
             // Register for log messages
-            Application.logMessageReceived += OnLogMessageReceived;
+            Application.logMessageReceivedThreaded += OnLogMessageReceived;
 
 #if UNITY_6000_0_OR_NEWER
             // Unity 6 specific implementation
@@ -55,7 +55,7 @@ public void StartListening()
         public void StopListening()
         {
             // Unregister from log messages
-            Application.logMessageReceived -= OnLogMessageReceived;
+            Application.logMessageReceivedThreaded -= OnLogMessageReceived;
 
 #if UNITY_6000_0_OR_NEWER
             // Unity 6 specific implementation
@@ -145,13 +145,16 @@ private void CheckConsoleClearViaReflection()
         private void OnLogMessageReceived(string logString, string stackTrace, LogType type)
         {
             // Add the log entry to our collection
-            _logEntries.Add(new LogEntry
+            lock (_logEntries)
             {
-                Message = logString,
-                StackTrace = stackTrace,
-                Type = type,
-                Timestamp = DateTime.Now
-            });
+                _logEntries.Add(new LogEntry
+                {
+                    Message = logString,
+                    StackTrace = stackTrace,
+                    Type = type,
+                    Timestamp = DateTime.Now
+                });
+            }
         }
 
 #if UNITY_6000_0_OR_NEWER

Editor/Tools/UpdateComponentTool.cs

@@ -134,6 +134,7 @@ private GameObject FindGameObjectByPath(string path)
         {
             // Split the path by '/'
             string[] pathParts = path.Split('/');
+            GameObject[] rootGameObjects = UnityEngine.SceneManagement.SceneManager.GetActiveScene().GetRootGameObjects();
 
             // If the path is empty, return null
             if (pathParts.Length == 0)
@@ -142,7 +143,7 @@ private GameObject FindGameObjectByPath(string path)
             }
 
             // Search through all root GameObjects in all scenes
-            foreach (GameObject rootObj in UnityEngine.SceneManagement.SceneManager.GetActiveScene().GetRootGameObjects())
+            foreach (GameObject rootObj in rootGameObjects)
             {
                 if (rootObj.name == pathParts[0])
                 {

Editor/UnityBridge/McpUnityEditorWindow.cs

@@ -99,9 +99,7 @@ private void DrawServerTab()
                 settings.Port = newPort;
                 settings.SaveSettings();
                 mcpUnityServer.StopServer();
-                EditorUtility.DisplayDialog("Restart MCP Client",
-                    "Please restart your MCP Client (Windsurf, Cursor, Calude Destktop, etc.) to apply the changes.",
-                    "OK");
+                mcpUnityServer.StartServer(); // Restart the server.newPort
             }
             EditorGUILayout.EndHorizontal();
 
@@ -261,6 +259,10 @@ private void DrawServerTab()
                 {
                     EditorUtility.DisplayDialog("Success", "The MCP configuration was successfully added to the Cursor config file.", "OK");
                 }
+                else
+                {
+                    EditorUtility.DisplayDialog("Error", "The MCP configuration could not be added to the Cursor Desktop config file.", "OK");
+                }
             }
 
             EditorGUILayout.EndVertical();

Server~/src/index.ts

@@ -11,7 +11,7 @@ import { registerSendConsoleLogTool } from './tools/sendConsoleLogTool.js';
 import { registerUpdateComponentTool } from './tools/updateComponentTool.js';
 import { registerAddAssetToSceneTool } from './tools/addAssetToSceneTool.js';
 import { registerGetMenuItemsResource } from './resources/getMenuItemResource.js';
-import { registerGetConsoleLogsResource } from './resources/getConsoleLogResource.js';
+import { registerGetConsoleLogsResource } from './resources/getConsoleLogsResource.js';
 import { registerGetHierarchyResource } from './resources/getHierarchyResource.js';
 import { registerGetPackagesResource } from './resources/getPackagesResource.js';
 import { registerGetAssetsResource } from './resources/getAssetsResource.js';

Server~/src/resources/getConsoleLogResource.ts

@@ -0,0 +1,101 @@
+import { Logger } from '../utils/logger.js';
+import { ResourceTemplate, McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { ReadResourceResult } from '@modelcontextprotocol/sdk/types.js';
+import { McpUnity } from '../unity/mcpUnity.js';
+import { McpUnityError, ErrorType } from '../utils/errors.js';
+import { Variables } from '@modelcontextprotocol/sdk/shared/uriTemplate.js';
+
+// Constants for the resource
+const resourceName = 'get_console_logs';
+const resourceMimeType = 'application/json';
+const resourceUri = 'unity://logs/{logType}';
+const resourceTemplate = new ResourceTemplate(resourceUri, {
+  list: () => listLogTypes(resourceMimeType)
+});
+
+function listLogTypes(resourceMimeType: string) {
+  return {
+    resources: [
+      {
+        uri: `unity://logs/`,
+        name: "All logs",
+        description: "Retrieve all Unity console logs",
+        mimeType: resourceMimeType
+      },
+      {
+        uri: `unity://logs/error`,
+        name: "Error logs",
+        description: "Retrieve only error logs from the Unity console",
+        mimeType: resourceMimeType
+      },
+      {
+        uri: `unity://logs/warning`,
+        name: "Warning logs",
+        description: "Retrieve only warning logs from the Unity console",
+        mimeType: resourceMimeType
+      },
+      {
+        uri: `unity://logs/info`,
+        name: "Info logs",
+        description: "Retrieve only info logs from the Unity console",
+        mimeType: resourceMimeType
+      }
+    ]
+  };
+}
+
+/**
+ * Registers the get_console_logs resource with the MCP server
+ */
+export function registerGetConsoleLogsResource(server: McpServer, mcpUnity: McpUnity, logger: Logger) {
+  logger.info(`Registering resource: ${resourceName}`);
+
+  server.resource(
+    resourceName,
+    resourceTemplate,
+    {
+      description: 'Retrieve Unity console logs by type',
+      mimeType: resourceMimeType
+    },
+    async (uri, variables) => {
+      try {
+        return await resourceHandler(mcpUnity, uri, variables, logger);
+      } catch (error) {
+        logger.error(`Error handling resource ${resourceName}: ${error}`);
+        throw error;
+      }
+    }
+  );
+}
+
+/**
+ * Handles requests for Unity console logs by log type
+ */
+async function resourceHandler(mcpUnity: McpUnity, uri: URL, variables: Variables, logger: Logger): Promise<ReadResourceResult> {
+  // Extract and convert the parameter from the template variables
+  let logType = variables["logType"] ? decodeURIComponent(variables["logType"] as string) : undefined;
+  if (logType === '') logType = undefined;
+
+  // Send request to Unity
+  const response = await mcpUnity.sendRequest({
+    method: resourceName,
+    params: {
+      logType: logType
+    }
+  });
+
+  if (!response.success) {
+    throw new McpUnityError(
+      ErrorType.RESOURCE_FETCH,
+      response.message || 'Failed to fetch logs from Unity'
+    );
+  }
+
+  return {
+    contents: [{
+      uri: `unity://logs/${logType ?? ''}`,
+      mimeType: resourceMimeType,
+      text: JSON.stringify(response, null, 2)
+    }]
+  };
+}