sdd-skills 1.1.7 → 1.1.8
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
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: code-reviewer
|
|
3
|
-
description:
|
|
3
|
+
description: 代码审查专家,负责代码质量、安全审查和规范检查。当代码实现完成需要代码审查时激活。检查代码风格、最佳实践、安全漏洞、性能问题,审查不通过时触发钉钉通知。
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Code Reviewer
|
|
@@ -211,7 +211,7 @@ const HeavyComponent = defineAsyncComponent(() =>
|
|
|
211
211
|
- [ ] 避免魔法数字,使用常量
|
|
212
212
|
- [ ] 避免重复代码(DRY 原则)
|
|
213
213
|
- [ ] 复杂逻辑有注释说明
|
|
214
|
-
- [ ] 测试覆盖率达标(后端 >= 90%, 前端 >=
|
|
214
|
+
- [ ] 测试覆盖率达标(后端 >= 90%, 前端 >= 90%)
|
|
215
215
|
|
|
216
216
|
```go
|
|
217
217
|
// ❌ 可维护性问题:魔法数字
|
|
@@ -391,7 +391,7 @@ npm audit
|
|
|
391
391
|
- [ ] Pinia store 设计合理
|
|
392
392
|
- [ ] 类型定义完整
|
|
393
393
|
- [ ] 复杂逻辑有注释
|
|
394
|
-
- [ ] 测试覆盖率 >=
|
|
394
|
+
- [ ] 测试覆盖率 >= 90%
|
|
395
395
|
```
|
|
396
396
|
|
|
397
397
|
## 钉钉通知集成
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: git-engineer
|
|
3
|
-
description: Git工程师,负责代码提交、版本管理和MR
|
|
3
|
+
description: Git工程师,负责代码提交、版本管理和MR创建。当测试通过需要提交代码时激活。执行预检测(lint/build/test)、规范化提交、创建GitLab MR,失败时触发钉钉通知。
|
|
4
4
|
---
|
|
5
5
|
|
|
6
6
|
# Git Engineer
|
|
@@ -56,7 +56,7 @@ npm run test:unit
|
|
|
56
56
|
|
|
57
57
|
# 6. 检查覆盖率
|
|
58
58
|
npm run test:coverage
|
|
59
|
-
# 要求: >=
|
|
59
|
+
# 要求: >= 90%
|
|
60
60
|
```
|
|
61
61
|
|
|
62
62
|
✅ **预检测通过标准**:
|
|
@@ -202,7 +202,7 @@ glab mr create \
|
|
|
202
202
|
|
|
203
203
|
## 测试结果
|
|
204
204
|
- 后端覆盖率:92% (>= 90%)
|
|
205
|
-
- 前端覆盖率:
|
|
205
|
+
- 前端覆盖率:92% (>= 90%)
|
|
206
206
|
- E2E 测试:3/3 通过
|
|
207
207
|
|
|
208
208
|
## 验收标准
|
|
@@ -244,7 +244,7 @@ glab mr create
|
|
|
244
244
|
|
|
245
245
|
## 测试结果
|
|
246
246
|
- 后端覆盖率:XX% (>= 90%)
|
|
247
|
-
- 前端覆盖率:XX% (>=
|
|
247
|
+
- 前端覆盖率:XX% (>= 90%)
|
|
248
248
|
- E2E 测试:X/X 通过
|
|
249
249
|
|
|
250
250
|
## 验收标准
|
|
@@ -475,7 +475,7 @@ glab mr status
|
|
|
475
475
|
✅ 类型检查:通过
|
|
476
476
|
✅ 编译检查:通过
|
|
477
477
|
✅ 单元测试:8/8 通过
|
|
478
|
-
✅ 覆盖率:
|
|
478
|
+
✅ 覆盖率:92% (>= 90%)
|
|
479
479
|
|
|
480
480
|
✅ 所有预检测通过!
|
|
481
481
|
|