@akiojin/unity-mcp-server 2.26.0 → 2.26.1

package/src/handlers/gameobject/GameObjectModifyToolHandler.js

@@ -70,7 +70,7 @@ export class GameObjectModifyToolHandler extends BaseToolHandler {
         required: ['path']
       }
     );
-    
+
     this.unityConnection = unityConnection;
   }
 
@@ -81,23 +81,34 @@ export class GameObjectModifyToolHandler extends BaseToolHandler {
    */
   validate(params) {
     super.validate(params);
-    
+
     // Path is required
     validateGameObjectPath(params.path);
-    
+
     // At least one modification must be specified
-    const modifiableProps = ['name', 'position', 'rotation', 'scale', 'active', 'parentPath', 'tag', 'layer'];
-    const hasModification = modifiableProps.some(prop => params.hasOwnProperty(prop));
-    
+    const modifiableProps = [
+      'name',
+      'position',
+      'rotation',
+      'scale',
+      'active',
+      'parentPath',
+      'tag',
+      'layer'
+    ];
+    const hasModification = modifiableProps.some(prop =>
+      Object.prototype.hasOwnProperty.call(params, prop)
+    );
+
     if (!hasModification) {
       throw new Error('At least one property to modify must be specified');
     }
-    
+
     // Validate vector3 properties
     if (params.position) validateVector3(params.position, 'position');
     if (params.rotation) validateVector3(params.rotation, 'rotation');
     if (params.scale) validateVector3(params.scale, 'scale');
-    
+
     // Validate layer
     if (params.layer !== undefined) {
       validateLayer(Number(params.layer));
@@ -114,15 +125,15 @@ export class GameObjectModifyToolHandler extends BaseToolHandler {
     if (!this.unityConnection.isConnected()) {
       await this.unityConnection.connect();
     }
-    
+
     // Send modify_gameobject command
     const result = await this.unityConnection.sendCommand('modify_gameobject', params);
-    
+
     // Check for errors from Unity
     if (result.error) {
       throw new Error(result.error);
     }
-    
+
     return result;
   }
 }

package/src/handlers/menu/MenuItemExecuteToolHandler.js

@@ -18,7 +18,8 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
           action: {
             type: 'string',
             enum: ['execute', 'get_available_menus'],
-            description: 'Action to perform: execute menu item or get available menus (default: execute)'
+            description:
+              'Action to perform: execute menu item or get available menus (default: execute)'
           },
           alias: {
             type: 'string',
@@ -30,47 +31,48 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
           },
           safetyCheck: {
             type: 'boolean',
-            description: 'Enable safety checks to prevent execution of dangerous menu items (default: true)'
+            description:
+              'Enable safety checks to prevent execution of dangerous menu items (default: true)'
           }
         },
         required: ['menuPath']
       }
     );
-    
+
     this.unityConnection = unityConnection;
-    
+
     // Define blacklisted menu items for safety
     // Includes dialog-opening menus that cause MCP hanging
     this.blacklistedMenus = new Set([
       // Application control
       'File/Quit',
-      
+
       // Dialog-opening file operations (cause MCP hanging)
       'File/Open Scene',
-      'File/New Scene', 
+      'File/New Scene',
       'File/Save Scene As...',
       'File/Build Settings...',
       'File/Build And Run',
-      
+
       // Dialog-opening asset operations (cause MCP hanging)
       'Assets/Import New Asset...',
       'Assets/Import Package/Custom Package...',
       'Assets/Export Package...',
       'Assets/Delete',
-      
+
       // Dialog-opening preferences and settings (cause MCP hanging)
       'Edit/Preferences...',
       'Edit/Project Settings...',
-      
+
       // Dialog-opening window operations (may cause issues)
       'Window/Package Manager',
       'Window/Asset Store',
-      
+
       // Scene view operations that may require focus (potential hanging)
       'GameObject/Align With View',
       'GameObject/Align View to Selected'
     ]);
