job51-gitlab-cr-node-jt-1 2.9.6 → 2.9.7

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

@@ -21,40 +21,66 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      4. `-` 开头的删除行**不计数**，行号不增加
      5. 示例：`New Start: 1` 时，第 1 行代码行号=1，第 2 行行号=2，以此类推
    - **忽略元数据行**：文件末尾的 `# File Information` 等以 `# ` 开头的行都**不编号**，直接跳过
-   - **⚠️ 临时文件格式示例**（理解计数起点）：
-     ```
-     [代码行 1]                     <-- 空格开头，行号 = 29（New Start=29）
-     + [代码行 2]                   <-- + 开头，行号 = 30（29+1）
-     + [代码行 3]                   <-- + 开头，行号 = 31（30+1）
-     [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
-     ...（diff 内容结束）
-     # File Information
-     # New Path: b/src/Test.java
-     # New Start: 29                <-- 元数据在末尾，提供行号计算起点
-     ```
    - **⚠️ 强制验证步骤**（输出前必须执行）：
      1. 从元数据提取 `New Start` 和 `New Count`，计算行号范围：`[New Start, New Start + New Count - 1]`
      2. **逐个检查**每个问题的行号是否在该范围内
      3. **如果任何行号超出范围，必须重新计数**
      4. **禁止**直接使用从完整文件读取的行号
-   - **示例**：
-     - `New Start: 1, New Count: 56` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
-     - `New Start: 167, New Count: 6` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
-   - **⚠️ 重要**：读取完整文件仅用于理解代码逻辑，**不得**直接使用完整文件中的行号
+   - **⚠️ 重要**：读取完整文件用于理解代码逻辑，但问题行号必须基于 diff 块计算
    - 如果从完整文件中定位到问题行号，**必须验证**该行号是否在 diff 块范围内
    - **验证公式**：`new_start <= 行号 <= new_start + new_count - 1`
-   - **如果行号超出范围**：必须回到 diff 块内容，重新基于 `@@` 头之后逐行计数
 
 5. **执行审查**：按照 @.claude/rules/code-review-rules.md 中的规则进行审查
 
-6. **输出结果**：
+6. **重点检测问题类型**（详见规则文件对应章节）：
+
+   **A. 资源泄漏问题**：
+   - 检测数据库连接、文件流、网络资源是否正确关闭
+   - 搜索资源创建代码，检查是否有 try-with-resources 或 finally 关闭机制
+   - 循环内创建资源未关闭 → 必须报告
+
+   **B. 框架特性问题**：
+   - Spring @Async 方法返回具体类型 → 调用方拿到 null → 必须报告
+   - MyBatis-Plus 更新实体设置了与查询条件重复的非主键字段 → 必须报告
+   - 循环中使用乐观锁版本号未每次重新获取 → 必须报告
+
+   **C. 循环逻辑问题**：
+   - 循环内调用外部对象的方法修改其状态，后续迭代继续使用该对象的状态值 → 状态过期 → 必须报告
+   - 循环内累积计算与数据库实际状态不一致 → 必须报告
+
+   **D. 异常处理问题**：
+   - catch 块仅记录日志无后续处理 → 异常吞没 → 必须报告
+   - catch 块为空实现 → 必须报告
+
+   **E. 类型转换问题**：
+   - 包装类型用于条件判断或运算可能为空 → 自动拆箱空指针 → 必须报告
+   - 高精度转低精度可能溢出 → 必须报告
+
+   **F. 集合操作问题**：
+   - 遍历集合过程中直接修改集合 → 并发修改异常 → 必须报告
+   - 空集合直接获取特定位置元素 → 索引越界 → 必须报告
+
+   **G. 并发安全问题**：
+   - 共享变量多线程访问无同步机制 → 必须报告
+   - 分布式场景使用本地锁 → 必须报告
+
+   **H. 边界条件问题**：
+   - 循环首次/末次迭代特殊处理不正确 → 必须报告
+   - 空输入、零值、边界值未正确处理 → 必须报告
+
+   **I. 空指针问题**：
+   - 遵循规则文件中的深度验证流程
+   - 外部依赖返回值默认假设安全，禁止报告
+   - 项目内方法需读取源码确认是否可能返回空值
+
+7. **输出结果**：
    - 审查结果放入 `<REPORT>` 标签
    - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
    - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 18 条
    - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
-   - **⚠️ 关键格式要求**：`<LINE_INFO>` 标签必须放置在 `</REPORT>` 标签之后，**禁止**出现在 `</REPORT>` 之前或穿插在报告内容中
+   - **⚠️ 关键格式要求**：`<LINE_INFO>` 标签必须放置在 `</REPORT>` 标签之后
 
 ## 输出模板
 
@@ -63,15 +89,10 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 1. **报告开头必须包含 `## 🤖 AI 代码审查结果`**：
    - **正确**：`<REPORT>\n## 🤖 AI 代码审查结果\n\n### 🔴 严重问题`
    - **错误**：`<REPORT>\n### 🔴 严重问题`（缺少主标题）
