kld-sdd 2.6.2 → 2.6.4

package/templates/skills/kld-sdd/opsx-tdd-core/reference.md

@@ -82,16 +82,19 @@ description: "opsx-tdd-core 详细参考 — 执行步骤、拆分示例、DAG
 
 **策略 A**（Controller 拆 RED→GREEN，使用 @WebMvcTest）：
 
-| TASK-ID | 行为点 | 类型 | 验收标准 |
-|---------|--------|------|---------|
-| TASK-CTRL-RED-1 | POST /api/v1/auth/login 返回 Token | 测试-RED | @WebMvcTest 测试运行失败 |
-| TASK-CTRL-GREEN-1 | AuthController 接线 | 实现-GREEN | RED 测试通过 |
+| TASK-ID | 行为点 | 类型 | 依赖 | 验收标准 |
+|---------|--------|------|------|---------|
+| TASK-CTRL-RED-1 | POST /api/v1/auth/login 返回 Token | 测试-RED | TASK-05-GREEN-1 | @WebMvcTest 测试运行失败，失败原因：Controller 未实现 |
+| TASK-CTRL-GREEN-1 | AuthController.login() 接线 | 实现-GREEN | TASK-CTRL-RED-1 | RED 测试通过 |
+
+**策略 B**（Controller 先接线，再写 @WebMvcTest 验证）：
 
-**策略 B**（Controller 作为非 TDD 任务）：
+| TASK-ID | 行为点 | 类型 | 依赖 | 验收标准 |
+|---------|--------|------|------|---------|
+| TASK-CTRL-IMPL | AuthController 3 个接口接线 | 接口层 | TASK-05-GREEN-6 | Controller 编译通过，调用对应 Service 方法，返回 Result |
+| TASK-CTRL-TEST | AuthController HTTP 契约验证 | 接口测试 | TASK-CTRL-IMPL | @WebMvcTest 通过：URL 映射、请求绑定、响应 JSON 结构、错误处理 |
 
-| TASK-ID | 类型 | 验收标准 |
-|---------|------|---------|
-| TASK-CTRL | 接口层（非 TDD） | Controller 编译通过，调用对应 Service 方法，返回 Result |
+> 策略 B 本质是对 Controller 层单独应用 impl-first 模式：先实现接线，再写测试验证 HTTP 契约。
 
 ## §6 DAG 生成规则
 
@@ -112,7 +115,8 @@ description: "opsx-tdd-core 详细参考 — 执行步骤、拆分示例、DAG
 - 项目脚手架/配置 → 配置任务
 - 数据库迁移脚本（SQL DDL）→ 数据层任务
 - 前端路由/状态管理/API 封装 → UI层任务
-- Controller 层（可选）→ 接口层任务（当选择策略 B 时）
+
+> Controller 层不属于非 TDD 模块，必须通过策略 A 或策略 B 生成测试。
 
 ## §8 Telemetry 模板
 

package/templates/skills/kld-sdd/opsx-tdd-metrics/SKILL.md

@@ -7,6 +7,7 @@ description: "度量分析层 — 测试质量量化度量：隔离评分、命
 
 > **定位**：测试质量的量化度量，提供可计算的评分。
 > **参考来源**：tdd-guide `metrics_calculator.py`
+> **引用方式**：**可选引用**。本技能不参与强制流程，仅在需要量化评估测试质量时按需读取。`opsx-check` 和 `opsx-apply` 的强制检查引用 `opsx-tdd-core/checklist.md` 和 `opsx-tdd-review`，不引用本技能。
 
 ---
 

package/templates/skills/kld-sdd/opsx-tdd-review/SKILL.md

@@ -64,3 +64,42 @@ description: "测试审查层 — 从审查者角度检测'测试通过但没测
 | 不确定 | 举证责任在期望方（默认 FAIL） |
 
 > 不给予部分分数。每个断言要么通过要么失败。
