yx-commands-review

name	yx-commands-review
allowed-tools	Bash(yx-code diff:*), Bash(yx-code --help:*), Bash(git branch:*), Bash(git remote:*), Bash(git log:*), Bash(git show:*), Bash(git blame:*), Read, Grep, Glob
description	查看分支代码差异并进行代码审查

Context

• Current branch: !git branch --show-current
• Remote: !git remote -v
• Recent commits: !git log --oneline -5

Your task

对当前分支与目标分支之间的代码差异进行全面的代码审查。

步骤 1: 获取代码差异

首先使用 yx-code 命令获取差异：

yx-code diff --target <target> --source <source>

步骤 2: 展示代码差异

将 yx-code diff 的完整输出直接展示给用户，包括：

• 变更统计摘要（文件数、新增行数、删除行数）
• 每个变更文件的完整 diff 内容

保留原始的 diff 代码块格式，让用户能直接阅读代码差异。

步骤 3: 收集上下文信息

使用轻量级 agent 收集以下信息：

1. 获取 CLAUDE.md 文件列表：查找项目根目录及变更文件所在目录的 CLAUDE.md 文件
2. 获取变更摘要：分析差异内容，返回变更的简要总结
3. 检查审查资格：确认分支是否需要进行代码审查（排除已关闭、草稿、自动生成等）

步骤 4: 并行代码审查

启动 5 个并行 agent 独立进行代码审查：

Agent #1: CLAUDE.md 合规审查

• 检查变更是否遵循项目 CLAUDE.md 中的规范
• 注意 CLAUDE.md 是 Claude 编写代码的指南，审查时不一定所有条款都适用
• 返回问题列表及原因（CLAUDE.md 合规性）

Agent #2: Bug 浅层扫描

• 专注于变更本身，不额外读取上下文
• 关注明显的 Bug，避免小问题和吹毛求疵
• 忽略可能的误报
• 返回问题列表及原因（潜在 Bug）

Agent #3: 历史上下文审查

• 读取被修改代码的 git blame 和历史
• 基于历史上下文识别潜在问题
• 返回问题列表及原因（历史上下文）

Agent #4: 相关 MR 审查

• 查找之前修改过这些文件的 MR
• 检查之前 MR 的评论是否适用于当前变更
• 返回问题列表及原因（历史 MR 上下文）

Agent #5: 代码注释合规审查

• 读取被修改文件中的代码注释
• 确保变更遵循注释中的指导
• 返回问题列表及原因（代码注释合规性）

步骤 5: 问题置信度评分

对步骤 4 发现的每个问题，启动并行轻量级 agent 进行置信度评分：

评分标准（0-100 分）：

分数	含义
0	完全不确信。这是经不起推敲的误报，或是已存在的问题
25	有些确信。可能是真正的问题，但也可能是误报。如果是风格问题，相关 CLAUDE.md 中未明确指出
50	中等确信。验证了这是真正的问题，但可能只是小问题或不常发生。相对于整个 MR，不太重要
75	高度确信。验证了这是很可能在现实中发生的真正问题。当前方案不充分。问题很重要，直接影响功能，或在 CLAUDE.md 中明确提及
100	绝对确定。验证了这是肯定会在现实中频繁发生的真正问题。证据直接确认

步骤 6: 过滤并输出结果

1. 过滤问题：仅保留评分 >= 80 的问题
2. 重新检查资格：确认分支仍符合审查条件
3. 输出审查报告

使用方式

/yx-commands-review [-t target] [-s source]

参数：

• -t, --target: 目标分支（默认: develop）
• -s, --source: 源分支（默认: 当前分支）

示例

查看与 develop 的差异：

/yx-commands-review

查看 feature 分支与 main 的差异：

/yx-commands-review -t main -s feature-branch

输出格式

如果发现问题：

---

MR Code Review

发现 N 个问题：

1. <问题简要描述> (CLAUDE.md 规定: "<相关条款>")

<文件链接，包含完整 sha1 和行范围>

2. <问题简要描述> (<其他原因>)

<文件链接>

---

如果没有发现问题：

---

MR Code Review

未发现问题。已检查 Bug 和 CLAUDE.md 合规性。

---

链接格式要求

链接到代码时使用以下格式：

https://<云效域名>/<项目路径>/blob/<完整sha1>/<文件路径>#L<起始行>-L<结束行>

要求：

• 必须使用完整的 git sha（不能用 HEAD 等引用）
• 文件名后使用 # 符号
• 行范围格式: L[起始]-L[结束]
• 至少包含上下文各 1 行（如评论第 5-6 行，应链接 L4-L7）

误报示例（审查时排除）

• 已存在的问题（非本次变更引入）
• 看起来像 Bug 但实际不是
• 资深工程师不会提出的吹毛求疵
• Linter/TypeChecker/Compiler 会捕获的问题（导入、类型错误、格式问题）
• 一般代码质量问题（测试覆盖、一般安全问题、文档）
• CLAUDE.md 中提及但代码中明确静默的问题（如 lint ignore 注释）
• 可能是有意的功能性变更
• 真正的问题但不在用户修改的行上

注意事项

• 需要在 git 仓库目录中运行
• 目标分支需要存在于远程
• 不检查构建信号，不尝试构建或类型检查
• 大量变更时输出可能较长
• 对于复杂文件，优先使用 yx-code diff 获取准确差异