code-review-and-quality

技能简介

这是一个多维度代码审查技能，在合并前对每次变更执行质量检查。它从正确性、可读性、架构、安全和性能五个方面评估代码，帮助开发者或 Agent 发现潜在问题。审批标准是：只要变更确实改善整体代码健康度，即使不完美也应通过。

使用场景

• 合并任何 PR 或变更之前
• 完成功能实现后，自查代码
• 评估其他 Agent、模型或人类编写的代码
• 重构现有代码时
• 修复 Bug 后，审查修复代码及回归测试

使用方法

本技能用于支持 Skill 的 Agent 环境；将技能文件放入 Skills 目录或通过 Agent 配置加载即可。在审查场景中，可要求“对当前改动进行代码审查”，技能会按五个维度给出结论：

• 正确性：是否满足需求、边界与错误处理、测试是否有效。
• 可读性：命名清晰、控制流简单、消除死代码和过度设计。
• 架构：模块边界、依赖方向、是否真的降低了复杂度或只是转移了复杂度。
• 安全：输入校验、密钥保护、参数化查询、XSS 防御、依赖可信度。
• 性能：N+1 查询、无限循环、同步阻塞、缺失分页、多余渲染。

发现结构性问题时，技能会给出具体重构方向，例如用类型模型替代条件链、拆分重复分支、把特性逻辑移出共享模块、显式化类型边界等。

注意事项

• 不要因为代码不是“自己会那么写”就阻塞变更；只要符合项目约定并改善代码库，应批准。
• 优先选择删减或简化结构的方案，而不是制造新的抽象来重新分布复杂度。