codex-review-mcp 2.2.0 → 2.3.0

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,28 +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
+// Global error handlers to prevent server crashes and inform users
 process.on('unhandledRejection', (reason, promise) => {
-    console.error('[codex-review-mcp] Unhandled Rejection at:', promise, 'reason:', reason);
+    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] Uncaught Exception:', 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

@@ -5,44 +5,26 @@ export function buildPrompt({ diffText, context, focus, version, isStaticReview
     return [
         versionLine,
         '',
-        'You are an expert AI code reviewer. Be concise, specific, and actionable.',
-        'Prefer minimal diffs and direct fixes.',
+        'You are an expert React and GraphQL code reviewer. Be concise, specific, and actionable.',
         '',
-        'ENVIRONMENT ASSUMPTION: The runtime and tooling are POSIX (macOS/Linux). Do NOT suggest Windows/PowerShell/cmd-specific commands or paths.',
+        'PRIORITY HIERARCHY:',
+        '1. Project documentation (.cursor/rules/*, CODE_REVIEW.md, CONTRIBUTING.md)',
+        '2. Config files (tsconfig.json, .eslintrc, prettier)',
+        '3. Existing patterns in the same directory/module',
+        '4. General best practices (only when not conflicting above)',
         '',
-        '🚨 CRITICAL REPO-SPECIFIC GUIDANCE HIERARCHY:',
-        '1. HIGHEST PRIORITY: Project documentation (.cursor/rules/*, CODE_REVIEW.md, CONTRIBUTING.md, etc.)',
-        '2. Configuration files (tsconfig.json, .eslintrc, prettier config, etc.)',
-        '3. Existing patterns in similar files within the same directory',
-        '4. General best practices (ONLY when not conflicting with above)',
-        '',
-        'MANDATORY REVIEW REQUIREMENTS:',
-        '- 🔍 EXAMINE existing code patterns in the same directory/module before suggesting changes',
-        '- 📚 CHECK for existing utility functions, abstractions, and helpers before creating new ones',
-        '- 🎯 FOLLOW the exact naming conventions, file structure, and import patterns used in this repo',
-        '- 🧪 MATCH existing test patterns and testing approaches when reviewing test files',
-        '- ⚙️ VALIDATE against linting and formatting rules defined in the project config',
-        '- 🚫 NEVER suggest patterns that are inconsistent with the existing codebase',
-        '- 🚫 NEVER introduce external libraries/patterns not already in use',
-        '- ✅ PREFER reusing existing abstractions over creating new ones',
-        '',
-        'SPECIFIC TECHNICAL GUIDELINES:',
-        '- Follow ALL guidelines, conventions, and architectural patterns defined in the project documentation below',
-        '- Maintain consistency with the existing codebase styling, naming conventions, and structural themes',
-        '- Base your recommendations on the internal documentation, .cursor/rules files, and configuration files provided',
-        '- 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.',
+        'CORE PRINCIPLES:',
+        '- Match existing patterns, naming, structure, and imports in this codebase',
+        '- Reuse existing utilities and abstractions; never introduce new ones unnecessarily',
+        '- Never suggest external libraries or patterns not already in use',
+        '- Validate against project linting and formatting rules',
+        '- When project guidelines conflict with best practices, follow the project',
         focusLine,
         '',
-        'BEFORE making ANY suggestion, ask yourself:',
-        '1. Does this pattern already exist in the codebase? (Check similar files)',
-        '2. Is there an existing utility/helper that could be reused?',
-        '3. Does this follow the exact conventions shown in the project documentation?',
-        '4. Would this change maintain consistency with the surrounding code?',
+        'TECHNICAL REQUIREMENTS:',
+        '- Accessibility: WCAG 2.1 Level AA with tags ["wcag2a", "wcag2aa", "wcag21a", "wcag21aa"]. Flag misconfigured audits using WCAG 2.2, AAA, or only Level A.',
+        '- Z-index: Flag as HIGH RISK. Check entire stacking context before suggesting changes.',
+        '- MUI spacing: Use theme.spacing() for margins/padding/gaps. Flag hardcoded px/rem/em values.',
         '',
         `\n---\n${reviewType}${isStaticReview ? ' - Code Files' : ' (unified=0)'}:\n`,
         diffText,

package/dist/review/buildPrompt.test.js

@@ -61,10 +61,10 @@ describe('buildPrompt', () => {
             diffText: mockDiff,
         });
         // Verify key sections are present
-        expect(prompt).toContain('You are an expert AI code reviewer');
-        expect(prompt).toContain('CRITICAL REPO-SPECIFIC GUIDANCE HIERARCHY');
-        expect(prompt).toContain('MANDATORY REVIEW REQUIREMENTS');
-        expect(prompt).toContain('SPECIFIC TECHNICAL GUIDELINES');
+        expect(prompt).toContain('You are an expert React and GraphQL code reviewer');
+        expect(prompt).toContain('PRIORITY HIERARCHY');
+        expect(prompt).toContain('CORE PRINCIPLES');
+        expect(prompt).toContain('TECHNICAL REQUIREMENTS');
         expect(prompt).toContain('Output strictly as Markdown with the following sections');
     });
     it('should include the diff text', () => {
@@ -87,27 +87,20 @@ describe('buildPrompt', () => {
         expect(prompt).toContain('TypeScript best practices');
         expect(prompt).toContain(mockDiff);
     });
-    it('should include POSIX environment assumption', () => {
-        const prompt = buildPrompt({
-            diffText: mockDiff,
-        });
-        expect(prompt).toContain('ENVIRONMENT ASSUMPTION: The runtime and tooling are POSIX (macOS/Linux)');
-        expect(prompt).toContain('Do NOT suggest Windows/PowerShell/cmd-specific commands');
-    });
     it('should include z-index warning', () => {
         const prompt = buildPrompt({
             diffText: mockDiff,
         });
-        expect(prompt).toContain('CRITICAL Z-INDEX WARNING');
-        expect(prompt).toContain('z-index changes on modals, overlays, or components');
+        expect(prompt).toContain('Z-index: Flag as HIGH RISK');
+        expect(prompt).toContain('Check entire stacking context');
     });
     it('should include guidance hierarchy with proper priorities', () => {
         const prompt = buildPrompt({
             diffText: mockDiff,
         });
-        expect(prompt).toContain('1. HIGHEST PRIORITY: Project documentation');
-        expect(prompt).toContain('2. Configuration files');
-        expect(prompt).toContain('3. Existing patterns in similar files');
+        expect(prompt).toContain('1. Project documentation');
+        expect(prompt).toContain('2. Config files');
+        expect(prompt).toContain('3. Existing patterns in the same directory/module');
         expect(prompt).toContain('4. General best practices');
     });
 });

package/package.json

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