@j0hanz/code-review-analyst-mcp 1.5.2 → 1.5.3

package/dist/lib/tool-factory.js

@@ -136,15 +136,22 @@ function isRetryableUpstreamMessage(message) {
     return (RETRYABLE_UPSTREAM_ERROR_PATTERN.test(message) ||
         BUSY_ERROR_PATTERN.test(message));
 }
-function ignoreProgressInput(value) {
-    if (value === null) {
-        return;
-    }
-}
 function sendTaskProgress(extra, payload) {
-    ignoreProgressInput(extra);
-    ignoreProgressInput(payload);
-    return Promise.resolve();
+    const rawToken = extra._meta?.progressToken;
+    if (typeof rawToken !== 'string' && typeof rawToken !== 'number') {
+        return Promise.resolve();
+    }
+    const params = {
+        progressToken: rawToken,
+        progress: payload.current,
+        ...(payload.total !== undefined ? { total: payload.total } : {}),
+        ...(payload.message !== undefined ? { message: payload.message } : {}),
+    };
+    return extra
+        .sendNotification({ method: 'notifications/progress', params })
+        .catch(() => {
+        // Progress notifications are best-effort; never fail tool execution.
+    });
 }
 function createProgressReporter(extra) {
     let lastCurrent = 0;
@@ -349,16 +356,19 @@ export function registerStructuredToolTask(server, config) {
                         // statusMessage is best-effort; task may already be terminal.
                     }
                 };
+                const onLog = createGeminiLogger(server, task.taskId);
                 const storeResultSafely = async (status, result) => {
                     try {
                         await extra.taskStore.storeTaskResult(task.taskId, status, result);
                     }
-                    catch {
-                        // storing the result failed, possibly because the task is already marked as failed due to an uncaught error. There's not much we can do at this point, so we swallow the error to avoid unhandled rejections.
+                    catch (storeErr) {
+                        await onLog('error', {
+                            event: 'store_result_failed',
+                            error: getErrorMessage(storeErr),
+                        });
                     }
                 };
                 try {
-                    const onLog = createGeminiLogger(server, task.taskId);
                     const inputRecord = parseToolInput(input, config.fullInputSchema);
                     progressContext = normalizeProgressContext(config.progressContext?.(inputRecord));
                     // Snapshot the diff slot ONCE before any async work so that
@@ -438,9 +448,18 @@ export function registerStructuredToolTask(server, config) {
                     await reportProgressCompletionUpdate(reportProgress, config.name, progressContext, outcome);
                 }
             };
-            queueMicrotask(() => {
-                void runTask().catch((error) => {
-                    console.error(`[task-runner:${config.name}] ${getErrorMessage(error)}`);
+            setImmediate(() => {
+                void runTask().catch(async (error) => {
+                    try {
+                        await server.sendLoggingMessage({
+                            level: 'error',
+                            logger: 'task-runner',
+                            data: { task: config.name, error: getErrorMessage(error) },
+                        });
+                    }
+                    catch {
+                        console.error(`[task-runner:${config.name}] ${getErrorMessage(error)}`);
+                    }
                 });
             });
             return { task };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@j0hanz/code-review-analyst-mcp",
-  "version": "1.5.2",
+  "version": "1.5.3",
   "mcpName": "io.github.j0hanz/code-review-analyst",
   "description": "Gemini-powered MCP server for code review analysis.",
   "type": "module",