job51-gitlab-cr-node-skill-prompt-optimize 1.3.3 → 1.3.4
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/mr-review-template.md +10 -6
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -40,6 +40,11 @@
|
|
|
40
40
|
> - 代码重复或违反 DRY 原则
|
|
41
41
|
> - 类型安全警告（如原始类型使用、unchecked 转换）
|
|
42
42
|
> - **输出前检查**：若某行号范围的问题已在严重问题中报告，警告中**禁止**再出现该行号的任何条目
|
|
43
|
+
> - **强制排除**：警告中**不得包含**以下关键词描述的问题：
|
|
44
|
+
> - "空指针"、"NullPointerException"、"NPE"
|
|
45
|
+
> - "自动拆箱"、"Boolean 包装类"、"包装类转换"
|
|
46
|
+
> - "集合遍历"、"stream()"、"数组越界"
|
|
47
|
+
> - 任何可能导致运行时异常的描述
|
|
43
48
|
10. **问题描述准确性规则**：
|
|
44
49
|
> - 必须准确识别代码变更引入的实际风险，不得以次要风险（如性能影响）掩盖主要风险（如空指针异常）
|
|
45
50
|
> - 示例：删除 `CollectionUtils.isEmpty(jobids)` 空检查后，若后续直接调用 `jobids.stream()`，实际风险是 **NPE** 而非"不必要的 Redis 操作"或"性能影响"
|
|
@@ -66,16 +71,15 @@
|
|
|
66
71
|
> - **第二步：检查是否存在行号重叠**（完全相同或有交集即视为重叠）
|
|
67
72
|
> - **第三步：若发现重叠，仅保留最高优先级级别的问题**
|
|
68
73
|
> - 优先级顺序：严重问题 > 警告 > 优化建议
|
|
69
|
-
|
|
74
|
+
> - **强制删除**：低级别中与高级别行号重叠的条目必须删除，不得保留
|
|
75
|
+
> - **第四步：验证警告中是否包含运行时异常风险**
|
|
76
|
+
> - 检查警告问题描述中是否包含"空指针"、"NullPointerException"、"自动拆箱"等关键词
|
|
77
|
+
> - 若包含，必须将该问题升级为严重问题
|
|
70
78
|
> - **判定为同一变更的标准**（满足任一即视为同一问题）：
|
|
71
79
|
> - 文件相同且行号完全相同
|
|
72
|
-
|
|
80
|
+
> - 文件相同且行号范围重叠超过 50%
|
|
73
81
|
> - 指向同一处代码修改（如同样删除 `CollectionUtils.isEmpty()` 判断）
|
|
74
82
|
> - 问题描述存在因果关系（如"运行时异常"是"代码健壮性不足"的具体表现）
|
|
75
|
-
> - **示例**：若 87-93 行同时被识别为：
|
|
76
|
-
> - 严重问题："移除 jobids 空值检查可能导致空指针异常"
|
|
77
|
-
> - 警告："移除 jobids 空值检查可能导致后续不必要的 Redis 操作"
|
|
78
|
-
> - **处理**：仅保留严重问题，**强制删除**警告条目
|
|
79
83
|
> - **最终验证**：生成报告前必须确认严重问题、警告、优化建议三部分中**不存在任何行号重叠的条目**，如有重叠必须删除低级别条目
|
|
80
84
|
17. **代码变更方向识别规则**：
|
|
81
85
|
> - **变更内容格式说明**：
|