removed unecessary ping call and fixed case where mcp server would love connection with Unity

The issue where MCP server cannot reconnect with Unity still remains

Author: CoderGamester

Editor/UnityBridge/McpUnityEditorWindow.cs

@@ -111,7 +111,7 @@ private void DrawServerTab()
             GUI.enabled = !mcpUnityServer.IsListening;
             if (GUILayout.Button("Start Server", GUILayout.Height(30)))
             {
-                mcpUnityServer.StopServer();
+                mcpUnityServer.StartServer();
             }
 
             // Disconnect button - enabled only when connected

Editor/UnityBridge/McpUnitySocketHandler.cs

@@ -64,11 +64,6 @@ protected override async void OnMessage(MessageEventArgs e)
                 {
                     responseJson = CreateErrorResponse("Missing method in request", "invalid_request");
                 }
-                else if (method == "ping")
-                {
-                    // Handle MCP Protocol ping request - respond with an empty result object
-                    responseJson = new JObject();
-                }
                 else if (_server.TryGetTool(method, out var tool))
                 {
                     // Execute the tool

Server/build/resources/getAssetsResource.js

@@ -13,10 +13,6 @@ export function createGetAssetsResource(mcpUnity, logger) {
         handler: async (params) => {
             const assetType = params?.assetType;
             const searchPattern = params?.searchPattern;
-            logger.info(`Fetching assets from Asset Database${assetType ? ` of type ${assetType}` : ''}${searchPattern ? ` matching ${searchPattern}` : ''}`);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {

Server/build/resources/getConsoleLogResource.js

@@ -11,10 +11,6 @@ export function createGetConsoleLogsResource(mcpUnity, logger) {
             mimeType: resourceMimeType
         },
         handler: async (params) => {
-            logger.info(`Fetching Unity console logs`, params);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {}

Server/build/resources/getHierarchyResource.js

@@ -11,10 +11,6 @@ export function createGetHierarchyResource(mcpUnity, logger) {
             mimeType: resourceMimeType
         },
         handler: async (params) => {
-            logger.info(`Fetching Unity hierarchy`, params);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {}

Server/build/resources/getMenuItemResource.js

@@ -11,17 +11,14 @@ export function createGetMenuItemsResource(mcpUnity, logger) {
             mimeType: resourceMimeType
         },
         handler: async (params) => {
-            logger.info(`Fetching menu items list`, params);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {}
             });
             if (!response.success) {
                 throw new McpUnityError(ErrorType.RESOURCE_FETCH, response.message || 'Failed to fetch menu items from Unity');
             }
+            logger.info(`Fetching resource:`, response.menuItems);
             return {
                 contents: [{
                         uri: resourceUri,

Server/build/resources/getPackagesResource.js

@@ -11,10 +11,6 @@ export function createGetPackagesResource(mcpUnity, logger) {
             mimeType: resourceMimeType
         },
         handler: async (params) => {
-            logger.info('Fetching package list from Unity Package Manager');
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {}

Server/build/resources/getTestsResource.js

@@ -7,10 +7,6 @@ export function createGetTestsResource(mcpUnity, logger) {
         uri: `unity://${resourceName}`,
         handler: async (params) => {
             const { testMode, nameFilter } = params;
-            logger.info(`Fetching tests with filters: Mode=${testMode || 'All'}, Name=${nameFilter || 'none'}`);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: resourceName,
                 params: {

Server/build/tools/menuItemTool.js

@@ -9,10 +9,6 @@ export function createMenuItemTool(mcpUnity, logger) {
             menuPath: z.string().describe('The path to the menu item to execute (e.g. "GameObject/Create Empty")')
         }),
         handler: async ({ menuPath }) => {
-            logger.info(`Executing menu item: ${menuPath}`);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             const response = await mcpUnity.sendRequest({
                 method: toolName,
                 params: { menuPath }

Server/build/tools/notifyMessageTool.js

@@ -11,10 +11,6 @@ export function createNotifyMessageTool(mcpUnity, logger) {
         }),
         handler: async (params) => {
             const { message, type = 'info' } = params;
-            logger.info(`Sending notification to Unity console: ${message} (${type})`);
-            if (!mcpUnity.isConnected) {
-                throw new McpUnityError(ErrorType.CONNECTION, 'Not connected to Unity. Please ensure Unity is running with the MCP Unity plugin enabled.');
-            }
             // Send to Unity
             const response = await mcpUnity.sendRequest({
                 method: toolName,