job51-gitlab-cr-node-jt-1 2.2.9 → 2.3.0

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

@@ -3,14 +3,36 @@
 **核心原则：高准确率、低误报、深度上下文分析、风格一致性**
 
 **⚠️ 最高优先级强制规则（违反将导致严重误报）**：
-> - **报告运行时异常风险前，必须先用 Read 工具追踪读取相关方法/函数的实现代码**
-> - 适用场景：空指针/空引用异常、类型转换异常、集合越界等所有运行时异常
-> - 示例：看到 `obj = service.getData(id); obj.getProperty()` 时，**必须先读取** `service.getData` 方法的源码，确认其返回值是否可能为 null
-> - 如果方法/函数内部已有防护措施（如返回默认对象、空对象模式、边界检查等），则**禁止报告**该问题
-> - **禁止仅凭代码表面形式判断运行时异常风险**
-> - **⚠️ 特别强调**：当前 CR 流程处于版本开发的最后合并阶段，代码逻辑已经过开发自测和 QA 测试验证。审查时应**基于项目原有的代码风格**，不得强制要求添加项目惯例中不存在的内容。
-> - **⚠️ 风格一致性原则**：参数校验、异常处理、注解使用等应与项目原有风格匹配，**不得报告"建议添加 XX 注解/XX 校验/XX 异常处理"** 这类问题。
-> - **⚠️ 问题定位原则**：**只在真正会执行出错的代码行上报告问题**，其他地方（如类定义、导入语句、方法签名等）禁止报告
+
+1. **审查范围限制**（关键规则）：
+   > - **只审查当前 diff 块内的新增代码**（+ 开头的行）
+   > - **只报告当前 diff 块内能直接发现的问题**
+   > - **不要追踪方法调用链去其他文件中报告问题**（如"XX 方法返回 null，导致其他地方空指针"）
+   > - 可以分析方法返回值是否可能为 null，但**问题只能报告在当前调用处**
+
+2. **同一问题只报告一次**：
+   > - 同一个空指针问题可能在多处出现（import 行、依赖注入行、方法调用行）
+   > - **只在真正会执行出错的代码行上报告一次**
+   > - 示例：`@Autowired UserTaskSDK userTaskSDK;` 和 `userTaskSDK.xxx()` 都有空指针风险
+   > - **正确做法**：只在 `userTaskSDK.xxx()` 调用行报告，**禁止**在依赖注入行报告
+
+3. **禁止报告问题的位置**：
+   > - import 导入语句行
+   > - 类定义、package 声明行
+   > - 方法签名行（除非签名本身有安全问题）
+   > - 依赖注入/字段声明行（如 `@Autowired XXX xxx;`）
+   > - 仅定义变量但未使用的行
+   > - 其他不会直接导致运行时错误的位置
+
+4. **深度分析但精准报告**：
+   > - **报告运行时异常风险前，必须先用 Read 工具追踪读取相关方法/函数的实现代码**
+   > - 适用场景：空指针/空引用异常、类型转换异常、集合越界等所有运行时异常
+   > - 示例：看到 `obj = service.getData(id); obj.getProperty()` 时，**必须先读取** `service.getData` 方法的源码，确认其返回值是否可能为 null
+   > - 如果方法/函数内部已有防护措施（如返回默认对象、空对象模式、边界检查等），则**禁止报告**该问题
+   > - **禁止仅凭代码表面形式判断运行时异常风险**
+
+5. **⚠️ 特别强调**：当前 CR 流程处于版本开发的最后合并阶段，代码逻辑已经过开发自测和 QA 测试验证。审查时应**基于项目原有的代码风格**，不得强制要求添加项目惯例中不存在的内容。
+   > - **⚠️ 风格一致性原则**：参数校验、异常处理、注解使用等应与项目原有风格匹配，**不得报告"建议添加 XX 注解/XX 校验/XX 异常处理"** 这类问题。
 
 0. **Diff 数据结构与上下文读取规则**：
 > - **临时文件格式说明**：
@@ -88,6 +110,10 @@
         >   - 方法签名行（除非签名本身有安全问题）
         >   - 仅定义变量但未使用的行
         >   - 其他不会直接导致运行时错误的位置
