Skip to content

Commit 4a55cbf

Browse files
committed
同步上游 v5.0.6:用内联自检替代子代理审查循环
- brainstorming: 规格审查循环 → 规格自检(4 项内联检查清单) - writing-plans: 计划审查循环 → 自检(3 项检查清单)+ 新增"禁止占位符"章节 - writing-skills: 修正 frontmatter 描述,添加 agentskills.io 规范链接
1 parent 00b1e8b commit 4a55cbf

File tree

3 files changed

+35
-28
lines changed

3 files changed

+35
-28
lines changed

skills/brainstorming/SKILL.md

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ description: "在任何创造性工作之前必须使用此技能——创建功
2727
4. **提出 2-3 种方案** — 附带权衡分析和你的推荐
2828
5. **展示设计** — 按复杂度分节展示,每节展示后获得用户批准
2929
6. **编写设计文档** — 保存到 `docs/superpowers/specs/YYYY-MM-DD-<topic>-design.md` 并 commit
30-
7. **规格审查循环**调度 spec-document-reviewer 子代理,提供精心组织的审查上下文(绝不是你的会话历史);修复问题后重新调度直到通过(最多 3 次迭代,之后交给人工处理
30+
7. **规格自检**快速内联检查占位符、矛盾、模糊性、范围(详见下方
3131
8. **用户审查书面规格** — 在继续之前请用户审查规格文件
3232
9. **过渡到实现** — 调用 writing-plans 技能创建实现计划
3333

@@ -43,8 +43,7 @@ digraph brainstorming {
4343
"分节展示设计" [shape=box];
4444
"用户批准设计?" [shape=diamond];
4545
"编写设计文档" [shape=box];
46-
"规格审查循环" [shape=box];
47-
"规格审查通过?" [shape=diamond];
46+
"规格自检\n(内联修复)" [shape=box];
4847
"用户审查规格?" [shape=diamond];
4948
"调用 writing-plans 技能" [shape=doublecircle];
5049
@@ -57,10 +56,8 @@ digraph brainstorming {
5756
"分节展示设计" -> "用户批准设计?";
5857
"用户批准设计?" -> "分节展示设计" [label="否,修改"];
5958
"用户批准设计?" -> "编写设计文档" [label="是"];
60-
"编写设计文档" -> "规格审查循环";
61-
"规格审查循环" -> "规格审查通过?";
62-
"规格审查通过?" -> "规格审查循环" [label="发现问题,\n修复后重新调度"];
63-
"规格审查通过?" -> "用户审查规格?" [label="通过"];
59+
"编写设计文档" -> "规格自检\n(内联修复)";
60+
"规格自检\n(内联修复)" -> "用户审查规格?";
6461
"用户审查规格?" -> "编写设计文档" [label="要求修改"];
6562
"用户审查规格?" -> "调用 writing-plans 技能" [label="批准"];
6663
}
@@ -116,19 +113,22 @@ digraph brainstorming {
116113
- 如果可用,使用 elements-of-style:writing-clearly-and-concisely 技能
117114
- 将设计文档 commit 到 git
118115

119-
**规格审查循环**
120-
编写规格文档后:
116+
**规格自检**
117+
编写规格文档后,以全新的视角审视它
121118

122-
1. 调度 spec-document-reviewer 子代理(参见 spec-document-reviewer-prompt.md)
123-
2. 如果发现问题:修复,重新调度,重复直到通过
124-
3. 如果循环超过 3 次迭代,交给人工指导
119+
1. **占位符扫描:** 有没有"待定"、"TODO"、未完成的章节或模糊的需求?修复它们。
120+
2. **内部一致性:** 各章节之间有矛盾吗?架构和功能描述匹配吗?
121+
3. **范围检查:** 这是否聚焦到可以用一个实现计划覆盖,还是需要进一步拆分?
122+
4. **模糊性检查:** 有没有需求可以被两种方式理解?如果有,选择一种并明确写出来。
123+
124+
发现问题就直接内联修复。无需重新审查——修好继续推进。
125125

126126
**用户审查关卡:**
127-
规格审查循环通过后,请用户在继续之前审查书面规格:
127+
规格自检完成后,请用户在继续之前审查书面规格:
128128

129129
> "规格已编写并 commit 到 `<path>`。请审查一下,如果在我们开始编写实现计划之前你想做任何修改,请告诉我。"
130130
131-
等待用户回复。如果他们要求修改,做出修改并重新运行规格审查循环。只有在用户批准后才继续。
131+
等待用户回复。如果他们要求修改,做出修改并重新运行规格自检。只有在用户批准后才继续。
132132

133133
**实现:**
134134

skills/writing-plans/SKILL.md

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -103,26 +103,33 @@ git commit -m "feat: add specific feature"
103103
```
104104
````
105105

106+
## 禁止占位符
107+
108+
每个步骤都必须包含工程师需要的实际内容。以下是**计划缺陷**——绝不要写出来:
109+
- "待定"、"TODO"、"后续实现"、"补充细节"
110+
- "添加适当的错误处理" / "添加验证" / "处理边界情况"
111+
- "为上述代码编写测试"(没有实际测试代码)
112+
- "类似任务 N"(重复代码——工程师可能不按顺序阅读任务)
113+
- 只描述做什么而不展示怎么做的步骤(代码步骤必须有代码块)
114+
- 引用了未在任何任务中定义的类型、函数或方法
115+
106116
## 注意事项
107117
- 始终使用精确的文件路径
108-
- 计划中包含完整代码(而非"添加验证")
118+
- 每个步骤都包含完整代码——如果步骤涉及代码变更,就展示代码
109119
- 精确的命令和预期输出
110-
- 使用 @ 语法引用相关技能
111120
- DRY、YAGNI、TDD、频繁 commit
112121

113-
## 计划审查循环
122+
## 自检
123+
124+
编写完整计划后,以全新视角审视规格并对照检查计划。这是你自己执行的检查清单——不是子代理调度。
125+
126+
**1. 规格覆盖度:** 浏览规格中的每个章节/需求。你能指出实现它的任务吗?列出所有遗漏。
114127

115-
编写完整计划后:
128+
**2. 占位符扫描:** 搜索计划中的红旗——上方"禁止占位符"章节中的任何模式。修复它们。
116129

117-
1. 调度一个 plan-document-reviewer 子代理(参见 plan-document-reviewer-prompt.md),提供精心组织的审查上下文——绝不是你的会话历史。这样可以让审查员专注于计划本身,而非你的思考过程。
118-
- 提供:计划文档路径、规格文档路径
119-
2. 如果发现问题:修复问题,重新调度审查员审查整个计划
120-
3. 如果通过:进入执行交接
130+
**3. 类型一致性:** 后续任务中使用的类型、方法签名和属性名是否与前面任务中定义的一致?任务 3 中叫 `clearLayers()` 但任务 7 中叫 `clearFullLayers()` 就是 bug。
121131

122-
**审查循环指南:**
123-
- 编写计划的同一个代理负责修复(保留上下文)
124-
- 如果循环超过 3 次迭代,交给人工指导
125-
- 审查员是顾问性质的——如果你认为反馈不正确,可以解释你的理由
132+
如果发现问题,直接内联修复。无需重新审查——修好继续推进。如果发现规格中的需求没有对应任务,就添加任务。
126133

127134
## 执行交接
128135

skills/writing-skills/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ skills/
9292
## SKILL.md 结构
9393

9494
**Frontmatter(YAML):**
95-
- 仅支持两个字段`name``description`
95+
- 两个必需字段`name``description`(完整支持字段参见 [agentskills.io/specification](https://agentskills.io/specification)
9696
- 总计最多 1024 字符
9797
- `name`:只使用字母、数字和连字符(不要用括号、特殊字符)
9898
- `description`:第三人称,仅描述何时使用(不是做什么)
@@ -603,7 +603,7 @@ helper1、helper2、step3、pattern4
603603

604604
**绿色阶段 - 编写最小技能:**
605605
- [ ] 名称只使用字母、数字、连字符(无括号/特殊字符)
606-
- [ ] YAML frontmatter 仅包含 name 和 description(最多 1024 字符)
606+
- [ ] YAML frontmatter 包含必需的 `name``description` 字段(最多 1024 字符;参见 [spec](https://agentskills.io/specification)
607607
- [ ] 描述以"Use when..."开头并包含具体的触发条件/症状
608608
- [ ] 描述用第三人称
609609
- [ ] 全文包含搜索关键词(错误、症状、工具)

0 commit comments

Comments
 (0)