@atelierai/uco 1.0.7 → 1.0.8

package/vendor/plugin/com.atelierai.unity.copilot/Editor/Scripts/UnityCopilotPluginEditor.cs

@@ -1,69 +1,69 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  Author: Ivan Murzak (https://github.com/IvanMurzak)             │
-│  Repository: GitHub (https://github.com/IvanMurzak/Unity-MCP)    │
-│  Copyright (c) 2025 Ivan Murzak                                  │
-│  Licensed under the Apache License, Version 2.0.                 │
-│  See the LICENSE file in the project root for more information.  │
-└──────────────────────────────────────────────────────────────────┘
-*/
-
-#nullable enable
-using com.AtelierAI.Unity.Copilot.Runtime.Utils;
-
-namespace com.AtelierAI.Unity.Copilot
-{
-    /// <summary>
-    /// Editor-only singleton that owns the persistent MCP connection used
-    /// by Unity Editor tooling (Copilot window, configurators, etc.).
-    /// Lives in the Editor assembly — not accessible from Runtime code.
-    /// </summary>
-    public partial class UnityCopilotPluginEditor : UnityCopilotPlugin
-    {
-        /// <summary>
-        /// Captures runtime overrides (env vars / CLI flags) that were layered on top of the
-        /// on-disk config during construction. Used by <see cref="Save"/> to round-trip the
-        /// disk-baseline values to disk while keeping the runtime overrides on the in-memory
-        /// config — runtime-only overrides MUST NOT leak into the persisted JSON.
-        /// </summary>
-        internal EnvironmentUtils.OverrideRecord RuntimeOverrides { get; private set; }
-            = new EnvironmentUtils.OverrideRecord();
-
-        protected UnityCopilotPluginEditor() : base()
-        {
-            var config = GetOrCreateConfig(out var wasCreated);
-            unityConnectionConfig = config;
-            ApplyLogLevel(unityConnectionConfig.LogLevel);
-            // Layered loader: disk values are already on `config`; env vars and CLI flags
-            // are applied here as runtime overrides. The OverrideRecord captures the
-            // disk-baseline values so Save() can persist them without leaking the overrides.
-            RuntimeOverrides = EnvironmentUtils.ApplyEnvironmentOverrides(unityConnectionConfig);
-            if (wasCreated)
-                Save();
-            IncrementSingletonCount();
-        }
-
-        public void Validate()
-        {
-            var changed = false;
-            var data = unityConnectionConfig ??= new UnityConnectionConfig();
-
-            if (string.IsNullOrEmpty(data.LocalHost))
-            {
-                data.LocalHost = UnityConnectionConfig.DefaultHost;
-                changed = true;
-            }
-
-            // Data was changed during validation, need to notify subscribers
-            if (changed)
-                NotifyChanged(data);
-        }
-
-        public override void Dispose()
-        {
-            DecrementSingletonCount();
-            base.Dispose();
-            DisposeMcpPluginInstance();
-        }
-    }
-}
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  Author: Ivan Murzak (https://github.com/IvanMurzak)             │
+│  Repository: GitHub (https://github.com/IvanMurzak/Unity-MCP)    │
+│  Copyright (c) 2025 Ivan Murzak                                  │
+│  Licensed under the Apache License, Version 2.0.                 │
+│  See the LICENSE file in the project root for more information.  │
+└──────────────────────────────────────────────────────────────────┘
+*/
+
+#nullable enable
+using com.AtelierAI.Unity.Copilot.Runtime.Utils;
+
+namespace com.AtelierAI.Unity.Copilot
+{
+    /// <summary>
+    /// Editor-only singleton that owns the persistent Uco connection used
+    /// by Unity Editor tooling (Copilot window, configurators, etc.).
+    /// Lives in the Editor assembly — not accessible from Runtime code.
+    /// </summary>
+    public partial class UnityCopilotPluginEditor : UnityCopilotPlugin
+    {
+        /// <summary>
+        /// Captures runtime overrides (env vars / CLI flags) that were layered on top of the
+        /// on-disk config during construction. Used by <see cref="Save"/> to round-trip the
+        /// disk-baseline values to disk while keeping the runtime overrides on the in-memory
+        /// config — runtime-only overrides MUST NOT leak into the persisted JSON.
+        /// </summary>
+        internal EnvironmentUtils.OverrideRecord RuntimeOverrides { get; private set; }
+            = new EnvironmentUtils.OverrideRecord();
+
+        protected UnityCopilotPluginEditor() : base()
+        {
+            var config = GetOrCreateConfig(out var wasCreated);
+            unityConnectionConfig = config;
+            ApplyLogLevel(unityConnectionConfig.LogLevel);
+            // Layered loader: disk values are already on `config`; env vars and CLI flags
+            // are applied here as runtime overrides. The OverrideRecord captures the
+            // disk-baseline values so Save() can persist them without leaking the overrides.
+            RuntimeOverrides = EnvironmentUtils.ApplyEnvironmentOverrides(unityConnectionConfig);
+            if (wasCreated)
+                Save();
+            IncrementSingletonCount();
+        }
+
+        public void Validate()
+        {
+            var changed = false;
+            var data = unityConnectionConfig ??= new UnityConnectionConfig();
+
+            if (string.IsNullOrEmpty(data.LocalHost))
+            {
+                data.LocalHost = UnityConnectionConfig.DefaultHost;
+                changed = true;
+            }
+
+            // Data was changed during validation, need to notify subscribers
+            if (changed)
+                NotifyChanged(data);
+        }
+
+        public override void Dispose()
+        {
+            DecrementSingletonCount();
+            base.Dispose();
+            DisposeUcoPluginInstance();
+        }
+    }
+}

package/vendor/plugin/com.atelierai.unity.copilot/Editor/Scripts/Utils/PackageUtils.cs

@@ -1,168 +1,168 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  Author: Ivan Murzak (https://github.com/IvanMurzak)             │
-│  Repository: GitHub (https://github.com/IvanMurzak/Unity-MCP)    │
-│  Copyright (c) 2025 Ivan Murzak                                  │
-│  Licensed under the Apache License, Version 2.0.                 │
-│  See the LICENSE file in the project root for more information.  │
-└──────────────────────────────────────────────────────────────────┘
-*/
-
-#nullable enable
-using System.Collections.Generic;
-using System.Linq;
-using com.AtelierAI.Uco.Framework.Common.Model;
-using UnityEditor;
-
-namespace com.AtelierAI.Unity.Copilot.Editor.Utils
-{
-    /// <summary>
-    /// Utility class for handling package operations that may trigger domain reloads.
-    /// Uses SessionState to persist notification data across domain reloads.
-    /// </summary>
-    public static class PackageUtils
-    {
-        private const string PendingNotificationKeysKey = "MCP_PendingPackageNotificationKeys";
-        private const string NotificationDataSeparator = "<MCP_PKG_SEP>";
-
-        private static bool _processPendingScheduled = false;
-
-        public static void Init()
-        {
-            // Process any pending notifications after domain reload
-            ScheduleProcessPendingNotifications();
-        }
-
-        private static void ScheduleProcessPendingNotifications()
-        {
-            if (_processPendingScheduled)
-                return;
-
-            _processPendingScheduled = true;
-            EditorApplication.update += ProcessPendingNotificationsOnce;
-        }
-
-        private static void ProcessPendingNotificationsOnce()
-        {
-            EditorApplication.update -= ProcessPendingNotificationsOnce;
-
-            if (!_processPendingScheduled)
-                return;
-
-            _processPendingScheduled = false;
-            ProcessPendingNotifications();
-        }
-
-        /// <summary>
-        /// Schedules a notification to be sent after Unity domain reload completes.
-        /// Uses SessionState to persist across domain reloads.
-        /// </summary>
-        /// <param name="requestId">The request ID to track</param>
-        /// <param name="packageIdentifier">The package identifier that was operated on</param>
-        /// <param name="operationType">The type of operation performed (Add, Remove)</param>
-        /// <param name="expectedResult">Whether the operation was expected to succeed before domain reload</param>
-        public static void SchedulePostDomainReloadNotification(string requestId, string packageIdentifier, string operationType, bool expectedResult)
-        {
-            var notificationKey = $"MCP_PendingPackageNotification_{requestId}";
-            var notificationData = $"{requestId}{NotificationDataSeparator}{packageIdentifier}{NotificationDataSeparator}{operationType}{NotificationDataSeparator}{expectedResult}";
-
-            // Store the notification data
-            SessionState.SetString(notificationKey, notificationData);
-
-            // Add to the key list
-            var existingKeys = SessionState.GetString(PendingNotificationKeysKey, string.Empty);
-            var keyList = string.IsNullOrEmpty(existingKeys)
-                ? new List<string>()
-                : existingKeys.Split(',').Where(k => !string.IsNullOrEmpty(k)).ToList();
-
-            if (!keyList.Contains(notificationKey))
-            {
-                keyList.Add(notificationKey);
-                SessionState.SetString(PendingNotificationKeysKey, string.Join(",", keyList));
-            }
-        }
-
-        /// <summary>
-        /// Called by InitializeOnLoad to process any pending notifications after domain reload.
-        /// </summary>
-        public static void ProcessPendingNotifications()
-        {
-            // Get the list of pending notification keys from SessionState
-            var pendingKeys = SessionState.GetString(PendingNotificationKeysKey, string.Empty);
-            if (string.IsNullOrEmpty(pendingKeys))
-                return;
-
-            var keys = pendingKeys.Split(',').Where(k => !string.IsNullOrEmpty(k)).ToList();
-            if (keys.Count == 0)
-                return;
-
-            var processedKeys = new List<string>();
-
-            foreach (var key in keys)
-            {
-                var notificationData = SessionState.GetString(key, string.Empty);
-                if (string.IsNullOrEmpty(notificationData))
-                {
-                    processedKeys.Add(key);
-                    continue;
-                }
-
-                var parts = notificationData.Split(NotificationDataSeparator);
-                if (parts.Length != 4)
-                {
-                    processedKeys.Add(key);
-                    continue;
-                }
-
-                var requestId = parts[0];
-                var packageIdentifier = parts[1];
-                var operationType = parts[2];
-                var expectedResult = bool.Parse(parts[3]);
-
-                // Check for compilation errors after package operation
-                var hasCompilationErrors = ScriptUtils.HasCompilationErrors();
-
-                ResponseCallTool response;
-                if (hasCompilationErrors)
-                {
-                    var errorDetails = ScriptUtils.GetCompilationErrorDetails();
-                    var message = $"[Warning] Package {operationType} completed: {packageIdentifier}, but compilation errors occurred after domain reload. Details:\n{errorDetails}";
-                    response = ResponseCallTool.Success(message).SetRequestID(requestId);
-                }
-                else if (expectedResult)
-                {
-                    var message = $"[Success] Package {operationType} completed: {packageIdentifier}. Domain reload finished successfully.";
-                    response = ResponseCallTool.Success(message).SetRequestID(requestId);
-                }
-                else
-                {
-                    var message = $"[Error] Package {operationType} failed: {packageIdentifier}";
-                    response = ResponseCallTool.Error(message).SetRequestID(requestId);
-                }
-
-                // Send notification and mark for cleanup
-                _ = UnityCopilotPluginEditor.NotifyToolRequestCompleted(new RequestToolCompletedData
-                {
-                    RequestId = requestId,
-                    Result = response
-                });
-                processedKeys.Add(key);
-            }
-
-            // Clean up processed keys
-            foreach (var key in processedKeys)
-                SessionState.EraseString(key);
-
-            // Update the key list
-            var remainingKeys = keys.Except(processedKeys).ToList();
-            if (remainingKeys.Count > 0)
-            {
-                SessionState.SetString(PendingNotificationKeysKey, string.Join(",", remainingKeys));
-            }
-            else
-            {
-                SessionState.EraseString(PendingNotificationKeysKey);
-            }
-        }
-    }
-}
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  Author: Ivan Murzak (https://github.com/IvanMurzak)             │
+│  Repository: GitHub (https://github.com/IvanMurzak/Unity-MCP)    │
+│  Copyright (c) 2025 Ivan Murzak                                  │
+│  Licensed under the Apache License, Version 2.0.                 │
+│  See the LICENSE file in the project root for more information.  │
+└──────────────────────────────────────────────────────────────────┘
+*/
+
+#nullable enable
+using System.Collections.Generic;
+using System.Linq;
+using com.AtelierAI.Uco.Framework.Common.Model;
+using UnityEditor;
+
+namespace com.AtelierAI.Unity.Copilot.Editor.Utils
+{
+    /// <summary>
+    /// Utility class for handling package operations that may trigger domain reloads.
+    /// Uses SessionState to persist notification data across domain reloads.
+    /// </summary>
+    public static class PackageUtils
+    {
+        private const string PendingNotificationKeysKey = "UCO_PendingPackageNotificationKeys";
+        private const string NotificationDataSeparator = "<UCO_PKG_SEP>";
+
+        private static bool _processPendingScheduled = false;
+
+        public static void Init()
+        {
+            // Process any pending notifications after domain reload
+            ScheduleProcessPendingNotifications();
+        }
+
+        private static void ScheduleProcessPendingNotifications()
+        {
+            if (_processPendingScheduled)
+                return;
+
+            _processPendingScheduled = true;
+            EditorApplication.update += ProcessPendingNotificationsOnce;
+        }
+
+        private static void ProcessPendingNotificationsOnce()
+        {
+            EditorApplication.update -= ProcessPendingNotificationsOnce;
+
+            if (!_processPendingScheduled)
+                return;
+
+            _processPendingScheduled = false;
+            ProcessPendingNotifications();
+        }
+
+        /// <summary>
+        /// Schedules a notification to be sent after Unity domain reload completes.
+        /// Uses SessionState to persist across domain reloads.
+        /// </summary>
+        /// <param name="requestId">The request ID to track</param>
+        /// <param name="packageIdentifier">The package identifier that was operated on</param>
+        /// <param name="operationType">The type of operation performed (Add, Remove)</param>
+        /// <param name="expectedResult">Whether the operation was expected to succeed before domain reload</param>
+        public static void SchedulePostDomainReloadNotification(string requestId, string packageIdentifier, string operationType, bool expectedResult)
+        {
+            var notificationKey = $"UCO_PendingPackageNotification_{requestId}";
+            var notificationData = $"{requestId}{NotificationDataSeparator}{packageIdentifier}{NotificationDataSeparator}{operationType}{NotificationDataSeparator}{expectedResult}";
+
+            // Store the notification data
+            SessionState.SetString(notificationKey, notificationData);
+
+            // Add to the key list
+            var existingKeys = SessionState.GetString(PendingNotificationKeysKey, string.Empty);
+            var keyList = string.IsNullOrEmpty(existingKeys)
+                ? new List<string>()
+                : existingKeys.Split(',').Where(k => !string.IsNullOrEmpty(k)).ToList();
+
+            if (!keyList.Contains(notificationKey))
+            {
+                keyList.Add(notificationKey);
+                SessionState.SetString(PendingNotificationKeysKey, string.Join(",", keyList));
+            }
+        }
+
+        /// <summary>
+        /// Called by InitializeOnLoad to process any pending notifications after domain reload.
+        /// </summary>
+        public static void ProcessPendingNotifications()
+        {
+            // Get the list of pending notification keys from SessionState
+            var pendingKeys = SessionState.GetString(PendingNotificationKeysKey, string.Empty);
+            if (string.IsNullOrEmpty(pendingKeys))
+                return;
+
+            var keys = pendingKeys.Split(',').Where(k => !string.IsNullOrEmpty(k)).ToList();
+            if (keys.Count == 0)
+                return;
+
+            var processedKeys = new List<string>();
+
+            foreach (var key in keys)
+            {
+                var notificationData = SessionState.GetString(key, string.Empty);
+                if (string.IsNullOrEmpty(notificationData))
+                {
+                    processedKeys.Add(key);
+                    continue;
+                }
+
+                var parts = notificationData.Split(NotificationDataSeparator);
+                if (parts.Length != 4)
+                {
+                    processedKeys.Add(key);
+                    continue;
+                }
+
+                var requestId = parts[0];
+                var packageIdentifier = parts[1];
+                var operationType = parts[2];
+                var expectedResult = bool.Parse(parts[3]);
+
+                // Check for compilation errors after package operation
+                var hasCompilationErrors = ScriptUtils.HasCompilationErrors();
+
+                ResponseCallTool response;
+                if (hasCompilationErrors)
+                {
+                    var errorDetails = ScriptUtils.GetCompilationErrorDetails();
+                    var message = $"[Warning] Package {operationType} completed: {packageIdentifier}, but compilation errors occurred after domain reload. Details:\n{errorDetails}";
+                    response = ResponseCallTool.Success(message).SetRequestID(requestId);
+                }
+                else if (expectedResult)
+                {
+                    var message = $"[Success] Package {operationType} completed: {packageIdentifier}. Domain reload finished successfully.";
+                    response = ResponseCallTool.Success(message).SetRequestID(requestId);
+                }
+                else
+                {
+                    var message = $"[Error] Package {operationType} failed: {packageIdentifier}";
+                    response = ResponseCallTool.Error(message).SetRequestID(requestId);
+                }
+
+                // Send notification and mark for cleanup
+                _ = UnityCopilotPluginEditor.NotifyToolRequestCompleted(new RequestToolCompletedData
+                {
+                    RequestId = requestId,
+                    Result = response
+                });
+                processedKeys.Add(key);
+            }
+
+            // Clean up processed keys
+            foreach (var key in processedKeys)
+                SessionState.EraseString(key);
+
+            // Update the key list
+            var remainingKeys = keys.Except(processedKeys).ToList();
+            if (remainingKeys.Count > 0)
+            {
+                SessionState.SetString(PendingNotificationKeysKey, string.Join(",", remainingKeys));
+            }
+            else
+            {
+                SessionState.EraseString(PendingNotificationKeysKey);
+            }
+        }
+    }
+}