job51-gitlab-cr-node-jt-1 2.5.2 → 2.5.3
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.
|
@@ -182,14 +182,39 @@
|
|
|
182
182
|
|
|
183
183
|
13. **行号计算规范**：
|
|
184
184
|
> - **行号定义**：行号是变更后文件中的**绝对行号**（从 1 开始计数）
|
|
185
|
-
> -
|
|
186
|
-
> -
|
|
187
|
-
> -
|
|
188
|
-
> -
|
|
185
|
+
> - **⚠️ 关键规则：基于 diff 块内容逐行计数**（强制执行）：
|
|
186
|
+
> - **禁止**读取完整文件后计算绝对行号
|
|
187
|
+
> - **必须**基于 diff 块内容，从第一行开始逐行计数
|
|
188
|
+
> - 计数规则：
|
|
189
|
+
> - 从 diff 块的 `@@ ... @@` 头之后的第一行开始计数
|
|
190
|
+
> - `+` 开头的新增代码：计入行号
|
|
191
|
+
> - 空格开头的上下文代码：计入行号
|
|
192
|
+
> - `-` 开头的删除代码：**不计入**行号（已删除）
|
|
193
|
+
> - **问题行号 = 该行在 diff 块中的位置**
|
|
189
194
|
> - **示例**：
|
|
190
|
-
> - 新增文件 `@@ -0,0 +1,
|
|
191
|
-
|
|
192
|
-
|
|
195
|
+
> - 新增文件 `@@ -0,0 +1,75 @@`：
|
|
196
|
+
```
|
|
197
|
+
@@ -0,0 +1,75 @@
|
|
198
|
+
+package com.example; // 行号 = 1
|
|
199
|
+
+
|
|
200
|
+
+import java.util.List; // 行号 = 3
|
|
201
|
+
+
|
|
202
|
+
+public class Test { // 行号 = 5
|
|
203
|
+
+ public void method() { // 行号 = 6
|
|
204
|
+
+ Object obj = service.getData(); // 行号 = 7
|
|
205
|
+
+ obj.toString(); // 行号 = 8 ← 问题在这行，new_line = 8
|
|
206
|
+
+ }
|
|
207
|
+
+}
|
|
208
|
+
```
|
|
209
|
+
> - 已有文件 `@@ -42,3 +43,5 @@`：
|
|
210
|
+
```
|
|
211
|
+
@@ -42,3 +43,5 @@
|
|
212
|
+
public void method() { // 行号 = 43（上下文行，保留）
|
|
213
|
+
+ Object obj = service.getData(); // 行号 = 44
|
|
214
|
+
+ obj.toString(); // 行号 = 45 ← 问题在这行，new_line = 45
|
|
215
|
+
}
|
|
216
|
+
```
|
|
217
|
+
> - **关键**：问题行号必须准确指向变更后代码在 diff 块中的位置
|
|
193
218
|
|
|
194
219
|
14. `<REPORT>` 标签示例中的问题块数量仅用于展示格式，实际数量由 review 结果决定；
|
|
195
220
|
|
|
@@ -13,10 +13,12 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
|
|
|
13
13
|
- `New Count`: 当前 diff 块的行数
|
|
14
14
|
- `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
|
|
15
15
|
|
|
16
|
-
3.
|
|
16
|
+
3. **解析行号**：**基于 diff 块内容逐行计数**
|
|
17
|
+
- **⚠️ 重要**：行号计算规则详见 @.claude/rules/code-review-rules.md 第 13 条
|
|
18
|
+
- **禁止**读取完整文件后计算绝对行号
|
|
19
|
+
- **必须**基于 diff 块内容，从第一行开始逐行计数
|
|
17
20
|
- 行号定义：行号是变更后文件中的绝对行号（从 1 开始计数）
|
|
18
|
-
-
|
|
19
|
-
- **如何确定行号**：读取变更后文件（基于 `New Path`），找到新增代码在文件中的实际行号
|
|
21
|
+
- 例如：如果新增代码在 diff 块的第 43 行，则 `new_line = 43`
|
|
20
22
|
|
|
21
23
|
4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
|
|
22
24
|
|