claude-codex-code-review 0.2.1 → 0.4.0

package/.claude/commands/codex-review.md

@@ -8,7 +8,7 @@ argument-hint: [optional review instructions]
 
 Run the project Codex review script first:
 
-!`scripts/codex-review.sh "$ARGUMENTS"`
+!`scripts/codex-review.sh $ARGUMENTS`
 
 Then read `.ai-review/codex-review.md`, `.ai-review/effective-config.env`, and `.codex-review.yml` if they exist.
 
@@ -34,7 +34,7 @@ If the project has no `.codex-review.yml` and the user passes no command argumen
 ```yaml
 mode: ask
 review_scope: uncommitted
-max_fix_rounds: 1
+max_fix_rounds: 2
 auto_fix_severities: []
 ```
 

package/.codex-review.yml

@@ -1,4 +1,4 @@
 mode: ask
 review_scope: uncommitted
-max_fix_rounds: 1
+max_fix_rounds: 2
 auto_fix_severities: []

package/README.md

@@ -1,8 +1,11 @@
 # claude-codex-code-review
 
-在 Claude Code 里安装一个项目级 `/codex-review` 命令，让你可以用 Codex CLI 审查 Claude 刚写完的代码，再由 Claude 读取 review 结果并按你的选择修复问题。
+Claude Code 和 Codex CLI 之间的交叉代码审查工具。
 
-典型流程：
+- **在 Claude Code 中**：用 `/codex-review` 调用 Codex CLI 审查代码，再由 Claude 修复问题。
+- **在 Codex CLI 中**：用 claude-review skill 调用 Claude 审查代码，再由 Codex 修复问题。
+
+典型流程（Claude Code → Codex 审查）：
 
 1. 你在 Claude Code 里让 Claude 写代码。
 2. 写完后手动运行 `/codex-review`。
@@ -11,41 +14,74 @@
 5. Claude 读取结果，列出问题，并询问你要修复哪些。
 6. 你确认后，Claude 只修复选中的问题。
 
+典型流程（Codex CLI → Claude 审查）：
+
+1. 你在 Codex CLI 里让 Codex 写代码。
+2. 写完后触发 claude-review skill。
+3. Claude 审查当前项目的未提交改动。
+4. review 结果保存到 `.ai-review/claude-review.md`。
+5. Codex 读取结果，列出问题，并询问你要修复哪些。
+6. 你确认后，Codex 只修复选中的问题。
+
 ## 安装
 
-### 使用 curl
+### 全局安装（默认）
 
-在目标项目根目录执行：
+安装后所有项目都可以直接使用 `/codex-review`，无需逐个项目安装。
+
+使用 npx：
+
+```bash
+npx claude-codex-code-review install
+```
+
+使用 curl：
 
 ```bash
 curl -fsSL https://raw.githubusercontent.com/lkkwxy/claude-codex-claude-codex-review-claude/main/scripts/install.sh | bash
 ```
 
-注意：`curl | bash` 必须使用 `raw.githubusercontent.com` 地址，不能使用 GitHub 页面里的 `blob/main` 地址。
+全局安装会：
+
+- 将 `/codex-review` 斜杠命令安装到 `~/.claude/commands/`（Claude Code 使用）
+- 将 `claude-review` skill 安装到 `~/.codex/skills/claude-review/`（Codex CLI 使用）
+- 将 `.ai-review/` 加入全局 gitignore，确保中间文件不会被任何项目提交
+
+如果某个项目需要自定义配置，在该项目根目录创建 `.codex-review.yml` 即可。
 
-### 使用 npx
+### 项目级安装
 
-发布到 npm 后，可以在目标项目根目录执行：
+如果只想在单个项目中使用，可以在项目根目录执行：
+
+#### 使用 curl
+
+在目标项目根目录执行：
 
 ```bash
-npx claude-codex-code-review install
+curl -fsSL https://raw.githubusercontent.com/lkkwxy/claude-codex-claude-codex-review-claude/main/scripts/install.sh | bash -s -- --local
 ```
 
