job51-gitlab-cr-node-jt-1 2.6.2 → 2.6.3

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

@@ -247,6 +247,15 @@
         >      - **正确做法**：放弃从完整文件读取的行号，重新基于 diff 块内容逐行计数
         >      - 示例：diff 块 `@@ -0,0 +1,56 @@` 范围是 [1, 56]，如果你在完整文件中定位到第 60 行 → **该行不在 diff 块内，不能用**
         >      - **原因**：GitLab API 只能评论 diff 块范围内的行
+        >   5. **⚠️ 临时文件元数据行处理**（关键）：
+        >      - 临时文件格式包含 8 行元数据（7 行信息 + 1 行`@@`），这些都**不编号**
+        >      - **必须从 `@@` 之后的第一行才开始编号为 1**
+        >      - 验证方法：检查你的第一个行号是否等于 `new_start`。如果不是，说明你把元数据行算进去了
+        >      - 示例：`@@ -0,0 +1,56 @@` 的第一个新增行（如 `+package`）的行号必须是 **1**
+        >      - 错误示例：如果你返回的行号是 9、10、43 等，说明你把 8 行元数据算进去了，**必须重新计数**
+        >   6. **输出前最终验证**：
+        >      - 对于每个问题的行号 X，问自己："X - new_start + 1 是否等于我在 diff 块中从 `@@` 之后开始数的位置？"
+        >      - 如果不等，说明计算错误，必须重新从 `@@` 之后逐行计数
         >   - 示例：`@@ -0,0 +1,56 @@` 的 diff 块，行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
         >   - 示例：`@@ -167,7 +167,6 @@` 的 diff 块，new_start=167, new_count=6，行号范围是 [167, 172]
     > - **关键**：绝对行号必须通过 diff 块内计数计算，且结果必须在 diff 块范围 [new_start, new_start + new_count - 1] 内

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

@@ -46,6 +46,10 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      2. **逐个检查**每个问题的行号是否在该范围内
      3. **如果任何行号超出范围，必须重新基于 diff 块计数**
      4. **禁止**直接使用从完整文件读取的行号
+     5. **验证计算过程**：在输出前，必须在心中默念验证：
+        - "这个行号 X 是否在 [new_start, new_start + new_count - 1] 范围内？"
+        - "如果 X 超出范围，说明我计数错误，必须从 `@@` 行之后重新逐行编号"
+        - "临时文件的前 8 行（7 行元数据 + 1 行`@@`）都不能编号，从第 9 行（`@@`之后的第一行）才开始编号为 1"
    - **示例**：
      - `@@ -0,0 +1,56 @@` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
      - `@@ -167,7 +167,6 @@` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**