关于 user-story-reviewer
user-story-reviewer 是用于独立审查单个故事 PR 的技能,面向代码审查者。它要求将审查与实现分离,并依据验收标准、证据和仓库策略作出审查与合并判断。
主要功能
- 01核对 PR 元数据与仓库规则
- 02逐项验证验收标准
- 03独立运行重点验证
- 04检查正确性、安全性与兼容性
- 05按策略审批或合并 PR
- 06记录审查证据并交接
适用场景
- 01实现者提交故事 PR 后进行独立审查
- 02检查变更是否满足验收标准
- 03在审批或合并前复核 PR 状态
- 04向协调者提交审查结论与阻塞项
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ eho/ agent-skills --skill user-story-reviewer安装后在 Claude Code 中输入/user-story-reviewer确认已加载。
附带文件scripts/approve_or_merge_pr.shtests/approve_or_merge_pr_test.sh
详细介绍
用户故事评审员
独立于实现者,准确评审一个用户故事 PR。只进行评审;将所有代码更改交还实现者,确保下一次评审仍然独立。
评审
- 阅读仓库说明、完整议题、验收标准、依赖项、以往发现和仓库合并政策。
- 检查 PR 元数据、目标分支、当前头部 SHA、草稿状态、提交、变更文件、检查项、可合并状态、关联议题和现有评审。
- 阅读差异和相关上下文代码。
- 根据具体实现和验证证据,检查每一项验收标准。
- 视情况评审正确性、回归问题、测试、失败路径、安全性、权限、数据安全、兼容性、迁移、并发、性能、诊断、运维和文档。
- 针对变更行为和最高风险,独立开展有针对性的验证。除非仓库政策或风险要求重新运行,否则复用与当前头部准确对应且可信的全面测试结果。
- 工具可用时,在适当的运行环境中体验有意义的用户可见行为;仅有自动化测试不足以证明视觉或交互验收标准已满足。
- 报告阻塞性问题,明确指出位置、错误行为、影响和所需更改。
任何推送的提交都会使签署失效。请在全新的独立上下文中评审新的头部。
决策和合并
- 仍存在任何验收、正确性、安全性、兼容性或重要测试可信度问题时,选择
Request changes。 - 所有标准和必要检查均通过,且允许正式批准时,选择
Approve。 - 实现没有问题,但当前 GitHub 身份无法正式批准时,选择
Comment only。 - 仅当仓库政策授权且所有前提条件均满足时,才选择
Merge。
批准或合并前,立即确认 PR 仍处于打开状态、不是草稿、基于预期分支、位于准确评审过的头部、可合并,并符合必要检查和评审政策。
解析并使用 scripts/approve_or_merge_pr.sh,以绑定提交的方式批准或合并。检查其 --help,并传入预期头部以及仓库授权的直接合并方式或队列模式。不要启用异步自动合并。
此辅助脚本会区分必要检查失败或待处理与 GitHub API/身份验证错误。如果 GitHub 明确报告该分支没有检查项或没有必要检查项,脚本会继续执行由服务器强制执行的合并门槛;不会将此状态视为 CI 通过或身份验证受阻。
合并后,重新读取 PR 和议题。报告实际的合并和关闭状态,不要假定命令已成功。
评审记录和交接
在 PR 上发布详细评审:
## Findings
- <blocking findings or "No blocking findings found.">
## Acceptance Criteria
| Criterion | Evidence | Result |
| --- | --- | --- |
## Verification
- Commands/checks:
- Not verified:
- Residual risk:
## Decision
Decision: Request changes | Approve | Comment only | Merge
采取评审操作后,仅向协调者返回以下简短交接内容:
- Issue:
- PR:
- Head SHA:
- Result:
- Acceptance-criteria evidence:
- Verification:
- Blocking findings:
- Merge result:
评论
支持 Markdown