-   - **错误**：`<REPORT>\n## 代码审查结果`（缺少 emoji）
 
 2. **问题格式**：每个问题必须以 `**问题 N**：` 开头（N 为阿拉伯数字 1,2,3...）
-   - **正确**：`**问题 1**：`、`**问题 2**：`
-   - **错误**：`1. **xxx**`、`### 问题 1`、`**问题一**：`、`**问题 1**: xxx（行 43）`
 
-3. **每个问题块只能有一个 `**文件及行号**：` 行**：
-   - **正确**：每个问题块中 `**文件及行号**：` 只出现一次
-   - **错误**：同一个问题块中出现两次 `**文件及行号**：` 行
+3. **每个问题块只能有一个 `**文件及行号**：` 行**
 
 4. **行号标注**：**禁止**在问题描述中写"（行 X）"，行号只能通过 `<LINE_INFO>` 标签提供
 
@@ -81,20 +102,6 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 7. **必须输出 `<LINE_INFO>` 标签**：即使无问题也要输出空数组 `<LINE_INFO>[]</LINE_INFO>`
 
-8. **修改建议中的代码格式**：
-   - **必须使用 markdown 代码块**（```language ... ```）展示建议代码
-   - **禁止**使用行内代码（\`xxx\`）展示多行代码示例
-   - 代码块的语言根据文件类型自动选择（如 \`java\`、\`js\`、\`ts\`、\`py\`、\`go\` 等）
-   - 示例：
-     \`\`\`
-     **修改建议**：添加空值检查
-     \`\`\`java
-     if ([变量] != null) {
-         [方法调用];
-     }
-     \`\`\`
-     \`\`\`
-
 <REPORT>
 ## 🤖 AI 代码审查结果
 
@@ -104,19 +111,15 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
 
-**问题 2**：[问题描述]<br/>
-**文件及行号**：[文件路径：行号]<br/>
-**修改建议**：[正确示例代码或说明]
-
 </REPORT>
 
 <LINE_INFO>
-[{"new_path":"文件路径","new_line":行号,"old_path":"文件路径","old_line":行号}]
+[{"new_path":"文件路径","new_line":行号}]
 </LINE_INFO>
 
 **说明**：
 - 无问题时省略 `### 🔴 严重问题` 标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
 - 每个问题以 `**问题 N**：` 开头，方便切分
-- **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**：第 1 个元素对应问题 1 的行号，第 2 个元素对应问题 2 的行号，以此类推
-- 行号：返回变更后文件中的绝对行号（从 1 开始计数），例如文件第 43 行则 `new_line = 43`
+- **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**
+- 行号：返回变更后文件中的绝对行号（从 1 开始计数）
 - **无严重问题时，LINE_INFO 必须输出空数组 `[]`**

package/index.js

@@ -459,16 +459,25 @@ ${allReportsText}
 文件路径：${filePath}
 
 **重要审查规则**：
-1. **只审查当前 diff 块内的新增代码**（+ 开头的行）
-2. **只报告当前 diff 块内能直接发现的问题**，不要追踪方法调用链去其他地方报告问题
-3. **同一问题只报告一次**：如 import 行、声明行、调用行都有问题，只在真正会出错的调用行报告一次
-4. **禁止报告的位置**：import 语句、类定义、方法签名、依赖注入声明行
+1. **严格按照 .claude/rules/code-review-rules.md 中的规则执行审查**，该文件包含详细的审查规则
+2. **重点检测以下问题类型**（详见规则文件）：
+   - 资源泄漏问题：JDBC连接、文件流、网络资源未关闭
+   - 框架特性问题：Spring @Async返回值、MyBatis-Plus更新实体、乐观锁版本号
+   - 循环逻辑问题：循环内状态失效、累积状态不一致
+   - 异常处理问题：异常吞没、catch块空实现
+   - 类型转换问题：精度丢失、装箱拆箱空指针
+   - 集合操作问题：遍历中修改、空集合访问
+   - 并发安全问题：线程安全、分布式锁
+   - 边界条件问题：循环边界、输入边界
+   - 空指针问题：需深度分析，遵循规则文件中的验证流程
+3. **上下文读取要求**：必须使用 Read 工具读取完整方法代码，分析循环逻辑、状态管理、框架使用
+4. **问题报告位置**：只报告在 diff 块内实际会出错的代码行，禁止在 import/类定义/方法签名等位置报告
 
 **输出要求**：
 1. 严格按照 .claude/skills/simple-code-review/SKILL.md 中定义的模板格式输出
 2. **每个模块不是必须的**：没有对应问题时，完全省略该模块（不输出标题）
 3. 必须以 <REPORT> 开始，以 </REPORT> 结束
-4. **必须输出 '<LINE_INFO>' 标签**，包含所有问题的行号信息（无问题时输出空数组 [])
+4. **必须输出 '<LINE_INFO>' 标签**，包含所有问题的行号信息（无问题时输出空数组 []）
 5. 不要输出任何额外的解释、问候或总结文本`;
     //打印
     debugLog(`[${blockIdentifier}] Claude命令: ${prompt}`);

package/package.json

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