review tool missing --skip-git-repo-check flag

[JVL1]

Bug

The review tool handler in dist/tools/handlers.js does not pass --skip-git-repo-check to the Codex CLI, while the codex (exec) tool handler does. This causes review to fail with:

Not inside a trusted directory and --skip-git-repo-check was not specified.

...when invoked via MCP from a parent orchestrator (e.g., Claude Code) even when the directory is trusted in ~/.codex/config.toml.

Repro

1. Configure Codex MCP server in Claude Code
2. Call mcp__codex-cli__review with workingDirectory set to a git repo
3. The review fails with the trust directory error

mcp__codex-cli__codex (which uses exec --skip-git-repo-check) works fine on the same directory.

Fix

Add --skip-git-repo-check to the review command args in ReviewToolHandler.execute(), matching the behavior of CodexToolHandler.execute():

// handlers.js, ReviewToolHandler.execute()
  cmdArgs.push('-c', `model="${selectedModel}"`);
+ cmdArgs.push('--skip-git-repo-check');
  cmdArgs.push('review');

Environment

• codex-mcp-server v1.4.2
• Codex CLI (codex binary at /opt/homebrew/bin/codex)
• macOS Darwin 25.3.0

[JVL1]

Update: Correct fix approach

My initial suggestion of adding --skip-git-repo-check before review was wrong — that flag is only available on the exec subcommand, not as a global flag.

The actual fix is to route the review through codex exec ... review instead of codex review. The exec review subcommand inherits all exec flags including -C (working directory) and --skip-git-repo-check.

Working command:

codex exec --model <model> -C <workingDirectory> --skip-git-repo-check review --base main

Suggested patch for ReviewToolHandler.execute() in handlers.js:

- const cmdArgs = [];
- const selectedModel = ...;
- cmdArgs.push('-c', `model="${selectedModel}"`);
- cmdArgs.push('review');
- // ... review flags
- if (workingDirectory) {
-     cmdArgs.push('-C', workingDirectory);
- }
+ // Route through 'codex exec ... review' instead of 'codex review'
+ // so we get -C (working directory) and --skip-git-repo-check support.
+ const cmdArgs = ['exec'];
+ const selectedModel = ...;
+ cmdArgs.push('--model', selectedModel);
+ if (workingDirectory) {
+     cmdArgs.push('-C', workingDirectory);
+ }
+ cmdArgs.push('--skip-git-repo-check');
+ cmdArgs.push('review');
+ // ... review flags (--base, --commit, --uncommitted, --title, prompt)

Key changes:

1. Start with exec subcommand instead of going directly to review
2. Use --model flag (exec flag) instead of -c model="X" (config override)
3. Add -C workingDirectory as an exec flag (before review)
4. Add --skip-git-repo-check as an exec flag (before review)
5. Review-specific flags (--base, --commit, etc.) go after review

Verified this works on Codex CLI v0.112.0 / codex-mcp-server v1.4.2.