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

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

@@ -1,4 +1,4 @@
----
+---
 name: simple-code-review
 description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_INFO
 ---
@@ -21,6 +21,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      - 第 2 行 `+` 或空格：行号 = `New Start + 1`
      - 以此类推，`-` 开头的删除行不计入行号
    - **重要**：行号是变更后文件中的绝对行号，必须使用 `New Start` 作为基准计算
+   - **只报告当前 diff 块内的问题**：问题的行号必须在 `[New Start, New Start + New Count - 1]` 范围内
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
 
@@ -29,7 +30,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 6. **输出结果**：
    - 审查结果放入 `<REPORT>` 标签
    - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
-   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 19 条
+   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 18 条
    - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
 
 ## 输出模板
@@ -39,16 +40,13 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 ### 🔴 严重问题
 
----
 **问题 1**：[问题描述]<br/>
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
----
 
 **问题 2**：[问题描述]<br/>
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
----
 
 </REPORT>
 
@@ -57,7 +55,8 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 </LINE_INFO>
 
 **说明**：
-- 无问题时省略对应标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
-- 每个问题用 `---` 分割线分隔，方便切分
+- 无问题时省略 `### 🔴 严重问题` 标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
+- 每个问题以 `**问题 N**：` 开头，方便切分
 - **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**：第 1 个元素对应问题 1 的行号，第 2 个元素对应问题 2 的行号，以此类推
 - 行号计算：使用 `New Start` 作为基准，第 1 行 `+` 或空格的行号 = `New Start`，第 2 行 = `New Start + 1`，依此类推
+- **无严重问题时，LINE_INFO 必须输出空数组 `[]`**

package/index.js

@@ -102,7 +102,7 @@ class GitLabCodeReviewer {
         // 无严重问题 → 直接返回标准空格式
         if (!hasSeriousProblem) {
           debugLog(`【决策】报告无严重问题，返回标准空格式`);
-          return { reportContent: claudeResult, lineInfo: lineInfoContent };
+          return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
         // 有严重问题 → 检查标题是否符合要求
@@ -297,7 +297,7 @@ ${diffObject.diff}`;
         // 无严重问题 → 直接返回标准空格式
         if (!hasSeriousProblem) {
           debugLog(`【决策】报告无严重问题，返回标准空格式`);
-          return { reportContent: claudeResult, lineInfo: lineInfoContent };
+          return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
         // 有严重问题 → 检查标题是否符合要求
@@ -514,29 +514,32 @@ ${diffObject.diff}`;
    */
   extractSingleProblemReport(fullReport, problemIndex) {
     const lines = fullReport.split('\n');
-    // 使用 --- 分割线来切分问题
+    // 使用 **问题 N**： 模式来切分问题
     const problemBlocks = [];
     let currentBlock = [];
-    let inBlock = false;
+    let currentBlockIndex = 0;
 
     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;
+      // 检查是否是新的问题开始（如 **问题 1**：, **问题 2**：等）
+      const problemMatch = line.match(/\*\*问题\s*(\d+)\*\*/);
+      if (problemMatch) {
+        // 保存上一个问题块
+        if (currentBlock.length > 0 && currentBlockIndex > 0) {
+          problemBlocks.push(currentBlock.join('\n'));
+          currentBlock = [];
         }
-      } else if (inBlock) {
+        currentBlockIndex = parseInt(problemMatch[1]);
+        currentBlock.push(line);
+      } else {
         currentBlock.push(line);
       }
     }
 
+    // 保存最后一个问题块
+    if (currentBlock.length > 0 && currentBlockIndex > 0) {
+      problemBlocks.push(currentBlock.join('\n'));
+    }
+
     // 返回第 problemIndex 个问题（索引从 1 开始）
     if (problemIndex >= 1 && problemIndex <= problemBlocks.length) {
       return problemBlocks[problemIndex - 1];