Update on three different changes

1. Add McpLog that could log mcp calls and errors under Asset/
2. Solve issues CoplayDev#816 via incluging str in annotation, extracted AssignedObjectreference() helper that verify assignments and resolve component types from gameobjects.

Author: Scriptwonder

MCPForUnity/Editor/Constants/EditorPrefKeys.cs

@@ -67,5 +67,6 @@ internal static class EditorPrefKeys
         internal const string ApiKey = "MCPForUnity.ApiKey";
 
         internal const string BatchExecuteMaxCommands = "MCPForUnity.BatchExecute.MaxCommands";
+        internal const string LogRecordEnabled = "MCPForUnity.LogRecordEnabled";
     }
 }

MCPForUnity/Editor/Helpers/ComponentOps.cs

@@ -592,12 +592,14 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
                     error = $"No object found with instanceID {id}.";
                     return false;
                 }
-                prop.objectReferenceValue = resolved;
-                return true;
+                return AssignObjectReference(prop, resolved, null, out error);
             }
 
             if (value is JObject jObj)
             {
+                // Optional component type filter — e.g. {"instanceID": 123, "component": "Button"}
+                string componentFilter = jObj["component"]?.ToString();
+
                 var idToken = jObj["instanceID"];
                 if (idToken != null)
                 {
@@ -608,8 +610,7 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
                         error = $"No object found with instanceID {id}.";
                         return false;
                     }
-                    prop.objectReferenceValue = resolved;
-                    return true;
+                    return AssignObjectReference(prop, resolved, componentFilter, out error);
                 }
 
                 var guidToken = jObj["guid"];
@@ -665,8 +666,8 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
                         return false;
                     }
 
-                    prop.objectReferenceValue = AssetDatabase.LoadAssetAtPath<UnityEngine.Object>(path);
-                    return true;
+                    var loaded = AssetDatabase.LoadAssetAtPath<UnityEngine.Object>(path);
+                    return AssignObjectReference(prop, loaded, componentFilter, out error);
                 }
 
                 var pathToken = jObj["path"];
@@ -679,8 +680,7 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
                         error = $"No asset found at path '{pathToken}'.";
                         return false;
                     }
-                    prop.objectReferenceValue = resolved;
-                    return true;
+                    return AssignObjectReference(prop, resolved, componentFilter, out error);
                 }
 
                 var nameToken = jObj["name"];
@@ -696,6 +696,16 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
             if (value.Type == JTokenType.String)
             {
                 string strVal = value.ToString();
+
+                // Try as instanceID if the string is purely numeric
+                if (int.TryParse(strVal, out int parsedId))
+                {
+                    var resolved = GameObjectLookup.ResolveInstanceID(parsedId);
+                    if (resolved != null)
+                        return AssignObjectReference(prop, resolved, null, out error);
+                    // Not a valid instanceID — fall through to path/name resolution
+                }
+
                 if (strVal.StartsWith("Assets/", StringComparison.OrdinalIgnoreCase) || strVal.Contains("/"))
                 {
                     string sanitized = AssetPathUtility.SanitizeAssetPath(strVal);
@@ -705,8 +715,7 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
                         error = $"No asset found at path '{strVal}'.";
                         return false;
                     }
-                    prop.objectReferenceValue = resolved;
-                    return true;
+                    return AssignObjectReference(prop, resolved, null, out error);
                 }
 
                 // Fall back to scene hierarchy lookup by name.
@@ -717,6 +726,66 @@ private static bool SetObjectReference(SerializedProperty prop, JToken value, ou
             return false;
         }
 
