job51-gitlab-cr-node-skill-prompt-optimize 1.1.8 → 1.2.0
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 +4 -4
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -8,11 +8,11 @@
|
|
|
8
8
|
5. 使用当前系统时间填充 `[当前时间]`，并根据实际变更内容计算 `[文件数量]`；
|
|
9
9
|
6. 所有占位符（如 `[具体问题描述]`）必须被真实内容替换，不可保留；
|
|
10
10
|
7. 若无某类问题（如无严重问题），则**完全省略该部分**（例如不输出“🔴 严重问题”标题）；
|
|
11
|
-
8.
|
|
11
|
+
8. 若当前审核的代码块中相同级别问题有不止一个，在对应级别的标题下按原代码顺序依次展示，如存在两个严重问题时，按其在代码中的原有顺序在严重问题标题下依次展示；
|
|
12
12
|
9. 严重问题的定义如下
|
|
13
13
|
> - 存在安全漏洞（如 SQL 注入、命令注入、敏感信息泄露）
|
|
14
14
|
> - 可能导致数据丢失/损坏/不一致
|
|
15
|
-
> - 引发空指针（如：null对象调用方法/属性；null集合/数组遍历访问；包装类null
|
|
15
|
+
> - 引发空指针（如：null对象调用方法/属性；null集合/数组遍历访问；包装类null自动拆箱赋值给基础类型或直接用于条件判断；；Optional误用；方法返回值/入参未判空即使用）、资源泄漏、死锁等运行时崩溃风险
|
|
16
16
|
> - 违反事务一致性或并发安全原则
|
|
17
17
|
> - 导致核心功能失效或产生错误业务结果
|
|
18
18
|
> - 引入构建失败、测试失败或高危依赖漏洞
|
|
@@ -42,13 +42,13 @@
|
|
|
42
42
|
|
|
43
43
|
### 🔴 严重问题
|
|
44
44
|
|
|
45
|
-
[
|
|
45
|
+
[带圈数字序号，如：①，字体格式和后续问题描述一致]：[具体问题描述]<br/>
|
|
46
46
|
**文件及行号**：[问题所在文件相关代码块开始至结束行号，格式类似：ApplyPushServiceImpl.java:176-180]<br/>
|
|
47
47
|
**修改建议**：[修复建议，展示错误代码及对应修改后的示例代码，注意：错误代码和正确示例代码是必须存在的部分！！！]
|
|
48
48
|
|
|
49
49
|
### 🟡 警告
|
|
50
50
|
|
|
51
|
-
[
|
|
51
|
+
[带圈数字序号，如：①，字体格式和后续问题描述一致]：[具体问题描述]<br/>
|
|
52
52
|
**文件及行号**：[问题所在文件相关代码块开始至结束行号，格式类似：ApplyPushServiceImpl.java:176-180]<br/>
|
|
53
53
|
**修改建议**：[改进建议，展示相应代码及对应修改后的示例代码]
|
|
54
54
|
|