sdd-skills 1.1.2 → 1.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sdd-skills",
-  "version": "1.1.2",
+  "version": "1.1.4",
   "description": "Spec-Driven Development Skills for Claude Code - SAE/ADE workflow automation",
   "type": "module",
   "main": "install.js",

package/skills/backend-engineer/SKILL.md

@@ -54,6 +54,17 @@ description: 高级Golang后端工程师，负责后端API设计与实现。当
    - 单元测试（覆盖率 >= 90%）
 
 3. **验证检查（必须全部通过才能流转）**
+
+   ⚠️ **提测前必须使用项目 Taskfile 命令验证**：
+   ```bash
+   # 使用项目 Taskfile 进行快速验证（推荐）
+   task lint      # 代码规范检查
+   task build     # 编译检查
+   task test      # 运行单元测试
+   task coverage  # 检查测试覆盖率
+   ```
+
+   或手动执行以下命令：
    ```bash
    cd backend
 
@@ -90,6 +101,62 @@ description: 高级Golang后端工程师，负责后端API设计与实现。当
    - 提交所有实现文件和测试文件
    - 确保已通过所有验证检查
 
+### 阶段3：修复模式（来自 Tester 的修复请求）
+
+⚠️ **当收到 Tester 的修复请求时，执行此流程**
+
+#### 1. 分析问题
+
+- 仔细阅读 Tester 提供的错误信息
+- 定位问题代码文件和行号
+- 理解失败的测试用例和预期行为
+
+#### 2. 修复问题
+
+根据问题类型进行修复：
+
+| 问题类型 | 修复方式 |
+|---------|---------|
+| 单元测试失败 | 修复业务逻辑或修正测试断言 |
+| 覆盖率不达标 | 补充缺失的测试用例 |
+| 编译错误 | 修复语法和类型错误 |
+| Lint 错误 | 按规范修正代码风格 |
+| API 响应错误 | 修复 Handler/Service 逻辑 |
+
+#### 3. 本地验证
+
+修复后必须在本地验证：
+```bash
+# 使用 Taskfile（推荐）
+task lint && task build && task test && task coverage
+
+# 或手动执行
+cd backend
+go vet ./... && go build ./... && golangci-lint run && go test ./... -v
+go test ./... -coverprofile=coverage.out
+go tool cover -func=coverage.out | grep total  # 确保 >= 90%
+```
+
+#### 4. 自动重新提测
+
+✅ **修复并验证通过后，必须自动调用 `/tester` 重新提测**：
+
+```
+/tester 后端修复已完成，请重新执行测试验证：
+
+修复内容：
+- [具体修复的问题描述]
+- [修改的文件列表]
+
+本地验证结果：
+- 编译：✅ 通过
+- Lint：✅ 通过
+- 单元测试：✅ 全部通过
+- 覆盖率：✅ XX% (>= 90%)
+```
+
+⚠️ **重要**：修复完成后 **必须自动调用 `/tester`**，无需等待人工指令！
+
 ## 代码规范
 
 ### 项目结构
@@ -336,6 +403,9 @@ mockRepo.EXPECT().FindByEmail("test@example.com").Return(&user, nil)
 1. **上游**: 接收 SAE 生成的需求规格
 2. **并行**: 与 Frontend Engineer 并行工作，通过 API 契约协作
 3. **下游**: 代码完成后交给 Tester 进行测试
+4. **修复循环**:
+   - 收到 Tester 修复请求 → 修复问题 → 本地验证 → **自动调用** `/tester` 重新提测
+   - 循环直到测试全部通过
 
 ## 在 Pre-Execution Review 中的角色
 

package/skills/frontend-engineer/SKILL.md

@@ -70,6 +70,17 @@ description: 高级前端工程师，擅长Vue和React多个框架。当需要
    - 组件测试（覆盖率 >= 90%）
 
 3. **验证检查（必须全部通过才能流转）**
+
+   ⚠️ **提测前必须使用项目 Taskfile 命令验证**：
+   ```bash
+   # 使用项目 Taskfile 进行快速验证（推荐）
+   task lint      # 代码规范检查
+   task build     # 编译检查
+   task test      # 运行单元测试
+   task coverage  # 检查测试覆盖率
+   ```
+
+   或手动执行以下命令：
    ```bash
    cd frontend
 
@@ -105,6 +116,64 @@ description: 高级前端工程师，擅长Vue和React多个框架。当需要
    - 提交所有实现文件和测试文件
    - 确保已通过所有验证检查
 
