ue-mcp 1.2.1 → 1.2.2

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/EngineStatusHooks.cpp

@@ -1,76 +1,76 @@
-#include "EngineStatusHooks.h"
-#include "MCPEngineStatus.h"
-#include "UE_MCP_BridgeModule.h"
-#include "GameThreadExecutor.h"
-#include "Handlers/DialogHandlers.h"
-
-#include "Framework/Application/SlateApplication.h"
-
-#include "AssetCompilingManager.h"
-#include "ShaderCompiler.h"
-
-namespace
-{
-	FDelegateHandle GPreTickHandle;
-	FDelegateHandle GModalLoopHandle;
-}
-
-void FMCPEngineStatusHooks::Install()
-{
-	FMCPEngineStatus& Status = FMCPEngineStatus::Get();
-
-	Status.SetModalProvider([](FString& OutTitle, FString& OutMessage, TArray<FString>& OutButtons)
-	{
-		return FDialogHandlers::DescribeActiveModal(OutTitle, OutMessage, OutButtons);
-	});
-
-	Status.SetCompileProvider([](int32& OutShaderJobs, int32& OutAssetCompiles)
-	{
-		OutShaderJobs = GShaderCompilingManager ? GShaderCompilingManager->GetNumRemainingJobs() : 0;
-		OutAssetCompiles = FAssetCompilingManager::Get().GetNumRemainingAssets();
-	});
-
-	if (!FSlateApplication::IsInitialized())
-	{
-		UE_LOG(LogMCPBridge, Warning, TEXT("[UE-MCP] Slate not initialized at bridge startup - engine status keeps its core-only hooks"));
-		return;
-	}
-
-	// Slate's pre-tick keeps firing while a long operation pumps the UI to draw
-	// its progress bar, which is exactly the window where the core ticker is
-	// suspended and every bridge request times out.
-	GPreTickHandle = FSlateApplication::Get().OnPreTick().AddLambda([](float)
-	{
-		FMCPEngineStatus::Get().CaptureNow();
-	});
-
-	GModalLoopHandle = FSlateApplication::Get().GetOnModalLoopTickEvent().AddLambda([](float)
-	{
-		FMCPEngineStatus::Get().CaptureNow();
-		// Seeing the dialog is only half of it: the call that could answer it
-		// is queued behind the same blocked game thread. Modal-safe handlers
-		// run here so a dialog can be cleared from the outside.
-		FMCPGameThreadExecutor::DrainModalSafeQueue();
-	});
-}
-
-void FMCPEngineStatusHooks::Remove()
-{
-	FMCPEngineStatus& Status = FMCPEngineStatus::Get();
-	Status.SetModalProvider(nullptr);
-	Status.SetCompileProvider(nullptr);
-
-	if (FSlateApplication::IsInitialized())
-	{
-		if (GPreTickHandle.IsValid())
-		{
-			FSlateApplication::Get().OnPreTick().Remove(GPreTickHandle);
-			GPreTickHandle.Reset();
-		}
-		if (GModalLoopHandle.IsValid())
-		{
-			FSlateApplication::Get().GetOnModalLoopTickEvent().Remove(GModalLoopHandle);
-			GModalLoopHandle.Reset();
-		}
-	}
-}
+#include "EngineStatusHooks.h"
+#include "MCPEngineStatus.h"
+#include "UE_MCP_BridgeModule.h"
+#include "GameThreadExecutor.h"
+#include "Handlers/DialogHandlers.h"
+
+#include "Framework/Application/SlateApplication.h"
+
+#include "AssetCompilingManager.h"
+#include "ShaderCompiler.h"
+
+namespace
+{
+	FDelegateHandle GPreTickHandle;
+	FDelegateHandle GModalLoopHandle;
+}
+
+void FMCPEngineStatusHooks::Install()
+{
+	FMCPEngineStatus& Status = FMCPEngineStatus::Get();
+
+	Status.SetModalProvider([](FString& OutTitle, FString& OutMessage, TArray<FString>& OutButtons)
+	{
+		return FDialogHandlers::DescribeActiveModal(OutTitle, OutMessage, OutButtons);
+	});
+
+	Status.SetCompileProvider([](int32& OutShaderJobs, int32& OutAssetCompiles)
+	{
+		OutShaderJobs = GShaderCompilingManager ? GShaderCompilingManager->GetNumRemainingJobs() : 0;
+		OutAssetCompiles = FAssetCompilingManager::Get().GetNumRemainingAssets();
+	});
+
+	if (!FSlateApplication::IsInitialized())
+	{
+		UE_LOG(LogMCPBridge, Warning, TEXT("[UE-MCP] Slate not initialized at bridge startup - engine status keeps its core-only hooks"));
+		return;
+	}
+
+	// Slate's pre-tick keeps firing while a long operation pumps the UI to draw
+	// its progress bar, which is exactly the window where the core ticker is
+	// suspended and every bridge request times out.
+	GPreTickHandle = FSlateApplication::Get().OnPreTick().AddLambda([](float)
+	{
+		FMCPEngineStatus::Get().CaptureNow();
+	});
+
+	GModalLoopHandle = FSlateApplication::Get().GetOnModalLoopTickEvent().AddLambda([](float)
+	{
+		FMCPEngineStatus::Get().CaptureNow();
+		// Seeing the dialog is only half of it: the call that could answer it
+		// is queued behind the same blocked game thread. Modal-safe handlers
+		// run here so a dialog can be cleared from the outside.
+		FMCPGameThreadExecutor::DrainModalSafeQueue();
+	});
+}
+
+void FMCPEngineStatusHooks::Remove()
+{
+	FMCPEngineStatus& Status = FMCPEngineStatus::Get();
+	Status.SetModalProvider(nullptr);
+	Status.SetCompileProvider(nullptr);
+
+	if (FSlateApplication::IsInitialized())
+	{
+		if (GPreTickHandle.IsValid())
+		{
+			FSlateApplication::Get().OnPreTick().Remove(GPreTickHandle);
+			GPreTickHandle.Reset();
+		}
+		if (GModalLoopHandle.IsValid())
+		{
+			FSlateApplication::Get().GetOnModalLoopTickEvent().Remove(GModalLoopHandle);
+			GModalLoopHandle.Reset();
+		}
+	}
+}

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/EngineStatusHooks.h

