项目根目录/
├── .git/hooks/pre-commit      # Git hook入口
├── scripts/claude-review.sh   # Claude Code调用脚本
└── CLAUDE.md                  # 审查规则配置（已有则追加）

#!/bin/bash
# Claude Code Pre-commit Review Hook
# 每次commit前用Claude Code审查staged代码

set -e

# 颜色定义
RED='\033[0;31m'
YELLOW='\033[1;33m'
GREEN='\033[0;32m'
NC='\033[0m'

echo -e "${YELLOW}🔍 Claude Code 正在审查你的代码...${NC}"

# 获取staged文件的diff
DIFF=$(git diff --cached)

# 如果没有staged文件，跳过
if [ -z "$DIFF" ]; then
    echo -e "${GREEN}✅ 没有staged文件，跳过审查${NC}"
    exit 0
fi

# 构建审查prompt
PROMPT="你是一个严格的代码审查员。请审查以下git diff，检查：
1. 空指针/空值引用
2. SQL注入、XSS等安全漏洞
3. 明显的逻辑错误
4. 死代码或未使用的变量
5. 资源泄漏（未关闭的文件/连接）

只输出3种结论之一：
- PASS: 没有问题
- WARN: 有小问题但不应阻止提交，简要说明
- BLOCK: 有严重问题必须修复，列出具体文件和行号

Diff内容：
$DIFF"

# 调用Claude Code（-p模式：print-only，不进入交互）
RESULT=$(claude -p "$PROMPT" 2>&1) || true

echo -e "\n${YELLOW}📋 审查结果：${NC}"
echo "$RESULT"
echo ""

# 检查是否需要阻止提交
if echo "$RESULT" | grep -q "^BLOCK"; then
    echo -e "${RED}❌ 提交被阻止！请修复上述问题后重试${NC}"
    echo -e "${YELLOW}💡 提示：如果你确信没问题，可以用 git commit --no-verify 跳过审查${NC}"
    exit 1
elif echo "$RESULT" | grep -q "^WARN"; then
    echo -e "${YELLOW}⚠️  有警告但允许提交${NC}"
else
    echo -e "${GREEN}✅ 审查通过${NC}"
fi

exit 0

chmod +x .git/hooks/pre-commit

## Code Review Rules

当审查代码时，按以下规则执行：

### 关键检查项（BLOCK级别）
- Python: 任何使用字符串拼接构建SQL查询（必须用参数化查询）
- TypeScript: 任何 `any` 类型的使用
- Go: 未检查的 `err` 返回值
- 所有语言: 硬编码的密码/API Key/Token

### 警告项（WARN级别）
- Python: 裸except（except: 而不是 except Exception）
- JavaScript: console.log 未移除
- 所有语言: TODO注释超过30天未处理
- 函数超过80行（建议拆分）

### 项目特定规则
- 日志使用 `logger.xxx()` 而不是 `print()`
- HTTP调用超时不得小于5秒
- 数据库查询必须有limit（防止全表扫描）

# bad.py
def get_user(username):
    query = "SELECT * FROM users WHERE name = '" + username + "'"
    return db.execute(query)

BLOCK: bad.py:3 - SQL注入漏洞
- 使用字符串拼接构建SQL查询
- 修复：使用参数化查询 db.execute("SELECT * FROM users WHERE name = ?", [username])

# 紧急情况下跳过（不推荐日常使用）
git commit --no-verify -m "hotfix"

# 或者在hook开头加条件判断
if [ "$SKIP_CLAUDE_REVIEW" = "1" ]; then
    exit 0
fi

# 只审查以下扩展名的文件
STAGED_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E '\.(py|js|ts|go|rs|java)$' || true)

if [ -z "$STAGED_FILES" ]; then
    echo "✅ 没有需要审查的代码文件"
    exit 0
fi

DIFF_SIZE=$(echo "$DIFF" | wc -c)
MAX_SIZE=50000  # 50KB

if [ "$DIFF_SIZE" -gt "$MAX_SIZE" ]; then
    echo "⚠️  Diff过大（${DIFF_SIZE}字节），跳过自动审查"
    echo "💡 建议：拆分commit，每个commit保持小粒度"
    exit 0
fi