-    
+
     // Common menu aliases
     this.menuAliases = new Map([
       ['refresh', 'Assets/Refresh'],
@@ -104,12 +106,18 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
 
     // Safety check for blacklisted items with security normalization (BEFORE format validation)
     if (safetyCheck && this.isMenuPathBlacklisted(menuPath)) {
-      throw new Error(`Menu item is blacklisted for safety: ${menuPath}. Use safetyCheck: false to override.`);
+      throw new Error(
+        `Menu item is blacklisted for safety: ${menuPath}. Use safetyCheck: false to override.`
+      );
     }
 
     // Validate menu path format (should contain at least one slash) - after normalization for security
     const normalizedForValidation = this.normalizeMenuPath(menuPath);
-    if (!normalizedForValidation.includes('/') || normalizedForValidation.startsWith('/') || normalizedForValidation.endsWith('/')) {
+    if (
+      !normalizedForValidation.includes('/') ||
+      normalizedForValidation.startsWith('/') ||
+      normalizedForValidation.endsWith('/')
+    ) {
       throw new Error('menuPath must be in format "Category/MenuItem" (e.g., "Assets/Refresh")');
     }
 
@@ -125,13 +133,7 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
    * @returns {Promise<Object>} The result of the menu operation
    */
   async execute(params) {
-    const {
-      menuPath,
-      action = 'execute',
-      alias,
-      parameters,
-      safetyCheck = true
-    } = params;
+    const { menuPath, action = 'execute', alias, parameters, safetyCheck = true } = params;
 
     // Ensure connection to Unity
     if (!this.unityConnection.isConnected()) {
@@ -242,7 +244,7 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
   isMenuPathBlacklisted(menuPath) {
     // Normalize the input path to prevent bypass attacks
     const normalizedPath = this.normalizeMenuPath(menuPath);
-    
+
     // Check against normalized blacklist entries
     for (const blacklistedItem of this.blacklistedMenus) {
       const normalizedBlacklistItem = this.normalizeMenuPath(blacklistedItem);
@@ -250,7 +252,7 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
         return true;
       }
     }
-    
+
     return false;
   }
 
@@ -265,40 +267,76 @@ export class MenuItemExecuteToolHandler extends BaseToolHandler {
     }
 
     // Step 1: Remove zero-width and invisible Unicode characters
-    let normalized = menuPath.replace(/[\u200B-\u200D\uFEFF\u00AD\u034F\u061C\u180E\u2060-\u2069]/g, '');
-    
+    // eslint-disable-next-line no-misleading-character-class
+    let normalized = menuPath.replace(
+      /[\u200B-\u200D\uFEFF\u00AD\u034F\u061C\u180E\u2060-\u2069]/gu,
+      ''
+    );
+
     // Step 2: Normalize Unicode to canonical form (handles homograph attacks)
     normalized = normalized.normalize('NFC');
-    
+
     // Step 3: Convert to lowercase for case-insensitive comparison
     normalized = normalized.toLowerCase();
-    
+
     // Step 4: Trim whitespace and remove all internal whitespace (security bypass prevention)
     normalized = normalized.trim().replace(/\s+/g, '');
-    
+
     // Step 5: Normalize path separators (convert backslashes to forward slashes)
     normalized = normalized.replace(/\\/g, '/');
-    
+
     // Step 6: Remove duplicate path separators
     normalized = normalized.replace(/\/+/g, '/');
-    
+
     // Step 7: Handle common homograph substitutions for ASCII characters
     const homographMap = {
       // Cyrillic lookalikes
-      'а': 'a', 'е': 'e', 'о': 'o', 'р': 'p', 'с': 'c', 'х': 'x', 'у': 'y',
-      'і': 'i', 'ј': 'j', 'ѕ': 's', 'һ': 'h', 'ց': 'q', 'ԁ': 'd', 'ɡ': 'g',
+      а: 'a',
+      е: 'e',
+      о: 'o',
+      р: 'p',
+      с: 'c',
+      х: 'x',
+      у: 'y',
+      і: 'i',
+      ј: 'j',
+      ѕ: 's',
+      һ: 'h',
+      ց: 'q',
+      ԁ: 'd',
+      ɡ: 'g',
       // Greek lookalikes
-      'α': 'a', 'β': 'b', 'γ': 'g', 'δ': 'd', 'ε': 'e', 'ζ': 'z', 'η': 'h',
-      'θ': 'o', 'ι': 'i', 'κ': 'k', 'λ': 'l', 'μ': 'm', 'ν': 'n', 'ξ': 'x',
-      'ο': 'o', 'π': 'p', 'ρ': 'p', 'σ': 's', 'τ': 't', 'υ': 'u', 'φ': 'f',
-      'χ': 'x', 'ψ': 'y', 'ω': 'w'
+      α: 'a',
+      β: 'b',
+      γ: 'g',
+      δ: 'd',
+      ε: 'e',
+      ζ: 'z',
+      η: 'h',
+      θ: 'o',
+      ι: 'i',
+      κ: 'k',
+      λ: 'l',
+      μ: 'm',
+      ν: 'n',
+      ξ: 'x',
+      ο: 'o',
+      π: 'p',
+      ρ: 'p',
+      σ: 's',
+      τ: 't',
+      υ: 'u',
+      φ: 'f',
+      χ: 'x',
+      ψ: 'y',
+      ω: 'w'
     };
-    
+
     // Replace homographs
     for (const [homograph, ascii] of Object.entries(homographMap)) {
       normalized = normalized.replace(new RegExp(homograph, 'g'), ascii);
     }
-    
+
     return normalized;
   }
 }

package/src/handlers/screenshot/ScreenshotAnalyzeToolHandler.js

@@ -1,5 +1,4 @@
 import { BaseToolHandler } from '../base/BaseToolHandler.js';
-import fs from 'fs/promises';
 import path from 'path';
 
 /**
@@ -24,18 +23,20 @@ export class ScreenshotAnalyzeToolHandler extends BaseToolHandler {
           analysisType: {
             type: 'string',
             enum: ['basic', 'ui', 'content', 'full'],
-            
-            description: 'Type of analysis: basic (colors, dimensions), ui (UI element detection), content (scene content), full (all)'
+
+            description:
+              'Type of analysis: basic (colors, dimensions), ui (UI element detection), content (scene content), full (all)'
           },
           prompt: {
             type: 'string',
-            description: 'Optional prompt for AI-based analysis (e.g., "Find all buttons in the UI")'
+            description:
+              'Optional prompt for AI-based analysis (e.g., "Find all buttons in the UI")'
           }
         },
         required: []
       }
     );
-    
+
     this.unityConnection = unityConnection;
   }
 
@@ -62,7 +63,7 @@ export class ScreenshotAnalyzeToolHandler extends BaseToolHandler {
       if (!imagePath.startsWith('Assets/')) {
         throw new Error('imagePath must be within the Assets folder');
       }
-      
+
       const ext = path.extname(imagePath).toLowerCase();
       if (!['.png', '.jpg', '.jpeg'].includes(ext)) {
         throw new Error('imagePath must be a PNG or JPEG file');
@@ -159,24 +160,25 @@ export class ScreenshotAnalyzeToolHandler extends BaseToolHandler {
         result.analysis = {
           note: 'Basic analysis of base64 images requires image processing library integration',
           fileSize: fileSize,
-          estimatedFormat: fileSize > 100000 ? 'Likely PNG or high-quality JPEG' : 'Likely compressed JPEG'
+          estimatedFormat:
+            fileSize > 100000 ? 'Likely PNG or high-quality JPEG' : 'Likely compressed JPEG'
         };
         break;
-        
+
       case 'ui':
         result.uiAnalysis = {
           note: 'UI element detection requires computer vision integration',
           placeholder: 'This would detect buttons, text fields, panels, etc.'
         };
         break;
-        
+
       case 'content':
         result.contentAnalysis = {
           note: 'Content analysis requires scene understanding models',
           placeholder: 'This would identify GameObjects, lighting, materials, etc.'
         };
         break;
-        
+
       case 'full':
         result.fullAnalysis = {
           basic: { note: 'Requires image processing library' },