fix: remove console logging

Author: GhostTypes

src/index.ts

@@ -141,8 +141,8 @@ class MinecraftDevMCPServer {
       await verifyJavaVersion(17);
     } catch (error) {
       logger.error('Java verification failed', error);
-      console.error('\n❌ Java 17+ is required but not found or not working.');
-      console.error('Please install Java 17 or higher and ensure it is in your PATH.\n');
+      // Don't use console.error - it breaks MCP stdio protocol
+      // Error will be logged to file and server will exit
       process.exit(1);
     }
 
@@ -159,6 +159,6 @@ class MinecraftDevMCPServer {
 const server = new MinecraftDevMCPServer();
 server.start().catch((error) => {
   logger.error('Failed to start server', error);
-  console.error('Failed to start server:', error);
+  // Don't use console.error - it breaks MCP stdio protocol
   process.exit(1);
 });

src/utils/logger.ts

@@ -38,19 +38,12 @@ class Logger {
 
     const formatted = this.format(levelName, message, meta);
 
-    // Console output
-    if (level >= LogLevel.WARN) {
-      console.error(formatted.trim());
-    } else {
-      console.log(formatted.trim());
-    }
-
-    // File output
+    // File output only - MCP servers must not write to stdout/stderr
+    // as it interferes with JSON-RPC communication over stdio
     try {
       appendFileSync(this.logFile, formatted, 'utf8');
     } catch (error) {
-      // Fail silently for file writes
-      console.error('Failed to write to log file:', error);
+      // Fail silently for file writes - cannot use console in MCP servers
     }
   }