codex-review-mcp 2.2.1 → 2.3.1

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/dist/review/formatOutput.js

@@ -1,10 +1,13 @@
-export function formatOutput(agentMarkdown) {
+export function formatOutput(agentMarkdown, version) {
     // For MVP, trust agent output if non-empty; otherwise return a minimal stub.
-    if (agentMarkdown && agentMarkdown.trim().length > 0)
-        return agentMarkdown;
+    const versionFooter = version ? `\n\n---\n*Generated by codex-review-mcp v${version}*` : '';
+    if (agentMarkdown && agentMarkdown.trim().length > 0) {
+        return agentMarkdown + versionFooter;
+    }
     return [
         '# Code Review',
         '',
         'No issues detected or empty output from agent.',
+        versionFooter,
     ].join('\n');
 }

package/dist/review/formatOutput.test.js

@@ -2,10 +2,18 @@ import { describe, it, expect } from 'vitest';
 import { formatOutput } from './formatOutput.js';
 describe('formatOutput', () => {
     describe('Valid Agent Output', () => {
-        it('should return agent markdown as-is when valid', () => {
+        it('should return agent markdown without version when version not provided', () => {
             const agentOutput = '# Code Review\n\n## Issues\n\n- Issue 1\n- Issue 2';
             const result = formatOutput(agentOutput);
             expect(result).toBe(agentOutput);
+            expect(result).not.toContain('Generated by codex-review-mcp');
+        });
+        it('should append version footer when version is provided', () => {
+            const agentOutput = '# Code Review\n\n## Issues\n\n- Issue 1';
+            const result = formatOutput(agentOutput, '2.3.0');
+            expect(result).toContain(agentOutput);
+            expect(result).toContain('---');
+            expect(result).toContain('*Generated by codex-review-mcp v2.3.0*');
         });
         it('should preserve formatting and special characters', () => {
             const agentOutput = `# Review
@@ -58,6 +66,12 @@ function fix() {
             expect(result).toContain('# Code Review');
             expect(result).toContain('No issues detected or empty output from agent');
         });
+        it('should include version in fallback message when provided', () => {
+            const result = formatOutput('', '2.3.0');
+            expect(result).toContain('# Code Review');
+            expect(result).toContain('No issues detected or empty output from agent');
+            expect(result).toContain('*Generated by codex-review-mcp v2.3.0*');
+        });
         it('should return fallback message when output is only whitespace', () => {
             const result = formatOutput('   \n\n  \t  ');
             expect(result).toContain('# Code Review');

package/dist/tools/performCodeReview.js

@@ -59,7 +59,7 @@ export async function performCodeReview(input, onProgress) {
     await debugLog(`Review produced ${agentMd.length} chars`);
     // Format output
     await onProgress?.('Formatting output…', 90, 100);
-    const out = formatOutput(agentMd);
+    const out = formatOutput(agentMd, VERSION);
     await onProgress?.('Done', 100, 100);
     return out;
 }

package/package.json

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