job51-gitlab-cr-node-jt-1 2.2.2 → 2.2.4

package/.claude/rules/code-review-rules.md

@@ -1,5 +1,12 @@
 # 代码审查规则
 
+**⚠️ 最高优先级强制规则（违反将导致严重误报）**：
+> - **报告运行时异常风险前，必须先用 Read 工具追踪读取相关方法/函数的实现代码**
+> - 适用场景：空指针/空引用异常、类型转换异常、集合越界等所有运行时异常
+> - 示例：看到 `obj = service.getData(id); obj.getProperty()` 时，**必须先读取** `service.getData` 方法的源码，确认其返回值是否可能为 null
+> - 如果方法/函数内部已有防护措施（如返回默认对象、空对象模式、边界检查等），则**禁止报告**该问题
+> - **禁止仅凭代码表面形式判断运行时异常风险**
+
 0. **Diff 数据结构与上下文读取规则**：
 > - **临时文件格式说明**：
     >   - 文件分为两部分：`=== File Information ===` 和 `=== Diff Content ===`
@@ -142,7 +149,7 @@
     > - **第一步：收集所有问题的文件及行号信息**
     > - **第二步：检查是否存在行号重叠**（完全相同或有交集即视为重叠）
     > - **第三步：若发现重叠，仅保留最高优先级级别的问题**
-        >   - 优先级顺序：严重问题 > 警告 > 优化建议
+        >   - 优先级顺序：严重问题 > 警告
         >   - **强制删除**：低级别中与高级别行号重叠的条目必须删除，不得保留
     > - **第四步：验证警告中是否包含运行时异常风险**
         >   - 检查警告问题描述中是否包含"空指针"、"NullPointerException"、"自动拆箱"等关键词
@@ -154,7 +161,7 @@
         >   - 检查安全问题涉及的敏感信息（如令牌、密码等）是否在变更中已被删除
         >   - 若已删除（`-` 开头），必须删除该安全问题条目
     > - **最终验证**：生成报告前必须确认：
-        >   - 严重问题、警告、优化建议中不存在任何行号重叠的条目
+        >   - 严重问题、警告中不存在任何行号重叠的条目
         >   - 所有错误代码示例展示的是变更后实际存在的代码（无 `-` 前缀）
         >   - 不存在针对已删除代码的安全问题报告
 17. **代码变更方向识别规则**：

package/.claude/skills/simple-code-review/SKILL.md

@@ -52,16 +52,4 @@ description: 你是一个专业的代码审查助手。请严格根据代码规
 **文件及行号**：[完整文件路径：起始行 - 结束行]<br/>
 **修改建议**：[改进建议 + 代码示例]
 
-### 🟢 优化建议
-
-[带圈数字序号][优化建议]<br/>
-**文件及行号**：[完整文件路径：起始行 - 结束行]<br/>
-**示例代码**：[优化后的代码]
-
-### 🔄 上下文影响分析
-
-[带圈数字序号][影响描述]<br/>
-**受影响组件**: [受影响的类/方法/模块]<br/>
-**验证建议**：[如何验证]
-
 </REPORT>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "2.2.2",
+  "version": "2.2.4",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {