job51-gitlab-cr-node-skill-prompt-optimize 1.2.9 → 1.3.1

package/mr-review-template.md

@@ -25,24 +25,25 @@
    > - 引入构建失败、测试失败或高危依赖漏洞
 9. 警告问题的定义如下（优先级低于严重问题，不会造成立即崩溃或数据丢失，但存在隐患，**不包含任何空指针风险**）
    > - 代码可读性差（如魔法数字、命名不规范、缺少必要注释）
-   > - 性能潜在问题（如低效算法、不必要的数据库查询、内存使用不当）**但不得与严重问题行号重叠**
+   > - 性能潜在问题（如低效算法、不必要的数据库查询、内存使用不当）
    > - 边界条件处理不完善（如空集合、极端值未妥善处理）**但不包括直接导致 NPE 的情况**
    > - 异常处理不完善（如捕获异常未记录或吞掉异常）
    > - 资源使用不规范（如未关闭资源但无泄漏风险、连接池配置不当）
    > - 代码重复或违反 DRY 原则
    > - 类型安全警告（如原始类型使用、unchecked 转换）
-   > - **若某问题已在严重问题中报告，警告中不得再以其他角度（如性能、健壮性）重复描述同一变更**
+   > - **输出前检查**：若某行号的问题已在严重问题中报告，警告中**禁止**再出现该行号的任何条目
+   > - **强制规则**：同一行号的问题不得同时出现在严重问题和警告中
 10. **问题描述准确性规则**：
-    > - 必须准确识别代码变更引入的实际风险，不得以次要风险（如性能影响）掩盖主要风险（如空指针异常）
-    > - 示例：删除 `CollectionUtils.isEmpty(jobids)` 空检查后，若后续直接调用 `jobids.stream()`，实际风险是 **NPE** 而非"不必要的 Redis 操作"或"性能影响"
-    > - 判定原则：若代码变更可能导致运行时异常，必须优先描述异常风险，不得降级为性能或健壮性描述
+  > - 必须准确识别代码变更引入的实际风险，不得以次要风险（如性能影响）掩盖主要风险（如空指针异常）
+  > - 示例：删除 `CollectionUtils.isEmpty(jobids)` 空检查后，若后续直接调用 `jobids.stream()`，实际风险是 **NPE** 而非"不必要的 Redis 操作"或"性能影响"
+  > - 判定原则：若代码变更可能导致运行时异常，必须优先描述异常风险，不得降级为性能或健壮性描述
 11. **问题分级规则**：
-   > - 若问题涉及以下任一情况，**必须**归类为严重问题：
-   > - 可能导致 `NullPointerException`
-   > - 可能导致数据丢失或损坏
-   > - 可能导致安全漏洞
-   > - 若同一代码块存在多个问题，**全部归入最高风险级别**
-   > - 示例：若某一个代码块同时存在"集合未判空"和"Boolean 拆箱"，两者均为严重问题，应在严重问题标题下并列展示
+  > - 若问题涉及以下任一情况，**必须**归类为严重问题：
+    >   - 可能导致 **运行时异常**（`RuntimeException` 及其子类，如 `NullPointerException`、`ArrayIndexOutOfBoundsException`、`ClassCastException`、`NoSuchElementException` 等）
+    >   - 可能导致数据丢失或损坏
+    >   - 可能导致安全漏洞
+  > - 若同一代码块存在多个问题，**全部归入同一风险级别，并依次展示**
+    >   - 示例：若某一个代码块同时存在"集合未判空"和"Boolean 拆箱"，两者均为严重问题，应在严重问题标题下并列展示
 12. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束，确保内容可被程序安全提取。
 13. 需要分析的变更内容位于下方<CHANGE_CONTENT>标签内。
 14. 变更内容的行信息位于下方<LINE_INFO>标签内，里面的内容格式为：“@@ -84,9 +84,11 @@ public class UserCompetitiveAnalyseLogServiceImpl extends ServiceImpl<UserCompet”，使用变更后区域计算新代码块的起始行和结束行。
@@ -53,11 +54,20 @@
     > - 若问题涉及多行，行号范围应覆盖完整问题代码块（如 84-93）
 15. `<REPORT>`标签的每一个标题内都有两个问题块，这里只是进行示例，表示有不止一个问题时应该怎么展示，不是说每一个标题下必须有两个问题，具体问题的数量有review的结果决定，按原始代码块中对应代码的顺序依次展示即可
 16. **输出前必须执行去重检查**：
-    > - 遍历所有已识别问题，若发现同一文件及行号范围的问题出现在多个级别，仅保留最高优先级级别
-    > - 判定为同一问题的标准：文件相同且行号范围重叠，且问题描述存在包含关系（如严重问题中的"移除空集合检查可能导致后续代码出现空指针异常"已包含警告中的"移除边界条件检查可能影响代码健壮性"，后者为前者的笼统表述）
-    > - 规则：当严重问题与警告指向同一代码变更且存在描述上的包含关系时，仅保留严重问题，删除警告中的笼统表述
-    > - 示例：若 84-93 行同时被识别为"严重问题"（空指针风险）和"警告"（边界条件检查/代码健壮性），因空指针风险本身就是边界条件处理不当的具体表现，仅保留严重问题，完全删除警告中的该条目
-    > - 生成报告前必须验证：严重问题、警告、优化建议三部分中不存在行号重叠且描述存在包含关系的重复条目
+    > - **第一步：收集所有问题的文件及行号信息**
+    > - **第二步：检查是否存在行号重叠**（完全相同或有交集即视为重叠）
+    > - **第三步：若发现重叠，仅保留最高优先级级别的问题**
+        >   - 优先级顺序：严重问题 > 警告 > 优化建议
+    >   - **强制删除**：低级别中与高级别行号重叠的条目必须删除，不得保留
+    > - **判定为同一变更的标准**（满足任一即视为同一问题）：
+        >   - 文件相同且行号完全相同
+    >   - 文件相同且行号范围重叠超过 50%
+    >   - 指向同一处代码修改（如同样修改 `if (hasKey)` 判断）
+    > - **示例**：若第 88 行同时被识别为：
+        >   - 严重问题："Boolean 对象直接用于条件判断可能导致空指针异常"
+    >   - 警告："简化布尔值比较可能影响代码健壮性"
+    >   - **处理**：仅保留严重问题，**强制删除**警告条目
+    > - **最终验证**：生成报告前必须确认严重问题、警告、优化建议三部分中**不存在任何行号重叠的条目**，如有重叠必须删除低级别条目
 
 <REPORT>
 ## 🤖 AI代码审查结果

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-skill-prompt-optimize",
-  "version": "1.2.9",
+  "version": "1.3.1",
   "description": "GitLab merge request code review tool with AI-powered analysis",
   "main": "index.js",
   "bin": {