@atelierai/uco 1.0.7 → 1.0.8

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

@@ -1,65 +1,65 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  Author: AtelierAI                                                 │
-│  Copyright (c) 2025 AtelierAI                                     │
-│  Licensed under the MIT License.                                  │
-└──────────────────────────────────────────────────────────────────┘
-*/
-#nullable enable
-
-using System.Linq;
-using com.AtelierAI.Uco.Framework;
-using UnityEditor;
-
-namespace com.AtelierAI.Unity.Copilot.Editor.Utils
-{
-    /// <summary>
-    /// Invalidates every in-memory confirmation plan before a domain reload
-    /// or Editor quit. The plan store is process-local by design; scripts,
-    /// objects, and policy code may change across a reload, so a token that
-    /// was issued by the previous AppDomain must not authorise an execution
-    /// afterwards. It is deliberately not a durable operation registry hook.
-    /// </summary>
-    [InitializeOnLoad]
-    public static class UnityAuthoringSafetyReloadGuard
-    {
-        static UnityAuthoringSafetyReloadGuard()
-        {
-            AssemblyReloadEvents.beforeAssemblyReload += InvalidateConfirmationPlansHandler;
-            EditorApplication.quitting += InvalidateConfirmationPlansHandler;
-        }
-
-        /// <summary>
-        /// Forgets every issued plan on the live pipeline. Returns the number
-        /// of policy stores that were invalidated (tests assert on it).
-        /// </summary>
-        public static int InvalidateConfirmationPlans()
-        {
-            if (!UnityCopilotPluginEditor.HasInstance || !UnityCopilotPluginEditor.Instance.HasMcpPluginInstance)
-                return 0;
-
-            var invalidated = 0;
-            foreach (var pipeline in Pipelines())
-            {
-                foreach (var middleware in pipeline.Middleware.OfType<AuthoringSafetyMiddleware>())
-                {
-                    middleware.Policy.ConfirmationPlans.Invalidate();
-                    invalidated++;
-                }
-            }
-            return invalidated;
-        }
-
-        private static System.Collections.Generic.IEnumerable<ToolExecutionPipeline> Pipelines()
-        {
-            var plugin = UnityCopilotPluginEditor.Instance;
-            if (plugin.Tools is UcoToolManager tools)
-                yield return tools.ExecutionPipeline;
-            if (plugin.UcoPluginInstance?.UcoManager.SystemToolManager is UcoSystemToolManager systemTools
-                && !(plugin.Tools is UcoToolManager regular && ReferenceEquals(regular.ExecutionPipeline, systemTools.ExecutionPipeline)))
-                yield return systemTools.ExecutionPipeline;
-        }
-
-        private static void InvalidateConfirmationPlansHandler() => InvalidateConfirmationPlans();
-    }
-}
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  Author: AtelierAI                                                 │
+│  Copyright (c) 2025 AtelierAI                                     │
+│  Licensed under the MIT License.                                  │
+└──────────────────────────────────────────────────────────────────┘
+*/
+#nullable enable
+
+using System.Linq;
+using com.AtelierAI.Uco.Framework;
+using UnityEditor;
+
+namespace com.AtelierAI.Unity.Copilot.Editor.Utils
+{
+    /// <summary>
+    /// Invalidates every in-memory confirmation plan before a domain reload
+    /// or Editor quit. The plan store is process-local by design; scripts,
+    /// objects, and policy code may change across a reload, so a token that
+    /// was issued by the previous AppDomain must not authorise an execution
+    /// afterwards. It is deliberately not a durable operation registry hook.
+    /// </summary>
+    [InitializeOnLoad]
+    public static class UnityAuthoringSafetyReloadGuard
+    {
+        static UnityAuthoringSafetyReloadGuard()
+        {
+            AssemblyReloadEvents.beforeAssemblyReload += InvalidateConfirmationPlansHandler;
+            EditorApplication.quitting += InvalidateConfirmationPlansHandler;
+        }
+
+        /// <summary>
+        /// Forgets every issued plan on the live pipeline. Returns the number
+        /// of policy stores that were invalidated (tests assert on it).
+        /// </summary>
+        public static int InvalidateConfirmationPlans()
+        {
+            if (!UnityCopilotPluginEditor.HasInstance || !UnityCopilotPluginEditor.Instance.HasUcoPluginInstance)
+                return 0;
+
+            var invalidated = 0;
+            foreach (var pipeline in Pipelines())
+            {
+                foreach (var middleware in pipeline.Middleware.OfType<AuthoringSafetyMiddleware>())
+                {
+                    middleware.Policy.ConfirmationPlans.Invalidate();
+                    invalidated++;
+                }
+            }
+            return invalidated;
+        }
+
+        private static System.Collections.Generic.IEnumerable<ToolExecutionPipeline> Pipelines()
+        {
+            var plugin = UnityCopilotPluginEditor.Instance;
+            if (plugin.Tools is UcoToolManager tools)
+                yield return tools.ExecutionPipeline;
+            if (plugin.UcoPluginInstance?.UcoManager.SystemToolManager is UcoSystemToolManager systemTools
+                && !(plugin.Tools is UcoToolManager regular && ReferenceEquals(regular.ExecutionPipeline, systemTools.ExecutionPipeline)))
+                yield return systemTools.ExecutionPipeline;
+        }
+
+        private static void InvalidateConfirmationPlansHandler() => InvalidateConfirmationPlans();
+    }
+}

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

