ue-mcp 0.4.28 → 0.4.30

package/dist/tools/material.js

@@ -27,8 +27,8 @@ export const materialTool = categoryTool("material", "Materials: create, read, p
 - set_blend_mode: Set blend mode. Params: assetPath, blendMode (Opaque|Masked|Translucent|Additive|Modulate|AlphaComposite|AlphaHoldout)
 - set_base_color: Set base color. Params: assetPath, color {r,g,b,a?}
 - connect_texture: Connect texture to property. Params: materialPath, texturePath, property
-- add_expression: Add expression node. Params: materialPath, expressionType, x?, y?, properties?
-- connect_expressions: Wire two expressions. Params: materialPath, sourceExpression, sourceOutput?, targetExpression, targetInput?
+- add_expression: Add expression node. Returns nodeId (use as sourceExpression/targetExpression). Params: materialPath, expressionType, name?, parameterName?, positionX?, positionY?
+- connect_expressions: Wire two expressions. Use nodeId from add_expression, or expression name/index/class. Params: materialPath, sourceExpression, sourceOutput?, targetExpression, targetInput?
 - connect_to_property: Wire expression to material output. Params: materialPath, expressionName, outputName?, property (BaseColor|Normal|Roughness|Metallic|Emissive|...)
 - list_expressions: List expression nodes. Params: materialPath
 - delete_expression: Remove expression. Params: materialPath, expressionName

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ue-mcp",
-  "version": "0.4.28",
+  "version": "0.4.30",
   "description": "Unreal Engine MCP server — 19 tools, 300+ actions for AI-driven editor control",
   "type": "module",
   "main": "dist/index.js",

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/Handlers/AnimationHandlers.cpp

@@ -35,9 +35,9 @@
 #include "AnimStateTransitionNode.h"
 #include "AnimStateEntryNode.h"
 #include "AnimationStateMachineGraph.h"
+#include "AnimGraphNode_AssetPlayerBase.h"
 #include "AnimGraphNode_SequencePlayer.h"
 #include "AnimGraphNode_BlendSpacePlayer.h"
-#include "AnimGraphNode_Inertialization.h"
 #include "Kismet2/BlueprintEditorUtils.h"
 #include "Kismet2/KismetEditorUtilities.h"
 
@@ -1697,14 +1697,19 @@ TSharedPtr<FJsonValue> FAnimationHandlers::CreateStateMachine(const TSharedPtr<F
 
 	// Create the state machine container node in the AnimGraph
 	UAnimGraphNode_StateMachine* SMNode = NewObject<UAnimGraphNode_StateMachine>(TargetGraph);
-	SMNode->GetStateMachineNode()->SetStateMachineName(FName(*Name));
 	TargetGraph->AddNode(SMNode, false, false);
 	SMNode->CreateNewGuid();
-	SMNode->PostPlacedNewNode();
+	SMNode->PostPlacedNewNode();  // This creates the EditorStateMachineGraph sub-graph
 	SMNode->AllocateDefaultPins();
 	SMNode->NodePosX = 200;
 	SMNode->NodePosY = 0;
 
+	// Rename the state machine graph to the desired name
+	if (SMNode->EditorStateMachineGraph)
+	{
+		SMNode->EditorStateMachineGraph->Rename(*Name);
+	}
+
 	CompileAndSave(AnimBP);
 
 	Result->SetStringField(TEXT("assetPath"), AssetPath);
@@ -1992,7 +1997,7 @@ TSharedPtr<FJsonValue> FAnimationHandlers::SetStateAnimation(const TSharedPtr<FJ
 			SeqPlayer->PostPlacedNewNode();
 			SeqPlayer->AllocateDefaultPins();
 		}
-		SeqPlayer->GetSequencePlayerNode()->SetSequence(Seq);
+		SeqPlayer->SetAnimationAsset(Seq);
 	}
 	else if (UBlendSpace* BS = Cast<UBlendSpace>(AnimAsset))
 	{
@@ -2004,7 +2009,7 @@ TSharedPtr<FJsonValue> FAnimationHandlers::SetStateAnimation(const TSharedPtr<FJ
 			BSPlayer->PostPlacedNewNode();
 			BSPlayer->AllocateDefaultPins();
 		}
-		BSPlayer->GetBlendSpacePlayerNode()->SetBlendSpace(BS);
+		BSPlayer->SetAnimationAsset(BS);
 	}
 	else
 	{
@@ -2177,18 +2182,11 @@ TSharedPtr<FJsonValue> FAnimationHandlers::ReadStateMachine(const TSharedPtr<FJs
 			{
 				for (UEdGraphNode* Inner : State->BoundGraph->Nodes)
 				{
-					if (UAnimGraphNode_SequencePlayer* SP = Cast<UAnimGraphNode_SequencePlayer>(Inner))
-					{
-						if (UAnimSequence* Seq = SP->GetSequencePlayerNode()->GetSequence())
-						{
-							StateObj->SetStringField(TEXT("animAsset"), Seq->GetPathName());
-						}
-					}
-					else if (UAnimGraphNode_BlendSpacePlayer* BSP = Cast<UAnimGraphNode_BlendSpacePlayer>(Inner))
+					if (UAnimGraphNode_AssetPlayerBase* AssetNode = Cast<UAnimGraphNode_AssetPlayerBase>(Inner))
 					{
-						if (UBlendSpace* BS = BSP->GetBlendSpacePlayerNode()->GetBlendSpace())
+						if (UAnimationAsset* Asset = AssetNode->GetAnimationAsset())
 						{
-							StateObj->SetStringField(TEXT("animAsset"), BS->GetPathName());
+							StateObj->SetStringField(TEXT("animAsset"), Asset->GetPathName());
 						}
 					}
 				}

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/Handlers/MaterialHandlers.cpp

@@ -615,11 +615,67 @@ TSharedPtr<FJsonValue> FMaterialHandlers::AddMaterialExpression(const TSharedPtr
 	UMaterialExpression* NewExpression = NewObject<UMaterialExpression>(Material, ExpressionClass);
 	Material->GetExpressionCollection().AddExpression(NewExpression);
 
+	// Apply optional properties
+	FString ExpressionName;
+	if (Params->TryGetStringField(TEXT("name"), ExpressionName) || Params->TryGetStringField(TEXT("expressionName"), ExpressionName))
+	{
+		NewExpression->Desc = ExpressionName;
+	}
+
+	// Set parameter name for parameter expressions
+	FString ParameterName;
+	if (Params->TryGetStringField(TEXT("parameterName"), ParameterName))
+	{
+		if (UMaterialExpressionScalarParameter* ScalarParam = Cast<UMaterialExpressionScalarParameter>(NewExpression))
+		{
+			ScalarParam->ParameterName = FName(*ParameterName);
+		}
+		else if (UMaterialExpressionVectorParameter* VectorParam = Cast<UMaterialExpressionVectorParameter>(NewExpression))
+		{
+			VectorParam->ParameterName = FName(*ParameterName);
+		}
+		else if (UMaterialExpressionTextureObjectParameter* TexParam = Cast<UMaterialExpressionTextureObjectParameter>(NewExpression))
+		{
+			TexParam->ParameterName = FName(*ParameterName);
+		}
+		// If name not set via Desc, use parameterName as the description too
+		if (NewExpression->Desc.IsEmpty())
+		{
+			NewExpression->Desc = ParameterName;
+		}
+	}
+
+	// Set position
+	double PosX = 0, PosY = 0;
+	if (Params->TryGetNumberField(TEXT("positionX"), PosX))
+	{
+		NewExpression->MaterialExpressionEditorX = static_cast<int32>(PosX);
+	}
+	if (Params->TryGetNumberField(TEXT("positionY"), PosY))
+	{
+		NewExpression->MaterialExpressionEditorY = static_cast<int32>(PosY);
+	}
+
 	Material->PostEditChange();
-	Material->MarkPackageDirty();
+
+	// Save the package so subsequent list/connect calls see the expression
+	UPackage* Package = Material->GetOutermost();
+	if (Package)
+	{
+		Package->MarkPackageDirty();
+		FString PackageFileName = FPackageName::LongPackageNameToFilename(Package->GetName(), FPackageName::GetAssetPackageExtension());
+		FSavePackageArgs SaveArgs;
+		SaveArgs.TopLevelFlags = RF_Standalone;
+		UPackage::SavePackage(Package, nullptr, *PackageFileName, SaveArgs);
+	}
+
+	// Return the index as nodeId for use with connect_expressions and other operations
+	int32 NodeIndex = Material->GetExpressions().Num() - 1;
 
 	Result->SetStringField(TEXT("expressionType"), ExpressionType);
 	Result->SetStringField(TEXT("expressionClass"), NewExpression->GetClass()->GetName());
+	Result->SetStringField(TEXT("nodeId"), FString::FromInt(NodeIndex));
+	Result->SetStringField(TEXT("description"), NewExpression->GetDescription());
 	Result->SetStringField(TEXT("materialPath"), Material->GetPathName());
 	Result->SetNumberField(TEXT("expressionCount"), Material->GetExpressions().Num());
 	Result->SetBoolField(TEXT("success"), true);
@@ -648,16 +704,30 @@ TSharedPtr<FJsonValue> FMaterialHandlers::ListMaterialExpressions(const TSharedP
 	}
 
 	TArray<TSharedPtr<FJsonValue>> ExpressionsArray;
-	for (UMaterialExpression* Expression : Material->GetExpressions())
+	auto Expressions = Material->GetExpressions();
+	for (int32 i = 0; i < Expressions.Num(); i++)
 	{
+		UMaterialExpression* Expression = Expressions[i];
 		if (!Expression) continue;
 
 		TSharedPtr<FJsonObject> ExprObj = MakeShared<FJsonObject>();
+		ExprObj->SetStringField(TEXT("nodeId"), FString::FromInt(i));
 		ExprObj->SetStringField(TEXT("class"), Expression->GetClass()->GetName());
 		ExprObj->SetStringField(TEXT("description"), Expression->GetDescription());
+		ExprObj->SetStringField(TEXT("name"), Expression->Desc);
 		ExprObj->SetNumberField(TEXT("positionX"), Expression->MaterialExpressionEditorX);
 		ExprObj->SetNumberField(TEXT("positionY"), Expression->MaterialExpressionEditorY);
 
+		// Include parameter name if applicable
+		if (UMaterialExpressionScalarParameter* SP = Cast<UMaterialExpressionScalarParameter>(Expression))
+		{
+			ExprObj->SetStringField(TEXT("parameterName"), SP->ParameterName.ToString());
+		}
+		else if (UMaterialExpressionVectorParameter* VP = Cast<UMaterialExpressionVectorParameter>(Expression))
+		{
+			ExprObj->SetStringField(TEXT("parameterName"), VP->ParameterName.ToString());
+		}
+
 		ExpressionsArray.Add(MakeShared<FJsonValueObject>(ExprObj));
 	}