code-review 是这份来自 Sentry 的代码审查 Skill。你拿到一组待合入的改动,想找实际风险而非争论个人写法,可以按它的清单检查。
它关注运行错误、查询性能、意外行为、兼容性、权限和秘密暴露,再看组件关系是否符合项目架构。测试也要覆盖真实要求、交互和关键用户路径,不只是证明某段代码存在。
涉及数据库结构、接口约定、新框架或敏感功能时,原文建议交给资深工程师进一步看。反馈说明可操作的改法,不确定时提问;只剩轻微问题可以认可,不用风格偏好阻挡合入。
这些是 Sentry 工程实践下的评审方向,要结合自己的仓库要求;本文未读外链完整制度,不据此声称适用所有团队,也没有实际给任何代码批准。