@j0hanz/fetch-url-mcp 1.11.4 → 1.11.6

package/dist/tasks/handlers.js

@@ -91,15 +91,14 @@ export function registerTaskHandlers(server, options) {
             throwTaskNotFound();
         try {
             if (task.status === 'failed') {
-                if (task.error) {
-                    throw new McpError(task.error.code, task.error.message, task.error.data);
-                }
                 const failedResult = (task.result ?? null);
                 const fallback = failedResult ?? {
                     content: [
                         {
                             type: 'text',
-                            text: task.statusMessage ?? 'Task execution failed',
+                            text: JSON.stringify({
+                                error: task.statusMessage ?? 'Task execution failed',
+                            }),
                         },
                     ],
                     isError: true,

package/dist/tasks/manager.js

@@ -35,9 +35,9 @@ function resolveNextTaskStatus(task, updates) {
     return nextStatus;
 }
 function normalizeTaskTtl(ttl) {
-    if (!Number.isFinite(ttl))
+    if (ttl === undefined || !Number.isFinite(ttl))
         return DEFAULT_TTL_MS;
-    return Math.max(MIN_TTL_MS, Math.min(Math.trunc(Number(ttl)), MAX_TTL_MS));
+    return Math.max(MIN_TTL_MS, Math.min(Math.trunc(ttl), MAX_TTL_MS));
 }
 class TaskManager {
     tasks = new Map();
@@ -218,7 +218,7 @@ class TaskManager {
         }
         const anchorIndex = validTasks.findIndex((t) => t.taskId === anchorTaskId);
         if (anchorIndex === -1) {
-            return [];
+            throw new McpError(ErrorCode.InvalidParams, 'Invalid cursor');
         }
         return validTasks.slice(anchorIndex + 1, anchorIndex + 1 + pageSize + 1);
     }

package/dist/tasks/waiters.js

@@ -89,7 +89,7 @@ export async function waitForTerminalTask(options) {
     };
     if (options.signal?.aborted) {
         onAbort();
-        return;
+        return promise;
     }
     options.registry.add(options.taskId, waiter);
     if (options.signal) {

package/dist/tools/fetch-url.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fetch-url.d.ts","sourceRoot":"","sources":["../../src/tools/fetch-url.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AACzE,OAAO,KAAK,EACV,YAAY,EAEb,MAAM,oCAAoC,CAAC;AAE5C,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oCAAoC,CAAC;AACvE,OAAO,KAAK,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAc7B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAC;AAEjE,OAAO,EAEL,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,EACtB,MAAM,uBAAuB,CAAC;AAU/B,OAAO,EACL,mBAAmB,EAIpB,MAAM,eAAe,CAAC;AAEvB,OAAO,EAIL,KAAK,sBAAsB,EAE5B,MAAM,sBAAsB,CAAC;AAE9B,KAAK,aAAa,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,mBAAmB,CAAC,CAAC;AAEzD,UAAU,gBAAgB;IACxB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;IACvB,OAAO,EAAE,YAAY,EAAE,CAAC;IACxB,iBAAiB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,CAAC;IACxD,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED,eAAO,MAAM,mBAAmB,cAAc,CAAC;AAwI/C,wBAAgB,+BAA+B,CAC7C,MAAM,EAAE,YAAY,GACnB,MAAM,GAAG,SAAS,CAUpB;AAED,qBAAa,oBAAoB;IAI7B,OAAO,CAAC,QAAQ,CAAC,QAAQ;IACzB,OAAO,CAAC,QAAQ,CAAC,OAAO;IAJ1B,OAAO,CAAC,WAAW,CAA0B;gBAG1B,QAAQ,EAAE,gBAAgB,EAC1B,OAAO,EAAE,MAAM;IAGlC,WAAW,IAAI,IAAI;IAInB,WAAW,CAAC,KAAK,EAAE,gBAAgB,GAAG,IAAI;IAO1C,aAAa,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI;IAIxC,aAAa,CAAC,SAAS,EAAE,OAAO,GAAG,IAAI;IAIvC,OAAO,CAAC,iBAAiB;IAYzB,OAAO,CAAC,QAAQ;CAmCjB;AAuED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,aAAa,EACpB,KAAK,CAAC,EAAE,gBAAgB,GACvB,OAAO,CAAC,gBAAgB,CAAC,CAQ3B;AAqBD,MAAM,WAAW,wBAAwB;IACvC,cAAc,EAAE,CAAC,OAAO,EAAE,sBAAsB,KAAK,IAAI,CAAC;CAC3D;AAqBD,wBAAgB,aAAa,CAAC,MAAM,EAAE,SAAS,GAAG,wBAAwB,CAsCzE"}
+{"version":3,"file":"fetch-url.d.ts","sourceRoot":"","sources":["../../src/tools/fetch-url.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AACzE,OAAO,KAAK,EACV,YAAY,EAEb,MAAM,oCAAoC,CAAC;AAE5C,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oCAAoC,CAAC;AACvE,OAAO,KAAK,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAc7B,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAC;AACjE,OAAO,EAGL,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,EACtB,MAAM,uBAAuB,CAAC;AAU/B,OAAO,EACL,mBAAmB,EAIpB,MAAM,eAAe,CAAC;AAEvB,OAAO,EAIL,KAAK,sBAAsB,EAE5B,MAAM,sBAAsB,CAAC;AAE9B,KAAK,aAAa,GAAG,CAAC,CAAC,KAAK,CAAC,OAAO,mBAAmB,CAAC,CAAC;AAEzD,UAAU,gBAAgB;IACxB,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC;IACvB,OAAO,EAAE,YAAY,EAAE,CAAC;IACxB,iBAAiB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,CAAC;IACxD,OAAO,CAAC,EAAE,OAAO,CAAC;CACnB;AAED,eAAO,MAAM,mBAAmB,cAAc,CAAC;AAwI/C,wBAAgB,+BAA+B,CAC7C,MAAM,EAAE,YAAY,GACnB,MAAM,GAAG,SAAS,CAUpB;AAED,qBAAa,oBAAoB;IAI7B,OAAO,CAAC,QAAQ,CAAC,QAAQ;IACzB,OAAO,CAAC,QAAQ,CAAC,OAAO;IAJ1B,OAAO,CAAC,WAAW,CAA0B;gBAG1B,QAAQ,EAAE,gBAAgB,EAC1B,OAAO,EAAE,MAAM;IAGlC,WAAW,IAAI,IAAI;IAInB,WAAW,CAAC,KAAK,EAAE,gBAAgB,GAAG,IAAI;IAO1C,aAAa,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI;IAIxC,aAAa,CAAC,SAAS,EAAE,OAAO,GAAG,IAAI;IAIvC,OAAO,CAAC,iBAAiB;IAYzB,OAAO,CAAC,QAAQ;CAmCjB;AAuED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,aAAa,EACpB,KAAK,CAAC,EAAE,gBAAgB,GACvB,OAAO,CAAC,gBAAgB,CAAC,CAQ3B;AAqBD,MAAM,WAAW,wBAAwB;IACvC,cAAc,EAAE,CAAC,OAAO,EAAE,sBAAsB,KAAK,IAAI,CAAC;CAC3D;AAqBD,wBAAgB,aAAa,CAAC,MAAM,EAAE,SAAS,GAAG,wBAAwB,CAoCzE"}

package/dist/tools/fetch-url.js

@@ -1,10 +1,8 @@
 import { ErrorCode, McpError } from '@modelcontextprotocol/sdk/types.js';
 import { config, logDebug, logError, logWarn } from '../lib/core.js';
 import { finalizeInlineMarkdown, markdownTransform, parseCachedMarkdownResult, performSharedFetch, serializeMarkdownResult, withSignal, } from '../lib/fetch-pipeline.js';
-import { handleToolError } from '../lib/mcp-interop.js';
-import { createProgressReporter, } from '../lib/mcp-interop.js';
-import { composeAbortSignal, isAbortError, parseUrlOrNull, toError, } from '../lib/utils.js';
-import { isObject } from '../lib/utils.js';
+import { createProgressReporter, handleToolError, } from '../lib/mcp-interop.js';
+import { composeAbortSignal, isAbortError, isObject, parseUrlOrNull, toError, } from '../lib/utils.js';
 import { formatZodError } from '../lib/zod.js';
 import { fetchUrlInputSchema, fetchUrlOutputSchema, normalizeExtractedMetadata, normalizePageTitle, } from '../schemas.js';
 import { withRequestContextIfMissing } from '../tasks/owner.js';
@@ -287,10 +285,9 @@ export function registerTools(server) {
         execution: { taskSupport: 'optional' },
         icons: [TOOL_ICON],
     }, withRequestContextIfMissing(TOOL_DEFINITION.handler));
-    const registeredToolRecord = registeredTool;
     const updateTaskSupport = (support) => {
         setTaskCapableToolSupport(FETCH_URL_TOOL_NAME, support);
-        registeredToolRecord.execution = { taskSupport: support };
+        registeredTool.execution = { taskSupport: support };
     };
     updateTaskSupport('optional');
     return { setTaskSupport: updateTaskSupport };

package/dist/transform/transform.js

@@ -1,8 +1,7 @@
 import diagnosticsChannel from 'node:diagnostics_channel';
 import { isProbablyReaderable, Readability } from '@mozilla/readability';
 import { parseHTML } from 'linkedom';
-import { config } from '../lib/core.js';
-import { getOperationId, getRequestId, logDebug, logError, logInfo, logWarn, redactUrl, } from '../lib/core.js';
+import { config, getOperationId, getRequestId, logDebug, logError, logInfo, logWarn, redactUrl, } from '../lib/core.js';
 import { isRawTextContentUrl } from '../lib/http.js';
 import { composeAbortSignal, FetchError, getErrorMessage, getUtf8ByteLength, isAsciiOnly, isObject, throwIfAborted, toError, trimDanglingTagFragment, truncateToUtf8Boundary, } from '../lib/utils.js';
 import { evaluateArticleContent, extractNoscriptImages, getVisibleTextLength, normalizeTabContent, prepareDocumentForMarkdown, removeNoiseFromHtml, serializeDocumentForMarkdown, stripDocsControls, stripScreenReaderText, surfaceCodeEditorContent, } from './dom-prep.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/fetch-url-mcp",
-  "version": "1.11.4",
+  "version": "1.11.6",
   "mcpName": "io.github.j0hanz/fetch-url-mcp",
   "description": "A web content fetcher MCP server that converts HTML to clean, AI and human readable markdown.",
   "type": "module",

package/dist/lib/task-handlers.d.ts

@@ -1,11 +0,0 @@
-import { type McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-export { cancelTasksForOwner, abortAllTaskExecutions, } from '../tasks/execution.js';
-interface TaskHandlerRegistrationOptions {
-    requireInterception?: boolean;
-}
-interface TaskHandlerRegistrationResult {
-    interceptedToolsCall: boolean;
-    taskCapableToolsRegistered: boolean;
-}
-export declare function registerTaskHandlers(server: McpServer, options?: TaskHandlerRegistrationOptions): TaskHandlerRegistrationResult;
-//# sourceMappingURL=task-handlers.d.ts.map

package/dist/lib/task-handlers.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"task-handlers.d.ts","sourceRoot":"","sources":["../../src/lib/task-handlers.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAkCzE,OAAO,EACL,mBAAmB,EACnB,sBAAsB,GACvB,MAAM,uBAAuB,CAAC;AA4D/B,UAAU,8BAA8B;IACtC,mBAAmB,CAAC,EAAE,OAAO,CAAC;CAC/B;AAED,UAAU,6BAA6B;IACrC,oBAAoB,EAAE,OAAO,CAAC;IAC9B,0BAA0B,EAAE,OAAO,CAAC;CACrC;AACD,wBAAgB,oBAAoB,CAClC,MAAM,EAAE,SAAS,EACjB,OAAO,CAAC,EAAE,8BAA8B,GACvC,6BAA6B,CA+J/B"}

package/dist/lib/task-handlers.js

@@ -1,151 +0,0 @@
-import { randomUUID } from 'node:crypto';
-import {} from '@modelcontextprotocol/sdk/server/mcp.js';
-import { CallToolRequestSchema, ErrorCode, McpError, } from '@modelcontextprotocol/sdk/types.js';
-import { z } from 'zod';
-import { parseExtendedCallToolRequest, withRelatedTaskMeta, } from '../tasks/call-contract.js';
-import { abortTaskExecution, emitTaskStatusNotification, handleToolCallRequest, throwTaskNotFound, toTaskSummary, } from '../tasks/execution.js';
-import { taskManager } from '../tasks/manager.js';
-import { isServerResult, parseHandlerExtra, resolveTaskOwnerKey, resolveToolCallContext, } from '../tasks/owner.js';
-import { hasRegisteredTaskCapableTools, hasTaskCapableTool, } from '../tasks/registry.js';
-import { logWarn, runWithRequestContext } from './core.js';
-import { getSdkCallToolHandler } from './mcp-interop.js';
-export { cancelTasksForOwner, abortAllTaskExecutions, } from '../tasks/execution.js';
-/* -------------------------------------------------------------------------------------------------
- * Task handler schemas and registration
- * ------------------------------------------------------------------------------------------------- */
-const TaskGetSchema = z.looseObject({
-    method: z.literal('tasks/get', 'Expected "tasks/get"'),
-    params: z.looseObject({ taskId: z.string('Expected string') }, 'Expected object'),
-}, 'Invalid request');
-const TaskListSchema = z.looseObject({
-    method: z.literal('tasks/list', 'Expected "tasks/list"'),
-    params: z
-        .looseObject({
-        cursor: z.string('Expected string').optional(),
-    }, 'Expected object')
-        .optional(),
-}, 'Invalid request');
-const TaskCancelSchema = z.looseObject({
-    method: z.literal('tasks/cancel', 'Expected "tasks/cancel"'),
-    params: z.looseObject({ taskId: z.string('Expected string') }, 'Expected object'),
-}, 'Invalid request');
-const TaskResultSchema = z.looseObject({
-    method: z.literal('tasks/result', 'Expected "tasks/result"'),
-    params: z.looseObject({ taskId: z.string('Expected string') }, 'Expected object'),
-}, 'Invalid request');
-function resolveOwnerScopedExtra(extra) {
-    const parsedExtra = parseHandlerExtra(extra);
-    return {
-        parsedExtra,
-        ownerKey: resolveTaskOwnerKey(parsedExtra),
-    };
-}
-export function registerTaskHandlers(server, options) {
-    const sdkCallToolHandler = getSdkCallToolHandler(server);
-    const taskCapableToolsRegistered = hasRegisteredTaskCapableTools();
-    const requireInterception = options?.requireInterception ?? true;
-    if (!sdkCallToolHandler) {
-        if (taskCapableToolsRegistered && requireInterception) {
-            throw new Error('Task-capable tools are registered but SDK tools/call interception is unavailable. Upgrade compatibility or disable strict interception with TASKS_REQUIRE_INTERCEPTION=false.');
-        }
-        logWarn('Task call interception disabled: SDK tools/call handler unavailable; task-capable tools require MCP SDK compatibility update', { sdkVersion: 'unknown' });
-    }
-    if (sdkCallToolHandler) {
-        server.server.setRequestHandler(CallToolRequestSchema, async (request, extra) => {
-            const parsedExtra = parseHandlerExtra(extra);
-            const requestId = parsedExtra?.requestId !== undefined
-                ? String(parsedExtra.requestId)
-                : randomUUID();
-            return runWithRequestContext({
-                requestId,
-                operationId: requestId,
-                ...(parsedExtra?.sessionId
-                    ? { sessionId: parsedExtra.sessionId }
-                    : {}),
-            }, () => {
-                const toolName = request.params.name;
-                // Only intercept task-capable tools managed by the local task registry.
-                // Delegate all other tools to the SDK handler to avoid shadowing future tools.
-                if (!hasTaskCapableTool(toolName)) {
-                    return sdkCallToolHandler(request, extra);
-                }
-                const parsed = parseExtendedCallToolRequest(request);
-                const context = resolveToolCallContext(parsedExtra, parsed.params._meta);
-                return handleToolCallRequest(server, parsed, context);
-            });
-        });
-    }
-    server.server.setRequestHandler(TaskGetSchema, (request, extra) => {
-        const { taskId } = request.params;
-        const { ownerKey } = resolveOwnerScopedExtra(extra);
-        const task = taskManager.getTask(taskId, ownerKey);
-        if (!task)
-            throwTaskNotFound();
-        return toTaskSummary(task);
-    });
-    server.server.setRequestHandler(TaskResultSchema, async (request, extra) => {
-        const { taskId } = request.params;
-        const { parsedExtra, ownerKey } = resolveOwnerScopedExtra(extra);
-        const task = await taskManager.waitForTerminalTask(taskId, ownerKey, parsedExtra?.signal);
-        if (!task)
-            throwTaskNotFound();
-        try {
-            if (task.status === 'failed') {
-                if (task.error) {
-                    throw new McpError(task.error.code, task.error.message, task.error.data);
-                }
-                const failedResult = (task.result ?? null);
-                const fallback = failedResult ?? {
-                    content: [
-                        {
-                            type: 'text',
-                            text: task.statusMessage ?? 'Task execution failed',
-                        },
-                    ],
-                    isError: true,
-                };
-                return withRelatedTaskMeta(fallback, task.taskId);
-            }
-            if (task.status === 'cancelled') {
-                throw new McpError(ErrorCode.InvalidRequest, 'Task was cancelled', {
-                    taskId: task.taskId,
-                    status: 'cancelled',
-                    ...(task.statusMessage ? { statusMessage: task.statusMessage } : {}),
-                });
-            }
-            const result = isServerResult(task.result)
-                ? task.result
-                : { content: [] };
-            return withRelatedTaskMeta(result, task.taskId);
-        }
-        finally {
-            // Shrink TTL only after the result has been fully constructed and
-            // is about to be delivered — avoids premature expiry if result
-            // construction throws.
-            taskManager.shrinkTtlAfterDelivery(taskId);
-        }
-    });
-    server.server.setRequestHandler(TaskListSchema, (request, extra) => {
-        const { ownerKey } = resolveOwnerScopedExtra(extra);
-        const cursor = request.params?.cursor;
-        const { tasks, nextCursor } = taskManager.listTasks(cursor === undefined ? { ownerKey } : { ownerKey, cursor });
-        return {
-            tasks: tasks.map((task) => toTaskSummary(task)),
-            nextCursor,
-        };
-    });
-    server.server.setRequestHandler(TaskCancelSchema, (request, extra) => {
-        const { taskId } = request.params;
-        const { ownerKey } = resolveOwnerScopedExtra(extra);
-        const task = taskManager.cancelTask(taskId, ownerKey);
-        if (!task)
-            throwTaskNotFound();
-        abortTaskExecution(taskId);
-        emitTaskStatusNotification(server, task);
-        return toTaskSummary(task);
-    });
-    return {
-        interceptedToolsCall: sdkCallToolHandler !== null,
-        taskCapableToolsRegistered,
-    };
-}