+    >   - **Feign Client 远程调用返回值校验**：
+        >   - Feign Client 接口调用（如 `xxClient.getXxx()`）的返回值**不纳入空指针审查范围**
+        >   - Feign 框架会在调用失败时抛出异常，而不是返回 null
+        >   - 示例：`Result<UserTaskPop> userTaskPop = userTaskClient.getUserTaskPop(accountId);` 后续调用 `userTaskPop.requestSucceeded()` **不应报告 NPE 问题**
 5. 使用当前系统时间填充 `[当前时间]`，并根据实际变更内容计算 `[文件数量]`；
 6. 所有占位符必须被真实内容替换；
    > - **修改建议中的代码示例**：
@@ -119,51 +145,38 @@
        >   - 示例：`UserTaskPop pop = sdk.method(); log.debug(pop.getId())` 问题应报告在 `pop.getId()` 行，不是 `sdk.method()` 或变量定义行
        >   - **每个 diff 块只报告本块内实际会出错的代码**，不要在无关的 diff 块（如 package 行、import 行、类定义行）报告问题
 
-10. **警告问题（⚠️）**：
-    > - 代码可读性差（魔法数字、命名不规范、缺少必要注释）
-    > - 性能潜在问题（低效算法、不必要的数据库查询、内存使用不当）
-    > - 边界条件处理不完善（但不导致运行时异常）
-    > - 异常处理不完善（捕获异常未记录或吞掉异常）
-    > - 资源使用不规范（未关闭资源但无泄漏风险）
-    > - 代码重复或违反 DRY 原则
-    > - 类型安全警告（原始类型使用、unchecked 转换）
-    > - **强制排除**：警告中**不得包含**空指针/空引用、类型转换/拆箱、集合越界等运行时异常描述
-    > - **行号去重**：若某行号已在严重问题中报告，警告中**禁止**再出现
-    > - **风格一致性排除**：**不得报告"建议添加 XX 注解/XX 校验/XX 异常处理"** 这类问题，除非项目已有明确惯例而此处遗漏
-
-11. **问题描述准确性**：
-    > - 若代码变更可能导致运行时异常，必须优先描述异常风险，不得降级为性能或健壮性描述
-
-12. **问题分级规则**：
+11. **问题分级规则**：
     > - 可能导致运行时异常/错误、数据丢失/损坏、安全漏洞的，**必须**归类为严重问题
     > - 同一代码块的多个问题，全部归入同一风险级别并依次展示
 
-13. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束；
-
-14. **行号计算规范**：
-    > - **Diff 头信息格式**：`@@ -84,9 +84,11 @@` 表示变更后代码从原始文件的第 84 行开始，共有 11 行
-    > - **行号计算**：
-        >   - 空格开头（上下文）：行号 = 起始行号 + 已处理的上下文行数
-        >   - `-` 开头（删除）：不计入变更后行号
-        >   - `+` 开头（新增）：行号 = 起始行号 + 已处理的保留/新增行数
-    > - **问题行号范围**：
-        >   - 单行问题：`UserService.java:87`
-        >   - 多行问题：`UserService.java:90-91`
-    > - **关键**：问题行号必须准确指向变更后代码的实际行号，不得引用已删除的行
-
-15. `<REPORT>` 标签示例中的问题块数量仅用于展示格式，实际数量由 review 结果决定；
-
-16. **输出前验证流程**：
-    > - **去重检查**：检查是否存在行号重叠，若发现重叠仅保留最高优先级（严重问题 > 警告）
-    > - **跨块重复问题去重**：**同一文件同一问题在不同 diff 块中出现时，只报告一次**
-        >   - 示例：同一方法 `userTaskSDK.userTaskModuleControl()` 在块 1、块 2、块 3 中被调用，未判空问题**只报告一次**
-        >   - 方法：**按「文件 + 问题类型 + 根本原因」进行去重**，相同组合的问题只保留第一个
-    > - **警告验证**：检查警告中是否包含运行时异常关键词，若包含则升级为严重问题
+12. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束；
+
+13. **行号计算规范**：
+    > - **Diff 头信息格式**：`@@ -84,9 +84,11 @@`
+        >   - `-84,9`：旧文件从第 84 行开始，共 9 行
+        >   - `+84,11`：新文件从第 84 行开始，共 11 行
+    > - **行号计算**（绝对行号，非相对偏移）：
+        >   - 初始值 = `new_start`（从 diff 头解析）
+        >   - 空格开头（上下文）：当前行号 = 计数器值，计数器 +1
+        >   - `-` 开头（删除）：跳过，计数器不变
+        >   - `+` 开头（新增）：当前行号 = 计数器值，计数器 +1
+    > - **示例**：`@@ -0,0 +1,49 @@`（新增文件）
+        >   - 第 1 行 `+`：行号 = 1
+        >   - 第 2 行 `+`：行号 = 2
+    > - **关键**：问题行号必须准确指向变更后代码的实际行号（绝对行号）
+
+14. `<REPORT>` 标签示例中的问题块数量仅用于展示格式，实际数量由 review 结果决定；
+
+15. **输出前验证流程**：
+    > - **块内去重检查**：**同一 diff 块内由同一根本原因导致的问题，只报告一次**
+        >   - 示例：同一个变量 `userTaskSDK` 在块内多处被调用，若都存在未判空问题，**合并为一条报告**
+        >   - 方法：**按「变量名 + 问题类型 + 根本原因」进行去重**，相同组合的问题只保留第一个
+        >   - **注意**：每个 diff 块是独立审查的，无法跨块去重，只需保证当前块内不重复
     > - **代码示例验证**：确保错误代码示例不包含 `-` 开头的已删除代码
     > - **删除代码验证**：**逐条检查问题涉及的代码行，确认不是 `-` 开头的已删除代码**
     > - **安全问题验证**：确保不针对已删除代码报告安全问题
 
