ue-mcp 1.2.0 → 1.2.1

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

@@ -1,230 +1,230 @@
-// Control Rig RigVM graph inspection (#774).
-//
-// list_control_rig_variables reported each graph as just a name, class and node
-// count, which is not enough to verify solver wiring - you cannot tell what a
-// rig actually computes from a node count. Reading the RigVM models, their
-// nodes, pins and links required dropping to Python.
-//
-// Translation-unit partition of FAnimationHandlers; registration lives in
-// AnimationHandlers.cpp.
-
-#include "AnimationHandlers.h"
-
-#include "HandlerUtils.h"
-
-#include "EditorAssetLibrary.h"
-#include "Engine/Blueprint.h"
-#include "UObject/UObjectGlobals.h"
-#include "UObject/UObjectHash.h"
-
-#include "RigVMModel/RigVMGraph.h"
-#include "RigVMModel/RigVMLink.h"
-#include "RigVMModel/RigVMNode.h"
-#include "RigVMModel/RigVMPin.h"
-
-namespace
-{
-	FString PinDirectionName(ERigVMPinDirection Direction)
-	{
-		switch (Direction)
-		{
-			case ERigVMPinDirection::Input:   return TEXT("Input");
-			case ERigVMPinDirection::Output:  return TEXT("Output");
-			case ERigVMPinDirection::IO:      return TEXT("IO");
-			case ERigVMPinDirection::Visible: return TEXT("Visible");
-			case ERigVMPinDirection::Hidden:  return TEXT("Hidden");
-			default:                          return TEXT("Invalid");
-		}
-	}
-
-	TSharedPtr<FJsonObject> PinToJson(URigVMPin* Pin, bool bIncludeDefaults, int32 Depth)
-	{
-		TSharedPtr<FJsonObject> Obj = MakeShared<FJsonObject>();
-		Obj->SetStringField(TEXT("name"), Pin->GetName());
-		Obj->SetStringField(TEXT("pinPath"), Pin->GetPinPath());
-		Obj->SetStringField(TEXT("cppType"), Pin->GetCPPType());
-		Obj->SetStringField(TEXT("direction"), PinDirectionName(Pin->GetDirection()));
-		Obj->SetBoolField(TEXT("isExecute"), Pin->IsExecuteContext());
-		if (bIncludeDefaults)
-		{
-			const FString Default = Pin->GetDefaultValue();
-			if (!Default.IsEmpty()) Obj->SetStringField(TEXT("defaultValue"), Default);
-		}
-
-		// Struct pins expand into sub-pins; that nesting IS the wiring for
-		// things like a transform's individual channels, so keep it, bounded.
-		const TArray<URigVMPin*>& SubPins = Pin->GetSubPins();
-		if (SubPins.Num() > 0 && Depth >= 3)
-		{
-			// Say so rather than returning a partial pin tree that reads complete.
-			Obj->SetBoolField(TEXT("subPinsTruncated"), true);
-		}
-		if (SubPins.Num() > 0 && Depth < 3)
-		{
-			TArray<TSharedPtr<FJsonValue>> SubArray;
-			for (URigVMPin* Sub : SubPins)
-			{
-				if (Sub) SubArray.Add(MakeShared<FJsonValueObject>(PinToJson(Sub, bIncludeDefaults, Depth + 1)));
-			}
-			Obj->SetArrayField(TEXT("subPins"), SubArray);
-		}
-		return Obj;
-	}
-}
-
-TSharedPtr<FJsonValue> FAnimationHandlers::ReadControlRigGraph(const TSharedPtr<FJsonObject>& Params)
-{
-	FString AssetPath;
-	if (auto Err = RequireStringAlt(Params, TEXT("assetPath"), TEXT("path"), AssetPath)) return Err;
-
-	UObject* LoadedAsset = UEditorAssetLibrary::LoadAsset(AssetPath);
-	UBlueprint* Blueprint = Cast<UBlueprint>(LoadedAsset);
-	if (!Blueprint)
-	{
-		return MCPError(FString::Printf(TEXT("Failed to load Blueprint at '%s'"), *AssetPath));
-	}
-
-	const FString GraphFilter = OptionalString(Params, TEXT("graphName"));
-	const bool bIncludePins = OptionalBool(Params, TEXT("includePins"), true);
-	const bool bIncludeDefaults = OptionalBool(Params, TEXT("includeDefaults"), true);
-	const bool bIncludeLinks = OptionalBool(Params, TEXT("includeLinks"), true);
-	const int32 NodeLimit = FMath::Max(1, OptionalInt(Params, TEXT("limit"), 200));
-
-	// ControlRigBlueprint was removed in 5.7 and the client-host interface
-	// moved, so reach the models by walking the blueprint's own subobjects for
-	// URigVMGraph. That is version-tolerant and needs no editor-only host API.
-	TArray<UObject*> Subobjects;
-#if UE_MCP_HAS_5_8_API
-	GetObjectsWithOuter(Blueprint, Subobjects, EGetObjectsFlags::IncludeNestedObjects);
-#else
-	GetObjectsWithOuter(Blueprint, Subobjects, /*bIncludeNestedObjects=*/true);
-#endif
-
-	TArray<URigVMGraph*> Models;
-	for (UObject* Object : Subobjects)
-	{
-		if (URigVMGraph* Graph = Cast<URigVMGraph>(Object))
-		{
-			// Nested graphs (function library entries, collapsed nodes) are
-			// reachable from their own outer; keep them all but de-duplicate.
-			Models.AddUnique(Graph);
-		}
-	}
-
-	if (Models.Num() == 0)
-	{
-		return MCPError(FString::Printf(
-			TEXT("'%s' has no RigVM models. Is this a Control Rig / RigVM asset? (loaded class: %s)"),
-			*AssetPath, *Blueprint->GetClass()->GetName()));
-	}
-
-	TArray<TSharedPtr<FJsonValue>> GraphsArray;
-	int32 TotalNodes = 0;
-	int32 TotalLinks = 0;
-	bool bTruncatedAny = false;
-
-	for (URigVMGraph* Graph : Models)
-	{
-		const FString GraphName = Graph->GetName();
-		if (!GraphFilter.IsEmpty() && !GraphName.Contains(GraphFilter, ESearchCase::IgnoreCase))
-		{
-			continue;
-		}
-
-		TSharedPtr<FJsonObject> GraphObj = MakeShared<FJsonObject>();
-		GraphObj->SetStringField(TEXT("name"), GraphName);
-		GraphObj->SetStringField(TEXT("path"), Graph->GetPathName());
-
-		const TArray<URigVMNode*>& Nodes = Graph->GetNodes();
-		GraphObj->SetNumberField(TEXT("nodeCount"), Nodes.Num());
-		TotalNodes += Nodes.Num();
-
-		TArray<TSharedPtr<FJsonValue>> NodesArray;
-		bool bGraphTruncated = false;
-		for (URigVMNode* Node : Nodes)
-		{
-			if (!Node) continue;
-			if (NodesArray.Num() >= NodeLimit) { bTruncatedAny = true; bGraphTruncated = true; break; }
-
-			TSharedPtr<FJsonObject> NodeObj = MakeShared<FJsonObject>();
-			NodeObj->SetStringField(TEXT("name"), Node->GetName());
-			NodeObj->SetStringField(TEXT("nodePath"), Node->GetNodePath());
-			NodeObj->SetStringField(TEXT("class"), Node->GetClass()->GetName());
-
-			if (bIncludePins)
-			{
-				TArray<TSharedPtr<FJsonValue>> PinsArray;
-				for (URigVMPin* Pin : Node->GetPins())
-				{
-					if (Pin) PinsArray.Add(MakeShared<FJsonValueObject>(PinToJson(Pin, bIncludeDefaults, 0)));
-				}
-				NodeObj->SetArrayField(TEXT("pins"), PinsArray);
-			}
-			NodesArray.Add(MakeShared<FJsonValueObject>(NodeObj));
-		}
-		GraphObj->SetArrayField(TEXT("nodes"), NodesArray);
-		// Set at the break site: deriving it from counts false-positives on a
-		// graph that merely contains a null node.
-		GraphObj->SetBoolField(TEXT("nodesTruncated"), bGraphTruncated);
-
-		if (bIncludeLinks)
-		{
-			const TArray<URigVMLink*>& Links = Graph->GetLinks();
-			TotalLinks += Links.Num();
-			TArray<TSharedPtr<FJsonValue>> LinksArray;
-			for (URigVMLink* Link : Links)
-			{
-				if (!Link) continue;
-				URigVMPin* Source = Link->GetSourcePin();
-				URigVMPin* Target = Link->GetTargetPin();
-				if (!Source || !Target) continue;
-				TSharedPtr<FJsonObject> LinkObj = MakeShared<FJsonObject>();
-				LinkObj->SetStringField(TEXT("source"), Source->GetPinPath(/*bUseNodePath=*/true));
-				LinkObj->SetStringField(TEXT("target"), Target->GetPinPath(/*bUseNodePath=*/true));
-				LinksArray.Add(MakeShared<FJsonValueObject>(LinkObj));
-			}
-			GraphObj->SetArrayField(TEXT("links"), LinksArray);
-			GraphObj->SetNumberField(TEXT("linkCount"), LinksArray.Num());
-		}
-
-		GraphsArray.Add(MakeShared<FJsonValueObject>(GraphObj));
-	}
-
-	// Member variables with their full metadata, not just a name and category.
-	TArray<TSharedPtr<FJsonValue>> VariablesArray;
-	for (const FBPVariableDescription& Var : Blueprint->NewVariables)
-	{
-		TSharedPtr<FJsonObject> VarObj = MakeShared<FJsonObject>();
-		VarObj->SetStringField(TEXT("name"), Var.VarName.ToString());
-		VarObj->SetStringField(TEXT("type"), Var.VarType.PinCategory.ToString());
-		VarObj->SetStringField(TEXT("subType"), Var.VarType.PinSubCategory.ToString());
-		if (Var.VarType.PinSubCategoryObject.IsValid())
-		{
-			VarObj->SetStringField(TEXT("subTypeObject"), Var.VarType.PinSubCategoryObject->GetPathName());
-		}
-		VarObj->SetBoolField(TEXT("isArray"), Var.VarType.IsArray());
-		VarObj->SetStringField(TEXT("defaultValue"), Var.DefaultValue);
-		VarObj->SetBoolField(TEXT("isPublic"), (Var.PropertyFlags & CPF_BlueprintVisible) != 0);
-		VarObj->SetBoolField(TEXT("readOnly"), (Var.PropertyFlags & CPF_BlueprintReadOnly) != 0);
-		VariablesArray.Add(MakeShared<FJsonValueObject>(VarObj));
-	}
-
-	auto Result = MCPSuccess();
-	Result->SetStringField(TEXT("assetPath"), AssetPath);
-	Result->SetStringField(TEXT("name"), Blueprint->GetName());
-	if (Blueprint->ParentClass)
-	{
-		Result->SetStringField(TEXT("parentClass"), Blueprint->ParentClass->GetName());
-	}
-	Result->SetArrayField(TEXT("models"), GraphsArray);
-	Result->SetNumberField(TEXT("modelCount"), GraphsArray.Num());
-	Result->SetNumberField(TEXT("totalNodes"), TotalNodes);
-	Result->SetNumberField(TEXT("totalLinks"), TotalLinks);
-	Result->SetArrayField(TEXT("variables"), VariablesArray);
-	Result->SetNumberField(TEXT("variableCount"), VariablesArray.Num());
-	if (bTruncatedAny)
-	{
-		Result->SetStringField(TEXT("note"), TEXT("Some graphs hit the node limit; raise 'limit' or narrow with 'graphName'."));
-	}
-	return MCPResult(Result);
-}
+// Control Rig RigVM graph inspection (#774).
+//
+// list_control_rig_variables reported each graph as just a name, class and node
+// count, which is not enough to verify solver wiring - you cannot tell what a
+// rig actually computes from a node count. Reading the RigVM models, their
+// nodes, pins and links required dropping to Python.
+//
+// Translation-unit partition of FAnimationHandlers; registration lives in
+// AnimationHandlers.cpp.
+
+#include "AnimationHandlers.h"
+
+#include "HandlerUtils.h"
+
+#include "EditorAssetLibrary.h"
+#include "Engine/Blueprint.h"
+#include "UObject/UObjectGlobals.h"
+#include "UObject/UObjectHash.h"
+
+#include "RigVMModel/RigVMGraph.h"
+#include "RigVMModel/RigVMLink.h"
+#include "RigVMModel/RigVMNode.h"
+#include "RigVMModel/RigVMPin.h"
+
+namespace
+{
+	FString PinDirectionName(ERigVMPinDirection Direction)
+	{
+		switch (Direction)
+		{
+			case ERigVMPinDirection::Input:   return TEXT("Input");
+			case ERigVMPinDirection::Output:  return TEXT("Output");
+			case ERigVMPinDirection::IO:      return TEXT("IO");
+			case ERigVMPinDirection::Visible: return TEXT("Visible");
+			case ERigVMPinDirection::Hidden:  return TEXT("Hidden");
+			default:                          return TEXT("Invalid");
+		}
+	}
+
+	TSharedPtr<FJsonObject> PinToJson(URigVMPin* Pin, bool bIncludeDefaults, int32 Depth)
+	{
+		TSharedPtr<FJsonObject> Obj = MakeShared<FJsonObject>();
+		Obj->SetStringField(TEXT("name"), Pin->GetName());
+		Obj->SetStringField(TEXT("pinPath"), Pin->GetPinPath());
+		Obj->SetStringField(TEXT("cppType"), Pin->GetCPPType());
+		Obj->SetStringField(TEXT("direction"), PinDirectionName(Pin->GetDirection()));
+		Obj->SetBoolField(TEXT("isExecute"), Pin->IsExecuteContext());
+		if (bIncludeDefaults)
+		{
+			const FString Default = Pin->GetDefaultValue();
+			if (!Default.IsEmpty()) Obj->SetStringField(TEXT("defaultValue"), Default);
+		}
+
+		// Struct pins expand into sub-pins; that nesting IS the wiring for
+		// things like a transform's individual channels, so keep it, bounded.
+		const TArray<URigVMPin*>& SubPins = Pin->GetSubPins();
+		if (SubPins.Num() > 0 && Depth >= 3)
+		{
+			// Say so rather than returning a partial pin tree that reads complete.
+			Obj->SetBoolField(TEXT("subPinsTruncated"), true);
+		}
+		if (SubPins.Num() > 0 && Depth < 3)
+		{
+			TArray<TSharedPtr<FJsonValue>> SubArray;
+			for (URigVMPin* Sub : SubPins)
+			{
+				if (Sub) SubArray.Add(MakeShared<FJsonValueObject>(PinToJson(Sub, bIncludeDefaults, Depth + 1)));
+			}
+			Obj->SetArrayField(TEXT("subPins"), SubArray);
+		}
+		return Obj;
+	}
+}
+
+TSharedPtr<FJsonValue> FAnimationHandlers::ReadControlRigGraph(const TSharedPtr<FJsonObject>& Params)
+{
+	FString AssetPath;
+	if (auto Err = RequireStringAlt(Params, TEXT("assetPath"), TEXT("path"), AssetPath)) return Err;
+
+	UObject* LoadedAsset = UEditorAssetLibrary::LoadAsset(AssetPath);
+	UBlueprint* Blueprint = Cast<UBlueprint>(LoadedAsset);
+	if (!Blueprint)
+	{
+		return MCPError(FString::Printf(TEXT("Failed to load Blueprint at '%s'"), *AssetPath));
+	}
+
+	const FString GraphFilter = OptionalString(Params, TEXT("graphName"));
+	const bool bIncludePins = OptionalBool(Params, TEXT("includePins"), true);
+	const bool bIncludeDefaults = OptionalBool(Params, TEXT("includeDefaults"), true);
+	const bool bIncludeLinks = OptionalBool(Params, TEXT("includeLinks"), true);
+	const int32 NodeLimit = FMath::Max(1, OptionalInt(Params, TEXT("limit"), 200));
+
+	// ControlRigBlueprint was removed in 5.7 and the client-host interface
+	// moved, so reach the models by walking the blueprint's own subobjects for
+	// URigVMGraph. That is version-tolerant and needs no editor-only host API.
+	TArray<UObject*> Subobjects;
+#if UE_MCP_HAS_5_8_API
+	GetObjectsWithOuter(Blueprint, Subobjects, EGetObjectsFlags::IncludeNestedObjects);
+#else
+	GetObjectsWithOuter(Blueprint, Subobjects, /*bIncludeNestedObjects=*/true);
+#endif
+
+	TArray<URigVMGraph*> Models;
+	for (UObject* Object : Subobjects)
+	{
+		if (URigVMGraph* Graph = Cast<URigVMGraph>(Object))
+		{
+			// Nested graphs (function library entries, collapsed nodes) are
+			// reachable from their own outer; keep them all but de-duplicate.
+			Models.AddUnique(Graph);
+		}
+	}
+
+	if (Models.Num() == 0)
+	{
+		return MCPError(FString::Printf(
+			TEXT("'%s' has no RigVM models. Is this a Control Rig / RigVM asset? (loaded class: %s)"),
+			*AssetPath, *Blueprint->GetClass()->GetName()));
+	}
+
+	TArray<TSharedPtr<FJsonValue>> GraphsArray;
+	int32 TotalNodes = 0;
+	int32 TotalLinks = 0;
+	bool bTruncatedAny = false;
+
+	for (URigVMGraph* Graph : Models)
+	{
+		const FString GraphName = Graph->GetName();
+		if (!GraphFilter.IsEmpty() && !GraphName.Contains(GraphFilter, ESearchCase::IgnoreCase))
+		{
+			continue;
+		}
+
+		TSharedPtr<FJsonObject> GraphObj = MakeShared<FJsonObject>();
+		GraphObj->SetStringField(TEXT("name"), GraphName);
+		GraphObj->SetStringField(TEXT("path"), Graph->GetPathName());
+
+		const TArray<URigVMNode*>& Nodes = Graph->GetNodes();
+		GraphObj->SetNumberField(TEXT("nodeCount"), Nodes.Num());
+		TotalNodes += Nodes.Num();
+
+		TArray<TSharedPtr<FJsonValue>> NodesArray;
+		bool bGraphTruncated = false;
+		for (URigVMNode* Node : Nodes)
+		{
+			if (!Node) continue;
+			if (NodesArray.Num() >= NodeLimit) { bTruncatedAny = true; bGraphTruncated = true; break; }
+
+			TSharedPtr<FJsonObject> NodeObj = MakeShared<FJsonObject>();
+			NodeObj->SetStringField(TEXT("name"), Node->GetName());
+			NodeObj->SetStringField(TEXT("nodePath"), Node->GetNodePath());
+			NodeObj->SetStringField(TEXT("class"), Node->GetClass()->GetName());
+
+			if (bIncludePins)
+			{
+				TArray<TSharedPtr<FJsonValue>> PinsArray;
+				for (URigVMPin* Pin : Node->GetPins())
+				{
+					if (Pin) PinsArray.Add(MakeShared<FJsonValueObject>(PinToJson(Pin, bIncludeDefaults, 0)));
+				}
+				NodeObj->SetArrayField(TEXT("pins"), PinsArray);
+			}
+			NodesArray.Add(MakeShared<FJsonValueObject>(NodeObj));
+		}
+		GraphObj->SetArrayField(TEXT("nodes"), NodesArray);
+		// Set at the break site: deriving it from counts false-positives on a
+		// graph that merely contains a null node.
+		GraphObj->SetBoolField(TEXT("nodesTruncated"), bGraphTruncated);
+
+		if (bIncludeLinks)
+		{
+			const TArray<URigVMLink*>& Links = Graph->GetLinks();
+			TotalLinks += Links.Num();
+			TArray<TSharedPtr<FJsonValue>> LinksArray;
+			for (URigVMLink* Link : Links)
+			{
+				if (!Link) continue;
+				URigVMPin* Source = Link->GetSourcePin();
+				URigVMPin* Target = Link->GetTargetPin();
+				if (!Source || !Target) continue;
+				TSharedPtr<FJsonObject> LinkObj = MakeShared<FJsonObject>();
+				LinkObj->SetStringField(TEXT("source"), Source->GetPinPath(/*bUseNodePath=*/true));
+				LinkObj->SetStringField(TEXT("target"), Target->GetPinPath(/*bUseNodePath=*/true));
+				LinksArray.Add(MakeShared<FJsonValueObject>(LinkObj));
+			}
+			GraphObj->SetArrayField(TEXT("links"), LinksArray);
+			GraphObj->SetNumberField(TEXT("linkCount"), LinksArray.Num());
+		}
+
+		GraphsArray.Add(MakeShared<FJsonValueObject>(GraphObj));
+	}
+
+	// Member variables with their full metadata, not just a name and category.
+	TArray<TSharedPtr<FJsonValue>> VariablesArray;
+	for (const FBPVariableDescription& Var : Blueprint->NewVariables)
+	{
+		TSharedPtr<FJsonObject> VarObj = MakeShared<FJsonObject>();
+		VarObj->SetStringField(TEXT("name"), Var.VarName.ToString());
+		VarObj->SetStringField(TEXT("type"), Var.VarType.PinCategory.ToString());
+		VarObj->SetStringField(TEXT("subType"), Var.VarType.PinSubCategory.ToString());
+		if (Var.VarType.PinSubCategoryObject.IsValid())
+		{
+			VarObj->SetStringField(TEXT("subTypeObject"), Var.VarType.PinSubCategoryObject->GetPathName());
+		}
+		VarObj->SetBoolField(TEXT("isArray"), Var.VarType.IsArray());
+		VarObj->SetStringField(TEXT("defaultValue"), Var.DefaultValue);
+		VarObj->SetBoolField(TEXT("isPublic"), (Var.PropertyFlags & CPF_BlueprintVisible) != 0);
+		VarObj->SetBoolField(TEXT("readOnly"), (Var.PropertyFlags & CPF_BlueprintReadOnly) != 0);
+		VariablesArray.Add(MakeShared<FJsonValueObject>(VarObj));
+	}
+
+	auto Result = MCPSuccess();
+	Result->SetStringField(TEXT("assetPath"), AssetPath);
+	Result->SetStringField(TEXT("name"), Blueprint->GetName());
+	if (Blueprint->ParentClass)
+	{
+		Result->SetStringField(TEXT("parentClass"), Blueprint->ParentClass->GetName());
+	}
+	Result->SetArrayField(TEXT("models"), GraphsArray);
+	Result->SetNumberField(TEXT("modelCount"), GraphsArray.Num());
+	Result->SetNumberField(TEXT("totalNodes"), TotalNodes);
+	Result->SetNumberField(TEXT("totalLinks"), TotalLinks);
+	Result->SetArrayField(TEXT("variables"), VariablesArray);
+	Result->SetNumberField(TEXT("variableCount"), VariablesArray.Num());
+	if (bTruncatedAny)
+	{
+		Result->SetStringField(TEXT("note"), TEXT("Some graphs hit the node limit; raise 'limit' or narrow with 'graphName'."));
+	}
+	return MCPResult(Result);
+}

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