@@ -1,27 +1,27 @@
-#pragma once
-
-#include "CoreMinimal.h"
-
-/**
- * Wires the editor-only sensors into FMCPEngineStatus, which lives in the
- * UE_MCP_BridgeStatus module and has been publishing since PostConfigInit on
- * Core-only hooks.
- *
- * Split this way because the snapshot has to exist long before Slate, the
- * asset compiler or the shader compiler do. Once the bridge module loads, the
- * same snapshot gains:
- *
- *   - Slate's pre-tick, which keeps firing while a long operation pumps the UI;
- *   - the modal loop tick, which fires while a dialog owns the main loop (and
- *     is also where modal-safe handlers get their chance to run);
- *   - a description of the active modal dialog, from the Slate widget walk;
- *   - remaining shader jobs and asset compiles.
- */
-namespace FMCPEngineStatusHooks
-{
-	/** Register the Slate hooks and the Slate/Engine-backed providers. */
-	void Install();
-
-	/** Unregister everything registered by Install. Safe to call twice. */
-	void Remove();
-}
+#pragma once
+
+#include "CoreMinimal.h"
+
+/**
+ * Wires the editor-only sensors into FMCPEngineStatus, which lives in the
+ * UE_MCP_BridgeStatus module and has been publishing since PostConfigInit on
+ * Core-only hooks.
+ *
+ * Split this way because the snapshot has to exist long before Slate, the
+ * asset compiler or the shader compiler do. Once the bridge module loads, the
+ * same snapshot gains:
+ *
+ *   - Slate's pre-tick, which keeps firing while a long operation pumps the UI;
+ *   - the modal loop tick, which fires while a dialog owns the main loop (and
+ *     is also where modal-safe handlers get their chance to run);
+ *   - a description of the active modal dialog, from the Slate widget walk;
+ *   - remaining shader jobs and asset compiles.
+ */
+namespace FMCPEngineStatusHooks
+{
+	/** Register the Slate hooks and the Slate/Engine-backed providers. */
+	void Install();
+
+	/** Unregister everything registered by Install. Safe to call twice. */
+	void Remove();
+}

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/HandlerRegistry.cpp

