codex-review-mcp 1.3.1 → 1.4.0

package/README.md

@@ -11,6 +11,7 @@ With this MCP server, your AI assistant can call out for a code review **right i
 ## Features
 
 - 🎯 **Zero Configuration** - Just add your OpenAI API key
+- 📋 **Respects Your Cursor Rules** - Automatically reads `.cursor/rules/` and enforces YOUR coding standards
 - 🔍 Automatically reviews your uncommitted changes
 - 🤖 Powered by GPT-5 Codex for intelligent code analysis
 - 📊 Returns actionable feedback in Markdown format
@@ -100,9 +101,9 @@ Most users won't need these, but they're available if you want more control:
 
 1. **Detects Repository**: Automatically finds your git repository root
 2. **Collects Diff**: Runs git diff to get the changes
-3. **Gathers Context**: Collects relevant context about your codebase
-4. **AI Review**: Sends to GPT-5 Codex for intelligent analysis
-5. **Returns Feedback**: Provides actionable Markdown feedback
+3. **Gathers Context**: Reads your `.cursor/rules/`, `CODE_REVIEW.md`, `.eslintrc`, and other project files
+4. **AI Review**: Sends to GPT-5 Codex with YOUR project's coding standards
+5. **Returns Feedback**: Provides actionable Markdown feedback that respects YOUR rules
 
 ## Requirements
 

package/dist/review/buildPrompt.js

@@ -2,12 +2,46 @@ export function buildPrompt({ diffText, context, focus }) {
     const focusLine = focus ? `Focus areas: ${focus}.` : '';
     return [
         'You are an expert AI code reviewer. Be concise, specific, and actionable.',
-        'Prefer minimal diffs and direct fixes. Respect project guidelines when provided.',
+        'Prefer minimal diffs and direct fixes.',
+        '',
+        'ENVIRONMENT ASSUMPTION: The runtime and tooling are POSIX (macOS/Linux). Do NOT suggest Windows/PowerShell/cmd-specific commands or paths.',
+        '',
+        '🚨 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',
+        '- 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.',
         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?',
+        '',
         '\n---\nScope and Diff (unified=0):\n',
         diffText,
-        context ? '\n---\nProject context and guidelines:\n' + context : '',
+        context ? '\n---\nProject context and guidelines (FOLLOW THESE STRICTLY):\n' + context : '',
         '\n---\nOutput strictly as Markdown with the following sections:\n',
-        '1) Title + scope summary\n2) Quick Summary (3–6 bullets)\n3) Issues table: severity | file:lines | category | explanation | suggested fix\n4) Inline suggested edits for top issues\n5) Positive notes\n6) Next steps',
+        '1) Title + scope summary\n2) Quick Summary (3–6 bullets)\n3) Issues table: severity | file:lines | category | explanation | suggested fix\n4) Inline suggested edits for top issues\n5) Positive notes (mention when code follows project guidelines well)\n6) Next steps',
     ].join('\n');
 }

package/dist/review/gatherContext.js

@@ -3,17 +3,37 @@ import { join } from 'node:path';
 const CANDIDATE_FILES = [
     '.cursor/rules/tsx.mdc',
     '.cursor/rules/project.mdc',
-    '.cursor/rules/**/*.md',
+    '.cursor/rules/', // directory to scan for all .md and .mdc files
     'CODE_REVIEW.md',
     'CONTRIBUTING.md',
+    'ARCHITECTURE.md',
+    'STYLE_GUIDE.md',
+    'CODING_STANDARDS.md',
+    'GUIDELINES.md',
     'SECURITY.md',
     '.eslintrc',
     '.eslintrc.cjs',
     '.eslintrc.js',
     '.eslintrc.json',
+    '.prettierrc',
+    '.prettierrc.json',
+    '.prettierrc.js',
+    'prettier.config.js',
     'package.json',
     'tsconfig.json',
+    'vitest.config.ts',
+    'jest.config.js',
 ];
+// Directories to skip when scanning
+const SKIP_DIRS = new Set([
+    'node_modules',
+    '.git',
+    'dist',
+    'build',
+    'coverage',
+    '.next',
+    '.turbo',
+]);
 async function readIfExists(path) {
     try {
         const data = await fs.readFile(path, 'utf8');
@@ -23,21 +43,89 @@ async function readIfExists(path) {
         return null;
     }
 }
+async function scanDirectory(dirPath, extensions, visited = new Set()) {
+    const files = [];
+    // Protect against circular symlinks
+    try {
+        const realPath = await fs.realpath(dirPath);
+        if (visited.has(realPath))
+            return files;
+        visited.add(realPath);
+    }
+    catch {
+        // Can't resolve real path, skip
+        return files;
+    }
+    try {
+        const entries = await fs.readdir(dirPath, { withFileTypes: true });
+        for (const entry of entries) {
+            // Skip common large directories
+            if (entry.isDirectory() && SKIP_DIRS.has(entry.name)) {
+                continue;
+            }
+            const fullPath = join(dirPath, entry.name);
+            if (entry.isDirectory()) {
+                // Recursively scan subdirectories
+                const subFiles = await scanDirectory(fullPath, extensions, visited);
+                files.push(...subFiles);
+            }
+            else if (entry.isFile()) {
+                // Check if file has one of the desired extensions
+                if (extensions.some(ext => entry.name.endsWith(ext))) {
+                    files.push(fullPath);
+                }
+            }
+        }
+    }
+    catch {
+        // Directory doesn't exist or can't be read
+    }
+    return files;
+}
 export async function gatherContext() {
     const chunks = [];
-    for (const rel of CANDIDATE_FILES) {
-        // naive glob: only handle literal paths here to keep MVP simple
-        if (rel.includes('*'))
-            continue;
-        const data = await readIfExists(join(process.cwd(), rel));
-        if (!data)
-            continue;
-        // bound each file contribution to ~8k chars
-        const capped = data.slice(0, 8000);
-        chunks.push(`\n<!-- ${rel} -->\n${capped}`);
-        // global size guard ~50k chars
+    const processedPaths = new Set();
+    const cwd = process.cwd();
+    for (const pattern of CANDIDATE_FILES) {
+        // Check global size guard before processing each pattern
         if (chunks.join('').length > 50_000)
             break;
+        // Handle directory paths - scan for .md and .mdc files
+        if (pattern.endsWith('/')) {
+            const dirPath = join(cwd, pattern);
+            const files = await scanDirectory(dirPath, ['.md', '.mdc']);
+            for (const filePath of files) {
+                const relativePath = filePath.replace(cwd + '/', '');
+                if (processedPaths.has(relativePath))
+                    continue;
+                processedPaths.add(relativePath);
+                const data = await readIfExists(filePath);
+                if (!data)
+                    continue;
+                const capped = data.slice(0, 8000);
+                chunks.push(`\n<!-- ${relativePath} -->\n${capped}`);
+                // Check size after each file
+                if (chunks.join('').length > 50_000)
+                    break;
+            }
+            // Check size after processing directory
+            if (chunks.join('').length > 50_000)
+                break;
+        }
+        else {
+            // Handle literal file paths
+            if (processedPaths.has(pattern))
+                continue;
+            processedPaths.add(pattern);
+            const data = await readIfExists(join(cwd, pattern));
+            if (!data)
+                continue;
+            const capped = data.slice(0, 8000);
+            chunks.push(`\n<!-- ${pattern} -->\n${capped}`);
+            // Check size after each literal file
+            if (chunks.join('').length > 50_000)
+                break;
+        }
     }
     return chunks.join('\n');
 }

package/package.json

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