codex-review-mcp 2.1.1 → 2.2.1

package/dist/mcp-server.js

@@ -22,7 +22,8 @@ server.registerTool('get_version', {
                 type: 'text',
                 text: `codex-review-mcp version ${VERSION}`,
                 mimeType: 'text/plain'
-            }]
+            }],
+        _meta: { version: VERSION }
     };
 });
 server.registerTool('perform_code_review', {
@@ -93,19 +94,55 @@ server.registerTool('perform_code_review', {
             }, extra?.requestId ? { relatedRequestId: extra.requestId } : undefined);
         };
         const markdown = await performCodeReview(reviewInput, onProgress);
-        return { content: [{ type: 'text', text: markdown, mimeType: 'text/markdown' }] };
+        return {
+            content: [{ type: 'text', text: markdown, mimeType: 'text/markdown' }],
+            _meta: { version: VERSION }
+        };
     }
     catch (error) {
         const errorMessage = error?.message || String(error);
+        const errorStack = error?.stack || '';
+        // Log detailed error to stderr for MCP logs
+        console.error(`[codex-review-mcp v${VERSION}] Tool error:`, errorMessage);
+        if (errorStack) {
+            console.error('[codex-review-mcp] Stack trace:', errorStack);
+        }
         return {
             content: [{
                     type: 'text',
-                    text: `❌ Code Review Failed\n\n${errorMessage}`,
+                    text: `❌ Code Review Failed\n\n**Error:** ${errorMessage}\n\n**Version:** ${VERSION}\n\nCheck MCP server logs for details.`,
                     mimeType: 'text/markdown'
                 }],
-            isError: true
+            isError: true,
+            _meta: { version: VERSION, error: errorMessage }
         };
     }
 });
+// Global error handlers to prevent server crashes and inform users
+process.on('unhandledRejection', (reason, promise) => {
+    const errorMsg = reason instanceof Error ? reason.message : String(reason);
+    const errorStack = reason instanceof Error ? reason.stack : '';
+    console.error(`[codex-review-mcp v${VERSION}] ⚠️  UNHANDLED REJECTION`);
+    console.error(`[codex-review-mcp] Reason:`, errorMsg);
+    if (errorStack) {
+        console.error(`[codex-review-mcp] Stack:`, errorStack);
+    }
+    console.error(`[codex-review-mcp] Promise:`, promise);
+    console.error(`[codex-review-mcp] Server will continue running, but this indicates a bug that should be fixed.`);
+    // Don't exit - let the server continue running
+});
+process.on('uncaughtException', (error) => {
+    console.error(`[codex-review-mcp v${VERSION}] ⚠️  UNCAUGHT EXCEPTION`);
+    console.error(`[codex-review-mcp] Error:`, error.message);
+    console.error(`[codex-review-mcp] Stack:`, error.stack);
+    console.error(`[codex-review-mcp] Server will continue running, but this indicates a serious bug.`);
+    // Don't exit - let the server continue running
+});
+// Log startup information
+console.error(`[codex-review-mcp v${VERSION}] Starting server...`);
+console.error(`[codex-review-mcp] Node version: ${process.version}`);
+console.error(`[codex-review-mcp] Platform: ${process.platform}`);
 const transport = new StdioServerTransport();
 await server.connect(transport);
+console.error(`[codex-review-mcp v${VERSION}] ✅ Server started successfully`);
+console.error(`[codex-review-mcp] Ready to accept tool requests`);

package/dist/review/buildPrompt.js

@@ -33,6 +33,7 @@ export function buildPrompt({ diffText, context, focus, version, isStaticReview
         '- Do NOT suggest changes that deviate from the project\'s established patterns or introduce external patterns',
         '- When project guidelines conflict with general best practices, prioritize the project guidelines',
         '- For accessibility audits: apply axe-core best practices and WCAG 2.1 Level AA guidelines',
+        '- When reviewing accessibility audit results or configs: VERIFY the audit tool is configured for WCAG 2.1 Level AA only. FLAG if audits are misconfigured (using WCAG 2.2, Level AAA, or only Level A tags) as these will report violations that don\'t match the Axe browser extension defaults or project compliance requirements. Correct axe-core tags should be: ["wcag2a", "wcag2aa", "wcag21a", "wcag21aa"].',
         '- CRITICAL Z-INDEX WARNING: Be extremely cautious with z-index changes on modals, overlays, or components. Analyze the entire z-index hierarchy before suggesting changes. Flag any z-index modifications as HIGH RISK and verify they won\'t break stacking context upstream or downstream. Always check for existing z-index patterns in the codebase first.',
         '- MATERIAL-UI/MUI SPACING: Use theme.spacing() for all margin, padding, and gap values instead of hardcoded px/rem/em values to avoid magic numbers (e.g., gap: theme.spacing(3) for 24px). Flag any hardcoded spacing values.',
         focusLine,

package/package.json

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