job51-gitlab-cr-node-jt-1 2.9.8 → 3.0.0

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

@@ -11,26 +11,22 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
    - `New Start`: 当前 diff 块在变更后文件中的起始行号
    - `New Count`: 当前 diff 块的行数
    - `New Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
+   - `Is Large File`: 如果为 true，表示这是完整文件审查（GitLab diff 截断）
    - **注意**：元数据在文件末尾，`@@` 行已被移除，不影响行号计数
 
-3. **解析行号**：**基于 diff 块内容逐行计数，计算绝对行号**
-   - **行号计算方法**：
-     1. 从元数据中读取 `New Start` 值
-     2. 从第 1 行代码开始，行号 = `New Start`
-     3. 每往下一行（`+` 开头或空格开头），行号 +1
-     4. `-` 开头的删除行**不计数**，行号不增加
-     5. 示例：`New Start: 1` 时，第 1 行代码行号=1，第 2 行行号=2，以此类推
-   - **忽略元数据行**：文件末尾的 `# File Information` 等以 `# ` 开头的行都**不编号**，直接跳过
-   - **⚠️ 强制验证步骤**（输出前必须执行）：
-     1. 从元数据提取 `New Start` 和 `New Count`，计算行号范围：`[New Start, New Start + New Count - 1]`
-     2. **逐个检查**每个问题的行号是否在该范围内
-     3. **如果任何行号超出范围，必须重新计数**
-     4. **禁止**直接使用从完整文件读取的行号
+3. **解析行号**：
+   - **大文件模式（Is Large File: true）**：
+     - 这是完整文件内容审查，所有代码都是新增
+     - 行号从 1 开始，逐行计数
+     - 发现问题时，直接使用实际行号
+   - **正常 diff 模式**：
+     - 从元数据中读取 `New Start` 值
+     - 从第 1 行代码开始，行号 = `New Start`
+     - 每往下一行（`+` 开头或空格开头），行号 +1
+     - `-` 开头的删除行**不计数**
+   - **忽略元数据行**：文件末尾的 `# File Information` 等以 `# ` 开头的行都**不编号**
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
-   - **⚠️ 重要**：读取完整文件用于理解代码逻辑，但问题行号必须基于 diff 块计算
-   - 如果从完整文件中定位到问题行号，**必须验证**该行号是否在 diff 块范围内
-   - **验证公式**：`new_start <= 行号 <= new_start + new_count - 1`
 
 5. **执行审查**：按照 @.claude/rules/code-review-rules.md 中的规则进行审查
 
@@ -77,11 +73,21 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 7. **输出结果**：
    - 审查结果放入 `<REPORT>` 标签
-   - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
-   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 18 条
+   - **⚠️ 必须输出 `<LINE_INFO>` 标签**（这是强制要求，没有例外）
+   - LINE_INFO 格式：`[{"new_path":"文件路径","new_line":行号}]`
    - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
+   - **无问题时**：LINE_INFO 必须输出空数组 `<LINE_INFO>[]</LINE_INFO>`
    - **⚠️ 关键格式要求**：`<LINE_INFO>` 标签必须放置在 `</REPORT>` 标签之后
 
+## ⚠️ 大文件审查特殊要求
+
+当 `Is Large File: true` 时：
+1. 这是完整文件内容审查，所有代码行都是新增（`+` 开头）
+2. **必须审查整个文件**，不要只看部分代码
+3. **发现任何问题时，必须输出 `<LINE_INFO>` 标签**，包含准确的行号
+4. 行号基于完整文件内容计算，从第 1 行开始
+5. **即使只有一个问题，也必须输出 LINE_INFO**
+
 ## 输出模板
 
 **⚠️ 强制格式要求（违反会导致解析失败）**：
@@ -101,6 +107,8 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 6. **禁止输出错误代码示例**：已经有行号定位了，不需要重复输出错误代码
 
 7. **必须输出 `<LINE_INFO>` 标签**：即使无问题也要输出空数组 `<LINE_INFO>[]</LINE_INFO>`
+   - **⚠️ 再次强调**：如果发现严重问题，必须同时输出 LINE_INFO 标签包含行号信息
+   - **如果只输出 REPORT 而不输出 LINE_INFO，审查结果会被系统丢弃**
 
 <REPORT>
 ## 🤖 AI 代码审查结果
