中文代码审查规范助手是面向软件开发团队设计的AI技能，提供系统化的代码审查流程和标准化审查清单。技能内置缺陷分类体系，支持按严重程度（致命/严重/一般/建议）对代码问题进行分级标注，同时提供针对性的改进建议和沟通话术，帮助审查双方高效协作，提升代码审查质量和团队技术积累。
中文代码审查规范助手：分级标注与高效沟通的AI审查流程
【适用场景】
场景一：功能代码合并审查
开发工程师完成一个功能模块开发后，提交 Pull Request（PR）进行代码审查。使用本技能可对 PR 中的代码变更进行系统化审查，按严重程度分级标注发现的问题（如空指针风险、逻辑漏洞、安全隐患等），并给出具体的修改建议，确保合并到主分支的代码质量达标。场景二：重构代码安全审查
在进行代码重构（如技术债务偿还、架构升级）时，需要评估重构方案对现有系统的影响。技能可辅助审查重构变更，识别潜在风险点，并提供重构前后的对比分析，帮助团队做出是否推进重构的决策。场景三：新成员代码风格指导
团队新成员提交的代码往往存在风格不统一的问题。通过技能审查新成员的代码，既能保证代码质量，又能为新成员提供具体、可操作的改进建议，加速其融入团队编码规范。场景四：安全与合规专项审查
针对安全敏感模块（如用户认证、支付、权限控制等），需要进行专项安全审查。技能内置安全审查清单，可识别常见安全漏洞（如 SQL 注入、XSS 跨站脚本、敏感信息泄露等），并提供修复建议，确保代码符合安全规范。【操作步骤】
Step 1：提交代码变更与上下文
在发起代码审查前，准备好以下信息并提交给技能： - 代码变更内容：需审查的代码或 Pull Request 链接 - 变更背景：本次修改的目的和主要变更点 - 自测情况：本地测试或自动化测试的通过情况 - 重点关注：是否有特别需要审查关注的领域（如性能、安全等）Step 2：执行系统化审查
技能对提交代码进行系统化审查，按照标准审查清单逐项检查： - 正确性：逻辑是否正确、边界条件是否处理 - 安全性：是否存在安全漏洞、敏感信息是否泄露 - 性能：是否存在性能瓶颈、大数据量处理是否优化 - 可读性：命名是否清晰、注释是否充分、函数长度是否合理 - 测试覆盖：是否有遗漏的测试场景Step 3：获取分级审查报告
技能输出包含以下内容的审查报告： - 问题列表：按严重程度分级（致命/严重/一般/建议） - 具体位置：问题代码的行号和上下文 - 修改建议：针对每个问题的具体改进方案 - 沟通话术：审查意见的礼貌表达方式Step 4：修复与复核
开发工程师根据审查报告修复问题后，可将修复后的代码再次提交复核。技能对比修复前后的变更，确认问题是否真正解决，避免"修一个问题引出三个新问题"的情况。【代码模板】
```markdown
代码审查报告
提交人：[开发者姓名] 审查时间：[日期] 变更摘要：[简要描述本次修改内容]
审查结论
| 严重程度 | 数量 | 状态 |
|---|---|---|
| 致命 | X | [待修复/已修复] |
| 严重 | X | [待修复/已修复] |
| 一般 | X | [待修复/已修复] |
| 建议 | X | [待修复/已修复] |
详细问题列表
致命问题
[编号] [问题标题] - 文件：[文件名]:[行号] - 问题描述：[具体描述] - 修改建议：[建议的修复方案]
严重问题
[编号] [问题标题] - 文件：[文件名]:[行号] - 问题描述：[具体描述] - 修改建议：[建议的修复方案]
本报告由 dev-skills-zh 代码审查技能生成 ```
【复盘要点】
代码审查质量三问： 1. 全面性：审查是否覆盖了代码的正确性、安全性、性能、可读性等关键维度？避免只关注功能正确性而忽略其他维度。 2. 建设性：审查意见是否具体、可操作？"这段代码不好"不是审查，"建议将这个循环内的重复计算提取到循环外，可将时间复杂度从 O(n²) 降至 O(n)"才是有效审查。 3. 时效性：审查反馈是否及时？等待超过 24 小时的审查会严重拖慢开发节奏，建议建立 SLA 机制。
常见失败模式： - 审查变成"挑刺大会"，忽视代码的整体质量和开发者的努力 - 审查意见过于笼统（如"代码风格不好"），缺乏具体的修改建议 - 审查人员和开发人员陷入对立情绪，影响团队协作氛围
优化建议： 建立团队代码审查规范（包括审查清单、严重程度定义、沟通话术），本技能可作为审查辅助工具。定期统计审查发现的高频问题类型，针对性开展团队培训，从源头减少共性问题。
来源：GitHub Yangge521/dev-skills-zh https://github.com/Yangge521/dev-skills-zh/blob/master/skills/engineering/code-review-cn/SKILL.md