codex-review-mcp 2.0.1 → 2.1.0

package/dist/review/invokeAgent.js

@@ -1,52 +1,14 @@
-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();
-    // 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: 25, // Allow more turns for file reading (default is 10)
-        });
+        const result = await run(agent, prompt);
         const out = result.finalOutput ?? '';
         await debugLog(`Agent model=${model} outputLen=${out.length}`);
         return out;

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.1",
+  "version": "2.1.0",
   "main": "index.js",
   "scripts": {
     "build": "tsc",