job51-gitlab-cr-node-skill-prompt-optimize 1.1.4 → 1.1.6
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 +13 -5
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -11,13 +11,21 @@
|
|
|
11
11
|
8. 严重问题的定义如下
|
|
12
12
|
> - 存在安全漏洞（如 SQL 注入、命令注入、敏感信息泄露）
|
|
13
13
|
> - 可能导致数据丢失/损坏/不一致
|
|
14
|
-
> -
|
|
14
|
+
> - 引发空指针（如：null对象调用方法/属性；null集合/数组遍历访问；包装类null自动拆箱；Optional误用；方法返回值/入参未判空即使用）、资源泄漏、死锁等运行时崩溃风险
|
|
15
15
|
> - 违反事务一致性或并发安全原则
|
|
16
16
|
> - 导致核心功能失效或产生错误业务结果
|
|
17
17
|
> - 引入构建失败、测试失败或高危依赖漏洞
|
|
18
|
-
9.
|
|
19
|
-
|
|
20
|
-
|
|
18
|
+
9. 警告问题的定义如下（优先级低于严重问题，不会造成立即崩溃或数据丢失，但存在隐患）
|
|
19
|
+
> - 代码可读性差（如魔法数字、命名不规范、缺少必要注释）
|
|
20
|
+
> - 性能潜在问题（如低效算法、不必要的数据库查询、内存使用不当）
|
|
21
|
+
> - 边界条件处理不完善（如空集合、极端值未妥善处理）
|
|
22
|
+
> - 异常处理不完善（如捕获异常未记录或吞掉异常）
|
|
23
|
+
> - 资源使用不规范（如未关闭资源但无泄漏风险、连接池配置不当）
|
|
24
|
+
> - 代码重复或违反DRY原则
|
|
25
|
+
> - 类型安全警告（如原始类型使用、unchecked 转换）
|
|
26
|
+
10. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束，确保内容可被程序安全提取。
|
|
27
|
+
11. 需要分析的变更内容位于下方<CHANGE_CONTENT>标签内。
|
|
28
|
+
12. 变更内容的行信息位于下方<LINE_INFO>标签内，里面的内容格式为：“@@ -84,9 +84,11 @@ public class UserCompetitiveAnalyseLogServiceImpl extends ServiceImpl<UserCompet”，使用变更后区域计算新代码块的起始行和结束行。
|
|
21
29
|
|
|
22
30
|
<REPORT>
|
|
23
31
|
## 🤖 AI代码审查结果
|
|
@@ -35,7 +43,7 @@
|
|
|
35
43
|
|
|
36
44
|
**文件及行号**: [问题所在文件相关代码块开始至结束行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176-180]<br/>
|
|
37
45
|
**问题描述**: [具体问题描述]<br/>
|
|
38
|
-
**修改建议**: [
|
|
46
|
+
**修改建议**: [修复建议，展示错误代码及对应修改后的示例代码，注意：错误代码和正确示例代码是必须存在的部分！！！]
|
|
39
47
|
|
|
40
48
|
### 🟡 警告
|
|
41
49
|
|