job51-gitlab-cr-node-skill-prompt-optimize 1.2.8 → 1.2.9
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 +2 -1
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -25,12 +25,13 @@
|
|
|
25
25
|
> - 引入构建失败、测试失败或高危依赖漏洞
|
|
26
26
|
9. 警告问题的定义如下（优先级低于严重问题，不会造成立即崩溃或数据丢失，但存在隐患，**不包含任何空指针风险**）
|
|
27
27
|
> - 代码可读性差（如魔法数字、命名不规范、缺少必要注释）
|
|
28
|
-
> -
|
|
28
|
+
> - 性能潜在问题（如低效算法、不必要的数据库查询、内存使用不当）**但不得与严重问题行号重叠**
|
|
29
29
|
> - 边界条件处理不完善（如空集合、极端值未妥善处理）**但不包括直接导致 NPE 的情况**
|
|
30
30
|
> - 异常处理不完善（如捕获异常未记录或吞掉异常）
|
|
31
31
|
> - 资源使用不规范（如未关闭资源但无泄漏风险、连接池配置不当）
|
|
32
32
|
> - 代码重复或违反 DRY 原则
|
|
33
33
|
> - 类型安全警告（如原始类型使用、unchecked 转换）
|
|
34
|
+
> - **若某问题已在严重问题中报告，警告中不得再以其他角度（如性能、健壮性）重复描述同一变更**
|
|
34
35
|
10. **问题描述准确性规则**：
|
|
35
36
|
> - 必须准确识别代码变更引入的实际风险，不得以次要风险（如性能影响）掩盖主要风险（如空指针异常）
|
|
36
37
|
> - 示例：删除 `CollectionUtils.isEmpty(jobids)` 空检查后，若后续直接调用 `jobids.stream()`，实际风险是 **NPE** 而非"不必要的 Redis 操作"或"性能影响"
|