dev-playbooks-cn 1.5.2 → 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/package.json
CHANGED
|
@@ -53,6 +53,34 @@ allowed-tools:
|
|
|
53
53
|
- 必须对照 `verification.md` 检查测试是否覆盖所有 AC
|
|
54
54
|
- 如发现测试缺失，明确指出缺失的 AC-ID
|
|
55
55
|
|
|
56
|
+
### CON-ROLE-004：只评审测试代码质量，不评论运行结果
|
|
57
|
+
> 核心理念：Test Reviewer 关注的是"**测试写得好不好**"，而非"**测试跑过没有**"
|
|
58
|
+
|
|
59
|
+
**禁止**：
|
|
60
|
+
- 讨论测试是 Pass/Fail、Red/Green、Skip
|
|
61
|
+
- 评论"功能未实现"或"依赖功能缺失"
|
|
62
|
+
- 给 Coder 提实现建议（如"实现 xxx 命令"、"添加 xxx 字段"）
|
|
63
|
+
- 引用工作流阶段（如"Red 基线阶段"、"这是预期行为"）
|
|
64
|
+
- 基于测试运行结果判断测试质量
|
|
65
|
+
|
|
66
|
+
**允许**：
|
|
67
|
+
- 评审测试断言的正确性和清晰度
|
|
68
|
+
- 评审测试结构和组织
|
|
69
|
+
- 评审测试覆盖的场景完整性（基于代码分析，非运行结果）
|
|
70
|
+
|
|
71
|
+
### CON-ROLE-005：覆盖率 = 测试存在性，非测试通过性
|
|
72
|
+
|
|
73
|
+
| 覆盖状态 | 定义 | 判断依据 |
|
|
74
|
+
|----------|------|----------|
|
|
75
|
+
| ✅ 已覆盖 | 存在对应 AC 的测试用例 | 测试文件中有对应的 test/it block |
|
|
76
|
+
| ⚠️ 部分覆盖 | 测试存在但场景不完整 | 缺少边界条件、错误路径等 |
|
|
77
|
+
| ❌ 缺失 | 没有对应测试用例 | 找不到对应的测试代码 |
|
|
78
|
+
|
|
79
|
+
**禁止**将以下状态纳入覆盖率判断：
|
|
80
|
+
- 测试 Skip 状态（skip 的测试仍算"已覆盖"）
|
|
81
|
+
- 测试 Fail 状态（失败的测试仍算"已覆盖"）
|
|
82
|
+
- 测试运行时间或性能
|
|
83
|
+
|
|
56
84
|
---
|
|
57
85
|
|
|
58
86
|
## 评审维度
|
|
@@ -104,11 +132,13 @@ allowed-tools:
|
|
|
104
132
|
|
|
105
133
|
## 覆盖率分析
|
|
106
134
|
|
|
135
|
+
> **注意**：覆盖状态基于测试代码存在性判断，与测试运行结果（Pass/Fail/Skip）无关。
|
|
136
|
+
|
|
107
137
|
| AC-ID | 测试文件 | 覆盖状态 | 备注 |
|
|
108
138
|
|-------|----------|----------|------|
|
|
109
|
-
| AC-001 | test-a.ts | ✅ 已覆盖 |
|
|
110
|
-
| AC-002 | - | ❌ 缺失 |
|
|
111
|
-
| AC-003 | test-b.ts | ⚠️ 部分覆盖 |
|
|
139
|
+
| AC-001 | test-a.ts | ✅ 已覆盖 | 测试用例存在 |
|
|
140
|
+
| AC-002 | - | ❌ 缺失 | 无对应测试代码 |
|
|
141
|
+
| AC-003 | test-b.ts | ⚠️ 部分覆盖 | 测试存在但缺少边界条件 |
|
|
112
142
|
|
|
113
143
|
## 问题清单
|
|
114
144
|
|
|
@@ -178,7 +208,7 @@ allowed-tools:
|
|
|
178
208
|
| Skill 名称 | devbooks-test-reviewer |
|
|
179
209
|
| 阶段 | Apply |
|
|
180
210
|
| 产物 | 评审报告（不写入变更包） |
|
|
181
|
-
| 约束 | CON-ROLE-001~
|
|
211
|
+
| 约束 | CON-ROLE-001~005 |
|
|
182
212
|
|
|
183
213
|
---
|
|
184
214
|
|