@@ -111,10 +119,14 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
 
+**问题 2**：[问题描述]<br/>
+**文件及行号**：[文件路径：行号]<br/>
+**修改建议**：[正确示例代码或说明]
+
 </REPORT>
 
 <LINE_INFO>
-[{"new_path":"文件路径","new_line":行号}]
+[{"new_path":"文件路径","new_line":行号},{"new_path":"文件路径","new_line":行号}]
 </LINE_INFO>
 
 **说明**：
@@ -123,3 +135,4 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 - **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**
 - 行号：返回变更后文件中的绝对行号（从 1 开始计数）
 - **无严重问题时，LINE_INFO 必须输出空数组 `[]`**
+- **⚠️ 有严重问题时，LINE_INFO 必须包含对应的行号信息，否则审查结果会被丢弃**

package/index.js

@@ -549,40 +549,54 @@ ${allReportsText}
 
     // 大文件模式下的特殊提示
     const largeFilePrompt = isLargeFile ? `
-**⚠️ 大文件审查模式**：
-这是一个完整文件内容的审查（因为 GitLab diff 被截断）。
-1. 所有代码行都是新增或修改的内容
-2. 必须审查整个文件的业务逻辑，包括：
-   - 所有方法中的循环逻辑、状态管理
-   - 所有数据库操作和资源管理
-   - 所有框架特性使用（如 @Async、MyBatis-Plus）
-3. 发现问题时，准确报告行号（基于文件内容的实际行号）
+**⚠️ 大文件审查模式 - 强制格式要求**：
+这是完整文件审查（GitLab diff截断），所有代码都是新增内容。
+1. 直接输出审查报告，不要先分析后格式化
+2. 发现问题后立即按模板格式写入 REPORT 和 LINE_INFO
+3. 行号基于完整文件实际行号计算
+4. **严禁在输出中写 "Let me format" 等中间过程文本**
+5. **必须在 </REPORT> 后立即输出 <LINE_INFO> 标签，包含每个问题的行号**
 ` : '';
 
     const prompt = `请调用 simple-code-review 技能审核代码变更。
 文件路径：${filePath}
 ${largeFilePrompt}
-**重要审查规则**：
-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>' 标签**，包含所有问题的行号信息（无问题时输出空数组 []）
-5. 不要输出任何额外的解释、问候或总结文本`;
+**⚠️ 强制输出格式要求（违反会导致审查结果丢失）**：
+
+你必须直接输出以下格式，不要输出任何分析过程、思考过程或中间文本：
+
+<REPORT>
+## 🤖 AI 代码审查结果
+
+### 🔴 严重问题
+
+**问题 1**：[问题描述]<br/>
+**文件及行号**：[文件路径：实际行号]<br/>
+**修改建议**：[修复建议]
+
+**问题 2**：[问题描述]<br/>
+**文件及行号**：[文件路径：实际行号]<br/>
+**修改建议**：[修复建议]
+
+...（更多问题按同样格式）
+
+</REPORT>
+
+<LINE_INFO>
+[{"new_path":"文件路径","new_line":实际行号},{"new_path":"文件路径","new_line":实际行号}]
+</LINE_INFO>
+
+**审查重点**：
+1. 资源泄漏：JDBC/文件流未关闭 → 报告资源创建行号
+2. 框架问题：@Async返回具体类型 → 报告方法定义行号
+3. 循环逻辑：状态过期、累积不一致 → 报告循环内出错行号
+4. 异常吞没：catch块空实现 → 报告catch行号
+5. 空指针：包装类型拆箱 → 报告使用行号
+
+**关键**：
+- 有严重问题时，LINE_INFO 必须包含对应行号，否则结果被丢弃
+- 无问题时输出 <LINE_INFO>[]</LINE_INFO>
+- 不要输出 "Let me analyze"、"Let me format" 等文本`;
     //打印
     debugLog(`[${blockIdentifier}] Claude命令: ${prompt}`);
     // 最多重试5次，直到结果包含"🤖 AI 代码审查结果"或达到最大重试次数

package/package.json

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