@@ -1,66 +1,66 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  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 UnityEditor;
-using UnityEngine;
-
-namespace com.AtelierAI.Unity.Copilot.Editor.Utils
-{
-    /// <summary>
-    /// Team-shared settings for the Unity-MCP update checker, persisted under
-    /// <c>ProjectSettings/Copilot-UpdateSettings.asset</c>.
-    /// </summary>
-    /// <remarks>
-    /// Uses Unity's <see cref="ScriptableSingleton{T}"/> + <see cref="FilePathAttribute"/> pattern
-    /// for project-scoped editor settings. The asset file is plain YAML, diff-friendly, and
-    /// lives in the folder teams commit to VCS — so flipping <see cref="DisableUpdateNotificationsForTeam"/>
-    /// once and committing the resulting asset disables the update popup for every team member
-    /// who pulls that commit.
-    ///
-    /// This complements (does not replace) the per-user "Do not show again" flag stored via
-    /// <see cref="EditorPrefs"/>. Precedence is enforced in
-    /// <see cref="UpdateChecker.ShouldCheckForUpdates"/>: the project flag short-circuits
-    /// the check before the per-user flag is consulted.
-    ///
-    /// See <see href="https://github.com/IvanMurzak/uco-plugin/issues/768"/>.
-    /// </remarks>
-    /// <remarks>
-    /// <see cref="FilePathAttribute.Location.ProjectFolder"/> resolves the relative path against
-    /// the Unity project root (the folder that contains <c>Assets/</c>, <c>Packages/</c>, and
-    /// <c>ProjectSettings/</c>) — NOT against <c>Assets/</c>. That places the asset alongside
-    /// other team-shared editor settings such as <c>ProjectSettings/EditorSettings.asset</c>.
-    /// </remarks>
-    [FilePath("ProjectSettings/Copilot-UpdateSettings.asset", FilePathAttribute.Location.ProjectFolder)]
-    internal sealed class UnityCopilotUpdateProjectSettings : ScriptableSingleton<UnityCopilotUpdateProjectSettings>
-    {
-        [SerializeField] private bool disableUpdateNotificationsForTeam;
-
-        /// <summary>
-        /// When <c>true</c>, the update popup is suppressed for every team member who has this
-        /// asset in their checkout — regardless of any per-user <see cref="EditorPrefs"/> state.
-        /// </summary>
-        /// <remarks>
-        /// Setter persists immediately via <see cref="ScriptableSingleton{T}.Save"/> with
-        /// <c>saveAsText: true</c> so the resulting asset is YAML (diff-friendly).
-        /// </remarks>
-        public bool DisableUpdateNotificationsForTeam
-        {
-            get => disableUpdateNotificationsForTeam;
-            set
-            {
-                if (disableUpdateNotificationsForTeam == value)
-                    return;
-                disableUpdateNotificationsForTeam = value;
-                Save(true);
-            }
-        }
-    }
-}
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  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 UnityEditor;
+using UnityEngine;
+
+namespace com.AtelierAI.Unity.Copilot.Editor.Utils
+{
+    /// <summary>
+    /// Team-shared settings for the Unity-Uco update checker, persisted under
+    /// <c>ProjectSettings/Copilot-UpdateSettings.asset</c>.
+    /// </summary>
+    /// <remarks>
+    /// Uses Unity's <see cref="ScriptableSingleton{T}"/> + <see cref="FilePathAttribute"/> pattern
+    /// for project-scoped editor settings. The asset file is plain YAML, diff-friendly, and
+    /// lives in the folder teams commit to VCS — so flipping <see cref="DisableUpdateNotificationsForTeam"/>
+    /// once and committing the resulting asset disables the update popup for every team member
+    /// who pulls that commit.
+    ///
+    /// This complements (does not replace) the per-user "Do not show again" flag stored via
+    /// <see cref="EditorPrefs"/>. Precedence is enforced in
+    /// <see cref="UpdateChecker.ShouldCheckForUpdates"/>: the project flag short-circuits
+    /// the check before the per-user flag is consulted.
+    ///
+    /// See <see href="https://github.com/IvanMurzak/uco-plugin/issues/768"/>.
+    /// </remarks>
+    /// <remarks>
+    /// <see cref="FilePathAttribute.Location.ProjectFolder"/> resolves the relative path against
+    /// the Unity project root (the folder that contains <c>Assets/</c>, <c>Packages/</c>, and
+    /// <c>ProjectSettings/</c>) — NOT against <c>Assets/</c>. That places the asset alongside
+    /// other team-shared editor settings such as <c>ProjectSettings/EditorSettings.asset</c>.
+    /// </remarks>
+    [FilePath("ProjectSettings/Copilot-UpdateSettings.asset", FilePathAttribute.Location.ProjectFolder)]
+    internal sealed class UnityCopilotUpdateProjectSettings : ScriptableSingleton<UnityCopilotUpdateProjectSettings>
+    {
+        [SerializeField] private bool disableUpdateNotificationsForTeam;
+
+        /// <summary>
+        /// When <c>true</c>, the update popup is suppressed for every team member who has this
+        /// asset in their checkout — regardless of any per-user <see cref="EditorPrefs"/> state.
+        /// </summary>
+        /// <remarks>
+        /// Setter persists immediately via <see cref="ScriptableSingleton{T}.Save"/> with
+        /// <c>saveAsText: true</c> so the resulting asset is YAML (diff-friendly).
+        /// </remarks>
+        public bool DisableUpdateNotificationsForTeam
+        {
+            get => disableUpdateNotificationsForTeam;
+            set
+            {
+                if (disableUpdateNotificationsForTeam == value)
+                    return;
+                disableUpdateNotificationsForTeam = value;
+                Save(true);
+            }
+        }
+    }
+}

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/Converter/Json/Structures/EntityIdConverter.cs

