ue-mcp 0.4.29 → 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.29",
+  "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/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));
 	}