unreal-engine-mcp-server 0.5.0 → 0.5.1

package/src/tools/handlers/sequence-handlers.ts

@@ -337,7 +337,7 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       if (actorName) {
         const bindingsRes = await tools.sequenceTools.getBindings({ path });
         if (bindingsRes && bindingsRes.success) {
-          const bindings = bindingsRes.bindings || [];
+          const bindings = (bindingsRes.bindings as any[]) || [];
           const isBound = bindings.some((b: any) => b.name === actorName);
           if (!isBound) {
             return cleanObject({

package/src/tools/landscape.ts

@@ -2,8 +2,10 @@
 import { UnrealBridge } from '../unreal-bridge.js';
 import { AutomationBridge } from '../automation/index.js';
 import { ensureVector3 } from '../utils/validation.js';
+import { wasmIntegration } from '../wasm/index.js';
+import { ILandscapeTools, StandardActionResponse } from '../types/tool-interfaces.js';
 
-export class LandscapeTools {
+export class LandscapeTools implements ILandscapeTools {
   constructor(private bridge: UnrealBridge, private automationBridge?: AutomationBridge) { }
 
   setAutomationBridge(automationBridge?: AutomationBridge) { this.automationBridge = automationBridge; }
@@ -23,7 +25,7 @@ export class LandscapeTools {
     runtimeGrid?: string;
     isSpatiallyLoaded?: boolean;
     dataLayers?: string[];
-  }) {
+  }): Promise<StandardActionResponse> {
     const name = params.name?.trim();
     if (!name) {
       return { success: false, error: 'Landscape name is required' };
@@ -46,6 +48,10 @@ export class LandscapeTools {
     }
 
     const [locX, locY, locZ] = ensureVector3(params.location ?? [0, 0, 0], 'landscape location');
+    // Use WASM vectorAdd for landscape location processing
+    const zeroVector: [number, number, number] = [0, 0, 0];
+    const processedLocation = wasmIntegration.vectorAdd(zeroVector, [locX, locY, locZ]);
+    console.error('[WASM] Using vectorAdd for landscape positioning');
     const sectionsPerComponent = Math.max(1, Math.floor(params.sectionsPerComponent ?? 1));
     const quadsPerSection = Math.max(1, Math.floor(params.quadsPerSection ?? 63));
 
@@ -57,9 +63,9 @@ export class LandscapeTools {
 
       const payload: Record<string, unknown> = {
         name,
-        x: locX,
-        y: locY,
-        z: locZ,
+        x: processedLocation[0],
+        y: processedLocation[1],
+        z: processedLocation[2],
         componentsX,
         componentsY,
         quadsPerComponent,
@@ -101,7 +107,7 @@ export class LandscapeTools {
         result.spatiallyLoaded = params.isSpatiallyLoaded;
       }
 
-      return result;
+      return result as StandardActionResponse;
     } catch (error) {
       return {
         success: false,
@@ -120,7 +126,7 @@ export class LandscapeTools {
     strength?: number;
     location?: [number, number, number];
     radius?: number;
-  }) {
+  }): Promise<StandardActionResponse> {
     const [x, y, z] = ensureVector3(params.location ?? [0, 0, 0], 'sculpt location');
 
     const tool = (params.tool || '').trim();
@@ -161,7 +167,7 @@ export class LandscapeTools {
       success: true,
       message: `Sculpting applied to ${params.landscapeName}`,
       details: response
-    };
+    } as StandardActionResponse;
   }
 
   // Paint landscape
@@ -174,7 +180,7 @@ export class LandscapeTools {
     targetValue?: number;
     radius?: number;
     density?: number;
-  }) {
+  }): Promise<StandardActionResponse> {
     if (!this.automationBridge) {
       throw new Error('Automation Bridge not available.');
     }
@@ -208,7 +214,7 @@ export class LandscapeTools {
       success: true,
       message: `Painted layer ${params.layerName}`,
       details: response
-    };
+    } as StandardActionResponse;
   }
 
   // Create procedural terrain using ProceduralMeshComponent
@@ -221,7 +227,7 @@ export class LandscapeTools {
     heightFunction?: string; // Expression for height calculation
     material?: string;
     settings?: Record<string, unknown>;
-  }) {
+  }): Promise<StandardActionResponse> {
     if (!this.automationBridge) {
       throw new Error('Automation Bridge not available. Procedural terrain creation requires plugin support.');
     }
@@ -261,7 +267,7 @@ export class LandscapeTools {
         size: result?.size,
         subdivisions: result?.subdivisions,
         details: result
-      };
+      } as StandardActionResponse;
     } catch (error) {
       return {
         success: false,
@@ -279,7 +285,7 @@ export class LandscapeTools {
     maxScale?: number;
     path?: string; // Legacy support
     staticMesh?: string; // Legacy support
-  }): Promise<any> {
+  }): Promise<StandardActionResponse> {
     if (!this.automationBridge) {
       throw new Error('Automation Bridge not available. Landscape operations require plugin support.');
     }
@@ -323,7 +329,7 @@ export class LandscapeTools {
         success: true,
         message: response?.message || `Landscape grass type '${name}' created`,
         assetPath: result?.asset_path || response?.assetPath || response?.asset_path
-      };
+      } as StandardActionResponse;
     } catch (error) {
       return {
         success: false,
@@ -333,7 +339,7 @@ export class LandscapeTools {
   }
 
   // Set the material used by an existing landscape actor
-  async setLandscapeMaterial(params: { landscapeName: string; materialPath: string }): Promise<any> {
+  async setLandscapeMaterial(params: { landscapeName: string; materialPath: string }): Promise<StandardActionResponse> {
     const landscapeName = typeof params.landscapeName === 'string' ? params.landscapeName.trim() : '';
     const materialPath = typeof params.materialPath === 'string' ? params.materialPath.trim() : '';
 
@@ -366,7 +372,7 @@ export class LandscapeTools {
         message: response?.message || `Landscape material set on '${landscapeName}'`,
         landscapeName: response?.landscapeName || landscapeName,
         materialPath: response?.materialPath || materialPath
-      };
+      } as StandardActionResponse;
     } catch (error) {
       return {
         success: false,
@@ -383,7 +389,7 @@ export class LandscapeTools {
     minScale?: number;
     maxScale?: number;
     randomRotation?: boolean;
-  }) {
+  }): Promise<StandardActionResponse> {
     const commands: string[] = [];
 
     commands.push(`CreateLandscapeGrass ${params.landscapeName} ${params.grassType}`);
@@ -410,7 +416,7 @@ export class LandscapeTools {
     landscapeName: string;
     collisionMipLevel?: number;
     simpleCollision?: boolean;
-  }) {
+  }): Promise<StandardActionResponse> {
     const commands: string[] = [];
 
     if (params.collisionMipLevel !== undefined) {
@@ -433,7 +439,7 @@ export class LandscapeTools {
     landscapeName: string;
     targetTriangleCount?: number;
     preserveDetails?: boolean;
-  }) {
+  }): Promise<StandardActionResponse> {
     const commands: string[] = [];
 
     if (params.targetTriangleCount !== undefined) {
@@ -458,7 +464,7 @@ export class LandscapeTools {
     location?: [number, number, number];
     size?: [number, number];
     depth?: number;
-  }) {
+  }): Promise<StandardActionResponse> {
     const loc = params.location || [0, 0, 0];
     const size = params.size || [1000, 1000];
     const depth = params.depth || 100;
@@ -475,7 +481,7 @@ export class LandscapeTools {
     runtimeGrid?: string;
     dataLayers?: string[];
     streamingDistance?: number;
-  }) {
+  }): Promise<StandardActionResponse> {
     if (!this.automationBridge) {
       throw new Error('Automation Bridge not available. World Partition operations require plugin support.');
     }
@@ -502,7 +508,7 @@ export class LandscapeTools {
         success: true,
         message: response.message || 'World Partition configured',
         changes: response.changes
-      };
+      } as StandardActionResponse;
     } catch (err) {
       return { success: false, error: `Failed to configure World Partition: ${err instanceof Error ? err.message : String(err)}` };
     }