+
+## §7 子代理审查提示模板
+
+> 当 `opsx-apply` 在 GREEN 任务完成后派发独立审查子代理时使用此模板。参考 Superpowers `subagent-driven-development` 的 task-reviewer 模式：implementer 自审查 ≠ 独立审查。
+
+```
+Agent (general-purpose):
+  description: "TDD 审查任务 [TASK-ID]: [任务描述]"
+  prompt: |
+    你是 TDD 测试质量审查者。你的职责是审查 implementer 的测试代码和实现代码，
+    检测"测试通过但没测到关键点"的问题。
+
+    ## 审查范围
+
+    - 任务 ID: [TASK-ID]
+    - 任务类型: [测试-RED / 实现-GREEN / 重构-REFACTOR]
+    - 审审查文件: [implementer 修改的文件列表]
+
+    ## 审查清单
+
+    执行 opsx-tdd-review/SKILL.md §3（8 项质量审查）+ §4（缺失测试检测）+ §5（测试异味检测）。
+    对于 GREEN 任务，额外检查：
+    - GREEN Scope: 生产代码中是否有未被当前 RED 断言覆盖的逻辑分支？
+    - YAGNI 围栏: 是否提前实现了后续 RED 的行为？
+    - 反模式: 执行 opsx-tdd-anti-patterns/SKILL.md §4（12 种 GREEN 后反模式）
+
+    ## 审查标准
+
+    - PASS: 需明确证据且证据反映真实任务完成
+    - FAIL: 无证据、证据矛盾、证据表面化、巧合满足
+    - 不确定: 举证责任在期望方（默认 FAIL）
+
+    ## 报告格式
+
+    - **审查结果**: PASS / FAIL / 不确定
+    - **通过项**: [列表]
+    - **失败项**: [列表，含具体文件:行号和问题描述]
+    - **建议修复**: [具体修复方案]
+```

package/templates/skills/kld-sdd/opsx-tdd-rules/rules/controller-strategy.md

@@ -2,30 +2,52 @@
 
 > 影响等级：HIGH
 
-## 策略 A：拆 RED→GREEN
+## 核心原则
+
+Controller 层**必须**有测试。两种策略的区别在于测试方式（TDD 红绿循环 vs impl-first 验证），而非"有无测试"。
+
+## 策略 A：TDD 红绿循环
 
 为每个 Controller 接口生成 RED 测试（使用 @WebMvcTest），拆为 RED→GREEN 对。
 
-适用：Controller 含业务逻辑。
+适用：Controller 含业务逻辑（参数校验、条件转发、响应转换等）。
 
 示例任务表：
 
-| TASK-ID | 行为点 | 类型 | 验收标准 |
-|---------|--------|------|---------|
-| CTRL-RED-1 | POST /api/v1/auth/login 返回 Token | 测试-RED | @WebMvcTest 测试运行失败 |
-| CTRL-GREEN-1 | AuthController 接线 | 实现-GREEN | RED 测试通过 |
+| TASK-ID | 行为点 | 类型 | 依赖 | 验收标准 |
+|---------|--------|------|------|---------|
+| CTRL-RED-1 | POST /api/v1/auth/login 返回 Token | 测试-RED | GREEN-Service | @WebMvcTest 测试运行失败，失败原因：Controller 未实现 |
+| CTRL-GREEN-1 | AuthController.login() 接线 | 实现-GREEN | CTRL-RED-1 | RED 测试通过 |
 
-## 策略 B：非 TDD 接线任务
+## 策略 B：Impl-First 接线测试
 
-Controller 作为非 TDD 模块，独立接口层任务，仅做编译检查。
+Controller 先实现接线，再写 @WebMvcTest 验证 HTTP 契约。不拆 RED→GREEN 循环，但**必须生成测试任务**。
 
 适用：Controller 为纯接线（调用 Service + 返回 Result）。
 
 示例任务表：
 
