用途
code-review 是一项代码审查技能,定位在“实现变更之后、交付之前”这一环节,用于验证自己的工作是否达到可交付标准。它把审查过程拆解为可执行的清单与步骤,帮助使用者在提交或交付前发现并修复问题。
主要能力
该技能提供一份四维度的审查清单:
- 正确性:变更是否解决了原始问题/任务;是否对现有功能产生非预期副作用;边界情况是否处理;错误处理是否恰当(而非过度)。
- 代码质量:代码是否符合既有风格与模式;是否存在不必要的复杂度或抽象;变量与函数命名是否清晰;是否遗留死代码、被注释掉的代码或 TODO。
- 测试:新功能是否有测试覆盖;既有测试是否仍然通过;测试是否同时覆盖正常路径与错误情况;测试是否脆弱(是否在测试实现细节)。
- 安全性:是否存在硬编码的密钥或凭据;用户输入是否在边界处校验;是否存在 SQL 注入、XSS 或命令注入风险;文件操作是否使用安全路径。
此外,技能附带辅助脚本 /skills/code-review/lint_check.py,用于扫描 Python 文件中的缺失文档字符串、长函数(超过 50 行)以及裸 except: 子句。
使用方式
技能给出了明确的执行流程:
- 端到端阅读每个被修改的文件(不只是看 diff)。
- 运行测试套件:execute("python -m pytest -v")。
- 如可用则运行 linter:execute("ruff check .")。
- 运行内置 lint 检查:execute("python /skills/code-review/lint_check.py .")。
- 对照审查清单逐项检查。
- 若发现问题,修复后重新审查。
- 全部通过后,审查完成。
辅助脚本的调用方式为:execute("python /skills/code-review/lint_check.py [path ...]")。
适用场景
适用于在完成代码修改后、交付或提交前进行自检,尤其适合需要同时兼顾正确性、风格一致性、测试覆盖与安全性的开发流程。
限制
资料未说明该技能支持的语言范围(除辅助脚本针对 Python 外)、是否集成到特定 IDE 或 CI 系统、是否有收费或价格信息,这些内容暂未提供。