关于 simplify
用于质量门禁的 skill,依据需求和项目规范审查任务实现。它将发现记录在任务文档中,并在进入测试前给出 PASS 或 FAIL。
主要功能
- 01仅审查任务范围和变更文件
- 02核对需求与验收标准
- 03按严重程度分类偏差
- 04在任务文档中记录持久发现
- 05返回 PASS 或 FAIL
适用场景
- 01在测试前检查实现
- 02标记范围扩张或重大偏差
- 03审查没有任务文档的小改动
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ eljun/ claude-workflow-skills --skill simplify安装后在 Claude Code 中输入/simplify确认已加载。
详细介绍
simplify - 质量门禁阶段
不变原则
- 仅审查任务范围和已更改的文件。
- 不得增加新的产品范围。
- 在测试前阻止重大偏差。
- 将需要长期留存的发现写入任务文档。
- 如果任务文档包含
Implementation Notes一节,但TASKS.md中的任务行仍显示为Planned或In Progress,则继续操作前将该行状态更新为Testing——实现已运行,但遗漏了任务跟踪器的更新。如果没有Implementation Notes,则不要更新状态;这表示实现实际上并未运行,应告知用户。
运行时适配器可以将此阶段呈现为斜杠命令、菜单操作或自然语言技能调用。可移植的阶段名称为 simplify。
工作流
- 阅读
AGENTS.md。 - 确定任务 ID,并阅读
docs/task/{ID}-{task-name}.md。 - 根据
Implementation Notes和git diff --name-only确定已更改的文件。 - 仅阅读已更改的文件。
- 检查编码标准和可维护性。
- 对照需求、文件变更、范围外边界和验收标准检查实现。
- 对偏差进行分类。
- 将
Quality Gate Notes追加到任务文档。 - 返回 PASS 或 FAIL。
没有任务文档
对于用户直接提出的小改动,且没有 docs/task/ 文档(快速修复或润色):
- 根据用户请求审查相对于基准分支的差异(
git diff),而不是任务文档。 - 应用相同的标准检查清单和偏差等级。“范围”即用户提出的需求。
- 将结果(
PASS/FAIL、发现和偏差)记录在拉取请求正文或提交消息中,而不是新建任务文档。 - 除非项目要求,否则不要为此创建
TASKS.md任务行。
标准检查清单
在适用时检查:
- 没有未使用代码、死代码或注释掉的实现
- 项目有类型化替代方案时,不使用宽泛的
any或无类型的逃逸机制 - 避免深层嵌套;如有必要,使用卫语句以提高可读性
- 函数和文件职责明确
- 名称能准确描述行为
- 重复逻辑会增加维护风险时,应提取复用
- 有意处理错误
- 生产路径中没有密钥、凭据或调试日志
- 有意更新断言旧行为或旧措辞的测试,并添加说明原因的注释,绝不可直接删除
- 新增或更改的测试应尽可能检查行为或输出;仅断言源文本或结构的测试应标记为证据薄弱
- 任务文档中的每项验收标准都必须有可能失败的检查;标记任何没有检查的标准
- 遵循
AGENTS.md、LEARNINGS.md和任务文档中的项目约定
偏差分类
| 级别 | 含义 | 结果 |
|---|---|---|
| Minor | 满足范围要求的细微实现差异。 | 记录并继续。 |
| Medium | 应让用户知晓的实质性差异。 | 仅在风险可接受时记录并继续。 |
| Major | 违反需求、扩大范围,或未经批准更改架构。 | FAIL 并停止。 |
在任务文档中的输出
## Quality Gate Notes
### Result
PASS | FAIL
### Standards Review
- {Finding or "No blocking issues"}
### Deviations
- {Deviation level and rationale}
### Required Fixes
- {Only if FAIL}
交接
PASS:
Quality gate passed for task {ID}
Next stage: test {ID}
FAIL:
Quality gate failed for task {ID}
Return to implementation with the required fixes listed in the task document.
评论
支持 Markdown