-也可以指定安装目录：
+注意：`curl | bash` 必须使用 `raw.githubusercontent.com` 地址，不能使用 GitHub 页面里的 `blob/main` 地址。
+
+#### 使用 npx
+
+在目标项目根目录执行：
 
 ```bash
-npx claude-codex-code-review install -- --target-dir /path/to/project
+npx claude-codex-code-review install -- --local
 ```
 
-如果需要覆盖已有文件：
+也可以指定安装目录：
 
 ```bash
-npx claude-codex-code-review install -- --force
+npx claude-codex-code-review install -- --target-dir /path/to/project
 ```
 
 ## 使用
 
-安装后，在目标项目里打开 Claude Code，运行：
+### 在 Claude Code 中（用 Codex 审查）
+
+安装后，在任意项目里打开 Claude Code，运行：
 
 ```text
 /codex-review
@@ -57,9 +93,25 @@ npx claude-codex-code-review install -- --force
 /codex-review --mode severity --auto-fix-severities P0,P1
 ```
 
+### 在 Codex CLI 中（用 Claude 审查）
+
+安装后，在任意项目里打开 Codex CLI，Codex 会自动发现 `claude-review` skill。你可以：
+
+- 直接告诉 Codex「用 Claude 审查我的代码」
+- 通过 `/skills` 菜单选择 `claude-review`
+
 ## 安装内容
 
-安装器会写入这些文件：
+全局安装会写入这些文件：
+
+```text
+~/.claude/commands/codex-review.md
+~/.claude/scripts/codex-review.sh
+~/.codex/skills/claude-review/SKILL.md
+~/.codex/skills/claude-review/claude-review.sh
+```
+
+项目级安装会写入这些文件：
 
 ```text
 .claude/commands/codex-review.md
@@ -68,11 +120,12 @@ scripts/codex-review.sh
 .gitignore
 ```
 
-运行 `/codex-review` 后，会生成：
+运行审查后，会生成：
 
 ```text
-.ai-review/codex-review.md
-.ai-review/codex-review.log
+.ai-review/codex-review.md    (Codex 审查结果)
+.ai-review/claude-review.md   (Claude 审查结果)
+.ai-review/*.log
 .ai-review/effective-config.env
 ```
 
@@ -85,7 +138,7 @@ scripts/codex-review.sh
 ```yaml
 mode: ask
 review_scope: uncommitted
-max_fix_rounds: 1
+max_fix_rounds: 2
 auto_fix_severities: []
 ```
 
@@ -100,7 +153,7 @@ auto_fix_severities: []
 ```yaml
 mode: ask
 review_scope: uncommitted
-max_fix_rounds: 1
+max_fix_rounds: 2
 auto_fix_severities: []
 ```
 
@@ -137,8 +190,8 @@ codex --help
 
 ## 设计原则
 
-- Codex review 阶段只审查，不修改代码。
-- Claude 修复阶段只处理 Codex 明确指出的问题。
+- 审查阶段只审查，不修改代码（无论 Codex 审查还是 Claude 审查）。
+- 修复阶段只处理审查方明确指出的问题。
 - 默认不自动修复，先由用户判断。
 - 不自动 commit，最终提交仍由开发者确认。
 - 最多执行配置允许的修复轮次，避免自动循环。

package/bin/codex-review

@@ -10,7 +10,7 @@ while [ -L "$SOURCE" ]; do
 done
 PACKAGE_ROOT="$(cd "$(dirname "$SOURCE")/.." && pwd -P)"
 TARGET_DIR="$(pwd)"
-FORCE=0
+GLOBAL=1
 
 usage() {
   cat <<'USAGE'
@@ -18,17 +18,17 @@ Usage:
   claude-codex-code-review install [options]
   claude-codex-code-review --help
 
-Installs the Claude Code /codex-review command into the current project.
+Installs the Claude Code /codex-review command globally (default) or into a specific project.
 
 Options:
-  --target-dir <dir>  Install into a specific project directory. Default: current directory.
-  --force             Overwrite existing codex-review files.
-  -h, --help          Show this help.
+  --local               Install into the current project instead of globally.
+  --target-dir <dir>    Install into a specific project directory (implies --local).
+  -h, --help            Show this help.
 
 Examples:
   npx claude-codex-code-review install
+  npx claude-codex-code-review install -- --local
   npx claude-codex-code-review install -- --target-dir /path/to/project
-  npx claude-codex-code-review install -- --force
 USAGE
 }
 
