ue-mcp 1.3.10-beta.13 → 1.3.10-beta.14

package/dist/tool-counts.json

@@ -34,6 +34,6 @@
   },
   "nativeToolsets": 52,
   "nativeToolActions": 830,
-  "generatedAt": "2026-10-06T00:55:16.577Z",
-  "version": "1.3.10-beta.13"
+  "generatedAt": "2026-10-06T12:57:09.853Z",
+  "version": "1.3.10-beta.14"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ue-mcp",
-  "version": "1.3.10-beta.13",
+  "version": "1.3.10-beta.14",
   "description": "Unreal Engine MCP server - 26 tools, 1972+ actions for AI-driven editor control",
   "type": "module",
   "main": "dist/index.js",

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/Handlers/PCG/PCGHandlers.cpp

@@ -786,9 +786,7 @@ TSharedPtr<FJsonValue> FPCGHandlers::ConnectPCGNodes(const TSharedPtr<FJsonObjec
 	return MCPResult(Result);
 }
 
-// #346: per-edge removal. UPCGGraph has no public RemoveEdge; do it manually by
-// finding the matching UPCGEdge on the source pin and clearing it from both
-// pins' Edges arrays, then PostEditChange + save.
+// #346: per-edge removal through UPCGGraph::RemoveEdge, so the graph recompiles.
 TSharedPtr<FJsonValue> FPCGHandlers::DisconnectPCGNodes(const TSharedPtr<FJsonObject>& Params)
 {
 	FString AssetPath;
@@ -842,21 +840,27 @@ TSharedPtr<FJsonValue> FPCGHandlers::DisconnectPCGNodes(const TSharedPtr<FJsonOb
 		if (!OutPin) continue;
 		if (!SourcePinLabel.IsEmpty() && OutPin->Properties.Label != FName(*SourcePinLabel)) continue;
 
-		// Mutating the Edges array during iteration is unsafe; collect first.
-		TArray<UPCGEdge*> ToRemove;
+		// RemoveEdge mutates the Edges array; collect the target labels first.
+		TArray<FName> ToRemove;
 		for (const TObjectPtr<UPCGEdge>& Edge : OutPin->Edges)
 		{
 			if (!Edge || !Edge->OutputPin) continue;
 			UPCGNode* EdgeDstNode = Edge->OutputPin->Node;
 			if (EdgeDstNode != TargetNode) continue;
 			if (!TargetPinLabel.IsEmpty() && Edge->OutputPin->Properties.Label != FName(*TargetPinLabel)) continue;
-			ToRemove.Add(Edge);
+			ToRemove.Add(Edge->OutputPin->Properties.Label);
 		}
-		for (UPCGEdge* Edge : ToRemove)
+		for (const FName& TargetLabel : ToRemove)
 		{
-			if (!Edge) continue;
+			// The graph's own RemoveEdge notifies a structural change, which recompiles the graph and
+			// drops cached results; editing the pins' Edges arrays directly left both stale.
+			if (!Graph->RemoveEdge(SourceNode, OutPin->Properties.Label, TargetNode, TargetLabel))
+			{
+				return MCPError(FString::Printf(TEXT("UPCGGraph::RemoveEdge refused %s.%s -> %s.%s after %d edge(s) were removed"),
+					*SourceNodeName, *OutPin->Properties.Label.ToString(), *TargetNodeName, *TargetLabel.ToString(), RemovedCount));
+			}
 			const FString CutSourcePin = OutPin->Properties.Label.ToString();
-			const FString CutTargetPin = Edge->OutputPin ? Edge->OutputPin->Properties.Label.ToString() : FString();
+			const FString CutTargetPin = TargetLabel.ToString();
 			TSharedPtr<FJsonObject> Cut = MakeShared<FJsonObject>();
 			Cut->SetStringField(TEXT("sourcePinLabel"), CutSourcePin);
 			Cut->SetStringField(TEXT("targetPinLabel"), CutTargetPin);
@@ -866,11 +870,6 @@ TSharedPtr<FJsonValue> FPCGHandlers::DisconnectPCGNodes(const TSharedPtr<FJsonOb
 				FirstRemovedSourcePin = CutSourcePin;
 				FirstRemovedTargetPin = CutTargetPin;
 			}
-			if (Edge->OutputPin)
-			{
-				Edge->OutputPin->Edges.Remove(Edge);
-			}
-			OutPin->Edges.Remove(Edge);
 			RemovedCount++;
 		}
 	}

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/Tests/PCGDisconnectNotifyTests.cpp

@@ -0,0 +1,67 @@
+// disconnect_pcg_nodes must tell the graph its structure changed; without that the compiled graph and
+// cached results keep the removed edge until something else invalidates them.
+#if WITH_DEV_AUTOMATION_TESTS
+
+#include "HandlerRegistry.h"
+#include "Handlers/PCG/PCGHandlers.h"
+#include "Dom/JsonObject.h"
+#include "Dom/JsonValue.h"
+#include "Elements/PCGAddTag.h"
+#include "Misc/AutomationTest.h"
+#include "Misc/Guid.h"
+#include "PCGEdge.h"
+#include "PCGGraph.h"
+#include "PCGNode.h"
+#include "PCGPin.h"
+#include "UObject/Package.h"
+#include "Tests/MCPScopedTestMount.h"
+
+IMPLEMENT_SIMPLE_AUTOMATION_TEST(FMCPPCGDisconnectNotifiesTest,
+	"UE.MCP.PCG.Disconnect.NotifiesStructureChange",
+	EAutomationTestFlags::EditorContext | EAutomationTestFlags::EngineFilter)
+
+bool FMCPPCGDisconnectNotifiesTest::RunTest(const FString& Parameters)
+{
+	FMCPScopedTestMount Mount(
+		TEXT("/UEMCPPCGDisconnect_") + FGuid::NewGuid().ToString(EGuidFormats::Digits) + TEXT("/"), TEXT("UEMCPPCGDisconnect"));
+	const FString PackageName = Mount.RootPath + TEXT("PG_DisconnectProbe");
+
+	UPackage* Package = CreatePackage(*PackageName);
+	UPCGGraph* Graph = NewObject<UPCGGraph>(Package, TEXT("PG_DisconnectProbe"), RF_Public | RF_Standalone | RF_Transactional);
+	if (!TestNotNull(TEXT("PCG graph fixture created"), Graph)) return false;
+
+	UPCGAddTagSettings* FromSettings = nullptr;
+	UPCGAddTagSettings* ToSettings = nullptr;
+	UPCGNode* From = Graph->AddNodeOfType(FromSettings);
+	UPCGNode* To = Graph->AddNodeOfType(ToSettings);
+	if (!TestTrue(TEXT("two nodes added"), From && To)) return false;
+	if (!TestNotNull(TEXT("edge added"), Graph->AddEdge(From, PCGPinConstants::DefaultOutputLabel, To, PCGPinConstants::DefaultInputLabel))) return false;
+
+	int32 StructuralEdgeChanges = 0;
+	const FDelegateHandle Handle = Graph->OnGraphChangedDelegate.AddLambda([&StructuralEdgeChanges](UPCGGraphInterface*, EPCGChangeType ChangeType)
+	{
+		if (EnumHasAnyFlags(ChangeType, EPCGChangeType::Structural | EPCGChangeType::Edge)) ++StructuralEdgeChanges;
+	});
+
+	FMCPHandlerRegistry Registry;
+	FPCGHandlers::RegisterHandlers(Registry);
+	auto Params = MakeShared<FJsonObject>();
+	Params->SetStringField(TEXT("assetPath"), PackageName);
+	Params->SetStringField(TEXT("sourceNode"), From->GetName());
+	Params->SetStringField(TEXT("targetNode"), To->GetName());
+	const TSharedPtr<FJsonValue> Value = Registry.ExecuteHandler(TEXT("disconnect_pcg_nodes"), Params);
+	Graph->OnGraphChangedDelegate.Remove(Handle);
+
+	const TSharedPtr<FJsonObject> Result = Value.IsValid() && Value->Type == EJson::Object ? Value->AsObject() : nullptr;
+	double Removed = 0;
+	if (!TestTrue(TEXT("disconnect_pcg_nodes reports one removed edge"), Result.IsValid() && Result->TryGetNumberField(TEXT("removedEdges"), Removed) && Removed == 1)) return false;
+
+	const UPCGPin* OutPin = From->GetOutputPin(PCGPinConstants::DefaultOutputLabel);
+	const UPCGPin* InPin = To->GetInputPin(PCGPinConstants::DefaultInputLabel);
+	TestTrue(TEXT("the source pin has no edges"), OutPin && OutPin->Edges.Num() == 0);
+	TestTrue(TEXT("the target pin has no edges"), InPin && InPin->Edges.Num() == 0);
+	TestTrue(TEXT("the graph broadcast a structural or edge change"), StructuralEdgeChanges > 0);
+	return true;
+}
+
+#endif

package/plugin/ue_mcp_bridge/UE_MCP_Bridge.uplugin

@@ -1,7 +1,7 @@
 {
 	"FileVersion": 3,
 	"Version": 1,
-	"VersionName": "1.3.10-beta.13",
+	"VersionName": "1.3.10-beta.14",
 	"FriendlyName": "UE MCP Bridge",
 	"Description": "C++ WebSocket bridge for UE-MCP, providing 1911+ editor actions over the MCP protocol",
 	"Category": "Developer Tools",