Merge pull request #77 from Snowblaze/am/create-prefab-tool-field-fix

fix: use the proper create prefab field

Author: CoderGamester

Editor/Tools/CreatePrefabTool.cs

@@ -25,7 +25,7 @@ public CreatePrefabTool()
         public override JObject Execute(JObject parameters)
         {
             // Extract parameters
-            string scriptName = parameters["scriptName"]?.ToObject<string>();
+            string componentName = parameters["componentName"]?.ToObject<string>();
             string prefabName = parameters["prefabName"]?.ToObject<string>();
             JObject fieldValues = parameters["fieldValues"]?.ToObject<JObject>();
 
@@ -42,20 +42,20 @@ public override JObject Execute(JObject parameters)
             GameObject tempObject = new GameObject(prefabName);
 
             // Add component if provided
-            if (!string.IsNullOrEmpty(scriptName))
+            if (!string.IsNullOrEmpty(componentName))
             {
                 try
                 {
                     // Add component
-                    Component component = AddComponent(tempObject, scriptName);
+                    Component component = AddComponent(tempObject, componentName);
 
                     // Apply field values if provided and component exists
                     ApplyFieldValues(fieldValues, component);
                 }
                 catch (Exception ex)
                 {
                     return McpUnitySocketHandler.CreateErrorResponse(
-                        $"Failed to add component '{scriptName}' to GameObject", 
+                        $"Failed to add component '{componentName}' to GameObject", 
                         "component_error"
                     );
                 }
@@ -71,7 +71,8 @@ public override JObject Execute(JObject parameters)
             }
 
             // Create the prefab
-            GameObject prefab = PrefabUtility.SaveAsPrefabAsset(tempObject, prefabPath);
+            bool success = false;
+            PrefabUtility.SaveAsPrefabAsset(tempObject, prefabPath, out success);
 
             // Clean up temporary object
             UnityEngine.Object.DestroyImmediate(tempObject);
@@ -80,34 +81,36 @@ public override JObject Execute(JObject parameters)
             AssetDatabase.Refresh();
 
             // Log the action
-            McpLogger.LogInfo($"Created prefab '{prefab.name}' at path '{prefabPath}' from script '{scriptName}'");
+            McpLogger.LogInfo($"Created prefab '{prefabName}' at path '{prefabPath}' from script '{componentName}'");
+
+            string message = success ? $"Successfully created prefab '{prefabName}' at path '{prefabPath}'" : $"Failed to create prefab '{prefabName}' at path '{prefabPath}'";
 
             // Create the response
             return new JObject
             {
-                ["success"] = true,
+                ["success"] = success,
                 ["type"] = "text",
-                ["message"] = $"Successfully created prefab '{prefab.name}' at path '{prefabPath}'",
+                ["message"] = message,
                 ["prefabPath"] = prefabPath
             };
         }
 
-        private Component AddComponent(GameObject gameObject, string scriptName)
+        private Component AddComponent(GameObject gameObject, string componentName)
         {
             // Find the script type
-            Type scriptType = Type.GetType($"{scriptName}, Assembly-CSharp");
+            Type scriptType = Type.GetType($"{componentName}, Assembly-CSharp");
             if (scriptType == null)
             {
                 // Try with just the class name
-                scriptType = Type.GetType(scriptName);
+                scriptType = Type.GetType(componentName);
             }
 
             if (scriptType == null)
             {
                 // Try to find the type using AppDomain
                 foreach (var assembly in AppDomain.CurrentDomain.GetAssemblies())
                 {
-                    scriptType = assembly.GetType(scriptName);
+                    scriptType = assembly.GetType(componentName);
                     if (scriptType != null)
                         break;
                 }

Server~/src/tools/createPrefabTool.ts

@@ -51,14 +51,7 @@ export function registerCreatePrefabTool(server: McpServer, mcpUnity: McpUnity,
  * @returns A promise that resolves to the tool execution result
  * @throws McpUnityError if validation fails or the request to Unity fails
  */
-async function toolHandler(mcpUnity: McpUnity, params: any) {  
-  if (!params.scriptName) {
-    throw new McpUnityError(
-      ErrorType.VALIDATION,
-      "'scriptName' must be provided"
-    );
-  }
-  
+async function toolHandler(mcpUnity: McpUnity, params: any) {
   if (!params.prefabName) {
     throw new McpUnityError(
       ErrorType.VALIDATION,