+        /// <summary>
+        /// Assigns a resolved object to a SerializedProperty, with automatic component fallback.
+        /// If the resolved object is a GameObject but the property expects a Component type,
+        /// searches the GameObject's components for a compatible one.
+        /// Optionally filters by component type name (e.g. "Button", "Rigidbody").
+        /// </summary>
+        private static bool AssignObjectReference(SerializedProperty prop, UnityEngine.Object resolved, string componentFilter, out string error)
+        {
+            error = null;
+            if (resolved == null)
+            {
+                error = "Resolved object is null.";
+                return false;
+            }
+
+            // If a component filter is specified and the resolved object is a GameObject,
+            // find the specific component by type name.
+            if (!string.IsNullOrEmpty(componentFilter) && resolved is GameObject filterGo)
+            {
+                var components = filterGo.GetComponents<Component>();
+                foreach (var comp in components)
+                {
+                    if (comp == null) continue;
+                    if (string.Equals(comp.GetType().Name, componentFilter, StringComparison.OrdinalIgnoreCase) ||
+                        string.Equals(comp.GetType().FullName, componentFilter, StringComparison.OrdinalIgnoreCase))
+                    {
+                        prop.objectReferenceValue = comp;
+                        if (prop.objectReferenceValue != null)
+                            return true;
+                    }
+                }
+                error = $"Component '{componentFilter}' not found on GameObject '{filterGo.name}'.";
+                return false;
+            }
+
+            // Try direct assignment first
+            prop.objectReferenceValue = resolved;
+            if (prop.objectReferenceValue != null)
+                return true;
+
+            // If the resolved object is a GameObject but the property expects a Component,
+            // try each component on the GameObject until one is accepted.
+            if (resolved is GameObject go)
+            {
+                var components = go.GetComponents<Component>();
+                foreach (var comp in components)
+                {
+                    if (comp == null) continue;
+                    prop.objectReferenceValue = comp;
+                    if (prop.objectReferenceValue != null)
+                        return true;
+                }
+                error = $"GameObject '{go.name}' found but no compatible component for the property type.";
+                return false;
+            }
+
+            error = $"Object '{resolved.name}' (type: {resolved.GetType().Name}) is not compatible with the property type.";
+            return false;
+        }
+
         /// <summary>
         /// Resolves a scene GameObject by name and assigns it (or a component on it)
         /// to a SerializedProperty. Uses GameObjectLookup for robust search
@@ -747,24 +816,7 @@ private static bool ResolveSceneObjectByName(SerializedProperty prop, string nam
                 return false;
             }
 
-            // If the property accepts a GameObject directly, assign it.
-            prop.objectReferenceValue = go;
-            if (prop.objectReferenceValue != null)
-                return true;
-
-            // The field type may expect a specific Component (e.g. Transform, Rigidbody).
-            // Try each component on the GameObject until one is accepted.
-            var components = go.GetComponents<Component>();
-            foreach (var comp in components)
-            {
-                if (comp == null) continue;
-                prop.objectReferenceValue = comp;
-                if (prop.objectReferenceValue != null)
-                    return true;
-            }
-
-            error = $"GameObject '{name}' found but no compatible component for property type.";
-            return false;
+            return AssignObjectReference(prop, go, null, out error);
         }
 
         /// <summary>

MCPForUnity/Editor/Helpers/McpLogRecord.cs