-17. **代码变更方向识别**：
+16. **代码变更方向识别**：
     > - `-` 开头：**已删除的代码**，**禁止**作为审查对象，**禁止**报告任何问题
     > - `+` 开头：**新增的代码**，是审查的唯一重点
     > - 空格开头：未变更的上下文代码，仅作参考
@@ -172,9 +185,12 @@
     > - **⚠️ 问题报告位置规则**：
         >   - **空指针/未判空问题**：只报告在**实际调用方法访问返回值的行**（如 `obj.getXxx()`），而不是方法调用或变量定义的行
         >   - **示例**：`UserTaskPop pop = sdk.method(); pop.getId()` 问题应报告在 `pop.getId()` 这一行，而不是 `sdk.method()` 或变量定义行
+        >   - **依赖注入/字段声明**：如 `@Autowired UserTaskSDK userTaskSDK;`，**禁止**在此行报告空指针问题
+        >   - **Import 语句**：**禁止**在 import 行报告任何问题
+        >   - **同一变量多次使用**：只在第一次可能出错的地方报告一次
         >   - **禁止**在类定义、package、import、方法签名等位置报告与这些行无关的问题
 
-18. **变更内容解析步骤**（分析前必须按顺序执行）：
+17. **变更内容解析步骤**（分析前必须按顺序执行）：
     > - **步骤 1：识别变更类型**
         >   - 遍历代码块，标记每行类型：`-`（删除）、`+`（新增）、空格（保留）
     > - **步骤 2：丢弃已删除代码**（关键步骤）
@@ -194,9 +210,22 @@
         >   - 空指针问题：报告在实际调用 `obj.getXxx()` 的行，不是变量定义行
         >   - 禁止在 package、import、类定义、方法签名等位置报告无关问题
 
