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

package/mr-review-template.md

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

package/package.json

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