code-review
Installation
SKILL.md
对 HEAD 与固定点之间的 diff 分别做标准审查和任务要求审查:
- 标准:代码是否遵守本仓库已记录的编码标准?
- 任务要求:代码是否忠实地实现了原始 issue?
审查是只读的:不编辑文件、不提交、不跑构建或测试。发现的问题报出来,修复交给调用方。
流程
1. 确定审查目标
从参数按这个顺序认定固定点:
- PR/issue 引用(
#473、GitLab!67):按docs/agents/issue-tracker.md的流程拉取,PR 描述留给步骤 2 当任务要求,固定点取 PR 的 base 分支(origin/<base>)。 - 明确的 git 引用:commit SHA、分支名、tag、
main、HEAD~5、区间(main...my-feature)。 - 什么都没给:用
@{upstream}(上游跟踪分支)。 - 认不出,或没有上游跟踪:问用户。
把 diff 命令一次性记下来:git diff <fixed-point>...HEAD(三点,比较的是 merge-base),外加 git diff HEAD 覆盖已跟踪文件的未提交改动,再用 git status --short 找未跟踪文件(它们整体都是新改动,直接读全文)。没提交的改动同样算审查范围,除非用户明确只审已提交内容。同时记下 commit 列表:git log <fixed-point>..HEAD --oneline。