codex-review-mcp 2.3.8 → 2.4.0

package/dist/mcp-server.js

@@ -86,11 +86,11 @@ server.registerTool('perform_code_review', {
             maxTokens: input.maxTokens,
         };
         const onProgress = async (message, progress, total) => {
-            // Only send progress notifications if we have an explicit progress token
-            // Note: Do NOT fall back to requestId - it's a different thing in MCP protocol
+            // Only send progress notifications if we have a valid string token (like toolu_xxx)
+            // Numeric tokens cause "unknown token" errors in the MCP client
             const progressToken = extra?._meta?.progressToken;
-            if (progressToken === undefined || progressToken === null) {
-                // Skip progress notifications if no explicit token provided
+            if (!progressToken || typeof progressToken !== 'string') {
+                // Skip progress notifications if no valid string token
                 return;
             }
             await server.server.notification({

package/dist/mcp-server.test.js

@@ -50,11 +50,11 @@ describe('MCP Server Integration Tests', () => {
             },
         }, async (input, extra) => {
             const onProgress = async (message, progress, total) => {
-                // Only send progress notifications if we have an explicit progress token
-                // Note: Do NOT fall back to requestId - it's a different thing in MCP protocol
+                // Only send progress notifications if we have a valid string token (like toolu_xxx)
+                // Numeric tokens cause "unknown token" errors in the MCP client
                 const progressToken = extra?._meta?.progressToken;
-                if (progressToken === undefined || progressToken === null) {
-                    // Skip progress notifications if no explicit token provided
+                if (!progressToken || typeof progressToken !== 'string') {
+                    // Skip progress notifications if no valid string token
                     return;
                 }
                 await server.server.notification({

package/dist/tools/performCodeReview.js

@@ -37,16 +37,20 @@ export async function performCodeReview(input, onProgress) {
         : shouldSkip
             ? ''
             : await gatherContext(input.workspaceDir);
-    // Debug: Log what context was gathered (to stderr for MCP logs)
-    console.error(`[codex-review-mcp] Context gathered: ${context.length} chars`);
+    // Log context gathering results (informational, not errors!)
     const hasCursorRules = context.includes('.cursor/rules');
-    console.error(`[codex-review-mcp] Contains .cursor/rules: ${hasCursorRules}`);
     if (hasCursorRules) {
         const cursorFiles = (context.match(/<!-- \.cursor\/rules\/[^>]+ -->/g) || []);
-        console.error(`[codex-review-mcp] Cursor rules files: ${cursorFiles.join(', ')}`);
+        console.error(`[codex-review-mcp] Context: ${context.length} chars with .cursor/rules → ${cursorFiles.map(f => f.replace('<!-- ', '').replace(' -->', '')).join(', ')}`);
+    }
+    else if (context.length > 0) {
+        console.error(`[codex-review-mcp] Context: ${context.length} chars from package.json, tsconfig.json, etc. (no .cursor/rules in repo)`);
+    }
+    else {
+        console.error(`[codex-review-mcp] No project context found (generic review mode)`);
     }
     // Also use debugLog for persistent logging
-    await debugLog(`Context gathered: ${context.length} chars, .cursor/rules: ${hasCursorRules}`);
+    await debugLog(`Context: ${context.length} chars, .cursor/rules: ${hasCursorRules}`);
     // Build expert prompt
     await onProgress?.('Building expert prompt…', 50, 100);
     const isStaticReview = input.contentType === 'code';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-review-mcp",
-  "version": "2.3.8",
+  "version": "2.4.0",
   "main": "index.js",
   "scripts": {
     "build": "tsc",