@akiojin/unity-mcp-server 2.40.2 → 2.40.3

package/src/handlers/script/CodeIndexStatusToolHandler.js

@@ -24,8 +24,9 @@ export class CodeIndexStatusToolHandler extends BaseToolHandler {
 
   async execute() {
     const jobManager = this.jobManager;
-    const buildJobs = jobManager.getAllJobs()
-      .filter((job) => typeof job?.id === 'string' && job.id.startsWith('build-'))
+    const buildJobs = jobManager
+      .getAllJobs()
+      .filter(job => typeof job?.id === 'string' && job.id.startsWith('build-'))
       .sort((a, b) => new Date(b.startedAt || 0).getTime() - new Date(a.startedAt || 0).getTime());
     const latestBuildJob = buildJobs.length > 0 ? buildJobs[0] : null;
 
@@ -61,7 +62,7 @@ export class CodeIndexStatusToolHandler extends BaseToolHandler {
     let totalFiles = 0;
     const breakdown = { assets: 0, packages: 0, packageCache: 0, other: 0 };
 
-    const visit = (targetPath) => {
+    const visit = targetPath => {
       try {
         if (!fs.existsSync(targetPath)) return;
         const stats = fs.statSync(targetPath);

package/src/handlers/script/CodeIndexUpdateToolHandler.js

@@ -159,7 +159,9 @@ export class CodeIndexUpdateToolHandler extends BaseToolHandler {
     let lastErr = null;
     for (let attempt = 0; attempt <= retry; attempt += 1) {
       try {
-        const res = await this.lsp.request('textDocument/documentSymbol', { textDocument: { uri } });
+        const res = await this.lsp.request('textDocument/documentSymbol', {
+          textDocument: { uri }
+        });
         return res?.result ?? res;
       } catch (err) {
         lastErr = err;
@@ -212,23 +214,31 @@ export class CodeIndexUpdateToolHandler extends BaseToolHandler {
   _toRelative(absPath, projectRoot) {
     const normAbs = String(absPath).replace(/\\/g, '/');
     const normRoot = String(projectRoot).replace(/\\/g, '/').replace(/\/$/, '');
-    return normAbs.startsWith(normRoot)
-      ? normAbs.substring(normRoot.length + 1)
-      : normAbs;
+    return normAbs.startsWith(normRoot) ? normAbs.substring(normRoot.length + 1) : normAbs;
   }
 
   _kindFromLsp(kind) {
     switch (kind) {
-      case 5: return 'class';
-      case 23: return 'struct';
-      case 11: return 'interface';
-      case 10: return 'enum';
-      case 6: return 'method';
-      case 7: return 'property';
-      case 8: return 'field';
-      case 3: return 'namespace';
-      case 9: return 'constructor';
-      default: return 'symbol';
+      case 5:
+        return 'class';
+      case 23:
+        return 'struct';
+      case 11:
+        return 'interface';
+      case 10:
+        return 'enum';
+      case 6:
+        return 'method';
+      case 7:
+        return 'property';
+      case 8:
+        return 'field';
+      case 3:
+        return 'namespace';
+      case 9:
+        return 'constructor';
+      default:
+        return 'symbol';
     }
   }
 }

package/src/handlers/script/ScriptCreateClassToolHandler.js

@@ -11,26 +11,54 @@ export class ScriptCreateClassToolHandler extends BaseToolHandler {
       {
         type: 'object',
         properties: {
-          path: { type: 'string', description: 'Project-relative C# file path (e.g., Assets/Scripts/MyClass.cs)' },
+          path: {
+            type: 'string',
+            description: 'Project-relative C# file path (e.g., Assets/Scripts/MyClass.cs)'
+          },
           className: { type: 'string', description: 'Class name' },
           namespace: { type: 'string', description: 'Optional namespace' },
           baseType: { type: 'string', description: 'Optional base type (e.g., MonoBehaviour)' },
-          usings: { type: 'string', description: 'Comma-separated using directives (e.g., System,Newtonsoft.Json)' },
+          usings: {
+            type: 'string',
+            description: 'Comma-separated using directives (e.g., System,Newtonsoft.Json)'
+          },
           partial: { type: 'boolean', description: 'Create as partial class (default: false)' },
           apply: { type: 'boolean', description: 'Apply immediately (default: false)' }
         },
-        required: ['path','className']
+        required: ['path', 'className']
       }
     );
     this.unityConnection = unityConnection;
   }
 
   async execute(params) {
-    const { path: rel, className, namespace: ns, baseType, usings, partial = false, apply = false } = params;
-    const relative = String(rel).replace(/\\\\/g, '/').endsWith('.cs') ? String(rel).replace(/\\\\/g, '/') : String(rel).replace(/\\\\/g, '/') + '.cs';
-    const code = this._buildSource({ className: String(className), ns: ns ? String(ns) : null, baseType: baseType ? String(baseType) : null, usings: usings ? String(usings) : null, partial: !!partial });
+    const {
+      path: rel,
+      className,
+      namespace: ns,
+      baseType,
+      usings,
+      partial = false,
+      apply = false
+    } = params;
+    const relative = String(rel).replace(/\\\\/g, '/').endsWith('.cs')
+      ? String(rel).replace(/\\\\/g, '/')
+      : String(rel).replace(/\\\\/g, '/') + '.cs';
+    const code = this._buildSource({
+      className: String(className),
+      ns: ns ? String(ns) : null,
+      baseType: baseType ? String(baseType) : null,
+      usings: usings ? String(usings) : null,
+      partial: !!partial
+    });
     if (!apply) {
-      return { success: true, applied: false, preview: code.slice(0, 1000), previewTruncated: code.length > 1000, relative: relative };
+      return {
+        success: true,
+        applied: false,
+        preview: code.slice(0, 1000),
+        previewTruncated: code.length > 1000,
+        relative: relative
+      };
     }
     const full = this._resolveFullPath(relative);
     fs.mkdirSync(path.dirname(full), { recursive: true });
@@ -46,8 +74,15 @@ export class ScriptCreateClassToolHandler extends BaseToolHandler {
 
   _buildSource({ className, ns, baseType, usings, partial }) {
     const useList = [];
-    if (usings) useList.push(...usings.split(',').map(s => s.trim()).filter(Boolean));
-    if (baseType === 'MonoBehaviour' && !useList.includes('UnityEngine')) useList.push('UnityEngine');
+    if (usings)
+      useList.push(
+        ...usings
+          .split(',')
+          .map(s => s.trim())
+          .filter(Boolean)
+      );
+    if (baseType === 'MonoBehaviour' && !useList.includes('UnityEngine'))
+      useList.push('UnityEngine');
     const header = useList.length ? useList.map(u => `using ${u};`).join('\n') + '\n\n' : '';
     const partialKw = partial ? ' partial' : '';
     const baseClause = baseType ? ` : ${baseType}` : '';

package/src/handlers/script/ScriptPackagesListToolHandler.js

@@ -5,99 +5,99 @@ import { ProjectInfoProvider } from '../../core/projectInfo.js';
 import { logger } from '../../core/config.js';
 
 export class ScriptPackagesListToolHandler extends BaseToolHandler {
-    constructor(unityConnection) {
-        super(
-            'script_packages_list',
-            'List Unity packages in the project (optionally include built‑in). BEST PRACTICES: Use to discover available packages and their paths. Set includeBuiltIn=false to see only user packages. Returns package IDs, versions, and resolved paths. Embedded packages can be edited directly. Essential for understanding project dependencies.',
-            {
-                type: 'object',
-                properties: {
-                    includeBuiltIn: {
-                        type: 'boolean',
-                        description: 'If true, includes built‑in packages in results (default: false).'
-                    }
-                },
-                required: []
-            }
-        );
-        this.unityConnection = unityConnection;
-        this.projectInfo = new ProjectInfoProvider(unityConnection);
-    }
+  constructor(unityConnection) {
+    super(
+      'script_packages_list',
+      'List Unity packages in the project (optionally include built‑in). BEST PRACTICES: Use to discover available packages and their paths. Set includeBuiltIn=false to see only user packages. Returns package IDs, versions, and resolved paths. Embedded packages can be edited directly. Essential for understanding project dependencies.',
+      {
+        type: 'object',
+        properties: {
+          includeBuiltIn: {
+            type: 'boolean',
+            description: 'If true, includes built‑in packages in results (default: false).'
+          }
+        },
+        required: []
+      }
+    );
+    this.unityConnection = unityConnection;
+    this.projectInfo = new ProjectInfoProvider(unityConnection);
+  }
 
-    async execute(params) {
-        const { includeBuiltIn = false } = params;
-        const info = await this.projectInfo.get();
+  async execute(params) {
+    const { includeBuiltIn = false } = params;
+    const info = await this.projectInfo.get();
 
-        // Prefer packages-lock.json for authoritative list (includes builtin/embedded/registry)
-        const lockPath = path.join(info.projectRoot, 'Packages', 'packages-lock.json');
-        const manifestPath = path.join(info.projectRoot, 'Packages', 'manifest.json');
-        let results = [];
-        try {
-            if (fs.existsSync(lockPath)) {
-                const json = JSON.parse(fs.readFileSync(lockPath, 'utf8'));
-                const deps = json?.dependencies || {};
-                for (const [name, meta] of Object.entries(deps)) {
-                    const source = String(meta.source || '').toLowerCase();
-                    if (!includeBuiltIn && source === 'builtin') continue;
-                    const version = String(meta.version || '');
-                    let resolvedPath = null;
-                    let isEmbedded = source === 'embedded';
-                    if (isEmbedded) {
-                        // For embedded, manifest specifies file:folder
-                        try {
-                            const man = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
-                            const spec = man?.dependencies?.[name];
-                            if (typeof spec === 'string' && spec.startsWith('file:')) {
-                                const folder = spec.substring('file:'.length);
-                                resolvedPath = path.join(info.projectRoot, 'Packages', folder).replace(/\\/g, '/');
-                            }
-                        } catch {}
-                    }
-                    results.push({
-                        packageId: `${name}@${version}`,
-                        name,
-                        displayName: name,
-                        version,
-                        source,
-                        isEmbedded,
-                        resolvedPath
-                    });
-                }
-            } else if (fs.existsSync(manifestPath)) {
-                const man = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
-                const deps = man?.dependencies || {};
-                for (const [name, spec] of Object.entries(deps)) {
-                    let source = 'registry';
-                    let version = String(spec);
-                    let resolvedPath = null;
-                    let isEmbedded = false;
-                    if (version.startsWith('file:')) {
-                        source = 'embedded';
-                        isEmbedded = true;
-                        const folder = version.substring('file:'.length);
-                        resolvedPath = path.join(info.projectRoot, 'Packages', folder).replace(/\\/g, '/');
-                    }
-                    if (!includeBuiltIn && source === 'builtin') continue;
-                    results.push({
-                        packageId: `${name}@${version}`,
-                        name,
-                        displayName: name,
-                        version,
-                        source,
-                        isEmbedded,
-                        resolvedPath
-                    });
-                }
-            } else {
-                return { success: true, packages: [], totalCount: 0 };
-            }
-        } catch (e) {
-            logger.error(`[script_packages_list] local parse failed: ${e.message}`);
-            return { error: e.message };
+    // Prefer packages-lock.json for authoritative list (includes builtin/embedded/registry)
+    const lockPath = path.join(info.projectRoot, 'Packages', 'packages-lock.json');
+    const manifestPath = path.join(info.projectRoot, 'Packages', 'manifest.json');
+    let results = [];
+    try {
+      if (fs.existsSync(lockPath)) {
+        const json = JSON.parse(fs.readFileSync(lockPath, 'utf8'));
+        const deps = json?.dependencies || {};
+        for (const [name, meta] of Object.entries(deps)) {
+          const source = String(meta.source || '').toLowerCase();
+          if (!includeBuiltIn && source === 'builtin') continue;
+          const version = String(meta.version || '');
+          let resolvedPath = null;
+          let isEmbedded = source === 'embedded';
+          if (isEmbedded) {
+            // For embedded, manifest specifies file:folder
+            try {
+              const man = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
+              const spec = man?.dependencies?.[name];
+              if (typeof spec === 'string' && spec.startsWith('file:')) {
+                const folder = spec.substring('file:'.length);
+                resolvedPath = path.join(info.projectRoot, 'Packages', folder).replace(/\\/g, '/');
+              }
+            } catch {}
+          }
+          results.push({
+            packageId: `${name}@${version}`,
+            name,
+            displayName: name,
+            version,
+            source,
+            isEmbedded,
+            resolvedPath
+          });
         }
-
-        // Sort by name
-        results.sort((a, b) => a.name.localeCompare(b.name));
-        return { success: true, packages: results, totalCount: results.length };
+      } else if (fs.existsSync(manifestPath)) {
+        const man = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
+        const deps = man?.dependencies || {};
+        for (const [name, spec] of Object.entries(deps)) {
+          let source = 'registry';
+          let version = String(spec);
+          let resolvedPath = null;
+          let isEmbedded = false;
+          if (version.startsWith('file:')) {
+            source = 'embedded';
+            isEmbedded = true;
+            const folder = version.substring('file:'.length);
+            resolvedPath = path.join(info.projectRoot, 'Packages', folder).replace(/\\/g, '/');
+          }
+          if (!includeBuiltIn && source === 'builtin') continue;
+          results.push({
+            packageId: `${name}@${version}`,
+            name,
+            displayName: name,
+            version,
+            source,
+            isEmbedded,
+            resolvedPath
+          });
+        }
+      } else {
+        return { success: true, packages: [], totalCount: 0 };
+      }
+    } catch (e) {
+      logger.error(`[script_packages_list] local parse failed: ${e.message}`);
+      return { error: e.message };
     }
+
+    // Sort by name
+    results.sort((a, b) => a.name.localeCompare(b.name));
+    return { success: true, packages: results, totalCount: results.length };
+  }
 }

package/src/handlers/script/ScriptRefactorRenameToolHandler.js

@@ -3,81 +3,90 @@ import { LspRpcClient } from '../../lsp/LspRpcClient.js';
 import { ProjectInfoProvider } from '../../core/projectInfo.js';
 
 export class ScriptRefactorRenameToolHandler extends BaseToolHandler {
-    constructor(unityConnection) {
-        super(
-            'script_refactor_rename',
-            'Refactor: rename a symbol across the project using the bundled C# LSP. Required params: relative (file path starting with Assets/ or Packages/), namePath (container path like Outer/Nested/Member), newName. Guidance: resolve targets first (script_symbols_get/script_symbol_find), prefer fully-qualified namePath to avoid ambiguity, and use preview for diagnostics only (apply proceeds even if diagnostics exist; errors are returned in response). Responses are summarized (errors≤30, message≤200 chars, large text≤1000 chars).',
-            {
-                type: 'object',
-                properties: {
-                    relative: { type: 'string', description: 'Project-relative file path (Assets/ or Packages/)' },
-                    namePath: { type: 'string', description: 'Symbol path like Class/Method' },
-                    newName: { type: 'string', description: 'New name' },
-                    preview: { type: 'boolean', description: 'Preview changes before applying (default: true)' }
-                },
-                required: ['relative', 'namePath', 'newName']
-            }
-        );
-        this.projectInfo = new ProjectInfoProvider(unityConnection);
-        this.lsp = null;
-    }
+  constructor(unityConnection) {
+    super(
+      'script_refactor_rename',
+      'Refactor: rename a symbol across the project using the bundled C# LSP. Required params: relative (file path starting with Assets/ or Packages/), namePath (container path like Outer/Nested/Member), newName. Guidance: resolve targets first (script_symbols_get/script_symbol_find), prefer fully-qualified namePath to avoid ambiguity, and use preview for diagnostics only (apply proceeds even if diagnostics exist; errors are returned in response). Responses are summarized (errors≤30, message≤200 chars, large text≤1000 chars).',
+      {
+        type: 'object',
+        properties: {
+          relative: {
+            type: 'string',
+            description: 'Project-relative file path (Assets/ or Packages/)'
+          },
+          namePath: { type: 'string', description: 'Symbol path like Class/Method' },
+          newName: { type: 'string', description: 'New name' },
+          preview: {
+            type: 'boolean',
+            description: 'Preview changes before applying (default: true)'
+          }
+        },
+        required: ['relative', 'namePath', 'newName']
+      }
+    );
+    this.projectInfo = new ProjectInfoProvider(unityConnection);
+    this.lsp = null;
+  }
 
-    validate(params) {
-        super.validate(params);
-        const { relative, namePath, newName } = params;
-        if (!relative || !namePath || !newName) throw new Error('relative, namePath, newName are required');
-    }
+  validate(params) {
+    super.validate(params);
+    const { relative, namePath, newName } = params;
+    if (!relative || !namePath || !newName)
+      throw new Error('relative, namePath, newName are required');
+  }
 
-    async execute(params) {
-        const { relative, namePath, newName, preview = true } = params;
-        const info = await this.projectInfo.get();
-        if (!this.lsp) this.lsp = new LspRpcClient(info.projectRoot);
-        const resp = await this.lsp.request('mcp/renameByNamePath', {
-            relative: String(relative).replace(/\\\\/g, '/'),
-            namePath: String(namePath),
-            newName: String(newName),
-            apply: !preview
-        });
-        const r = resp?.result ?? resp;
-        return this._summarizeResult(r);
-    }
+  async execute(params) {
+    const { relative, namePath, newName, preview = true } = params;
+    const info = await this.projectInfo.get();
+    if (!this.lsp) this.lsp = new LspRpcClient(info.projectRoot);
+    const resp = await this.lsp.request('mcp/renameByNamePath', {
+      relative: String(relative).replace(/\\\\/g, '/'),
+      namePath: String(namePath),
+      newName: String(newName),
+      apply: !preview
+    });
+    const r = resp?.result ?? resp;
+    return this._summarizeResult(r);
+  }
 
-    _summarizeResult(res) {
-        if (!res || typeof res !== 'object') return res;
-        const MAX_ERRORS = 30;
-        const MAX_MSG_LEN = 200;
-        const MAX_TEXT_LEN = 1000;
-        const out = {};
-        if ('id' in res) out.id = res.id;
-        if ('success' in res) out.success = !!res.success;
-        if ('applied' in res) out.applied = !!res.applied;
-        if (Array.isArray(res.errors)) {
-            const trimmed = res.errors.slice(0, MAX_ERRORS).map(e => {
-                const o = {};
-                if (e && typeof e === 'object') {
-                    if ('id' in e) o.id = e.id;
-                    if ('message' in e) o.message = String(e.message).slice(0, MAX_MSG_LEN);
-                    if ('file' in e) o.file = String(e.file).slice(0, 260);
-                    if ('line' in e) o.line = e.line;
-                    if ('column' in e) o.column = e.column;
-                } else { o.message = String(e).slice(0, MAX_MSG_LEN); }
-                return o;
-            });
-            out.errorCount = trimmed.length;
-            out.totalErrors = res.errors.length;
-            out.errors = trimmed;
+  _summarizeResult(res) {
+    if (!res || typeof res !== 'object') return res;
+    const MAX_ERRORS = 30;
+    const MAX_MSG_LEN = 200;
+    const MAX_TEXT_LEN = 1000;
+    const out = {};
+    if ('id' in res) out.id = res.id;
+    if ('success' in res) out.success = !!res.success;
+    if ('applied' in res) out.applied = !!res.applied;
+    if (Array.isArray(res.errors)) {
+      const trimmed = res.errors.slice(0, MAX_ERRORS).map(e => {
+        const o = {};
+        if (e && typeof e === 'object') {
+          if ('id' in e) o.id = e.id;
+          if ('message' in e) o.message = String(e.message).slice(0, MAX_MSG_LEN);
+          if ('file' in e) o.file = String(e.file).slice(0, 260);
+          if ('line' in e) o.line = e.line;
+          if ('column' in e) o.column = e.column;
+        } else {
+          o.message = String(e).slice(0, MAX_MSG_LEN);
         }
-        // workspace情報は返さない（厳格: .sln必須のため）
+        return o;
+      });
+      out.errorCount = trimmed.length;
+      out.totalErrors = res.errors.length;
+      out.errors = trimmed;
+    }
+    // workspace情報は返さない（厳格: .sln必須のため）
 
-        for (const k of ['preview','diff','text','content']) {
-            if (typeof res[k] === 'string' && res[k].length > 0) {
-                out[k] = res[k].slice(0, MAX_TEXT_LEN);
-                if (res[k].length > MAX_TEXT_LEN) out[`${k}Truncated`] = true;
-            }
-        }
-        for (const k of ['operation','path','relative','symbolName']) {
-            if (res[k] !== undefined) out[k] = res[k];
-        }
-        return Object.keys(out).length ? out : res;
+    for (const k of ['preview', 'diff', 'text', 'content']) {
+      if (typeof res[k] === 'string' && res[k].length > 0) {
+        out[k] = res[k].slice(0, MAX_TEXT_LEN);
+        if (res[k].length > MAX_TEXT_LEN) out[`${k}Truncated`] = true;
+      }
+    }
+    for (const k of ['operation', 'path', 'relative', 'symbolName']) {
+      if (res[k] !== undefined) out[k] = res[k];
     }
+    return Object.keys(out).length ? out : res;
+  }
 }

package/src/handlers/script/ScriptRemoveSymbolToolHandler.js

@@ -14,10 +14,16 @@ export class ScriptRemoveSymbolToolHandler extends BaseToolHandler {
           path: { type: 'string', description: 'Project-relative C# file path' },
           namePath: { type: 'string', description: 'Symbol path like Outer/Nested/Member' },
           apply: { type: 'boolean', description: 'Apply changes immediately (default: false)' },
-          failOnReferences: { type: 'boolean', description: 'Fail if symbol has references (default: true)' },
-          removeEmptyFile: { type: 'boolean', description: 'Remove file if it becomes empty (default: false)' }
+          failOnReferences: {
+            type: 'boolean',
+            description: 'Fail if symbol has references (default: true)'
+          },
+          removeEmptyFile: {
+            type: 'boolean',
+            description: 'Remove file if it becomes empty (default: false)'
+          }
         },
-        required: ['path','namePath']
+        required: ['path', 'namePath']
       }
     );
     this.projectInfo = new ProjectInfoProvider(unityConnection);
@@ -25,7 +31,13 @@ export class ScriptRemoveSymbolToolHandler extends BaseToolHandler {
   }
 
   async execute(params) {
-    const { path, namePath, apply = false, failOnReferences = true, removeEmptyFile = false } = params;
+    const {
+      path,
+      namePath,
+      apply = false,
+      failOnReferences = true,
+      removeEmptyFile = false
+    } = params;
     const info = await this.projectInfo.get();
     if (!this.lsp) this.lsp = new LspRpcClient(info.projectRoot);
     const resp = await this.lsp.request('mcp/removeSymbol', {
@@ -56,7 +68,9 @@ export class ScriptRemoveSymbolToolHandler extends BaseToolHandler {
           if ('file' in e) o.file = String(e.file).slice(0, 260);
           if ('line' in e) o.line = e.line;
           if ('column' in e) o.column = e.column;
-        } else { o.message = String(e).slice(0, MAX_MSG_LEN); }
+        } else {
+          o.message = String(e).slice(0, MAX_MSG_LEN);
+        }
         return o;
       });
       out.errorCount = trimmed.length;
@@ -65,13 +79,13 @@ export class ScriptRemoveSymbolToolHandler extends BaseToolHandler {
     }
     // workspace情報は返さない（厳格: .sln必須のため）
 
-    for (const k of ['preview','diff','text','content']) {
+    for (const k of ['preview', 'diff', 'text', 'content']) {
       if (typeof res[k] === 'string' && res[k].length > 0) {
         out[k] = res[k].slice(0, MAX_TEXT_LEN);
         if (res[k].length > MAX_TEXT_LEN) out[`${k}Truncated`] = true;
       }
     }
-    for (const k of ['operation','path','relative','symbolName']) {
+    for (const k of ['operation', 'path', 'relative', 'symbolName']) {
       if (res[k] !== undefined) out[k] = res[k];
     }
     return Object.keys(out).length ? out : res;