@j0hanz/filesystem-mcp 1.6.1 → 1.7.0

package/dist/tools/task-support.js

@@ -134,6 +134,10 @@ function normalizeCallToolResult(value) {
 function getToolResultErrorCode(result) {
     if (!isRecord(result) || result['isError'] !== true)
         return undefined;
+    // First check for a dedicated errorCode property to avoid regex parsing of the content for structured error results produced by newer code.
+    if (typeof result['errorCode'] === 'string')
+        return result['errorCode'];
+    // Fallback to regex parsing of the human-readable error message for older error results that lack a structured errorCode property.
     const { content } = result;
     if (!Array.isArray(content) || content.length === 0)
         return undefined;
@@ -293,6 +297,21 @@ async function runTaskInBackground(run, args, extra, taskStore, taskId, toolName
         }
         catch (innerError) {
             console.error(`Failed to store task failure result for task ${taskId}:`, innerError);
+            // If storing the failure result also fails, there's not much we can do. The task will remain in 'working' status until it expires, which is not ideal but at least prevents clients from receiving incorrect results or hanging indefinitely waiting for a result that will never arrive. We log the error to aid debugging, and we attempt to notify the client of the failure if possible, but we don't want to throw further errors that could crash the server or cause cascading failures.
+            const syntheticTask = {
+                taskId,
+                status: 'failed',
+                ttl: null,
+                createdAt: new Date().toISOString(),
+                lastUpdatedAt: new Date().toISOString(),
+            };
+            const { sendNotification } = extra;
+            if (typeof sendNotification === 'function') {
+                void sendNotification({
+                    method: TASK_STATUS_NOTIFICATION_METHOD,
+                    params: buildTaskStatusNotificationParams(syntheticTask),
+                });
+            }
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/filesystem-mcp",
-  "version": "1.6.1",
+  "version": "1.7.0",
   "mcpName": "io.github.j0hanz/filesystem-mcp",
   "description": "MCP Server that enables LLMs to interact with the local filesystem.",
   "type": "module",
@@ -37,8 +37,6 @@
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "test": "node scripts/tasks.mjs test",
-    "test:fast": "node --test --import tsx/esm src/__tests__/**/*.test.ts node-tests/**/*.test.ts",
-    "test:coverage": "node scripts/tasks.mjs test --coverage",
     "knip": "knip",
     "knip:fix": "knip --fix",
     "inspector": "npm run build && npx -y @modelcontextprotocol/inspector node dist/index.js ${workspaceFolder}",