@n0zer0d4y/vulcan-file-ops 1.2.10 → 1.2.11

package/CHANGELOG.md

@@ -9,6 +9,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+## [1.2.11] - 2026-02-21
+
+### Fixed
+
+- **CRITICAL**: Fixed MCP server toggle failure and "no tools configured" error in Claude Desktop and Cursor.
+  - **Root Cause**: ES Module execution order caused `parseArguments()` to run before console suppression logic. This allowed warning messages to be emitted to the protocol stream, corrupting JSON-RPC communication.
+  - **Solution**: Moved console suppression to the absolute top of `server/index.ts` and deferred argument parsing to the `runServer()` function.
+  - Improved MCP mode detection to exclude `--help` and `--version` flags.
+- See `local_docs/RCA-MCP-Toggle-Failure-Stdout-Pollution-2026-02-21.md` for full details.
+
 ## [1.2.10] - 2026-01-31
 
 ### Fixed

package/dist/cli.js

@@ -1,9 +1,11 @@
 #!/usr/bin/env node
 // CRITICAL: Detect MCP mode and suppress console output BEFORE any imports
 // MCP servers use stdin/stdout for JSON-RPC via stdio transport
-// Detection: stdin/stdout are NOT TTY (piped/redirected) = MCP mode
-const isMCP = (!process.stdin.isTTY && !process.stdout.isTTY) ||
-    process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio"));
+// Detection: stdin/stdout are NOT TTY (piped/redirected) OR explicit MCP flags are present.
+// Note: We exclude help/version flags to allow CLI usage even when piped.
+const isMCP = ((!process.stdin.isTTY && !process.stdout.isTTY) ||
+    process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio") || arg.includes("inspector"))) &&
+    !process.argv.some((arg) => arg === "--help" || arg === "-h" || arg === "--version" || arg === "-v");
 if (isMCP) {
     // Suppress all console methods (but NOT stdout/stderr streams - MCP SDK needs those)
     const noop = () => { };

package/dist/server/index.js

@@ -1,4 +1,19 @@
 #!/usr/bin/env node
+// CRITICAL: Console suppression MUST happen BEFORE any other imports
+// due to ES Module static import execution order (imports are evaluated BEFORE top-level code).
+// Detection: stdin/stdout are NOT TTY (piped/redirected) OR explicit MCP flags are present.
+// Note: We exclude help/version flags to allow CLI usage even when piped.
+const _isMCP = ((!process.stdin.isTTY && !process.stdout.isTTY) ||
+    process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio") || arg.includes("inspector"))) &&
+    !process.argv.some((arg) => arg === "--help" || arg === "-h" || arg === "--version" || arg === "-v");
+if (_isMCP) {
+    const noop = () => { };
+    console.log = noop;
+    console.error = noop;
+    console.warn = noop;
+    console.info = noop;
+    console.debug = noop;
+}
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { CallToolRequestSchema, ListToolsRequestSchema, InitializeRequestSchema, PingRequestSchema, RootsListChangedNotificationSchema, } from "@modelcontextprotocol/sdk/types.js";
@@ -183,21 +198,12 @@ function parseArguments() {
     }
     return directories;
 }
-const directoryArgs = parseArguments();
+// Defer parsing until runServer is called
+let directoryArgs = [];
 // Async initialization function to be called in runServer()
 async function initializeDirectories() {
     // Detect MCP mode: stdin/stdout are NOT TTY (piped) = MCP mode
-    const isMCP = (!process.stdin.isTTY && !process.stdout.isTTY) ||
-        process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio"));
-    // During MCP operation, suppress ALL console output to prevent protocol corruption
-    if (isMCP) {
-        const noop = () => { };
-        console.error = noop;
-        console.log = noop;
-        console.warn = noop;
-        console.info = noop;
-        console.debug = noop;
-    }
+    const isMCP = _isMCP;
     if (!isMCP &&
         (approvedFoldersFromArgs.length > 0 ||
             directoryArgs.length > 0 ||
@@ -631,6 +637,7 @@ export async function runServer() {
     // Initialize directories before starting server
     // BUT: Don't exit on errors during MCP mode - just log and continue
     try {
+        directoryArgs = parseArguments();
         await initializeDirectories();
     }
     catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@n0zer0d4y/vulcan-file-ops",
-  "version": "1.2.10",
+  "version": "1.2.11",
   "mcpName": "io.github.n0zer0d4y/vulcan-file-ops",
   "description": "MCP server for AI assistants: read, write, edit, and manage files securely on local filesystem.",
   "license": "MIT",