@atelierai/uco 1.0.8 → 1.0.10

package/vendor/plugin/com.atelierai.unity.copilot/README.md

@@ -1,88 +1,88 @@
-<div align="center">
-  <h1>Unity Copilot</h1>
-  <p>Drive the Unity Editor from any AI agent over plain REST + WebSocket — no MCP protocol on the wire.</p>
-  <p><code>com.atelierai.unity.copilot</code></p>
-</div>
-
-Unity Copilot is a Unity Editor/Runtime plugin that exposes the Editor as an
-HTTP service: 160+ tools covering scenes, GameObjects, components, assets,
-prefabs, scripts, packages, screenshots, console, tests, and builds — plus
-prompts and resources. A self-hosted Node bridge connects the Editor to the
-outside world; the **uco** CLI drives the whole surface from any agent or
-terminal.
-
-## Highlights
-
-- **160+ Editor tools** — authoring, code (including Roslyn compile-and-execute
-  with confirmation gating for risky operations), diagnostics, build & tests,
-  visuals, physics
-- **Three-surface AI skills** — `uco setup-skills` generates focused skill
-  bundles (setup / lifecycle / live-editor) for Claude Code, Codex, Cursor,
-  and other agents, refreshed from the live tool catalog
-- **Deterministic, safe connectivity** — one port per project (hashed from the
-  project path), bearer-token auth, localhost by default, no vendor lock-in:
-  anything that can speak HTTP or run a CLI can drive the Editor
-- **Durable calls** — async invocations return a call id queryable later
-  (`uco call get <id>`); structured error envelopes survive the whole stack
-- **Dual-editor support** — gates run against Unity 2022.3 LTS and 6000.5
-
-## Requirements
-
-- Unity **2022.3 or newer** (tested on 2022.3 LTS and 6000.5)
-- **Node.js 20+** for the bridge server (the plugin auto-discovers the `uco`
-  npm package — global or project-local — or set `nodeServerPath` in the
-  config to point at a specific install)
-
-## Install
-
-### Via the uco CLI (recommended)
-
-```bash
-npm install -g @atelierai/uco
-uco install <path-to-your-unity-project>
-```
-
-This embeds the plugin package, stages the framework DLLs, writes the initial
-config, and can generate agent skills and wrappers (`uco setup-skills`).
-
-### Via OpenUPM
-
-Add the scoped registry to `Packages/manifest.json` and reference the package:
-
-```json
-{
-  "dependencies": {
-    "com.atelierai.unity.copilot": "1.0.0"
-  },
-  "scopedRegistries": [
-    {
-      "name": "package.openupm.com",
-      "url": "https://package.openupm.com",
-      "scopes": [ "com.atelierai.unity.copilot" ]
-    }
-  ]
-}
-```
-
-## Quick start
-
-```bash
-uco ping                        # health-check the bridge
-uco call unity-tool-list        # list every tool the Editor exposes
-uco exec --method-body --return-type int --code 'return GameObject.FindObjectsOfType<Camera>().Length;'
-```
-
-Configuration lives in `UserSettings/uco-config.json` (migrated automatically
-from the legacy filename by older→newer versions of the plugin).
-
-## Wire protocol
-
-Plain HTTP + WebSocket with JSON envelopes (`/api/tools/...`). There is no MCP
-protocol on the wire — the bridge was deliberately stripped down to REST so
-any agent, script, or CI job can use it without an MCP SDK.
-
-## Attribution & license
-
-Derived from [IvanMurzak/Unity-MCP](https://github.com/IvanMurzak/Unity-MCP)
-(Apache-2.0). Licensed under the Apache License 2.0 — see [LICENSE](LICENSE)
-for the dual copyright notice.
+<div align="center">
+  <h1>Unity Copilot</h1>
+  <p>Drive the Unity Editor from any AI agent over plain REST + WebSocket — a single simple wire protocol on the wire.</p>
+  <p><code>com.atelierai.unity.copilot</code></p>
+</div>
+
+Unity Copilot is a Unity Editor/Runtime plugin that exposes the Editor as an
+HTTP service: 160+ tools covering scenes, GameObjects, components, assets,
+prefabs, scripts, packages, screenshots, console, tests, and builds — plus
+prompts and resources. A self-hosted Node bridge connects the Editor to the
+outside world; the **uco** CLI drives the whole surface from any agent or
+terminal.
+
+## Highlights
+
+- **160+ Editor tools** — authoring, code (including Roslyn compile-and-execute
+  with confirmation gating for risky operations), diagnostics, build & tests,
+  visuals, physics
+- **Three-surface AI skills** — `uco setup-skills` generates focused skill
+  bundles (setup / lifecycle / live-editor) for Claude Code, Codex, Cursor,
+  and other agents, refreshed from the live tool catalog
+- **Deterministic, safe connectivity** — one port per project (hashed from the
+  project path), bearer-token auth, localhost by default, no vendor lock-in:
+  anything that can speak HTTP or run a CLI can drive the Editor
+- **Durable calls** — async invocations return a call id queryable later
+  (`uco call get <id>`); structured error envelopes survive the whole stack
+- **Dual-editor support** — gates run against Unity 2022.3 LTS and 6000.5
+
+## Requirements
+
+- Unity **2022.3 or newer** (tested on 2022.3 LTS and 6000.5)
+- **Node.js 20+** for the bridge server (the plugin auto-discovers the `uco`
+  npm package — global or project-local — or set `nodeServerPath` in the
+  config to point at a specific install)
+
+## Install
+
+### Via the uco CLI (recommended)
+
+```bash
+npm install -g @atelierai/uco
+uco install <path-to-your-unity-project>
+```
+
+This embeds the plugin package, stages the framework DLLs, writes the initial
+config, and can generate agent skills and wrappers (`uco setup-skills`).
+
+### Via OpenUPM
+
+Add the scoped registry to `Packages/manifest.json` and reference the package:
+
+```json
+{
+  "dependencies": {
+    "com.atelierai.unity.copilot": "1.0.0"
+  },
+  "scopedRegistries": [
+    {
+      "name": "package.openupm.com",
+      "url": "https://package.openupm.com",
+      "scopes": [ "com.atelierai.unity.copilot" ]
+    }
+  ]
+}
+```
+
+## Quick start
+
+```bash
+uco ping                        # health-check the bridge
+uco call unity-tool-list        # list every tool the Editor exposes
+uco exec --method-body --return-type int --code 'return GameObject.FindObjectsOfType<Camera>().Length;'
+```
+
+Configuration lives in `UserSettings/uco-config.json` (migrated automatically
+from the legacy filename by older→newer versions of the plugin).
+
+## Wire protocol
+
+Plain HTTP + WebSocket with JSON envelopes (`/api/tools/...`). The wire is plain JSON
+protocol on the wire — the bridge was deliberately stripped down to REST so
+any agent, script, or CI job can drive it with nothing but an HTTP client.
+
+## Attribution & license
+
+Derived from [IvanMurzak/Unity-MCP](https://github.com/IvanMurzak/Unity-MCP)
+(Apache-2.0). Licensed under the Apache License 2.0 — see [LICENSE](LICENSE)
+for the dual copyright notice.

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/Unity/Logs/BufferedFileLogStorage.cs

@@ -1,207 +1,208 @@
-/*
-┌──────────────────────────────────────────────────────────────────┐
-│  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;
-using System.Collections.Generic;
-using System.IO;
-using System.Linq;
-using System.Text.Json;
-using System.Threading.Tasks;
-using Microsoft.Extensions.Logging;
-using UnityEngine;
-
-namespace com.AtelierAI.Unity.Copilot
-{
-    using ILogger = Microsoft.Extensions.Logging.ILogger;
-    public class BufferedFileLogStorage : FileLogStorage
-    {
-        protected readonly int _flushEntriesThreshold;
-        protected readonly LogEntry[] _logEntriesBuffer;
-        protected int _logEntriesBufferLength;
-
-        public BufferedFileLogStorage(
-            ILogger? logger = null,
-            int flushEntriesThreshold = 100,
-            string? cacheFilePath = null,
-            string? cacheFileName = null,
-            int fileBufferSize = 4096,
-            int maxFileSizeMB = DefaultMaxFileSizeMB,
-            JsonSerializerOptions? jsonOptions = null)
-            : base(logger, cacheFilePath, cacheFileName, fileBufferSize, maxFileSizeMB, jsonOptions)
-        {
-            if (flushEntriesThreshold <= 0)
-                throw new ArgumentOutOfRangeException(nameof(flushEntriesThreshold), "Flush entries threshold must be greater than zero.");
-
-            _flushEntriesThreshold = flushEntriesThreshold;
-            _logEntriesBuffer = new LogEntry[flushEntriesThreshold];
-            _logEntriesBufferLength = 0;
-        }
-
-        public override void Flush()
-        {
-            if (_isDisposed.Value)
-            {
-                _logger.LogWarning("{method} called but already disposed, ignored.",
-                    nameof(Flush));
-                return;
-            }
-            lock (_fileMutex)
-            {
-                // Flush buffered entries to file
-                if (_logEntriesBufferLength > 0)
-                {
-                    var entriesToFlush = new LogEntry[_logEntriesBufferLength];
-                    Array.Copy(_logEntriesBuffer, entriesToFlush, _logEntriesBufferLength);
-                    base.AppendInternal(entriesToFlush);
-                    _logEntriesBufferLength = 0;
-                }
-                fileWriteStream?.Flush();
-            }
-        }
-        public override Task FlushAsync()
-        {
-            if (_isDisposed.Value)
-            {
-                _logger.LogWarning("{method} called but already disposed, ignored.",
-                    nameof(FlushAsync));
-                return Task.CompletedTask;
-            }
-            lock (_fileMutex)
-            {
-                // Flush buffered entries to file
-                if (_logEntriesBufferLength > 0)
-                {
-                    var entriesToFlush = new LogEntry[_logEntriesBufferLength];
-                    Array.Copy(_logEntriesBuffer, entriesToFlush, _logEntriesBufferLength);
-                    base.AppendInternal(entriesToFlush);
-                    _logEntriesBufferLength = 0;
-                }
-                fileWriteStream?.Flush();
-            }
-            return Task.CompletedTask;
-        }
-
-        protected override void AppendInternal(params LogEntry[] entries)
-        {
-            if (_isDisposed.Value)
-            {
-                _logger.LogWarning("{method} called but already disposed, ignored.",
-                    nameof(AppendInternal));
-                return;
-            }
-            if (_logEntriesBufferLength >= _flushEntriesThreshold)
-            {
-                base.AppendInternal(_logEntriesBuffer);
-                _logEntriesBufferLength = 0;
-            }
-            foreach (var entry in entries)
-            {
-                _logEntriesBuffer[_logEntriesBufferLength] = entry;
-                _logEntriesBufferLength++;
-
-                if (_logEntriesBufferLength >= _flushEntriesThreshold)
-                {
-                    base.AppendInternal(_logEntriesBuffer);
-                    _logEntriesBufferLength = 0;
-                }
-            }
-        }
-
-        /// <summary>
-        /// Closes and disposes the current file stream if open. Clears the log cache file.
-        /// </summary>
-        public override LogClearResult Clear()
-        {
-            if (_isDisposed.Value)
-            {
-                _logger.LogWarning("{method} called but already disposed, ignored.",
-                    nameof(Clear));
-                return new LogClearResult
-                {
-                    Ok = false,
-                    Strategy = "storage-disposed",
-                    Path = filePath,
-                    Error = "The log storage is disposed."
-                };
-            }
-            lock (_fileMutex)
-            {
-                _logEntriesBufferLength = 0;
-                return ClearFileLocked();
-            }
-        }
-
-        public override LogEntry[] Query(
-            int maxEntries = 100,
-            LogType? logTypeFilter = null,
-            bool includeStackTrace = false,
-            int lastMinutes = 0)
-        {
-            if (_isDisposed.Value)
-            {
-                _logger.LogWarning("{method} called but already disposed, ignored.",
-                    nameof(Query));
-                return Array.Empty<LogEntry>();
-            }
-            lock (_fileMutex)
-            {
-                return QueryInternal(maxEntries, logTypeFilter, includeStackTrace, lastMinutes);
-            }
-        }
-
-        protected override LogEntry[] QueryInternal(
-            int maxEntries = 100,
-            LogType? logTypeFilter = null,
-            bool includeStackTrace = false,
-            int lastMinutes = 0)
-        {
-            var result = new List<LogEntry>();
-            var cutoffTime = lastMinutes > 0
-                ? System.DateTime.Now.AddMinutes(-lastMinutes)
-                : System.DateTime.MinValue;
-
-            // 1. Get from buffer (Newest are at the end of buffer)
-            for (int i = _logEntriesBufferLength - 1; i >= 0; i--)
-            {
-                var entry = _logEntriesBuffer[i];
-                if (logTypeFilter.HasValue && entry.LogType != logTypeFilter.Value)
-                    continue;
-
-                if (lastMinutes > 0)
-                {
-                    if (entry.Timestamp < cutoffTime)
-                    {
-                        return result.AsEnumerable().Reverse().ToArray();
-                    }
-                }
-
-                result.Add(entry);
-                if (result.Count >= maxEntries)
-                    return result.AsEnumerable().Reverse().ToArray();
-            }
-
-            // 2. Exit if we already have enough entries
-            var neededLogsCount = maxEntries - result.Count;
-            if (neededLogsCount <= 0)
-                return result.AsEnumerable().Reverse().ToArray();
-
-            result.Reverse();
-
-            // 3. Get from file
-            var fileEntries = base.QueryInternal(neededLogsCount, logTypeFilter, includeStackTrace, lastMinutes);
-            result.AddRange(fileEntries);
-
-            return result.ToArray();
-        }
-
-        ~BufferedFileLogStorage() => Dispose();
-    }
-}
+/*
+┌──────────────────────────────────────────────────────────────────┐
+│  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;
+using System.Collections.Generic;
+using System.IO;
+using System.Linq;
+using System.Text.Json;
+using System.Threading.Tasks;
+using Microsoft.Extensions.Logging;
+using UnityEngine;
+
+namespace com.AtelierAI.Unity.Copilot
+{
+    using ILogger = Microsoft.Extensions.Logging.ILogger;
+    public class BufferedFileLogStorage : FileLogStorage
+    {
+        protected readonly int _flushEntriesThreshold;
+        protected readonly LogEntry[] _logEntriesBuffer;
+        protected int _logEntriesBufferLength;
+
+        public BufferedFileLogStorage(
+            ILogger? logger = null,
+            int flushEntriesThreshold = 100,
+            string? cacheFilePath = null,
+            string? cacheFileName = null,
+            int fileBufferSize = 4096,
+            int maxFileSizeMB = DefaultMaxFileSizeMB,
+            JsonSerializerOptions? jsonOptions = null)
+            : base(logger, cacheFilePath, cacheFileName, fileBufferSize, maxFileSizeMB, jsonOptions)
+        {
+            if (flushEntriesThreshold <= 0)
+                throw new ArgumentOutOfRangeException(nameof(flushEntriesThreshold), "Flush entries threshold must be greater than zero.");
+
+            _flushEntriesThreshold = flushEntriesThreshold;
+            _logEntriesBuffer = new LogEntry[flushEntriesThreshold];
+            _logEntriesBufferLength = 0;
+        }
+
+        public override void Flush()
+        {
+            if (_isDisposed.Value)
+            {
+                _logger.LogWarning("{method} called but already disposed, ignored.",
+                    nameof(Flush));
+                return;
+            }
+            lock (_fileMutex)
+            {
+                // Flush buffered entries to file
+                if (_logEntriesBufferLength > 0)
+                {
+                    var entriesToFlush = new LogEntry[_logEntriesBufferLength];
+                    Array.Copy(_logEntriesBuffer, entriesToFlush, _logEntriesBufferLength);
+                    base.AppendInternal(entriesToFlush);
+                    _logEntriesBufferLength = 0;
+                }
+                fileWriteStream?.Flush();
+            }
+        }
+        public override Task FlushAsync()
+        {
+            if (_isDisposed.Value)
+            {
+                _logger.LogWarning("{method} called but already disposed, ignored.",
+                    nameof(FlushAsync));
+                return Task.CompletedTask;
+            }
+            lock (_fileMutex)
+            {
+                // Flush buffered entries to file
+                if (_logEntriesBufferLength > 0)
+                {
+                    var entriesToFlush = new LogEntry[_logEntriesBufferLength];
+                    Array.Copy(_logEntriesBuffer, entriesToFlush, _logEntriesBufferLength);
+                    base.AppendInternal(entriesToFlush);
+                    _logEntriesBufferLength = 0;
+                }
+                fileWriteStream?.Flush();
+            }
+            return Task.CompletedTask;
+        }
+
+        protected override void AppendInternal(params LogEntry[] entries)
+        {
+            if (_isDisposed.Value)
+            {
+                _logger.LogWarning("{method} called but already disposed, ignored.",
+                    nameof(AppendInternal));
+                return;
+            }
+            if (_logEntriesBufferLength >= _flushEntriesThreshold)
+            {
+                base.AppendInternal(_logEntriesBuffer);
+                _logEntriesBufferLength = 0;
+            }
+            foreach (var entry in entries)
+            {
+                _logEntriesBuffer[_logEntriesBufferLength] = entry;
+                _logEntriesBufferLength++;
+
+                if (_logEntriesBufferLength >= _flushEntriesThreshold)
+                {
+                    base.AppendInternal(_logEntriesBuffer);
+                    _logEntriesBufferLength = 0;
+                }
+            }
+        }
+
+        /// <summary>
+        /// Closes and disposes the current file stream if open. Clears the log cache file.
+        /// </summary>
+        public override LogClearResult Clear()
+        {
+            if (_isDisposed.Value)
+            {
+                _logger.LogWarning("{method} called but already disposed, ignored.",
+                    nameof(Clear));
+                return new LogClearResult
+                {
+                    Ok = false,
+                    Strategy = "storage-disposed",
+                    Path = filePath,
+                    Error = "The log storage is disposed."
+                };
+            }
+            lock (_fileMutex)
+            {
+                _logEntriesBufferLength = 0;
+                return ClearFileLocked();
+            }
+        }
+
+        public override LogEntry[] Query(
+            int maxEntries = 100,
+            LogType? logTypeFilter = null,
+            bool includeStackTrace = false,
+            int lastMinutes = 0)
+        {
+            if (_isDisposed.Value)
+            {
+                _logger.LogWarning("{method} called but already disposed, ignored.",
+                    nameof(Query));
+                return Array.Empty<LogEntry>();
+            }
+            lock (_fileMutex)
+            {
+                return QueryInternal(maxEntries, logTypeFilter, includeStackTrace, lastMinutes);
+            }
+        }
+
+        protected override LogEntry[] QueryInternal(
+            int maxEntries = 100,
+            LogType? logTypeFilter = null,
+            bool includeStackTrace = false,
+            int lastMinutes = 0)
+        {
+            var result = new List<LogEntry>();
+            var cutoffTime = lastMinutes > 0
+                ? System.DateTime.Now.AddMinutes(-lastMinutes)
+                : System.DateTime.MinValue;
+
+            // 1. Get from buffer (Newest are at the end of buffer)
+            for (int i = _logEntriesBufferLength - 1; i >= 0; i--)
+            {
+                var entry = _logEntriesBuffer[i];
+                if (logTypeFilter.HasValue && entry.LogType != logTypeFilter.Value)
+                    continue;
+
+                if (lastMinutes > 0)
+                {
+                    if (entry.Timestamp < cutoffTime)
+                    {
+                        return result.AsEnumerable().Reverse().ToArray();
+                    }
+                }
+
+                result.Add(entry);
+                if (result.Count >= maxEntries)
+                    return result.AsEnumerable().Reverse().ToArray();
+            }
+
+            // 2. Exit if we already have enough entries
+            var neededLogsCount = maxEntries - result.Count;
+            if (neededLogsCount <= 0)
+                return result.AsEnumerable().Reverse().ToArray();
+
+            result.Reverse();
+
+            // 3. Get from file
+            var fileEntries = base.QueryInternal(neededLogsCount, logTypeFilter, includeStackTrace, lastMinutes);
+            result.AddRange(fileEntries);
+
+            return result.ToArray();
+        }
+
+// No finalizer: Dispose is plain managed cleanup; running arbitrary code on
+        // the finalizer thread during domain unload is never safe.
+    }
+}

package/vendor/plugin/com.atelierai.unity.copilot/Runtime/Unity/Logs/FileLogStorage.cs

@@ -542,6 +542,8 @@ namespace com.AtelierAI.Unity.Copilot
             GC.SuppressFinalize(this);
         }
 
-        ~FileLogStorage() => Dispose();
+// No finalizer: Dispose performs file I/O / Unity static-event unsubscription,
+        // which must never run on the finalizer thread during domain unload
+        // (same crash class as uco-domain-reload-crash-20260919).
     }
 }