-19. **输出格式**：严格参照 SKILL.md 模板，无某类问题时省略对应部分；
-
-20. **误报防控检查清单**（报告前必须确认）：
+18. **输出格式**：严格参照 SKILL.md 模板，无某类问题时省略对应部分；
+    > - **必须输出 `<LINE_INFO>` 标签**：包含所有问题的行号信息
+    > - **LINE_INFO 格式**（与 targetLine 结构一致）：
+        ```json
+        [{"new_path":"文件路径","new_line":行号,"old_path":"文件路径","old_line":行号}]
+        ```
+    > - **字段说明**：
+        - `new_path`: 变更后文件路径（必填）
+        - `new_line`: 变更后代码的**绝对行号**（必填，指向 `+` 开头的新增代码在文件中的实际行号）
+        - `old_path`: 变更前文件路径（可选，仅删除代码相关问题使用）
+        - `old_line`: 变更前代码的**绝对行号**（可选，仅删除代码相关问题使用）
+    > - **行号计算**：参考第 13 条 行号计算规范
+    > - **行号定位**：参考第 16 条 问题报告位置规则
+    > - 无问题时输出空数组：`<LINE_INFO>[]</LINE_INFO>`
+
+19. **误报防控检查清单**（报告前必须确认）：
     > - [ ] 已追踪读取相关方法实现，确认可能返回危险值
     > - [ ] 已跨 diff 块分析，确认不是已被新逻辑替代的删除代码
     > - [ ] 已分析方法内联上下文，确认没有前置判空保护

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

@@ -1,55 +1,52 @@
----
-name: simple-code-review
-description: 你是一个专业的代码审查助手。请严格根据代码规范、安全规则（如 SQL 注入防护、XSS 防护）、性能要求（如时间复杂度优化、内存占用控制）、可读性要求（如注释完整性、命名规范性）及功能正确性标准处理指定文件中的代码变更内容
----
-
-## 处理步骤
-1. 首先使用 Read 工具读取 diff 文件：$ARGUMENTS
-2. **解析文件信息**：从文件内容中提取以下信息
-   - New Path: 变更后的文件路径（可能包含 `b/` 前缀）
-   - Old Path: 变更前的文件路径（可能包含 `a/` 前缀）
-   - Diff Content: 实际的代码变更内容
-3. **处理文件路径**：去掉路径中的 `a/` 或 `b/` 前缀，得到实际的项目相对路径
-4. **读取项目上下文文件**：基于解析出的文件路径，主动读取相关的项目文件（详见规则文档）
-5. **深度分析代码上下文（必须执行）**：
-   - 对于diff中涉及的方法/函数调用，**必须**使用 Read 工具追踪读取该方法的实现代码
-   - 示例：若看到 `accountInfo = userSDK.getAccountInfo(id, false); accountInfo.getXxx()`，必须读取 `userSDK.getAccountInfo` 方法的实现，确认其返回值是否可能为 null
-   - **禁止**仅凭代码表面形式判断存在空指针/空引用等运行时异常
-   - **跨 diff 块分析**：同一文件可能有多个 diff 块，必须合并分析。如块A删除了 `obj.setValue(xxx)`，但块B新增了 `if(condition) obj.setValue(yyy)`，不应报告"删除了set值会影响数据"
-   - 只有在确认方法/函数确实可能返回危险值时，才能报告严重问题
-6. **结合项目上下文**，按照 @.claude/rules/code-review-rules.md 中的规则进行审查
-7. 输出格式必须以下方模板为准
-
-## 输出模板
-
-``**说明**：
-- 必须以 `<REPORT>` 开始，以 `</REPORT>` 结束
-- 若某类问题不存在，则**完全省略该模块**（不输出对应标题）
-- 所有占位符必须被真实内容替换
-- 带圈数字序号：① ② ③ ④ ⑤ ⑥ ⑦ ⑧ ⑨ ⑩``
-
-<REPORT>
-## 🤖 AI 代码审查结果
-
-**生成时间**: [当前时间]
-**审查范围**: [文件数量] 个文件
-
----
-
-### 审查总览
-
-[简要总结]
-
-### 🔴 严重问题
-
-[带圈数字序号][问题描述]<br/>
-**文件及行号**：[完整文件路径：起始行 - 结束行，如：src/main/java/com/example/UserService.java:87-91]<br/>
-**修改建议**：[修复建议 + 错误代码 + 正确示例]
-
-### 🟡 警告
-
-[带圈数字序号][问题描述]<br/>
-**文件及行号**：[完整文件路径：起始行 - 结束行]<br/>
-**修改建议**：[改进建议 + 代码示例]
-
-</REPORT>
+---
+name: simple-code-review
+description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_INFO
+---
+
+## 处理步骤
+
+1. **读取 diff 文件**：使用 Read 工具读取文件：$ARGUMENTS
+
+2. **解析文件信息**：从 `=== File Information ===` 部分提取
+   - `Block Index`: 当前块索引号
+   - `Line Info`: 行号信息 JSON 对象
+   - `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
+
+3. **解析行号**：从 `Line Info` 中获取 `new_start`，遍历 diff 内容计算每行行号
+   - 行号计算规则：
+     - 从 diff 头 `@@ -old_start,old_count +new_start,new_count @@` 中提取 `new_start`
+     - 遍历 diff 内容，只计数 `+` 开头（新增）和空格开头（上下文）的行
+     - 第 1 行 `+` 或空格：行号 = `new_start`
+     - 第 2 行 `+` 或空格：行号 = `new_start + 1`
+     - 以此类推，`-` 开头的删除行不计入行号
+   - **重要**：行号是变更后文件中的绝对行号，必须准确对应实际代码位置
+
+4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
+
+5. **执行审查**：按照 @.claude/rules/code-review-rules.md 中的规则进行审查
+
+6. **输出结果**：
+   - 审查结果放入 `<REPORT>` 标签
+   - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
+   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 19 条
+
+## 输出模板
+
+<REPORT>
+## 🤖 AI 代码审查结果
+
+### 🔴 严重问题
+
+[带圈数字序号][问题描述]<br/>
+**文件及行号**：[文件路径:行号，如：51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java:40]<br/>
+**修改建议**：[正确示例代码]
+
+</REPORT>
+
+<LINE_INFO>
+[{"new_path":"文件路径","new_line":行号,"old_path":"文件路径","old_line":行号}]
+</LINE_INFO>
+
+**说明**：
+- 无问题时省略对应标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
+- 行号格式参考 @.claude/rules/code-review-rules.md 第 19 条

package/index.js

@@ -78,6 +78,7 @@ class GitLabCodeReviewer {
 
 
           // 提取REPORT标签内容并返回
+          // 提取 REPORT 标签和 LINE_INFO 标签内容并返回对象
           return extractReportContent(claudeResult);
         } else {
           debugLog(`AI审核结果不包含"🤖 AI 代码审查结果" (尝试 ${attempts})，将重试...`);
@@ -85,7 +86,8 @@ class GitLabCodeReviewer {
             debugLog(`已达到最大重试次数 ${maxAttempts}，返回最后一次结果，${claudeResult}`);
 
             // 提取REPORT标签内容并返回
-            return extractReportContent(claudeResult);
+            // 提取 REPORT 标签和 LINE_INFO 标签内容并返回对象
+          return extractReportContent(claudeResult);
           }
         }
       } catch (error) {
@@ -138,9 +140,8 @@ ${diffObject.diff}`;
         const blockObj = { ...diffObject, review_result, temp_file_path: tmpFileName };
 
         // 检查审查结果中是否包含严重问题，只有包含严重问题才发布评论
