关于 coding-review-simplify
coding-review-simplify 是供实现者和评审者使用的技能,用于在不改变行为的前提下简化已完成的代码差异。它要求删除前先核验证据,并将评审限制在具体问题内。
主要功能
- 01保持行为的代码差异简化
- 02删除前进行证据核验
- 03按角色执行实现或评审
- 04改动后运行针对性验证
- 05分析复杂边界中的共因关系
适用场景
- 01评审已完成改动中的具体缺陷
- 02核对调用方和行为后删除代码
- 03简化跨边界或结构复杂的差异
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ robsonrung/ rar-skills --skill coding-review-simplify安装后在 Claude Code 中输入/coding-review-simplify确认已加载。
包含的文件
除 SKILL.md 外,安装时一起下载的参考资料和脚本。
references/team-checklists.mdreferences/personas/code-quality-reviewer.mdcode-reuse-reviewer.mdefficiency-reviewer.md详细介绍
编码审查简化
先选择分支,再选择模型。协调工作和确定性命令在调用授权范围内执行。在指定的实现者或审查者上下文中应用此方法。实现者执行已授权的清理;独立审查者保持只读。此方法不会派遣工作者。要派遣工作者,请使用 shared/references/model-preview.md 及其中唯一获批的路由快照。嵌套技能和运行器使用该快照,无需再次选择模型或工作者。
趁实现上下文还新鲜,收紧已完成的差异。结果应是更小、更连贯的形态,或有证据支持的具体问题。下一个使用者是获批的审查计划。
这是一次保持行为不变的审查,不会创建新的审查路线。获批的路由计划决定之后是否需要进行聚焦审查、边界审查或深入审查。
工作流
- 阅读任务验收约定、已更改的文件、差异和已记录的验证结果。从已更改的范围入手,仅在跨越实际边界时扩展检查范围。
- 根据预期的可观察行为检查变更。只有在可能导致错误、回归、维护陷阱、数据风险、验证不足或令读者明显困惑时,才报告问题。
- 仅在结果仍保持行为不变时进行简化。除非有证据证明以下检查已无用,否则应保留信任边界验证、授权检查、不变量断言、编码和无障碍保障。删除前,检查受支持的调用方和入口,包括运行时加载情况。必须有证据证明删除不会改变副作用、失败行为和必要的保障措施,并进行相关验证。调用可达性未知或缺少保持行为不变的证据时,不得删除。
- 如果差异新增了辅助函数、改变了所有权,或影响了高频或异步路径,则在当前角色中针对相关审查问题进行检查。仅针对该问题阅读
references/personas/中匹配的角色说明。存储状态或异步行为发生变化时,应用data-systems-coding-lens。只有存在重大风险或未解决的问题时,才通过获批路线派遣独立专家。 - 仅当差异跨越边界或显得纠缠不清时,才执行 Connascence Pass。识别彼此共变的部分,然后说明其强度、局部性、度数和最小安全补救措施。如果提取会增加间接层且不能提高安全性,就保留局部静态耦合。
- 在当前角色中,针对边界范围内的设计问题应用相关视角。不要借此启动更广泛的重构。
- 将每个已接受的问题转化为具体的修复、测试、检查或明确的后续事项。仅重新运行受已实施变更影响的验证。
简化提示
使用 clean-code 术语。留意没有做出决策的辅助函数、掩盖了别扭约定的类型、不必要的记忆化或副作用、没有调用方的兼容代码,以及无关的差异改动。好名称可能胜过包装器。如果概念可能分化,重复代码可以保留。
输出
仅审查工作中,按严重程度列出问题,并注明位置、后果、最小的有效修复方案和验证结果。
实现工作中,返回:
- 结果:无需变更即可安全、已简化、需要针对性修复,或需要升级设计评估。
- 已做的变更。
- 已记录的验证结果。
- 剩余风险。
仅在交接或部署前,才使用 references/team-checklists.md。不要以删除的行数衡量成效;衡量标准是:在保持可观察行为不变的同时,让变更更简单。
评论
支持 Markdown