unreal-engine-mcp-server 0.5.4 → 0.5.5

package/plugins/McpAutomationBridge/Source/McpAutomationBridge/Private/McpAutomationBridge_LandscapeHandlers.cpp

@@ -1,1164 +0,0 @@
-#include "McpAutomationBridgeGlobals.h"
-#include "McpAutomationBridgeHelpers.h"
-#include "McpAutomationBridgeSubsystem.h"
-#include "Runtime/Launch/Resources/Version.h"
-
-
-#if WITH_EDITOR
-#include "Async/Async.h"
-#include "EditorAssetLibrary.h"
-#include "Engine/World.h"
-#include "Landscape.h"
-#include "LandscapeComponent.h"
-#include "LandscapeDataAccess.h"
-#include "LandscapeEdit.h"
-#include "LandscapeEditorObject.h"
-#include "LandscapeEditorUtils.h"
-#include "LandscapeGrassType.h"
-#include "LandscapeInfo.h"
-#include "LandscapeProxy.h"
-#include "LandscapeStreamingProxy.h"
-#include "Materials/Material.h"
-#include "Materials/MaterialInstanceConstant.h"
-#include "Misc/ScopedSlowTask.h"
-#include "UObject/SavePackage.h"
-
-#if __has_include("Subsystems/EditorActorSubsystem.h")
-#include "Subsystems/EditorActorSubsystem.h"
-#elif __has_include("EditorActorSubsystem.h")
-#include "EditorActorSubsystem.h"
-#endif
-#endif
-
-bool UMcpAutomationBridgeSubsystem::HandleEditLandscape(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  // Dispatch to specific edit operations implemented below
-  if (HandleModifyHeightmap(RequestId, Action, Payload, RequestingSocket))
-    return true;
-  if (HandlePaintLandscapeLayer(RequestId, Action, Payload, RequestingSocket))
-    return true;
-  if (HandleSculptLandscape(RequestId, Action, Payload, RequestingSocket))
-    return true;
-  if (HandleSetLandscapeMaterial(RequestId, Action, Payload, RequestingSocket))
-    return true;
-  return false;
-}
-
-bool UMcpAutomationBridgeSubsystem::HandleCreateLandscape(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("create_landscape"), ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("create_landscape payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  // Parse inputs (accept multiple shapes)
-  double X = 0.0, Y = 0.0, Z = 0.0;
-  if (!Payload->TryGetNumberField(TEXT("x"), X) ||
-      !Payload->TryGetNumberField(TEXT("y"), Y) ||
-      !Payload->TryGetNumberField(TEXT("z"), Z)) {
-    // Try location object { x, y, z }
-    const TSharedPtr<FJsonObject> *LocObj = nullptr;
-    if (Payload->TryGetObjectField(TEXT("location"), LocObj) && LocObj) {
-      (*LocObj)->TryGetNumberField(TEXT("x"), X);
-      (*LocObj)->TryGetNumberField(TEXT("y"), Y);
-      (*LocObj)->TryGetNumberField(TEXT("z"), Z);
-    } else {
-      // Try location as array [x,y,z]
-      const TArray<TSharedPtr<FJsonValue>> *LocArr = nullptr;
-      if (Payload->TryGetArrayField(TEXT("location"), LocArr) && LocArr &&
-          LocArr->Num() >= 3) {
-        X = (*LocArr)[0]->AsNumber();
-        Y = (*LocArr)[1]->AsNumber();
-        Z = (*LocArr)[2]->AsNumber();
-      }
-    }
-  }
-
-  int32 ComponentsX = 8, ComponentsY = 8;
-  bool bHasCX = Payload->TryGetNumberField(TEXT("componentsX"), ComponentsX);
-  bool bHasCY = Payload->TryGetNumberField(TEXT("componentsY"), ComponentsY);
-
-  int32 ComponentCount = 0;
-  Payload->TryGetNumberField(TEXT("componentCount"), ComponentCount);
-  if (!bHasCX && ComponentCount > 0) {
-    ComponentsX = ComponentCount;
-  }
-  if (!bHasCY && ComponentCount > 0) {
-    ComponentsY = ComponentCount;
-  }
-
-  // If sizeX/sizeY provided (world units), derive a coarse components estimate
-  double SizeXUnits = 0.0, SizeYUnits = 0.0;
-  if (Payload->TryGetNumberField(TEXT("sizeX"), SizeXUnits) && SizeXUnits > 0 &&
-      !bHasCX) {
-    ComponentsX =
-        FMath::Max(1, static_cast<int32>(FMath::Floor(SizeXUnits / 1000.0)));
-  }
-  if (Payload->TryGetNumberField(TEXT("sizeY"), SizeYUnits) && SizeYUnits > 0 &&
-      !bHasCY) {
-    ComponentsY =
-        FMath::Max(1, static_cast<int32>(FMath::Floor(SizeYUnits / 1000.0)));
-  }
-
-  int32 QuadsPerComponent = 63;
-  if (!Payload->TryGetNumberField(TEXT("quadsPerComponent"),
-                                  QuadsPerComponent)) {
-    // Accept quadsPerSection synonym from some clients
-    Payload->TryGetNumberField(TEXT("quadsPerSection"), QuadsPerComponent);
-  }
-
-  int32 SectionsPerComponent = 1;
-  Payload->TryGetNumberField(TEXT("sectionsPerComponent"),
-                             SectionsPerComponent);
-
-  FString MaterialPath;
-  Payload->TryGetStringField(TEXT("materialPath"), MaterialPath);
-  if (MaterialPath.IsEmpty()) {
-    // Default to simple WorldGridMaterial if none provided to ensure visibility
-    MaterialPath = TEXT("/Engine/EngineMaterials/WorldGridMaterial");
-  }
-
-  // ... inside HandleCreateLandscape ...
-  if (!GEditor || !GEditor->GetEditorWorldContext().World()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("Editor world not available"),
-                        TEXT("EDITOR_NOT_AVAILABLE"));
-    return true;
-  }
-
-  FString NameOverride;
-  if (!Payload->TryGetStringField(TEXT("name"), NameOverride) ||
-      NameOverride.IsEmpty()) {
-    Payload->TryGetStringField(TEXT("landscapeName"), NameOverride);
-  }
-
-  // Capture parameters by value for the async task
-  const int32 CaptComponentsX = ComponentsX;
-  const int32 CaptComponentsY = ComponentsY;
-  const int32 CaptQuadsPerComponent = QuadsPerComponent;
-  const int32 CaptSectionsPerComponent = SectionsPerComponent;
-  const FVector CaptLocation(X, Y, Z);
-  const FString CaptMaterialPath = MaterialPath;
-  const FString CaptName = NameOverride;
-
-  // Debug log to confirm name capture
-  UE_LOG(LogMcpAutomationBridgeSubsystem, Display,
-         TEXT("HandleCreateLandscape: Captured name '%s' (from override '%s')"),
-         *CaptName, *NameOverride);
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  // Execute on Game Thread to ensure thread safety for Actor spawning and
-  // Landscape operations
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, CaptComponentsX,
-                                        CaptComponentsY, CaptQuadsPerComponent,
-                                        CaptSectionsPerComponent, CaptLocation,
-                                        CaptMaterialPath, CaptName]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    if (!GEditor)
-      return;
-    UWorld *World = GEditor->GetEditorWorldContext().World();
-    if (!World)
-      return;
-
-    FActorSpawnParameters SpawnParams;
-    SpawnParams.SpawnCollisionHandlingOverride =
-        ESpawnActorCollisionHandlingMethod::AlwaysSpawn;
-    ALandscape *Landscape =
-        World->SpawnActor<ALandscape>(ALandscape::StaticClass(), CaptLocation,
-                                      FRotator::ZeroRotator, SpawnParams);
-    if (!Landscape) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to spawn landscape actor"),
-                                     TEXT("SPAWN_FAILED"));
-      return;
-    }
-
-    if (!CaptName.IsEmpty()) {
-      Landscape->SetActorLabel(CaptName);
-    } else {
-      Landscape->SetActorLabel(FString::Printf(
-          TEXT("Landscape_%dx%d"), CaptComponentsX, CaptComponentsY));
-    }
-    Landscape->ComponentSizeQuads = CaptQuadsPerComponent;
-    Landscape->SubsectionSizeQuads =
-        CaptQuadsPerComponent / CaptSectionsPerComponent;
-    Landscape->NumSubsections = CaptSectionsPerComponent;
-
-    if (!CaptMaterialPath.IsEmpty()) {
-      UMaterialInterface *Mat =
-          LoadObject<UMaterialInterface>(nullptr, *CaptMaterialPath);
-      if (Mat) {
-        Landscape->LandscapeMaterial = Mat;
-      }
-    }
-
-    // CRITICAL INITIALIZATION ORDER:
-    // 1. Set Landscape GUID first. CreateLandscapeInfo depends on this.
-    if (!Landscape->GetLandscapeGuid().IsValid()) {
-      Landscape->SetLandscapeGuid(FGuid::NewGuid());
-    }
-
-    // 2. Create Landscape Info. This will register itself with the Landscape's
-    // GUID.
-    Landscape->CreateLandscapeInfo();
-
-    const int32 VertX = CaptComponentsX * CaptQuadsPerComponent + 1;
-    const int32 VertY = CaptComponentsY * CaptQuadsPerComponent + 1;
-
-    TArray<uint16> HeightArray;
-    HeightArray.Init(32768, VertX * VertY);
-
-    const int32 InMinX = 0;
-    const int32 InMinY = 0;
-    const int32 InMaxX = CaptComponentsX * CaptQuadsPerComponent;
-    const int32 InMaxY = CaptComponentsY * CaptQuadsPerComponent;
-    const int32 NumSubsections = CaptSectionsPerComponent;
-    const int32 SubsectionSizeQuads =
-        CaptQuadsPerComponent / FMath::Max(1, CaptSectionsPerComponent);
-
-    // 3. Use a valid GUID for Import call, but zero GUID for map keys.
-    // Analysis of Landscape.cpp shows:
-    // - Import() asserts InGuid.IsValid()
-    // - BUT Import() uses FGuid() (zero) to look up data in the maps:
-    // InImportHeightData.FindChecked(FinalLayerGuid) where FinalLayerGuid is
-    // default constructed.
-    const FGuid ImportGuid =
-        FGuid::NewGuid(); // Valid GUID for the function call
-    const FGuid DataKey;  // Zero GUID for the map keys
-
-    // 3. Populate maps with FGuid() keys because ALandscape::Import uses
-    // default GUID to look up data regardless of the GUID passed to the
-    // function (which is used for the layer definition itself).
-    TMap<FGuid, TArray<uint16>> ImportHeightData;
-    ImportHeightData.Add(FGuid(), HeightArray);
-
-    TMap<FGuid, TArray<FLandscapeImportLayerInfo>> ImportLayerInfos;
-    ImportLayerInfos.Add(FGuid(), TArray<FLandscapeImportLayerInfo>());
-
-    TArray<FLandscapeLayer> EditLayers;
-
-    // Use a transaction to ensure undo/redo and proper notification
-    {
-      const FScopedTransaction Transaction(
-          FText::FromString(TEXT("Create Landscape")));
-      Landscape->Modify();
-
-#if ENGINE_MAJOR_VERSION == 5 && ENGINE_MINOR_VERSION >= 7
-      // UE 5.7+: The Import() function has a known issue with fresh landscapes.
-      // Use CreateDefaultLayer instead to initialize a valid landscape
-      // structure. Note: bCanHaveLayersContent is deprecated/removed in 5.7 as
-      // all landscapes use edit layers.
-
-      // Create default edit layer to enable modification
-      if (Landscape->GetLayersConst().Num() == 0) {
-        Landscape->CreateDefaultLayer();
-      }
-
-      // Explicitly request layer initialization to ensure components are ready
-      // Landscape->RequestLayersInitialization(true, true); // Removed to
-      // prevent crash: LandscapeEditLayers.cpp confirms this resets init state
-      // which is unstable here
-
-      // Note: We bypass feeding ImportHeightData here because doing so via
-      // Import() is what causes the crash in 5.7. A flat empty landscape is
-      // created instead.
-
-#else
-            // UE 5.6 and older: Use standard Import() workflow
-            Landscape->Import(FGuid::NewGuid(), 0, 0, CaptComponentsX - 1, CaptComponentsY - 1, CaptSectionsPerComponent, CaptQuadsPerComponent, ImportHeightData, nullptr, ImportLayerInfos, ELandscapeImportAlphamapType::Layered, TArrayView<const FLandscapeLayer>(EditLayers));
-            Landscape->CreateDefaultLayer();
-#endif
-    }
-
-    // Initialize properties AFTER import to avoid conflicts during component
-    // creation
-    if (CaptName.IsEmpty()) {
-      Landscape->SetActorLabel(FString::Printf(
-          TEXT("Landscape_%dx%d"), CaptComponentsX, CaptComponentsY));
-    } else {
-      Landscape->SetActorLabel(CaptName);
-      UE_LOG(LogMcpAutomationBridgeSubsystem, Display,
-             TEXT("HandleCreateLandscape: Set ActorLabel to '%s'"), *CaptName);
-    }
-
-    if (!CaptMaterialPath.IsEmpty()) {
-      UMaterialInterface *Mat =
-          LoadObject<UMaterialInterface>(nullptr, *CaptMaterialPath);
-      if (Mat) {
-        Landscape->LandscapeMaterial = Mat;
-        // Re-assign material effectively
-        Landscape->PostEditChange();
-      }
-    }
-
-    // Register components if Import didn't do it (it usually does re-register)
-    if (Landscape->GetRootComponent() &&
-        !Landscape->GetRootComponent()->IsRegistered()) {
-      Landscape->RegisterAllComponents();
-    }
-
-    // Register components if Import didn't do it (it usually does re-register)
-    if (Landscape->GetRootComponent() &&
-        !Landscape->GetRootComponent()->IsRegistered()) {
-      Landscape->RegisterAllComponents();
-    }
-
-    // Only call PostEditChange if the landscape is still valid and not pending
-    // kill
-    if (IsValid(Landscape)) {
-      Landscape->PostEditChange();
-    }
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("landscapePath"), Landscape->GetPathName());
-    Resp->SetStringField(TEXT("actorLabel"), Landscape->GetActorLabel());
-    Resp->SetNumberField(TEXT("componentsX"), CaptComponentsX);
-    Resp->SetNumberField(TEXT("componentsY"), CaptComponentsY);
-    Resp->SetNumberField(TEXT("quadsPerComponent"), CaptQuadsPerComponent);
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Landscape created successfully"),
-                                      Resp, FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(RequestingSocket, RequestId, false,
-                         TEXT("create_landscape requires editor build."),
-                         nullptr, TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}
-
-bool UMcpAutomationBridgeSubsystem::HandleModifyHeightmap(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("modify_heightmap"), ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("modify_heightmap payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  FString LandscapePath;
-  Payload->TryGetStringField(TEXT("landscapePath"), LandscapePath);
-  FString LandscapeName;
-  Payload->TryGetStringField(TEXT("landscapeName"), LandscapeName);
-
-  const TArray<TSharedPtr<FJsonValue>> *HeightDataArray = nullptr;
-  if (!Payload->TryGetArrayField(TEXT("heightData"), HeightDataArray) ||
-      !HeightDataArray || HeightDataArray->Num() == 0) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("heightData array required"),
-                        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-
-  // Copy height data for async task
-  TArray<uint16> HeightValues;
-  for (const TSharedPtr<FJsonValue> &Val : *HeightDataArray) {
-    if (Val.IsValid() && Val->Type == EJson::Number) {
-      HeightValues.Add(
-          static_cast<uint16>(FMath::Clamp(Val->AsNumber(), 0.0, 65535.0)));
-    }
-  }
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  // Dispatch to Game Thread
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, LandscapePath,
-                                        LandscapeName,
-                                        HeightValues =
-                                            MoveTemp(HeightValues)]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    ALandscape *Landscape = nullptr;
-    if (!LandscapePath.IsEmpty()) {
-      Landscape = Cast<ALandscape>(
-          StaticLoadObject(ALandscape::StaticClass(), nullptr, *LandscapePath));
-    }
-
-    // Find landscape with fallback to single instance
-    if (!Landscape && GEditor) {
-      if (UEditorActorSubsystem *ActorSS =
-              GEditor->GetEditorSubsystem<UEditorActorSubsystem>()) {
-        TArray<AActor *> AllActors = ActorSS->GetAllLevelActors();
-        ALandscape *Fallback = nullptr;
-        int32 Count = 0;
-
-        for (AActor *A : AllActors) {
-          if (ALandscape *L = Cast<ALandscape>(A)) {
-            Count++;
-            Fallback = L;
-            if (!LandscapeName.IsEmpty() &&
-                L->GetActorLabel().Equals(LandscapeName,
-                                          ESearchCase::IgnoreCase)) {
-              Landscape = L;
-              break;
-            }
-          }
-        }
-
-        if (!Landscape && Count == 1) {
-          Landscape = Fallback;
-        }
-      }
-    }
-    if (!Landscape) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to find landscape"),
-                                     TEXT("LOAD_FAILED"));
-      return;
-    }
-
-    ULandscapeInfo *LandscapeInfo = Landscape->GetLandscapeInfo();
-    if (!LandscapeInfo) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Landscape has no info"),
-                                     TEXT("INVALID_LANDSCAPE"));
-      return;
-    }
-
-    FScopedSlowTask SlowTask(2.0f,
-                             FText::FromString(TEXT("Modifying heightmap...")));
-    SlowTask.MakeDialog();
-
-    int32 MinX, MinY, MaxX, MaxY;
-    if (!LandscapeInfo->GetLandscapeExtent(MinX, MinY, MaxX, MaxY)) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to get landscape extent"),
-                                     TEXT("INVALID_LANDSCAPE"));
-      return;
-    }
-
-    SlowTask.EnterProgressFrame(
-        1.0f, FText::FromString(TEXT("Writing heightmap data")));
-
-    const int32 SizeX = (MaxX - MinX + 1);
-    const int32 SizeY = (MaxY - MinY + 1);
-
-    if (HeightValues.Num() != SizeX * SizeY) {
-      Subsystem->SendAutomationError(
-          RequestingSocket, RequestId,
-          FString::Printf(TEXT("Height data size mismatch. Expected %d x %d = "
-                               "%d values, got %d"),
-                          SizeX, SizeY, SizeX * SizeY, HeightValues.Num()),
-          TEXT("INVALID_ARGUMENT"));
-      return;
-    }
-
-    FLandscapeEditDataInterface LandscapeEdit(LandscapeInfo);
-    LandscapeEdit.SetHeightData(MinX, MinY, MaxX, MaxY, HeightValues.GetData(),
-                                SizeX, true);
-
-    SlowTask.EnterProgressFrame(
-        1.0f, FText::FromString(TEXT("Rebuilding collision")));
-    LandscapeEdit.Flush();
-    Landscape->PostEditChange();
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("landscapePath"), LandscapePath);
-    Resp->SetNumberField(TEXT("modifiedVertices"), HeightValues.Num());
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Heightmap modified successfully"),
-                                      Resp, FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(RequestingSocket, RequestId, false,
-                         TEXT("modify_heightmap requires editor build."),
-                         nullptr, TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}
-
-bool UMcpAutomationBridgeSubsystem::HandlePaintLandscapeLayer(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("paint_landscape_layer"), ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("paint_landscape_layer payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  FString LandscapePath;
-  Payload->TryGetStringField(TEXT("landscapePath"), LandscapePath);
-  FString LandscapeName;
-  Payload->TryGetStringField(TEXT("landscapeName"), LandscapeName);
-
-  FString LayerName;
-  if (!Payload->TryGetStringField(TEXT("layerName"), LayerName) ||
-      LayerName.IsEmpty()) {
-    SendAutomationError(RequestingSocket, RequestId, TEXT("layerName required"),
-                        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-
-  // Paint region (optional - if not specified, paint entire landscape)
-  int32 MinX = -1, MinY = -1, MaxX = -1, MaxY = -1;
-  const TSharedPtr<FJsonObject> *RegionObj = nullptr;
-  if (Payload->TryGetObjectField(TEXT("region"), RegionObj) && RegionObj) {
-    (*RegionObj)->TryGetNumberField(TEXT("minX"), MinX);
-    (*RegionObj)->TryGetNumberField(TEXT("minY"), MinY);
-    (*RegionObj)->TryGetNumberField(TEXT("maxX"), MaxX);
-    (*RegionObj)->TryGetNumberField(TEXT("maxY"), MaxY);
-  }
-
-  double Strength = 1.0;
-  Payload->TryGetNumberField(TEXT("strength"), Strength);
-  Strength = FMath::Clamp(Strength, 0.0, 1.0);
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, LandscapePath,
-                                        LandscapeName, LayerName, MinX, MinY,
-                                        MaxX, MaxY, Strength]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    ALandscape *Landscape = nullptr;
-    if (!LandscapePath.IsEmpty()) {
-      Landscape = Cast<ALandscape>(
-          StaticLoadObject(ALandscape::StaticClass(), nullptr, *LandscapePath));
-    }
-    if (!Landscape && !LandscapeName.IsEmpty()) {
-      if (UEditorActorSubsystem *ActorSS =
-              GEditor->GetEditorSubsystem<UEditorActorSubsystem>()) {
-        TArray<AActor *> AllActors = ActorSS->GetAllLevelActors();
-        for (AActor *A : AllActors) {
-          if (A && A->IsA<ALandscape>() &&
-              A->GetActorLabel().Equals(LandscapeName,
-                                        ESearchCase::IgnoreCase)) {
-            Landscape = Cast<ALandscape>(A);
-            break;
-          }
-        }
-      }
-    }
-    if (!Landscape) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to find landscape"),
-                                     TEXT("LOAD_FAILED"));
-      return;
-    }
-
-    ULandscapeInfo *LandscapeInfo = Landscape->GetLandscapeInfo();
-    if (!LandscapeInfo) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Landscape has no info"),
-                                     TEXT("INVALID_LANDSCAPE"));
-      return;
-    }
-
-    ULandscapeLayerInfoObject *LayerInfo = nullptr;
-    for (const FLandscapeInfoLayerSettings &Layer : LandscapeInfo->Layers) {
-      if (Layer.LayerName == FName(*LayerName)) {
-        LayerInfo = Layer.LayerInfoObj;
-        break;
-      }
-    }
-
-    if (!LayerInfo) {
-      Subsystem->SendAutomationError(
-          RequestingSocket, RequestId,
-          FString::Printf(TEXT("Layer '%s' not found. Create layer first using "
-                               "landscape editor."),
-                          *LayerName),
-          TEXT("LAYER_NOT_FOUND"));
-      return;
-    }
-
-    FScopedSlowTask SlowTask(
-        1.0f, FText::FromString(TEXT("Painting landscape layer...")));
-    SlowTask.MakeDialog();
-
-    int32 PaintMinX = MinX;
-    int32 PaintMinY = MinY;
-    int32 PaintMaxX = MaxX;
-    int32 PaintMaxY = MaxY;
-    if (PaintMinX < 0 || PaintMaxX < 0) {
-      LandscapeInfo->GetLandscapeExtent(PaintMinX, PaintMinY, PaintMaxX,
-                                        PaintMaxY);
-    }
-
-    FLandscapeEditDataInterface LandscapeEdit(LandscapeInfo);
-    const uint8 PaintValue = static_cast<uint8>(Strength * 255.0);
-    const int32 RegionSizeX = (PaintMaxX - PaintMinX + 1);
-    const int32 RegionSizeY = (PaintMaxY - PaintMinY + 1);
-
-    TArray<uint8> AlphaData;
-    AlphaData.Init(PaintValue, RegionSizeX * RegionSizeY);
-
-    LandscapeEdit.SetAlphaData(LayerInfo, PaintMinX, PaintMinY, PaintMaxX,
-                               PaintMaxY, AlphaData.GetData(), RegionSizeX);
-    LandscapeEdit.Flush();
-    Landscape->PostEditChange();
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("landscapePath"), LandscapePath);
-    Resp->SetStringField(TEXT("layerName"), LayerName);
-    Resp->SetNumberField(TEXT("strength"), Strength);
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Layer painted successfully"), Resp,
-                                      FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(RequestingSocket, RequestId, false,
-                         TEXT("paint_landscape_layer requires editor build."),
-                         nullptr, TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}
-
-bool UMcpAutomationBridgeSubsystem::HandleSculptLandscape(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("sculpt_landscape"), ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("sculpt_landscape payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  FString LandscapePath;
-  Payload->TryGetStringField(TEXT("landscapePath"), LandscapePath);
-  FString LandscapeName;
-  Payload->TryGetStringField(TEXT("landscapeName"), LandscapeName);
-
-  UE_LOG(LogMcpAutomationBridgeSubsystem, Warning,
-         TEXT("HandleSculptLandscape: RequestId=%s Path='%s' Name='%s'"),
-         *RequestId, *LandscapePath, *LandscapeName);
-
-  double LocX = 0, LocY = 0, LocZ = 0;
-  const TSharedPtr<FJsonObject> *LocObj = nullptr;
-  // Accept both 'location' and 'position' parameter names for consistency
-  if (Payload->TryGetObjectField(TEXT("location"), LocObj) && LocObj) {
-    (*LocObj)->TryGetNumberField(TEXT("x"), LocX);
-    (*LocObj)->TryGetNumberField(TEXT("y"), LocY);
-    (*LocObj)->TryGetNumberField(TEXT("z"), LocZ);
-  } else if (Payload->TryGetObjectField(TEXT("position"), LocObj) && LocObj) {
-    (*LocObj)->TryGetNumberField(TEXT("x"), LocX);
-    (*LocObj)->TryGetNumberField(TEXT("y"), LocY);
-    (*LocObj)->TryGetNumberField(TEXT("z"), LocZ);
-  } else {
-    SendAutomationError(
-        RequestingSocket, RequestId,
-        TEXT("location or position required. Example: {\"location\": {\"x\": "
-             "0, \"y\": 0, \"z\": 100}}"),
-        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-  FVector TargetLocation(LocX, LocY, LocZ);
-
-  FString ToolMode = TEXT("Raise");
-  Payload->TryGetStringField(TEXT("toolMode"), ToolMode);
-
-  double BrushRadius = 1000.0;
-  Payload->TryGetNumberField(TEXT("brushRadius"), BrushRadius);
-
-  double BrushFalloff = 0.5;
-  Payload->TryGetNumberField(TEXT("brushFalloff"), BrushFalloff);
-
-  double Strength = 0.1;
-  Payload->TryGetNumberField(TEXT("strength"), Strength);
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, LandscapePath,
-                                        LandscapeName, TargetLocation, ToolMode,
-                                        BrushRadius, BrushFalloff, Strength]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    ALandscape *Landscape = nullptr;
-    if (!LandscapePath.IsEmpty()) {
-      Landscape = Cast<ALandscape>(
-          StaticLoadObject(ALandscape::StaticClass(), nullptr, *LandscapePath));
-    }
-
-    if (!Landscape && GEditor) {
-      if (UEditorActorSubsystem *ActorSS =
-              GEditor->GetEditorSubsystem<UEditorActorSubsystem>()) {
-        TArray<AActor *> AllActors = ActorSS->GetAllLevelActors();
-        ALandscape *Fallback = nullptr;
-        int32 LandscapeCount = 0;
-
-        for (AActor *A : AllActors) {
-          if (ALandscape *L = Cast<ALandscape>(A)) {
-            LandscapeCount++;
-            Fallback = L;
-
-            if (!LandscapeName.IsEmpty() &&
-                L->GetActorLabel().Equals(LandscapeName,
-                                          ESearchCase::IgnoreCase)) {
-              Landscape = L;
-              break;
-            }
-          }
-        }
-
-        if (!Landscape && LandscapeCount == 1) {
-          Landscape = Fallback;
-          UE_LOG(LogMcpAutomationBridgeSubsystem, Warning,
-                 TEXT("HandleSculptLandscape: Exact match for '%s' not found, "
-                      "using single available Landscape: '%s'"),
-                 *LandscapeName, *Landscape->GetActorLabel());
-        }
-      }
-    }
-    if (!Landscape) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to find landscape"),
-                                     TEXT("LOAD_FAILED"));
-      return;
-    }
-
-    ULandscapeInfo *LandscapeInfo = Landscape->GetLandscapeInfo();
-    if (!LandscapeInfo) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Landscape has no info"),
-                                     TEXT("INVALID_LANDSCAPE"));
-      return;
-    }
-
-    // Convert World Location to Landscape Local Space
-    FVector LocalPos =
-        Landscape->GetActorTransform().InverseTransformPosition(TargetLocation);
-    int32 CenterX = FMath::RoundToInt(LocalPos.X);
-    int32 CenterY = FMath::RoundToInt(LocalPos.Y);
-
-    // Convert Brush Radius to Vertex Units (assuming uniform scale for
-    // simplicity, or use X)
-    float ScaleX = Landscape->GetActorScale3D().X;
-    int32 RadiusVerts = FMath::Max(1, FMath::RoundToInt(BrushRadius / ScaleX));
-    int32 FalloffVerts = FMath::RoundToInt(RadiusVerts * BrushFalloff);
-
-    int32 MinX = CenterX - RadiusVerts;
-    int32 MaxX = CenterX + RadiusVerts;
-    int32 MinY = CenterY - RadiusVerts;
-    int32 MaxY = CenterY + RadiusVerts;
-
-    // Clamp to landscape extents
-    int32 LMinX, LMinY, LMaxX, LMaxY;
-    if (LandscapeInfo->GetLandscapeExtent(LMinX, LMinY, LMaxX, LMaxY)) {
-      MinX = FMath::Max(MinX, LMinX);
-      MinY = FMath::Max(MinY, LMinY);
-      MaxX = FMath::Min(MaxX, LMaxX);
-      MaxY = FMath::Min(MaxY, LMaxY);
-    }
-
-    if (MinX > MaxX || MinY > MaxY) {
-      Subsystem->SendAutomationResponse(RequestingSocket, RequestId, false,
-                                        TEXT("Brush outside landscape bounds"),
-                                        nullptr, TEXT("OUT_OF_BOUNDS"));
-      return;
-    }
-
-    int32 SizeX = MaxX - MinX + 1;
-    int32 SizeY = MaxY - MinY + 1;
-    TArray<uint16> HeightData;
-    HeightData.SetNumZeroed(SizeX * SizeY);
-
-    FLandscapeEditDataInterface LandscapeEdit(LandscapeInfo);
-    LandscapeEdit.GetHeightData(MinX, MinY, MaxX, MaxY, HeightData.GetData(),
-                                0);
-
-    bool bModified = false;
-    for (int32 Y = MinY; Y <= MaxY; ++Y) {
-      for (int32 X = MinX; X <= MaxX; ++X) {
-        float Dist = FMath::Sqrt(FMath::Square((float)(X - CenterX)) +
-                                 FMath::Square((float)(Y - CenterY)));
-        if (Dist > RadiusVerts)
-          continue;
-
-        float Alpha = 1.0f;
-        if (Dist > (RadiusVerts - FalloffVerts)) {
-          Alpha = 1.0f -
-                  ((Dist - (RadiusVerts - FalloffVerts)) / (float)FalloffVerts);
-        }
-        Alpha = FMath::Clamp(Alpha, 0.0f, 1.0f);
-
-        int32 Index = (Y - MinY) * SizeX + (X - MinX);
-        if (Index < 0 || Index >= HeightData.Num())
-          continue;
-
-        uint16 CurrentHeight = HeightData[Index];
-
-        float ScaleZ = Landscape->GetActorScale3D().Z;
-        float HeightScale =
-            128.0f / ScaleZ; // Conversion factor from World Z to uint16
-
-        float Delta = 0.0f;
-        if (ToolMode.Equals(TEXT("Raise"), ESearchCase::IgnoreCase)) {
-          Delta = Strength * Alpha * 100.0f *
-                  HeightScale; // Arbitrary strength multiplier
-        } else if (ToolMode.Equals(TEXT("Lower"), ESearchCase::IgnoreCase)) {
-          Delta = -Strength * Alpha * 100.0f * HeightScale;
-        } else if (ToolMode.Equals(TEXT("Flatten"), ESearchCase::IgnoreCase)) {
-          float CurrentVal = (float)CurrentHeight;
-          float Target = (TargetLocation.Z - Landscape->GetActorLocation().Z) /
-                             ScaleZ * 128.0f +
-                         32768.0f;
-          Delta = (Target - CurrentVal) * Strength * Alpha;
-        }
-
-        int32 NewHeight =
-            FMath::Clamp((int32)(CurrentHeight + Delta), 0, 65535);
-        if (NewHeight != CurrentHeight) {
-          HeightData[Index] = (uint16)NewHeight;
-          bModified = true;
-        }
-      }
-    }
-
-    if (bModified) {
-      LandscapeEdit.SetHeightData(MinX, MinY, MaxX, MaxY, HeightData.GetData(),
-                                  0, true);
-      LandscapeEdit.Flush();
-      Landscape->PostEditChange();
-    }
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("toolMode"), ToolMode);
-    Resp->SetNumberField(TEXT("modifiedVertices"),
-                         bModified ? HeightData.Num() : 0);
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Landscape sculpted"), Resp,
-                                      FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(RequestingSocket, RequestId, false,
-                         TEXT("sculpt_landscape requires editor build."),
-                         nullptr, TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}
-
-bool UMcpAutomationBridgeSubsystem::HandleSetLandscapeMaterial(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("set_landscape_material"), ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("set_landscape_material payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  FString LandscapePath;
-  Payload->TryGetStringField(TEXT("landscapePath"), LandscapePath);
-  FString LandscapeName;
-  Payload->TryGetStringField(TEXT("landscapeName"), LandscapeName);
-  FString MaterialPath;
-  if (!Payload->TryGetStringField(TEXT("materialPath"), MaterialPath) ||
-      MaterialPath.IsEmpty()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("materialPath required"),
-                        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, LandscapePath,
-                                        LandscapeName, MaterialPath]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    ALandscape *Landscape = nullptr;
-    if (!LandscapePath.IsEmpty()) {
-      Landscape = Cast<ALandscape>(
-          StaticLoadObject(ALandscape::StaticClass(), nullptr, *LandscapePath));
-    }
-    if (!Landscape && !LandscapeName.IsEmpty()) {
-      if (UEditorActorSubsystem *ActorSS =
-              GEditor->GetEditorSubsystem<UEditorActorSubsystem>()) {
-        TArray<AActor *> AllActors = ActorSS->GetAllLevelActors();
-        for (AActor *A : AllActors) {
-          if (A && A->IsA<ALandscape>() &&
-              A->GetActorLabel().Equals(LandscapeName,
-                                        ESearchCase::IgnoreCase)) {
-            Landscape = Cast<ALandscape>(A);
-            break;
-          }
-        }
-      }
-    }
-
-    // Fallback: If no path/name provided (or name not found but let's be
-    // generous if no path was given), find first available landscape
-    if (!Landscape && LandscapePath.IsEmpty() && LandscapeName.IsEmpty()) {
-      if (UEditorActorSubsystem *ActorSS =
-              GEditor->GetEditorSubsystem<UEditorActorSubsystem>()) {
-        TArray<AActor *> AllActors = ActorSS->GetAllLevelActors();
-        for (AActor *A : AllActors) {
-          if (ALandscape *L = Cast<ALandscape>(A)) {
-            Landscape = L;
-            break;
-          }
-        }
-      }
-    }
-    if (!Landscape) {
-      Subsystem->SendAutomationError(
-          RequestingSocket, RequestId,
-          TEXT("Failed to find landscape and no name provided"),
-          TEXT("LOAD_FAILED"));
-      return;
-    }
-
-    // Use Silent load to avoid engine warnings if path is invalid or type
-    // mismatch
-    UMaterialInterface *Mat = Cast<UMaterialInterface>(
-        StaticLoadObject(UMaterialInterface::StaticClass(), nullptr,
-                         *MaterialPath, nullptr, LOAD_NoWarn));
-
-    if (!Mat) {
-      // Check existence separately only if load failed, to distinguish error
-      // type (optional)
-      if (!UEditorAssetLibrary::DoesAssetExist(MaterialPath)) {
-        Subsystem->SendAutomationError(
-            RequestingSocket, RequestId,
-            FString::Printf(TEXT("Material asset not found: %s"),
-                            *MaterialPath),
-            TEXT("ASSET_NOT_FOUND"));
-      } else {
-        Subsystem->SendAutomationError(
-            RequestingSocket, RequestId,
-            TEXT("Failed to load material (invalid type?)"),
-            TEXT("LOAD_FAILED"));
-      }
-      return;
-    }
-
-    Landscape->LandscapeMaterial = Mat;
-    Landscape->PostEditChange();
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("landscapePath"), Landscape->GetPathName());
-    Resp->SetStringField(TEXT("materialPath"), MaterialPath);
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Landscape material set"), Resp,
-                                      FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(RequestingSocket, RequestId, false,
-                         TEXT("set_landscape_material requires editor build."),
-                         nullptr, TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}
-
-bool UMcpAutomationBridgeSubsystem::HandleCreateLandscapeGrassType(
-    const FString &RequestId, const FString &Action,
-    const TSharedPtr<FJsonObject> &Payload,
-    TSharedPtr<FMcpBridgeWebSocket> RequestingSocket) {
-  const FString Lower = Action.ToLower();
-  if (!Lower.Equals(TEXT("create_landscape_grass_type"),
-                    ESearchCase::IgnoreCase)) {
-    return false;
-  }
-
-#if WITH_EDITOR
-  if (!Payload.IsValid()) {
-    SendAutomationError(RequestingSocket, RequestId,
-                        TEXT("create_landscape_grass_type payload missing"),
-                        TEXT("INVALID_PAYLOAD"));
-    return true;
-  }
-
-  FString Name;
-  if (!Payload->TryGetStringField(TEXT("name"), Name) || Name.IsEmpty()) {
-    SendAutomationError(RequestingSocket, RequestId, TEXT("name required"),
-                        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-
-  FString MeshPath;
-  if (!Payload->TryGetStringField(TEXT("meshPath"), MeshPath) ||
-      MeshPath.IsEmpty()) {
-    SendAutomationError(RequestingSocket, RequestId, TEXT("meshPath required"),
-                        TEXT("INVALID_ARGUMENT"));
-    return true;
-  }
-
-  double Density = 1.0;
-  Payload->TryGetNumberField(TEXT("density"), Density);
-
-  double MinScale = 0.8;
-  Payload->TryGetNumberField(TEXT("minScale"), MinScale);
-
-  double MaxScale = 1.2;
-  Payload->TryGetNumberField(TEXT("maxScale"), MaxScale);
-
-  TWeakObjectPtr<UMcpAutomationBridgeSubsystem> WeakSubsystem(this);
-
-  AsyncTask(ENamedThreads::GameThread, [WeakSubsystem, RequestId,
-                                        RequestingSocket, Name, MeshPath,
-                                        Density, MinScale, MaxScale]() {
-    UMcpAutomationBridgeSubsystem *Subsystem = WeakSubsystem.Get();
-    if (!Subsystem)
-      return;
-
-    // Use Silent load to avoid engine warnings
-    UStaticMesh *StaticMesh = Cast<UStaticMesh>(StaticLoadObject(
-        UStaticMesh::StaticClass(), nullptr, *MeshPath, nullptr, LOAD_NoWarn));
-    if (!StaticMesh) {
-      Subsystem->SendAutomationError(
-          RequestingSocket, RequestId,
-          FString::Printf(TEXT("Static mesh not found: %s"), *MeshPath),
-          TEXT("ASSET_NOT_FOUND"));
-      return;
-    }
-
-    FString PackagePath = TEXT("/Game/Landscape");
-    FString AssetName = Name;
-    FString FullPackagePath =
-        FString::Printf(TEXT("%s/%s"), *PackagePath, *AssetName);
-
-    // Check if already exists
-    if (UObject *ExistingAsset = StaticLoadObject(
-            ULandscapeGrassType::StaticClass(), nullptr, *FullPackagePath)) {
-      TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-      Resp->SetBoolField(TEXT("success"), true);
-      Resp->SetStringField(TEXT("asset_path"), ExistingAsset->GetPathName());
-      Resp->SetStringField(TEXT("message"), TEXT("Asset already exists"));
-      Subsystem->SendAutomationResponse(
-          RequestingSocket, RequestId, true,
-          TEXT("Landscape grass type already exists"), Resp, FString());
-      return;
-    }
-
-    UPackage *Package = CreatePackage(*FullPackagePath);
-    ULandscapeGrassType *GrassType = NewObject<ULandscapeGrassType>(
-        Package, FName(*AssetName), RF_Public | RF_Standalone);
-    if (!GrassType) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to create grass type asset"),
-                                     TEXT("CREATION_FAILED"));
-      return;
-    }
-
-    FGrassVariety Variety;
-    Variety.GrassMesh = StaticMesh;
-    Variety.GrassDensity.Default = static_cast<float>(Density);
-
-    Variety.ScaleX = FFloatInterval(static_cast<float>(MinScale),
-                                    static_cast<float>(MaxScale));
-    Variety.ScaleY = FFloatInterval(static_cast<float>(MinScale),
-                                    static_cast<float>(MaxScale));
-    Variety.ScaleZ = FFloatInterval(static_cast<float>(MinScale),
-                                    static_cast<float>(MaxScale));
-
-    Variety.RandomRotation = true;
-    Variety.AlignToSurface = true;
-
-    GrassType->GrassVarieties.Add(Variety);
-
-    Package->MarkPackageDirty();
-    FAssetRegistryModule::AssetCreated(GrassType);
-
-    FString PackageFileName = FPackageName::LongPackageNameToFilename(
-        FullPackagePath, FPackageName::GetAssetPackageExtension());
-    FSavePackageArgs SaveArgs;
-    SaveArgs.TopLevelFlags = RF_Public | RF_Standalone;
-    SaveArgs.Error = GError;
-    SaveArgs.SaveFlags = SAVE_NoError;
-    bool bSaved =
-        UPackage::SavePackage(Package, GrassType, *PackageFileName, SaveArgs);
-
-    if (!bSaved) {
-      Subsystem->SendAutomationError(RequestingSocket, RequestId,
-                                     TEXT("Failed to save grass type asset"),
-                                     TEXT("SAVE_FAILED"));
-      return;
-    }
-
-    TSharedPtr<FJsonObject> Resp = MakeShared<FJsonObject>();
-    Resp->SetBoolField(TEXT("success"), true);
-    Resp->SetStringField(TEXT("asset_path"), GrassType->GetPathName());
-
-    Subsystem->SendAutomationResponse(RequestingSocket, RequestId, true,
-                                      TEXT("Landscape grass type created"),
-                                      Resp, FString());
-  });
-
-  return true;
-#else
-  SendAutomationResponse(
-      RequestingSocket, RequestId, false,
-      TEXT("create_landscape_grass_type requires editor build."), nullptr,
-      TEXT("NOT_IMPLEMENTED"));
-  return true;
-#endif
-}