unreal-engine-mcp-server 0.5.1 → 0.5.2

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

@@ -1,7 +1,18 @@
 import { cleanObject } from '../../utils/safe-json.js';
-import { ITools } from '../../types/tool-interfaces.js';
+import { ITools, StandardActionResponse } from '../../types/tool-interfaces.js';
 import { executeAutomationRequest, requireNonEmptyString } from './common-handlers.js';
 
+/** Extended response with common sequence fields */
+interface SequenceActionResponse extends StandardActionResponse {
+  result?: {
+    sequencePath?: string;
+    results?: Array<{ success?: boolean; error?: string }>;
+    [key: string]: unknown;
+  };
+  bindings?: Array<{ name?: string;[key: string]: unknown }>;
+  message?: string;
+}
+
 const managedSequences = new Set<string>();
 const deletedSequences = new Set<string>();
 
@@ -22,37 +33,48 @@ function markSequenceDeleted(path: unknown) {
   const norm = normalizeSequencePath(path);
   if (!norm) return;
   managedSequences.delete(norm);
-  deletedSequences.add(norm);
+  deletedSequences.delete(norm);
+}
+
+/** Helper to safely get string from error/message */
+function getErrorString(res: SequenceActionResponse | null | undefined): string {
+  if (!res) return '';
+  return typeof res.error === 'string' ? res.error : '';
+}
+
+function getMessageString(res: SequenceActionResponse | null | undefined): string {
+  if (!res) return '';
+  return typeof res.message === 'string' ? res.message : '';
 }
 
 
 