@@ -75,32 +75,10 @@
 #include "AI/Navigation/NavCollisionBase.h"
 
 // ─── Protected mount guardrail ──────────────────────────────────────────
-// Engine-shipped content (/Engine/, /Script/, /Memory/, /Temp/) and Verse
-// runtime classes must never be mutated through the bridge. UE's
-// UEditorAssetLibrary::DeleteAsset will happily destroy files under
-// <engineRoot>/Engine/Content/ if not stopped - verified the hard way.
-// Apply this check to every handler that deletes, moves, or renames an
-// asset. Plugin content roots (mounted under /<PluginName>/) are NOT
-// protected here; per-project plugin content is expected to be writable.
+// The rule itself is MCPIsProtectedAssetPath in HandlerUtils.h, shared by
+// every asset translation unit so the write paths cannot drift apart.
 namespace
 {
-	bool IsProtectedAssetPath(const FString& Path)
-	{
-		FString P = Path;
-		P.TrimStartAndEndInline();
-		if (P.IsEmpty()) return false;
-		// Tolerate leading whitespace and the surface form (no leading slash).
-		if (!P.StartsWith(TEXT("/"))) P = TEXT("/") + P;
-		const FString L = P.ToLower();
-		if (L.StartsWith(TEXT("/engine/"))) return true;
-		if (L.StartsWith(TEXT("/script/"))) return true;
-		if (L.StartsWith(TEXT("/memory/"))) return true;
-		if (L.StartsWith(TEXT("/temp/"))) return true;
-		// Verse runtime objects surface as /Script/CoreUObject.* etc.
-		if (L.Contains(TEXT("/script/"))) return true;
-		return false;
-	}
-
 	TSharedPtr<FJsonValue> MakeProtectedPathError(const FString& Path)
 	{
 		return MCPError(FString::Printf(
@@ -1488,8 +1466,8 @@ TSharedPtr<FJsonValue> FAssetHandlers::RenameAsset(const TSharedPtr<FJsonObject>
 		return MCPError(TEXT("Missing 'sourcePath'+'destinationPath' or 'assetPath'+'newName'"));
 	}
 
-	if (IsProtectedAssetPath(SourcePath)) return MakeProtectedPathError(SourcePath);
-	if (IsProtectedAssetPath(DestPath))   return MakeProtectedPathError(DestPath);
+	if (MCPIsProtectedAssetPath(SourcePath)) return MakeProtectedPathError(SourcePath);
+	if (MCPIsProtectedAssetPath(DestPath))   return MakeProtectedPathError(DestPath);
 
 	// Idempotency: if already at destination, no-op.
 	if (SourcePath == DestPath)
@@ -1697,7 +1675,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::DeleteAsset(const TSharedPtr<FJsonObject>
 	FString AssetPath;
 	if (auto Err = RequireStringAlt(Params, TEXT("path"), TEXT("assetPath"), AssetPath)) return Err;
 
-	if (IsProtectedAssetPath(AssetPath)) return MakeProtectedPathError(AssetPath);
+	if (MCPIsProtectedAssetPath(AssetPath)) return MakeProtectedPathError(AssetPath);
 
 	const bool bForce = OptionalBool(Params, TEXT("force"), false);
 
@@ -1763,7 +1741,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::DeleteAssetBatch(const TSharedPtr<FJsonOb
 		TSharedPtr<FJsonObject> Entry = MakeShared<FJsonObject>();
 		Entry->SetStringField(TEXT("path"), Path);
 
-		if (IsProtectedAssetPath(Path))
+		if (MCPIsProtectedAssetPath(Path))
 		{
 			Entry->SetStringField(TEXT("status"), TEXT("protected"));
 			Entry->SetStringField(TEXT("reason"), TEXT("Engine/Script/Memory/Temp mounts are read-only via the bridge"));
@@ -1892,7 +1870,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::BulkRename(const TSharedPtr<FJsonObject>&
 			continue;
 		}
 
-		if (IsProtectedAssetPath(SourcePath) || IsProtectedAssetPath(NewPackagePath))
+		if (MCPIsProtectedAssetPath(SourcePath) || MCPIsProtectedAssetPath(NewPackagePath))
 		{
 			Record->SetStringField(TEXT("status"), TEXT("protected"));
 			Record->SetStringField(TEXT("reason"), TEXT("Engine/Script/Memory/Temp mounts are read-only via the bridge"));
@@ -2555,8 +2533,8 @@ TSharedPtr<FJsonValue> FAssetHandlers::MoveFolder(const TSharedPtr<FJsonObject>&
 	SourcePath.RemoveFromEnd(TEXT("/"));
 	DestinationPath.RemoveFromEnd(TEXT("/"));
 
-	if (IsProtectedAssetPath(SourcePath))      return MakeProtectedPathError(SourcePath);
-	if (IsProtectedAssetPath(DestinationPath)) return MakeProtectedPathError(DestinationPath);
+	if (MCPIsProtectedAssetPath(SourcePath))      return MakeProtectedPathError(SourcePath);
+	if (MCPIsProtectedAssetPath(DestinationPath)) return MakeProtectedPathError(DestinationPath);
 
 	// Scan source path to discover all assets
 	IAssetRegistry& AR = FModuleManager::LoadModuleChecked<FAssetRegistryModule>(TEXT("AssetRegistry")).Get();
@@ -2738,7 +2716,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::DeleteFolder(const TSharedPtr<FJsonObject
 			continue;
 		}
 
-		if (IsProtectedAssetPath(Norm))
+		if (MCPIsProtectedAssetPath(Norm))
 		{
 			Entry->SetStringField(TEXT("status"), TEXT("failed"));
 			Entry->SetStringField(TEXT("reason"), TEXT("protected_path"));

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

@@ -64,26 +64,6 @@ namespace
 		bool PassedPreflight() const { return Error.IsEmpty(); }
 	};
 
-	// Mirrors IsProtectedAssetPath in AssetHandlers.cpp, which is file-local to
-	// that translation unit. Keep the two rule sets identical: a write path that
-	// enforces a weaker rule than its neighbours is how engine content gets
-	// mutated through the bridge.
-	bool IsProtectedBulkAssetPath(const FString& Path)
-	{
-		FString Normalized = Path;
-		Normalized.TrimStartAndEndInline();
-		if (Normalized.IsEmpty()) return false;
-		if (!Normalized.StartsWith(TEXT("/"))) Normalized = TEXT("/") + Normalized;
-		const FString Lower = Normalized.ToLower();
-		if (Lower.StartsWith(TEXT("/engine/"))) return true;
-		if (Lower.StartsWith(TEXT("/script/"))) return true;
-		if (Lower.StartsWith(TEXT("/memory/"))) return true;
-		if (Lower.StartsWith(TEXT("/temp/"))) return true;
-		// Verse runtime objects surface as /Script/CoreUObject.* etc.
-		if (Lower.Contains(TEXT("/script/"))) return true;
-		return false;
-	}
-
 	TSharedPtr<FJsonObject> MakePropertyReadback(
 		const FPreparedPropertyWrite& Prepared,
 		const TSharedPtr<FJsonValue>& ActualValue,
@@ -198,7 +178,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::BulkSetAssetProperties(const TSharedPtr<F
 		}
 		PreparedAsset.AssetPath = AssetPath;
 
-		if (IsProtectedBulkAssetPath(AssetPath))
+		if (MCPIsProtectedAssetPath(AssetPath))
 		{
 			RejectItem(BulkStatusProtected, FString::Printf(
 				TEXT("Refusing to mutate protected mount: %s. Engine, /Script/, /Memory/, /Temp/ are read-only via the bridge."),

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

@@ -50,21 +50,6 @@ struct FPreparedUpsertItem
 	TArray<FPreparedProperty> Properties;
 };
 
-bool IsProtectedAssetPath(const FString& Path)
-{
-	FString Normalized = Path;
-	Normalized.TrimStartAndEndInline();
-	if (!Normalized.StartsWith(TEXT("/")))
-	{
-		Normalized = TEXT("/") + Normalized;
-	}
-	const FString Lower = Normalized.ToLower();
-	return Lower.StartsWith(TEXT("/engine/"))
-		|| Lower.StartsWith(TEXT("/script/"))
-		|| Lower.StartsWith(TEXT("/memory/"))
-		|| Lower.StartsWith(TEXT("/temp/"));
-}
-
 UClass* ResolveDataAssetClass(const FString& ClassName)
 {
 	UClass* DataClass = nullptr;
@@ -144,7 +129,7 @@ bool ParseAssetIdentity(
 	}
 
 	const FString LongPackageName = OutItem.PackagePath + TEXT("/") + OutItem.Name;
-	if (IsProtectedAssetPath(LongPackageName))
+	if (MCPIsProtectedAssetPath(LongPackageName))
 	{
 		OutError = FString::Printf(TEXT("Refusing to mutate protected package '%s'"), *LongPackageName);
 		return false;
@@ -357,7 +342,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::BulkRestoreDataAssets(const TSharedPtr<FJ
 			FString AssetPath;
 			const TSharedPtr<FJsonObject>* Properties = nullptr;
 			if (!(*Item)->TryGetStringField(TEXT("assetPath"), AssetPath)
-				|| IsProtectedAssetPath(AssetPath)
+				|| MCPIsProtectedAssetPath(AssetPath)
 				|| !(*Item)->TryGetObjectField(TEXT("properties"), Properties)
 				|| !Properties
 				|| !Properties->IsValid())
@@ -414,7 +399,7 @@ TSharedPtr<FJsonValue> FAssetHandlers::BulkRestoreDataAssets(const TSharedPtr<FJ
 			FString AssetPath;
 			if (!(*CreatedAssetPaths)[PathIndex].IsValid()
 				|| !(*CreatedAssetPaths)[PathIndex]->TryGetString(AssetPath)
-				|| IsProtectedAssetPath(AssetPath))
+				|| MCPIsProtectedAssetPath(AssetPath))
 			{
 				Errors.Add(MakeShared<FJsonValueString>(FString::Printf(TEXT("createdAssetPaths[%d] is invalid"), PathIndex)));
 				continue;