@atelierai/uco 1.0.2 → 1.0.4

package/vendor/plugin/com.atelierai.unity.copilot/Editor/Scripts/UI/Window/MainWindowEditor.cs

@@ -23,11 +23,7 @@ namespace com.AtelierAI.Unity.Copilot.Editor.UI
         readonly CompositeDisposable _disposables = new();
 
         Button? _btnConnect;
-        Button? _btnAuthorize;
-        Action? _startAuthorizeAction;
         VisualElement? _timelinePointUnity;
-        AlertPanel? _connectionAuthAlert;
-        AlertPanel? _connectionConnectAlert;
 
         protected override string WindowTitle => "Unity Copilot";
         protected override string[] WindowUxmlPaths => _windowUxmlPaths;
@@ -73,76 +69,11 @@ namespace com.AtelierAI.Unity.Copilot.Editor.UI
             _authRejectedSubscription.Dispose();
         }
 
-        internal static (bool needsAuth, bool hasToken, bool isCloud) ComputeCloudAuthState(ConnectionMode mode, string? token)
-        {
-            var isCloud = mode == ConnectionMode.Cloud;
-            var hasToken = !string.IsNullOrEmpty(token);
-            var needsAuth = isCloud && !hasToken;
-            return (needsAuth, hasToken, isCloud);
-        }
-
-        private void UpdateCloudAuthState()
-        {
-            var (needsAuth, hasToken, isCloud) = ComputeCloudAuthState(UnityCopilotPluginEditor.ConnectionMode, UnityCopilotPluginEditor.CloudToken);
-
-            if (_timelinePointUnity != null)
-            {
-                _timelinePointUnity.SetEnabled(!needsAuth);
-                _timelinePointUnity.tooltip = needsAuth
-                    ? "Cloud token is required. Press the Authorize button to authenticate."
-                    : "";
-            }
-            if (_btnConnect != null)
-            {
-                if (needsAuth)
-                {
-                    _btnConnect.text = ServerButtonText_Connect;
-                    _btnConnect.EnableInClassList("btn-primary", false);
-                    _btnConnect.EnableInClassList("btn-secondary", true);
-                }
-                else if (isCloud && hasToken
-                    && _btnConnect.text == ServerButtonText_Connect)
-                {
-                    _btnConnect.EnableInClassList("btn-primary", true);
-                    _btnConnect.EnableInClassList("btn-secondary", false);
-                }
-            }
-            if (_btnAuthorize != null)
-            {
-                _btnAuthorize.EnableInClassList("btn-primary", !hasToken);
-            }
-            _connectionAuthAlert?.SetVisible(needsAuth);
-
-            // Show connect alert when authorized in Cloud mode but not connected and not trying
-            if (_connectionConnectAlert != null)
-            {
-                var connectionState = UnityCopilotPluginEditor.ConnectionState.CurrentValue;
-                var keepConnected = UnityCopilotPluginEditor.KeepConnected;
-                var isDisconnected = connectionState == ConnectionState.Disconnected;
-                var needsConnect = isCloud && hasToken && isDisconnected && !keepConnected;
-                _connectionConnectAlert.SetVisible(needsConnect);
-            }
-        }
-
         private static void UnityBuildAndConnect()
         {
             UnityCopilotPluginEditor.Instance.BuildMcpPluginIfNeeded();
             UnityCopilotPluginEditor.Instance.AddUnityLogCollectorIfNeeded(() => new BufferedFileLogStorage());
             UnityCopilotPluginEditor.ConnectIfNeeded();
         }
-
-        /// <summary>
-        /// Disconnects, disposes the current MCP plugin, rebuilds it (picking up the new Host/Token
-        /// from the changed ConnectionMode), and reconnects if KeepConnected is enabled.
-        /// Called when switching between Local and Cloud modes.
-        /// </summary>
-        private static void ReconnectAfterModeSwitch()
-        {
-            if (UnityCopilotPluginEditor.Instance.HasMcpPluginInstance)
-            {
-                UnityCopilotPluginEditor.Instance.DisposeMcpPluginInstance();
-            }
-            UnityBuildAndConnect();
-        }
     }
 }

package/vendor/plugin/com.atelierai.unity.copilot/Editor/Scripts/UI/Window/UpdatePopupWindow.cs

@@ -11,6 +11,7 @@
 #nullable enable
 
 using System;
+using System.IO;
 using com.AtelierAI.Unity.Copilot.Editor.DependencyResolver;
 using com.AtelierAI.Unity.Copilot.Editor.Utils;
 using R3;
@@ -170,6 +171,20 @@ namespace com.AtelierAI.Unity.Copilot.Editor.UI
             if (installButton != null)
                 installButton.text = "Installing...";
 
