refactor: update package ID from justinpbarnett to coplaydev across codebase

Author: msanatan

UnityMcpBridge/Editor/Helpers/McpPathResolver.cs

@@ -90,9 +90,9 @@ private static bool IsDevelopmentMode()
                 string manifestContent = File.ReadAllText(manifestPath);
                 // Look specifically for our package dependency set to a file: URL
                 // This avoids auto-enabling dev mode just because a repo exists elsewhere on disk
-                if (manifestContent.IndexOf("\"com.justinpbarnett.unity-mcp\"", StringComparison.OrdinalIgnoreCase) >= 0)
+                if (manifestContent.IndexOf("\"com.coplaydev.unity-mcp\"", StringComparison.OrdinalIgnoreCase) >= 0)
                 {
-                    int idx = manifestContent.IndexOf("com.justinpbarnett.unity-mcp", StringComparison.OrdinalIgnoreCase);
+                    int idx = manifestContent.IndexOf("com.coplaydev.unity-mcp", StringComparison.OrdinalIgnoreCase);
                     // Crude but effective: check for "file:" in the same line/value
                     if (manifestContent.IndexOf("file:", idx, StringComparison.OrdinalIgnoreCase) >= 0
                         && manifestContent.IndexOf("\n", idx, StringComparison.OrdinalIgnoreCase) > manifestContent.IndexOf("file:", idx, StringComparison.OrdinalIgnoreCase))

UnityMcpBridge/Editor/Helpers/ServerPathResolver.cs

@@ -99,24 +99,16 @@ public static bool TryFindEmbeddedServerSource(out string srcPath, bool warnOnLe
             return false;
         }
 
-        private static bool TryResolveWithinPackage(UnityEditor.PackageManager.PackageInfo p, out string srcPath, bool warnOnLegacyPackageId)
+        private static bool TryResolveWithinPackage(UnityEditor.PackageManager.PackageInfo p, out string srcPath)
         {
             const string CurrentId = "com.coplaydev.unity-mcp";
-            const string LegacyId = "com.justinpbarnett.unity-mcp";
 
             srcPath = null;
-            if (p == null || (p.name != CurrentId && p.name != LegacyId))
+            if (p == null || p.name != CurrentId)
             {
                 return false;
             }
 
-            if (warnOnLegacyPackageId && p.name == LegacyId)
-            {
-                Debug.LogWarning(
-                    "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.");
-            }
-
             string packagePath = p.resolvedPath;
 
             // Preferred tilde folder (embedded but excluded from import)

UnityMcpBridge/Editor/Windows/MCPForUnityEditorWindow.cs

@@ -1169,35 +1169,6 @@ private string FindPackagePythonDirectory()
             return McpPathResolver.FindPackagePythonDirectory(debugLogsEnabled);
         }
 
-        private bool IsDevelopmentMode()
-        {
-            try
-            {
-                // Only treat as development if manifest explicitly references a local file path for the package
-                string manifestPath = Path.Combine(Application.dataPath, "..", "Packages", "manifest.json");
-                if (!File.Exists(manifestPath)) return false;
-
-                string manifestContent = File.ReadAllText(manifestPath);
-                // Look specifically for our package dependency set to a file: URL
-                // This avoids auto-enabling dev mode just because a repo exists elsewhere on disk
-                if (manifestContent.IndexOf("\"com.justinpbarnett.unity-mcp\"", StringComparison.OrdinalIgnoreCase) >= 0)
-                {
-                    int idx = manifestContent.IndexOf("com.justinpbarnett.unity-mcp", StringComparison.OrdinalIgnoreCase);
-                    // Crude but effective: check for "file:" in the same line/value
-                    if (manifestContent.IndexOf("file:", idx, StringComparison.OrdinalIgnoreCase) >= 0
-                        && manifestContent.IndexOf("\n", idx, StringComparison.OrdinalIgnoreCase) > manifestContent.IndexOf("file:", idx, StringComparison.OrdinalIgnoreCase))
-                    {
-                        return true;
-                    }
-                }
-                return false;
-            }
-            catch
-            {
-                return false;
-            }
-        }
-
         private string ConfigureMcpClient(McpClient mcpClient)
         {
             try