@akiojin/unity-mcp-server 2.43.2 → 2.43.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akiojin/unity-mcp-server",
-  "version": "2.43.2",
+  "version": "2.43.3",
   "description": "MCP server and Unity Editor bridge — enables AI assistants to control Unity for AI-assisted workflows",
   "type": "module",
   "main": "src/core/server.js",

package/src/core/server.js

@@ -10,7 +10,7 @@ import { UnityConnection } from './unityConnection.js';
 import { createHandlers } from '../handlers/index.js';
 import { config, logger } from './config.js';
 import { IndexWatcher } from './indexWatcher.js';
-import { HybridStdioServerTransport } from './transports/HybridStdioServerTransport.js';
+import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
 
 // Create Unity connection
 const unityConnection = new UnityConnection();
@@ -189,7 +189,7 @@ export async function startServer(options = {}) {
     let transport;
     if (runtimeConfig.stdioEnabled !== false) {
       console.error(`[unity-mcp-server] MCP transport connecting...`);
-      transport = new HybridStdioServerTransport();
+      transport = new StdioServerTransport();
       await server.connect(transport);
       console.error(`[unity-mcp-server] MCP transport connected`);
 
@@ -261,41 +261,40 @@ export async function startServer(options = {}) {
     process.on('SIGTERM', stopWatch);
 
     // Auto-initialize code index if DB doesn't exist
-    // DISABLED: Investigating Issue #168 - npx timeout with Claude Code
-    // (async () => {
-    //   try {
-    //     const { CodeIndex } = await import('./codeIndex.js');
-    //     const index = new CodeIndex(unityConnection);
-    //     const ready = await index.isReady();
-
-    //     if (!ready) {
-    //       if (index.disabled) {
-    //         logger.warning(
-    //           `[startup] Code index disabled: ${index.disableReason || 'SQLite native binding missing'}. Skipping auto-build.`
-    //         );
-    //         return;
-    //       }
-    //       logger.info('[startup] Code index DB not ready. Starting auto-build...');
-    //       const { CodeIndexBuildToolHandler } = await import(
-    //         '../handlers/script/CodeIndexBuildToolHandler.js'
-    //       );
-    //       const builder = new CodeIndexBuildToolHandler(unityConnection);
-    //       const result = await builder.execute({});
-
-    //       if (result.success) {
-    //         logger.info(
-    //           `[startup] Code index auto-build started: jobId=${result.jobId}. Use code_index_status to check progress.`
-    //         );
-    //       } else {
-    //         logger.warning(`[startup] Code index auto-build failed: ${result.message}`);
-    //       }
-    //     } else {
-    //       logger.info('[startup] Code index DB already exists. Skipping auto-build.');
-    //     }
-    //   } catch (e) {
-    //     logger.warning(`[startup] Code index auto-init failed: ${e.message}`);
-    //   }
-    // })();
+    (async () => {
+      try {
+        const { CodeIndex } = await import('./codeIndex.js');
+        const index = new CodeIndex(unityConnection);
+        const ready = await index.isReady();
+
+        if (!ready) {
+          if (index.disabled) {
+            logger.warning(
+              `[startup] Code index disabled: ${index.disableReason || 'SQLite native binding missing'}. Skipping auto-build.`
+            );
+            return;
+          }
+          logger.info('[startup] Code index DB not ready. Starting auto-build...');
+          const { CodeIndexBuildToolHandler } = await import(
+            '../handlers/script/CodeIndexBuildToolHandler.js'
+          );
+          const builder = new CodeIndexBuildToolHandler(unityConnection);
+          const result = await builder.execute({});
+
+          if (result.success) {
+            logger.info(
+              `[startup] Code index auto-build started: jobId=${result.jobId}. Use code_index_status to check progress.`
+            );
+          } else {
+            logger.warning(`[startup] Code index auto-build failed: ${result.message}`);
+          }
+        } else {
+          logger.info('[startup] Code index DB already exists. Skipping auto-build.');
+        }
+      } catch (e) {
+        logger.warning(`[startup] Code index auto-init failed: ${e.message}`);
+      }
+    })();
 
     // Handle shutdown
     process.on('SIGINT', async () => {

package/src/core/transports/HybridStdioServerTransport.js

@@ -1,191 +0,0 @@
-import process from 'node:process';
-import { JSONRPCMessageSchema } from '@modelcontextprotocol/sdk/types.js';
-
-const HEADER_END = '\r\n\r\n';
-const HEADER_RE = /Content-Length:\s*(\d+)/i;
-const DEFAULT_BUFFER = Buffer.alloc(0);
-
-function encodeContentLength(message) {
-  const json = JSON.stringify(message);
-  const header = `Content-Length: ${Buffer.byteLength(json, 'utf8')}${HEADER_END}`;
-  return header + json;
-}
-
-function parseJson(text) {
-  return JSONRPCMessageSchema.parse(JSON.parse(text));
-}
-
-export class HybridStdioServerTransport {
-  constructor(stdin = process.stdin, stdout = process.stdout) {
-    this._stdin = stdin;
-    this._stdout = stdout;
-    this._buffer = DEFAULT_BUFFER;
-    this._started = false;
-    this._mode = null; // 'content-length' | 'ndjson'
-
-    this._onData = chunk => {
-      this._buffer = this._buffer.length
-        ? Buffer.concat([this._buffer, chunk])
-        : Buffer.from(chunk);
-      this._processBuffer();
-    };
-
-    this._onError = error => {
-      this.onerror?.(error);
-    };
-  }
-
-  get framingMode() {
-    return this._mode;
-  }
-
-  async start() {
-    if (this._started) {
-      throw new Error('HybridStdioServerTransport already started');
-    }
-    this._started = true;
-    this._stdin.on('data', this._onData);
-    this._stdin.on('error', this._onError);
-  }
-
-  async close() {
-    if (!this._started) return;
-    this._stdin.off('data', this._onData);
-    this._stdin.off('error', this._onError);
-    this._buffer = DEFAULT_BUFFER;
-    this._started = false;
-    this.onclose?.();
-  }
-
-  send(message) {
-    return new Promise(resolve => {
-      // Always use Content-Length framing for output (MCP protocol standard)
-      // Input remains hybrid (Content-Length or NDJSON) for client compatibility
-      const payload = encodeContentLength(message);
-      if (this._stdout.write(payload)) {
-        resolve();
-      } else {
-        this._stdout.once('drain', resolve);
-      }
-    });
-  }
-
-  _processBuffer() {
-    while (true) {
-      const message = this._readMessage();
-      if (message === null) {
-        break;
-      }
-      this.onmessage?.(message);
-    }
-  }
-
-  _readMessage() {
-    if (!this._buffer || this._buffer.length === 0) {
-      return null;
-    }
-
-    if (this._mode === 'content-length') {
-      return this._readContentLengthMessage();
-    }
-    if (this._mode === 'ndjson') {
-      return this._readNdjsonMessage();
-    }
-
-    const prefix = this._peekPrefix();
-    if (!prefix.length) {
-      return null;
-    }
-
-    if ('content-length:'.startsWith(prefix.toLowerCase())) {
-      return null; // Wait for full header keyword before deciding
-    }
-
-    if (prefix.toLowerCase().startsWith('content-length:')) {
-      this._mode = 'content-length';
-      return this._readContentLengthMessage();
-    }
-
-    const newlineIndex = this._buffer.indexOf(0x0a); // '\n'
-    if (newlineIndex === -1) {
-      return null;
-    }
-
-    this._mode = 'ndjson';
-    return this._readNdjsonMessage();
-  }
-
-  _peekPrefix() {
-    const length = Math.min(this._buffer.length, 32);
-    return this._buffer.toString('utf8', 0, length).trimStart();
-  }
-
-  _readContentLengthMessage() {
-    const { headerEndIndex, separatorLength } = (() => {
-      const crlfIndex = this._buffer.indexOf('\r\n\r\n');
-      const lfIndex = this._buffer.indexOf('\n\n');
-
-      if (crlfIndex === -1 && lfIndex === -1) return { headerEndIndex: -1, separatorLength: 0 };
-      if (crlfIndex === -1) return { headerEndIndex: lfIndex, separatorLength: 2 };
-      if (lfIndex === -1) return { headerEndIndex: crlfIndex, separatorLength: 4 };
-
-      return crlfIndex < lfIndex
-        ? { headerEndIndex: crlfIndex, separatorLength: 4 }
-        : { headerEndIndex: lfIndex, separatorLength: 2 };
-    })();
-
-    if (headerEndIndex === -1) {
-      return null;
-    }
-
-    const header = this._buffer.toString('utf8', 0, headerEndIndex);
-    const match = header.match(HEADER_RE);
-    if (!match) {
-      this._buffer = this._buffer.subarray(headerEndIndex + separatorLength);
-      this.onerror?.(new Error('Invalid Content-Length header'));
-      return null;
-    }
-
-    const length = Number(match[1]);
-    const totalMessageLength = headerEndIndex + separatorLength + length;
-    if (this._buffer.length < totalMessageLength) {
-      return null;
-    }
-
-    const json = this._buffer.toString(
-      'utf8',
-      headerEndIndex + separatorLength,
-      totalMessageLength
-    );
-    this._buffer = this._buffer.subarray(totalMessageLength);
-
-    try {
-      return parseJson(json);
-    } catch (error) {
-      this.onerror?.(error);
-      return null;
-    }
-  }
-
-  _readNdjsonMessage() {
-    while (true) {
-      const newlineIndex = this._buffer.indexOf(0x0a);
-      if (newlineIndex === -1) {
-        return null;
-      }
-
-      let line = this._buffer.toString('utf8', 0, newlineIndex);
-      this._buffer = this._buffer.subarray(newlineIndex + 1);
-      line = line.trim();
-      if (!line) {
-        continue;
-      }
-
-      try {
-        return parseJson(line);
-      } catch (error) {
-        this.onerror?.(error);
-      }
-    }
-  }
-}