+            // Embedded packages (installed by `uco install` as a physical copy into
+            // Packages/) cannot be updated through the registry — Unity refuses
+            // Client.Add on an embedded id. Detect this and direct the user to
+            // the CLI instead of failing with an opaque UPM error.
+            if (IsEmbeddedInstall())
+            {
+                if (installButton != null)
+                    installButton.text = "Use: uco install";
+                Debug.Log("[Unity Copilot] This project uses an embedded install (uco install). " +
+                    "Run `uco install <project-path>` from a terminal to upgrade — the in-editor " +
+                    "updater cannot replace embedded packages.");
+                return;
+            }
+
             // Force a clean post-install recompile so an unrelated user-asmdef
             // error can't leave the OLD plugin AppDomain loaded with the new
             // package files on disk.
@@ -179,6 +194,21 @@ namespace com.AtelierAI.Unity.Copilot.Editor.UI
             EditorApplication.update += OnPackageInstallProgress;
         }
 
+        /// <summary>
+        /// True when this package is physically embedded in the project's
+        /// Packages folder (installed via `uco install`), as opposed to
+        /// resolved from a registry or git URL. Embedded packages cannot be
+        /// updated through Unity's package manager.
+        /// </summary>
+        private static bool IsEmbeddedInstall()
+        {
+            return Directory.Exists(
+                Path.Combine(ProjectRootPath, "Packages", "com.atelierai.unity.copilot"));
+        }
+
+        private static string ProjectRootPath =>
+            Path.GetDirectoryName(Application.dataPath);
+
         private void OnPackageInstallProgress()
         {
             if (addRequest == null)

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

@@ -138,7 +138,12 @@ namespace com.AtelierAI.Unity.Copilot
                 {
                     config = string.IsNullOrWhiteSpace(json)
                         ? null
-                        : JsonSerializer.Deserialize<UnityConnectionConfig>(json!, new JsonSerializerOptions
+                        : JsonSerializer.Deserialize<UnityConnectionConfig>(
+                            // Legacy 1.0.2-and-earlier configs may carry the removed Cloud
+                            // connection mode; JsonStringEnumConverter would throw on it.
+                            json!.Replace("\"connectionMode\": \"Cloud\"", "\"connectionMode\": \"Custom\"")
+                                  .Replace("\"ConnectionMode\": \"Cloud\"", "\"ConnectionMode\": \"Custom\""),
+                            new JsonSerializerOptions
                         {
                             PropertyNameCaseInsensitive = true,
                             Converters = { new JsonStringEnumConverter() }

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

@@ -101,6 +101,15 @@ namespace com.AtelierAI.Unity.Copilot
                 NotifyChanged(Instance.unityConnectionConfig);
             }
         }
+        public static string? LocalToken
+        {
+            get => Instance.unityConnectionConfig.LocalToken;
+            set
+            {
+                Instance.unityConnectionConfig.LocalToken = value;
+                NotifyChanged(Instance.unityConnectionConfig);
+            }
+        }
         public static string LocalHost
         {
             get => Instance.unityConnectionConfig.LocalHost;
@@ -229,18 +238,6 @@ namespace com.AtelierAI.Unity.Copilot
                 NotifyChanged(Instance.unityConnectionConfig);
             }
         }
-        public static string CloudServerUrl => UnityCopilotPlugin.UnityConnectionConfig.CloudServerUrl;
-        public static string? CloudToken
-        {
-            get => Instance.unityConnectionConfig.CloudToken;
-            set
-            {
-                Instance.unityConnectionConfig.CloudToken = value;
-                NotifyChanged(Instance.unityConnectionConfig);
-            }
-        }
-
-
 
         // 'new' is intentional: static dispatch on the subtype, instance logic lives in the base.
         public static new ReadOnlyReactiveProperty<WsState> ConnectionState

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/UnityCopilotPlugin.Config.cs

@@ -35,87 +35,36 @@ namespace com.AtelierAI.Unity.Copilot
             public static List<CopilotFeature> DefaultResources => new();
 
             /// <summary>
-            /// Backing field for the local server URL. Serialized as "host" in JSON.
-            /// Use <see cref="Host"/> for the active connection URL (routes through Cloud mode).
+            /// The local server URL. Serialized as "host" in JSON.
             /// </summary>
             [JsonPropertyName("host")]
             public string LocalHost { get; set; } = DefaultHost;
 
             /// <summary>
-            /// Backing field for the local auth token. Serialized as "token" in JSON.
-            /// Use <see cref="Token"/> for the active token (routes through Cloud mode).
+            /// The local auth token. Serialized as "token" in JSON.
             /// </summary>
             [JsonPropertyName("token")]
             public string? LocalToken { get; set; }
 
-            public const string DefaultCloudServerBaseUrl = "https://ai-game.dev";
-
-            public static string CloudServerBaseUrl
-            {
-                get
-                {
-                    var args = ArgsUtils.ParseCommandLineArguments();
-                    var envValue = args.GetValueOrDefault(EnvironmentUtils.EnvCloudUrl)
-                        ?? Environment.GetEnvironmentVariable(EnvironmentUtils.EnvCloudUrl);
-
-                    if (string.IsNullOrWhiteSpace(envValue))
-                        return DefaultCloudServerBaseUrl;
-
-                    var normalized = envValue.Trim().Trim('"').TrimEnd('/');
-
-                    if (!Uri.TryCreate(normalized, UriKind.Absolute, out var uri) ||
-                        (uri.Scheme != Uri.UriSchemeHttp && uri.Scheme != Uri.UriSchemeHttps))
-                    {
-                        return DefaultCloudServerBaseUrl;
-                    }
-
-                    // Strip trailing "/mcp" so CloudServerUrl doesn't produce "/mcp/mcp"
-                    if (normalized.EndsWith("/mcp", StringComparison.OrdinalIgnoreCase))
-                        normalized = normalized[..^4];
-
-                    return normalized;
-                }
-            }
-
-            public static string CloudServerUrl => CloudServerBaseUrl + "/mcp";
-
-            /// <summary>
-            /// Returns the active connection host based on <see cref="ConnectionMode"/>.
-            /// In Cloud mode, returns <see cref="CloudServerUrl"/>.
-            /// In Local mode, returns <see cref="LocalHost"/>.
-            /// </summary>
             [JsonIgnore]
             public override string Host
             {
-                get => ConnectionMode == ConnectionMode.Cloud ? CloudServerUrl : LocalHost;
+                get => LocalHost;
                 set => LocalHost = value;
             }
 
-            /// <summary>
-            /// Gets/sets the active auth token based on <see cref="ConnectionMode"/>.
-            /// In Cloud mode, routes to <see cref="CloudToken"/>.
-            /// In Local mode, routes to <see cref="LocalToken"/>.
-            /// Setter mirrors the getter so env-var / CLI overrides (which write via
-            /// the generic Token property) land on the right field regardless of mode.
-            /// </summary>
             [JsonIgnore]
             public override string? Token
             {
-                get => ConnectionMode == ConnectionMode.Cloud ? CloudToken : LocalToken;
-                set
-                {
-                    if (ConnectionMode == ConnectionMode.Cloud)
-                        CloudToken = value;
-                    else
-                        LocalToken = value;
-                }
+                get => LocalToken;
+                set => LocalToken = value;
             }
 
             public LogLevel LogLevel { get; set; } = LogLevel.Warning;
             public bool KeepServerRunning { get; set; } = false;
             public TransportMethod TransportMethod { get; set; } = TransportMethod.streamableHttp;
             public AuthOption AuthOption { get; set; } = AuthOption.none;
-            public ConnectionMode ConnectionMode { get; set; } = ConnectionMode.Cloud;
+            public ConnectionMode ConnectionMode { get; set; } = ConnectionMode.Custom;
 
             /// <summary>
             /// Optional path to the Node.js MCP server entry script (cocli's bin/server.mjs).
@@ -165,7 +114,6 @@ namespace com.AtelierAI.Unity.Copilot
             /// <c>auth=none</c> (not recommended).
             /// </summary>
             public bool ForceTokenWhenLanBind { get; set; } = true;
-            public string? CloudToken { get; set; }
             public List<CopilotFeature> Tools { get; set; } = new();
             public List<CopilotFeature> Prompts { get; set; } = new();
             public List<CopilotFeature> Resources { get; set; } = new();
@@ -191,9 +139,8 @@ namespace com.AtelierAI.Unity.Copilot
                 KeepServerRunning = !isCi;
                 TransportMethod = TransportMethod.streamableHttp;
                 AuthOption = AuthOption.none;
-                ConnectionMode = ConnectionMode.Cloud;
+                ConnectionMode = ConnectionMode.Custom;
                 NodeServerPath = null;
-                CloudToken = null;
                 LogLevel = LogLevel.Warning;
                 TimeoutMs = Consts.Hub.DefaultTimeoutMs;
                 Tools = DefaultTools;
@@ -218,9 +165,11 @@ namespace com.AtelierAI.Unity.Copilot
         }
     }
 
