@mrxkun/mcfast-mcp 3.5.3 → 3.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mrxkun/mcfast-mcp",
-  "version": "3.5.3",
+  "version": "3.5.5",
   "description": "Ultra-fast code editing with WASM acceleration, fuzzy patching, multi-layer caching, and 8 unified tools. Optimized for AI code assistants with 80-98% latency reduction.",
   "type": "module",
   "bin": {

package/src/index.js

@@ -203,7 +203,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             // CORE TOOL 1: edit (consolidates apply_fast + edit_file + apply_search_replace)
             {
                 name: "edit",
-                description: "**PRIMARY TOOL FOR EDITING FILES** - Intelligent auto-switching strategies: (1) Search & Replace (Fastest) - use 'Replace X with Y', (2) Placeholder (Efficient) - use '// ... existing code ...', (3) Mercury AI (Intelligent) - for complex refactoring. Includes Auto-Rollback for syntax errors.",
+                description: "**PRIMARY TOOL FOR EDITING FILES** - Intelligent auto-switching strategies: (1) Search & Replace (Fastest) - use 'Replace X with Y', (2) Placeholder (Efficient) - use '// ... existing code ...' placeholders to save tokens.", (3) Mercury AI (Intelligent) - for complex refactoring. Includes Auto-Rollback for syntax errors.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -514,6 +514,17 @@ async function reportAudit(data) {
     try {
         // Only report if dashboard logging is enabled
         if (process.env.MCFAST_ENABLE_AUDIT !== 'true') {
+            if (VERBOSE) {
+                console.error(`${colors.dim}[Audit] Skipped - MCFAST_ENABLE_AUDIT not set to 'true'${colors.reset}`);
+            }
+            return;
+        }
+
+        // Check if we have a token
+        if (!TOKEN) {
+            if (VERBOSE) {
+                console.error(`${colors.yellow}[Audit] Skipped - No MCFAST_TOKEN provided${colors.reset}`);
+            }
             return;
         }
 
@@ -533,33 +544,46 @@ async function reportAudit(data) {
         };
 
         // Send to dashboard API
-        const DASHBOARD_API_URL = process.env.MCFAST_DASHBOARD_URL || 'https://mcfast.vercel.app/api/v1/audit';
+        const DASHBOARD_API_URL = process.env.MCFAST_DASHBOARD_URL || 'https://mcfast.vercel.app/api/v1/logs/audit';
         
-        if (TOKEN) {
-            fetch(DASHBOARD_API_URL, {
-                method: 'POST',
-                headers: {
-                    'Content-Type': 'application/json',
-                    'Authorization': `Bearer ${TOKEN}`
-                },
-                body: JSON.stringify(auditData)
-            }).catch(err => {
-                // Silent fail - don't break operations if audit fails
-                if (VERBOSE) {
-                    console.error(`${colors.dim}Audit log failed (non-critical):${colors.reset}`, err.message);
-                }
-            });
+        if (VERBOSE) {
+            console.error(`${colors.dim}[Audit] Sending to: ${DASHBOARD_API_URL}${colors.reset}`);
+            console.error(`${colors.dim}[Audit] Tool: ${auditData.tool}, Status: ${auditData.status}${colors.reset}`);
+        }
+        
+        const response = await fetch(DASHBOARD_API_URL, {
+            method: 'POST',
+            headers: {
+                'Content-Type': 'application/json',
+                'Authorization': `Bearer ${TOKEN}`
+            },
+            body: JSON.stringify(auditData)
+        });
+
+        if (!response.ok) {
+            const errorText = await response.text().catch(() => 'Unknown error');
+            if (VERBOSE) {
+                console.error(`${colors.yellow}[Audit] Failed - HTTP ${response.status}: ${errorText}${colors.reset}`);
+            }
+        } else {
+            if (VERBOSE) {
+                console.error(`${colors.green}[Audit] Logged successfully ✓${colors.reset}`);
+            }
         }
 
         // Also log to local file if configured
         if (process.env.MCFAST_AUDIT_FILE) {
             const logLine = JSON.stringify(auditData) + '\n';
-            await fs.appendFile(process.env.MCFAST_AUDIT_FILE, logLine).catch(() => {});
+            await fs.appendFile(process.env.MCFAST_AUDIT_FILE, logLine).catch((err) => {
+                if (VERBOSE) {
+                    console.error(`${colors.yellow}[Audit] Failed to write to file:${colors.reset}`, err.message);
+                }
+            });
         }
     } catch (error) {
         // Silent fail - audit should never break the tool
         if (VERBOSE) {
-            console.error(`${colors.dim}Audit error (non-critical):${colors.reset}`, error.message);
+            console.error(`${colors.yellow}[Audit] Error (non-critical):${colors.reset}`, error.message);
         }
     }
 }
@@ -1192,6 +1216,7 @@ async function handleReadFileInternal({ path: filePath, start_line, end_line, ma
     let endLine = end_line ? parseInt(end_line) : -1;
     let outputContent;
     let totalLines;
+    let lineRangeInfo = '';
 
     if ((stats.size > STREAM_THRESHOLD && (start_line || end_line)) || stats.size > 10 * 1024 * 1024) {
         const { Readable } = await import('stream');
@@ -1590,6 +1615,7 @@ async function handleReadFile({ path: filePath, start_line, end_line }) {
         let endLine = end_line ? parseInt(end_line) : -1;
         let outputContent;
         let totalLines;
+        let lineRangeInfo = '';
 
         if ((stats.size > STREAM_THRESHOLD && (start_line || end_line)) || stats.size > 10 * 1024 * 1024) {
             const { Readable } = await import('stream');
@@ -1817,6 +1843,13 @@ async function handleSearchCodeAI({ query, files, contextLines = 2 }) {
 async function handleGetDefinition({ path: filePath, symbol }) {
     if (!filePath || !symbol) throw new Error("Missing path or symbol");
 
+    // Check if path is a directory
+    const absolutePath = path.resolve(filePath);
+    const stats = await fs.stat(absolutePath);
+    if (!stats.isFile()) {
+        throw new Error(`Path is not a file: ${filePath}`);
+    }
+
     // Read file content first
     const content = await fs.readFile(filePath, 'utf8');
     const definitions = await findDefinition(content, filePath, symbol);
@@ -1837,6 +1870,13 @@ async function handleGetDefinition({ path: filePath, symbol }) {
 async function handleFindReferences({ path: filePath, symbol }) {
     if (!filePath || !symbol) throw new Error("Missing path or symbol");
 
+    // Check if path is a directory
+    const absolutePath = path.resolve(filePath);
+    const stats = await fs.stat(absolutePath);
+    if (!stats.isFile()) {
+        throw new Error(`Path is not a file: ${filePath}`);
+    }
+
     const content = await fs.readFile(filePath, 'utf8');
     const references = await findReferences(content, filePath, symbol);