+### 阶段3：修复模式（来自 Tester 的修复请求）
+
+⚠️ **当收到 Tester 的修复请求时，执行此流程**
+
+#### 1. 分析问题
+
+- 仔细阅读 Tester 提供的错误信息
+- 定位问题代码文件和行号
+- 理解失败的测试用例和预期行为
+
+#### 2. 修复问题
+
+根据问题类型进行修复：
+
+| 问题类型 | 修复方式 |
+|---------|---------|
+| 单元测试失败 | 修复组件逻辑或修正测试断言 |
+| 覆盖率不达标 | 补充缺失的测试用例 |
+| E2E 测试失败 | 修复页面交互逻辑、路由跳转等 |
+| TypeScript 类型错误 | 修复类型定义和类型断言 |
+| 构建失败 | 修复编译错误和依赖问题 |
+| Lint 错误 | 按规范修正代码风格 |
+
+#### 3. 本地验证
+
+修复后必须在本地验证：
+```bash
+# 使用 Taskfile（推荐）
+task lint && task build && task test && task coverage
+
+# 或手动执行
+cd frontend
+npm run type-check && npm run build && npm run lint
+npm run test:unit
+npm run test:coverage  # 确保 >= 90%
+```
+
+#### 4. 自动重新提测
+
+✅ **修复并验证通过后，必须自动调用 `/tester` 重新提测**：
+
+```
+/tester 前端修复已完成，请重新执行测试验证：
+
+修复内容：
+- [具体修复的问题描述]
+- [修改的文件列表]
+
+本地验证结果：
+- TypeScript 类型检查：✅ 通过
+- 构建：✅ 通过
+- Lint：✅ 通过
+- 单元测试：✅ 全部通过
+- 覆盖率：✅ XX% (>= 90%)
+```
+
+⚠️ **重要**：修复完成后 **必须自动调用 `/tester`**，无需等待人工指令！
+
 ## 代码规范
 
 ### 项目结构
