← ClaudeAtlas

git-pr-reviewlisted

当用户明确要求“review 某个 GitHub PR”“评估某个 pull request 是否值得 merge”“帮我判断这个 PR 怎么处理”时使用。基于用户提供的 GitHub 仓库地址、PR 编号/链接和补充说明,进行只读、证据驱动的 PR 审查:理解 PR 解决的问题、评估方案优劣与局限、默认优先使用内置“好 PR”标准并在必要时联网补充、识别恶意或高风险改动,并输出是否建议 merge 的 Markdown 决策报告。⚠️ 不适用:用户要你直接修改 PR 代码、直接 merge PR、或在本地执行 PR 分支中的不可信代码。
huangwb8/skills · ★ 37 · Code & Development · score 86
Install: claude install-skill huangwb8/skills
# Git PR Review ## 与 bensz-collect-bugs 的协作约定 - 因本 skill 设计缺陷导致的 bug,先用 `bensz-collect-bugs` 规范记录到 `~/.bensz-skills/bugs/`,不要直接修改用户本地已安装的 skill 源码;若有 workaround,先记 bug,再继续完成任务。 - 只有用户明确要求“report bensz skills bugs”等公开上报时,才用本地 `gh` 上传新增 bug 到 `huangwb8/bensz-bugs`;不要 pull / clone 整个仓库。 面向“**帮助用户决策如何处理某个 GitHub PR**”的只读评审技能。 ## 核心原则 - **只读优先**:默认只能读取 GitHub 页面、API 返回、diff、评论、CI 状态、相关 issue/文档;不要修改源代码。 - **绝不主动 merge**:除非用户明确要求,否则不要执行 merge、rebase、squash、approve、request review 等操作。 - **绝不执行不可信 PR 代码**:不要 `gh pr checkout`、不要运行 PR 分支脚本、不要安装 PR 引入的依赖、不要触发可疑 CI/CD。 - **中间文件隔离**:所有中间文件必须保存在工作目录下的隐藏目录 `.git-pr-review/`;若用户另有指定,才使用用户指定目录。 - **证据驱动**:所有结论都要回到证据,明确��用 diff、评论、CI、issue、社区资料或官方文档。 - **合规不忽略**:如果 PR 触及依赖、vendored 代码、复制粘贴第三方内容、资源资产或许可证文件,必须显式审查 license 风险与兼容性。 ## 你需要确认的输入 1. `github_repo`(必需) - GitHub 仓库根地址或 `owner/repo`,例如 `https://github.com/owner/repo` - 不接受 `issues/`、`pull/`、`tree/` 这类子页面 URL 2. `github_pr`(必需) - PR URL、`#123`、`123` 或 `pr-123` 3. `extra_instructions`(可选) - 用户已有判断、关注点、禁区、参考材料、团队背景 4. `review_count`(可选) - 独立评审次数,默认 5;用户明确指定时以用户要求为准 - 当你调用 `build_parallel_review_plan.py` 时,把它映射为 `--n <review_count>` 5. `workspace_dir`(可选) - 默认 `.git-pr-review/` 6. `report_dir`(可选) - 默认当前工作目录(项目根) ## 标准工作流 ### 1. 初始化隔离工作区 优先使用确定性脚本创建本次评审目录与建议输出文件名: ```bash python3 git-pr-review/scripts/prepare_review_job.py \ --repo "https://github.com/owner/repo" \ --pr "https://github.com/owner/repo/pull/123" ``` 脚本会: - 校验仓库地址与 PR URL 是否属于同一 GitHub 仓库