用 CLI Agent 做代码审查

# 1. 看一眼当前分支相对 main 的改动范围
git diff --stat main...HEAD

# 2. 把 diff 喂给 Claude Code 审一遍（也可以直接在交互模式下贴提示词）
claude -p "$(cat <<'EOF'
请对当前分支相对 main 的改动做一次代码审查，按以下结构输出：

## 1. 变更概览
- 一句话讲清这个 PR 在做什么
- 影响面：动了哪些模块/包
- 风险等级（low / medium / high）

## 2. 分文件审查
对每个改动文件按下面格式输出：

### <文件路径>
- [严重度] <一句话问题描述>
  - 行号：<L<起>-L<止>>
  - 原因：<为什么是问题，引用 CLAUDE.md/规范条款时直接抄原文>
  - 建议：<具体改法，能给 diff 就给 diff>

严重度只用四档：Critical / Major / Minor / Nit
- Critical：合并后会上线出事（数据丢失、鉴权绕过、崩溃）
- Major：影响正确性或可维护性（边界没考虑、命名误导、测试缺失）
- Minor：风格/重构建议
- Nit：可改可不改

## 3. 推荐动作
- 必须修改（blocker 列表）
- 建议修改（major/minor 列表）
- 可忽略（nit 列表）

约束：
- 只看 git diff main...HEAD 范围内的改动
- 不要重复 lint/format 工具能抓的问题
- 没有把握的发现宁可不报，假阳性比漏报代价大
- 引用代码时必须给出文件:行号
EOF
)"

## 1. 变更概览
- 给订单接口加上幂等键支持，避免重复下单
- 影响：src/order/service.ts、src/order/repository.ts、对应单测
- 风险等级：medium（涉及钱，但只新增不改老链路）

## 2. 分文件审查
### src/order/service.ts
- [Critical] 幂等键命中后直接返回旧订单，但没校验请求 body 是否一致
  - 行号：L88-L102
  - 原因：CLAUDE.md 第 4.2 条要求"幂等键 + 不同 payload 视为冲突"
  - 建议：命中后比对 payload hash，不一致则返回 409
- [Major] 新加的 `idempotencyKey` 入参没有长度校验
  ...

## 3. 推荐动作
- 必须修改：service.ts L88-L102（幂等冲突检测）
- 建议修改：service.ts L75 长度校验、repository.ts L40 索引补 unique
- 可忽略：测试用例命名风格

# ~/.bashrc 或 ~/.zshrc
review_pr() {
  local pr_num="$1"
  if [ -z "$pr_num" ]; then
    echo "用法: review_pr <PR 号>" >&2; return 1
  fi

  # 1. 用 gh CLI 拉 PR 元信息和 diff
  local pr_meta diff
  pr_meta=$(gh pr view "$pr_num" --json title,body,author,baseRefName,headRefName,files)
  diff=$(gh pr diff "$pr_num")

  # 2. 把 PR 元信息 + diff 一起喂给 Claude Code
  claude -p "请审查下面这个 PR：

PR 元信息（JSON）：
\`\`\`json
$pr_meta
\`\`\`

完整 diff：
\`\`\`diff
$diff
\`\`\`

按以下结构输出审查结果（结构同本地审查模板）：
1. 变更概览
2. 分文件审查（带严重度 Critical/Major/Minor/Nit）
3. 推荐动作

要求引用 PR 标题/描述里声明的目标，判断改动是否对得上 PR 的初衷。
"
}

# 粒度 1：作为整段 PR 评论回写（最常用，gh pr comment 原生支持）
review_pr 123 | tee /tmp/review.md
gh pr comment 123 --body-file /tmp/review.md

# 粒度 2：走 review 体系（带 approve / request-changes 状态）
gh pr review 123 --comment --body-file /tmp/review.md     # 只评论不阻拦
gh pr review 123 --request-changes --body "见上文 Critical 项"  # 显式 request changes
gh pr review 123 --approve --body "LGTM, ship it"          # 通过

# 粒度 3：行级 inline 评论（gh CLI 原生不支持，需要 gh extension 或 gh api 直调）
gh extension install agynio/gh-pr-review   # 一次性安装，提供 inline 评论

claude -p "审查 git diff main...HEAD，只输出 JSON，不要任何解释文字。schema 如下：
{
  \"summary\": \"一句话变更说明\",
  \"risk\": \"low|medium|high\",
  \"findings\": [
    {
      \"file\": \"src/foo.ts\",
      \"line_start\": 12,
      \"line_end\": 18,
      \"severity\": \"critical|major|minor|nit\",
      \"category\": \"bug|security|perf|style|test|docs\",
      \"message\": \"问题描述\",
      \"suggestion\": \"具体改法\"
    }
  ]
}" | jq '.findings | map(select(.severity == "critical")) | length'
# 输出 0 → 通过；>0 → CI 退出非 0