@@ -0,0 +1,100 @@
+using System;
+using System.IO;
+using MCPForUnity.Editor.Constants;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using UnityEditor;
+using UnityEngine;
+
+namespace MCPForUnity.Editor.Helpers
+{
+    internal static class McpLogRecord
+    {
+        private static readonly string LogPath = Path.Combine(Application.dataPath, "mcp.log");
+        private static readonly string ErrorLogPath = Path.Combine(Application.dataPath, "mcpError.log");
+        private const long MaxLogSizeBytes = 1024 * 1024; // 1 MB
+        private static bool _sessionStarted;
+
+        internal static bool IsEnabled
+        {
+            get => EditorPrefs.GetBool(EditorPrefKeys.LogRecordEnabled, false);
+            set => EditorPrefs.SetBool(EditorPrefKeys.LogRecordEnabled, value);
+        }
+
+        internal static void Log(string commandType, JObject parameters, string type, string status, long durationMs, string error = null)
+        {
+            if (!IsEnabled) return;
+
+            try
+            {
+                if (!_sessionStarted)
+                {
+                    _sessionStarted = true;
+                    RotateIfNeeded(LogPath);
+                    var sessionEntry = new JObject
+                    {
+                        ["ts"] = DateTime.UtcNow.ToString("yyyy-MM-ddTHH:mm:ss.fffZ"),
+                        ["event"] = "session_start",
+                        ["unity"] = Application.unityVersion
+                    };
+                    AppendLine(LogPath, sessionEntry.ToString(Formatting.None));
+                }
+
+                var entry = new JObject
+                {
+                    ["ts"] = DateTime.UtcNow.ToString("yyyy-MM-ddTHH:mm:ss.fffZ"),
+                    ["tool"] = commandType,
+                    ["type"] = type,
+                    ["status"] = status,
+                    ["ms"] = durationMs
+                };
+
+                var action = parameters?.Value<string>("action");
+                if (!string.IsNullOrEmpty(action))
+                    entry["action"] = action;
+
+                if (parameters != null)
+                    entry["params"] = parameters;
+
+                if (error != null)
+                    entry["error"] = error;
+
+                var line = entry.ToString(Formatting.None);
+                AppendLine(LogPath, line);
+
+                if (status == "ERROR")
+                {
+                    RotateIfNeeded(ErrorLogPath);
+                    AppendLine(ErrorLogPath, line);
+                }
+            }
+            catch (Exception ex)
+            {
+                McpLog.Warn($"[McpLogRecord] Failed to write log: {ex.Message}");
+            }
+        }
+
+        private static void AppendLine(string path, string line)
+        {
+            File.AppendAllText(path, line + Environment.NewLine);
+        }
+
+        private static void RotateIfNeeded(string path)
+        {
+            try
+            {
+                if (!File.Exists(path)) return;
+                var info = new FileInfo(path);
+                if (info.Length <= MaxLogSizeBytes) return;
+
+                var lines = File.ReadAllLines(path);
+                var half = lines.Length / 2;
+                File.WriteAllLines(path, lines[half..]);
+            }
+            catch
+            {
+                // Best-effort rotation
+            }
+        }
+    }
+}

MCPForUnity/Editor/Helpers/McpLogRecord.cs.meta

@@ -361,18 +361,31 @@ private static void ProcessCommand(string id, PendingCommand pending)
                     return;
                 }
 
+                var logType = resourceMeta != null ? "resource" : toolMeta != null ? "tool" : "unknown";
+                var sw = McpLogRecord.IsEnabled ? System.Diagnostics.Stopwatch.StartNew() : null;
                 var result = CommandRegistry.ExecuteCommand(command.type, parameters, pending.CompletionSource);
 
                 if (result == null)
                 {
                     // Async command – cleanup after completion on next editor frame to preserve order.
-                    pending.CompletionSource.Task.ContinueWith(_ =>
+                    var capturedType = command.type;
+                    var capturedParams = parameters;
+                    var capturedLogType = logType;
+                    pending.CompletionSource.Task.ContinueWith(t =>
                     {
+                        sw?.Stop();
+                        McpLogRecord.Log(capturedType, capturedParams, capturedLogType,
+                            t.IsFaulted ? "ERROR" : "SUCCESS",
+                            sw?.ElapsedMilliseconds ?? 0,
+                            t.IsFaulted ? t.Exception?.InnerException?.Message : null);
                         EditorApplication.delayCall += () => RemovePending(id, pending);
                     }, TaskScheduler.Default);
                     return;
                 }
 
+                sw?.Stop();
+                McpLogRecord.Log(command.type, parameters, logType, "SUCCESS", sw?.ElapsedMilliseconds ?? 0);
+
                 var response = new { status = "success", result };
                 pending.TrySetResult(JsonConvert.SerializeObject(response));
                 RemovePending(id, pending);

MCPForUnity/Editor/Services/Transport/TransportCommandDispatcher.cs

@@ -25,6 +25,7 @@ public class McpAdvancedSection
         private Button browseGitUrlButton;
         private Button clearGitUrlButton;
         private Toggle debugLogsToggle;
+        private Toggle logRecordToggle;
         private Toggle devModeForceRefreshToggle;
         private Toggle allowLanHttpBindToggle;
         private Toggle allowInsecureRemoteHttpToggle;
@@ -66,6 +67,7 @@ private void CacheUIElements()
             browseGitUrlButton = Root.Q<Button>("browse-git-url-button");
             clearGitUrlButton = Root.Q<Button>("clear-git-url-button");
             debugLogsToggle = Root.Q<Toggle>("debug-logs-toggle");
+            logRecordToggle = Root.Q<Toggle>("log-record-toggle");
             devModeForceRefreshToggle = Root.Q<Toggle>("dev-mode-force-refresh-toggle");
             allowLanHttpBindToggle = Root.Q<Toggle>("allow-lan-http-bind-toggle");
             allowInsecureRemoteHttpToggle = Root.Q<Toggle>("allow-insecure-remote-http-toggle");
@@ -96,6 +98,13 @@ private void InitializeUI()
                 if (debugLabel != null)
                     debugLabel.tooltip = debugLogsToggle.tooltip;
             }