-export async function handleSequenceTools(action: string, args: any, tools: ITools) {
+export async function handleSequenceTools(action: string, args: Record<string, unknown>, tools: ITools) {
   const seqAction = String(action || '').trim();
   switch (seqAction) {
     case 'create': {
       const name = requireNonEmptyString(args.name, 'name', 'Missing required parameter: name');
-      const res = await tools.sequenceTools.create({ name, path: args.path });
+      const res = await tools.sequenceTools.create({ name, path: args.path as string | undefined }) as SequenceActionResponse;
 
       let sequencePath: string | undefined;
-      if (res && (res as any).result && typeof (res as any).result.sequencePath === 'string') {
-        sequencePath = (res as any).result.sequencePath;
+      if (res && res.result && typeof res.result.sequencePath === 'string') {
+        sequencePath = res.result.sequencePath;
       } else if (typeof args.path === 'string' && args.path.trim().length > 0) {
         const basePath = args.path.trim().replace(/\/$/, '');
         sequencePath = `${basePath}/${name}`;
       }
-      if (sequencePath && res && (res as any).success !== false) {
+      if (sequencePath && res && res.success !== false) {
         markSequenceCreated(sequencePath);
       }
 
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
-      const msgLower = String((res && (res as any).message) || '').toLowerCase();
-      if (res && (res as any).success === false && (errorCode === 'FACTORY_NOT_AVAILABLE' || msgLower.includes('ulevelsequencefactorynew not available'))) {
+      const errorCode = getErrorString(res).toUpperCase();
+      const msgLower = getMessageString(res).toLowerCase();
+      if (res && res.success === false && (errorCode === 'FACTORY_NOT_AVAILABLE' || msgLower.includes('ulevelsequencefactorynew not available'))) {
         const path = sequencePath || (typeof args.path === 'string' ? args.path : undefined);
         return cleanObject({
           success: false,
           error: 'FACTORY_NOT_AVAILABLE',
-          message: (res as any).message || 'Sequence creation failed: factory not available',
+          message: res.message || 'Sequence creation failed: factory not available',
           action: 'create',
           name,
           path,
@@ -69,7 +91,7 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       return cleanObject(res);
     }
     case 'add_camera': {
-      const res = await tools.sequenceTools.addCamera({ spawnable: args.spawnable, path: args.path });
+      const res = await tools.sequenceTools.addCamera({ spawnable: args.spawnable as boolean | undefined, path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'add_actor': {
@@ -82,18 +104,18 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
         subAction: 'add_actor'
       };
 
-      const res = await executeAutomationRequest(tools, 'manage_sequence', payload);
+      const res = await executeAutomationRequest(tools, 'manage_sequence', payload) as SequenceActionResponse;
 
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
-      const msgLower = String((res && (res as any).message) || '').toLowerCase();
+      const errorCode = getErrorString(res).toUpperCase();
+      const msgLower = getMessageString(res).toLowerCase();
 
-      if (res && (res as any).success === false && path) {
+      if (res && res.success === false && path) {
         const isInvalidSequence = errorCode === 'INVALID_SEQUENCE' || msgLower.includes('sequence_add_actor requires a sequence path') || msgLower.includes('sequence not found');
         if (isInvalidSequence) {
           return cleanObject({
             success: false,
             error: 'NOT_FOUND',
-            message: (res as any).message || 'Sequence not found',
+            message: res.message || 'Sequence not found',
             action: 'add_actor',
             path,
             actorName
@@ -101,8 +123,8 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
         }
       }
 
-      const results = res && (res as any).result && Array.isArray((res as any).result.results)
-        ? (res as any).result.results as any[]
+      const results = res && res.result && Array.isArray(res.result.results)
+        ? res.result.results
         : undefined;
       if (results && results.length) {
         const failed = results.find((item) => item && item.success === false && typeof item.error === 'string');
@@ -124,24 +146,24 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       return cleanObject(res);
     }
     case 'add_actors': {
-      const actorNames: string[] = Array.isArray(args.actorNames) ? args.actorNames : [];
-      const res = await tools.sequenceTools.addActors({ actorNames, path: args.path });
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
-      const msgLower = String((res && (res as any).message) || '').toLowerCase();
-      if (actorNames.length === 0 && res && (res as any).success === false && errorCode === 'INVALID_ARGUMENT') {
+      const actorNames: string[] = Array.isArray(args.actorNames) ? args.actorNames as string[] : [];
+      const res = await tools.sequenceTools.addActors({ actorNames, path: args.path as string | undefined }) as SequenceActionResponse;
+      const errorCode = getErrorString(res).toUpperCase();
+      const msgLower = getMessageString(res).toLowerCase();
+      if (actorNames.length === 0 && res && res.success === false && errorCode === 'INVALID_ARGUMENT') {
         return cleanObject({
           success: false,
           error: 'INVALID_ARGUMENT',
-          message: (res as any).message || 'Invalid argument: actorNames required',
+          message: res.message || 'Invalid argument: actorNames required',
           action: 'add_actors',
           actorNames
         });
       }
-      if (res && (res as any).success === false && msgLower.includes('actor not found')) {
+      if (res && res.success === false && msgLower.includes('actor not found')) {
         return cleanObject({
           success: false,
           error: 'NOT_FOUND',
-          message: (res as any).message || 'Actor not found',
+          message: res.message || 'Actor not found',
           action: 'add_actors',
           actorNames
         });
@@ -149,8 +171,8 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       return cleanObject(res);
     }
     case 'remove_actors': {
-      const actorNames: string[] = Array.isArray(args.actorNames) ? args.actorNames : [];
-      const res = await tools.sequenceTools.removeActors({ actorNames, path: args.path });
+      const actorNames: string[] = Array.isArray(args.actorNames) ? args.actorNames as string[] : [];
+      const res = await tools.sequenceTools.removeActors({ actorNames, path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'get_bindings': {
@@ -164,7 +186,7 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       const property = typeof args.property === 'string' ? args.property : undefined;
       const frame = typeof args.frame === 'number' ? args.frame : Number(args.frame);
 
-      const payload = {
+      const payload: Record<string, unknown> = {
         ...args,
         path: path || args.path,
         actorName,
@@ -185,16 +207,16 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
         payload.value = { scale: args.value };
       }
 
-      const res = await executeAutomationRequest(tools, 'manage_sequence', payload);
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
-      const msgLower = String((res && (res as any).message) || '').toLowerCase();
+      const res = await executeAutomationRequest(tools, 'manage_sequence', payload) as SequenceActionResponse;
+      const errorCode = getErrorString(res).toUpperCase();
+      const msgLower = getMessageString(res).toLowerCase();
 
       // Keep explicit INVALID_ARGUMENT for missing frame as a real error
       if (errorCode === 'INVALID_ARGUMENT' || msgLower.includes('frame number is required')) {
         return cleanObject(res);
       }
 
-      if (res && (res as any).success === false) {
+      if (res && res.success === false) {
         const isBindingIssue = errorCode === 'BINDING_NOT_FOUND' || msgLower.includes('binding not found');
         const isUnsupported = errorCode === 'UNSUPPORTED_PROPERTY' || msgLower.includes('unsupported property') || msgLower.includes('invalid_sequence_type');
         const isInvalidSeq = errorCode === 'INVALID_SEQUENCE' || msgLower.includes('sequence not found') || msgLower.includes('requires a sequence path');
@@ -203,7 +225,7 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
           return cleanObject({
             success: false,
             error: 'NOT_FOUND',
-            message: (res as any).message || 'Sequence not found',
+            message: res.message || 'Sequence not found',
             action: 'add_keyframe',
             path,
             actorName,
@@ -222,28 +244,28 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
     }
     case 'add_spawnable_from_class': {
       const className = requireNonEmptyString(args.className, 'className', 'Missing required parameter: className');
-      const res = await tools.sequenceTools.addSpawnableFromClass({ className, path: args.path });
+      const res = await tools.sequenceTools.addSpawnableFromClass({ className, path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'play': {
-      const res = await tools.sequenceTools.play({ path: args.path, startTime: args.startTime, loopMode: args.loopMode });
+      const res = await tools.sequenceTools.play({ path: args.path as string | undefined, startTime: args.startTime as number | undefined, loopMode: args.loopMode as 'once' | 'loop' | 'pingpong' | undefined });
       return cleanObject(res);
     }
     case 'pause': {
-      const res = await tools.sequenceTools.pause({ path: args.path });
+      const res = await tools.sequenceTools.pause({ path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'stop': {
-      const res = await tools.sequenceTools.stop({ path: args.path });
+      const res = await tools.sequenceTools.stop({ path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'set_properties': {
       const res = await tools.sequenceTools.setSequenceProperties({
-        path: args.path,
-        frameRate: args.frameRate,
-        lengthInFrames: args.lengthInFrames,
-        playbackStart: args.playbackStart,
-        playbackEnd: args.playbackEnd
+        path: args.path as string | undefined,
+        frameRate: args.frameRate as number | undefined,
+        lengthInFrames: args.lengthInFrames as number | undefined,
+        playbackStart: args.playbackStart as number | undefined,
+        playbackEnd: args.playbackEnd as number | undefined
       });
       return cleanObject(res);
     }
@@ -258,31 +280,32 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
         throw new Error('Invalid speed: must be a positive number');
       }
       // Try setting speed
-      let res = await tools.sequenceTools.setPlaybackSpeed({ speed, path: args.path });
+      let res = await tools.sequenceTools.setPlaybackSpeed({ speed, path: args.path as string | undefined }) as SequenceActionResponse;
 
       // Fix: Auto-open if editor not open
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
+      const errorCode = getErrorString(res).toUpperCase();
       if ((!res || res.success === false) && errorCode === 'EDITOR_NOT_OPEN' && args.path) {
         // Attempt to open the sequence
-        await tools.sequenceTools.open({ path: args.path });
+        await tools.sequenceTools.open({ path: args.path as string });
 
         // Wait a short moment for editor to initialize on game thread
         await new Promise(resolve => setTimeout(resolve, 1000));
 
         // Retry
-        res = await tools.sequenceTools.setPlaybackSpeed({ speed, path: args.path });
+        res = await tools.sequenceTools.setPlaybackSpeed({ speed, path: args.path as string | undefined }) as SequenceActionResponse;
       }
 
       return cleanObject(res);
     }
     case 'list': {
-      const res = await tools.sequenceTools.list({ path: args.path });
+      const res = await tools.sequenceTools.list({ path: args.path as string | undefined });
       return cleanObject(res);
     }
     case 'duplicate': {
       const path = requireNonEmptyString(args.path, 'path', 'Missing required parameter: path');
       const destDir = requireNonEmptyString(args.destinationPath, 'destinationPath', 'Missing required parameter: destinationPath');
-      const newName = requireNonEmptyString(args.newName || path.split('/').pop(), 'newName', 'Missing required parameter: newName');
+      const defaultNewName = path.split('/').pop() || '';
+      const newName = requireNonEmptyString(args.newName || defaultNewName, 'newName', 'Missing required parameter: newName');
       const baseDir = destDir.replace(/\/$/, '');
       const destPath = `${baseDir}/${newName}`;
       const res = await tools.sequenceTools.duplicate({ path, destinationPath: destPath });
@@ -291,15 +314,15 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
     case 'rename': {
       const path = requireNonEmptyString(args.path, 'path', 'Missing required parameter: path');
       const newName = requireNonEmptyString(args.newName, 'newName', 'Missing required parameter: newName');
-      const res = await tools.sequenceTools.rename({ path, newName });
-      const errorCode = String((res && (res as any).error) || '').toUpperCase();
-      const msgLower = String((res && (res as any).message) || '').toLowerCase();
-      if (res && (res as any).success === false && (errorCode === 'OPERATION_FAILED' || msgLower.includes('failed to rename sequence'))) {
+      const res = await tools.sequenceTools.rename({ path, newName }) as SequenceActionResponse;
+      const errorCode = getErrorString(res).toUpperCase();
+      const msgLower = getMessageString(res).toLowerCase();
+      if (res && res.success === false && (errorCode === 'OPERATION_FAILED' || msgLower.includes('failed to rename sequence'))) {
         // Return actual failure, not best-effort success - rename is a destructive operation
         return cleanObject({
           success: false,
           error: 'OPERATION_FAILED',
-          message: (res as any).message || 'Failed to rename sequence',
+          message: res.message || 'Failed to rename sequence',
           action: 'rename',
           path,
           newName
@@ -309,20 +332,20 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
     }
     case 'delete': {
       const path = requireNonEmptyString(args.path, 'path', 'Missing required parameter: path');
-      const res = await tools.sequenceTools.deleteSequence({ path });
+      const res = await tools.sequenceTools.deleteSequence({ path }) as SequenceActionResponse;
 
-      if (res && (res as any).success !== false) {
+      if (res && res.success !== false) {
         markSequenceDeleted(path);
       }
       return cleanObject(res);
     }
     case 'get_metadata': {
-      const res = await tools.sequenceTools.getMetadata({ path: args.path });
+      const res = await tools.sequenceTools.getMetadata({ path: args.path as string });
       return cleanObject(res);
     }
     case 'set_metadata': {
       const path = requireNonEmptyString(args.path, 'path', 'Missing required parameter: path');
-      const metadata = (args.metadata && typeof args.metadata === 'object') ? args.metadata : {};
+      const metadata = (args.metadata && typeof args.metadata === 'object') ? args.metadata as Record<string, unknown> : {};
       const res = await executeAutomationRequest(tools, 'set_metadata', { assetPath: path, metadata });
       return cleanObject(res);
     }
@@ -335,10 +358,10 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
 
       // Fix: Check if actor is bound before adding track
       if (actorName) {
-        const bindingsRes = await tools.sequenceTools.getBindings({ path });
+        const bindingsRes = await tools.sequenceTools.getBindings({ path }) as SequenceActionResponse;
         if (bindingsRes && bindingsRes.success) {
-          const bindings = (bindingsRes.bindings as any[]) || [];
-          const isBound = bindings.some((b: any) => b.name === actorName);
+          const bindings = bindingsRes.bindings || [];
+          const isBound = bindings.some((b) => b.name === actorName);
           if (!isBound) {
             return cleanObject({
               success: false,
@@ -399,7 +422,7 @@ export async function handleSequenceTools(action: string, args: any, tools: IToo
       if (!Number.isFinite(end)) throw new Error('Invalid end: must be a number');
 
       const res = await tools.sequenceTools.setWorkRange({
-        path: args.path,
+        path: args.path as string | undefined,
         start,
         end
       });

package/src/tools/introspection.ts

@@ -35,7 +35,7 @@ export interface PropertyInfo {
   type: string;
   value?: any;
   flags?: string[];
-  metadata?: Record<string, any>;
+  metadata?: Record<string, unknown>;
   category?: string;
   tooltip?: string;
   description?: string;
@@ -141,7 +141,7 @@ export class IntrospectionTools {
     return value;
   }
 
-  private isPlainObject(value: any): value is Record<string, any> {
+  private isPlainObject(value: any): value is Record<string, unknown> {
     return !!value && typeof value === 'object' && !Array.isArray(value);
   }
 
@@ -200,7 +200,7 @@ export class IntrospectionTools {
     const rawValue = entry.value ?? entry.currentValue ?? entry.defaultValue ?? entry.data ?? entry;
     const value = this.convertPropertyValue(rawValue, type);
     const flags: string[] | undefined = entry.flags ?? entry.attributes;
-    const metadata: Record<string, any> | undefined = entry.metadata ?? entry.annotations;
+    const metadata: Record<string, unknown> | undefined = entry.metadata ?? entry.annotations;
     const filtered = this.shouldFilterProperty(name, value, flags, detailed);
     const dictionaryEntry = lookupPropertyMetadata(name);
     const propertyInfo: PropertyInfo = {
@@ -220,12 +220,12 @@ export class IntrospectionTools {
     return propertyInfo;
   }
 
-  private flattenPropertyMap(source: Record<string, any>, prefix = '', detailed = false): PropertyInfo[] {
+  private flattenPropertyMap(source: Record<string, unknown>, prefix = '', detailed = false): PropertyInfo[] {
     const properties: PropertyInfo[] = [];
     for (const [rawKey, rawValue] of Object.entries(source)) {
       const name = prefix ? `${prefix}.${rawKey}` : rawKey;
       if (this.isLikelyPropertyDescriptor(rawValue)) {
-        const normalized = this.normalizePropertyEntry({ ...rawValue, name }, detailed);
+        const normalized = this.normalizePropertyEntry({ ...(rawValue as Record<string, unknown>), name }, detailed);
         if (normalized) properties.push(normalized);
         continue;
       }
@@ -441,8 +441,8 @@ export class IntrospectionTools {
         }
 
         return res;
-      } catch (err: any) {
-        const errorMsg = err?.message || String(err);
+      } catch (err: unknown) {
+        const errorMsg = (err instanceof Error ? err.message : undefined) || String(err);
         if (errorMsg.includes('404')) {
           return { success: false, error: `Property '${params.propertyName}' not found on object '${params.objectPath}'` };
         }

package/src/tools/level.ts

@@ -50,12 +50,12 @@ export class LevelTools extends BaseTool implements ILevelTools {
     // Security validation
     try {
       formatted = sanitizePath(formatted);
-    } catch (e: any) {
+    } catch (e: unknown) {
       // If sanitizePath fails, we should probably propagate that error, 
       // but normalizeLevelPath signature expects to return an object.
       // For now, let's log and rethrow or fallback? 
       // Throwing is safer as it prevents operation on invalid path.
-      throw new Error(`Security validation failed for level path: ${e.message}`);
+      throw new Error(`Security validation failed for level path: ${e instanceof Error ? e.message : String(e)}`);
     }
 
     formatted = formatted.replace(/\.umap$/i, '');
@@ -322,8 +322,8 @@ export class LevelTools extends BaseTool implements ILevelTools {
         exportPath: params.exportPath,
         details: res
       } as StandardActionResponse;
-    } catch (e: any) {
-      return { success: false, error: `Export failed: ${e.message}` };
+    } catch (e: unknown) {
+      return { success: false, error: `Export failed: ${e instanceof Error ? e.message : String(e)}` };
     }
   }
 
@@ -356,8 +356,8 @@ export class LevelTools extends BaseTool implements ILevelTools {
         streaming: Boolean(params.streaming),
         details: res
       } as StandardActionResponse;
-    } catch (e: any) {
-      return { success: false, error: `Import failed: ${e.message}` };
+    } catch (e: unknown) {
+      return { success: false, error: `Import failed: ${e instanceof Error ? e.message : String(e)}` };
     }
   }
 

package/src/tools/lighting.ts

@@ -40,7 +40,7 @@ export class LightingTools {
       name: string;
       location?: [number, number, number];
       rotation?: [number, number, number] | { pitch: number, yaw: number, roll: number };
-      properties?: Record<string, any>;
+      properties?: Record<string, unknown>;
     }
   ) {
     if (!this.automationBridge) {
@@ -48,7 +48,7 @@ export class LightingTools {
     }
 
     try {
-      const payload: Record<string, any> = {
+      const payload: Record<string, unknown> = {
         lightClass,
         name: params.name,
       };
@@ -98,7 +98,7 @@ export class LightingTools {
     castShadows?: boolean;
     temperature?: number;
     useAsAtmosphereSunLight?: boolean;
-    properties?: Record<string, any>;
+    properties?: Record<string, unknown>;
   }) {
     const name = this.normalizeName(params.name);
     if (!this.automationBridge) {
@@ -149,7 +149,7 @@ export class LightingTools {
     const rot = params.rotation || [0, 0, 0];
 
     // Build properties for the light
-    const properties: Record<string, any> = params.properties || {};
+    const properties: Record<string, unknown> = params.properties || {};
     if (params.intensity !== undefined) {
       properties.intensity = params.intensity;
     }
@@ -175,9 +175,9 @@ export class LightingTools {
       });
 
       return { success: true, message: `Directional light '${name}' spawned` };
-    } catch (e: any) {
+    } catch (e: unknown) {
       // Don't mask errors as "not implemented" - report the actual error from the bridge
-      return { success: false, error: `Failed to create directional light: ${e?.message ?? e}` } as any;
+      return { success: false, error: `Failed to create directional light: ${(e instanceof Error ? e.message : String(e)) ?? e}` };
     }
   }
 
@@ -247,7 +247,7 @@ export class LightingTools {
     }
 
     // Build properties for the light
-    const properties: Record<string, any> = {};
+    const properties: Record<string, unknown> = {};
     if (params.intensity !== undefined) {
       properties.intensity = params.intensity;
     }
@@ -273,9 +273,9 @@ export class LightingTools {
       });
 
       return { success: true, message: `Point light '${name}' spawned at ${location.join(', ')}` };
-    } catch (e: any) {
+    } catch (e: unknown) {
       // Don't mask errors as "not implemented" - report the actual error from the bridge
-      return { success: false, error: `Failed to create point light: ${e?.message ?? e}` } as any;
+      return { success: false, error: `Failed to create point light: ${(e instanceof Error ? e.message : String(e)) ?? e}` };
     }
   }
 
@@ -369,7 +369,7 @@ export class LightingTools {
       }
     }
     // Build properties for the light
-    const properties: Record<string, any> = {};
+    const properties: Record<string, unknown> = {};
     if (params.intensity !== undefined) {
       properties.intensity = params.intensity;
     }
@@ -398,9 +398,9 @@ export class LightingTools {
       });
 
       return { success: true, message: `Spot light '${name}' spawned at ${params.location.join(', ')}` };
-    } catch (e: any) {
+    } catch (e: unknown) {
       // Don't mask errors as "not implemented" - report the actual error from the bridge
-      return { success: false, error: `Failed to create spot light: ${e?.message ?? e}` } as any;
+      return { success: false, error: `Failed to create spot light: ${(e instanceof Error ? e.message : String(e)) ?? e}` };
     }
   }
 
@@ -485,7 +485,7 @@ export class LightingTools {
       }
     }
     // Build properties for the light
-    const properties: Record<string, any> = {};
+    const properties: Record<string, unknown> = {};
     if (params.intensity !== undefined) {
       properties.intensity = params.intensity;
     }
@@ -508,9 +508,9 @@ export class LightingTools {
       });
 
       return { success: true, message: `Rect light '${name}' spawned at ${params.location.join(', ')}` };
-    } catch (e: any) {
+    } catch (e: unknown) {
       // Don't mask errors as "not implemented" - report the actual error from the bridge
-      return { success: false, error: `Failed to create rect light: ${e?.message ?? e}` } as any;
+      return { success: false, error: `Failed to create rect light: ${(e instanceof Error ? e.message : String(e)) ?? e}` };
     }
   }
 
@@ -581,13 +581,13 @@ export class LightingTools {
     }
 
     try {
-      const properties: Record<string, any> = {};
+      const properties: Record<string, unknown> = {};
       if (params.intensity !== undefined) properties.Intensity = params.intensity;
       if (params.castShadows !== undefined) properties.CastShadows = params.castShadows;
       if (params.realTimeCapture !== undefined) properties.RealTimeCapture = params.realTimeCapture;
       if (params.color) properties.LightColor = { r: params.color[0], g: params.color[1], b: params.color[2], a: 1.0 };
 
-      const payload: Record<string, any> = {
+      const payload: Record<string, unknown> = {
         name,
         sourceType: params.sourceType || 'CapturedScene',
         location: params.location,
@@ -811,12 +811,12 @@ export class LightingTools {
         success: true,
         message: response.message || 'Lighting build started',
         ...(response.result || {})
-      } as any;
+      };
     } catch (error) {
       return {
         success: false,
         error: `Failed to build lighting: ${error instanceof Error ? error.message : String(error)}`
-      } as any;
+      };
     }
   }
 

package/src/tools/materials.ts

@@ -120,7 +120,7 @@ export class MaterialTools {
     }
   }
 
-  async createMaterialInstance(name: string, path: string, parentMaterial: string, parameters?: Record<string, any>) {
+  async createMaterialInstance(name: string, path: string, parentMaterial: string, parameters?: Record<string, unknown>) {
     try {
       if (!name || name.trim() === '') {
         return { success: false, error: 'Material instance name cannot be empty' };

package/src/tools/sequence.ts

@@ -51,7 +51,7 @@ export class SequenceTools extends BaseTool implements ISequenceTools {
       const result = response.result ?? response;
 
       return { success, message: response.message ?? undefined, error: response.success === false ? (response.error ?? response.message) : undefined, result, requestId: response.requestId } as any;
-    } catch (err: any) {
+    } catch (err: unknown) {
       return { success: false, error: String(err), message: String(err) } as const;
     }
   }

package/src/tools/ui.ts

@@ -59,7 +59,7 @@ export class UITools {
         package_path: path,
         factory_class: 'WidgetBlueprintFactory',
         asset_class: 'unreal.WidgetBlueprint'
-      } as Record<string, any>;
+      } as Record<string, unknown>;
 
       const resp = await this.bridge.executeEditorFunction('CREATE_ASSET', payload as any);
       const result = resp && typeof resp === 'object' ? (resp.result ?? resp) : resp;

package/src/types/tool-types.ts

@@ -9,6 +9,10 @@ export interface BaseToolResponse {
   message?: string;
   error?: string;
   warning?: string;
+  /** Whether this error is retriable (e.g., connection failures) */
+  retriable?: boolean;
+  /** Scope/context for the error (e.g., 'tool-call/manage_asset') */
+  scope?: string;
 }
 
 // Asset Management Types