refactor: update branding from UNITY-MCP to MCP-FOR-UNITY across all log messages and warnings

msanatan · msanatan · commit fb7ebfd2a4fe · 2025-08-17T22:57:43.000-04:00
diff --git a/UnityMcpBridge/Editor/Helpers/PortManager.cs b/UnityMcpBridge/Editor/Helpers/PortManager.cs
@@ -48,7 +48,7 @@ public static int GetPortWithFallback()
                 string.Equals(storedConfig.project_path ?? string.Empty, Application.dataPath ?? string.Empty, StringComparison.OrdinalIgnoreCase) &&
                 IsPortAvailable(storedConfig.unity_port))
             {
-                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Using stored port {storedConfig.unity_port} for current project");
+                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Using stored port {storedConfig.unity_port} for current project");
                 return storedConfig.unity_port;
             }
 
@@ -57,7 +57,7 @@ public static int GetPortWithFallback()
             {
                 if (WaitForPortRelease(storedConfig.unity_port, 1500))
                 {
-                    if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Stored port {storedConfig.unity_port} became available after short wait");
+                    if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Stored port {storedConfig.unity_port} became available after short wait");
                     return storedConfig.unity_port;
                 }
                 // Prefer sticking to the same port; let the caller handle bind retries/fallbacks
@@ -78,7 +78,7 @@ public static int DiscoverNewPort()
         {
             int newPort = FindAvailablePort();
             SavePort(newPort);
-            if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Discovered and saved new port: {newPort}");
+            if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Discovered and saved new port: {newPort}");
             return newPort;
         }
 
@@ -91,18 +91,18 @@ private static int FindAvailablePort()
             // Always try default port first
             if (IsPortAvailable(DefaultPort))
             {
-                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Using default port {DefaultPort}");
+                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Using default port {DefaultPort}");
                 return DefaultPort;
             }
 
-            if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Default port {DefaultPort} is in use, searching for alternative...");
+            if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Default port {DefaultPort} is in use, searching for alternative...");
 
             // Search for alternatives
             for (int port = DefaultPort + 1; port < DefaultPort + MaxPortAttempts; port++)
             {
                 if (IsPortAvailable(port))
                 {
-                    if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Found available port {port}");
+                    if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Found available port {port}");
                     return port;
                 }
             }
@@ -211,7 +211,7 @@ private static void SavePort(int port)
                 string legacy = Path.Combine(GetRegistryDirectory(), RegistryFileName);
                 File.WriteAllText(legacy, json);
 
-                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Saved port {port} to storage");
+                if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Saved port {port} to storage");
             }
             catch (Exception ex)
             {
diff --git a/UnityMcpBridge/Editor/Helpers/ServerInstaller.cs b/UnityMcpBridge/Editor/Helpers/ServerInstaller.cs
@@ -246,7 +246,7 @@ public static bool RepairPythonEnvironment()
                     return false;
                 }
 
-                Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: Python environment repaired successfully.");
+                Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Python environment repaired successfully.");
                 return true;
             }
             catch (Exception ex)