-        if (blockObj.review_result && blockObj.review_result.includes('🔴 严重问题')) {
+        if (blockObj.review_result && blockObj.review_result.reportContent && blockObj.review_result.reportContent.includes('🔴 严重问题')) {
           // 立即发布评论
-          console.log(`🤖 AI 代码审查结果: ${blockObj.review_result}`);
           await this.postSingleCommentToGitLab(projectId, mergeRequestIid, {
             diff_info: blockObj,
             block_index: blockObj.block_index,
@@ -201,11 +202,18 @@ ${diffObject.diff}`;
     const prompt = `请调用 simple-code-review 技能审核代码变更。
 文件路径：${filePath}
 
-输出要求：
+**重要审查规则**：
+1. **只审查当前 diff 块内的新增代码**（+ 开头的行）
+2. **只报告当前 diff 块内能直接发现的问题**，不要追踪方法调用链去其他地方报告问题
+3. **同一问题只报告一次**：如 import 行、声明行、调用行都有问题，只在真正会出错的调用行报告一次
+4. **禁止报告的位置**：import 语句、类定义、方法签名、依赖注入声明行
+
+**输出要求**：
 1. 严格按照 .claude/skills/simple-code-review/SKILL.md 中定义的模板格式输出
 2. **每个模块不是必须的**：没有对应问题时，完全省略该模块（不输出标题）
 3. 必须以 <REPORT> 开始，以 </REPORT> 结束
-4. 不要输出任何额外的解释、问候或总结文本`;
+4. **必须输出 '<LINE_INFO>' 标签**，包含所有问题的行号信息（无问题时输出空数组 [])
+5. 不要输出任何额外的解释、问候或总结文本`;
     //打印
     debugLog(`Claude命令: ${prompt}`);
     // 最多重试5次，直到结果包含"🤖 AI 代码审查结果"或达到最大重试次数
@@ -229,6 +237,7 @@ ${diffObject.diff}`;
           debugLog(`AI审核成功，包含"🤖 AI 代码审查结果" (尝试 ${attempts})`);
 
           // 提取REPORT标签内容并返回
+          // 提取 REPORT 标签和 LINE_INFO 标签内容并返回对象
           return extractReportContent(claudeResult);
         } else {
           debugLog(`AI审核结果不包含"🤖 AI 代码审查结果" (尝试 ${attempts})，将重试...`);
@@ -236,7 +245,8 @@ ${diffObject.diff}`;
             debugLog(`已达到最大重试次数 ${maxAttempts}，返回最后一次结果，${claudeResult}`);
 
             // 提取REPORT标签内容并返回
-            return extractReportContent(claudeResult);
+            // 提取 REPORT 标签和 LINE_INFO 标签内容并返回对象
+          return extractReportContent(claudeResult);
           }
         }
       } catch (error) {
@@ -352,6 +362,48 @@ ${diffObject.diff}`;
       });
   }
 
+  /**
+   * 从 LINE_INFO 标签字符串中解析行号信息
+   * @param {string} lineInfoTag LINE_INFO 标签字符串，如 "<LINE_INFO>[...]</LINE_INFO>"
+   * @returns {Object|null} 行号信息对象 {new_path, new_line, old_path, old_line} 或 null
+   */
+  parseLineInfoFromReviewResult(lineInfoTag) {
+    if (!lineInfoTag) return null;
+
+    try {
+      // 从标签中提取 JSON 内容：<LINE_INFO>[{...}]</LINE_INFO>
+      const jsonContent = lineInfoTag.replace(/<LINE_INFO>\s*/g, '').replace(/\s*<\/LINE_INFO>/g, '').trim();
+
+      debugLog(`解析 LINE_INFO 原始内容：${lineInfoTag}`);
+      debugLog(`解析 LINE_INFO JSON 内容：${jsonContent}`);
+
+      // 如果是空数组，返回 null 使用后备方案
+      if (jsonContent === '[]') {
+        debugLog('LINE_INFO 为空数组，使用 diff 块起始行号作为后备方案');
+        return null;
+      }
+
+      const lineInfoArray = JSON.parse(jsonContent);
+      debugLog(`解析 LINE_INFO JSON 成功：${JSON.stringify(lineInfoArray)}`);
+
+      if (Array.isArray(lineInfoArray) && lineInfoArray.length > 0) {
+        debugLog(`从 LINE_INFO 中解析出行号信息：${JSON.stringify(lineInfoArray)}`);
+        // 返回第一个问题的行号信息
+        const lineInfo = lineInfoArray[0];
+        // GitLab API 只需要 new_path 和 new_line
+        return {
+          new_path: lineInfo.new_path,
+          new_line: lineInfo.new_line
+        };
+      }
+    } catch (error) {
+      debugLog(`解析 LINE_INFO JSON 失败：${error.message}`);
+    }
+
+    debugLog('无法从 LINE_INFO 中解析行号，将使用 diff 块起始行号作为后备方案');
+    return null;
+  }
+
   /**
    * 获取合并请求的最新版本信息
    * @param {number} projectId GitLab项目ID
@@ -380,6 +432,9 @@ ${diffObject.diff}`;
   async postSingleCommentToGitLab(projectId, mergeRequestIid, result) {
     try {
       const { diff_info, review_result, block_index } = result;
+      // review_result 现在是对象：{ reportContent, lineInfo }
+      const reviewContent = review_result?.reportContent || '';
+      const lineInfoTag = review_result?.lineInfo || null;
       const file_path = diff_info.new_path || diff_info.old_path;
       const file_path_with_line = `${file_path}#L${block_index}`;
 
@@ -399,19 +454,37 @@ ${diffObject.diff}`;
       // 直接使用diff_info中的line_info
       const lineInfo = diff_info.line_info;
 
-      if (!lineInfo) {
+      if (!lineInfo && !lineInfoTag) {
         // 如果没有行号信息，创建一般讨论
-        await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, review_result);
+        await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, reviewContent);
         debugLog(`评论已发布到文件 ${file_path_with_line} (无法解析行号)`);
         return;
       }
 
-      // 如果lineInfo里的firstLineFirstChar为+，则targetLine为new_path的new_start行；为-，则targetLine为old_path的old_start行；否则new和old都要赋值
-      const targetLine = lineInfo.firstLineFirstChar === '+'
+      // 如果 lineInfo 里的 firstLineFirstChar 为 +，则 targetLine 为 new_path 的 new_start 行；为 -，则 targetLine 为 old_path 的 old_start 行
+      // GitLab API 要求：diff 评论只能传递 new_line 或 old_line 中的一个，不能同时传递
+      let targetLine = lineInfo.firstLineFirstChar === '+'
         ? { new_line: lineInfo.new_start, new_path: diff_info.new_path }
         : lineInfo.firstLineFirstChar === '-'
           ? { old_line: lineInfo.old_start, old_path: diff_info.old_path }
-          : { new_line: lineInfo.new_start, new_path: diff_info.new_path, old_line: lineInfo.old_start, old_path: diff_info.old_path };
+          : { new_line: lineInfo.new_start, new_path: diff_info.new_path };
+
+      // 尝试从审查结果中解析更精确的行号信息（从 LINE_INFO 标签）
+      const parsedLineInfo = lineInfoTag ? this.parseLineInfoFromReviewResult(lineInfoTag) : null;
+      if (parsedLineInfo) {
+        // 验证解析的行号是否在当前 diff 块范围内
+        const newStart = lineInfo.new_start;
+        const newCount = lineInfo.new_count || 0;
+        const newEnd = newStart + newCount - 1;
+        debugLog(`diff 块信息：new_start=${newStart}, new_count=${newCount}, 范围=[${newStart}, ${newEnd}]`);
+        debugLog(`解析的行号：${parsedLineInfo.new_line}`);
+        if (parsedLineInfo.new_line >= newStart && parsedLineInfo.new_line <= newEnd) {
+          debugLog(`从 LINE_INFO 中解析出行号成功，使用解析后的行号覆盖 diff 块起始行号`);
+          targetLine = parsedLineInfo;
+        } else {
+          debugLog(`解析的行号 ${parsedLineInfo.new_line} 不在 diff 块范围 [${newStart}, ${newEnd}] 内，使用 diff 块起始行号`);
+        }
+      }
 
       //打印targetLine
       debugLog(`targetLine: ${JSON.stringify(targetLine)}`);
@@ -419,7 +492,7 @@ ${diffObject.diff}`;
       if (targetLine) {
         // 创建diff评论
         const payload = {
-          body: review_result,
+          body: reviewContent,
           position: {
             position_type: 'text',
             base_sha: baseSha,
@@ -433,13 +506,15 @@ ${diffObject.diff}`;
           await this.createDiffDiscussion(projectId, mergeRequestIid, payload);
           debugLog(`评论已发布到文件 ${file_path_with_line} 的相关变更区域`);
         } catch (error) {
+          // 打印详细的错误响应信息
+          debugLog(`GitLab API 错误详情：${JSON.stringify(error.response?.data || error.message)}`);
           console.error(`发布评论到文件 ${file_path_with_line} 的变更区域失败，改用一般讨论:`, error.message);
-          await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, review_result);
+          await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, reviewContent);
           debugLog(`评论已发布到文件 ${file_path_with_line} (作为一般讨论)`);
         }
       } else {
         // 如果没有找到任何行号，创建一般讨论
-        await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, review_result);
+        await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, reviewContent);
         debugLog(`评论已发布到文件 ${file_path_with_line} (无特定行号)`);
       }
     } catch (error) {

package/package.json

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

package/utils.js

@@ -1,67 +1,74 @@
-/**
- * 工具函数集合
- */
-
-const axios = require('axios');
-
-// 调试日志函数
-function debugLog(message) {
-  console.log('[DEBUG]', new Date().toISOString(), message);
-}
-
-/**
- * 提取REPORT标签之间的内容
- * @param {string} text 包含REPORT标签的文本
- * @returns {string} 提取后的内容
- */
-function extractReportContent(text) {
-  const reportRegex = /<REPORT>([\s\S]*?)<\/REPORT>/;
-  const match = text.match(reportRegex);
-  return match ? match[1].trim() : text;
-}
-
-// 创建一个可配置的GitLab API客户端
-class GitLabAPIClient {
-  constructor(gitlabToken, gitlabUrl = null) {
-    this.gitlabToken = gitlabToken;
-    this.gitlabUrl = gitlabUrl || 'https://gitdev.51job.com';
-    this.axiosInstance = axios.create({
-      headers: {
-        'PRIVATE-TOKEN': this.gitlabToken,
-      },
-    });
-  }
-
-  /**
-   * 通用的GitLab API调用函数
-   * @param {string} endpoint API端点
-   * @param {Object} options 额外选项（method, data等）
-   * @returns {Promise} API调用结果
-   */
-  async callGitLabAPI(endpoint, options = {}) {
-    const { method = 'GET', data = null, headers = {} } = options;
-
-    try {
-      const response = await this.axiosInstance({
-        method,
-        url: `${this.gitlabUrl}${endpoint}`,
-        headers: {
-          'Content-Type': 'application/json',
-          ...headers
-        },
-        data
-      });
-
-      return response.data;
-    } catch (error) {
-      console.error(`GitLab API调用失败: ${method} ${endpoint}`, error.message);
-      throw error;
-    }
-  }
-}
-
-module.exports = {
-  GitLabAPIClient,
-  debugLog,
-  extractReportContent
-};
+/**
+ * 工具函数集合
+ */
+
+const axios = require('axios');
+
+// 调试日志函数
+function debugLog(message) {
+  console.log('[DEBUG]', new Date().toISOString(), message);
+}
+
+/**
+ * 提取 REPORT 标签和 LINE_INFO 标签的内容
+ * @param {string} text 包含 REPORT 和 LINE_INFO 标签的文本
+ * @returns {{reportContent: string, lineInfo: string|null}} 包含 REPORT 内容和 LINE_INFO 标签的对象
+ */
+function extractReportContent(text) {
+  const reportRegex = /<REPORT>([\s\S]*?)<\/REPORT>/;
+  const lineInfoRegex = /<LINE_INFO>[\s\S]*?<\/LINE_INFO>/;
+
+  const reportMatch = text.match(reportRegex);
+  const lineInfoMatch = text.match(lineInfoRegex);
+
+  const reportContent = reportMatch ? reportMatch[1].trim() : text;
+  const lineInfo = lineInfoMatch ? lineInfoMatch[0] : null;
+
+  return { reportContent, lineInfo };
+}
+
+// 创建一个可配置的 GitLab API 客户端
+class GitLabAPIClient {
+  constructor(gitlabToken, gitlabUrl = null) {
+    this.gitlabToken = gitlabToken;
+    this.gitlabUrl = gitlabUrl || 'https://gitdev.51job.com';
+    this.axiosInstance = axios.create({
+      headers: {
+        'PRIVATE-TOKEN': this.gitlabToken,
+      },
+    });
+  }
+
+  /**
+   * 通用的 GitLab API 调用函数
+   * @param {string} endpoint API 端点
+   * @param {Object} options 额外选项（method, data 等）
+   * @returns {Promise} API 调用结果
+   */
+  async callGitLabAPI(endpoint, options = {}) {
+    const { method = 'GET', data = null, headers = {} } = options;
+
+    try {
+      const response = await this.axiosInstance({
+        method,
+        url: `${this.gitlabUrl}${endpoint}`,
+        headers: {
+          'Content-Type': 'application/json',
+          ...headers
+        },
+        data
+      });
+
+      return response.data;
+    } catch (error) {
+      console.error(`GitLab API 调用失败：${method} ${endpoint}`, error.message);
+      throw error;
+    }
+  }
+}
+
+module.exports = {
+  GitLabAPIClient,
+  debugLog,
+  extractReportContent
+};