codex-review-mcp 2.0.2 → 2.1.1

package/dist/review/buildPrompt.js

@@ -34,6 +34,7 @@ export function buildPrompt({ diffText, context, focus, version, isStaticReview
         '- 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.',
+        '- 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,
         '',
         'BEFORE making ANY suggestion, ask yourself:',

package/dist/review/invokeAgent.js

@@ -1,68 +1,20 @@
-import { Agent, run, tool } from '@openai/agents';
+import { Agent, run } from '@openai/agents';
 import { debugLog } from '../util/debug.js';
-import { promises as fs } from 'node:fs';
-import { join } from 'node:path';
-import { z } from 'zod';
 export async function invokeAgent({ prompt, maxTokens, workspaceDir }) {
     const model = process.env.CODEX_MODEL || 'gpt-5-codex';
-    const cwd = workspaceDir || process.cwd();
-    // Allow configuring max turns via env var (default: 100)
-    // Each tool call (read_file) counts as 1 turn
-    // 100 allows agent to read many files if needed for thorough review
-    const maxTurns = parseInt(process.env.CODEX_MAX_TURNS || '100', 10);
-    // Tool: Read file from the codebase
-    const readFileTool = tool({
-        name: 'read_file',
-        description: 'Read the contents of a file from the codebase to examine existing patterns, utilities, or configuration. Use this to check for existing code before suggesting changes.',
-        parameters: z.object({
-            path: z.string().describe('Relative path to the file from the repository root'),
-        }),
-        execute: async (params) => {
-            try {
-                // Basic path sandboxing to keep reads under cwd
-                const requested = params.path.replace(/\\/g, '/');
-                if (requested.includes('..')) {
-                    return 'Access denied: parent directory traversal is not allowed.';
-                }
-                const fullPath = join(cwd, requested);
-                if (!fullPath.startsWith(cwd)) {
-                    return 'Access denied: path escapes workspace boundary.';
-                }
-                const content = await fs.readFile(fullPath, 'utf8');
-                const MAX_RETURN = 50_000; // cap very large files
-                const body = content.length > MAX_RETURN ? content.slice(0, MAX_RETURN) + '\n... [truncated]' : content;
-                await debugLog(`Agent read file: ${params.path} (${content.length} chars)`);
-                return `File: ${params.path}\n\n${body}`;
-            }
-            catch (error) {
-                const errorMsg = `Unable to read ${params.path}: ${error.message}`;
-                await debugLog(`Agent file read error: ${errorMsg}`);
-                return errorMsg;
-            }
-        },
-    });
     const agent = new Agent({
         name: 'Code Reviewer',
-        instructions: 'You are a precise code-review agent. Follow the output contract exactly. Use minimal verbosity. When you need to examine existing code patterns, utilities, or configurations to provide accurate recommendations, use the read_file tool.',
+        instructions: 'You are a precise code-review agent. Follow the output contract exactly. Use minimal verbosity.',
         model,
-        tools: [readFileTool],
     });
     try {
-        const result = await run(agent, prompt, {
-            maxTurns, // Configurable via CODEX_MAX_TURNS env var (default: 100)
-        });
+        const result = await run(agent, prompt);
         const out = result.finalOutput ?? '';
-        await debugLog(`Agent model=${model} maxTurns=${maxTurns} outputLen=${out.length}`);
+        await debugLog(`Agent model=${model} outputLen=${out.length}`);
         return out;
     }
     catch (error) {
         await debugLog(`Agent error: ${error?.message || String(error)}`);
-        // If max turns exceeded, provide helpful error message
-        if (error?.message?.includes('Max turns')) {
-            throw new Error(`OpenAI API error: ${error.message}. ` +
-                `The agent made too many tool calls (read_file). ` +
-                `Current limit: ${maxTurns}. Increase with CODEX_MAX_TURNS env var.`);
-        }
         throw new Error(`OpenAI API error: ${error?.message || 'Unknown error occurred'}`);
     }
 }

package/dist/review/invokeAgent.test.js

@@ -5,7 +5,6 @@ import { run } from '@openai/agents';
 vi.mock('@openai/agents', () => ({
     Agent: vi.fn().mockImplementation((config) => config),
     run: vi.fn(),
-    tool: vi.fn().mockImplementation((config) => config),
 }));
 vi.mock('../util/debug', () => ({
     debugLog: vi.fn(),
@@ -75,26 +74,7 @@ describe('invokeAgent', () => {
             process.env.CODEX_MODEL = originalEnv;
         }
     });
-    it('should create agent with read_file tool for examining codebase', async () => {
-        vi.mocked(run).mockResolvedValue({
-            finalOutput: '# Review',
-        });
-        await invokeAgent({
-            prompt: 'Review this code',
-            workspaceDir: '/test/workspace'
-        });
-        const { Agent, tool } = await import('@openai/agents');
-        // Verify tool was created
-        expect(tool).toHaveBeenCalledWith(expect.objectContaining({
-            name: 'read_file',
-            description: expect.stringContaining('examine existing patterns'),
-        }));
-        // Verify agent was created with tools
-        expect(Agent).toHaveBeenCalledWith(expect.objectContaining({
-            tools: expect.arrayContaining([expect.anything()]),
-        }));
-    });
-    it('should pass workspaceDir to control file reading scope', async () => {
+    it('should pass workspaceDir parameter (for future use)', async () => {
         vi.mocked(run).mockResolvedValue({
             finalOutput: '# Review',
         });

package/package.json

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