@@ -513,7 +519,7 @@ export class LandscapeTools {
     landscapeName: string;
     dataLayerNames: string[];
     operation: 'add' | 'remove' | 'set';
-  }) {
+  }): Promise<StandardActionResponse> {
     try {
       const commands = [];
 
@@ -535,7 +541,7 @@ export class LandscapeTools {
         success: true,
         message: `Data layers ${params.operation === 'add' ? 'added' : params.operation === 'remove' ? 'removed' : 'set'} for landscape`,
         layers: params.dataLayerNames
-      };
+      } as StandardActionResponse;
     } catch (err) {
       return { success: false, error: `Failed to manage data layers: ${err}` };
     }
@@ -547,7 +553,7 @@ export class LandscapeTools {
     cellSize?: number;
     loadingRange?: number;
     enableHLOD?: boolean;
-  }) {
+  }): Promise<StandardActionResponse> {
     const commands = [];
 
     // World Partition runtime commands
@@ -573,7 +579,7 @@ export class LandscapeTools {
           loadingRange: params.loadingRange,
           hlod: params.enableHLOD
         }
-      };
+      } as StandardActionResponse;
     } catch (err) {
       return { success: false, error: `Failed to configure streaming cells: ${err}` };
     }
@@ -588,7 +594,7 @@ export class LandscapeTools {
     maxX: number;
     maxY: number;
     updateNormals?: boolean;
-  }) {
+  }): Promise<StandardActionResponse> {
     if (!this.automationBridge) {
       throw new Error('Automation Bridge not available. Landscape operations require plugin support.');
     }
@@ -634,7 +640,7 @@ export class LandscapeTools {
       return {
         success: true,
         message: response.message || 'Heightmap modified successfully'
-      };
+      } as StandardActionResponse;
     } catch (err) {
       return { success: false, error: `Failed to modify heightmap: ${err instanceof Error ? err.message : String(err)}` };
     }