unreal-engine-mcp-server 0.5.0 → 0.5.1

package/dist/tools/assets.d.ts

@@ -1,5 +1,5 @@
 import { BaseTool } from './base-tool.js';
-import { IAssetTools } from '../types/tool-interfaces.js';
+import { IAssetTools, StandardActionResponse, SourceControlState } from '../types/tool-interfaces.js';
 export declare class AssetTools extends BaseTool implements IAssetTools {
     private normalizeAssetPath;
     importAsset(params: {
@@ -7,124 +7,72 @@ export declare class AssetTools extends BaseTool implements IAssetTools {
         destinationPath: string;
         overwrite?: boolean;
         save?: boolean;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     duplicateAsset(params: {
         sourcePath: string;
         destinationPath: string;
         overwrite?: boolean;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     renameAsset(params: {
         sourcePath: string;
         destinationPath: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     moveAsset(params: {
         sourcePath: string;
         destinationPath: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     findByTag(params: {
         tag: string;
         value?: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     deleteAssets(params: {
         paths: string[];
         fixupRedirectors?: boolean;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     searchAssets(params: {
         classNames?: string[];
         packagePaths?: string[];
         recursivePaths?: boolean;
         recursiveClasses?: boolean;
         limit?: number;
-    }): Promise<{
-        success: boolean;
-        error: any;
-        message?: undefined;
-        assets?: undefined;
-        count?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        assets: any[];
-        count: number;
-        error?: undefined;
-    }>;
-    saveAsset(assetPath: string): Promise<any>;
-    createFolder(folderPath: string): Promise<any>;
+    }): Promise<StandardActionResponse>;
+    saveAsset(assetPath: string): Promise<StandardActionResponse>;
+    createFolder(folderPath: string): Promise<StandardActionResponse>;
     getDependencies(params: {
         assetPath: string;
         recursive?: boolean;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getSourceControlState(params: {
         assetPath: string;
-    }): Promise<any>;
+    }): Promise<SourceControlState | StandardActionResponse>;
     getMetadata(params: {
         assetPath: string;
-    }): Promise<{
-        success: boolean;
-        message: string;
-    }>;
+    }): Promise<StandardActionResponse>;
     analyzeGraph(params: {
         assetPath: string;
         maxDepth?: number;
-    }): Promise<{
-        success: boolean;
-        error: any;
-        message?: undefined;
-        analysis?: undefined;
-    } | {
-        success: boolean;
-        message: string;
-        analysis: {
-            asset: any;
-            dependencies: any[];
-            totalDependencyCount: any;
-            requestedMaxDepth: number;
-            maxDepthUsed: number;
-            circularDependencies: string[][];
-            topologicalOrder: string[];
-            stats: {
-                nodeCount: number;
-                leafCount: number;
-            };
-        };
-        error?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     createThumbnail(params: {
         assetPath: string;
         width?: number;
         height?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setTags(params: {
         assetPath: string;
         tags: string[];
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     generateReport(params: {
         directory: string;
         reportType?: string;
         outputPath?: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     validate(params: {
         assetPath: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     generateLODs(params: {
         assetPath: string;
         lodCount: number;
-    }): Promise<{
-        success: boolean;
-        error: string;
-        details?: undefined;
-    } | {
-        success: boolean;
-        error: any;
-        details: any;
-    } | {
-        success: boolean;
-        message: any;
-        assetPath: string;
-        lodCount: number;
-        error?: undefined;
-        details?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
 }
 //# sourceMappingURL=assets.d.ts.map

package/dist/tools/assets.js

@@ -1,5 +1,9 @@
 import { BaseTool } from './base-tool.js';
 import { wasmIntegration } from '../wasm/index.js';
+import { Logger } from '../utils/logger.js';
+import { sanitizePath } from '../utils/path-security.js';
+import { DEFAULT_ASSET_OP_TIMEOUT_MS, EXTENDED_ASSET_OP_TIMEOUT_MS, LONG_RUNNING_OP_TIMEOUT_MS } from '../constants.js';
+const log = new Logger('AssetTools');
 export class AssetTools extends BaseTool {
     normalizeAssetPath(path) {
         if (!path)
@@ -15,13 +19,14 @@ export class AssetTools extends BaseTool {
             else
                 normalized = '/Game/' + normalized;
         }
-        return normalized.replace(/\/+/g, '/');
+        normalized = normalized.replace(/\/+/g, '/');
+        return sanitizePath(normalized);
     }
     async importAsset(params) {
         const res = await this.sendRequest('manage_asset', {
             ...params,
             subAction: 'import'
-        }, 'manage_asset', { timeoutMs: 120000 });
+        }, 'manage_asset', { timeoutMs: EXTENDED_ASSET_OP_TIMEOUT_MS });
         if (res && res.success) {
             return { ...res, asset: this.normalizeAssetPath(params.destinationPath), source: params.sourcePath };
         }
@@ -35,7 +40,7 @@ export class AssetTools extends BaseTool {
             destinationPath,
             overwrite: params.overwrite ?? false,
             subAction: 'duplicate'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
         if (res && res.success) {
             return { ...res, asset: destinationPath, source: sourcePath };
         }
@@ -48,7 +53,7 @@ export class AssetTools extends BaseTool {
             sourcePath,
             destinationPath,
             subAction: 'rename'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
         if (res && res.success) {
             return { ...res, asset: destinationPath, oldName: sourcePath };
         }
@@ -61,7 +66,7 @@ export class AssetTools extends BaseTool {
             sourcePath,
             destinationPath,
             subAction: 'move'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
         if (res && res.success) {
             return { ...res, asset: destinationPath, from: sourcePath };
         }
@@ -71,7 +76,7 @@ export class AssetTools extends BaseTool {
         return this.sendRequest('asset_query', {
             ...params,
             subAction: 'find_by_tag'
-        }, 'asset_query', { timeoutMs: 60000 });
+        }, 'asset_query', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
     }
     async deleteAssets(params) {
         const assetPaths = (Array.isArray(params.paths) ? params.paths : [])
@@ -80,7 +85,7 @@ export class AssetTools extends BaseTool {
             assetPaths,
             fixupRedirectors: params.fixupRedirectors,
             subAction: 'delete'
-        }, 'manage_asset', { timeoutMs: params.timeoutMs || 120000 });
+        }, 'manage_asset', { timeoutMs: params.timeoutMs || EXTENDED_ASSET_OP_TIMEOUT_MS });
     }
     async searchAssets(params) {
         const packagePaths = params.packagePaths
@@ -90,10 +95,10 @@ export class AssetTools extends BaseTool {
             ...params,
             packagePaths,
             subAction: 'search_assets'
-        }, 'asset_query', { timeoutMs: 60000 });
+        }, 'asset_query', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
         if (!response.success) {
-            const errorMsg = response.error || `Failed to search assets. Raw response: ${JSON.stringify(response)}`;
-            return { success: false, error: errorMsg };
+            const errorMsg = typeof response.error === 'string' ? response.error : JSON.stringify(response.error);
+            return { success: false, error: errorMsg || 'Failed to search assets' };
         }
         const assetsRaw = response.assets || response.data || response.result;
         const assets = Array.isArray(assetsRaw) ? assetsRaw : [];
@@ -110,7 +115,7 @@ export class AssetTools extends BaseTool {
             const bridge = this.getAutomationBridge();
             if (bridge && typeof bridge.sendAutomationRequest === 'function') {
                 try {
-                    const response = await bridge.sendAutomationRequest('manage_asset', { assetPath: normalizedPath, subAction: 'save_asset' }, { timeoutMs: 60000 });
+                    const response = await bridge.sendAutomationRequest('manage_asset', { assetPath: normalizedPath, subAction: 'save_asset' }, { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
                     if (response && response.success !== false) {
                         return {
                             success: true,
@@ -120,7 +125,8 @@ export class AssetTools extends BaseTool {
                         };
                     }
                 }
-                catch (_err) {
+                catch (primaryError) {
+                    log.debug('saveAsset primary method failed, trying fallback', primaryError);
                 }
             }
             const res = await this.bridge.executeEditorFunction('SAVE_ASSET', { path: normalizedPath });
@@ -131,7 +137,7 @@ export class AssetTools extends BaseTool {
             return { success: false, error: res?.error ?? 'Failed to save asset' };
         }
         catch (err) {
-            return { success: false, error: `Failed to save asset: ${err}` };
+            return { success: false, error: `Failed to save asset: ${err} ` };
         }
     }
     async createFolder(folderPath) {
@@ -139,7 +145,7 @@ export class AssetTools extends BaseTool {
         return this.sendRequest('manage_asset', {
             path,
             subAction: 'create_folder'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
     }
     async getDependencies(params) {
         return this.sendRequest('manage_asset', {
@@ -176,7 +182,7 @@ export class AssetTools extends BaseTool {
                 assetPath,
                 maxDepth,
                 subAction: 'get_asset_graph'
-            }, 'manage_asset', { timeoutMs: 60000 });
+            }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
             if (!response.success || !response.graph) {
                 return { success: false, error: response.error || 'Failed to retrieve asset graph from engine' };
             }
@@ -216,7 +222,7 @@ export class AssetTools extends BaseTool {
             };
         }
         catch (e) {
-            return { success: false, error: `Analysis failed: ${e.message}` };
+            return { success: false, error: `Analysis failed: ${e.message} ` };
         }
     }
     async createThumbnail(params) {
@@ -224,28 +230,28 @@ export class AssetTools extends BaseTool {
             ...params,
             assetPath: this.normalizeAssetPath(params.assetPath),
             subAction: 'generate_thumbnail'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
     }
     async setTags(params) {
         return this.sendRequest('manage_asset', {
             ...params,
             assetPath: this.normalizeAssetPath(params.assetPath),
             subAction: 'set_tags'
-        }, 'manage_asset', { timeoutMs: 60000 });
+        }, 'manage_asset', { timeoutMs: DEFAULT_ASSET_OP_TIMEOUT_MS });
     }
     async generateReport(params) {
         return this.sendRequest('manage_asset', {
             ...params,
             directory: this.normalizeAssetPath(params.directory),
             subAction: 'generate_report'
-        }, 'manage_asset', { timeoutMs: 300000 });
+        }, 'manage_asset', { timeoutMs: LONG_RUNNING_OP_TIMEOUT_MS });
     }
     async validate(params) {
         return this.sendRequest('manage_asset', {
             ...params,
             assetPath: this.normalizeAssetPath(params.assetPath),
             subAction: 'validate'
-        }, 'manage_asset', { timeoutMs: 300000 });
+        }, 'manage_asset', { timeoutMs: LONG_RUNNING_OP_TIMEOUT_MS });
     }
     async generateLODs(params) {
         const assetPath = this.normalizeAssetPath(String(params.assetPath ?? '').trim());
@@ -263,7 +269,7 @@ export class AssetTools extends BaseTool {
                 assetPaths: [assetPath],
                 numLODs: lodCount,
                 subAction: 'generate_lods'
-            }, { timeoutMs: 120000 });
+            }, { timeoutMs: EXTENDED_ASSET_OP_TIMEOUT_MS });
             if (!response || response.success === false) {
                 return {
                     success: false,
@@ -283,7 +289,7 @@ export class AssetTools extends BaseTool {
         catch (error) {
             return {
                 success: false,
-                error: `Failed to generate LODs: ${error instanceof Error ? error.message : String(error)}`
+                error: `Failed to generate LODs: ${error instanceof Error ? error.message : String(error)} `
             };
         }
     }

package/dist/tools/base-tool.d.ts

@@ -5,13 +5,13 @@ export declare abstract class BaseTool implements IBaseTool {
     protected bridge: UnrealBridge;
     constructor(bridge: UnrealBridge);
     getAutomationBridge(): AutomationBridge;
-    protected sendRequest(action: string, params: Record<string, unknown>, toolName?: string, options?: {
+    protected sendRequest<T = unknown>(action: string, params: Record<string, unknown>, toolName?: string, options?: {
         timeoutMs?: number;
-    }): Promise<any>;
-    protected sendAutomationRequest(action: string, params?: Record<string, unknown>, options?: {
+    }): Promise<T>;
+    protected sendAutomationRequest<T = unknown>(action: string, params?: Record<string, unknown>, options?: {
         timeoutMs?: number;
         waitForEvent?: boolean;
         waitForEventTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<T>;
 }
 //# sourceMappingURL=base-tool.d.ts.map

package/dist/tools/base-tool.js

@@ -33,7 +33,7 @@ export class BaseTool {
             }
             throw new Error(errorMessage);
         }
-        return response.result ?? response;
+        return (response.result ?? response);
     }
     async sendAutomationRequest(action, params = {}, options) {
         const automation = this.getAutomationBridge();

package/dist/tools/blueprint.d.ts

@@ -1,5 +1,5 @@
 import { BaseTool } from './base-tool.js';
-import { IBlueprintTools } from '../types/tool-interfaces.js';
+import { IBlueprintTools, StandardActionResponse } from '../types/tool-interfaces.js';
 export declare class BlueprintTools extends BaseTool implements IBlueprintTools {
     private log;
     private pluginBlueprintActionsAvailable;
@@ -15,7 +15,7 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     modifyConstructionScript(params: {
         blueprintPath: string;
         operations: any[];
@@ -24,7 +24,7 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addComponent(params: {
         blueprintName: string;
         componentType: string;
@@ -37,40 +37,24 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
-    waitForBlueprint(blueprintRef: string | string[], timeoutMs?: number): Promise<any>;
+    }): Promise<StandardActionResponse>;
+    waitForBlueprint(blueprintRef: string | string[], timeoutMs?: number): Promise<StandardActionResponse>;
     getBlueprint(params: {
         blueprintName: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getBlueprintInfo(params: {
         blueprintPath: string;
         timeoutMs?: number;
-    }): Promise<{
-        success: boolean;
-        message: any;
-        blueprintPath: any;
-        blueprint: any;
-        result: any;
-        requestId: any;
-        error?: undefined;
-    } | {
-        readonly success: false;
-        readonly error: any;
-        readonly message: any;
-        blueprintPath?: undefined;
-        blueprint?: undefined;
-        result?: undefined;
-        requestId?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     probeSubobjectDataHandle(opts?: {
         componentClass?: string;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setBlueprintDefault(params: {
         blueprintName: string;
         propertyName: string;
         value: unknown;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addVariable(params: {
         blueprintName: string;
         variableName: string;
@@ -83,14 +67,14 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     removeVariable(params: {
         blueprintName: string;
         variableName: string;
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     renameVariable(params: {
         blueprintName: string;
         oldName: string;
@@ -98,7 +82,7 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addEvent(params: {
         blueprintName: string;
         eventType: string;
@@ -110,7 +94,7 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     removeEvent(params: {
         blueprintName: string;
         eventName: string;
@@ -118,7 +102,7 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addFunction(params: {
         blueprintName: string;
         functionName: string;
@@ -135,28 +119,28 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setVariableMetadata(params: {
         blueprintName: string;
         variableName: string;
         metadata: Record<string, unknown>;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addConstructionScript(params: {
         blueprintName: string;
         scriptName: string;
         timeoutMs?: number;
         waitForCompletion?: boolean;
         waitForCompletionTimeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     compileBlueprint(params: {
         blueprintName: string;
         saveAfterCompile?: boolean;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getBlueprintSCS(params: {
         blueprintPath: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     addSCSComponent(params: {
         blueprintPath: string;
         componentClass: string;
@@ -165,18 +149,18 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         meshPath?: string;
         materialPath?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     removeSCSComponent(params: {
         blueprintPath: string;
         componentName: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     reparentSCSComponent(params: {
         blueprintPath: string;
         componentName: string;
         newParent: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setSCSComponentTransform(params: {
         blueprintPath: string;
         componentName: string;
@@ -184,31 +168,19 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         rotation?: [number, number, number];
         scale?: [number, number, number];
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setSCSComponentProperty(params: {
         blueprintPath: string;
         componentName: string;
         propertyName: string;
         propertyValue: any;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getNodes(params: {
         blueprintPath: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<{
-        success: boolean;
-        nodes: any;
-        graphName: any;
-        error?: undefined;
-        message?: undefined;
-    } | {
-        readonly success: false;
-        readonly error: any;
-        readonly message: any;
-        nodes?: undefined;
-        graphName?: undefined;
-    }>;
+    }): Promise<StandardActionResponse>;
     addNode(params: {
         blueprintName: string;
         nodeType: string;
@@ -221,20 +193,20 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         posX?: number;
         posY?: number;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     deleteNode(params: {
         blueprintPath: string;
         nodeId: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     createRerouteNode(params: {
         blueprintPath: string;
         graphName?: string;
         x: number;
         y: number;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     setNodeProperty(params: {
         blueprintPath: string;
         nodeId: string;
@@ -242,25 +214,25 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         value: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getNodeDetails(params: {
         blueprintPath: string;
         nodeId: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getGraphDetails(params: {
         blueprintPath: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     getPinDetails(params: {
         blueprintPath: string;
         nodeId: string;
         pinName?: string;
         graphName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
     connectPins(params: {
         blueprintName: string;
         sourceNodeGuid: string;
@@ -268,6 +240,6 @@ export declare class BlueprintTools extends BaseTool implements IBlueprintTools
         sourcePinName?: string;
         targetPinName?: string;
         timeoutMs?: number;
-    }): Promise<any>;
+    }): Promise<StandardActionResponse>;
 }
 //# sourceMappingURL=blueprint.d.ts.map