job51-gitlab-cr-node-jt-1 2.4.3 → 2.4.5

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

@@ -1,4 +1,4 @@
----
+---
 name: simple-code-review
 description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_INFO
 ---
@@ -9,17 +9,18 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 2. **解析文件信息**：从 `=== File Information ===` 部分提取
    - `Block Index`: 当前块索引号
-   - `Line Info`: 行号信息 JSON 对象
+   - `New Start`: 当前 diff 块在变更后文件中的起始行号（**关键：用于计算绝对行号**）
+   - `New Count`: 当前 diff 块的行数
    - `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
 
-3. **解析行号**：从 `Line Info` 中获取 `new_start`，遍历 diff 内容计算每行行号
+3. **解析行号**：使用 `New Start` 作为基准，遍历 diff 内容计算每行的绝对行号
    - 行号计算规则：
-     - 从 diff 头 `@@ -old_start,old_count +new_start,new_count @@` 中提取 `new_start`
+     - **基准行号** = `New Start`（从 File Information 中获取）
      - 遍历 diff 内容，只计数 `+` 开头（新增）和空格开头（上下文）的行
-     - 第 1 行 `+` 或空格：行号 = `new_start`
-     - 第 2 行 `+` 或空格：行号 = `new_start + 1`
+     - 第 1 行 `+` 或空格：行号 = `New Start`
+     - 第 2 行 `+` 或空格：行号 = `New Start + 1`
      - 以此类推，`-` 开头的删除行不计入行号
-   - **重要**：行号是变更后文件中的绝对行号，必须准确对应实际代码位置
+   - **重要**：行号是变更后文件中的绝对行号，必须使用 `New Start` 作为基准计算
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
 
@@ -29,6 +30,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
    - 审查结果放入 `<REPORT>` 标签
    - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
    - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 19 条
+   - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
 
 ## 输出模板
 
@@ -37,9 +39,16 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 ### 🔴 严重问题
 
-[带圈数字序号][问题描述]<br/>
-**文件及行号**：[文件路径:行号，如：51job-dev-boot-bbs/src/main/java/com/job51/dev/bbs/service/impl/ForumCategoryForumServiceImpl.java:40]<br/>
-**修改建议**：[正确示例代码]
+---
+**问题 1**：[问题描述]<br/>
+**文件及行号**：[文件路径：行号]<br/>
+**修改建议**：[正确示例代码或说明]
+---
+
+**问题 2**：[问题描述]<br/>
+**文件及行号**：[文件路径：行号]<br/>
+**修改建议**：[正确示例代码或说明]
+---
 
 </REPORT>
 
@@ -49,4 +58,6 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 **说明**：
 - 无问题时省略对应标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
-- 行号格式参考 @.claude/rules/code-review-rules.md 第 19 条
+- 每个问题用 `---` 分割线分隔，方便切分
+- **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**：第 1 个元素对应问题 1 的行号，第 2 个元素对应问题 2 的行号，以此类推
+- 行号计算：使用 `New Start` 作为基准，第 1 行 `+` 或空格的行号 = `New Start`，第 2 行 = `New Start + 1`，依此类推

package/index.js

@@ -160,6 +160,8 @@ class GitLabCodeReviewer {
 New Path: ${diffObject.new_path || 'N/A'}
 Old Path: ${diffObject.old_path || 'N/A'}
 Block Index: ${blockIndex}
+New Start: ${diffObject.line_info?.new_start || 1}
+New Count: ${diffObject.line_info?.new_count || 1}
 === Diff Content ===
 ${diffObject.diff}`;
 
@@ -512,30 +514,32 @@ ${diffObject.diff}`;
    */
   extractSingleProblemReport(fullReport, problemIndex) {
     const lines = fullReport.split('\n');
-    // 匹配带圈数字序号（1️⃣、2️⃣、3️⃣等）或普通数字
-    const problemRegex = new RegExp(`^\\s*${problemIndex}[️⃣\\d\\.\\)]`);
-
-    let startIndex = -1;
-    let endIndex = lines.length;
-
-    for (let i = 0; i < lines.length; i++) {
-      if (problemRegex.test(lines[i])) {
-        startIndex = i;
-        // 查找下一个问题的开始
-        const nextProblemRegex = new RegExp(`^\\s*(${problemIndex + 1}|\\d+)[️⃣\\.\\)]`);
-        for (let j = i + 1; j < lines.length; j++) {
-          if (nextProblemRegex.test(lines[j])) {
-            endIndex = j;
-            break;
+    // 使用 --- 分割线来切分问题
+    const problemBlocks = [];
+    let currentBlock = [];
+    let inBlock = false;
+
+    for (const line of lines) {
+      if (line.trim() === '---') {
+        if (inBlock) {
+          // 块结束
+          if (currentBlock.length > 0) {
+            problemBlocks.push(currentBlock.join('\n'));
+            currentBlock = [];
           }
+          inBlock = false;
+        } else {
+          // 块开始
+          inBlock = true;
         }
-        break;
+      } else if (inBlock) {
+        currentBlock.push(line);
       }
     }
 
-    if (startIndex !== -1) {
-      const problemLines = lines.slice(startIndex, endIndex);
-      return problemLines.join('\n');
+    // 返回第 problemIndex 个问题（索引从 1 开始）
+    if (problemIndex >= 1 && problemIndex <= problemBlocks.length) {
+      return problemBlocks[problemIndex - 1];
     }
 
     // 如果无法提取，返回完整报告（降级处理）