@@ -1,120 +1,120 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  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.  │
-└──────────────────────────────────────────────────────────────────┘
-*/
-
-// Wire contract for EntityId (Unity 6.5+):
-//   On the wire: JSON string of decimal digits, e.g. "568105584918935294".
-//                No sign, no leading zeros, no exponent.
-//   Schema:      { "type": "string", "pattern": "^[0-9]+$" }.
-//
-//   Outbound: always emitted as a JSON string (via WriteStringValue).
-//   Inbound:  accepts both a JSON string (preferred) and a JSON number
-//             (back-compat — legacy clients pre-#759 wrote a number).
-//
-// Rationale: Unity 6.5's EntityId is a 64-bit ulong. JS-based MCP clients
-// (Claude Agent SDK, etc.) parse JSON numbers as IEEE-754 doubles, so any
-// value past 2^53 - 1 rounds. Serializing as a string makes the value
-// opaque to every JSON parser, preserving full precision through any
-// language boundary. This is the same pattern used by Twitter, YouTube,
-// Discord, Stripe, and gRPC-JSON for 64-bit IDs. See #759 / #754.
-
-#nullable enable
-#if UNITY_6000_5_OR_NEWER
-using System;
-using System.Globalization;
-using System.Text.Json;
-using System.Text.Json.Nodes;
-using com.IvanMurzak.ReflectorNet.Json;
-using com.IvanMurzak.ReflectorNet.Utils;
-using com.AtelierAI.Unity.Copilot.Runtime.Utils;
-using UnityEngine;
-
-namespace com.AtelierAI.Unity.Copilot.JsonConverters
-{
-    public class EntityIdConverter : JsonSchemaConverter<EntityId>, IJsonSchemaConverter
-    {
-        public override JsonNode GetSchema() => new JsonObject
-        {
-            [JsonSchema.Type] = JsonSchema.String,
-            [JsonSchema.Pattern] = "^[0-9]+$"
-        };
-        public override JsonNode GetSchemaRef() => new JsonObject
-        {
-            [JsonSchema.Ref] = JsonSchema.RefValue + Id
-        };
-
-        public override EntityId Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
-        {
-            if (reader.TokenType == JsonTokenType.Null)
-                return EntityId.None;
-
-            if (reader.TokenType != JsonTokenType.String && reader.TokenType != JsonTokenType.Number)
-                throw new JsonException($"Expected string or number token for {nameof(EntityId)}, got {reader.TokenType}.");
-
-            return ReadEntityIdValue(ref reader);
-        }
-
-        // Accept three JSON representations:
-        //   1. String token — the new on-wire format (decimal digits, see top-of-file).
-        //   2. Legacy int form from EntityId.ToString (handwritten JSON).
-        //   3. Raw ulong from EntityId.ToULong (machine-serialized JSON).
-        // All three round-trip correctly.
-        internal static EntityId ReadEntityIdValue(ref Utf8JsonReader reader)
-        {
-            // String token — the new on-wire format.
-            if (reader.TokenType == JsonTokenType.String)
-            {
-                var stringValue = reader.GetString();
-                // Reject empty string up-front: the schema regex "^[0-9]+$" requires
-                // one or more digits, but EntityIdUtils.FromString("") returns
-                // EntityId.None silently. Throw here so the runtime matches the
-                // published schema (see #759 wire contract at top-of-file).
-                if (string.IsNullOrEmpty(stringValue))
-                    throw new JsonException($"{nameof(EntityId)} string must be one or more decimal digits, got empty string.");
-                return EntityIdUtils.FromString(stringValue);
-            }
-
-            if (reader.TryGetInt64(out var signedValue))
-                return EntityIdUtils.FromNumber(signedValue);
-
-            if (reader.TryGetUInt64(out var unsignedValue))
-                return EntityIdUtils.FromRawValue(unsignedValue);
-
-            // Fallback for numeric tokens STJ 8's TryGet* helpers reject
-            // (e.g. numbers re-serialized through JsonElement that end up with
-            // a fractional trailing zero). Parse the raw token text ourselves.
-            byte[] rawBytes;
-            if (reader.HasValueSequence)
-            {
-                var seq = reader.ValueSequence;
-                rawBytes = new byte[seq.Length];
-                var offset = 0;
-                foreach (var mem in seq)
-                {
-                    mem.Span.CopyTo(new Span<byte>(rawBytes, offset, mem.Length));
-                    offset += mem.Length;
-                }
-            }
-            else
-            {
-                rawBytes = reader.ValueSpan.ToArray();
-            }
-            return EntityIdUtils.FromString(System.Text.Encoding.UTF8.GetString(rawBytes));
-        }
-
-        public override void Write(Utf8JsonWriter writer, EntityId value, JsonSerializerOptions options)
-        {
-            // Outbound: always a JSON string of decimal digits so JS clients
-            // can JSON.parse the payload without IEEE-754 precision loss
-            // (see top-of-file wire contract).
-            writer.WriteStringValue(EntityId.ToULong(value).ToString(CultureInfo.InvariantCulture));
-        }
-    }
-}
-#endif
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  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.  │
+└──────────────────────────────────────────────────────────────────┘
+*/
+
+// Wire contract for EntityId (Unity 6.5+):
+//   On the wire: JSON string of decimal digits, e.g. "568105584918935294".
+//                No sign, no leading zeros, no exponent.
+//   Schema:      { "type": "string", "pattern": "^[0-9]+$" }.
+//
+//   Outbound: always emitted as a JSON string (via WriteStringValue).
+//   Inbound:  accepts both a JSON string (preferred) and a JSON number
+//             (back-compat — legacy clients pre-#759 wrote a number).
+//
+// Rationale: Unity 6.5's EntityId is a 64-bit ulong. JS-based Uco clients
+// (Claude Agent SDK, etc.) parse JSON numbers as IEEE-754 doubles, so any
+// value past 2^53 - 1 rounds. Serializing as a string makes the value
+// opaque to every JSON parser, preserving full precision through any
+// language boundary. This is the same pattern used by Twitter, YouTube,
+// Discord, Stripe, and gRPC-JSON for 64-bit IDs. See #759 / #754.
+
+#nullable enable
+#if UNITY_6000_5_OR_NEWER
+using System;
+using System.Globalization;
+using System.Text.Json;
+using System.Text.Json.Nodes;
+using com.IvanMurzak.ReflectorNet.Json;
+using com.IvanMurzak.ReflectorNet.Utils;
+using com.AtelierAI.Unity.Copilot.Runtime.Utils;
+using UnityEngine;
+
+namespace com.AtelierAI.Unity.Copilot.JsonConverters
+{
+    public class EntityIdConverter : JsonSchemaConverter<EntityId>, IJsonSchemaConverter
+    {
+        public override JsonNode GetSchema() => new JsonObject
+        {
+            [JsonSchema.Type] = JsonSchema.String,
+            [JsonSchema.Pattern] = "^[0-9]+$"
+        };
+        public override JsonNode GetSchemaRef() => new JsonObject
+        {
+            [JsonSchema.Ref] = JsonSchema.RefValue + Id
+        };
+
+        public override EntityId Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
+        {
+            if (reader.TokenType == JsonTokenType.Null)
+                return EntityId.None;
+
+            if (reader.TokenType != JsonTokenType.String && reader.TokenType != JsonTokenType.Number)
+                throw new JsonException($"Expected string or number token for {nameof(EntityId)}, got {reader.TokenType}.");
+
+            return ReadEntityIdValue(ref reader);
+        }
+
+        // Accept three JSON representations:
+        //   1. String token — the new on-wire format (decimal digits, see top-of-file).
+        //   2. Legacy int form from EntityId.ToString (handwritten JSON).
+        //   3. Raw ulong from EntityId.ToULong (machine-serialized JSON).
+        // All three round-trip correctly.
+        internal static EntityId ReadEntityIdValue(ref Utf8JsonReader reader)
+        {
+            // String token — the new on-wire format.
+            if (reader.TokenType == JsonTokenType.String)
+            {
+                var stringValue = reader.GetString();
+                // Reject empty string up-front: the schema regex "^[0-9]+$" requires
+                // one or more digits, but EntityIdUtils.FromString("") returns
+                // EntityId.None silently. Throw here so the runtime matches the
+                // published schema (see #759 wire contract at top-of-file).
+                if (string.IsNullOrEmpty(stringValue))
+                    throw new JsonException($"{nameof(EntityId)} string must be one or more decimal digits, got empty string.");
+                return EntityIdUtils.FromString(stringValue);
+            }
+
+            if (reader.TryGetInt64(out var signedValue))
+                return EntityIdUtils.FromNumber(signedValue);
+
+            if (reader.TryGetUInt64(out var unsignedValue))
+                return EntityIdUtils.FromRawValue(unsignedValue);
+
+            // Fallback for numeric tokens STJ 8's TryGet* helpers reject
+            // (e.g. numbers re-serialized through JsonElement that end up with
+            // a fractional trailing zero). Parse the raw token text ourselves.
+            byte[] rawBytes;
+            if (reader.HasValueSequence)
+            {
+                var seq = reader.ValueSequence;
+                rawBytes = new byte[seq.Length];
+                var offset = 0;
+                foreach (var mem in seq)
+                {
+                    mem.Span.CopyTo(new Span<byte>(rawBytes, offset, mem.Length));
+                    offset += mem.Length;
+                }
+            }
+            else
+            {
+                rawBytes = reader.ValueSpan.ToArray();
+            }
+            return EntityIdUtils.FromString(System.Text.Encoding.UTF8.GetString(rawBytes));
+        }
+
+        public override void Write(Utf8JsonWriter writer, EntityId value, JsonSerializerOptions options)
+        {
+            // Outbound: always a JSON string of decimal digits so JS clients
+            // can JSON.parse the payload without IEEE-754 precision loss
+            // (see top-of-file wire contract).
+            writer.WriteStringValue(EntityId.ToULong(value).ToString(CultureInfo.InvariantCulture));
+        }
+    }
+}
+#endif