unreal-engine-mcp-server 0.4.0 → 0.4.3

package/dist/tools/performance.js

@@ -1,21 +1,9 @@
+import { coerceBoolean, coerceNumber, interpretStandardResult } from '../utils/result-helpers.js';
 export class PerformanceTools {
     bridge;
     constructor(bridge) {
         this.bridge = bridge;
     }
-    // Execute console command
-    async _executeCommand(command) {
-        return this.bridge.httpCall('/remote/object/call', 'PUT', {
-            objectPath: '/Script/Engine.Default__KismetSystemLibrary',
-            functionName: 'ExecuteConsoleCommand',
-            parameters: {
-                WorldContextObject: null,
-                Command: command,
-                SpecificPlayer: null
-            },
-            generateTransaction: false
-        });
-    }
     // Start profiling
     async startProfiling(params) {
         const commands = [];
@@ -44,9 +32,7 @@ export class PerformanceTools {
         if (params.duration) {
             commands.push(`stat stopfile ${params.duration}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: `${params.type} profiling started` };
     }
     // Stop profiling
@@ -55,9 +41,7 @@ export class PerformanceTools {
             'stat stopfile',
             'stat none'
         ];
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Profiling stopped' };
     }
     // Show FPS
@@ -112,10 +96,17 @@ export class PerformanceTools {
             Foliage: 'Foliage',
             Shading: 'Shading',
         };
+        const requestedLevel = Number(params.level);
+        if (!Number.isInteger(requestedLevel) || requestedLevel < 0 || requestedLevel > 4) {
+            return {
+                success: false,
+                error: 'Invalid scalability level. Expected integer between 0 and 4.'
+            };
+        }
         const base = categoryBaseMap[params.category] || params.category;
         // Use direct console command to set with highest priority (SetByConsole)
         // This avoids conflicts with the scalability system
-        const setCommand = `sg.${base}Quality ${params.level}`;
+        const setCommand = `sg.${base}Quality ${requestedLevel}`;
         // Apply the console command directly
         await this.bridge.executeConsoleCommand(setCommand);
         // Skip GameUserSettings entirely to avoid any scalability triggers
@@ -123,7 +114,7 @@ export class PerformanceTools {
         /* eslint-disable no-useless-escape */
         const py = `
 import unreal, json
-result = {'success': True, 'category': '${base}', 'requested': ${params.level}, 'actual': ${params.level}, 'method': 'ConsoleOnly'}
+result = {'success': True, 'category': '${base}', 'requested': ${requestedLevel}, 'actual': ${requestedLevel}, 'method': 'ConsoleOnly'}
 
 # Simply verify the console variable was set correctly
 try:
@@ -167,32 +158,20 @@ print('RESULT:' + json.dumps(result))
         // Always try to apply through Python for consistency
         try {
             const pyResp = await this.bridge.executePython(py);
-            let out = '';
-            if (pyResp?.LogOutput && Array.isArray(pyResp.LogOutput)) {
-                out = pyResp.LogOutput.map((l) => l.Output || '').join('');
-            }
-            else if (typeof pyResp === 'string') {
-                out = pyResp;
-            }
-            else {
-                out = JSON.stringify(pyResp);
-            }
-            const m = out.match(/RESULT:({.*})/);
-            if (m) {
-                try {
-                    const parsed = JSON.parse(m[1]);
-                    const verified = parsed.success && (parsed.actual === params.level);
-                    return {
-                        success: true,
-                        message: `${params.category} quality set to level ${params.level}`,
-                        verified,
-                        readback: parsed.actual,
-                        method: parsed.method || 'Unknown'
-                    };
-                }
-                catch {
-                    // Fall through to simple success
-                }
+            const interpreted = interpretStandardResult(pyResp, {
+                successMessage: `${params.category} quality set to level ${requestedLevel}`,
+                failureMessage: `Failed to set ${params.category} quality`
+            });
+            if (interpreted.success) {
+                const actual = coerceNumber(interpreted.payload.actual) ?? requestedLevel;
+                const verified = coerceBoolean(interpreted.payload.success, true) === true && actual === requestedLevel;
+                return {
+                    success: true,
+                    message: interpreted.message,
+                    verified,
+                    readback: actual,
+                    method: interpreted.payload.method || 'ConsoleOnly'
+                };
             }
         }
         catch {
@@ -201,7 +180,7 @@ print('RESULT:' + json.dumps(result))
         // If Python fails, the console command was still applied
         return {
             success: true,
-            message: `${params.category} quality set to level ${params.level}`,
+            message: `${params.category} quality set to level ${requestedLevel}`,
             method: 'CVarOnly'
         };
     }
@@ -257,9 +236,7 @@ print('RESULT:' + json.dumps(result))
         if (params.outputPath) {
             commands.push(`obj savepackage ${params.outputPath}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Memory report generated' };
     }
     // Texture streaming
@@ -272,9 +249,7 @@ print('RESULT:' + json.dumps(result))
         if (params.boostPlayerLocation !== undefined) {
             commands.push(`r.Streaming.UseFixedPoolSize ${params.boostPlayerLocation ? 1 : 0}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Texture streaming configured' };
     }
     // LOD settings
@@ -292,9 +267,7 @@ print('RESULT:' + json.dumps(result))
             commands.push(`r.StaticMeshLODDistanceScale ${params.distanceScale}`);
             commands.push(`r.SkeletalMeshLODDistanceScale ${params.distanceScale}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'LOD settings configured' };
     }
     // Apply a baseline performance profile (explicit CVar enforcement)
@@ -314,9 +287,7 @@ print('RESULT:' + json.dumps(result))
             `r.VSync ${p.vsync ? 1 : 0}`,
             `t.MaxFPS ${p.maxFPS}`,
         ];
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Baseline performance settings applied', params: p };
     }
     // Draw call optimization
@@ -329,9 +300,7 @@ print('RESULT:' + json.dumps(result))
         if (params.mergeActors) {
             commands.push('MergeActors');
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Draw call optimization configured' };
     }
     // Occlusion culling
@@ -343,9 +312,7 @@ print('RESULT:' + json.dumps(result))
         if (params.freezeRendering !== undefined) {
             commands.push(`FreezeRendering ${params.freezeRendering ? 1 : 0}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Occlusion culling configured' };
     }
     // Shader compilation
@@ -360,9 +327,7 @@ print('RESULT:' + json.dumps(result))
         if (params.reducePermutations) {
             commands.push('RecompileShaders changed');
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Shader optimization configured' };
     }
     // Nanite settings
@@ -375,9 +340,7 @@ print('RESULT:' + json.dumps(result))
         if (params.streamingPoolSize !== undefined) {
             commands.push(`r.Nanite.StreamingPoolSize ${params.streamingPoolSize}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'Nanite configured' };
     }
     // World Partition streaming
@@ -390,22 +353,18 @@ print('RESULT:' + json.dumps(result))
         if (params.cellSize !== undefined) {
             commands.push(`wp.Runtime.CellSize ${params.cellSize}`);
         }
-        for (const cmd of commands) {
-            await this.bridge.executeConsoleCommand(cmd);
-        }
+        await this.bridge.executeConsoleCommands(commands);
         return { success: true, message: 'World Partition configured' };
     }
     // Benchmark
     async runBenchmark(params) {
         const duration = params.duration || 60;
         // Start recording and GPU profiling
-        await this.bridge.executeConsoleCommand('stat startfile');
-        await this.bridge.executeConsoleCommand('profilegpu');
+        await this.bridge.executeConsoleCommands(['stat startfile', 'profilegpu']);
         // Wait for the requested duration
         await new Promise(resolve => setTimeout(resolve, duration * 1000));
         // Stop recording and clear stats
-        await this.bridge.executeConsoleCommand('stat stopfile');
-        await this.bridge.executeConsoleCommand('stat none');
+        await this.bridge.executeConsoleCommands(['stat stopfile', 'stat none']);
         return { success: true, message: `Benchmark completed for ${duration} seconds` };
     }
 }

package/dist/tools/physics.d.ts

@@ -28,20 +28,30 @@ export declare class PhysicsTools {
             };
         }>;
     }): Promise<{
+        success: true;
+        message: string;
+        path: string;
+        existingAsset?: boolean;
+        warnings?: string[];
+        details?: string[];
+    } | {
         success: boolean;
         message: string;
         error: string | undefined;
-        path?: undefined;
+        warnings?: undefined;
+        details?: undefined;
     } | {
-        success: boolean;
+        success: false;
         message: string;
-        path: string;
-        error?: undefined;
+        error: string;
+        warnings: string[] | undefined;
+        details: string[] | undefined;
     } | {
         success: boolean;
         error: string;
         message?: undefined;
-        path?: undefined;
+        warnings?: undefined;
+        details?: undefined;
     }>;
     /**
      * Create Physics Constraint
@@ -133,7 +143,25 @@ export declare class PhysicsTools {
         vector: [number, number, number];
         boneName?: string;
         isLocal?: boolean;
-    }): Promise<any>;
+    }): Promise<{
+        success: boolean;
+        message: string;
+        availableActors: string[] | undefined;
+        details: string[] | undefined;
+        error?: undefined;
+    } | {
+        success: boolean;
+        error: string;
+        availableActors: string[] | undefined;
+        details: string[] | undefined;
+        message?: undefined;
+    } | {
+        success: boolean;
+        error: string;
+        message?: undefined;
+        availableActors?: undefined;
+        details?: undefined;
+    }>;
     /**
      * Configure Cloth Simulation
      */
@@ -181,9 +209,5 @@ export declare class PhysicsTools {
         error: string;
         message?: undefined;
     }>;
-    /**
-     * Helper function to execute console commands
-     */
-    private _executeCommand;
 }
 //# sourceMappingURL=physics.d.ts.map