@@ -49,11 +49,6 @@ copy_file() {
     fail "package template is missing: $source_path"
   fi
 
-  if [ -f "$dest_path" ] && [ "$FORCE" -ne 1 ]; then
-    log "Keeping existing $dest_path"
-    return
-  fi
-
   mkdir -p "$(dirname "$dest_path")"
   cp "$source_path" "$dest_path"
 }
@@ -64,11 +59,58 @@ append_gitignore_once() {
 
   touch "$gitignore_path"
   if grep -Fxq "$entry" "$gitignore_path"; then
-    log "Keeping existing .gitignore entry: $entry"
-  else
-    printf "\n%s\n" "$entry" >> "$gitignore_path"
-    log "Added .gitignore entry: $entry"
+    return
   fi
+  printf "\n%s\n" "$entry" >> "$gitignore_path"
+}
+
+install_global() {
+  local global_commands_dir="$HOME/.claude/commands"
+  local global_scripts_dir="$HOME/.claude/scripts"
+  local script_path="$global_scripts_dir/codex-review.sh"
+  local source_template="$PACKAGE_ROOT/.claude/commands/codex-review.md"
+  local source_script="$PACKAGE_ROOT/scripts/codex-review.sh"
+
+  if [ ! -f "$source_template" ]; then
+    fail "package template is missing: $source_template"
+  fi
+  if [ ! -f "$source_script" ]; then
+    fail "package template is missing: $source_script"
+  fi
+
+  mkdir -p "$global_commands_dir"
+  mkdir -p "$global_scripts_dir"
+
+  cp "$source_script" "$global_scripts_dir/codex-review.sh"
+  chmod +x "$global_scripts_dir/codex-review.sh"
+
+  sed "s|Bash(scripts/codex-review.sh:|Bash(${script_path}:|g;
+       s|scripts/codex-review.sh |${script_path} |g" \
+    "$source_template" > "$global_commands_dir/codex-review.md"
+
+  local excludes_file
+  excludes_file="$(git config --global core.excludesFile 2>/dev/null || true)"
+  if [ -z "$excludes_file" ]; then
+    excludes_file="$HOME/.gitignore_global"
+    git config --global core.excludesFile "$excludes_file"
+  fi
+  append_gitignore_once "$excludes_file"
+
+  local codex_skill_dir="$HOME/.codex/skills/claude-review"
+  local source_skill="$PACKAGE_ROOT/skills/claude-review/SKILL.md"
+  local source_claude_script="$PACKAGE_ROOT/scripts/claude-review.sh"
+
+  if [ -f "$source_skill" ] && [ -f "$source_claude_script" ]; then
+    mkdir -p "$codex_skill_dir"
+    cp "$source_claude_script" "$codex_skill_dir/claude-review.sh"
+    chmod +x "$codex_skill_dir/claude-review.sh"
+    sed "s|~/.codex/skills/claude-review/claude-review.sh|${codex_skill_dir}/claude-review.sh|g" \
+      "$source_skill" > "$codex_skill_dir/SKILL.md"
+  fi
+
+  log "Global install complete."
+  log "/codex-review is now available in all Claude Code projects."
+  log "claude-review skill is now available in Codex CLI."
 }
 
 COMMAND="${1:-install}"
@@ -87,17 +129,19 @@ esac
 
 while [ "$#" -gt 0 ]; do
   case "$1" in
+    --local)
+      GLOBAL=0
+      shift
+      ;;
     --target-dir)
       [ "$#" -ge 2 ] || fail "--target-dir requires a value."
       TARGET_DIR="$2"
+      GLOBAL=0
       shift 2
       ;;
     --target-dir=*)
       TARGET_DIR="${1#*=}"
-      shift
-      ;;
-    --force)
-      FORCE=1
+      GLOBAL=0
       shift
       ;;
     -h|--help)
