job51-gitlab-cr-node-jt-1 2.5.9 → 2.6.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude/skills/simple-code-review/SKILL.md +13 -4
- package/docs/GITLAB_CR_NODE_TECHNICAL_DOCS.md +1 -0
- package/log.txt +1709 -1890
- package/package.json +1 -1
|
@@ -24,12 +24,21 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
|
|
|
24
24
|
- **简化计算**：
|
|
25
25
|
- 对于 `@@ -0,0 +1,N @@`（新增文件）：`绝对行号 = 编号`（因为 new_start=1）
|
|
26
26
|
- 对于 `@@ -old_start,old_count +new_start,new_count @@`（修改文件）：`绝对行号 = 编号 + new_start - 1`
|
|
27
|
-
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
27
|
+
- **⚠️ 强制验证步骤**（输出前必须执行）：
|
|
28
|
+
1. 从 diff 头提取 `new_start` 和 `new_count`，计算行号范围：`[new_start, new_start + new_count - 1]`
|
|
29
|
+
2. **逐个检查**每个问题的行号是否在该范围内
|
|
30
|
+
3. **如果任何行号超出范围，必须重新基于 diff 块计数**
|
|
31
|
+
4. **禁止**直接使用从完整文件读取的行号
|
|
32
|
+
- **示例**：
|
|
33
|
+
- `@@ -0,0 +1,56 @@` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
|
|
34
|
+
- `@@ -167,7 +167,6 @@` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**
|
|
35
|
+
- 如果完整文件中某行在第 60 行，但 diff 块范围是 [1, 56]，说明该行不在当前 diff 块内，**必须放弃该行号**
|
|
31
36
|
|
|
32
37
|
4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
|
|
38
|
+
- **⚠️ 重要**：读取完整文件仅用于理解代码逻辑，**不得**直接使用完整文件中的行号
|
|
39
|
+
- 如果从完整文件中定位到问题行号，**必须验证**该行号是否在 diff 块范围内
|
|
40
|
+
- **验证公式**：`new_start <= 行号 <= new_start + new_count - 1`
|
|
41
|
+
- **如果行号超出范围**：必须回到 diff 块内容，重新基于 `@@` 头之后逐行计数
|
|
33
42
|
|
|
34
43
|
5. **执行审查**：按照 @.claude/rules/code-review-rules.md 中的规则进行审查
|
|
35
44
|
|
|
@@ -849,6 +849,7 @@ npm run dev
|
|
|
849
849
|
|
|
850
850
|
| 版本 | 日期 | 变更说明 |
|
|
851
851
|
|------|------|----------|
|
|
852
|
+
| **2.5.9** | 2026-04-16 | **行号计算强制验证修复**：修复 AI 返回的行号与代码对不上的问题。问题根因：AI 读取完整文件后，没有基于 diff 块内容逐行计数，而是直接返回了错误的行号（如 43、45、60），这些行号既不是 diff 块内的行号，也不是完整文件中的正确行号。修复方案：(1) 更新 `SKILL.md` 第 3 步，添加**强制验证步骤**，要求输出前必须逐个检查行号是否在 diff 块范围内；(2) 在 `读取上下文` 步骤中添加强制要求：读取完整文件仅用于理解代码逻辑，**不得**直接使用完整文件中的行号；(3) 添加验证公式：`new_start <= 行号 <= new_start + new_count - 1`；(4) 明确如果行号超出范围，必须回到 diff 块内容重新基于 `@@` 头之后逐行计数 |
|
|
852
853
|
| **2.5.8** | 2026-04-16 | **行号计算规则修正**：修正行号定义为"文件的绝对行号"，但强调必须通过 diff 块内计数计算。问题根因：(1) 规则第 13 条错误地将行号定义为"diff 块内的相对行号"，与 GitLab API 需要的绝对行号冲突；(2) AI 读取完整文件后定位到 diff 块范围之外的行（如第 60 行），导致行号验证失败。修复方案：(1) 更新 `code-review-rules.md` 第 13 条，明确行号是**变更后文件中的绝对行号**；(2) 修正计算方法：`绝对行号 = 从 @@ 头之后的编号 + new_start - 1`；(3) 添加强制验证步骤：输出前必须检查 `new_start <= 行号 <= new_start + new_count - 1`；(4) 添加特别场景处理：读取完整文件后必须验证行号是否在 diff 块范围内，超出范围则必须放弃该行号，重新基于 diff 块计数；(5) 更新 `SKILL.md` 第 3 步，同步更新行号计算方法 |
|
|
853
854
|
| **2.5.7** | 2026-04-16 | **行号计算规则修复**：修复 AI 返回的行号超出 diff 块范围的问题。问题根因：AI 读取完整源文件后返回文件的绝对行号，而不是基于 diff 块的相对行号。修复方案：(1) 更新 `code-review-rules.md` 第 13 条，明确行号是**diff 块内的相对行号**，不是文件的绝对行号；(2) 添加行号验证规则：最大行号不能超过 diff 块的 `new_count` 值；(3) 简化 `SKILL.md` 中的行号计算说明，将详细规则移至 rules 文件；(4) 在 rules 中添加强制验证步骤：输出前必须检查 diff 头的 `new_count` 值，确保所有问题行号 ≤ new_count |
|
|
854
855
|
| **2.5.6** | 2026-04-16 | **当前版本**：技术文档更新 |
|