job51-gitlab-cr-node-skill-prompt-optimize 1.3.0 → 1.3.1
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 +20 -18
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -31,18 +31,19 @@
|
|
|
31
31
|
> - 资源使用不规范（如未关闭资源但无泄漏风险、连接池配置不当）
|
|
32
32
|
> - 代码重复或违反 DRY 原则
|
|
33
33
|
> - 类型安全警告（如原始类型使用、unchecked 转换）
|
|
34
|
-
> -
|
|
34
|
+
> - **输出前检查**：若某行号的问题已在严重问题中报告，警告中**禁止**再出现该行号的任何条目
|
|
35
|
+
> - **强制规则**：同一行号的问题不得同时出现在严重问题和警告中
|
|
35
36
|
10. **问题描述准确性规则**：
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
37
|
+
> - 必须准确识别代码变更引入的实际风险，不得以次要风险（如性能影响）掩盖主要风险（如空指针异常）
|
|
38
|
+
> - 示例：删除 `CollectionUtils.isEmpty(jobids)` 空检查后，若后续直接调用 `jobids.stream()`，实际风险是 **NPE** 而非"不必要的 Redis 操作"或"性能影响"
|
|
39
|
+
> - 判定原则：若代码变更可能导致运行时异常，必须优先描述异常风险，不得降级为性能或健壮性描述
|
|
39
40
|
11. **问题分级规则**：
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
41
|
+
> - 若问题涉及以下任一情况，**必须**归类为严重问题：
|
|
42
|
+
> - 可能导致 **运行时异常**（`RuntimeException` 及其子类，如 `NullPointerException`、`ArrayIndexOutOfBoundsException`、`ClassCastException`、`NoSuchElementException` 等）
|
|
43
|
+
> - 可能导致数据丢失或损坏
|
|
44
|
+
> - 可能导致安全漏洞
|
|
45
|
+
> - 若同一代码块存在多个问题，**全部归入同一风险级别，并依次展示**
|
|
46
|
+
> - 示例：若某一个代码块同时存在"集合未判空"和"Boolean 拆箱"，两者均为严重问题，应在严重问题标题下并列展示
|
|
46
47
|
12. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束，确保内容可被程序安全提取。
|
|
47
48
|
13. 需要分析的变更内容位于下方<CHANGE_CONTENT>标签内。
|
|
48
49
|
14. 变更内容的行信息位于下方<LINE_INFO>标签内，里面的内容格式为：“@@ -84,9 +84,11 @@ public class UserCompetitiveAnalyseLogServiceImpl extends ServiceImpl<UserCompet”，使用变更后区域计算新代码块的起始行和结束行。
|
|
@@ -57,15 +58,16 @@
|
|
|
57
58
|
> - **第二步：检查是否存在行号重叠**（完全相同或有交集即视为重叠）
|
|
58
59
|
> - **第三步：若发现重叠，仅保留最高优先级级别的问题**
|
|
59
60
|
> - 优先级顺序：严重问题 > 警告 > 优化建议
|
|
61
|
+
> - **强制删除**：低级别中与高级别行号重叠的条目必须删除，不得保留
|
|
60
62
|
> - **判定为同一变更的标准**（满足任一即视为同一问题）：
|
|
61
|
-
> -
|
|
62
|
-
> -
|
|
63
|
-
> -
|
|
64
|
-
> -
|
|
65
|
-
> - 严重问题："
|
|
66
|
-
> - 警告："
|
|
67
|
-
> -
|
|
68
|
-
> -
|
|
63
|
+
> - 文件相同且行号完全相同
|
|
64
|
+
> - 文件相同且行号范围重叠超过 50%
|
|
65
|
+
> - 指向同一处代码修改（如同样修改 `if (hasKey)` 判断）
|
|
66
|
+
> - **示例**：若第 88 行同时被识别为：
|
|
67
|
+
> - 严重问题："Boolean 对象直接用于条件判断可能导致空指针异常"
|
|
68
|
+
> - 警告："简化布尔值比较可能影响代码健壮性"
|
|
69
|
+
> - **处理**：仅保留严重问题，**强制删除**警告条目
|
|
70
|
+
> - **最终验证**：生成报告前必须确认严重问题、警告、优化建议三部分中**不存在任何行号重叠的条目**，如有重叠必须删除低级别条目
|
|
69
71
|
|
|
70
72
|
<REPORT>
|
|
71
73
|
## 🤖 AI代码审查结果
|