@@ -110,6 +154,11 @@ while [ "$#" -gt 0 ]; do
   esac
 done
 
+if [ "$GLOBAL" -eq 1 ]; then
+  install_global
+  exit 0
+fi
+
 mkdir -p "$TARGET_DIR"
 cd "$TARGET_DIR" || fail "cannot enter target directory: $TARGET_DIR"
 
@@ -120,5 +169,3 @@ chmod +x "scripts/codex-review.sh"
 append_gitignore_once ".gitignore"
 
 log "Installed /codex-review into $TARGET_DIR"
-log "Usage in Claude Code: /codex-review"
-log "Optional: /codex-review --mode severity --auto-fix-severities P0,P1"

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "claude-codex-code-review",
-  "version": "0.2.1",
-  "description": "Install a Claude Code /codex-review command that runs Codex CLI review and lets Claude apply selected fixes.",
+  "version": "0.4.0",
+  "description": "Cross-review between Claude Code and Codex CLI. Provides /codex-review in Claude Code and claude-review skill in Codex CLI.",
   "license": "MIT",
   "bin": {
     "claude-codex-code-review": "bin/codex-review"
@@ -10,10 +10,12 @@
     ".claude/commands/codex-review.md",
     ".codex-review.yml",
     "bin/codex-review",
-    "scripts/codex-review.sh"
+    "scripts/codex-review.sh",
+    "scripts/claude-review.sh",
+    "skills/claude-review/SKILL.md"
   ],
   "scripts": {
-    "test": "bash -n scripts/codex-review.sh scripts/install.sh bin/codex-review"
+    "test": "bash -n scripts/codex-review.sh scripts/claude-review.sh scripts/install.sh bin/codex-review"
   },
   "engines": {
     "node": ">=18"

package/scripts/claude-review.sh

@@ -0,0 +1,287 @@
+#!/usr/bin/env bash
+set -u
+set -o pipefail
+
+OUTPUT_DIR=".ai-review"
+REVIEW_FILE="${OUTPUT_DIR}/claude-review.md"
+LOG_FILE="${OUTPUT_DIR}/claude-review.log"
+EFFECTIVE_CONFIG_FILE="${OUTPUT_DIR}/effective-config.env"
+CONFIG_FILE=".codex-review.yml"
+DEFAULT_MODE="ask"
+DEFAULT_REVIEW_SCOPE="uncommitted"
+DEFAULT_MAX_FIX_ROUNDS="2"
+DEFAULT_AUTO_FIX_SEVERITIES="[]"
+MODE_ARG=""
+REVIEW_SCOPE_ARG=""
+MAX_FIX_ROUNDS_ARG=""
+AUTO_FIX_SEVERITIES_ARG=""
+EXTRA_INSTRUCTIONS=()
+
+timestamp() {
+  date "+%Y-%m-%d %H:%M:%S %z"
+}
+
+usage() {
+  cat <<'USAGE'
+Usage: claude-review.sh [options] [follow-up instructions]
+
+Runs Claude Code review on uncommitted changes and writes the result to
+.ai-review/claude-review.md.
+
+Options:
+  --mode <ask|auto|severity>
+      Fix policy Codex should apply after reading the review. Default: ask.
+
+  --review-scope <uncommitted>
+      Review scope. Default: uncommitted.
+
+  --max-fix-rounds <number>
+      Maximum automatic fix rounds. Default: 1.
+
+  --auto-fix-severities <list>
+      Comma-separated severities for mode=severity, e.g. P0,P1. Default: empty.
+
+  -h, --help
+      Show this help.
+USAGE
+}
+
+fail() {
+  local message="$1"
+  mkdir -p "$OUTPUT_DIR"
+  {
+    printf "[%s] ERROR: %s\n" "$(timestamp)" "$message"
+  } | tee -a "$LOG_FILE" >&2
+  exit 1
+}
+
+read_config_value() {
+  local key="$1"
+  local default_value="$2"
+
+  if [ ! -f "$CONFIG_FILE" ]; then
+    printf "%s" "$default_value"
+    return
+  fi
+
+  local value
+  value="$(awk -F ':' -v key="$key" '
+    $1 == key {
+      sub(/^[[:space:]]+/, "", $2)
+      sub(/[[:space:]]+$/, "", $2)
+      gsub(/^["'"'"']|["'"'"']$/, "", $2)
+      print $2
+      exit
+    }
+  ' "$CONFIG_FILE")"
+
+  if [ -n "$value" ]; then
+    printf "%s" "$value"
+  else
+    printf "%s" "$default_value"
+  fi
+}
+
+while [ "$#" -gt 0 ]; do
+  case "$1" in
+    --mode)
+      [ "$#" -ge 2 ] || fail "--mode requires a value."
+      MODE_ARG="$2"
+      shift 2
+      ;;
+    --mode=*)
+      MODE_ARG="${1#*=}"
+      shift
+      ;;
+    --review-scope)
+      [ "$#" -ge 2 ] || fail "--review-scope requires a value."
+      REVIEW_SCOPE_ARG="$2"
+      shift 2
+      ;;
+    --review-scope=*)
+      REVIEW_SCOPE_ARG="${1#*=}"
+      shift
+      ;;
+    --max-fix-rounds)
+      [ "$#" -ge 2 ] || fail "--max-fix-rounds requires a value."
+      MAX_FIX_ROUNDS_ARG="$2"
+      shift 2
+      ;;
+    --max-fix-rounds=*)
+      MAX_FIX_ROUNDS_ARG="${1#*=}"
+      shift
+      ;;
+    --auto-fix-severities)
+      [ "$#" -ge 2 ] || fail "--auto-fix-severities requires a value."
+      AUTO_FIX_SEVERITIES_ARG="$2"
+      shift 2
+      ;;
+    --auto-fix-severities=*)
+      AUTO_FIX_SEVERITIES_ARG="${1#*=}"
+      shift
+      ;;
+    -h|--help)
+      usage
+      exit 0
+      ;;
+    --)
+      shift
+      while [ "$#" -gt 0 ]; do
+        EXTRA_INSTRUCTIONS+=("$1")
+        shift
+      done
+      ;;
+    -*)
+      fail "unknown option: $1"
+      ;;
+    *)
+      EXTRA_INSTRUCTIONS+=("$1")
+      shift
+      ;;
+  esac
+done
+
+if ! command -v git >/dev/null 2>&1; then
+  fail "git is not available on PATH."
+fi
+
+if ! git rev-parse --is-inside-work-tree >/dev/null 2>&1; then
+  fail "current directory is not inside a git repository."
+fi
+
+if ! command -v claude >/dev/null 2>&1; then
+  fail "claude CLI is not available on PATH."
+fi
+
+MODE="${MODE_ARG:-$(read_config_value "mode" "$DEFAULT_MODE")}"
+REVIEW_SCOPE="${REVIEW_SCOPE_ARG:-$(read_config_value "review_scope" "$DEFAULT_REVIEW_SCOPE")}"
+MAX_FIX_ROUNDS="${MAX_FIX_ROUNDS_ARG:-$(read_config_value "max_fix_rounds" "$DEFAULT_MAX_FIX_ROUNDS")}"
+AUTO_FIX_SEVERITIES="${AUTO_FIX_SEVERITIES_ARG:-$(read_config_value "auto_fix_severities" "$DEFAULT_AUTO_FIX_SEVERITIES")}"
+EXTRA_INSTRUCTIONS_TEXT="${EXTRA_INSTRUCTIONS[*]:-}"
+
+case "$MODE" in
+  ask|auto|severity) ;;
+  *) fail "invalid --mode '$MODE'. Expected ask, auto, or severity." ;;
+esac
+
+case "$REVIEW_SCOPE" in
+  uncommitted) ;;
+  *) fail "invalid --review-scope '$REVIEW_SCOPE'. Only uncommitted is currently supported." ;;
+esac
+
+case "$MAX_FIX_ROUNDS" in
+  ''|*[!0-9]*) fail "invalid --max-fix-rounds '$MAX_FIX_ROUNDS'. Expected a non-negative integer." ;;
+  *) ;;
+esac
+
+PENDING_CHANGES="$(git status --porcelain)"
+
+mkdir -p "$OUTPUT_DIR"
+: > "$LOG_FILE"
+{
+  printf "MODE=%s\n" "$MODE"
+  printf "REVIEW_SCOPE=%s\n" "$REVIEW_SCOPE"
+  printf "MAX_FIX_ROUNDS=%s\n" "$MAX_FIX_ROUNDS"
+  printf "AUTO_FIX_SEVERITIES=%s\n" "$AUTO_FIX_SEVERITIES"
+} > "$EFFECTIVE_CONFIG_FILE"
+
+{
+  printf "[%s] Starting Claude review\n" "$(timestamp)"
+  printf "Working directory: %s\n" "$(pwd)"
+  printf "Review file: %s\n" "$REVIEW_FILE"
+  printf "Log file: %s\n" "$LOG_FILE"
+  printf "Effective mode: %s\n" "$MODE"
+  printf "Effective review scope: %s\n" "$REVIEW_SCOPE"
+  printf "Effective max fix rounds: %s\n" "$MAX_FIX_ROUNDS"
+  printf "Effective auto-fix severities: %s\n" "$AUTO_FIX_SEVERITIES"
+  if [ -n "$EXTRA_INSTRUCTIONS_TEXT" ]; then
+    printf "Extra instructions: %s\n" "$EXTRA_INSTRUCTIONS_TEXT"
+  fi
+} >> "$LOG_FILE"
+
+if [ ! -f "$CONFIG_FILE" ]; then
+  {
+    printf "[%s] Config file %s not found; using built-in defaults.\n" "$(timestamp)" "$CONFIG_FILE"
+  } >> "$LOG_FILE"
+fi
+
+if [ -z "$PENDING_CHANGES" ]; then
+  {
+    printf "# Claude Review\n\n"
+    printf "No staged, unstaged, or untracked changes were found.\n"
+  } > "$REVIEW_FILE"
+  {
+    printf "[%s] No uncommitted changes found; skipped Claude review.\n" "$(timestamp)"
+  } >> "$LOG_FILE"
+  printf "No uncommitted changes found. Wrote %s\n" "$REVIEW_FILE"
+  exit 0
+fi
+
+DIFF_CONTENT=""
+STAGED_DIFF="$(git diff --cached 2>/dev/null)"
+UNSTAGED_DIFF="$(git diff 2>/dev/null)"
+UNTRACKED_FILES="$(git ls-files --others --exclude-standard 2>/dev/null)"
+
+if [ -n "$STAGED_DIFF" ]; then
+  DIFF_CONTENT="${DIFF_CONTENT}
+=== Staged Changes ===
+${STAGED_DIFF}"
+fi
+
+if [ -n "$UNSTAGED_DIFF" ]; then
+  DIFF_CONTENT="${DIFF_CONTENT}
+=== Unstaged Changes ===
+${UNSTAGED_DIFF}"
+fi
+
+if [ -n "$UNTRACKED_FILES" ]; then
+  UNTRACKED_CONTENT=""
+  while IFS= read -r file; do
+    if [ -f "$file" ]; then
+      FILE_CONTENT="$(head -c 10000 "$file" 2>/dev/null)"
+      UNTRACKED_CONTENT="${UNTRACKED_CONTENT}
+--- ${file} (new file) ---
+${FILE_CONTENT}"
+    fi
+  done <<< "$UNTRACKED_FILES"
+  if [ -n "$UNTRACKED_CONTENT" ]; then
+    DIFF_CONTENT="${DIFF_CONTENT}
+=== Untracked Files ===
+${UNTRACKED_CONTENT}"
+  fi
+fi
+
+REVIEW_PROMPT="You are a senior code reviewer. Review the following uncommitted code changes.
+
+For each issue found, assign a severity level:
+- P0: Critical - security vulnerabilities, data loss, crashes
+- P1: High - bugs, logic errors, performance issues
+- P2: Medium - code quality, maintainability concerns
+- P3: Low - style, naming, minor suggestions
+
+Format your response as a Markdown document with:
+1. A summary section
+2. Issues grouped by severity (P0, P1, P2, P3)
+3. For each issue: file path, line reference if possible, description, and suggested fix
+
+If there are no issues, say so clearly.
+${EXTRA_INSTRUCTIONS_TEXT:+
+Additional review instructions: ${EXTRA_INSTRUCTIONS_TEXT}}"
+
+{
+  printf "[%s] Running: claude -p (review)\n" "$(timestamp)"
+} >> "$LOG_FILE"
+
+if printf "%s" "$DIFF_CONTENT" | claude -p "$REVIEW_PROMPT" > "$REVIEW_FILE" 2>> "$LOG_FILE"; then
+  {
+    printf "[%s] Claude review completed successfully.\n" "$(timestamp)"
+  } >> "$LOG_FILE"
+  printf "Claude review completed. Wrote %s\n" "$REVIEW_FILE"
+else
+  status=$?
+  {
+    printf "[%s] Claude review failed with exit code %s.\n" "$(timestamp)" "$status"
+  } >> "$LOG_FILE"
+  printf "# Claude Review Failed\n\nClaude review failed with exit code %s. See %s for details.\n" "$status" "$LOG_FILE" > "$REVIEW_FILE"
+  exit "$status"
+fi