-| TASK-ID | 类型 | 验收标准 |
-|---------|------|---------|
-| CTRL | 接口层（非 TDD） | Controller 编译通过，调用对应 Service 方法，返回 Result |
+| TASK-ID | 行为点 | 类型 | 依赖 | 验收标准 |
+|---------|--------|------|------|---------|
+| CTRL-IMPL | AuthController 3 个接口接线 | 接口层 | GREEN-Service | Controller 编译通过，调用对应 Service 方法，返回 Result |
+| CTRL-TEST | AuthController HTTP 契约验证 | 接口测试 | CTRL-IMPL | @WebMvcTest 通过：URL 映射正确、请求体绑定正确、响应 JSON 结构正确、错误响应正确 |
+
+### 策略 B 测试范围
+
+@WebMvcTest 必须覆盖以下 HTTP 契约点：
+
+1. **URL 映射**：请求路径和 HTTP 方法正确
+2. **请求绑定**：`@RequestBody`/`@RequestParam`/`@PathVariable` 正确反序列化
+3. **响应结构**：`jsonPath()` 断言响应 JSON 字段
+4. **状态码**：200/201/400/401/403/404 等正确返回
+5. **错误处理**：`@ControllerAdvice`/`@ExceptionHandler` 响应正确
+
+### 策略 B 与 impl-first 的关系
+
+策略 B 本质上是**对 Controller 层单独应用 impl-first 模式**：
+- 实现任务在前（CTRL-IMPL）
+- 测试任务 Depends-On 实现任务（CTRL-TEST）
+- 测试使用 @WebMvcTest（非 @SpringBootTest）
 
 ## 声明要求
 

package/templates/skills/kld-sdd/opsx-tdd-rules/rules/dag-generation-rules.md

@@ -18,3 +18,16 @@
 4. GREEN 任务依赖对应的 RED 任务
 5. 可选每 3-5 个行为点后插入 REFACTOR
 6. 模块末尾可保留 VERIFY 任务
+
+## Controller 层 DAG 规则
+
+Controller 层根据策略生成不同的 DAG 结构：
+
+| 策略 | DAG 结构 | 说明 |
+|------|----------|------|
+| 策略 A | CTRL-RED → CTRL-GREEN | 与 Service 层相同的红绿循环 |
+| 策略 B | CTRL-IMPL → CTRL-TEST | impl-first 模式：先接线，再测试 |
+
+- 策略 A 的 CTRL-RED 依赖对应 Service 的 GREEN 任务
+- 策略 B 的 CTRL-IMPL 依赖对应 Service 的 GREEN 任务
+- 两种策略都生成测试任务，区别是测试时序（先测试 vs 先实现）

package/templates/skills/kld-sdd/opsx-tdd-rules/rules/non-tdd-modules.md

@@ -10,8 +10,9 @@
 - 项目脚手架/配置 → 配置任务
 - 数据库迁移脚本（SQL DDL）→ 数据层任务
 - 前端路由/状态管理/API 封装 → UI层任务
-- Controller 层（可选）→ 接口层任务（当选择策略 B 时）
 
-## Controller 层处理策略
+## Controller 层说明
 
-Controller 层处理策略需在 tasks.md §2.0 中明确声明，不能默认选择。
+Controller 层**不属于**非 TDD 模块。Controller 必须有测试，通过策略 A（TDD 红绿循环）或策略 B（impl-first 接线测试）实现。
+
+> Controller 策略选择见 `opsx-tdd-rules/rules/controller-strategy.md`，必须在 tasks.md §2.0 中声明。

package/templates/skills/kld-sdd/opsx-tdd-rules/rules/task-type-definitions.md

@@ -11,6 +11,13 @@
 | 重构-REFACTOR | 测试全绿下的代码清理 | 所有测试仍绿，代码结构改善 |
 | 测试-验证 | 全量测试运行验证 | 全部测试通过 |
 
+## Controller 策略 B 任务类型
+
+| 类型 | 说明 | 验收标准要求 |
+|------|------|-------------|
+| 接口层 | Controller 接线实现 | Controller 编译通过，调用对应 Service 方法，返回 Result |
+| 接口测试 | Controller HTTP 契约验证 | @WebMvcTest 通过：URL 映射、请求绑定、响应结构、错误处理 |
+
 ## RED 任务额外必填字段
 
 - 测试方法名（`{method}_{state}_{outcome}` 格式）

package/templates/skills/kld-sdd/opsx-test/SKILL.md

@@ -146,7 +146,7 @@ node skywalk-sdd/log.cjs record --type=test_result --command=test --project=. --
 - [ ] 无"生产类中加测试专用方法"
 - [ ] 无"不理解依赖就 mock"
 
-> 完整 5 项反模式检测见 opsx-tdd-anti-patterns/SKILL.md §4
+> 完整 12 项反模式检测见 opsx-tdd-anti-patterns/SKILL.md §4
 > TDD 报告模式见 opsx-tdd-review/SKILL.md §3
 
 ### 6. 【交互引导】根据结果引导下一步