job51-gitlab-cr-node-jt-1 3.0.2 → 3.0.3
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.
|
@@ -30,48 +30,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
|
|
|
30
30
|
|
|
31
31
|
5. **执行审查**：按照 @.claude/rules/code-review-rules.md 中的规则进行审查
|
|
32
32
|
|
|
33
|
-
6.
|
|
34
|
-
|
|
35
|
-
**A. 资源泄漏问题**：
|
|
36
|
-
- 检测数据库连接、文件流、网络资源是否正确关闭
|
|
37
|
-
- 搜索资源创建代码，检查是否有 try-with-resources 或 finally 关闭机制
|
|
38
|
-
- 循环内创建资源未关闭 → 必须报告
|
|
39
|
-
|
|
40
|
-
**B. 框架特性问题**：
|
|
41
|
-
- Spring @Async 方法返回具体类型 → 调用方拿到 null → 必须报告
|
|
42
|
-
- MyBatis-Plus 更新实体设置了与查询条件重复的非主键字段 → 必须报告
|
|
43
|
-
- 循环中使用乐观锁版本号未每次重新获取 → 必须报告
|
|
44
|
-
|
|
45
|
-
**C. 循环逻辑问题**：
|
|
46
|
-
- 循环内调用外部对象的方法修改其状态，后续迭代继续使用该对象的状态值 → 状态过期 → 必须报告
|
|
47
|
-
- 循环内累积计算与数据库实际状态不一致 → 必须报告
|
|
48
|
-
|
|
49
|
-
**D. 异常处理问题**：
|
|
50
|
-
- catch 块仅记录日志无后续处理 → 异常吞没 → 必须报告
|
|
51
|
-
- catch 块为空实现 → 必须报告
|
|
52
|
-
|
|
53
|
-
**E. 类型转换问题**：
|
|
54
|
-
- 包装类型用于条件判断或运算可能为空 → 自动拆箱空指针 → 必须报告
|
|
55
|
-
- 高精度转低精度可能溢出 → 必须报告
|
|
56
|
-
|
|
57
|
-
**F. 集合操作问题**：
|
|
58
|
-
- 遍历集合过程中直接修改集合 → 并发修改异常 → 必须报告
|
|
59
|
-
- 空集合直接获取特定位置元素 → 索引越界 → 必须报告
|
|
60
|
-
|
|
61
|
-
**G. 并发安全问题**：
|
|
62
|
-
- 共享变量多线程访问无同步机制 → 必须报告
|
|
63
|
-
- 分布式场景使用本地锁 → 必须报告
|
|
64
|
-
|
|
65
|
-
**H. 边界条件问题**：
|
|
66
|
-
- 循环首次/末次迭代特殊处理不正确 → 必须报告
|
|
67
|
-
- 空输入、零值、边界值未正确处理 → 必须报告
|
|
68
|
-
|
|
69
|
-
**I. 空指针问题**：
|
|
70
|
-
- 遵循规则文件中的深度验证流程
|
|
71
|
-
- 外部依赖返回值默认假设安全，禁止报告
|
|
72
|
-
- 项目内方法需读取源码确认是否可能返回空值
|
|
73
|
-
|
|
74
|
-
7. **输出结果**：
|
|
33
|
+
6. **输出结果**：
|
|
75
34
|
- 审查结果放入 `<REPORT>` 标签
|
|
76
35
|
- **⚠️ 必须输出 `<LINE_INFO>` 标签**（这是强制要求，没有例外）
|
|
77
36
|
- LINE_INFO 格式：`[{"new_path":"文件路径","new_line":行号}]`
|