package/scripts/codex-review.sh

@@ -9,7 +9,7 @@ EFFECTIVE_CONFIG_FILE="${OUTPUT_DIR}/effective-config.env"
 CONFIG_FILE=".codex-review.yml"
 DEFAULT_MODE="ask"
 DEFAULT_REVIEW_SCOPE="uncommitted"
-DEFAULT_MAX_FIX_ROUNDS="1"
+DEFAULT_MAX_FIX_ROUNDS="2"
 DEFAULT_AUTO_FIX_SEVERITIES="[]"
 MODE_ARG=""
 REVIEW_SCOPE_ARG=""

package/skills/claude-review/SKILL.md

@@ -0,0 +1,74 @@
+---
+name: "claude-review"
+description: "Use Claude to review uncommitted code changes. Invoke this skill when the user asks to review code with Claude, run claude-review, or wants a second opinion from Claude on their changes."
+---
+
+# Claude Review
+
+Use this skill to review uncommitted code changes using Claude Code.
+
+## When to use
+
+- User asks to review code with Claude
+- User wants a code review of their uncommitted changes
+- User mentions "claude-review" or "claude review"
+
+## Workflow
+
+1. Run the review script to get Claude's analysis:
+
+```bash
+~/.codex/skills/claude-review/claude-review.sh
+```
+
+2. Read the review result from `.ai-review/claude-review.md` and the effective config from `.ai-review/effective-config.env`.
+
+3. Also read `.codex-review.yml` in the current project if it exists (project-specific config).
+
+4. Summarize whether Claude reported actionable findings.
+
+5. If there are no actionable findings, say that clearly and stop.
+
+6. If there are actionable findings, list them by severity and include file/line references when Claude provided them.
+
+7. Ask the user which findings to fix before editing anything.
+
+8. If the user chooses findings to fix, only fix those selected findings.
+
+9. Do not perform unrelated refactors.
+
+10. Do not create a commit.
+
+11. After fixing selected findings, offer to run the review again for confirmation.
+
+## Configuration
+
+The review script reads `.codex-review.yml` from the current project directory. If missing, it uses built-in defaults:
+
+```yaml
+mode: ask
+review_scope: uncommitted
+max_fix_rounds: 2
+auto_fix_severities: []
+```
+
+Configuration meanings:
+
+- `mode: ask` — always ask before fixing.
+- `mode: auto` — automatically fix actionable findings.
+- `mode: severity` — only auto-fix findings matching `auto_fix_severities`; ask for all others.
+- `max_fix_rounds` — limits automatic review/fix loops.
+
+The script also accepts command-line options:
+
+```
+--mode ask|auto|severity
+--review-scope uncommitted
+--max-fix-rounds 1
+--auto-fix-severities P0,P1
+```
+
+## Dependencies
+
+- git
+- claude (Claude Code CLI, authenticated)