diff --git a/UnityMcpBridge/Editor/Helpers/ServerPathResolver.cs b/UnityMcpBridge/Editor/Helpers/ServerPathResolver.cs
@@ -113,7 +113,7 @@ private static bool TryResolveWithinPackage(UnityEditor.PackageManager.PackageIn
             if (warnOnLegacyPackageId && p.name == LegacyId)
             {
                 Debug.LogWarning(
-                    "UnityMCP: Detected legacy package id 'com.justinpbarnett.unity-mcp'. " +
+                    "MCP For Unity: Detected legacy package id 'com.justinpbarnett.unity-mcp'. " +
                     "Please update Packages/manifest.json to 'com.coplaydev.unity-mcp' to avoid future breakage.");
             }
 
diff --git a/UnityMcpBridge/Editor/UnityMcpBridge.cs b/UnityMcpBridge/Editor/UnityMcpBridge.cs
@@ -46,7 +46,7 @@ private static void LogBreadcrumb(string stage)
         {
             if (IsDebugEnabled())
             {
-                Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: [{stage}]");
+                Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: [{stage}]");
             }
         }
 
@@ -227,7 +227,7 @@ public static void Start()
                 // Don't restart if already running on a working port
                 if (isRunning && listener != null)
                 {
-                    Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge already running on port {currentUnityPort}");
+                    Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge already running on port {currentUnityPort}");
                     return;
                 }
 
@@ -310,7 +310,7 @@ public static void Start()
 
                     isRunning = true;
                     isAutoConnectMode = false;
-                    Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge started on port {currentUnityPort}.");
+                    Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge started on port {currentUnityPort}.");
                     Task.Run(ListenerLoop);
                     EditorApplication.update += ProcessCommands;
                     // Write initial heartbeat immediately
@@ -343,7 +343,7 @@ public static void Stop()
                     listener?.Stop();
                     listener = null;
                     EditorApplication.update -= ProcessCommands;
-                    Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge stopped.");
+                    Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge stopped.");
                 }
                 catch (Exception ex)
                 {
diff --git a/UnityMcpBridge/Editor/Windows/UnityMcpEditorWindow.cs b/UnityMcpBridge/Editor/Windows/UnityMcpEditorWindow.cs
@@ -1623,7 +1623,7 @@ private void RegisterWithClaudeCode(string pythonDir)
                     var existingClient = mcpClients.clients.FirstOrDefault(c => c.mcpType == McpTypes.ClaudeCode);
                     if (existingClient != null) CheckClaudeCodeConfiguration(existingClient);
                     Repaint();
-                    UnityEngine.Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMCP already registered with Claude Code.");
+                    UnityEngine.Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMCP already registered with Claude Code.");
                 }
                 else
                 {
@@ -1636,7 +1636,7 @@ private void RegisterWithClaudeCode(string pythonDir)
             var claudeClient = mcpClients.clients.FirstOrDefault(c => c.mcpType == McpTypes.ClaudeCode);
             if (claudeClient != null) CheckClaudeCodeConfiguration(claudeClient);
             Repaint();
-            UnityEngine.Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: Registered with Claude Code.");
+            UnityEngine.Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Registered with Claude Code.");
         }
 
         private void UnregisterWithClaudeCode()

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public static int GetPortWithFallback()`
`48`	`48`	`string.Equals(storedConfig.project_path ?? string.Empty, Application.dataPath ?? string.Empty, StringComparison.OrdinalIgnoreCase) &&`
`49`	`49`	`IsPortAvailable(storedConfig.unity_port))`
`50`	`50`	`{`
`51`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Using stored port {storedConfig.unity_port} for current project");`
	`51`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Using stored port {storedConfig.unity_port} for current project");`
`52`	`52`	`return storedConfig.unity_port;`
`53`	`53`	`}`
`54`	`54`
`@@ -57,7 +57,7 @@ public static int GetPortWithFallback()`
`57`	`57`	`{`
`58`	`58`	`if (WaitForPortRelease(storedConfig.unity_port, 1500))`
`59`	`59`	`{`
`60`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Stored port {storedConfig.unity_port} became available after short wait");`
	`60`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Stored port {storedConfig.unity_port} became available after short wait");`
`61`	`61`	`return storedConfig.unity_port;`
`62`	`62`	`}`
`63`	`63`	`// Prefer sticking to the same port; let the caller handle bind retries/fallbacks`
`@@ -78,7 +78,7 @@ public static int DiscoverNewPort()`
`78`	`78`	`{`
`79`	`79`	`int newPort = FindAvailablePort();`
`80`	`80`	`SavePort(newPort);`
`81`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Discovered and saved new port: {newPort}");`
	`81`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Discovered and saved new port: {newPort}");`
`82`	`82`	`return newPort;`
`83`	`83`	`}`
`84`	`84`
`@@ -91,18 +91,18 @@ private static int FindAvailablePort()`
`91`	`91`	`// Always try default port first`
`92`	`92`	`if (IsPortAvailable(DefaultPort))`
`93`	`93`	`{`
`94`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Using default port {DefaultPort}");`
	`94`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Using default port {DefaultPort}");`
`95`	`95`	`return DefaultPort;`
`96`	`96`	`}`
`97`	`97`
`98`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Default port {DefaultPort} is in use, searching for alternative...");`
	`98`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Default port {DefaultPort} is in use, searching for alternative...");`
`99`	`99`
`100`	`100`	`// Search for alternatives`
`101`	`101`	`for (int port = DefaultPort + 1; port < DefaultPort + MaxPortAttempts; port++)`
`102`	`102`	`{`
`103`	`103`	`if (IsPortAvailable(port))`
`104`	`104`	`{`
`105`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Found available port {port}");`
	`105`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Found available port {port}");`
`106`	`106`	`return port;`
`107`	`107`	`}`
`108`	`108`	`}`
`@@ -211,7 +211,7 @@ private static void SavePort(int port)`
`211`	`211`	`string legacy = Path.Combine(GetRegistryDirectory(), RegistryFileName);`
`212`	`212`	`File.WriteAllText(legacy, json);`
`213`	`213`
`214`		`- if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: Saved port {port} to storage");`
	`214`	`+ if (IsDebugEnabled()) Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Saved port {port} to storage");`
`215`	`215`	`}`
`216`	`216`	`catch (Exception ex)`
`217`	`217`	`{`
Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ public static bool RepairPythonEnvironment()`
`246`	`246`	`return false;`
`247`	`247`	`}`
`248`	`248`
`249`		`- Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: Python environment repaired successfully.");`
	`249`	`+ Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Python environment repaired successfully.");`