+    /// <summary>Local/custom server connection. The legacy Cloud mode (remote
+    /// ai-game.dev endpoint) was removed in 1.0.3; configs that still carry
+    /// "Cloud" are mapped to Custom on load.</summary>
     public enum ConnectionMode
     {
-        Custom,
-        Cloud
+        Custom
     }
 }

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/UnityCopilotPlugin.cs

@@ -26,7 +26,7 @@ namespace com.AtelierAI.Unity.Copilot
 
     public partial class UnityCopilotPlugin : IDisposable
     {
-        public const string Version = "1.0.2";
+        public const string Version = "1.0.4";
 
         private static int _singletonCount = 0;
         public static bool HasAnyInstance => _singletonCount > 0;

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/Utils/EnvironmentUtils.cs

@@ -47,7 +47,6 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
         public const string EnvTools = "UNITY_MCP_TOOLS";
         public const string EnvStartServer = "UNITY_MCP_START_SERVER";
         public const string EnvTransport = "UNITY_MCP_TRANSPORT";
-        public const string EnvCloudUrl = "UNITY_MCP_CLOUD_URL";
         public const string EnvConnectionMode = "UNITY_MCP_CONNECTION_MODE";
 
         // Short flag aliases recognised by the in-plugin command-line parser.
@@ -64,7 +63,6 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
         public const string FieldKeepConnected = nameof(UnityCopilotPlugin.UnityConnectionConfig.KeepConnected);
         public const string FieldAuthOption = nameof(UnityCopilotPlugin.UnityConnectionConfig.AuthOption);
         public const string FieldLocalToken = nameof(UnityCopilotPlugin.UnityConnectionConfig.LocalToken);
-        public const string FieldCloudToken = nameof(UnityCopilotPlugin.UnityConnectionConfig.CloudToken);
         public const string FieldTools = nameof(UnityCopilotPlugin.UnityConnectionConfig.EnabledToolsOverride);
         public const string FieldStartServer = nameof(UnityCopilotPlugin.UnityConnectionConfig.KeepServerRunning);
         public const string FieldTransport = nameof(UnityCopilotPlugin.UnityConnectionConfig.TransportMethod);
@@ -176,9 +174,9 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
 
             string Sanitize(string raw) => raw.Trim().Trim('"');
 
-            // UNITY_MCP_HOST is the legacy alias for UNITY_MCP_CLOUD_URL.
+            // UNITY_MCP_HOST / --url override the local server host.
             string? sanitizedHost = null;
-            var rawHost = Resolve(EnvCloudUrl, FlagUrl) ?? Resolve(EnvHost, flagAlias: null);
+            var rawHost = Resolve(EnvHost, FlagUrl);
             if (rawHost != null)
             {
                 var host = Sanitize(rawHost).TrimEnd('/');
@@ -231,8 +229,7 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
                 _logger.LogInformation("[MCP] Override: {Key}={Value}", EnvAuthOption, ao);
             }
 
-            // Resolved AFTER ConnectionMode so we route to the correct underlying field
-            // (LocalToken in Custom mode, CloudToken in Cloud mode). We track the specific
+            // We track the specific
             // backing field rather than the abstract Token property because only the backing
             // fields are serialised — restoring the baseline must target the same field that
             // was clobbered.
@@ -240,23 +237,11 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
             if (rawToken != null)
             {
                 var token = Sanitize(rawToken);
-                if (config.ConnectionMode == ConnectionMode.Cloud)
+                if (!string.Equals(token, config.LocalToken, StringComparison.Ordinal))
                 {
-                    if (!string.Equals(token, config.CloudToken, StringComparison.Ordinal))
-                    {
-                        record.Track(FieldCloudToken, config.CloudToken, token);
-                        config.CloudToken = token;
-                        _logger.LogInformation("[MCP] Override: {Key}=*** (CloudToken)", EnvToken);
-                    }
-                }
-                else
-                {
-                    if (!string.Equals(token, config.LocalToken, StringComparison.Ordinal))
-                    {
-                        record.Track(FieldLocalToken, config.LocalToken, token);
-                        config.LocalToken = token;
-                        _logger.LogInformation("[MCP] Override: {Key}=*** (LocalToken)", EnvToken);
-                    }
+                    record.Track(FieldLocalToken, config.LocalToken, token);
+                    config.LocalToken = token;
+                    _logger.LogInformation("[MCP] Override: {Key}=*** (LocalToken)", EnvToken);
                 }
             }
 
@@ -341,9 +326,6 @@ namespace com.AtelierAI.Unity.Copilot.Runtime.Utils
                     case FieldLocalToken:
                         config.LocalToken = (string?)kvp.Value;
                         break;
-                    case FieldCloudToken:
-                        config.CloudToken = (string?)kvp.Value;
-                        break;
                     case FieldTools:
                         config.EnabledToolsOverride = (List<string>?)kvp.Value;
                         break;