@n0zer0d4y/vulcan-file-ops 1.2.4 → 1.2.5

package/dist/cli.js

@@ -2,8 +2,13 @@
 // 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"));
+//
+// EXCEPTION: --help and --version flags should always output to console
+// even when stdin/stdout are piped (e.g., in scripts or CI environments)
+const isHelpOrVersion = process.argv.some((arg) => arg === "--help" || arg === "-h" || arg === "--version" || arg === "-v");
+const isMCP = !isHelpOrVersion &&
+    ((!process.stdin.isTTY && !process.stdout.isTTY) ||
+        process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio")));
 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,24 @@
 #!/usr/bin/env node
+// CRITICAL: Suppress console output IMMEDIATELY, before any imports
+// This MUST be the first executable code due to ES Module execution order.
+// With static imports, child modules are evaluated BEFORE the importing
+// module's top-level code runs. By placing suppression here at the very top
+// of the main server module, we ensure no console output can corrupt the
+// MCP JSON-RPC protocol stream on stdout.
+// See: RCA-Claude-Desktop-MCP-Toggle-Failure-2026-01-31.md
+// Check for CLI info flags BEFORE suppression - these should always work
+const _isHelpOrVersion = process.argv.some((arg) => arg === "--help" || arg === "-h" || arg === "--version" || arg === "-v");
+const _mcpMode = !_isHelpOrVersion &&
+    ((!process.stdin.isTTY && !process.stdout.isTTY) ||
+        process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio")));
+if (_mcpMode) {
+    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,22 +203,13 @@ function parseArguments() {
     }
     return directories;
 }
-const directoryArgs = parseArguments();
+// Defer parseArguments() call to runServer() to avoid module-level side effects
+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;
-    }
-    if (!isMCP &&
+    // Use the already-determined MCP mode from top of file
+    // Console suppression is already done at module load time via _mcpMode
+    if (!_mcpMode &&
         (approvedFoldersFromArgs.length > 0 ||
             directoryArgs.length > 0 ||
             ignoredFolders.length > 0 ||
@@ -248,7 +259,7 @@ async function initializeDirectories() {
                     const errorMsg = `Error: Approved folder ${dir} is not a directory`;
                     console.error(errorMsg);
                     // In MCP mode, don't exit - just skip invalid directories
-                    if (!isMCP) {
+                    if (!_mcpMode) {
                         process.exit(1);
                     }
                 }
@@ -257,13 +268,13 @@ async function initializeDirectories() {
                 const errorMsg = `Error accessing approved folder ${dir}: ${error}`;
                 console.error(errorMsg);
                 // In MCP mode, don't exit - just skip invalid directories
-                if (!isMCP) {
+                if (!_mcpMode) {
                     process.exit(1);
                 }
             }
         }));
         // Only log initialization if not running under MCP
-        if (!isMCP) {
+        if (!_mcpMode) {
             console.error(`Initialized with ${allowedDirectories.length} approved ${allowedDirectories.length === 1 ? "directory" : "directories"}`);
         }
     }
@@ -291,7 +302,7 @@ async function initializeDirectories() {
                     const errorMsg = `Error: Legacy directory ${dir} is not a directory`;
                     console.error(errorMsg);
                     // In MCP mode, don't exit - just skip invalid directories
-                    if (!isMCP) {
+                    if (!_mcpMode) {
                         process.exit(1);
                     }
                 }
@@ -300,7 +311,7 @@ async function initializeDirectories() {
                 const errorMsg = `Error accessing legacy directory ${dir}: ${error}`;
                 console.error(errorMsg);
                 // In MCP mode, don't exit - just skip invalid directories
-                if (!isMCP) {
+                if (!_mcpMode) {
                     process.exit(1);
                 }
             }
@@ -322,7 +333,7 @@ async function initializeDirectories() {
     // Priority 1: --approved-commands from CLI (supersedes .env)
     if (approvedCommandsFromArgs.length > 0) {
         finalApprovedCommands = approvedCommandsFromArgs;
-        if (!isMCP) {
+        if (!_mcpMode) {
             console.error(`  Approved commands (from CLI): ${finalApprovedCommands.join(", ")}`);
         }
     }
@@ -335,13 +346,13 @@ async function initializeDirectories() {
                 finalApprovedCommands = process.env.APPROVED_COMMANDS.split(",")
                     .map((c) => c.trim())
                     .filter((c) => c.length > 0);
-                if (!isMCP) {
+                if (!_mcpMode) {
                     console.error(`  Approved commands (from .env): ${finalApprovedCommands.join(", ")}`);
                 }
             }
         }
         catch (error) {
-            if (!isMCP) {
+            if (!_mcpMode) {
                 console.error("  Note: Could not load .env file (this is okay if using CLI args)");
             }
         }
@@ -349,14 +360,14 @@ async function initializeDirectories() {
     // Initialize shell tool with approved commands
     if (finalApprovedCommands.length > 0) {
         initializeShellTool(finalApprovedCommands);
-        if (!isMCP) {
+        if (!_mcpMode) {
             console.error(`Initialized shell tool with ${finalApprovedCommands.length} approved command(s)`);
         }
     }
     else {
         initializeShellTool([]);
         // Only log shell initialization if not running under MCP
-        if (!isMCP) {
+        if (!_mcpMode) {
             console.error("Shell tool initialized with no pre-approved commands (all commands require approval)");
         }
     }
@@ -625,6 +636,8 @@ server.oninitialized = async () => {
 };
 // Start server
 export async function runServer() {
+    // Parse arguments here (not at module level) to ensure console suppression is active
+    directoryArgs = parseArguments();
     // Initialize directories before starting server
     // BUT: Don't exit on errors during MCP mode - just log and continue
     try {
@@ -633,9 +646,7 @@ export async function runServer() {
     catch (error) {
         // In MCP mode, don't crash the server on init errors
         // Just continue with empty configuration
-        const isMCP = (!process.stdin.isTTY && !process.stdout.isTTY) ||
-            process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio"));
-        if (!isMCP) {
+        if (!_mcpMode) {
             // In non-MCP mode, we can show errors and exit
             throw error;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@n0zer0d4y/vulcan-file-ops",
-  "version": "1.2.4",
+  "version": "1.2.5",
   "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",