ultimate-unreal-engine-mcp 0.1.12 → 0.1.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ultimate-unreal-engine-mcp",
-  "version": "0.1.12",
+  "version": "0.1.14",
   "description": "MCP server giving AI assistants full access to Unreal Engine 5.7 projects",
   "type": "module",
   "engines": {

package/unreal-plugin/MCPBridge/Source/MCPBridgeEditor/Private/MCPActorCommands.cpp

@@ -478,8 +478,35 @@ void RegisterActorCommands(FMCPCommandRouter& Router)
 			return;
 		}
 
-		// Find the UPROPERTY by name on the actor's class.
-		FProperty* Prop = TargetActor->GetClass()->FindPropertyByName(FName(*PropertyName));
+		// Optional component_name — if set, resolve property on the named component
+		// instead of the actor itself.  Enables "DoorMesh" + "StaticMesh" patterns.
+		UObject* PropertyOwner = TargetActor;
+		FString ComponentName;
+		if (Payload->TryGetStringField(TEXT("component_name"), ComponentName) && !ComponentName.IsEmpty())
+		{
+			UActorComponent* Comp = TargetActor->FindComponentByClass<UActorComponent>();
+			// Search all components by their subobject name.
+			TInlineComponentArray<UActorComponent*> Components;
+			TargetActor->GetComponents(Components);
+			UActorComponent* FoundComp = nullptr;
+			for (UActorComponent* C : Components)
+			{
+				if (C && C->GetName() == ComponentName)
+				{
+					FoundComp = C;
+					break;
+				}
+			}
+			if (!FoundComp)
+			{
+				SendResponse(BuildActorErrorResponse(CorrId, TEXT("component_not_found")) + TEXT("\n"));
+				return;
+			}
+			PropertyOwner = FoundComp;
+		}
+
+		// Find the UPROPERTY by name on the target object's class.
+		FProperty* Prop = PropertyOwner->GetClass()->FindPropertyByName(FName(*PropertyName));
 		if (!Prop)
 		{
 			SendResponse(BuildActorErrorResponse(CorrId, TEXT("property_not_found")) + TEXT("\n"));
@@ -487,7 +514,7 @@ void RegisterActorCommands(FMCPCommandRouter& Router)
 		}
 
 		TargetActor->Modify();
-		void* ValuePtr = Prop->ContainerPtrToValuePtr<void>(TargetActor);
+		void* ValuePtr = Prop->ContainerPtrToValuePtr<void>(PropertyOwner);
 
 		if (ValueType == TEXT("bool"))
 		{

package/unreal-plugin/MCPBridge/Source/MCPBridgeEditor/Private/MCPImportExportCommands.cpp

@@ -236,9 +236,15 @@ void RegisterImportExportCommands(FMCPCommandRouter& Router)
 		Payload->TryGetBoolField(TEXT("combine_meshes"), bCombineMeshes);
 		Payload->TryGetNumberField(TEXT("scale_factor"), ScaleFactor);
 
-		// Capture everything for async dispatch.
-		AsyncTask(ENamedThreads::GameThread, [CorrId, SendResponse, SourceFile, DestPath,
-			bImportMaterials, bCombineMeshes, ScaleFactor]()
+		// Defer the import to the next engine tick via FTSTicker.
+		// Reason: ImportAssetTasks triggers the Interchange pipeline which
+		// enqueues task-graph work.  Running it inside the router's
+		// AsyncTask(GameThread) hits the recursion guard (Assertion
+		// ++Queue.RecursionGuard == 1).  A ticker callback runs on the
+		// game thread but OUTSIDE the task-graph scope.
+		FTSTicker::GetCoreTicker().AddTicker(
+			FTickerDelegate::CreateLambda([CorrId, SendResponse, SourceFile, DestPath,
+				bImportMaterials, bCombineMeshes, ScaleFactor](float) -> bool
 		{
 			// Split dest_path into package path + asset name.
 			FString PackagePath = DestPath;
@@ -307,7 +313,8 @@ void RegisterImportExportCommands(FMCPCommandRouter& Router)
 			Data->SetNumberField(TEXT("count"), static_cast<double>(AssetsArray.Num()));
 
 			SendResponse(BuildImpSuccessResponse(CorrId, Data) + TEXT("\n"));
-		});
+			return false; // One-shot: remove ticker after execution
+		}), 0.0f);
 	});
 
 	// -----------------------------------------------------------------------
@@ -753,7 +760,9 @@ void RegisterImportExportCommands(FMCPCommandRouter& Router)
 		Payload->TryGetBoolField(TEXT("import_materials"), bImportMaterials);
 		Payload->TryGetNumberField(TEXT("scale_factor"), ScaleFactor);
 
-		AsyncTask(ENamedThreads::GameThread, [CorrId, SendResponse, Directory, DestPath, Extensions, bImportMaterials, ScaleFactor]()
+		// Defer to next tick to avoid Interchange task-graph recursion (same as import.fbx fix).
+		FTSTicker::GetCoreTicker().AddTicker(
+			FTickerDelegate::CreateLambda([CorrId, SendResponse, Directory, DestPath, Extensions, bImportMaterials, ScaleFactor](float) -> bool
 		{
 			// Enumerate all matching files in the directory.
 			TArray<FString> FoundFiles;
@@ -843,6 +852,7 @@ void RegisterImportExportCommands(FMCPCommandRouter& Router)
 			Data->SetArrayField(TEXT("errors"), ErrorsArray);
 
 			SendResponse(BuildImpSuccessResponse(CorrId, Data) + TEXT("\n"));
-		});
+			return false; // One-shot: remove ticker after execution
+		}), 0.0f);
 	});
 }