Code Review 变更代码评审技能

这是一款专注于"发布前审查改动代码"的 Agent 技能，帮助开发者在提交或发起 Pull Request 之前，系统性地排查安全漏洞、性能退化与项目规范问题。它通过 git diff 定位本次改动涉及的文件，再按结构化清单逐项审查，最终给出带严重等级与修复建议的评审结论。对于希望把代码质量前移到本地开发环节、减少线上事故与返工的团队来说，它是一道轻量却可靠的"自动预审关卡"。

核心能力

• 变更范围识别：基于 git diff 工作流自动找出本次修改涉及的文件，聚焦真正需要评审的代码。
• OWASP Top 10 安全审查：覆盖注入、越权、敏感数据暴露等常见 Web 安全风险，提前拦截高风险写法。
• 数据库性能检查：审查 SQLAlchemy 用法，识别 N+1 查询、缺失索引、低效加载等性能隐患。
• 逻辑与边界问题排查：检查逻辑错误、边界条件遗漏与异常处理缺失。
• 规范一致性对比：当项目存在编码规范时，将改动代码与既有标准对照，指出偏离之处。
• 分级结论输出：以严重等级呈现发现的问题，并附上可执行的修复建议，方便快速决策。
• 本地优先、可追溯：全程基于本地改动运行，评审结论可直接用于 PR 描述与自我复查。

安装配置

1. 安装技能：在 Claude、Codex 或 Claude Code 中执行 npx skillstore add daviddworetzky/code-review。
2. 准备 Git 仓库：技能依赖 git diff 读取改动，需在有效的 Git 工作区内使用。
3. 确保改动已暂存或提交：明确本次要审查的改动范围（工作区改动或目标分支差异）。
4. 准备项目规范（可选）：若仓库内存在编码规范文档，技能会据此做一致性对比，建议提前说明。

使用步骤

1. 提交前评审：在本地改动尚未提交时，让 Agent 审查变更，尽早发现缺陷与安全风险。
2. PR 准备：在发起 Pull Request 前检查目标分支，生成清晰、可执行的评审结论。
3. 聚焦安全：对涉及认证、权限、数据处理的改动，重点要求按 OWASP Top 10 审查。
4. 性能专项：对涉及数据库访问的改动，要求检查 N+1、索引缺失与加载效率。
5. 汇总报告：让 Agent 输出带严重等级与修复建议的清单，用于决定是否放行。

适用场景

• 本地提交前自查：在 commit 之前拦截 bug 与安全隐患，降低返工成本。
• Pull Request 预审：在人工评审前完成一轮自动化检查，提高评审效率。
• 安全敏感模块改动：对涉及鉴权、支付、用户数据的代码做重点安全审查。
• 数据库相关重构：在 ORM 用法调整后核验查询效率与索引正确性。
• 团队规范落地：用统一的评审口径推动编码规范在改动中落地。

适用人群

• 需要频繁提交代码的后端与全栈开发者
• 负责合并 PR、把关代码质量的 Tech Lead 与评审人
• 关注应用安全与数据库性能的工程团队

工作原理

技能以 git diff 为入口确定变更文件，随后加载一套结构化评审清单：安全层面对标 OWASP Top 10，性能层面分析 SQLAlchemy 的查询模式，逻辑层面对照边界与错误处理，规范层面则与项目既有标准比对。所有发现按严重程度分级并附修复建议，形成一份可直接用于决策的评审报告，且全程在本地运行，改动与数据不出本机。

官方链接

• 技能主页：https://skillstore.io/skills/daviddworetzky-code-review
• 源码仓库：https://github.com/DavidDworetzky/Geist/tree/main/.claude/skills/code-review
• 许可证：MIT