`250`	`250`	`return true;`
`251`	`251`	`}`
`252`	`252`	`catch (Exception ex)`
Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ private static bool TryResolveWithinPackage(UnityEditor.PackageManager.PackageIn`
`113`	`113`	`if (warnOnLegacyPackageId && p.name == LegacyId)`
`114`	`114`	`{`
`115`	`115`	`Debug.LogWarning(`
`116`		`- "UnityMCP: Detected legacy package id 'com.justinpbarnett.unity-mcp'. " +`
	`116`	`+ "MCP For Unity: Detected legacy package id 'com.justinpbarnett.unity-mcp'. " +`
`117`	`117`	`"Please update Packages/manifest.json to 'com.coplaydev.unity-mcp' to avoid future breakage.");`
`118`	`118`	`}`
`119`	`119`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ private static void LogBreadcrumb(string stage)`
`46`	`46`	`{`
`47`	`47`	`if (IsDebugEnabled())`
`48`	`48`	`{`
`49`		`- Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: [{stage}]");`
	`49`	`+ Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: [{stage}]");`
`50`	`50`	`}`
`51`	`51`	`}`
`52`	`52`
`@@ -227,7 +227,7 @@ public static void Start()`
`227`	`227`	`// Don't restart if already running on a working port`
`228`	`228`	`if (isRunning && listener != null)`
`229`	`229`	`{`
`230`		`- Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge already running on port {currentUnityPort}");`
	`230`	`+ Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge already running on port {currentUnityPort}");`
`231`	`231`	`return;`
`232`	`232`	`}`
`233`	`233`
`@@ -310,7 +310,7 @@ public static void Start()`
`310`	`310`
`311`	`311`	`isRunning = true;`
`312`	`312`	`isAutoConnectMode = false;`
`313`		`- Debug.Log($"<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge started on port {currentUnityPort}.");`
	`313`	`+ Debug.Log($"<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge started on port {currentUnityPort}.");`
`314`	`314`	`Task.Run(ListenerLoop);`
`315`	`315`	`EditorApplication.update += ProcessCommands;`
`316`	`316`	`// Write initial heartbeat immediately`
`@@ -343,7 +343,7 @@ public static void Stop()`
`343`	`343`	`listener?.Stop();`
`344`	`344`	`listener = null;`
`345`	`345`	`EditorApplication.update -= ProcessCommands;`
`346`		`- Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMcpBridge stopped.");`
	`346`	`+ Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMcpBridge stopped.");`
`347`	`347`	`}`
`348`	`348`	`catch (Exception ex)`
`349`	`349`	`{`
Original file line number	Diff line number	Diff line change
`@@ -1623,7 +1623,7 @@ private void RegisterWithClaudeCode(string pythonDir)`
`1623`	`1623`	`var existingClient = mcpClients.clients.FirstOrDefault(c => c.mcpType == McpTypes.ClaudeCode);`
`1624`	`1624`	`if (existingClient != null) CheckClaudeCodeConfiguration(existingClient);`
`1625`	`1625`	`Repaint();`
`1626`		`- UnityEngine.Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: UnityMCP already registered with Claude Code.");`
	`1626`	`+ UnityEngine.Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: UnityMCP already registered with Claude Code.");`
`1627`	`1627`	`}`
`1628`	`1628`	`else`
`1629`	`1629`	`{`
`@@ -1636,7 +1636,7 @@ private void RegisterWithClaudeCode(string pythonDir)`
`1636`	`1636`	`var claudeClient = mcpClients.clients.FirstOrDefault(c => c.mcpType == McpTypes.ClaudeCode);`
`1637`	`1637`	`if (claudeClient != null) CheckClaudeCodeConfiguration(claudeClient);`
`1638`	`1638`	`Repaint();`
`1639`		`- UnityEngine.Debug.Log("<b><color=#2EA3FF>UNITY-MCP</color></b>: Registered with Claude Code.");`
	`1639`	`+ UnityEngine.Debug.Log("<b><color=#2EA3FF>MCP-FOR-UNITY</color></b>: Registered with Claude Code.");`
`1640`	`1640`	`}`
`1641`	`1641`
`1642`	`1642`	`private void UnregisterWithClaudeCode()`