@@ -1,128 +1,128 @@
-#include "HandlerRegistry.h"
-#include "HAL/PlatformFileManager.h"
-#include "Misc/FileHelper.h"
-#include "Misc/Paths.h"
-#include "HandlerUtils.h"
-#include "MCPHandlerRegistration.h"
-
-FMCPHandlerRegistry::FMCPHandlerRegistry()
-{
-}
-
-FMCPHandlerRegistry::~FMCPHandlerRegistry()
-{
-	Clear();
-}
-
-void FMCPHandlerRegistry::RegisterHandler(const FString& MethodName, FHandlerFunction Handler)
-{
-	CppHandlers.Add(MethodName, Handler);
-}
-
-void FMCPHandlerRegistry::RegisterHandlerWithTimeout(const FString& MethodName, FHandlerFunction Handler, float TimeoutSeconds)
-{
-	CppHandlers.Add(MethodName, Handler);
-	if (TimeoutSeconds > 0.0f)
-	{
-		HandlerTimeouts.Add(MethodName, TimeoutSeconds);
-	}
-}
-
-float FMCPHandlerRegistry::GetHandlerTimeout(const FString& MethodName) const
-{
-	if (const float* V = HandlerTimeouts.Find(MethodName))
-	{
-		return *V;
-	}
-	// External (plugin-contributed) handlers may register their own timeout.
-	UEMCP::FExternalHandlerFn Unused;
-	float ExternalTimeout = 0.0f;
-	if (UEMCP::LookupExternalHandler(MethodName, Unused, ExternalTimeout) && ExternalTimeout > 0.0f)
-	{
-		return ExternalTimeout;
-	}
-	return 0.0f;
-}
-
-void FMCPHandlerRegistry::RegisterPythonHandler(const FString& MethodName, const FString& PythonScriptPath)
-{
-	FPythonHandlerInfo Info;
-	Info.ScriptPath = PythonScriptPath;
-	Info.HandlerName = MethodName;
-	PythonHandlers.Add(MethodName, Info);
-}
-
-TSharedPtr<FJsonValue> FMCPHandlerRegistry::ExecuteHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params)
-{
-	// Try C++ handler first
-	if (CppHandlers.Contains(MethodName))
-	{
-		return CppHandlers[MethodName](Params);
-	}
-
-	// Try Python handler
-	if (PythonHandlers.Contains(MethodName))
-	{
-		return ExecutePythonHandler(MethodName, Params);
-	}
-
-	// Plugin-contributed external handler (registered via UEMCP::RegisterExternalHandler).
-	{
-		UEMCP::FExternalHandlerFn External;
-		float Unused = 0.0f;
-		if (UEMCP::LookupExternalHandler(MethodName, External, Unused))
-		{
-			return External(Params);
-		}
-	}
-
-	// Handler not found - return nullptr so BridgeServer sends "Unknown method" error
-	return nullptr;
-}
-
-bool FMCPHandlerRegistry::HasHandler(const FString& MethodName) const
-{
-	if (CppHandlers.Contains(MethodName) || PythonHandlers.Contains(MethodName))
-	{
-		return true;
-	}
-	UEMCP::FExternalHandlerFn Unused;
-	float UnusedTimeout = 0.0f;
-	return UEMCP::LookupExternalHandler(MethodName, Unused, UnusedTimeout);
-}
-
-TArray<FString> FMCPHandlerRegistry::GetHandlerNames() const
-{
-	TArray<FString> Names;
-	CppHandlers.GetKeys(Names);
-
-	TArray<FString> PythonNames;
-	PythonHandlers.GetKeys(PythonNames);
-	Names.Append(PythonNames);
-
-	Names.Append(UEMCP::GetExternalHandlerNames());
-
-	return Names;
-}
-
-void FMCPHandlerRegistry::Clear()
-{
-	CppHandlers.Empty();
-	PythonHandlers.Empty();
-	HandlerTimeouts.Empty();
-}
-
-TSharedPtr<FJsonValue> FMCPHandlerRegistry::ExecutePythonHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& /*Params*/)
-{
-	// Python handler dispatch is not implemented. Prior behaviour returned an
-	// empty JSON object, which callers could not distinguish from a real
-	// empty-success result. Return a typed error instead so callers see the
-	// gap clearly; use `execute_python` for ad-hoc Python until the dispatch
-	// pipeline lands.
-	TSharedPtr<FJsonObject> Err = MakeShared<FJsonObject>();
-	Err->SetBoolField(TEXT("success"), false);
-	Err->SetStringField(TEXT("error"), FString::Printf(
-		TEXT("Python handler '%s' is registered but Python dispatch is not implemented. Use the 'execute_python' action instead."),
-		*MethodName));
-	return MakeShared<FJsonValueObject>(Err);
-}
+#include "HandlerRegistry.h"
+#include "HAL/PlatformFileManager.h"
+#include "Misc/FileHelper.h"
+#include "Misc/Paths.h"
+#include "HandlerUtils.h"
+#include "MCPHandlerRegistration.h"
+
+FMCPHandlerRegistry::FMCPHandlerRegistry()
+{
+}
+
+FMCPHandlerRegistry::~FMCPHandlerRegistry()
+{
+	Clear();
+}
+
+void FMCPHandlerRegistry::RegisterHandler(const FString& MethodName, FHandlerFunction Handler)
+{
+	CppHandlers.Add(MethodName, Handler);
+}
+
+void FMCPHandlerRegistry::RegisterHandlerWithTimeout(const FString& MethodName, FHandlerFunction Handler, float TimeoutSeconds)
+{
+	CppHandlers.Add(MethodName, Handler);
+	if (TimeoutSeconds > 0.0f)
+	{
+		HandlerTimeouts.Add(MethodName, TimeoutSeconds);
+	}
+}
+
+float FMCPHandlerRegistry::GetHandlerTimeout(const FString& MethodName) const
+{
+	if (const float* V = HandlerTimeouts.Find(MethodName))
+	{
+		return *V;
+	}
+	// External (plugin-contributed) handlers may register their own timeout.
+	UEMCP::FExternalHandlerFn Unused;
+	float ExternalTimeout = 0.0f;
+	if (UEMCP::LookupExternalHandler(MethodName, Unused, ExternalTimeout) && ExternalTimeout > 0.0f)
+	{
+		return ExternalTimeout;
+	}
+	return 0.0f;
+}
+
+void FMCPHandlerRegistry::RegisterPythonHandler(const FString& MethodName, const FString& PythonScriptPath)
+{
+	FPythonHandlerInfo Info;
+	Info.ScriptPath = PythonScriptPath;
+	Info.HandlerName = MethodName;
+	PythonHandlers.Add(MethodName, Info);
+}
+
+TSharedPtr<FJsonValue> FMCPHandlerRegistry::ExecuteHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params)
+{
+	// Try C++ handler first
+	if (CppHandlers.Contains(MethodName))
+	{
+		return CppHandlers[MethodName](Params);
+	}
+
+	// Try Python handler
+	if (PythonHandlers.Contains(MethodName))
+	{
+		return ExecutePythonHandler(MethodName, Params);
+	}
+
+	// Plugin-contributed external handler (registered via UEMCP::RegisterExternalHandler).
+	{
+		UEMCP::FExternalHandlerFn External;
+		float Unused = 0.0f;
+		if (UEMCP::LookupExternalHandler(MethodName, External, Unused))
+		{
+			return External(Params);
+		}
+	}
+
+	// Handler not found - return nullptr so BridgeServer sends "Unknown method" error
+	return nullptr;
+}
+
+bool FMCPHandlerRegistry::HasHandler(const FString& MethodName) const
+{
+	if (CppHandlers.Contains(MethodName) || PythonHandlers.Contains(MethodName))
+	{
+		return true;
+	}
+	UEMCP::FExternalHandlerFn Unused;
+	float UnusedTimeout = 0.0f;
+	return UEMCP::LookupExternalHandler(MethodName, Unused, UnusedTimeout);
+}
+
+TArray<FString> FMCPHandlerRegistry::GetHandlerNames() const
+{
+	TArray<FString> Names;
+	CppHandlers.GetKeys(Names);
+
+	TArray<FString> PythonNames;
+	PythonHandlers.GetKeys(PythonNames);
+	Names.Append(PythonNames);
+
+	Names.Append(UEMCP::GetExternalHandlerNames());
+
+	return Names;
+}
+
+void FMCPHandlerRegistry::Clear()
+{
+	CppHandlers.Empty();
+	PythonHandlers.Empty();
+	HandlerTimeouts.Empty();
+}
+
+TSharedPtr<FJsonValue> FMCPHandlerRegistry::ExecutePythonHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& /*Params*/)
+{
+	// Python handler dispatch is not implemented. Prior behaviour returned an
+	// empty JSON object, which callers could not distinguish from a real
+	// empty-success result. Return a typed error instead so callers see the
+	// gap clearly; use `execute_python` for ad-hoc Python until the dispatch
+	// pipeline lands.
+	TSharedPtr<FJsonObject> Err = MakeShared<FJsonObject>();
+	Err->SetBoolField(TEXT("success"), false);
+	Err->SetStringField(TEXT("error"), FString::Printf(
+		TEXT("Python handler '%s' is registered but Python dispatch is not implemented. Use the 'execute_python' action instead."),
+		*MethodName));
+	return MakeShared<FJsonValueObject>(Err);
+}