@@ -528,6 +597,9 @@ test('user can login', async ({ page }) => {
 1. **上游**: 接收 SAE 生成的需求规格
 2. **并行**: 与 Backend Engineer 并行工作，通过 API 契约协作
 3. **下游**: 代码完成后交给 Tester 进行 E2E 测试
+4. **修复循环**:
+   - 收到 Tester 修复请求 → 修复问题 → 本地验证 → **自动调用** `/tester` 重新提测
+   - 循环直到测试全部通过
 
 ## 在 Pre-Execution Review 中的角色
 

package/skills/tester/SKILL.md

@@ -94,7 +94,65 @@ npm run test:e2e -- --project=chromium
 - [ ] 标准 5：测试覆盖率达标（后端 >= 90%, 前端 >= 90%）
 ```
 
-### 阶段4：测试报告
+### 阶段4：自动修复与重测（关键）
+
+⚠️ **重要**：测试失败时 **必须自动触发修复**，无需人工介入！
+
+#### 自动修复流程
+
+```
+测试失败 → 分析问题归属 → 自动调用对应 Skill → 重新测试 → 循环直到通过
+```
+
+#### 1. 后端问题自动修复
+
+当检测到以下后端问题时，**立即调用 `/backend-engineer`**：
+
+- 后端单元测试失败
+- 后端覆盖率不达标 (< 90%)
+- 后端集成测试失败
+- Go 编译错误、lint 错误
+- API 响应不符合预期
+
+**自动修复指令**：
+```
+/backend-engineer 修复以下测试问题：
+[具体的错误信息和失败的测试用例]
+修复完成后重新运行测试验证。
+```
+
+#### 2. 前端问题自动修复
+
+当检测到以下前端问题时，**立即调用 `/frontend-engineer`**：
+
+- 前端单元测试失败
+- 前端覆盖率不达标 (< 90%)
+- E2E 测试失败
+- TypeScript 类型错误
+- 构建失败、lint 错误
+- UI 交互问题
+
+**自动修复指令**：
+```
+/frontend-engineer 修复以下测试问题：
+[具体的错误信息和失败的测试用例]
+修复完成后重新运行测试验证。
+```
+
+#### 3. 混合问题处理
+
+如果前端和后端都有问题，**按顺序处理**：
+1. 先调用 `/backend-engineer` 修复后端问题
+2. 后端修复完成后，调用 `/frontend-engineer` 修复前端问题
+3. 全部修复后重新执行完整测试
+
+#### 4. 修复循环控制
+
+- **最大修复轮次**：3 轮
+- 超过 3 轮仍未通过，发送钉钉通知并请求人工介入
+- 每轮修复后必须重新执行完整测试
+
+### 阶段5：测试报告
 
 **测试通过**：
 ```
@@ -104,7 +162,7 @@ npm run test:e2e -- --project=chromium
 - 后端单元测试：45/45 通过
 - 后端覆盖率：92%
 - 前端单元测试：32/32 通过
-- 前端覆盖率：75%
+- 前端覆盖率：91%
 - E2E 测试：8/8 通过
 
 ✅ 验收标准：5/5 满足
@@ -112,9 +170,9 @@ npm run test:e2e -- --project=chromium
 下一步请 Code Reviewer 进行代码审查。
 ```
 
-**测试失败**：
+**测试失败（自动修复中）**：
 ```
-❌ 测试验证失败
+❌ 测试验证失败（第 1 轮）
 
 ❌ 失败详情：
 1. 后端测试失败：
@@ -125,16 +183,21 @@ npm run test:e2e -- --project=chromium
    - 后端覆盖率：85% (要求 >= 90%)
    - 未覆盖模块：user_service.go:45-60
 
-3. E2E 测试失败：
-   - 用户登录流程测试失败
-   - 原因：登录成功后未正确跳转到 /dashboard
+🔄 自动修复：调用 /backend-engineer 修复后端问题...
+
+[Backend Engineer 修复完成后自动重新测试]
+```
+
+**测试失败（需人工介入）**：
+```
+❌ 测试验证失败（已尝试 3 轮自动修复）
+
+❌ 仍存在的问题：
+[问题详情]
 
-建议操作：
-1. 返回 Backend Engineer 修复 API 错误处理
-2. 补充 user_service.go 的单元测试
-3. 返回 Frontend Engineer 修复路由跳转逻辑
+⚠️ 已达到最大修复轮次，需要人工介入。
 
-📢 已发送钉钉通知（失败告警）
+📢 已发送钉钉通知（需人工介入）
 ```
 
 ## 测试规范
@@ -418,10 +481,14 @@ lhci autorun
 ## 与其他 Skills 的协作
 
 1. **上游**: 接收 Backend Engineer 和 Frontend Engineer 的代码实现
-2. **下游**:
+2. **自动修复循环**:
+   - 后端测试失败 → **自动调用** `/backend-engineer` 修复 → 重测
+   - 前端测试失败 → **自动调用** `/frontend-engineer` 修复 → 重测
+   - 循环直到全部通过或达到最大修复轮次(3轮)
+3. **下游**:
    - 测试通过 → 交给 Code Reviewer
-   - 测试失败 → 返回相应的 Engineer 修复
-3. **并行**: 无
+   - 超过 3 轮仍失败 → 发送钉钉通知，请求人工介入
+4. **并行**: 无
 
 ## 在 Pre-Execution Review 中的角色
 
@@ -444,7 +511,7 @@ lhci autorun
 
 📊 后端测试结果：
 - 单元测试：✅ 12/12 通过
-- 覆盖率：✅ 92% (>= 90%)
+- 覆盖率：❌ 85% (要求 >= 90%)
 - 集成测试：✅ 5/5 通过
 
 📊 前端测试结果：
@@ -452,32 +519,65 @@ lhci autorun
 - 覆盖率：✅ 92% (>= 90%)
 - E2E 测试：❌ 1/3 失败
 
-❌ E2E 测试失败详情：
-- 测试：用户登录成功后跳转到 Dashboard
-- 失败原因：登录成功后停留在 /login，未跳转到 /dashboard
-- 涉及文件：frontend/src/components/LoginForm.vue:45
+❌ 测试验证失败（第 1 轮）
+
+🔄 自动修复流程启动...
+
+---
+【自动调用 /backend-engineer】
+
+/backend-engineer 修复以下测试问题：
+
+1. 后端覆盖率不达标：85% (要求 >= 90%)
+   - 未覆盖文件：user_service.go:45-60
+   - 需要补充的测试场景：
+     - 用户邮箱格式验证
+     - 密码强度验证
+
+请补充单元测试并确保覆盖率 >= 90%。
+
+---
+[Backend Engineer 修复完成]
+
+🔄 重新执行后端测试...
+
+📊 后端测试结果：
+- 单元测试：✅ 15/15 通过
+- 覆盖率：✅ 92% (>= 90%)
+- 集成测试：✅ 5/5 通过
 
-验收标准检查：
-- [x] 用户可以成功完成核心操作
-- [x] 错误情况下有友好提示
-- [x] 性能满足要求
-- [x] 通过安全测试
-- [ ] E2E 测试全部通过 ❌
+✅ 后端测试通过！
 
-建议操作：
-返回 Frontend Engineer 修复路由跳转逻辑
+---
+【自动调用 /frontend-engineer】
 
-📢 已发送钉钉通知（失败告警）
+/frontend-engineer 修复以下测试问题：
 
-[Frontend Engineer 修复后]
+1. E2E 测试失败：用户登录成功后跳转到 Dashboard
+   - 失败原因：登录成功后停留在 /login，未跳转到 /dashboard
+   - 涉及文件：frontend/src/views/LoginView.vue:45
+   - 预期行为：登录成功后应跳转到 /dashboard
+
+请修复路由跳转逻辑。
+
+---
+[Frontend Engineer 修复完成]
 
-重新执行测试...
+🔄 重新执行前端测试...
 
-✅ 所有测试通过！
+📊 前端测试结果：
+- 单元测试：✅ 8/8 通过
+- 覆盖率：✅ 92% (>= 90%)
+- E2E 测试：✅ 3/3 通过
+
+✅ 前端测试通过！
+
+---
+✅ 所有测试通过！（第 2 轮）
 
 📊 最终测试统计：
-- 后端单元测试：12/12 通过，覆盖率 92%
-- 前端单元测试：8/8 通过，覆盖率 75%
+- 后端单元测试：15/15 通过，覆盖率 92%
+- 前端单元测试：8/8 通过，覆盖率 92%
 - E2E 测试：3/3 通过
 - 验收标准：5/5 满足