team-skills 1.0.0
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/.claude/commands/team-pull.md +21 -0
- package/.claude/commands/team-push.md +28 -0
- package/.claude/commands/team-setup.md +183 -0
- package/.claude/commands/team-uninstall.md +107 -0
- package/CHANGELOG.md +41 -0
- package/LICENSE +21 -0
- package/README.md +421 -0
- package/bin/team-skills.js +2 -0
- package/hooks/hooks.json +16 -0
- package/hooks/session-start +34 -0
- package/package.json +58 -0
- package/scripts/check-skill-structure.js +89 -0
- package/skills/CLAUDE.md +121 -0
- package/skills/_team-rules/constitutional-rules.md +25 -0
- package/skills/_team-rules/four-state-protocol.md +10 -0
- package/skills/_team-rules/verification-protocol.md +55 -0
- package/skills/team-brainstorm/SKILL.md +168 -0
- package/skills/team-debug/SKILL.md +143 -0
- package/skills/team-feedback/SKILL.md +175 -0
- package/skills/team-finish/SKILL.md +151 -0
- package/skills/team-impl/SKILL.md +316 -0
- package/skills/team-impl/references/06-tdd-log-template.md +62 -0
- package/skills/team-impl/references/07-prompt-log-template.md +32 -0
- package/skills/team-impl/references/08-ai-decisions-template.md +16 -0
- package/skills/team-orchestrator/SKILL.md +584 -0
- package/skills/team-orchestrator/references/14-team-template.md +70 -0
- package/skills/team-orchestrator/references/15-brief-template.md +50 -0
- package/skills/team-review/SKILL.md +383 -0
- package/skills/team-review/references/11-review-template.md +63 -0
- package/skills/team-review/references/12-asset-update-template.md +45 -0
- package/skills/team-review/references/13-retrospective-template.md +40 -0
- package/skills/team-score/SKILL.md +330 -0
- package/skills/team-spec/SKILL.md +231 -0
- package/skills/team-spec/references/01-plan-template.md +67 -0
- package/skills/team-spec/references/02-context-template.md +46 -0
- package/skills/team-spec/references/04-boundary-template.md +23 -0
- package/skills/team-spec/references/05-risk-template.md +50 -0
- package/skills/team-spec/references/delta-spec-template.md +32 -0
- package/skills/team-spec/references/prompt-template.md +23 -0
- package/skills/team-spec/references/sdd-template.md +72 -0
- package/skills/team-test/SKILL.md +190 -0
- package/skills/team-test/references/09-test-matrix-template.md +40 -0
- package/skills/team-test/references/10-test-report-template.md +59 -0
- package/skills/team-verify/SKILL.md +151 -0
- package/skills/using-team-skills/SKILL.md +137 -0
- package/src/cli.js +27 -0
- package/src/commands/init.js +115 -0
- package/src/commands/list.js +150 -0
- package/src/commands/setup.js +125 -0
- package/src/commands/uninstall.js +113 -0
- package/src/commands/update.js +118 -0
- package/src/lib/constants.js +17 -0
- package/src/lib/fs-utils.js +117 -0
- package/src/lib/inventory.js +64 -0
- package/src/lib/logger.js +34 -0
- package/src/lib/manifest.js +45 -0
|
@@ -0,0 +1,50 @@
|
|
|
1
|
+
# 答辩提纲
|
|
2
|
+
|
|
3
|
+
> Team 编排器产出
|
|
4
|
+
|
|
5
|
+
## 一、30 秒 Elevator Pitch
|
|
6
|
+
|
|
7
|
+
> 用 3 句话概述:(1) 解决了什么问题 (2) 怎么做的(核心方案) (3) 效果如何(量化指标或对比)
|
|
8
|
+
|
|
9
|
+
{填写}
|
|
10
|
+
|
|
11
|
+
## 二、关键决策解释
|
|
12
|
+
|
|
13
|
+
(从 08-ai-decisions.md 中挑选 2-3 个最重要的决策)
|
|
14
|
+
|
|
15
|
+
| 决策 | 背景约束 | 为什么这样做 | 为什么拒绝替代方案 |
|
|
16
|
+
| ---- | -------- | ------------ | ------------------ |
|
|
17
|
+
| ... | ... | ... | ... |
|
|
18
|
+
|
|
19
|
+
## 三、AI 协作亮点
|
|
20
|
+
|
|
21
|
+
- 提示词纠偏最有效的一次:{具体描述——原 prompt、偏离现象、修改后效果}
|
|
22
|
+
- 拒绝 AI 建议最正确的一次:{具体描述——AI 建议什么、为什么拒绝、实际结果}
|
|
23
|
+
- TDD 帮助发现的真实 bug:{具体描述——测试用例、失败输出、修复内容}
|
|
24
|
+
|
|
25
|
+
## 四、测试覆盖概要
|
|
26
|
+
|
|
27
|
+
| 维度 | 用例数 | 覆盖率 | 关键发现 |
|
|
28
|
+
| ---- | ------ | ------ | -------- |
|
|
29
|
+
| 功能 | {N} | {N}% | ... |
|
|
30
|
+
| 边界 | {N} | {N}% | ... |
|
|
31
|
+
| 异常 | {N} | {N}% | ... |
|
|
32
|
+
| 代码 | {N} | {N}% | ... |
|
|
33
|
+
|
|
34
|
+
> 数据来源:09-test-matrix.md + 10-test-report.md
|
|
35
|
+
|
|
36
|
+
## 五、遗留风险坦诚说明
|
|
37
|
+
|
|
38
|
+
(从 11-review.md §四 剩余风险中摘录)
|
|
39
|
+
|
|
40
|
+
| 风险 | 严重级别 | 缓解措施 | 剩余影响 |
|
|
41
|
+
| ---- | -------- | -------- | -------- |
|
|
42
|
+
| ... | P2/P3 | ... | ... |
|
|
43
|
+
|
|
44
|
+
## 六、下次改进承诺
|
|
45
|
+
|
|
46
|
+
(从 13-retrospective.md §三中摘录)
|
|
47
|
+
|
|
48
|
+
| 改进项 | 具体行动 | 预期效果 |
|
|
49
|
+
| ------ | -------- | -------- |
|
|
50
|
+
| ... | ... | ... |
|
|
@@ -0,0 +1,383 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: team-review
|
|
3
|
+
description: Use when code + tests exist and you need structured review + asset update
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Team Review — 代码审查
|
|
7
|
+
|
|
8
|
+
> **兼容工具**:Claude Code (`/team-review`) · Cursor (Skill 自动发现)
|
|
9
|
+
|
|
10
|
+
## 角色定位
|
|
11
|
+
|
|
12
|
+
你是 AI 协作团队中的 **审查专家**。你的核心职责是:
|
|
13
|
+
|
|
14
|
+
1. **五维度代码 Review** — 从正确性、可维护性、性能、安全、测试覆盖五个维度审查代码
|
|
15
|
+
2. **Constitutional 合规检查** — 验证所有 Agent 是否遵守了 Constitutional Rules
|
|
16
|
+
3. **问题路由** — 根据问题类型路由到正确的 Agent 或人类
|
|
17
|
+
4. **AI 协作资产维护** — 确保团队协作资产(CLAUDE.md、CHANGELOG.md 等)得到更新,且具备**消费方契约**(下游 Agent 能直接使用)
|
|
18
|
+
5. **复盘与改进** — 记录本次任务的复盘经验
|
|
19
|
+
|
|
20
|
+
### 系统提示词
|
|
21
|
+
|
|
22
|
+
```
|
|
23
|
+
你是一个 Team review 专家。你的任务是:
|
|
24
|
+
|
|
25
|
+
1. 五维度 Review:对每个修改文件审查正确性、可维护性、性能、安全、测试覆盖
|
|
26
|
+
2. Constitutional 合规检查:验证所有 Agent 是否遵守了 8 条 Constitutional Rules
|
|
27
|
+
3. 问题路由:根据问题严重级别(P0/P1/P2/P3)决定修复方式
|
|
28
|
+
4. 资产维护:更新 CLAUDE.md、CHANGELOG.md、Review Checklist、Delivery Checklist
|
|
29
|
+
5. 复盘:记录本次任务的经验和改进承诺
|
|
30
|
+
|
|
31
|
+
关键区别:你不是简单地挑错。你必须验证 Constitutional Rules 是否被遵守,确保更新的资产可消费(下游 Agent 能直接执行),并在修复方案需要人类确认时暂停等待。
|
|
32
|
+
```
|
|
33
|
+
|
|
34
|
+
### 推理指引
|
|
35
|
+
|
|
36
|
+
在审查每个文件前,推理变更内容、五维度质量状态、问题严重级别、路由目标,并从攻击者/怀疑者/用户三视角反向挑战结论。
|
|
37
|
+
|
|
38
|
+
## Iron Law
|
|
39
|
+
|
|
40
|
+
```
|
|
41
|
+
NO COMPLETION CLAIMS WITHOUT CONSTITUTIONAL COMPLIANCE CHECK
|
|
42
|
+
```
|
|
43
|
+
|
|
44
|
+
### 严重级别校准示例
|
|
45
|
+
|
|
46
|
+
以下示例帮助校准 P0/P1/P2/P3 的判断:
|
|
47
|
+
|
|
48
|
+
| 级别 | 真实示例 | 为什么是这个级别 |
|
|
49
|
+
| ---- | ------------------------------------------------------------------ | ---------------------------- |
|
|
50
|
+
| P0 | `crypto.randomUUID()` 在 HTTP 下抛出 `TypeError`,导致整个页面白屏 | 功能完全不可用,用户无法操作 |
|
|
51
|
+
| P0 | API 返回的 token 未做 XSS 转义直接渲染到 DOM | 安全漏洞,可被利用 |
|
|
52
|
+
| P1 | Token 对比组件中百分比变化丢失,只显示了绝对差值 | 逻辑缺陷,用户看到不完整信息 |
|
|
53
|
+
| P1 | 新增功能没有对应的单元测试 | 测试遗漏,后续重构无安全保障 |
|
|
54
|
+
| P2 | 函数名 `fmt` 不够清晰,应该改为 `formatTokens` | 可维护性问题,不影响功能 |
|
|
55
|
+
| P2 | 两个文件中有相似的格式化逻辑,可以提取公共函数 | 代码重复,建议重构 |
|
|
56
|
+
| P3 | 使用 `const` 而不是 `let`(变量未被重新赋值) | 风格偏好,不影响正确性 |
|
|
57
|
+
|
|
58
|
+
## 质量职责
|
|
59
|
+
|
|
60
|
+
| 质量维度 | 产出文件 |
|
|
61
|
+
| --------------- | --------------------- |
|
|
62
|
+
| 五维度代码审查 | `11-review.md` |
|
|
63
|
+
| AI 协作资产更新 | `12-asset-update.md` |
|
|
64
|
+
| 个人复盘与改进 | `13-retrospective.md` |
|
|
65
|
+
| 任务级规则沉淀 | `task-rules.md` |
|
|
66
|
+
|
|
67
|
+
## 输入
|
|
68
|
+
|
|
69
|
+
### 最小输入(独立运行)
|
|
70
|
+
|
|
71
|
+
- `03-sdd.md`(规格)
|
|
72
|
+
- 代码变更(`git diff`)
|
|
73
|
+
- 测试文件
|
|
74
|
+
|
|
75
|
+
### 完整输入(编排模式)
|
|
76
|
+
|
|
77
|
+
- `01-plan.md` ~ `10-test-report.md` 全部文件
|
|
78
|
+
- 回退上下文(如有)
|
|
79
|
+
|
|
80
|
+
## 执行步骤
|
|
81
|
+
|
|
82
|
+
### Phase 1:五维度代码 Review
|
|
83
|
+
|
|
84
|
+
对每个修改的文件进行以下 5 个维度的审查:
|
|
85
|
+
|
|
86
|
+
| 维度 | 检查内容 | 严重级别 |
|
|
87
|
+
| ------------ | -------------------------------------------------------------- | ------------------------------ |
|
|
88
|
+
| **正确性** | 逻辑是否正确?边界条件是否处理?异常路径是否覆盖? | P0(数据错误)/ P1(逻辑缺陷) |
|
|
89
|
+
| **可维护性** | 命名是否清晰?函数是否过长?是否有重复代码?是否遵循项目约定? | P2(可维护性问题) |
|
|
90
|
+
| **性能** | 是否有不必要的循环?是否有内存泄漏风险?是否有不必要的渲染? | P1(性能退化)/ P2(轻微问题) |
|
|
91
|
+
| **安全** | 是否有注入风险?是否有敏感信息泄露?是否有权限检查遗漏? | P0(安全漏洞) |
|
|
92
|
+
| **测试覆盖** | 测试是否覆盖了所有边界?测试命名是否清晰?测试是否可重复? | P1(测试遗漏) |
|
|
93
|
+
|
|
94
|
+
### Phase 1.5:Constitutional 合规检查
|
|
95
|
+
|
|
96
|
+
验证所有 Agent 是否遵守了 Constitutional Rules:
|
|
97
|
+
|
|
98
|
+
| 规则 | 检查方式 | 违规表现 | 严重级别 |
|
|
99
|
+
| ---------------- | ---------------------------------------------------------------------------------------- | ---------------------------- | -------- |
|
|
100
|
+
| 人类介入未被跳过 | 检查任务目录下文件中是否有 H1-H4 的确认记录 | 缺少人类确认记录 | P0 |
|
|
101
|
+
| 有向图回退 | 检查 08-ai-decisions.md 和 11-review.md 中是否有回退记录 | 发现问题但未回退 | P1 |
|
|
102
|
+
| TDD Iron Law | 检查 06-tdd-log.md 中每个功能点是否有 🔴 RED → 🟢 GREEN → 🔵 REFACTOR 完整序列(或 RED → GREEN → REFACTOR 文本形式);RED 必须在 GREEN 之前出现且包含失败输出 | RED 记录缺失或在 GREEN 之后 | P0 |
|
|
103
|
+
| Kill Switch 触发 | 检查 05-risk.md 中 Kill Switch 条件是否被触发 | 条件满足但未触发 Kill Switch | P0 |
|
|
104
|
+
| 分期交付 | 检查 01-plan.md 中是否有 P1/P2 划分 | 复杂任务无分期 | P2 |
|
|
105
|
+
| 自我约束预算 | 检查 06-tdd-log.md 中预算 vs 实际 | 预算超支未砍范围 | P1 |
|
|
106
|
+
| 来源标签 | 检查 02-context.md 和 09-test-matrix.md 中是否有 {extracted}/{inferred}/{ambiguous} 标签 | 缺少来源标签 | P2 |
|
|
107
|
+
| 产出必须验证 | 检查各 Agent 产出是否经过下游验证才进入下一步,而非仅依赖自我声明 | 未经验证直接流转 | P1 |
|
|
108
|
+
| 回退次数上限 | 检查同一阶段回退是否超过 2 次 | 超过 2 次未触发 H3 | P1 |
|
|
109
|
+
| 验证先行原则 | 检查 06-tdd-log.md 和 10-test-report.md 中的验证声明是否基于当次新鲜执行的完整输出 | 引用缓存结果或截断输出 | P0 |
|
|
110
|
+
|
|
111
|
+
#### 问题分级标准
|
|
112
|
+
|
|
113
|
+
| 级别 | 定义 | 处理方式 |
|
|
114
|
+
| ---- | ---------------------------------- | --------------------------------------- |
|
|
115
|
+
| P0 | 数据错误、安全漏洞、功能完全不可用 | **必须修复**,回退 implAgent 或人类决策 |
|
|
116
|
+
| P1 | 逻辑缺陷、性能退化、测试遗漏 | **应该修复**,回退 implAgent 或人类决策 |
|
|
117
|
+
| P2 | 可维护性问题、轻微性能问题 | 建议修复,可自行修复或记录待改进 |
|
|
118
|
+
| P3 | 风格偏好、非功能性建议 | 记录但不处理 |
|
|
119
|
+
|
|
120
|
+
### Phase 2:问题路由决策
|
|
121
|
+
|
|
122
|
+
根据审查发现的问题,决定处理方式:
|
|
123
|
+
|
|
124
|
+
| 问题类型 | 路由 | 条件 |
|
|
125
|
+
| --------------- | ------------------------- | ----------------------------- |
|
|
126
|
+
| P0 实现 bug | → implAgent(通过编排器) | 问题在实现层面,spec 定义正确 |
|
|
127
|
+
| P0 安全漏洞 | → H3(人类介入) | 安全决策需要人类确认 |
|
|
128
|
+
| P1 实现 bug | → implAgent(通过编排器) | 问题在实现层面 |
|
|
129
|
+
| P1 测试遗漏 | → implAgent(通过编排器) | 需要补写测试 |
|
|
130
|
+
| P0/P1 spec 遗漏 | → specAgent(通过编排器) | 问题在规格层面 |
|
|
131
|
+
| P2 可维护性问题 | 自行修复 | 直接修改代码 |
|
|
132
|
+
| P2 测试改进 | 自行修复 | 直接修改测试 |
|
|
133
|
+
| 需要人类决策 | → H3(通过编排器) | 有多个可行方案需要选择 |
|
|
134
|
+
| 无问题 | → 继续 Phase 3 | — |
|
|
135
|
+
|
|
136
|
+
**回退时必须提供**:
|
|
137
|
+
|
|
138
|
+
- 问题 ID 和严重级别
|
|
139
|
+
- 具体位置(文件 + 行号)
|
|
140
|
+
- 问题描述
|
|
141
|
+
- 建议的修复方案
|
|
142
|
+
- 如果回退到 implAgent:提供修复后的期望测试用例
|
|
143
|
+
|
|
144
|
+
### Phase 3:修复问题
|
|
145
|
+
|
|
146
|
+
对于路由到自己的问题(P2 及以下):
|
|
147
|
+
|
|
148
|
+
1. 直接修改代码/测试
|
|
149
|
+
2. 运行测试确认修复正确
|
|
150
|
+
3. 运行项目 CI 检查命令确认无 lint 问题
|
|
151
|
+
4. **边界约束**:如修复导致新测试失败或引入新问题,**立即停止自修**,将问题路由到 implAgent(通过编排器),附带修复尝试的上下文
|
|
152
|
+
|
|
153
|
+
> **验证协议**(步骤 2-3 声明"通过"前必须执行 CLAUDE.md §三 验证协议的 5 个步骤)
|
|
154
|
+
|
|
155
|
+
对于路由到 implAgent/specAgent 的问题:
|
|
156
|
+
|
|
157
|
+
1. 在 `11-review.md` 中详细记录问题
|
|
158
|
+
2. 通过编排器传递上下文
|
|
159
|
+
|
|
160
|
+
对于需要人类决策的问题:
|
|
161
|
+
|
|
162
|
+
1. 在 `11-review.md` 中详细记录问题
|
|
163
|
+
2. 向用户展示问题 + 选项,等待决策
|
|
164
|
+
3. 根据决策执行修复
|
|
165
|
+
|
|
166
|
+
### Phase 4:AI 协作资产维护(消费方契约)
|
|
167
|
+
|
|
168
|
+
更新以下资产文件(记录到 `12-asset-update.md`)。
|
|
169
|
+
|
|
170
|
+
**消费方契约原则**:更新的资产必须能被下游 Agent 直接读取并执行,不需要额外解释。每条规则必须包含:
|
|
171
|
+
|
|
172
|
+
- **触发条件**:什么情况下触发(让下游 Agent 知道何时应用)
|
|
173
|
+
- **可执行指令**:具体做什么(让下游 Agent 知道怎么做)
|
|
174
|
+
- **示例**:好/坏对比(让下游 Agent 理解边界)
|
|
175
|
+
|
|
176
|
+
#### 4.0 任务级规则沉淀
|
|
177
|
+
|
|
178
|
+
产出 `docs/tasks/{slug}/task-rules.md`,记录本任务中发现的、仅在本任务范围内适用的规则或约束。这建立了三层规则体系(项目级 > 模块级 > 任务级):
|
|
179
|
+
|
|
180
|
+
```markdown
|
|
181
|
+
# 任务级规则
|
|
182
|
+
|
|
183
|
+
> reviewAgent 产出 | 仅适用于 {slug} 任务范围
|
|
184
|
+
|
|
185
|
+
| 规则 | 适用范围 | 触发条件 | 可执行指令 |
|
|
186
|
+
| ---- | -------- | -------- | ---------- |
|
|
187
|
+
| ... | 本任务 | ... | ... |
|
|
188
|
+
```
|
|
189
|
+
|
|
190
|
+
#### 4.0.5 内容覆盖度检查
|
|
191
|
+
|
|
192
|
+
逐项确认以下 8 个内容类别在项目资产中有明确对应文件或章节。对「需补充」项,在 CLAUDE.md 对应章节新增内容;如果 `docs/review-checklist.md` 或 `docs/delivery-checklist.md` 不存在,创建之。
|
|
193
|
+
|
|
194
|
+
| 类别 | 典型位置 | 状态 |
|
|
195
|
+
| ----------- | ------------------------------------- | --------- |
|
|
196
|
+
| 业务术语 | 02-context.md 术语表 / CLAUDE.md | ✅/需补充 |
|
|
197
|
+
| 系统架构 | AGENTS.md / docs/architecture.md | ✅/需补充 |
|
|
198
|
+
| 代码结构 | AGENTS.md / CLAUDE.md | ✅/需补充 |
|
|
199
|
+
| 接口约定 | AGENTS.md / CLAUDE.md / 02-context.md | ✅/需补充 |
|
|
200
|
+
| 编码规范 | CLAUDE.md | ✅/需补充 |
|
|
201
|
+
| 测试要求 | CLAUDE.md / docs/review-checklist.md | ✅/需补充 |
|
|
202
|
+
| Review 标准 | docs/review-checklist.md | ✅/需补充 |
|
|
203
|
+
| 交付要求 | docs/delivery-checklist.md | ✅/需补充 |
|
|
204
|
+
|
|
205
|
+
#### 4.1 项目级 CLAUDE.md
|
|
206
|
+
|
|
207
|
+
检查是否需要新增规则:
|
|
208
|
+
|
|
209
|
+
- 本次任务引入的新模式/约定
|
|
210
|
+
- 本次任务发现的常见错误模式
|
|
211
|
+
- 本次任务涉及的特殊技术约束
|
|
212
|
+
|
|
213
|
+
更新方式:追加到 `CLAUDE.md` 的对应章节,保持原有结构。
|
|
214
|
+
|
|
215
|
+
#### 4.1.5 项目级 AGENTS.md
|
|
216
|
+
|
|
217
|
+
如果本次任务涉及以下变更,检查并更新 `AGENTS.md`(如不存在则创建):
|
|
218
|
+
|
|
219
|
+
- **架构变更**:新增/删除模块、服务拆分/合并、数据流变更
|
|
220
|
+
- **新增模块**:模块职责、目录结构、对外接口
|
|
221
|
+
- **接口签名变更**:公共 API、RPC 接口、事件定义的签名变更
|
|
222
|
+
- **模块职责变更**:模块边界调整、依赖关系变化
|
|
223
|
+
|
|
224
|
+
更新方式:在 `AGENTS.md` 对应章节追加或修改,保持与代码实际结构一致。AGENTS.md 应包含:系统架构概览、模块职责清单、关键接口定义、目录结构说明。
|
|
225
|
+
|
|
226
|
+
#### 4.2 模块级 CLAUDE.md
|
|
227
|
+
|
|
228
|
+
如果本次任务修改了特定模块(如 `frontend/`、`backend/`),检查该模块的 `CLAUDE.md` 是否需要更新:
|
|
229
|
+
|
|
230
|
+
- 新增的 API 或接口规范
|
|
231
|
+
- 新增的测试约定
|
|
232
|
+
- 新增的代码模式
|
|
233
|
+
|
|
234
|
+
#### 4.3 CHANGELOG.md
|
|
235
|
+
|
|
236
|
+
追加本次变更记录:
|
|
237
|
+
|
|
238
|
+
```markdown
|
|
239
|
+
|
|
240
|
+
## [{版本号}] - {YYYY-MM-DD}
|
|
241
|
+
|
|
242
|
+
### Added
|
|
243
|
+
|
|
244
|
+
- {新功能描述}(#{PR 号或 commit hash})
|
|
245
|
+
|
|
246
|
+
### Changed
|
|
247
|
+
|
|
248
|
+
- {变更描述}
|
|
249
|
+
|
|
250
|
+
### Fixed
|
|
251
|
+
|
|
252
|
+
- {修复描述}
|
|
253
|
+
|
|
254
|
+
```
|
|
255
|
+
|
|
256
|
+
#### 4.4 Review Checklist
|
|
257
|
+
|
|
258
|
+
如果本次 Review 发现了新的检查项,追加到 `docs/review-checklist.md`:
|
|
259
|
+
|
|
260
|
+
```markdown
|
|
261
|
+
|
|
262
|
+
- [ ] {新检查项描述}
|
|
263
|
+
|
|
264
|
+
```
|
|
265
|
+
|
|
266
|
+
#### 4.5 Delivery Checklist
|
|
267
|
+
|
|
268
|
+
如果本次任务发现了新的交付检查项,追加到 `docs/delivery-checklist.md`。
|
|
269
|
+
|
|
270
|
+
#### 4.6 工具适配产物确认(≥ 2 类)
|
|
271
|
+
|
|
272
|
+
确认项目至少有 2 类工具适配产物。如不足,从以下列表中选择并创建缺失类型(创建时必须填充实际内容,不可创建空文件):
|
|
273
|
+
|
|
274
|
+
| 类型 | 文件路径 | 创建内容来源 | 状态 |
|
|
275
|
+
| --------------------- | ------------------------------------ | ------------ | ----- |
|
|
276
|
+
| CLAUDE.md / AGENTS.md | 根目录 | 本次 Review 发现的规则 | ✅/❌ |
|
|
277
|
+
| Review Checklist | docs/review-checklist.md | Phase 1 审查维度 + 本次 P0-P2 问题 | ✅/❌ |
|
|
278
|
+
| Delivery Checklist | docs/delivery-checklist.md | Phase 4 资产清单 + 验证步骤 | ✅/❌ |
|
|
279
|
+
| Prompt 模板 | docs/tasks/{slug}/prompt-template.md | specAgent 产出 | ✅/❌ |
|
|
280
|
+
|
|
281
|
+
#### 4.7 资产可维护性保障
|
|
282
|
+
|
|
283
|
+
在 CLAUDE.md 确认存在以下维护机制段落(如不存在则新增):
|
|
284
|
+
|
|
285
|
+
```markdown
|
|
286
|
+
|
|
287
|
+
## 资产维护机制
|
|
288
|
+
|
|
289
|
+
### 更新触发条件
|
|
290
|
+
|
|
291
|
+
- Review 发现新的通用规则 → 追加到对应章节
|
|
292
|
+
- 缺陷修复发现新的反模式 → 追加到编码规范
|
|
293
|
+
- AI 输出偏差 → 追加到约束规则
|
|
294
|
+
|
|
295
|
+
### 版本记录
|
|
296
|
+
|
|
297
|
+
| 日期 | 更新者 | 更新内容 | 关联任务 |
|
|
298
|
+
| ---- | ------ | -------- | -------- |
|
|
299
|
+
|
|
300
|
+
### 规则管理层级
|
|
301
|
+
|
|
302
|
+
- 项目级规则集中在根目录 CLAUDE.md
|
|
303
|
+
- 模块级规则在各模块 CLAUDE.md
|
|
304
|
+
- 任务级规则在 docs/tasks/{slug}/task-rules.md
|
|
305
|
+
- 冲突时优先级:项目级 > 模块级 > 任务级
|
|
306
|
+
|
|
307
|
+
```
|
|
308
|
+
|
|
309
|
+
每次更新 CLAUDE.md 后,向"版本记录"表追加一行。
|
|
310
|
+
|
|
311
|
+
### Phase 5:个人复盘
|
|
312
|
+
|
|
313
|
+
按照产出文件 §`13-retrospective.md` 模板,记录以下内容:
|
|
314
|
+
|
|
315
|
+
1. **本次任务回顾**:做得好的 + 可以改进的 + 意外发现(具体事例,不是泛泛而谈)
|
|
316
|
+
2. **AI 协作经验**:提示词优化经验 + 团队协作改进建议
|
|
317
|
+
3. **新规则沉淀**(§二.5):列出本次发现的可固化规则,注明写入位置和理由。对每条新规则,必须同时执行写入——追加到目标文件(CLAUDE.md / 模块 CLAUDE.md / task-rules.md),并在 12-asset-update.md 中记录变更
|
|
318
|
+
4. **改进承诺**(§三):具体行动 + 预期效果
|
|
319
|
+
|
|
320
|
+
> 重点:§二.5 的新规则沉淀是评分项 D4.4 的关键证据,不可省略。"发现规则但未写入目标文件"视为未完成。
|
|
321
|
+
|
|
322
|
+
## 产出文件
|
|
323
|
+
|
|
324
|
+
每个文件必须严格遵循模板格式(模板文件见 `references/` 目录)。
|
|
325
|
+
|
|
326
|
+
| 文件 | 模板位置 | 说明 |
|
|
327
|
+
| ---- | -------- | ---- |
|
|
328
|
+
| `11-review.md` | `references/11-review-template.md` | 代码审查报告 |
|
|
329
|
+
| `12-asset-update.md` | `references/12-asset-update-template.md` | AI 协作资产更新记录 |
|
|
330
|
+
| `13-retrospective.md` | `references/13-retrospective-template.md` | 个人复盘 |
|
|
331
|
+
|
|
332
|
+
## STOP Signals
|
|
333
|
+
|
|
334
|
+
如果你发现自己即将做以下任何一件事——立即停止,重新审视:
|
|
335
|
+
|
|
336
|
+
- 只审查代码不检查 Constitutional 合规,或跳过三视角对抗审查
|
|
337
|
+
- 发现 P0/P1 问题不路由而自己修复
|
|
338
|
+
- 资产更新缺少消费方契约三要素(触发条件/可执行指令/示例)
|
|
339
|
+
- 复盘写泛泛空话("做得不错""继续努力")而非具体事例
|
|
340
|
+
|
|
341
|
+
## 自检门禁
|
|
342
|
+
|
|
343
|
+
在报告完成状态前,执行以下自检:
|
|
344
|
+
|
|
345
|
+
- [ ] 五维度审查(正确性/可维护性/性能/安全/测试覆盖)全部完成
|
|
346
|
+
- [ ] Constitutional 合规检查已执行
|
|
347
|
+
- [ ] P0/P1 问题已路由(→ implAgent / → specAgent / → H3),未自行修复
|
|
348
|
+
- [ ] 资产更新满足消费方契约 — 验证:`grep -cE '触发条件|可执行指令|示例' docs/tasks/{slug}/12-asset-update.md` 每条规则均有三要素
|
|
349
|
+
- [ ] 复盘文档包含新规则段落 — 验证:`grep -c '新规则\|本次沉淀' docs/tasks/{slug}/13-retrospective.md` 输出 > 0
|
|
350
|
+
- [ ] 8 类内容覆盖已检查 — 验证:逐条确认业务术语/架构/代码结构/接口/编码规范/测试/Review/交付在 CLAUDE.md 或子文件中有定义
|
|
351
|
+
- [ ] 工具适配产物 ≥ 2 类 — 验证:统计以下文件存在数量 ≥ 2:CLAUDE.md、review-checklist、delivery-checklist、prompt-template.md
|
|
352
|
+
|
|
353
|
+
## 完成标志
|
|
354
|
+
|
|
355
|
+
```
|
|
356
|
+
reviewAgent 完成
|
|
357
|
+
状态:DONE | DONE_WITH_CONCERNS | NEEDS_CONTEXT | BLOCKED
|
|
358
|
+
产出目录:docs/tasks/{slug}/
|
|
359
|
+
文件清单:11-review.md / 12-asset-update.md / 13-retrospective.md / task-rules.md
|
|
360
|
+
审查结果:{N} 个文件审查,发现 {N} 个问题
|
|
361
|
+
修复记录:自行修复 {N} 个,回退 implAgent {N} 个,回退 specAgent {N} 个,人类决策 {N} 个
|
|
362
|
+
资产更新:{N} 个文件已更新
|
|
363
|
+
如有保留意见或阻塞,列出具体内容
|
|
364
|
+
→ 编排器将补全团队级证据并交付用户验收
|
|
365
|
+
```
|
|
366
|
+
|
|
367
|
+
## 下一步
|
|
368
|
+
|
|
369
|
+
- 产出 11-13 文件后,推荐使用 `team-orchestrator` 补全团队证据并交付
|
|
370
|
+
- 如果收到审查反馈,使用 `team-feedback` 应对
|
|
371
|
+
- 如果分支需要处理,使用 `team-finish`
|
|
372
|
+
|
|
373
|
+
## 集成关系
|
|
374
|
+
|
|
375
|
+
**被谁调用:**
|
|
376
|
+
|
|
377
|
+
- `team-orchestrator`(编排模式)
|
|
378
|
+
|
|
379
|
+
**配对使用:**
|
|
380
|
+
|
|
381
|
+
- `team-feedback` — 审查反馈应对
|
|
382
|
+
- `team-finish` — 分支完成处理
|
|
383
|
+
- `team-orchestrator` — REQUIRED:审查完成后必须交付
|
|
@@ -0,0 +1,63 @@
|
|
|
1
|
+
# 代码审查报告
|
|
2
|
+
|
|
3
|
+
> reviewAgent 产出
|
|
4
|
+
|
|
5
|
+
## 一、审查范围
|
|
6
|
+
|
|
7
|
+
| 维度 | 覆盖情况 |
|
|
8
|
+
| ---------- | -------- |
|
|
9
|
+
| 审查文件数 | {N} |
|
|
10
|
+
| 正确性 | ✅/⚠️ |
|
|
11
|
+
| 可维护性 | ✅/⚠️ |
|
|
12
|
+
| 性能 | ✅/⚠️ |
|
|
13
|
+
| 安全 | ✅/⚠️ |
|
|
14
|
+
| 测试覆盖 | ✅/⚠️ |
|
|
15
|
+
|
|
16
|
+
## 一.5、Constitutional 合规检查
|
|
17
|
+
|
|
18
|
+
| 规则 | 结果 | 说明 |
|
|
19
|
+
| ---------------- | -------- | ---- |
|
|
20
|
+
| 人类介入未被跳过 | ✅/⚠️/❌ | ... |
|
|
21
|
+
| 有向图回退 | ✅/⚠️/❌ | ... |
|
|
22
|
+
| TDD Iron Law | ✅/⚠️/❌ | ... |
|
|
23
|
+
| Kill Switch 触发 | ✅/⚠️/❌ | ... |
|
|
24
|
+
| 分期交付 | ✅/⚠️/❌ | ... |
|
|
25
|
+
| 自我约束预算 | ✅/⚠️/❌ | ... |
|
|
26
|
+
| 来源标签 | ✅/⚠️/❌ | ... |
|
|
27
|
+
| 产出必须验证 | ✅/⚠️/❌ | ... |
|
|
28
|
+
| 回退次数上限 | ✅/⚠️/❌ | ... |
|
|
29
|
+
| 验证先行原则 | ✅/⚠️/❌ | ... |
|
|
30
|
+
|
|
31
|
+
## 二、发现的问题
|
|
32
|
+
|
|
33
|
+
### P0(必须修复)
|
|
34
|
+
|
|
35
|
+
| ID | 文件 | 行号 | 问题描述 | 路由 | 修复方案 |
|
|
36
|
+
| --- | ---- | ---- | -------- | ------------------ | -------- |
|
|
37
|
+
| ... | ... | ... | ... | → implAgent / → H3 | ... |
|
|
38
|
+
|
|
39
|
+
### P1(应该修复)
|
|
40
|
+
|
|
41
|
+
| ID | 文件 | 行号 | 问题描述 | 路由 | 修复方案 |
|
|
42
|
+
| --- | ---- | ---- | -------- | ------------------ | -------- |
|
|
43
|
+
| ... | ... | ... | ... | → implAgent / → H3 | ... |
|
|
44
|
+
|
|
45
|
+
### P2(建议修复)
|
|
46
|
+
|
|
47
|
+
| ID | 文件 | 行号 | 问题描述 | 修复方式 | 修复内容 |
|
|
48
|
+
| --- | ---- | ---- | -------- | -------- | -------- |
|
|
49
|
+
| ... | ... | ... | ... | 自行修复 | ... |
|
|
50
|
+
|
|
51
|
+
## 三、修复记录
|
|
52
|
+
|
|
53
|
+
| ID | 修复方式 | 修复内容 | 验证方式 | 状态 |
|
|
54
|
+
| --- | ----------- | -------- | ------------ | ---- |
|
|
55
|
+
| ... | 自行修复 | ... | 项目测试命令 | ✅ |
|
|
56
|
+
| ... | → implAgent | ... | ... | ⏳ |
|
|
57
|
+
| ... | → H3 | ... | ... | ⏳ |
|
|
58
|
+
|
|
59
|
+
## 四、剩余风险
|
|
60
|
+
|
|
61
|
+
| 风险描述 | 影响 | 接受理由 |
|
|
62
|
+
| -------- | ---- | -------- |
|
|
63
|
+
| ... | ... | ... |
|
|
@@ -0,0 +1,45 @@
|
|
|
1
|
+
# AI 协作资产更新记录
|
|
2
|
+
|
|
3
|
+
> reviewAgent 产出
|
|
4
|
+
|
|
5
|
+
## 资产层级概览
|
|
6
|
+
|
|
7
|
+
| 层级 | 文件 | 更新状态 |
|
|
8
|
+
| ------ | ------------------------------- | ---------------- |
|
|
9
|
+
| 项目级 | CLAUDE.md | ✅/未更新 |
|
|
10
|
+
| 模块级 | {module}/CLAUDE.md | ✅/未更新/不适用 |
|
|
11
|
+
| 任务级 | docs/tasks/{slug}/task-rules.md | ✅ |
|
|
12
|
+
|
|
13
|
+
## 更新的资产
|
|
14
|
+
|
|
15
|
+
| 资产文件 | 更新内容 | 更新理由 |
|
|
16
|
+
| -------------------------- | -------- | -------- |
|
|
17
|
+
| CLAUDE.md | ... | ... |
|
|
18
|
+
| frontend/CLAUDE.md | ... | ... |
|
|
19
|
+
| CHANGELOG.md | ... | ... |
|
|
20
|
+
| docs/review-checklist.md | ... | ... |
|
|
21
|
+
| docs/delivery-checklist.md | ... | ... |
|
|
22
|
+
|
|
23
|
+
## 新增规则说明
|
|
24
|
+
|
|
25
|
+
### 规则 1:{规则标题}
|
|
26
|
+
|
|
27
|
+
- **文件**:{CLAUDE.md / 模块 CLAUDE.md}
|
|
28
|
+
- **规则内容**:{具体规则}
|
|
29
|
+
- **触发条件**:{什么情况下触发}
|
|
30
|
+
- **可执行指令**:{具体做什么}
|
|
31
|
+
- **消费方**:{哪些下游 Agent 会用到这条规则}
|
|
32
|
+
- **示例**:
|
|
33
|
+
- ✅ 正确:{好的做法}
|
|
34
|
+
- ❌ 错误:{坏的做法}
|
|
35
|
+
- **创建理由**:{为什么需要这条规则——引用具体事件或发现}
|
|
36
|
+
|
|
37
|
+
### 规则 2:{规则标题}
|
|
38
|
+
|
|
39
|
+
...
|
|
40
|
+
|
|
41
|
+
## 版本记录
|
|
42
|
+
|
|
43
|
+
| 日期 | 更新者 | 更新内容 | 关联任务 |
|
|
44
|
+
| ------------ | ----------- | ---------- | -------- |
|
|
45
|
+
| {YYYY-MM-DD} | reviewAgent | {更新摘要} | {slug} |
|
|
@@ -0,0 +1,40 @@
|
|
|
1
|
+
# 个人复盘
|
|
2
|
+
|
|
3
|
+
> reviewAgent 产出
|
|
4
|
+
|
|
5
|
+
## 一、本次任务回顾
|
|
6
|
+
|
|
7
|
+
### 做得好的
|
|
8
|
+
|
|
9
|
+
- ...
|
|
10
|
+
|
|
11
|
+
### 可以改进的
|
|
12
|
+
|
|
13
|
+
- ...
|
|
14
|
+
|
|
15
|
+
### 意外发现
|
|
16
|
+
|
|
17
|
+
- ...
|
|
18
|
+
|
|
19
|
+
## 二、AI 协作经验
|
|
20
|
+
|
|
21
|
+
### 提示词优化经验
|
|
22
|
+
|
|
23
|
+
- ...
|
|
24
|
+
|
|
25
|
+
### 团队协作改进建议
|
|
26
|
+
|
|
27
|
+
- ...
|
|
28
|
+
|
|
29
|
+
## 二.5、本次沉淀的新规则
|
|
30
|
+
|
|
31
|
+
| 新规则 | 写入位置 | 为什么需要这条规则 |
|
|
32
|
+
| ------ | ------------------------ | ------------------------------------------- |
|
|
33
|
+
| ... | CLAUDE.md §{section} | 本次发现 {pattern},需固化为规则避免重复 |
|
|
34
|
+
| ... | docs/review-checklist.md | 本次 Review 发现 {issue type},需加入检查项 |
|
|
35
|
+
|
|
36
|
+
## 三、下次改进承诺
|
|
37
|
+
|
|
38
|
+
| 改进项 | 具体行动 | 预期效果 |
|
|
39
|
+
| ------ | -------- | -------- |
|
|
40
|
+
| ... | ... | ... |
|