关于 code-review
面向 Langfuse 变更审查的代码审查技能，供审查者使用，而非实现功能。它优先检查正确性、回归、安全、性能和有效测试覆盖，并按严重程度和位置报告发现。
主要功能
- 01读取仓库审查规范
- 02优先检查正确性与回归
- 03检查安全和租户隔离
- 04评估性能及高风险测试缺口
- 05按严重程度和位置报告问题
适用场景
- 01合并前审查代码变更
- 02评估集成中的安全风险
- 03检查后端变更的租户隔离
- 04判断高风险变更是否需要测试
安装
复制下面的命令，在终端中运行。
$
npx skills add https:/安装后在 Claude Code 中输入/code-review确认已加载。
附带文件references/review-checklist.md
详细介绍
代码审查
当任务是审查代码变更而非实现功能时，使用此技能。
从这里开始
- 阅读
references/review-checklist.md，了解仓库的规范审查规则。 - 阅读根目录
AGENTS.md以及被审查文件所在目录中最近的AGENTS.md。 - 如果审查涉及 ClickHouse，还要使用共享的
clickhouse-best-practices技能。 - 如果变更涉及后端代码，在适用时还要使用共享的
backend-dev-guidelines技能。 - 如果变更接受用户提供的 URL、添加出站 HTTP、引入新集成、涉及密钥、RBAC、重定向处理、产品分析、浏览器监控或会话回放，还要使用共享的
security-review技能。在签核前运行其references/checklist.md。
审查优先级
重点关注：
- 正确性缺陷
- 行为回归
- 安全和租户隔离风险
- 具有实际影响的性能问题
- 高风险变更中缺失或薄弱的测试
- 在认定覆盖不足之前，先明确每个建议测试能捕获哪种独特回归。不要要求添加唯一断言只是重复 diff 内容（例如间距值、标签）的测试，因为这会增加文件，却无法证明任何问题。优先合并到最接近的现有测试套件中。若多层重复测试同一判定逻辑，除非每项测试分别验证不同的传输、投影或执行边界，否则应指出这一点。
输出要求
- 先列出发现的问题，并按严重程度排序
- 每项发现都要提供文件和行号引用
- 仅在发现问题之后提供简短总结
- 如果没有发现问题，请明确说明，并指出任何残余风险或覆盖缺口
范围指导
使用 references/review-checklist.md 检查 Langfuse 特有事项，例如：
- ClickHouse 和 Postgres 迁移要求
- 项目级租户隔离检查
- Prisma 记录不存在检查（
isPrismaRecordNotFoundError/isRecordNotFoundError，而不是内联的P2025检查） - API/Fern 一致性
- 横幅偏移下的 UI 定位
- 环境变量访问模式
不要在临时提示词或特定工具的命令文件中重复这些规则。
评论
支持 Markdown