package/plugin/ue_mcp_bridge/Source/UE_MCP_Bridge/Private/HandlerRegistry.h

@@ -1,63 +1,63 @@
-#pragma once
-
-#include "CoreMinimal.h"
-#include "Dom/JsonValue.h"
-#include "Dom/JsonObject.h"
-
-class FMCPHandlerRegistry
-{
-public:
-	// Handler function signature: takes params JSON object, returns result JSON value
-	using FHandlerFunction = TFunction<TSharedPtr<FJsonValue>(const TSharedPtr<FJsonObject>& Params)>;
-
-	// Python handler info
-	struct FPythonHandlerInfo
-	{
-		FString ScriptPath;
-		FString HandlerName;
-	};
-
-	FMCPHandlerRegistry();
-	~FMCPHandlerRegistry();
-
-	// Register a C++ handler
-	void RegisterHandler(const FString& MethodName, FHandlerFunction Handler);
-
-	// Register a C++ handler with a non-default game-thread execution timeout.
-	// Most handlers finish in milliseconds, but a few (create_cpp_class
-	// regenerates IDE project files; build-related ops) legitimately need
-	// minutes. Pass TimeoutSeconds explicitly for those.
-	void RegisterHandlerWithTimeout(const FString& MethodName, FHandlerFunction Handler, float TimeoutSeconds);
-
-	// Look up a per-handler timeout. Returns 0 if no override registered,
-	// in which case the caller should use its default.
-	float GetHandlerTimeout(const FString& MethodName) const;
-
-	// Register a Python handler
-	void RegisterPythonHandler(const FString& MethodName, const FString& PythonScriptPath);
-
-	// Execute a handler
-	TSharedPtr<FJsonValue> ExecuteHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params);
-
-	// Check if handler exists
-	bool HasHandler(const FString& MethodName) const;
-
-	// Get all registered handler names
-	TArray<FString> GetHandlerNames() const;
-
-	// Clear all handlers
-	void Clear();
-
-private:
-	// C++ handlers
-	TMap<FString, FHandlerFunction> CppHandlers;
-
-	// Python handlers
-	TMap<FString, FPythonHandlerInfo> PythonHandlers;
-
-	// Per-handler game-thread timeouts (seconds). Absent = use default.
-	TMap<FString, float> HandlerTimeouts;
-
-	// Execute Python handler
-	TSharedPtr<FJsonValue> ExecutePythonHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params);
-};
+#pragma once
+
+#include "CoreMinimal.h"
+#include "Dom/JsonValue.h"
+#include "Dom/JsonObject.h"
+
+class FMCPHandlerRegistry
+{
+public:
+	// Handler function signature: takes params JSON object, returns result JSON value
+	using FHandlerFunction = TFunction<TSharedPtr<FJsonValue>(const TSharedPtr<FJsonObject>& Params)>;
+
+	// Python handler info
+	struct FPythonHandlerInfo
+	{
+		FString ScriptPath;
+		FString HandlerName;
+	};
+
+	FMCPHandlerRegistry();
+	~FMCPHandlerRegistry();
+
+	// Register a C++ handler
+	void RegisterHandler(const FString& MethodName, FHandlerFunction Handler);
+
+	// Register a C++ handler with a non-default game-thread execution timeout.
+	// Most handlers finish in milliseconds, but a few (create_cpp_class
+	// regenerates IDE project files; build-related ops) legitimately need
+	// minutes. Pass TimeoutSeconds explicitly for those.
+	void RegisterHandlerWithTimeout(const FString& MethodName, FHandlerFunction Handler, float TimeoutSeconds);
+
+	// Look up a per-handler timeout. Returns 0 if no override registered,
+	// in which case the caller should use its default.
+	float GetHandlerTimeout(const FString& MethodName) const;
+
+	// Register a Python handler
+	void RegisterPythonHandler(const FString& MethodName, const FString& PythonScriptPath);
+
+	// Execute a handler
+	TSharedPtr<FJsonValue> ExecuteHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params);
+
+	// Check if handler exists
+	bool HasHandler(const FString& MethodName) const;
+
+	// Get all registered handler names
+	TArray<FString> GetHandlerNames() const;
+
+	// Clear all handlers
+	void Clear();
+
+private:
+	// C++ handlers
+	TMap<FString, FHandlerFunction> CppHandlers;
+
+	// Python handlers
+	TMap<FString, FPythonHandlerInfo> PythonHandlers;
+
+	// Per-handler game-thread timeouts (seconds). Absent = use default.
+	TMap<FString, float> HandlerTimeouts;
+
+	// Execute Python handler
+	TSharedPtr<FJsonValue> ExecutePythonHandler(const FString& MethodName, const TSharedPtr<FJsonObject>& Params);
+};