@n0zer0d4y/vulcan-file-ops 1.2.6 → 1.2.8

package/dist/cli.js

@@ -2,19 +2,8 @@
 // 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
-//
-// 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");
-// MCP mode detection: suppress console if ANY of these are true:
-// 1. stdin is piped/redirected (not a TTY) - MCP clients pipe JSON-RPC via stdin
-// 2. stdout is piped/redirected (not a TTY) - MCP clients read JSON-RPC from stdout
-// 3. argv contains "mcp" or "stdio" - explicit MCP mode indicator
-// Using OR (||) instead of AND (&&) because MCP clients may only pipe one direction
-const isMCP = !isHelpOrVersion &&
-    (!process.stdin.isTTY ||
-        !process.stdout.isTTY ||
-        process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio")));
+const isMCP = (!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,33 +1,7 @@
 #!/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");
-// MCP mode detection: suppress console if ANY of these are true:
-// 1. stdin is piped/redirected (not a TTY) - MCP clients pipe JSON-RPC via stdin
-// 2. stdout is piped/redirected (not a TTY) - MCP clients read JSON-RPC from stdout
-// 3. argv contains "mcp" or "stdio" - explicit MCP mode indicator
-// Using OR (||) instead of AND (&&) because MCP clients may only pipe one direction
-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";
+import { CallToolRequestSchema, ListToolsRequestSchema, InitializeRequestSchema, PingRequestSchema, RootsListChangedNotificationSchema, LATEST_PROTOCOL_VERSION, } from "@modelcontextprotocol/sdk/types.js";
 import fs from "fs/promises";
 import path from "path";
 import dotenv from "dotenv";
@@ -209,13 +183,22 @@ function parseArguments() {
     }
     return directories;
 }
-// Defer parseArguments() call to runServer() to avoid module-level side effects
-let directoryArgs = [];
+const directoryArgs = parseArguments();
 // Async initialization function to be called in runServer()
 async function initializeDirectories() {
-    // Use the already-determined MCP mode from top of file
-    // Console suppression is already done at module load time via _mcpMode
-    if (!_mcpMode &&
+    // 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 &&
         (approvedFoldersFromArgs.length > 0 ||
             directoryArgs.length > 0 ||
             ignoredFolders.length > 0 ||
@@ -265,7 +248,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 (!_mcpMode) {
+                    if (!isMCP) {
                         process.exit(1);
                     }
                 }
@@ -274,13 +257,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 (!_mcpMode) {
+                if (!isMCP) {
                     process.exit(1);
                 }
             }
         }));
         // Only log initialization if not running under MCP
-        if (!_mcpMode) {
+        if (!isMCP) {
             console.error(`Initialized with ${allowedDirectories.length} approved ${allowedDirectories.length === 1 ? "directory" : "directories"}`);
         }
     }
@@ -308,7 +291,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 (!_mcpMode) {
+                    if (!isMCP) {
                         process.exit(1);
                     }
                 }
@@ -317,7 +300,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 (!_mcpMode) {
+                if (!isMCP) {
                     process.exit(1);
                 }
             }
@@ -339,7 +322,7 @@ async function initializeDirectories() {
     // Priority 1: --approved-commands from CLI (supersedes .env)
     if (approvedCommandsFromArgs.length > 0) {
         finalApprovedCommands = approvedCommandsFromArgs;
-        if (!_mcpMode) {
+        if (!isMCP) {
             console.error(`  Approved commands (from CLI): ${finalApprovedCommands.join(", ")}`);
         }
     }
@@ -347,19 +330,18 @@ async function initializeDirectories() {
         // Priority 2: Load from .env file
         try {
             const envPath = path.join(process.cwd(), ".env");
-            // CRITICAL: quiet: true prevents dotenv from logging to stdout which corrupts MCP JSON-RPC
-            dotenv.config({ path: envPath, quiet: true });
+            dotenv.config({ path: envPath });
             if (process.env.APPROVED_COMMANDS) {
                 finalApprovedCommands = process.env.APPROVED_COMMANDS.split(",")
                     .map((c) => c.trim())
                     .filter((c) => c.length > 0);
-                if (!_mcpMode) {
+                if (!isMCP) {
                     console.error(`  Approved commands (from .env): ${finalApprovedCommands.join(", ")}`);
                 }
             }
         }
         catch (error) {
-            if (!_mcpMode) {
+            if (!isMCP) {
                 console.error("  Note: Could not load .env file (this is okay if using CLI args)");
             }
         }
@@ -367,14 +349,14 @@ async function initializeDirectories() {
     // Initialize shell tool with approved commands
     if (finalApprovedCommands.length > 0) {
         initializeShellTool(finalApprovedCommands);
-        if (!_mcpMode) {
+        if (!isMCP) {
             console.error(`Initialized shell tool with ${finalApprovedCommands.length} approved command(s)`);
         }
     }
     else {
         initializeShellTool([]);
         // Only log shell initialization if not running under MCP
-        if (!_mcpMode) {
+        if (!isMCP) {
             console.error("Shell tool initialized with no pre-approved commands (all commands require approval)");
         }
     }
@@ -400,21 +382,21 @@ const server = new Server({
         tools: {
             listChanged: true,
         },
+        resources: {},
+        prompts: {},
     },
 });
 // Initialize handler - required for MCP protocol
 server.setRequestHandler(InitializeRequestSchema, async (request) => {
     const clientCapabilities = request.params.capabilities;
     return {
-        // CRITICAL FIX: Use 2025-06-18 for Claude Desktop compatibility
-        // Claude Desktop (as of 2026-01-21) only supports protocol version 2025-06-18
-        // Using LATEST_PROTOCOL_VERSION (2025-11-25) causes immediate disconnection
-        // See: RCA-Claude-Desktop-MCP-Connection-Failure-2026-01-21.md
-        protocolVersion: "2025-06-18",
+        protocolVersion: LATEST_PROTOCOL_VERSION,
         capabilities: {
             tools: {
                 listChanged: true,
             },
+            resources: {},
+            prompts: {},
         },
         serverInfo: {
             name: "vulcan-file-ops",
@@ -643,8 +625,6 @@ 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 {
@@ -653,7 +633,9 @@ export async function runServer() {
     catch (error) {
         // In MCP mode, don't crash the server on init errors
         // Just continue with empty configuration
-        if (!_mcpMode) {
+        const isMCP = (!process.stdin.isTTY && !process.stdout.isTTY) ||
+            process.argv.some((arg) => arg.includes("mcp") || arg.includes("stdio"));
+        if (!isMCP) {
             // 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.6",
+  "version": "1.2.8",
   "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",
@@ -65,7 +65,7 @@
     "lint": "npm run lint:json"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.25.2",
+    "@modelcontextprotocol/sdk": "1.20.0",
     "@turbodocx/html-to-docx": "^1.16.0",
     "diff": "^8.0.3",
     "docx": "^9.5.1",