dev-playbooks-cn 1.7.2 → 1.7.4
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/README.md +2 -2
- package/package.json +1 -1
- package/skills/Skills/344/275/277/347/224/250/350/257/264/346/230/216.md +2 -2
- package/skills/Skill/345/274/200/345/217/221/346/214/207/345/215/227.md +3 -3
- package/skills/_shared/MCP/345/242/236/345/274/272/346/250/241/346/235/277.md +1 -1
- package/skills/_shared/references//345/201/217/347/246/273/346/243/200/346/265/213/344/270/216/350/267/257/347/224/261/345/215/217/350/256/256.md +1 -1
- package/skills/_shared//345/267/245/344/275/234/346/265/201/344/270/213/344/270/200/346/255/245.md +3 -3
- package/skills/devbooks-brownfield-bootstrap/references//344/273/243/347/240/201/345/257/274/350/210/252/347/255/226/347/225/245.md +1 -1
- package/skills/devbooks-coder/references//346/227/245/345/277/227/350/247/204/350/214/203.md +1 -1
- package/skills/devbooks-coder/references//347/274/226/347/240/201/351/243/216/346/240/274/347/273/206/345/210/231.md +1 -1
- package/skills/devbooks-coder/references//351/224/231/350/257/257/347/240/201/350/247/204/350/214/203.md +1 -1
- package/skills/devbooks-delivery-workflow/SKILL.md +1 -1
- package/skills/devbooks-delivery-workflow/references//344/272/244/344/273/230/351/252/214/346/224/266/345/267/245/344/275/234/346/265/201.md +1 -1
- package/skills/devbooks-delivery-workflow/references//345/255/220Agent/350/260/203/347/224/250/350/247/204/350/214/203.md +1 -1
- package/skills/devbooks-design-doc/references//351/232/220/347/247/201/345/220/210/350/247/204/346/243/200/346/237/245/346/270/205/345/215/225.md +1 -1
- package/skills/devbooks-entropy-monitor/SKILL.md +1 -1
- package/skills/{devbooks-code-review → devbooks-reviewer}/SKILL.md +2 -2
- package/skills/{devbooks-code-review → devbooks-reviewer}/references/PR/346/250/241/346/235/277/344/270/216/346/214/207/345/215/227.md +1 -1
- package/skills/{devbooks-code-review → devbooks-reviewer}/references//345/235/217/345/221/263/351/201/223/351/200/237/346/237/245/350/241/250.md +2 -4
- package/skills/{devbooks-code-review → devbooks-reviewer}/references//350/265/204/346/272/220/347/256/241/347/220/206/345/256/241/346/237/245/346/270/205/345/215/225.md +1 -1
- package/skills/devbooks-router/SKILL.md +2 -2
- package/skills/devbooks-spec-contract/references/API/350/256/276/350/256/241/346/214/207/345/215/227.md +1 -1
- package/skills/devbooks-test-owner/SKILL.md +1 -1
- package/skills/devbooks-test-owner/references//346/265/213/350/257/225/345/210/206/345/261/202/347/255/226/347/225/245.md +2 -2
- package/skills/devbooks-test-owner/references//351/230/266/346/256/2652/350/257/201/346/215/256/345/256/241/350/256/241/346/270/205/345/215/225.md +1 -1
- package/templates/claude-agents/devbooks-reviewer.md +1 -1
- package/templates/claude-commands/devbooks/apply.md +2 -2
- package/templates/claude-commands/devbooks/review.md +2 -2
- package/templates/dev-playbooks/README.md +2 -2
- package/templates/dev-playbooks/docs/DevBooks/351/205/215/347/275/256/346/214/207/345/215/227.md +2 -2
- /package/skills/{devbooks-code-review → devbooks-reviewer}/references//344/273/243/347/240/201/350/257/204/345/256/241/346/217/220/347/244/272/350/257/215.md" +0 -0
package/README.md
CHANGED
|
@@ -164,7 +164,7 @@ DevBooks 使用两个目录根:
|
|
|
164
164
|
**3. Review 阶段**
|
|
165
165
|
|
|
166
166
|
```
|
|
167
|
-
请运行 devbooks-
|
|
167
|
+
请运行 devbooks-reviewer skill,变更 ID:add-oauth2
|
|
168
168
|
```
|
|
169
169
|
|
|
170
170
|
**4. Archive 阶段**
|
|
@@ -202,7 +202,7 @@ DevBooks 使用两个目录根:
|
|
|
202
202
|
|
|
203
203
|
| Skill | 说明 |
|
|
204
204
|
|-------|------|
|
|
205
|
-
| `devbooks-
|
|
205
|
+
| `devbooks-reviewer` | 代码评审(可读性/一致性) |
|
|
206
206
|
| `devbooks-test-reviewer` | 测试质量与覆盖率评审 |
|
|
207
207
|
|
|
208
208
|
### Archive 阶段
|
package/package.json
CHANGED
|
@@ -220,7 +220,7 @@ Skills 引用的共享资源(如 `_shared/references/`)位于 skills 全局
|
|
|
220
220
|
|
|
221
221
|
---
|
|
222
222
|
|
|
223
|
-
## `devbooks-
|
|
223
|
+
## `devbooks-reviewer`(Reviewer)
|
|
224
224
|
|
|
225
225
|
- 作用:以 Reviewer 角色做可读性/一致性/依赖健康/坏味道审查,只输出可执行建议,不讨论业务正确性。
|
|
226
226
|
- **热点优先审查**:审查前调用 `mcp__ckb__getHotspots` 获取项目热点,按风险排序审查
|
|
@@ -232,7 +232,7 @@ Skills 引用的共享资源(如 `_shared/references/`)位于 skills 全局
|
|
|
232
232
|
- 想发现耦合、依赖方向、复杂度、坏味道
|
|
233
233
|
- 使用话术:
|
|
234
234
|
```text
|
|
235
|
-
你现在是 Reviewer。请点名使用 `devbooks-
|
|
235
|
+
你现在是 Reviewer。请点名使用 `devbooks-reviewer`。
|
|
236
236
|
请只做可读性/一致性/依赖健康度/坏味道审查,不讨论业务正确性;不改 tests/,不改设计。
|
|
237
237
|
输入:本次变更涉及的代码 + `dev-playbooks/specs/**`(如需要项目画像/术语表/坑库)。
|
|
238
238
|
输出:严重问题 / 可维护性风险 / 一致性建议 / 需要新增的质量闸门建议。
|
|
@@ -18,7 +18,7 @@
|
|
|
18
18
|
|
|
19
19
|
| Skill 类型 | 幂等性要求 | 示例 |
|
|
20
20
|
|------------|------------|------|
|
|
21
|
-
| **验证/检查类** | 必须幂等(不修改文件) | `change-check.sh`、`guardrail-check.sh`、`devbooks-
|
|
21
|
+
| **验证/检查类** | 必须幂等(不修改文件) | `change-check.sh`、`guardrail-check.sh`、`devbooks-reviewer` |
|
|
22
22
|
| **生成类** | 必须明确"覆盖/增量"行为 | `change-scaffold.sh`、`devbooks-design-doc`、`devbooks-proposal-author` |
|
|
23
23
|
| **修改类** | 必须可安全重跑 | `devbooks-archiver`、`devbooks-design-backport` |
|
|
24
24
|
|
|
@@ -39,7 +39,7 @@
|
|
|
39
39
|
- [ ] 多次运行不产生累积副作用
|
|
40
40
|
- [ ] 提供"dry-run"模式预览变更
|
|
41
41
|
|
|
42
|
-
### 1.3
|
|
42
|
+
### 1.3 强制验证前置原则
|
|
43
43
|
|
|
44
44
|
**核心要求**:生成/修改类 Skill 在输出文件后,**必须运行验证**,验证失败时禁止进入下一步。
|
|
45
45
|
|
|
@@ -99,7 +99,7 @@ echo "ok: verification passed"
|
|
|
99
99
|
- **写入工作区**:Skill 的写入目标是 `<change-root>/<change-id>/`
|
|
100
100
|
- **归档即合并**:归档操作将工作区内容合并回真理源
|
|
101
101
|
|
|
102
|
-
### 1.6
|
|
102
|
+
### 1.6 资源清理原则
|
|
103
103
|
|
|
104
104
|
**要求**:无论成功失败,必须清理临时资源。
|
|
105
105
|
|
|
@@ -84,7 +84,7 @@
|
|
|
84
84
|
| Skill | MCP 依赖 | 增强功能 |
|
|
85
85
|
|-------|----------|----------|
|
|
86
86
|
| devbooks-coder | mcp__ckb__getHotspots | 热点文件预警 |
|
|
87
|
-
| devbooks-
|
|
87
|
+
| devbooks-reviewer | mcp__ckb__getHotspots | 热点文件高亮 |
|
|
88
88
|
| devbooks-impact-analysis | mcp__ckb__analyzeImpact, findReferences | 精确影响分析 |
|
|
89
89
|
| devbooks-brownfield-bootstrap | mcp__ckb__* | COD 模型生成 |
|
|
90
90
|
| devbooks-router | mcp__ckb__getStatus | 索引可用性检测 |
|
|
@@ -124,7 +124,7 @@ deviation-log.md 是**持久化文件**,不受 compact 影响。即使对话
|
|
|
124
124
|
| 当前 Skill | 完成状态 | 下一步 |
|
|
125
125
|
|------------|----------|--------|
|
|
126
126
|
| 任意 | COMPLETED_WITH_DEVIATION | `devbooks-design-backport` |
|
|
127
|
-
| coder | COMPLETED | `devbooks-
|
|
127
|
+
| coder | COMPLETED | `devbooks-reviewer` |
|
|
128
128
|
| coder | HANDOFF (测试问题) | `devbooks-test-owner` |
|
|
129
129
|
| test-owner | COMPLETED | `devbooks-coder` |
|
|
130
130
|
| test-owner | HANDOFF (设计问题) | `devbooks-design-backport` |
|
package/skills/_shared//345/267/245/344/275/234/346/265/201/344/270/213/344/270/200/346/255/245.md
CHANGED
|
@@ -44,9 +44,9 @@
|
|
|
44
44
|
| `devbooks-spec-contract` | `devbooks-implementation-plan` | 始终(绝不直接推荐 test-owner/coder) |
|
|
45
45
|
| `devbooks-implementation-plan` | `devbooks-test-owner` | 始终(必须单独会话) |
|
|
46
46
|
| `devbooks-test-owner` | `devbooks-coder` | Red 基线后(必须单独会话) |
|
|
47
|
-
| `devbooks-coder` | `devbooks-
|
|
48
|
-
| `devbooks-
|
|
49
|
-
| `devbooks-
|
|
47
|
+
| `devbooks-coder` | `devbooks-reviewer` | 所有任务完成后 |
|
|
48
|
+
| `devbooks-reviewer` | `devbooks-archiver` | 如果有 spec deltas |
|
|
49
|
+
| `devbooks-reviewer` | 归档完成 | 如果无 spec deltas |
|
|
50
50
|
| `devbooks-test-reviewer` | `devbooks-coder` | 如果发现测试问题,交回 |
|
|
51
51
|
| `devbooks-design-backport` | `devbooks-implementation-plan` | 设计更新后重跑计划 |
|
|
52
52
|
| `devbooks-archiver` | 归档完成 | 始终 |
|
|
@@ -163,7 +163,7 @@ allowed-tools:
|
|
|
163
163
|
| 7 | Test-Red | `devbooks-test-owner` | devbooks-test-owner | verification.md + tests/ | ✅ |
|
|
164
164
|
| 8 | Code | `devbooks-coder` | devbooks-coder | src/ 实现 | ✅ |
|
|
165
165
|
| 9 | Test-Review | `devbooks-reviewer` | devbooks-test-reviewer | 测试评审意见 | ✅ |
|
|
166
|
-
| 10 | Code-Review | `devbooks-reviewer` | devbooks-
|
|
166
|
+
| 10 | Code-Review | `devbooks-reviewer` | devbooks-reviewer | 代码评审意见 | ✅ |
|
|
167
167
|
| 11 | Green-Verify | `devbooks-test-owner` | devbooks-test-owner | evidence/green-final/ | ✅ |
|
|
168
168
|
| 12 | Archive | `devbooks-archiver` | devbooks-archiver | 归档到 archive/ | ✅ |
|
|
169
169
|
|
|
@@ -229,7 +229,7 @@ change-check.sh <change-id> --mode strict --project-root "$(pwd)" --change-root
|
|
|
229
229
|
- 运行本次范围内所有 A 类锚点(全套 tests + 静态检查 + build 校验)
|
|
230
230
|
- 静态检查优先使用机器可读输出(json/xml),便于机械式修复与可追溯存档。
|
|
231
231
|
- 执行 B/C 类:按 checklist 逐条勾选并记录证据(截图/录像/报表/签字)
|
|
232
|
-
- 可读性/依赖/坏味道审查(Reviewer):使用 `devbooks-
|
|
232
|
+
- 可读性/依赖/坏味道审查(Reviewer):使用 `devbooks-reviewer` Skill 输出审查意见
|
|
233
233
|
|
|
234
234
|
### Step 7:关账与沉淀(Close-out)
|
|
235
235
|
|
|
@@ -27,7 +27,7 @@
|
|
|
27
27
|
| 7 | devbooks-test-owner | "使用 devbooks-test-owner skill 为变更 [change-id] 编写测试并建立 Red 基线" |
|
|
28
28
|
| 8 | devbooks-coder | "使用 devbooks-coder skill 为变更 [change-id] 实现功能" |
|
|
29
29
|
| 9 | devbooks-reviewer | "使用 devbooks-test-reviewer skill 评审变更 [change-id] 的测试" |
|
|
30
|
-
| 10 | devbooks-reviewer | "使用 devbooks-
|
|
30
|
+
| 10 | devbooks-reviewer | "使用 devbooks-reviewer skill 评审变更 [change-id] 的代码" |
|
|
31
31
|
| 11 | devbooks-test-owner | "使用 devbooks-test-owner skill 运行变更 [change-id] 的所有测试并收集 Green 证据" |
|
|
32
32
|
| 12 | devbooks-archiver | "使用 devbooks-archiver skill 归档变更 [change-id]" |
|
|
33
33
|
|
|
@@ -105,7 +105,7 @@ allowed-tools:
|
|
|
105
105
|
|
|
106
106
|
| Skill | 关系 |
|
|
107
107
|
|-------|------|
|
|
108
|
-
| devbooks-
|
|
108
|
+
| devbooks-reviewer | 熵度量**不**嵌入每次 review,作为独立任务 |
|
|
109
109
|
| devbooks-proposal-author | 熵报告可作为重构提案的数据支撑 |
|
|
110
110
|
| devbooks-impact-analysis | 高熵区域变更需更谨慎的影响分析 |
|
|
111
111
|
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
|
-
name: devbooks-
|
|
3
|
-
description: devbooks-
|
|
2
|
+
name: devbooks-reviewer
|
|
3
|
+
description: devbooks-reviewer:以 Reviewer 角色做可读性/一致性/依赖健康/坏味道审查,只输出审查意见与可执行建议,不讨论业务正确性。用户说"帮我做代码评审/review 可维护性/坏味道/依赖风险/一致性建议",或在 DevBooks apply 阶段以 reviewer 执行时使用。
|
|
4
4
|
allowed-tools:
|
|
5
5
|
- Glob
|
|
6
6
|
- Grep
|
|
@@ -399,13 +399,11 @@ class PaymentService:
|
|
|
399
399
|
- 《重构:改善既有代码的设计》(第2版) - Martin Fowler
|
|
400
400
|
- 《架构整洁之道》- Robert C. Martin(第14章 组件耦合)
|
|
401
401
|
- dev-playbooks 辩论修订版评估报告
|
|
402
|
-
- devbooks-
|
|
402
|
+
- devbooks-reviewer 检查清单
|
|
403
403
|
|
|
404
404
|
---
|
|
405
405
|
|
|
406
|
-
## 10.
|
|
407
|
-
|
|
408
|
-
> 借鉴 VS Code ESLint 自定义规则,作为代码评审的自动化检查项
|
|
406
|
+
## 10. 代码卫生检查
|
|
409
407
|
|
|
410
408
|
### 禁止提交的模式(阻塞级)
|
|
411
409
|
|
|
@@ -68,7 +68,7 @@ fi
|
|
|
68
68
|
|
|
69
69
|
**降级模式说明**:
|
|
70
70
|
- 无索引时,`devbooks-impact-analysis` 退化为 Grep 文本搜索(准确度下降)
|
|
71
|
-
- 无索引时,`devbooks-
|
|
71
|
+
- 无索引时,`devbooks-reviewer` 无法获取调用图上下文
|
|
72
72
|
- 建议在 Apply 阶段前完成索引生成
|
|
73
73
|
|
|
74
74
|
## 你要做的事
|
|
@@ -228,7 +228,7 @@ LSC(大规模同质化修改)建议:
|
|
|
228
228
|
触发信号:用户说"review/坏味道/可维护性/依赖风险/一致性"等。
|
|
229
229
|
|
|
230
230
|
默认路由:
|
|
231
|
-
- `devbooks-
|
|
231
|
+
- `devbooks-reviewer`(输出可执行建议;不改业务结论、不改 tests)
|
|
232
232
|
- `devbooks-test-reviewer`(评审测试质量、覆盖率、边界条件)
|
|
233
233
|
|
|
234
234
|
### D) Docs Sync(文档同步)
|
|
@@ -231,7 +231,7 @@ proposal → design → [TEST-OWNER] → [CODER] → [TEST-OWNER] → code-revie
|
|
|
231
231
|
|
|
232
232
|
| 状态码 | 状态 | 判定条件 | 下一步 |
|
|
233
233
|
|:------:|------|----------|--------|
|
|
234
|
-
| ✅ | PHASE2_VERIFIED | 证据审计通过,AC 矩阵已打勾 | `devbooks-
|
|
234
|
+
| ✅ | PHASE2_VERIFIED | 证据审计通过,AC 矩阵已打勾 | `devbooks-reviewer` |
|
|
235
235
|
| ⏳ | PHASE2_WAITING | @full 测试仍在运行 | 等待 CI 完成 |
|
|
236
236
|
| ❌ | PHASE2_FAILED | @full 测试未通过 | 通知 Coder 修复 |
|
|
237
237
|
| 🔄 | PHASE2_HANDOFF | 发现测试本身有问题 | 修复测试后重新验证 |
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
# 测试分层策略
|
|
2
2
|
|
|
3
|
-
|
|
3
|
+
本文档定义了测试分层的最佳实践。
|
|
4
4
|
|
|
5
5
|
---
|
|
6
6
|
|
|
@@ -234,7 +234,7 @@ if (process.env.NODE_ENV === 'development') {
|
|
|
234
234
|
|
|
235
235
|
---
|
|
236
236
|
|
|
237
|
-
## 6)
|
|
237
|
+
## 6) 测试反脆弱策略
|
|
238
238
|
|
|
239
239
|
### 避免脆弱的选择器
|
|
240
240
|
|
|
@@ -101,7 +101,7 @@ npm test -- --grep "@critical"
|
|
|
101
101
|
|
|
102
102
|
| 状态码 | 状态 | 判定条件 | 下一步 |
|
|
103
103
|
|:------:|------|----------|--------|
|
|
104
|
-
| ✅ | PHASE2_VERIFIED | 证据审计通过,AC 矩阵已打勾 | `devbooks-
|
|
104
|
+
| ✅ | PHASE2_VERIFIED | 证据审计通过,AC 矩阵已打勾 | `devbooks-reviewer` |
|
|
105
105
|
| ⏳ | PHASE2_WAITING | @full 测试仍在运行 | 等待 CI 完成 |
|
|
106
106
|
| ❌ | PHASE2_FAILED | @full 测试未通过 | 通知 Coder 修复 |
|
|
107
107
|
| 🔄 | PHASE2_HANDOFF | 发现测试本身有问题 | 修复测试后重新验证 |
|
|
@@ -3,7 +3,7 @@ skill: multi-skill-combo
|
|
|
3
3
|
skills:
|
|
4
4
|
- devbooks-test-owner
|
|
5
5
|
- devbooks-coder
|
|
6
|
-
- devbooks-
|
|
6
|
+
- devbooks-reviewer
|
|
7
7
|
---
|
|
8
8
|
|
|
9
9
|
# DevBooks: 应用变更(向后兼容)
|
|
@@ -23,7 +23,7 @@ $ARGUMENTS
|
|
|
23
23
|
这是一个向后兼容命令,触发多 Skill 组合:
|
|
24
24
|
1. devbooks-test-owner:创建验收测试
|
|
25
25
|
2. devbooks-coder:实现功能代码
|
|
26
|
-
3. devbooks-
|
|
26
|
+
3. devbooks-reviewer:代码评审
|
|
27
27
|
|
|
28
28
|
建议使用更精确的直达命令:
|
|
29
29
|
- /devbooks:test - 验收测试
|
|
@@ -154,7 +154,7 @@ DevBooks 使用两个目录根:
|
|
|
154
154
|
**3. Review 阶段**
|
|
155
155
|
|
|
156
156
|
```
|
|
157
|
-
请运行 devbooks-
|
|
157
|
+
请运行 devbooks-reviewer skill,变更 ID:add-oauth2
|
|
158
158
|
```
|
|
159
159
|
|
|
160
160
|
**4. Archive 阶段**
|
|
@@ -192,7 +192,7 @@ DevBooks 使用两个目录根:
|
|
|
192
192
|
|
|
193
193
|
| Skill | 说明 |
|
|
194
194
|
|-------|------|
|
|
195
|
-
| `devbooks-
|
|
195
|
+
| `devbooks-reviewer` | 代码评审(可读性/一致性) |
|
|
196
196
|
| `devbooks-test-reviewer` | 测试质量与覆盖率评审 |
|
|
197
197
|
|
|
198
198
|
### Archive 阶段
|
package/templates/dev-playbooks/docs/DevBooks/351/205/215/347/275/256/346/214/207/345/215/227.md
CHANGED
|
@@ -131,7 +131,7 @@ change_root: dev-playbooks/changes/ # 变更包目录根
|
|
|
131
131
|
| Planner | `devbooks-implementation-plan` | `tasks.md` |
|
|
132
132
|
| Test Owner | `devbooks-test-owner` | `verification.md` + `tests/` |
|
|
133
133
|
| Coder | `devbooks-coder` | 按 tasks 实现(禁止改 tests) |
|
|
134
|
-
| Reviewer | `devbooks-
|
|
134
|
+
| Reviewer | `devbooks-reviewer` | 评审意见 |
|
|
135
135
|
| Archiver | `devbooks-archiver` | 归档修剪 + C4 合并 |
|
|
136
136
|
| Design Backport | `devbooks-design-backport` | 回写设计缺口 |
|
|
137
137
|
|
|
@@ -185,6 +185,6 @@ AI 可以根据用户意图自动选择 Skills:
|
|
|
185
185
|
| 用户意图 | 自动路由 |
|
|
186
186
|
|----------|----------|
|
|
187
187
|
| "修 bug"、"定位问题" | `devbooks-impact-analysis` → `devbooks-coder` |
|
|
188
|
-
| "重构"、"优化代码" | `devbooks-
|
|
188
|
+
| "重构"、"优化代码" | `devbooks-reviewer` → `devbooks-coder` |
|
|
189
189
|
| "新功能"、"实现 XX" | `devbooks-router` → 输出闭环路线 |
|
|
190
190
|
| "写测试"、"补测试" | `devbooks-test-owner` |
|