+            if (logRecordToggle != null)
+            {
+                logRecordToggle.tooltip = "Log every MCP tool execution (tool, action, status, duration) to Assets/mcp.log.";
+                var logRecordLabel = logRecordToggle?.parent?.Q<Label>();
+                if (logRecordLabel != null)
+                    logRecordLabel.tooltip = logRecordToggle.tooltip;
+            }
             if (devModeForceRefreshToggle != null)
             {
                 devModeForceRefreshToggle.tooltip = "When enabled, generated uvx commands add '--no-cache --refresh' before launching (slower startup, but avoids stale cached builds while iterating on the Server).";
@@ -146,6 +155,9 @@ private void InitializeUI()
             debugLogsToggle.value = debugEnabled;
             McpLog.SetDebugLoggingEnabled(debugEnabled);
 
+            if (logRecordToggle != null)
+                logRecordToggle.value = McpLogRecord.IsEnabled;
+
             devModeForceRefreshToggle.value = EditorPrefs.GetBool(EditorPrefKeys.DevModeForceServerRefresh, false);
             if (allowLanHttpBindToggle != null)
             {
@@ -199,6 +211,14 @@ private void RegisterCallbacks()
                 McpLog.SetDebugLoggingEnabled(evt.newValue);
             });
 
+            if (logRecordToggle != null)
+            {
+                logRecordToggle.RegisterValueChangedCallback(evt =>
+                {
+                    McpLogRecord.IsEnabled = evt.newValue;
+                });
+            }
+
             devModeForceRefreshToggle.RegisterValueChangedCallback(evt =>
             {
                 EditorPrefs.SetBool(EditorPrefKeys.DevModeForceServerRefresh, evt.newValue);
@@ -324,6 +344,8 @@ public void UpdatePathOverrides()
 
             gitUrlOverride.value = EditorPrefs.GetString(EditorPrefKeys.GitUrlOverride, "");
             debugLogsToggle.value = EditorPrefs.GetBool(EditorPrefKeys.DebugLogs, false);
+            if (logRecordToggle != null)
+                logRecordToggle.value = McpLogRecord.IsEnabled;
             devModeForceRefreshToggle.value = EditorPrefs.GetBool(EditorPrefKeys.DevModeForceServerRefresh, false);
             if (allowLanHttpBindToggle != null)
             {

MCPForUnity/Editor/Windows/Components/Advanced/McpAdvancedSection.cs

@@ -27,6 +27,11 @@
                 <ui:Toggle name="debug-logs-toggle" class="setting-toggle" />
             </ui:VisualElement>
 
+            <ui:VisualElement class="setting-row">
+                <ui:Label text="Log Record (Assets/mcp.log):" class="setting-label" />
+                <ui:Toggle name="log-record-toggle" class="setting-toggle" />
+            </ui:VisualElement>
+
             <ui:VisualElement class="setting-row">
                 <ui:Label text="Server Health:" class="setting-label" />
                 <ui:VisualElement class="status-container">

MCPForUnity/Editor/Windows/Components/Advanced/McpAdvancedSection.uxml

@@ -33,7 +33,7 @@ async def manage_asset(
     path: Annotated[str, "Asset path (e.g., 'Materials/MyMaterial.mat') or search scope (e.g., 'Assets')."],
     asset_type: Annotated[str,
                           "Asset type (e.g., 'Material', 'Folder') - required for 'create'. Note: For ScriptableObjects, use manage_scriptable_object."] | None = None,
-    properties: Annotated[dict[str, Any],
+    properties: Annotated[dict[str, Any] | str,
                           "Dictionary of properties for 'create'/'modify'. Keys are property names, values are property values."] | None = None,
     destination: Annotated[str,
                            "Target path for 'duplicate'/'move'."] | None = None,