Skip to content

Commit 5fa1dd5

Browse files
authored
fix(skills): hotfix 快车道对齐分级验证制——铁门禁改复现确认,S/M 级不强制创建测试文件 (#28)
1 parent 66b20e6 commit 5fa1dd5

2 files changed

Lines changed: 111 additions & 8 deletions

File tree

  • shadow-docs/changes/20261005-fix-hotfix-tiered-validation
  • skills/shadow-dev-hotfix
Lines changed: 98 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,98 @@
1+
---
2+
{
3+
"schema": "shadow-dev/v1",
4+
"name": "20261005-fix-hotfix-tiered-validation",
5+
"type": "fix",
6+
"scope": "shadow-dev-workflow/skills",
7+
"status": "reviewed",
8+
"baseBranch": "main",
9+
"branch": "fix/20261005-fix-hotfix-tiered-validation",
10+
"files": [
11+
"shadow-docs/changes/20261005-fix-hotfix-tiered-validation/brief.md",
12+
"skills/shadow-dev-hotfix/SKILL.md"
13+
],
14+
"github": {
15+
"repository": "stack-wuh/shadow-dev-workflow",
16+
"issue": 27,
17+
"issueUrl": "https://github.com/stack-wuh/shadow-dev-workflow/issues/27",
18+
"pullRequest": null,
19+
"pullRequestUrl": null
20+
},
21+
"review": {
22+
"conclusion": "passed",
23+
"verifiedCommit": "b1bbe74c91d3afed5664167a5c2a7548225a2d95",
24+
"verifiedAt": "2026-10-06T16:13:19.208Z"
25+
},
26+
"workflow": {
27+
"operation": null,
28+
"checkpoint": "issue:27",
29+
"planHash": "09a0d15309d8aef5122cc4e153e469ca7c06b99dfaf8d79d8f034653f95657de",
30+
"updatedAt": null,
31+
"lastError": null,
32+
"issuePlan": {
33+
"title": "[fix] hotfix 快车道验证强度对齐分级制——复现测试不再一刀切强制",
34+
"titleRaw": "hotfix 快车道验证强度对齐分级制——复现测试不再一刀切强制",
35+
"supplement": "Brief: shadow-docs/changes/20261005-fix-hotfix-tiered-validation/(复现确认按分级,L 才强制先红后绿)",
36+
"body": "## 动机\nPR #26 扫描发现 `skills/shadow-dev-hotfix/SKILL.md` 存在 6 处同款一刀切强制:「复现测试先红后绿(非协商)」出现在进场说明、修复步骤、不可省清单与 mini 任务模板四处,且 mini 正文声明「复杂度评级恒 S、期望验证深度 unit」自相矛盾(S 级按分级制不创建测试)。与 `norms/tdd-verification.md` 分级制(20260925 用户批准)及 PR #26 后的新铁律直接冲突:线上文案错字、配置错误类 S 级急修也被强制写测试文件。\n\n## 引用规范\n- `norms/tdd-verification.md`\n - 当前结论: S 不创建测试(走查+扫描)/ M 绿灯测试 / L 完整 TDD 先红后绿;过度测试阻塞\n - 适用 scope: hotfix 验证强度条款的逐字对齐目标\n\n## 决策\n- **选型:** 「复现测试先红后绿」降级为 L 级条件;快车道铁门禁改为「**复现确认**」——修复前必须以可重复手段复现问题(L 级=自动化测试先红;S/M 级=人工复现步骤/观察),保证根因不失准但手段按级别\n- **对比方案:** 维持强制测试——违背用户明确指令与分级制;完全删掉复现要求——丢失快车道最重要的一条安全性\n- **理由:** 复现确认与写测试是两件事:前者防猜测式修复(hotfix 的立身之本),后者仅是其 L 级实现形式。mini 模板同步修正评级恒 S 的矛盾——评级按触及面判定,命中契约/共享路径即 L\n\n## 任务\n### Phase 1\n- [ ] hotfix skill 分级制改写——`skills/shadow-dev-hotfix/SKILL.md` — 铁门禁措辞、mini 模板、任务示例、不可省清单、根因条款五处对齐\n\n## 补充\nBrief: shadow-docs/changes/20261005-fix-hotfix-tiered-validation/(复现确认按分级,L 才强制先红后绿)\n\n完整 brief:shadow-docs/changes/20261005-fix-hotfix-tiered-validation/brief.md\n\n<!-- shadow-dev:issue-metadata {\"name\":\"20261005-fix-hotfix-tiered-validation\",\"type\":\"fix\",\"scope\":\"shadow-dev-workflow/skills\",\"status\":\"proposed\",\"branch\":null,\"baseBranch\":\"main\",\"briefPath\":\"shadow-docs/changes/20261005-fix-hotfix-tiered-validation/brief.md\",\"cliVersion\":\"1.4.0\",\"prUrl\":null,\"issueNumber\":null} -->\n",
37+
"labels": [
38+
"fix"
39+
]
40+
},
41+
"release": {
42+
"files": [
43+
"shadow-docs/changes/20261005-fix-hotfix-tiered-validation/brief.md",
44+
"skills/shadow-dev-hotfix/SKILL.md"
45+
],
46+
"message": "fix(skills): hotfix 快车道对齐分级验证制——铁门禁改复现确认,S/M 级不强制创建测试文件",
47+
"title": "fix(skills): hotfix 对齐分级验证制——复现测试不再一刀切",
48+
"body": "Closes #27\n\n完整 brief:shadow-docs/changes/20261005-fix-hotfix-tiered-validation/brief.md"
49+
}
50+
},
51+
"knowledge": {
52+
"action": "无需变更",
53+
"target": null,
54+
"reason": "治理载体对齐既有批准分级制,无新长期事实"
55+
}
56+
}
57+
---
58+
59+
# hotfix 快车道验证强度对齐分级制——复现测试不再一刀切强制
60+
61+
## 动机
62+
63+
PR #26 扫描发现 `skills/shadow-dev-hotfix/SKILL.md` 存在 6 处同款一刀切强制:「复现测试先红后绿(非协商)」出现在进场说明、修复步骤、不可省清单与 mini 任务模板四处,且 mini 正文声明「复杂度评级恒 S、期望验证深度 unit」自相矛盾(S 级按分级制不创建测试)。与 `norms/tdd-verification.md` 分级制(20260925 用户批准)及 PR #26 后的新铁律直接冲突:线上文案错字、配置错误类 S 级急修也被强制写测试文件。
64+
65+
## 复杂度评级
66+
67+
- **评级:** S
68+
- **理由:** 纯流程文档措辞对齐,零运行时契约;方向已由 norm 分级制与 iron-laws 修订批准
69+
- **期望验证深度:** code-read
70+
71+
## 引用规范
72+
73+
- `norms/tdd-verification.md`
74+
- 当前结论: S 不创建测试(走查+扫描)/ M 绿灯测试 / L 完整 TDD 先红后绿;过度测试阻塞
75+
- 适用 scope: hotfix 验证强度条款的逐字对齐目标
76+
77+
## 决策
78+
79+
- **选型:** 「复现测试先红后绿」降级为 L 级条件;快车道铁门禁改为「**复现确认**」——修复前必须以可重复手段复现问题(L 级=自动化测试先红;S/M 级=人工复现步骤/观察),保证根因不失准但手段按级别
80+
- **对比方案:** 维持强制测试——违背用户明确指令与分级制;完全删掉复现要求——丢失快车道最重要的一条安全性
81+
- **理由:** 复现确认与写测试是两件事:前者防猜测式修复(hotfix 的立身之本),后者仅是其 L 级实现形式。mini 模板同步修正评级恒 S 的矛盾——评级按触及面判定,命中契约/共享路径即 L
82+
83+
## 任务
84+
85+
### Phase 1
86+
- [x] hotfix skill 分级制改写——`skills/shadow-dev-hotfix/SKILL.md` — 铁门禁措辞、mini 模板、任务示例、不可省清单、根因条款五处对齐
87+
88+
## 结果
89+
90+
- 实际耗时: 约 8 分钟
91+
- 验证: code-read——残留 grep 归零(旧一刀切措辞 0 处);新措辞「复现确认」6 处闭合;diff 走查单文件仅命中声明的 5 个改动点
92+
- 验证: —
93+
94+
## 知识评估
95+
96+
- **预期影响:** 无需变更
97+
- **候选卡片:** 无
98+
- **理由:** 治理载体自身对齐既有批准结论,不产生新长期事实

‎skills/shadow-dev-hotfix/SKILL.md‎

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
---
22
name: shadow-dev-hotfix
3-
description: 紧急修复快车道 — 线上/阻塞问题的最小修复直达 PR。省掉知识路由、方案比较、issue 与人工多维审查,只保留复现测试、两段式确认与 PR 合入。触发词:hotfix、线上急修、紧急修复、快修、快速上线。
3+
description: 紧急修复快车道 — 线上/阻塞问题的最小修复直达 PR。省掉知识路由、方案比较、issue 与人工多维审查,只保留复现确认、两段式确认与 PR 合入。触发词:hotfix、线上急修、紧急修复、快修、快速上线。
44
---
55
# Shadow Dev Hotfix — 紧急修复快车道
66

7-
只用于线上故障或阻塞级问题。省时间不省安全:复现测试、planHash 两段式确认、PR 合入三个铁门禁一个不减。
7+
只用于线上故障或阻塞级问题。省时间不省安全:复现确认、planHash 两段式确认、PR 合入三个铁门禁一个不减。复现确认的手段按复杂度分级(`norms/tdd-verification.md`),不是一律写测试。
88

99
**执行纪律:** AI 只负责推导、决策与审查 CLI 返回的结果。一切仓库与 GitHub 写操作一律经 `shadow-dev` CLI 完成;禁止原始 `git`/`gh` 写命令,禁止脚本旁路。运行测试与检查属于收集验证证据,允许直接执行,但结论必须引用真实输出。
1010

@@ -33,14 +33,19 @@ shadow-dev branch plan --name <name>
3333
shadow-dev branch execute --name <name> --confirm
3434
```
3535

36-
mini 正文结构(复杂度评级恒 S、期望验证深度 unit;无 issue,PR 即追踪载体):
36+
mini 正文结构(复杂度评级按 norm 三要素判定——文案/配置/展示层默认 S(期望验证深度 code-read/field)、触及行为局部 M、命中契约/共享路径/宿主核心 L;无 issue,PR 即追踪载体):
3737

3838
```markdown
3939
# <一句话标题>
4040

4141
## 现象与影响
4242
<现象、影响面、发现渠道>
4343

44+
## 复杂度评级
45+
- **评级:** <S/M/L,按 norm 三要素>
46+
- **理由:** <一句话>
47+
- **期望验证深度:** <code-read|field|unit|runtime>
48+
4449
## 根因与修复
4550
<根因一句话假设,现场验证可修正;最小修复方案>
4651

@@ -51,8 +56,8 @@ mini 正文结构(复杂度评级恒 S、期望验证深度 unit;无 issue
5156

5257
## 任务
5358
### Phase 1
54-
- [ ] 复现测试先红 — `test/<文件>` — 锁定 Bug 的最小复现
55-
- [ ] 修复并回归绿 — `<目标文件>` — 最小修复;相关测试全绿
59+
- [ ] 复现锁定 Bug — `test/<文件>` 或复现步骤记录 — L 写复现测试先红;S/M 以可重复手段复现(观察/查询/手动步骤)
60+
- [ ] 修复并回归 — `<目标文件>` — 最小修复;按评级强度回归验证(L 测试先红后绿,S/M 复现手段复验)
5661

5762
## 结果
5863
- 实际耗时: —
@@ -64,9 +69,9 @@ mini 正文结构(复杂度评级恒 S、期望验证深度 unit;无 issue
6469
- **理由:** 一次性修复过程不入 Knowledge
6570
```
6671

67-
### 3. 修复(唯一保留的 TDD 铁律)
72+
### 3. 修复(复现确认铁律,强度按分级)
6873

69-
复现测试先红后绿(`norms/tdd-verification.md`,非协商);同一个 Bug 的复现、根因确认、修复、回归在同一上下文完成(`knowledge/bug-investigation.md`)。快车道不豁免进度可见性:开工报数(测试 N 项 / 任务 M 项)、逐完成一行 `▶ [TDD] n/N … red|green`、卡点先报 `⏸`(`norms/tdd-verification.md`「进度可见性」)。只修改 brief 声明的文件;发现波及面超出准入判定时停下,退回 propose 重新立项。
74+
**复现确认非协商**:修复前必须以可重复手段复现问题、确认根因不失准——**L** 级写自动化复现测试先红后绿;**M** 级绿灯测试回归;**S** 级(文案/配置/展示层)以人工复现步骤或观察记录复现并复验,**不强制创建测试文件**(`norms/tdd-verification.md` 分级制)。同一个 Bug 的复现、根因确认、修复、回归在同一上下文完成(`knowledge/bug-investigation.md`)。快车道不豁免进度可见性:开工报数(任务 M 项,有测试则另报测试 N 项)、逐完成一行 `▶ [TDD|task] …`、卡点先报 `⏸`(`norms/tdd-verification.md`「进度可见性」)。只修改 brief 声明的文件;发现波及面超出准入判定时停下,退回 propose 重新立项。
7075

7176
### 4. 收尾门禁(压缩串行)
7277

@@ -93,7 +98,7 @@ shadow-dev release execute --name <name> --confirm
9398

9499
## 不可省清单
95100

96-
- 复现测试先红后绿(`norms/tdd-verification.md`,非协商)
101+
- 复现确认先于修复(`norms/tdd-verification.md` 分级制:L 复现测试先红后绿,S/M 人工复现复验;验证强度与评级匹配非协商)
97102
- 全部写操作 plan/execute 两段式 + `--confirm`
98103
- 显式文件清单,禁止 `git add .`
99104
- main 只接受 PR 合入

0 commit comments

Comments
 (0)