审查代码变更或仓库中的过度设计与不必要复杂度。用户要求简化 review、审查过度设计、检查多余抽象或防御代码、判断实现是否超出当前需求,或询问还能删什么时使用。只报告可删除、复用、合并或缩减的内容;不代替正确性、安全、性能审查,也不直接修改代码。
Resources
1Install
npx skillscat add timwongup/codex-orchestration/simplicity-review Install via the SkillsCat registry.
SKILL.md
简化审查
在用户指定的 diff、文件或仓库范围内进行只读审查。目标是在不改变当前合同和必要保护的前提下减少不必要复杂度。
审查顺序
按以下顺序检查,前一项已经解决问题时不继续设计新的替代结构:
- 删除:没有当前需求、调用方或有效行为的代码、配置、兼容逻辑和未来脚手架。
- 复用:仓库已有代码、既有模式、标准库、平台能力或现有依赖能够满足相同合同。
- 合并:单一实现的接口、只有一处调用的转发层、重复 helper、重复状态或分散在多个调用方的同类补丁。
- 缩减防护:没有当前合同、真实信任边界、具体风险或测量依据的哈希、校验和、缓存、重试、兼容处理、额外状态和重复校验。
- 缩减测试:没有独立行为保护价值的重复测试,以及仅为单一变化新增的测试框架、通用 fixture 层、测试基类、mock 基础设施、快照体系、参数矩阵或逐函数测试套件。
判断边界
- 每条发现都必须指出具体位置、当前缺少的需求或风险依据,以及能保留等价行为的最小替代方案。
- 不把个人风格、行数、文件数量或抽象偏好单独当作问题。
- 不建议删除真实信任边界的输入验证、安全与数据完整性要求、防止数据丢失的错误处理、可访问性、用户明确要求、项目门禁,或具有独立回归保护价值的测试。
- 现有机制已经提供保证时,可以建议删除内部重复保护;证据不足时说明不确定性,不把推测写成发现。
- 只报告与指定范围直接相关的内容,不借审查扩展为通用重构。
输出
按预期减少的复杂度从高到低列出发现。每项包含:
- 位置
- 类别:删除、复用、合并、防护或测试
- 当前复杂度为什么没有必要
- 最小替代方案
- 保留的行为或保护
没有实质发现时,直接说明当前范围内没有值得处理的简化项。不要为了凑数提出建议。