unreal-engine-mcp-server 0.5.0 → 0.5.1

package/dist/tools/landscape.d.ts

@@ -1,6 +1,7 @@
 import { UnrealBridge } from '../unreal-bridge.js';
 import { AutomationBridge } from '../automation/index.js';
-export declare class LandscapeTools {
+import { ILandscapeTools, StandardActionResponse } from '../types/tool-interfaces.js';
+export declare class LandscapeTools implements ILandscapeTools {
     private bridge;
     private automationBridge?;
     constructor(bridge: UnrealBridge, automationBridge?: AutomationBridge | undefined);
@@ -18,7 +19,7 @@ export declare class LandscapeTools {
         runtimeGrid?: string;
         isSpatiallyLoaded?: boolean;
         dataLayers?: string[];
-    }): Promise<Record<string, unknown>>;
+    }): Promise<StandardActionResponse>;
     sculptLandscape(params: {
         landscapeName: string;
         tool: string;
@@ -27,17 +28,7 @@ export declare class LandscapeTools {
         strength?: number;
         location?: [number, number, number];
         radius?: number;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        message?: undefined;
-        details?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        details: import("../automation/types.js").AutomationBridgeResponseMessage;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     paintLandscape(params: {
         landscapeName: string;
         layerName: string;
@@ -47,17 +38,7 @@ export declare class LandscapeTools {
         targetValue?: number;
         radius?: number;
         density?: number;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        message?: undefined;
-        details?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        details: import("../automation/types.js").AutomationBridgeResponseMessage;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     createProceduralTerrain(params: {
         name: string;
         location?: [number, number, number];
@@ -67,37 +48,7 @@ export declare class LandscapeTools {
         heightFunction?: string;
         material?: string;
         settings?: Record<string, unknown>;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        message: string;
-        actorName?: undefined;
-        vertices?: undefined;
-        triangles?: undefined;
-        size?: undefined;
-        subdivisions?: undefined;
-        details?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        actorName: any;
-        vertices: any;
-        triangles: any;
-        size: any;
-        subdivisions: any;
-        details: any;
-        error?: undefined;
-    } | {
-        success: boolean;
-        error: string;
-        message?: undefined;
-        actorName?: undefined;
-        vertices?: undefined;
-        triangles?: undefined;
-        size?: undefined;
-        subdivisions?: undefined;
-        details?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     createLandscapeGrassType(params: {
         name: string;
         meshPath: string;
@@ -106,11 +57,11 @@ export declare class LandscapeTools {
         maxScale?: number;
         path?: string;
         staticMesh?: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setLandscapeMaterial(params: {
         landscapeName: string;
         materialPath: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     createLandscapeGrass(params: {
         landscapeName: string;
         grassType: string;
@@ -118,85 +69,42 @@ export declare class LandscapeTools {
         minScale?: number;
         maxScale?: number;
         randomRotation?: boolean;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     updateLandscapeCollision(params: {
         landscapeName: string;
         collisionMipLevel?: number;
         simpleCollision?: boolean;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     retopologizeLandscape(params: {
         landscapeName: string;
         targetTriangleCount?: number;
         preserveDetails?: boolean;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     createWaterBody(params: {
         type: 'Ocean' | 'Lake' | 'River' | 'Stream';
         name: string;
         location?: [number, number, number];
         size?: [number, number];
         depth?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     configureWorldPartition(params: {
         landscapeName: string;
         enableSpatialLoading?: boolean;
         runtimeGrid?: string;
         dataLayers?: string[];
         streamingDistance?: number;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        message?: undefined;
-        changes?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        changes: unknown;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     setDataLayers(params: {
         landscapeName: string;
         dataLayerNames: string[];
         operation: 'add' | 'remove' | 'set';
-    }): Promise<{
-        success: boolean;
-        message: string;
-        layers: string[];
-        error?: undefined;
-    } | {
-        success: boolean;
-        error: string;
-        message?: undefined;
-        layers?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     configureStreamingCells(params: {
         landscapeName: string;
         cellSize?: number;
         loadingRange?: number;
         enableHLOD?: boolean;
-    }): Promise<{
-        success: boolean;
-        message: string;
-        settings: {
-            cellSize: number | undefined;
-            loadingRange: number | undefined;
-            hlod: boolean | undefined;
-        };
-        error?: undefined;
-    } | {
-        success: boolean;
-        error: string;
-        message?: undefined;
-        settings?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     modifyHeightmap(params: {
         landscapeName: string;
         heightData: number[];
@@ -205,14 +113,6 @@ export declare class LandscapeTools {
         maxX: number;
         maxY: number;
         updateNormals?: boolean;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        message?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
 }
 //# sourceMappingURL=landscape.d.ts.map

package/dist/tools/landscape.js

@@ -1,4 +1,5 @@
 import { ensureVector3 } from '../utils/validation.js';
+import { wasmIntegration } from '../wasm/index.js';
 export class LandscapeTools {
     bridge;
     automationBridge;
@@ -28,6 +29,9 @@ export class LandscapeTools {
             throw new Error('Automation Bridge not available. Landscape operations require plugin support.');
         }
         const [locX, locY, locZ] = ensureVector3(params.location ?? [0, 0, 0], 'landscape location');
+        const zeroVector = [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));
         try {
@@ -36,9 +40,9 @@ export class LandscapeTools {
             const quadsPerComponent = quadsPerSection;
             const payload = {
                 name,
-                x: locX,
-                y: locY,
-                z: locZ,
+                x: processedLocation[0],
+                y: processedLocation[1],
+                z: processedLocation[2],
                 componentsX,
                 componentsY,
                 quadsPerComponent,

package/dist/tools/level.d.ts

@@ -1,5 +1,5 @@
 import { BaseTool } from './base-tool.js';
-import { ILevelTools } from '../types/tool-interfaces.js';
+import { ILevelTools, StandardActionResponse } from '../types/tool-interfaces.js';
 export declare class LevelTools extends BaseTool implements ILevelTools {
     private managedLevels;
     private listCache?;
@@ -15,13 +15,8 @@ export declare class LevelTools extends BaseTool implements ILevelTools {
     private listManagedLevels;
     private summarizeLevel;
     private setCurrentLevel;
-    listLevels(): Promise<Record<string, unknown> | {
-        success: true;
-        message: string;
-        count: number;
-        levels: Array<Record<string, unknown>>;
-    }>;
-    getLevelSummary(levelPath?: string): Promise<Record<string, unknown>>;
+    listLevels(): Promise<StandardActionResponse>;
+    getLevelSummary(levelPath?: string): Promise<StandardActionResponse>;
     registerLight(levelPath: string | undefined, info: {
         name: string;
         type: string;
@@ -32,122 +27,52 @@ export declare class LevelTools extends BaseTool implements ILevelTools {
         exportPath: string;
         note?: string;
         timeoutMs?: number;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        levelPath?: undefined;
-        exportPath?: undefined;
-        details?: undefined;
-        message?: undefined;
-    } | {
-        success: boolean;
-        error: any;
-        levelPath: string;
-        exportPath: string;
-        details: any;
-        message?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        levelPath: string;
-        exportPath: string;
-        details: any;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     importLevel(params: {
         packagePath: string;
         destinationPath?: string;
         streaming?: boolean;
         timeoutMs?: number;
-    }): Promise<{
-        success: boolean;
-        error: any;
-        levelPath: string;
-        details: any;
-        message?: undefined;
-        partitioned?: undefined;
-        streaming?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        levelPath: string;
-        partitioned: boolean;
-        streaming: boolean;
-        details: any;
-        error?: undefined;
-    } | {
-        success: boolean;
-        error: string;
-        levelPath?: undefined;
-        details?: undefined;
-        message?: undefined;
-        partitioned?: undefined;
-        streaming?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     saveLevelAs(params: {
         sourcePath?: string;
         targetPath: string;
-    }): Promise<{
-        success: boolean;
-        error: any;
-        message?: undefined;
-        levelPath?: undefined;
-    } | {
-        success: boolean;
-        message: any;
-        levelPath: string;
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     deleteLevels(params: {
         levelPaths: string[];
-    }): Promise<{
-        success: boolean;
-        message: string;
-        removed: string[];
-    }>;
+    }): Promise<StandardActionResponse>;
     loadLevel(params: {
         levelPath: string;
         streaming?: boolean;
         position?: [number, number, number];
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     saveLevel(params: {
         levelName?: string;
         savePath?: string;
-    }): Promise<Record<string, unknown> | {
-        success: boolean;
-        error: any;
-    }>;
+    }): Promise<StandardActionResponse>;
     createLevel(params: {
         levelName: string;
         template?: 'Empty' | 'Default' | 'VR' | 'TimeOfDay';
         savePath?: string;
-    }): Promise<Record<string, unknown> | {
-        success: boolean;
-        error: any;
-        path: string;
-        partitioned: boolean;
-    }>;
+    }): Promise<StandardActionResponse>;
     addSubLevel(params: {
         parentLevel?: string;
         subLevelPath: string;
         streamingMethod?: 'Blueprint' | 'AlwaysLoaded';
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     streamLevel(params: {
         levelPath?: string;
         levelName?: string;
         shouldBeLoaded: boolean;
         shouldBeVisible?: boolean;
         position?: [number, number, number];
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setupWorldComposition(params: {
         enableComposition: boolean;
         tileSize?: number;
         distanceStreaming?: boolean;
         streamingDistance?: number;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     editLevelBlueprint(params: {
         eventType: 'BeginPlay' | 'EndPlay' | 'Tick' | 'Custom';
         customEventName?: string;
@@ -156,50 +81,44 @@ export declare class LevelTools extends BaseTool implements ILevelTools {
             position: [number, number];
             connections?: string[];
         }>;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     createSubLevel(params: {
         name: string;
         type: 'Persistent' | 'Streaming' | 'Lighting' | 'Gameplay';
         parent?: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setWorldSettings(params: {
         gravity?: number;
         worldScale?: number;
         gameMode?: string;
         defaultPawn?: string;
         killZ?: number;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     setLevelBounds(params: {
         min: [number, number, number];
         max: [number, number, number];
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     buildNavMesh(params: {
         rebuildAll?: boolean;
         selectedOnly?: boolean;
-    }): Promise<Record<string, unknown> | {
-        success: boolean;
-        error: any;
-    }>;
+    }): Promise<StandardActionResponse>;
     setLevelVisibility(params: {
         levelName: string;
         visible: boolean;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setWorldOrigin(params: {
         location: [number, number, number];
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     createStreamingVolume(params: {
         levelName: string;
         position: [number, number, number];
         size: [number, number, number];
         streamingDistance?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setLevelLOD(params: {
         levelName: string;
         lodLevel: number;
         distance: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
 }
 //# sourceMappingURL=level.d.ts.map

package/dist/tools/level.js

@@ -1,4 +1,6 @@
 import { BaseTool } from './base-tool.js';
+import { sanitizePath } from '../utils/path-security.js';
+import { DEFAULT_OPERATION_TIMEOUT_MS, DEFAULT_ASSET_OP_TIMEOUT_MS, LONG_RUNNING_OP_TIMEOUT_MS } from '../constants.js';
 export class LevelTools extends BaseTool {
     managedLevels = new Map();
     listCache;
@@ -18,6 +20,12 @@ export class LevelTools extends BaseTool {
         if (!formatted.startsWith('/Game/')) {
             formatted = `/Game/${formatted.replace(/^\/+/, '')}`;
         }
+        try {
+            formatted = sanitizePath(formatted);
+        }
+        catch (e) {
+            throw new Error(`Security validation failed for level path: ${e.message}`);
+        }
         formatted = formatted.replace(/\.umap$/i, '');
         if (formatted.endsWith('/')) {
             formatted = formatted.slice(0, -1);
@@ -180,6 +188,7 @@ export class LevelTools extends BaseTool {
                 const managedOnly = managed.levels.filter(m => !ueLevels.some(u => u.path === m.path));
                 const finalLevels = [...ueLevels, ...managedOnly];
                 const result = {
+                    ...response,
                     success: true,
                     message: 'Levels listed from Unreal Engine',
                     levels: finalLevels,
@@ -190,7 +199,6 @@ export class LevelTools extends BaseTool {
                         levels: finalLevels,
                         count: finalLevels.length
                     },
-                    ...response,
                     managedLevels: managed.levels,
                     managedLevelCount: managed.count
                 };
@@ -234,7 +242,7 @@ export class LevelTools extends BaseTool {
                 action: 'export_level',
                 levelPath: resolved,
                 exportPath: params.exportPath
-            }, { timeoutMs: params.timeoutMs ?? 300000 });
+            }, { timeoutMs: params.timeoutMs ?? LONG_RUNNING_OP_TIMEOUT_MS });
             if (res?.success === false) {
                 return {
                     success: false,
@@ -265,7 +273,7 @@ export class LevelTools extends BaseTool {
                 action: 'import_level',
                 packagePath: params.packagePath,
                 destinationPath: destination.path
-            }, { timeoutMs: params.timeoutMs ?? 300000 });
+            }, { timeoutMs: params.timeoutMs ?? LONG_RUNNING_OP_TIMEOUT_MS });
             if (res?.success === false) {
                 return {
                     success: false,
@@ -295,7 +303,7 @@ export class LevelTools extends BaseTool {
                 action: 'save_level_as',
                 savePath: target.path
             }, {
-                timeoutMs: 60000
+                timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS
             });
             if (response.success === false) {
                 return { success: false, error: response.error || response.message || 'Failed to save level as' };
@@ -382,7 +390,7 @@ export class LevelTools extends BaseTool {
                 const response = await this.sendAutomationRequest('manage_level', {
                     action: 'load',
                     levelPath: params.levelPath
-                }, { timeoutMs: 30000 });
+                }, { timeoutMs: DEFAULT_OPERATION_TIMEOUT_MS });
                 if (response.success) {
                     this.setCurrentLevel(normalizedPath);
                     this.mutateRecord(normalizedPath, {
@@ -391,11 +399,11 @@ export class LevelTools extends BaseTool {
                         visible: true
                     });
                     return {
+                        ...response,
                         success: true,
                         message: `Level loaded: ${params.levelPath}`,
                         level: normalizedPath,
-                        streaming: false,
-                        ...response
+                        streaming: false
                     };
                 }
             }
@@ -461,15 +469,15 @@ export class LevelTools extends BaseTool {
                 payload.savePath = params.savePath;
             }
             const response = await this.sendAutomationRequest('manage_level', payload, {
-                timeoutMs: 60000
+                timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS
             });
             if (response.success === false) {
                 return { success: false, error: response.error || response.message || 'Failed to save level' };
             }
             const result = {
+                ...response,
                 success: true,
-                message: response.message || 'Level saved',
-                ...response
+                message: response.message || 'Level saved'
             };
             if (response.skipped) {
                 result.skipped = response.skipped;
@@ -498,7 +506,7 @@ export class LevelTools extends BaseTool {
                 levelPath: fullPath,
                 useWorldPartition: isPartitioned
             }, {
-                timeoutMs: 60000
+                timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS
             });
             if (response.success === false) {
                 return {
@@ -509,13 +517,13 @@ export class LevelTools extends BaseTool {
                 };
             }
             const result = {
+                ...response,
                 success: true,
                 message: response.message || 'Level created',
                 path: response.levelPath || fullPath,
                 packagePath: response.packagePath ?? fullPath,
                 objectPath: response.objectPath,
-                partitioned: isPartitioned,
-                ...response
+                partitioned: isPartitioned
             };
             if (response.warnings) {
                 result.warnings = response.warnings;
@@ -551,7 +559,7 @@ export class LevelTools extends BaseTool {
                 subLevelPath: sub,
                 parentPath: parent,
                 streamingMethod: params.streamingMethod
-            }, { timeoutMs: 30000 });
+            }, { timeoutMs: DEFAULT_OPERATION_TIMEOUT_MS });
             if (response && (response.error === 'PACKAGE_NOT_FOUND' || response.error === 'ADD_FAILED') && !sub.endsWith('.umap')) {
                 const subWithExt = sub + '.umap';
                 response = await this.sendAutomationRequest('manage_level', {
@@ -560,7 +568,7 @@ export class LevelTools extends BaseTool {
                     subLevelPath: subWithExt,
                     parentPath: parent,
                     streamingMethod: params.streamingMethod
-                }, { timeoutMs: 30000 });
+                }, { timeoutMs: DEFAULT_OPERATION_TIMEOUT_MS });
             }
             if (response.success) {
                 this.ensureRecord(sub, { loaded: true, visible: true, streaming: true });
@@ -608,7 +616,7 @@ export class LevelTools extends BaseTool {
                 shouldBeLoaded: params.shouldBeLoaded,
                 shouldBeVisible
             }, {
-                timeoutMs: 60000
+                timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS
             });
             if (response.success === false) {
                 const errorCode = typeof response.error === 'string' ? response.error : '';

package/dist/tools/lighting.js

@@ -1,4 +1,5 @@
 import { ensureVector3 } from '../utils/validation.js';
+import { wasmIntegration } from '../wasm/index.js';
 export class LightingTools {
     bridge;
     automationBridge;
@@ -32,7 +33,10 @@ export class LightingTools {
                 name: params.name,
             };
             if (params.location) {
-                payload.location = { x: params.location[0], y: params.location[1], z: params.location[2] };
+                const zeroVector = [0, 0, 0];
+                const processedLocation = wasmIntegration.vectorAdd(zeroVector, params.location);
+                console.error('[WASM] Using vectorAdd for light positioning');
+                payload.location = { x: processedLocation[0], y: processedLocation[1], z: processedLocation[2] };
             }
             if (params.rotation) {
                 if (Array.isArray(params.rotation)) {

package/dist/tools/materials.js

@@ -129,7 +129,11 @@ export class MaterialTools {
                         return { success: false, error: resp?.error ?? resp?.message ?? 'CREATE_MATERIAL_INSTANCE_FAILED' };
                     }
                 }
-                catch (_e) {
+                catch (e) {
+                    const msg = e instanceof Error ? e.message : String(e);
+                    if (!msg.includes('unknown') && !msg.includes('UNKNOWN_PLUGIN_ACTION')) {
+                        console.warn(`[MaterialTools] Plugin create_material_instance failed with specific error (falling back to python): ${msg}`);
+                    }
                 }
             }
             const createParams = {