team-skills 1.5.1 → 1.5.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +18 -0
- package/package.json +1 -1
- package/skills/_team-rules/constitutional-rules.md +2 -0
- package/skills/team-brainstorm/SKILL.md +3 -1
- package/skills/team-debug/SKILL.md +7 -2
- package/skills/team-feedback/SKILL.md +3 -1
- package/skills/team-finish/SKILL.md +3 -1
- package/skills/team-impl/SKILL.md +7 -2
- package/skills/team-orchestrator/SKILL.md +19 -4
- package/skills/team-review/SKILL.md +42 -26
- package/skills/team-score/SKILL.md +3 -1
- package/skills/team-security/SKILL.md +3 -1
- package/skills/team-spec/SKILL.md +3 -1
- package/skills/team-test/SKILL.md +3 -1
- package/skills/team-verify/SKILL.md +3 -1
- package/skills/using-team-skills/SKILL.md +1 -1
package/CHANGELOG.md
CHANGED
|
@@ -7,6 +7,24 @@
|
|
|
7
7
|
|
|
8
8
|
## [Unreleased]
|
|
9
9
|
|
|
10
|
+
## [1.5.3] - 2026-06-28
|
|
11
|
+
|
|
12
|
+
### 修复
|
|
13
|
+
|
|
14
|
+
- 全部 12 个 SKILL.md 在 frontmatter 后增加顶级 CRITICAL 约束,阻止 LLM 使用 EnterPlanMode 绕过 Skill 自身的结构化流程(有向图/TDD/根因调查)
|
|
15
|
+
- constitutional-rules.md 新增 Rule #10「禁止外部规划工具替代 Skill 流程」+ 常见规避借口补充
|
|
16
|
+
- team-orchestrator/team-impl/team-debug: ROLE 约束、Step 1 TRAP、STOP_SIGNALS、SELF_CHECK 四层防护
|
|
17
|
+
- 清理 docs/tasks 历史产出并加入 .gitignore
|
|
18
|
+
|
|
19
|
+
## [1.5.2] - 2026-06-28
|
|
20
|
+
|
|
21
|
+
### 修复
|
|
22
|
+
|
|
23
|
+
- team-review: Phase 2 P0/P1 路由硬化——从被动"向编排器报告:建议路由到"改为显式 `route_target` 赋值 + `GOTO Phase 3`,消除顺序流滑入自修的路径
|
|
24
|
+
- team-review: Phase 3 重排为 MATCH `route_target` 优先分发——P0/P1 以 `DONE_WITH_CONCERNS` 硬终止执行,P2 自修入口增加 GATE 准入断言(`severity != P0 && severity != P1`)
|
|
25
|
+
- team-review: COMPLETION 补充 P0/P1 路由回退的 `DONE_WITH_CONCERNS` 状态定义(仅产出 `11-review.md`,Phase 4/5 跳过)
|
|
26
|
+
- team-orchestrator: Step 5 先 READ 路由决策再完成验证——P0/P1 回退场景跳过文件完整性检查(避免 team-review 提前终止后因缺少 12/13/task-rules 文件被误判导致死循环)
|
|
27
|
+
|
|
10
28
|
## [1.5.1] - 2026-06-28
|
|
11
29
|
|
|
12
30
|
### 修复
|
package/package.json
CHANGED
|
@@ -15,6 +15,7 @@
|
|
|
15
15
|
7. **回退次数上限** — 同阶段 ≤ 2 次,超过触发 ASK_HUMAN。两次未解决 = 信息不足,需人类介入(First Principle #1)
|
|
16
16
|
8. **验证先行** — "通过"声明须基于当次新鲜执行的完整输出,上一轮结果是历史而非当前事实(First Principle #4)
|
|
17
17
|
9. **TDD 顺序不可逆** — RED + commit 先于 GREEN + commit。后写测试 = 测试你构建的;先写测试 = 测试需求的(First Principle #2)
|
|
18
|
+
10. **禁止外部规划工具替代 Skill 流程** — 不得使用 EnterPlanMode 等工具替代 SKILL.md 定义的 STEPS。每个 Skill 自身的 Phase/Step 就是完整的规划与执行流程,外部规划工具会绕过纪律流程(TDD、根因调查、有向图调度)(First Principle #4)
|
|
18
19
|
|
|
19
20
|
## 常见规避借口(不成立)
|
|
20
21
|
|
|
@@ -26,6 +27,7 @@
|
|
|
26
27
|
| "改动太小不需要测试" | 至少运行相关测试 |
|
|
27
28
|
| "先实现再补测试" | 先测试再实现 |
|
|
28
29
|
| "代码已经写好了,补个测试就行" | 删除实现代码,从 RED 开始 |
|
|
30
|
+
| "先规划一下再按流程走" | Skill 的 STEPS 就是规划,不需要 EnterPlanMode |
|
|
29
31
|
| "先继续后面再修" | 立即修复,修复后重新验证 |
|
|
30
32
|
| "这个失败跟我的改动无关" | 验证无关性(git stash → 运行 → 仍失败 = 确认无关并记录);未验证 = 掩盖 |
|
|
31
33
|
| "用户没要求写文档" | 文档是流程一部分 |
|
|
@@ -5,6 +5,8 @@ description: Use when requirements are fuzzy, need to discuss and form a plan be
|
|
|
5
5
|
|
|
6
6
|
# Team Brainstorm — 讨论形成方案
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -245,7 +247,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
|
|
|
245
247
|
|
|
246
248
|
## CONSTITUTIONAL_RULES
|
|
247
249
|
|
|
248
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
250
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
249
251
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
250
252
|
|
|
251
253
|
brainstorm 阶段尤其注意:
|
|
@@ -5,6 +5,8 @@ description: Use when encountering any bug, test failure, or unexpected behavior
|
|
|
5
5
|
|
|
6
6
|
# Team Debug — 系统调试
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own investigation workflow (Phase 1→Phase 5). EnterPlanMode bypasses systematic root cause analysis, causing fix-before-investigation (violates IRON_LAW). Follow STEPS below directly, starting from Phase 1.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -60,6 +62,8 @@ NO FIXES WITHOUT ROOT CAUSE INVESTIGATION FIRST
|
|
|
60
62
|
|
|
61
63
|
> 收集所有症状的完整描述,不遗漏任何错误细节。"差不多记住了"不算收集。
|
|
62
64
|
|
|
65
|
+
> TRAP:不要使用 EnterPlanMode 来"先分析一下 bug"——本 SKILL.md 的 Phase 1→Phase 5 就是完整的调试流程。EnterPlanMode 会跳过系统性根因调查,导致基于初步印象直接写修复(违反 IRON_LAW)。
|
|
66
|
+
|
|
63
67
|
> TRAP:你会倾向于读完错误信息第一行就跳到修复方案("我觉得我知道问题在哪")。强制自己读完完整 stack trace 和错误上下文。第一行是症状,最后几行才是根因。
|
|
64
68
|
|
|
65
69
|
1. **READ** 完整错误信息 — 不跳过 stack trace、行号、错误码
|
|
@@ -173,6 +177,7 @@ NO FIXES WITHOUT ROOT CAUSE INVESTIGATION FIRST
|
|
|
173
177
|
|
|
174
178
|
## STOP_SIGNALS
|
|
175
179
|
|
|
180
|
+
- **使用** EnterPlanMode 或其他外部规划工具替代根因调查流程(plan mode 中的阅读 ≠ 系统性根因调查)
|
|
176
181
|
- **跳过**根因调查直接写修复代码
|
|
177
182
|
- **修改**多个变量同时进行,无法隔离有效改动
|
|
178
183
|
- **继续**尝试 3 次修复失败后仍不触发 `ASK_HUMAN`
|
|
@@ -204,7 +209,7 @@ NO FIXES WITHOUT ROOT CAUSE INVESTIGATION FIRST
|
|
|
204
209
|
|
|
205
210
|
## CONSTITUTIONAL_RULES
|
|
206
211
|
|
|
207
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
212
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
208
213
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
209
214
|
**REF** `_team-rules/verification-protocol.md` — 5 步验证协议
|
|
210
215
|
**REF** `_team-rules/spec-driven-workflow.md` — TDD 修复循环与有向图回退规则
|
|
@@ -226,7 +231,7 @@ NO FIXES WITHOUT ROOT CAUSE INVESTIGATION FIRST
|
|
|
226
231
|
- [ ] **ASSERT** `修复失败次数 < 3` || `ASK_HUMAN 已触发`
|
|
227
232
|
- [ ] **ASSERT** `同时修改变量数 <= 1`
|
|
228
233
|
- [ ] **ASSERT** `无占位符残留({N}、{slug} 等已被实际值替换)`
|
|
229
|
-
- [ ] **ASSERT** `IRON_LAW 遵守` —
|
|
234
|
+
- [ ] **ASSERT** `IRON_LAW 遵守` — 根因已确定后才修复,未跳过调查、未使用 EnterPlanMode 替代调查流程
|
|
230
235
|
- [ ] 我的假设如果错了,还有什么能解释所有已知症状?
|
|
231
236
|
- [ ] 我是在修根因还是在修症状?根因仍在时这个修复能撑多久?
|
|
232
237
|
|
|
@@ -5,6 +5,8 @@ description: Use when receiving code review feedback, before implementing sugges
|
|
|
5
5
|
|
|
6
6
|
# Team Feedback — 审查反馈应对
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -221,7 +223,7 @@ NO IMPLEMENTATION WITHOUT TECHNICAL VERIFICATION FIRST
|
|
|
221
223
|
|
|
222
224
|
## CONSTITUTIONAL_RULES
|
|
223
225
|
|
|
224
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
226
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
225
227
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
226
228
|
**REF** `_team-rules/spec-driven-workflow.md` — TDD 逐项验证与有向图回退规则
|
|
227
229
|
**REF** `_team-rules/verification-protocol.md` — verify_cmd 解析流程与 5 步验证协议
|
|
@@ -5,6 +5,8 @@ description: Use when implementation is complete, all tests pass, and you need t
|
|
|
5
5
|
|
|
6
6
|
# Team Finish — 分支完成处理
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
> **分支生命周期**:`team-orchestrator` 在 CONFIRM_GOAL 确认后创建功能分支(Step 1.5),本 Skill 在流程尾部(Step 7)负责分支收尾。
|
|
@@ -285,7 +287,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
|
|
|
285
287
|
|
|
286
288
|
## CONSTITUTIONAL_RULES
|
|
287
289
|
|
|
288
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
290
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
289
291
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
290
292
|
**REF** `_team-rules/verification-protocol.md` — verify_cmd 解析流程与 5 步验证协议
|
|
291
293
|
**REF** `_team-rules/task-lifecycle.md` — 进度追踪与知识合并(§3)
|
|
@@ -5,6 +5,8 @@ description: Use when SDD exists and you need TDD implementation with 06-08 docs
|
|
|
5
5
|
|
|
6
6
|
# Team Impl — 实现
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own TDD workflow (RED→GREEN→REFACTOR). EnterPlanMode bypasses the TDD cycle, causing implementation-before-test (violates IRON_LAW). Follow STEPS below directly, starting from Phase 1.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -115,6 +117,8 @@ NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST
|
|
|
115
117
|
|
|
116
118
|
> 通过红-绿-重构循环逐个实现功能点。每个 GREEN 完成时代码应是"能工作的最简单方案",不是"我能想到的最好方案"。
|
|
117
119
|
|
|
120
|
+
> TRAP:不要使用 EnterPlanMode 来"先规划实现方案"——本 SKILL.md 的 Phase 1→Phase 4 就是完整的实现流程。EnterPlanMode 会跳过 RED→GREEN→REFACTOR 循环,导致先写实现再补测试(违反 IRON_LAW)。
|
|
121
|
+
|
|
118
122
|
**FOR** `feature_point`(从 SDD 提取):
|
|
119
123
|
|
|
120
124
|
#### 循环 1:红(Red)— 写测试
|
|
@@ -343,13 +347,14 @@ NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST
|
|
|
343
347
|
|
|
344
348
|
## STOP_SIGNALS
|
|
345
349
|
|
|
350
|
+
- **使用** EnterPlanMode 或其他外部规划工具替代 TDD 循环(plan mode 完成 ≠ RED 阶段完成)
|
|
346
351
|
- **编码**前没 `READ` spec,或发现 spec 问题不 `ROLLBACK` 而自己决定
|
|
347
352
|
- **跳过** RED 阶段直接写实现,或先写实现再补测试
|
|
348
353
|
- **修改**测试让它通过(而非修改实现),或困惑不记录默默假设
|
|
349
354
|
|
|
350
355
|
## CONSTITUTIONAL_RULES
|
|
351
356
|
|
|
352
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
357
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
353
358
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
354
359
|
**REF** `_team-rules/verification-protocol.md` — 5 步验证协议
|
|
355
360
|
**REF** `_team-rules/spec-driven-workflow.md` — Spec-Driven 开发原则与 TDD 工作流
|
|
@@ -376,7 +381,7 @@ NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST
|
|
|
376
381
|
- [ ] **ASSERT** `实际消耗 <= 01-plan.md 自我约束预算`
|
|
377
382
|
- [ ] **ASSERT** `所有困惑已显式记录于 06-tdd-log.md 审计段落`
|
|
378
383
|
- [ ] **ASSERT** `无占位符残留({N}、{slug} 等已被实际值替换)`
|
|
379
|
-
- [ ] **ASSERT** `IRON_LAW 遵守` — 未自行假设 spec、未跳过 TDD
|
|
384
|
+
- [ ] **ASSERT** `IRON_LAW 遵守` — 未自行假设 spec、未跳过 TDD 顺序、未使用 EnterPlanMode 替代 TDD 循环
|
|
380
385
|
- [ ] **IF** 发现 spec 问题 → **ASSERT** `已 ROLLBACK team-spec`
|
|
381
386
|
- [ ] 我是否在某个 GREEN 阶段写了超出当前测试要求的代码?
|
|
382
387
|
- [ ] 我是否有任何"通过"声明是基于上一轮输出而非本轮刚执行的结果?
|
|
@@ -5,6 +5,8 @@ description: Use when task needs full spec→impl→test→review pipeline with
|
|
|
5
5
|
|
|
6
6
|
# Team Orchestrator — 流程编排器
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow (Step 1→Step 8). EnterPlanMode bypasses the directed graph, skips sub-skill dispatch (team-spec/team-impl/team-test/team-review), and skips human intervention points (CONFIRM_GOAL/HUMAN_ACCEPT). Follow STEPS below directly, starting from Step 1.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
```mermaid
|
|
@@ -45,6 +47,7 @@ flowchart TD
|
|
|
45
47
|
- CONFIRM_GOAL/HUMAN_ACCEPT 任何模式下不可省略
|
|
46
48
|
- 编排器不得自己写实现代码(必须 dispatch 子 Skill,不可亲自实现)
|
|
47
49
|
- 子 Skill 不可用时不得自动降级为自我执行,必须 ASK_HUMAN 请示用户
|
|
50
|
+
- 编排器不得使用 EnterPlanMode 等外部规划工具替代自身有向图流程——STEPS 定义的 Step 1→Step 8 就是规划与执行的全部流程
|
|
48
51
|
```
|
|
49
52
|
|
|
50
53
|
### 路由推理检查点
|
|
@@ -369,6 +372,8 @@ NO AGENT DISPATCH WITHOUT CONFIRM_GOAL HUMAN CONFIRMATION FIRST
|
|
|
369
372
|
|
|
370
373
|
> 确保任务目标被准确理解,用户对方案方向有明确确认。跳过 CONFIRM_GOAL 是编排器最危险的错误——后续所有 Agent 的工作都建立在这个确认之上。
|
|
371
374
|
|
|
375
|
+
> TRAP:不要使用 EnterPlanMode 来"先规划一下"——本 SKILL.md 的 Step 1→Step 8 就是完整的规划与执行流程。EnterPlanMode 会绕过有向图、跳过子 Skill 调度、跳过人类介入点,导致编排器退化为普通 Agent 直接写代码。
|
|
376
|
+
|
|
372
377
|
1. **READ** 用户参数 → 提取任务描述
|
|
373
378
|
2. **IF** `docs/tasks/` NOT_EXISTS → 创建目录,最大序号 = 0
|
|
374
379
|
**ELSE** → **READ** `docs/tasks/` 已有目录 → 提取所有匹配 `NNNN-*` 格式的目录名中的四位数字前缀 → 取最大值记为最大序号(无匹配目录则最大序号 = 0)
|
|
@@ -677,6 +682,17 @@ TDD 强制要求:每个功能点必须先 git commit 失败测试(test: {功
|
|
|
677
682
|
读取 skills/team-review/SKILL.md 获取完整执行步骤。
|
|
678
683
|
```
|
|
679
684
|
|
|
685
|
+
**READ** `11-review.md` 中 team-review 修复/回退决策
|
|
686
|
+
|
|
687
|
+
**IF** team-review 报告 `DONE_WITH_CONCERNS` 且 `route_target IN [team-impl, team-spec]`(P0/P1 回退场景,team-review 提前终止是正确行为):
|
|
688
|
+
|
|
689
|
+
> team-review 发现 P0/P1 问题后以 DONE_WITH_CONCERNS 终止执行,仅产出 11-review.md,不产出 12-asset-update.md / 13-retrospective.md / task-rules.md。这是有向图回退的正确行为,跳过完成验证直接进入回退检查。
|
|
690
|
+
|
|
691
|
+
→ **WRITE**(对话中)team-review 回退摘要:P0/P1 问题描述 + 路由决策(→ team-impl / → team-spec)
|
|
692
|
+
→ 直接进入下方回退检查
|
|
693
|
+
|
|
694
|
+
**ELSE**(team-review 正常完成):
|
|
695
|
+
|
|
680
696
|
**完成验证**(产出门禁):
|
|
681
697
|
|
|
682
698
|
**FOR** `file` **IN** [`11-review.md`, `12-asset-update.md`, `13-retrospective.md`, `task-rules.md`]:
|
|
@@ -685,8 +701,6 @@ TDD 强制要求:每个功能点必须先 git commit 失败测试(test: {功
|
|
|
685
701
|
- **ASSERT** `13-retrospective.md CONTAINS "新规则" || CONTAINS "本次沉淀"`
|
|
686
702
|
- 任一不通过 → **ROLLBACK** team-review,指明缺失文件名
|
|
687
703
|
|
|
688
|
-
**READ** `11-review.md` 中 team-review 修复/回退决策
|
|
689
|
-
|
|
690
704
|
**WRITE**(对话中)team-review 产出摘要:五维度审查结论 + P0/P1 问题数 + 路由决策(继续/回退)。**IF** `status == DONE_WITH_CONCERNS` → 完整展示 concerns 给用户。
|
|
691
705
|
|
|
692
706
|
**WRITE** checkpoint:`current_step=Step 6, next_step=Step 7, phase=review, completed_steps 追加 Step 5`
|
|
@@ -975,13 +989,14 @@ TDD 强制要求:每个功能点必须先 git commit 失败测试(test: {功
|
|
|
975
989
|
## STOP_SIGNALS
|
|
976
990
|
|
|
977
991
|
- **跳过** CONFIRM_GOAL 或 HUMAN_ACCEPT 人类介入点
|
|
992
|
+
- **使用** EnterPlanMode 或其他外部规划工具替代 STEPS 定义的有向图流程(plan mode 完成 ≠ CONFIRM_GOAL 完成)
|
|
978
993
|
- **延迟**回退("先记着后面一起修")
|
|
979
994
|
- **信任** Agent 自我声明而不验证产出
|
|
980
995
|
- **超出**预算不砍范围,或亲自执行实现代码而非 dispatch 子 Skill
|
|
981
996
|
|
|
982
997
|
## CONSTITUTIONAL_RULES
|
|
983
998
|
|
|
984
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
999
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
985
1000
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
986
1001
|
**REF** `_team-rules/ai-collaboration-standards.md` — AI 协作资产与 Prompt 工程规范
|
|
987
1002
|
**REF** `_team-rules/spec-driven-workflow.md` — 有向图回退规则与回退次数上限
|
|
@@ -1004,7 +1019,7 @@ TDD 强制要求:每个功能点必须先 git commit 失败测试(test: {功
|
|
|
1004
1019
|
- [ ] **IF** team-review 要求 → **ASSERT** `CHANGELOG.md 已更新`
|
|
1005
1020
|
- [ ] **ASSERT** `进度账本已更新`
|
|
1006
1021
|
- [ ] **ASSERT** `无占位符残留({N}、{slug} 等已被实际值替换)`
|
|
1007
|
-
- [ ] **ASSERT** `IRON_LAW 遵守` —
|
|
1022
|
+
- [ ] **ASSERT** `IRON_LAW 遵守` — 未自己写实现代码、未跳过人类介入点、未使用 EnterPlanMode 替代有向图流程
|
|
1008
1023
|
- [ ] 我是否因为"这步很简单"而跳过了某个人类介入点?
|
|
1009
1024
|
- [ ] 我是否把所有 Agent 的 concerns 都传达给了用户,还是悄悄忽略了某些?
|
|
1010
1025
|
- [ ] 我是否在回退时传递了完整的四要素上下文(问题、位置、期望、建议),还是只说了"有 bug"?
|
|
@@ -5,6 +5,8 @@ description: Use when code + tests exist and you need structured review + asset
|
|
|
5
5
|
|
|
6
6
|
# Team Review — 代码审查
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -177,15 +179,15 @@ NO COMPLETION CLAIMS WITHOUT CONSTITUTIONAL COMPLIANCE CHECK FIRST
|
|
|
177
179
|
**MATCH** `severity`:
|
|
178
180
|
|
|
179
181
|
- `P0` || `P1`
|
|
180
|
-
- `P0 实现 bug` && `spec 定义正确` →
|
|
181
|
-
- `P0 设计/架构缺陷` →
|
|
182
|
+
- `P0 实现 bug` && `spec 定义正确` → `route_target = team-impl` → **GOTO** Phase 3
|
|
183
|
+
- `P0 设计/架构缺陷` → `route_target = team-spec` → **GOTO** Phase 3
|
|
182
184
|
- `P0 安全漏洞` → **ASK_HUMAN**(安全决策需要人类确认)
|
|
183
|
-
- `P1 实现 bug` →
|
|
184
|
-
- `P1 测试遗漏` →
|
|
185
|
-
- `P0/P1 spec 遗漏` →
|
|
185
|
+
- `P1 实现 bug` → `route_target = team-impl` → **GOTO** Phase 3
|
|
186
|
+
- `P1 测试遗漏` → `route_target = team-impl`(需要补写测试) → **GOTO** Phase 3
|
|
187
|
+
- `P0/P1 spec 遗漏` → `route_target = team-spec` → **GOTO** Phase 3
|
|
186
188
|
- 需要人类决策 → **ASK_HUMAN**(有多个可行方案需要选择)
|
|
187
|
-
- `P2` →
|
|
188
|
-
- `P3` → 记录但不处理
|
|
189
|
+
- `P2` → `route_target = self` → **GOTO** Phase 3
|
|
190
|
+
- `P3` → 记录但不处理 → **GOTO** Phase 4
|
|
189
191
|
- *DEFAULT* → **GOTO** Phase 4
|
|
190
192
|
|
|
191
193
|
**回退时必须提供**:
|
|
@@ -198,32 +200,41 @@ NO COMPLETION CLAIMS WITHOUT CONSTITUTIONAL COMPLIANCE CHECK FIRST
|
|
|
198
200
|
|
|
199
201
|
### Phase 3:执行路由决策
|
|
200
202
|
|
|
201
|
-
>
|
|
202
|
-
|
|
203
|
-
> TRAP:自修时容易越界——"顺手"改了超过 20 行或触及了不在自己职责内的逻辑。超范围修改应回退 team-impl。
|
|
204
|
-
|
|
205
|
-
1. 直接修改代码/测试(**每个问题限 20 行以内的修改**——更大规模的重构记录为建议,不直接执行)
|
|
206
|
-
2. **EXEC** 项目测试命令 — 确认修复正确
|
|
207
|
-
**ASSERT** `exit_code == 0` — 测试失败 → 回滚修改 → **GOTO** Phase 2
|
|
208
|
-
3. **EXEC** 项目 CI 检查命令 — 确认无 lint 问题
|
|
209
|
-
**ASSERT** `exit_code == 0` — lint 失败 → 修复后重新执行
|
|
210
|
-
|
|
211
|
-
**验证协议**:步骤 2-3 声明"通过"前必须执行 `_team-rules/verification-protocol.md: 验证执行步骤`
|
|
212
|
-
|
|
213
|
-
4. **ASSERT** `exit_code == 0` && `failures == 0`
|
|
214
|
-
- 通过 → **WRITE** 修复详情(问题 ID + 修复内容 + 验证结果)到 `11-review.md` §三修复记录
|
|
215
|
-
- 修复导致新测试失败或引入新问题 → 立即停止自修 → **ROLLBACK** team-impl(通过编排器),附带修复尝试的上下文和失败详情
|
|
203
|
+
> 根据 Phase 2 的 `route_target` 执行对应动作。P0/P1 必须终止执行交还编排器,不可自修。
|
|
216
204
|
|
|
217
205
|
**MATCH** `route_target`:
|
|
218
206
|
|
|
219
207
|
- `team-impl` || `team-spec` →
|
|
220
|
-
1. **WRITE** 问题详情到 `11-review.md
|
|
221
|
-
2.
|
|
208
|
+
1. **WRITE** 问题详情到 `11-review.md`(包含"回退时必须提供"的完整信息)
|
|
209
|
+
2. 以 **DONE_WITH_CONCERNS** 终止执行,向编排器交付路由决策和问题上下文
|
|
210
|
+
|
|
211
|
+
> TRAP:到了这里不要"顺手修一下"——P0/P1 问题必须回退给专职 Skill,这是有向图回退的硬约束 `_team-rules/first-principles.md: First Principle #4`。
|
|
212
|
+
|
|
222
213
|
- `human` →
|
|
223
214
|
1. **WRITE** 问题详情到 `11-review.md`
|
|
224
215
|
2. 向用户展示问题 + 选项 → **ASK_HUMAN**,等待决策
|
|
225
216
|
3. 根据决策执行修复
|
|
226
|
-
|
|
217
|
+
|
|
218
|
+
- `self`(仅 P2 及以下) →
|
|
219
|
+
|
|
220
|
+
**GATE** 自修准入:
|
|
221
|
+
- **ASSERT** `severity != P0 && severity != P1` — P0/P1 不可自修,必须回退
|
|
222
|
+
|
|
223
|
+
> TRAP:自修时容易越界——"顺手"改了超过 20 行或触及了不在自己职责内的逻辑。超范围修改应回退 team-impl。
|
|
224
|
+
|
|
225
|
+
1. 直接修改代码/测试(**每个问题限 20 行以内的修改**——更大规模的重构记录为建议,不直接执行)
|
|
226
|
+
2. **EXEC** 项目测试命令 — 确认修复正确
|
|
227
|
+
**ASSERT** `exit_code == 0` — 测试失败 → 回滚修改 → **GOTO** Phase 2
|
|
228
|
+
3. **EXEC** 项目 CI 检查命令 — 确认无 lint 问题
|
|
229
|
+
**ASSERT** `exit_code == 0` — lint 失败 → 修复后重新执行
|
|
230
|
+
|
|
231
|
+
**验证协议**:步骤 2-3 声明"通过"前必须执行 `_team-rules/verification-protocol.md: 验证执行步骤`
|
|
232
|
+
|
|
233
|
+
4. **ASSERT** `exit_code == 0` && `failures == 0`
|
|
234
|
+
- 通过 → **WRITE** 修复详情(问题 ID + 修复内容 + 验证结果)到 `11-review.md` §三修复记录
|
|
235
|
+
- 修复导致新测试失败或引入新问题 → 立即停止自修 → `route_target = team-impl` → 以 **DONE_WITH_CONCERNS** 终止执行,附带修复尝试的上下文和失败详情
|
|
236
|
+
|
|
237
|
+
- *DEFAULT* → **GOTO** Phase 4
|
|
227
238
|
|
|
228
239
|
### Phase 4:AI 协作资产维护(消费方契约)
|
|
229
240
|
|
|
@@ -457,7 +468,7 @@ NO COMPLETION CLAIMS WITHOUT CONSTITUTIONAL COMPLIANCE CHECK FIRST
|
|
|
457
468
|
|
|
458
469
|
## CONSTITUTIONAL_RULES
|
|
459
470
|
|
|
460
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
471
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
461
472
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
462
473
|
**REF** `_team-rules/spec-driven-workflow.md` — SDD 验证链与有向图回退规则
|
|
463
474
|
**REF** `_team-rules/task-lifecycle.md` — 来源标签规范(§1.3)
|
|
@@ -499,6 +510,11 @@ NO COMPLETION CLAIMS WITHOUT CONSTITUTIONAL COMPLIANCE CHECK FIRST
|
|
|
499
510
|
- 修复记录:直接修复 `{N}` 个,回退 `team-impl` `{N}` 个,回退 `team-spec` `{N}` 个,人类决策 `{N}` 个
|
|
500
511
|
- 资产更新:`{N}` 个文件已更新
|
|
501
512
|
- → 编排器将补全团队级证据并交付用户验收
|
|
513
|
+
- 发现 P0/P1 问题,路由回退 → **DONE_WITH_CONCERNS**
|
|
514
|
+
- 产出文件:`11-review.md`(仅此文件,Phase 4/5 跳过)
|
|
515
|
+
- `route_target`:`team-impl` / `team-spec`
|
|
516
|
+
- 问题详情:包含"回退时必须提供"的完整信息
|
|
517
|
+
- → 编排器读取 route_target 执行有向图回退
|
|
502
518
|
- 全部通过但有保留意见(P2 建议未采纳等) → **DONE_WITH_CONCERNS**
|
|
503
519
|
- 缺少关键上下文(SDD 缺失、代码无法访问等) → **NEEDS_CONTEXT**
|
|
504
520
|
- P0/P1 问题阻塞且路由失败 → **BLOCKED**,触发 **ASK_HUMAN**
|
|
@@ -5,6 +5,8 @@ description: Use when evaluating AI collaboration maturity of a project
|
|
|
5
5
|
|
|
6
6
|
# Team Score — 协作评分
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -417,7 +419,7 @@ NO SCORE WITHOUT EVIDENCE FIRST
|
|
|
417
419
|
|
|
418
420
|
## CONSTITUTIONAL_RULES
|
|
419
421
|
|
|
420
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
422
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
421
423
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
422
424
|
**REF** `_team-rules/ai-collaboration-standards.md` — AI 协作资产与 Prompt 工程规范
|
|
423
425
|
**REF** `_team-rules/verification-protocol.md` — 5 步验证协议
|
|
@@ -5,6 +5,8 @@ description: Use when AI usage involves sensitive data, external services, or au
|
|
|
5
5
|
|
|
6
6
|
# Team Security — AI 安全红线合规检查
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -535,7 +537,7 @@ NO AI OPERATIONS WITHOUT RED LINE CHECK FIRST
|
|
|
535
537
|
|
|
536
538
|
## CONSTITUTIONAL_RULES
|
|
537
539
|
|
|
538
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
540
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
539
541
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
540
542
|
**REF** `_team-rules/verification-protocol.md` — 5 步验证协议
|
|
541
543
|
|
|
@@ -5,6 +5,8 @@ description: Use when starting a new feature, need SDD spec, or requirements are
|
|
|
5
5
|
|
|
6
6
|
# Team Spec — 规格制定
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -296,7 +298,7 @@ NO CODE WITHOUT SPEC FIRST
|
|
|
296
298
|
|
|
297
299
|
## CONSTITUTIONAL_RULES
|
|
298
300
|
|
|
299
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
301
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
300
302
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
301
303
|
**REF** `_team-rules/spec-driven-workflow.md` — Spec-Driven 开发原则与 TDD 工作流
|
|
302
304
|
**REF** `_team-rules/task-lifecycle.md` — 来源标签规范(§1.3)
|
|
@@ -5,6 +5,8 @@ description: Use when implementation exists and you need test matrix + coverage
|
|
|
5
5
|
|
|
6
6
|
# Team Test — 测试审计
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -246,7 +248,7 @@ Phase 1 只分析,不写测试代码。
|
|
|
246
248
|
|
|
247
249
|
## CONSTITUTIONAL_RULES
|
|
248
250
|
|
|
249
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
251
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
250
252
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
251
253
|
**REF** `_team-rules/spec-driven-workflow.md` — SDD 验证链与有向图回退规则
|
|
252
254
|
**REF** `_team-rules/verification-protocol.md` — verify_cmd 解析流程与 5 步验证协议
|
|
@@ -5,6 +5,8 @@ description: Use when about to claim work is complete, fixed, or passing - requi
|
|
|
5
5
|
|
|
6
6
|
# Team Verify — 验证协议
|
|
7
7
|
|
|
8
|
+
**CRITICAL: DO NOT use EnterPlanMode.** This skill defines its own structured workflow. Follow STEPS below directly.
|
|
9
|
+
|
|
8
10
|
## ROLE
|
|
9
11
|
|
|
10
12
|
### 系统提示词
|
|
@@ -164,7 +166,7 @@ NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE FIRST
|
|
|
164
166
|
|
|
165
167
|
## CONSTITUTIONAL_RULES
|
|
166
168
|
|
|
167
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
169
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
168
170
|
**REF** `_team-rules/verification-protocol.md` — 5 步验证协议
|
|
169
171
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
170
172
|
|
|
@@ -138,7 +138,7 @@ NO SKILL RECOMMENDATION WITHOUT SCENE ANALYSIS FIRST
|
|
|
138
138
|
|
|
139
139
|
## CONSTITUTIONAL_RULES
|
|
140
140
|
|
|
141
|
-
**REF** `_team-rules/constitutional-rules.md` —
|
|
141
|
+
**REF** `_team-rules/constitutional-rules.md` — 10 条 Constitutional Rules
|
|
142
142
|
**REF** `_team-rules/first-principles.md` — 4 条第一性原理(First Principle #1 ~ #4)
|
|
143
143
|
|
|
144
144
|
分诊阶段尤其注意:
|