job51-gitlab-cr-node-skill-prompt-optimize 1.3.9 → 1.4.1

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

@@ -1,68 +1,84 @@
----
-name: simple-code-review
-description: 你是一个专业的代码审查助手。请严格根据代码规范、安全规则（如 SQL 注入防护、XSS 防护）、性能要求（如时间复杂度优化、内存占用控制）、可读性要求（如注释完整性、命名规范性）及功能正确性标准处理指定文件中的代码变更内容
----
-
-## 处理步骤
-1. 首先使用 Read 工具读取 diff 文件：$ARGUMENTS
-2. **解析文件信息**：从文件内容中提取以下信息
-   - New Path: 变更后的文件路径（可能包含 `b/` 前缀）
-   - Old Path: 变更前的文件路径（可能包含 `a/` 前缀）
-   - Diff Content: 实际的代码变更内容
-3. **处理文件路径**：去掉路径中的 `a/` 或 `b/` 前缀，得到实际的项目相对路径
-4. **读取项目上下文文件**：基于解析出的文件路径，主动读取相关的项目文件（详见规则文档）
-5. **深度分析代码上下文（必须执行）**：
-   - 对于diff中涉及的方法/函数调用，**必须**使用 Read 工具追踪读取该方法的实现代码
-   - 示例：若看到 `accountInfo = userSDK.getAccountInfo(id, false); accountInfo.getXxx()`，必须读取 `userSDK.getAccountInfo` 方法的实现，确认其返回值是否可能为 null
-   - **禁止**仅凭代码表面形式判断存在空指针/空引用等运行时异常
-   - **跨 diff 块分析**：同一文件可能有多个 diff 块，必须合并分析。如块A删除了 `obj.setValue(xxx)`，但块B新增了 `if(condition) obj.setValue(yyy)`，不应报告"删除了set值会影响数据"
-   - 只有在确认方法/函数确实可能返回危险值时，才能报告严重问题
-6. **结合项目上下文**，按照 @.claude/rules/code-review-rules.md 中的规则进行审查
-7. **提取行号信息**：审查完成后，从审查结果中提取所有问题的行号信息
-   - 遍历每个问题，解析"**文件及行号**"行中的文件路径和行号
-   - 将行号信息以 JSON 数组格式放入 `<LINE_INFO>` 标签内
-   - **LINE_INFO 格式必须包含 `new_line` 和 `new_path` 字段**：
-   - 示例：`<LINE_INFO>[{"new_path":"src/main/java/UserService.java","new_line":87,"old_line":85,"old_path":"src/main/java/UserService.java"}]</LINE_INFO>`
-   - **字段说明**：
-     - `new_path`: 变更后文件路径（必填，用于新增/修改的代码）
-     - `new_line`: 变更后代码行号（必填，用于定位新增/修改的代码位置）
-     - `old_path`: 变更前文件路径（可选，用于删除的代码）
-     - `old_line`: 变更前代码行号（可选，用于定位删除的代码位置）
-8. 输出格式必须以下方模板为准
-
-## 输出模板
-
-``**说明**：
-- 必须以 `<REPORT>` 开始，以 `</REPORT>` 结束
-- 若某类问题不存在，则**完全省略该模块**（不输出对应标题）
-- 所有占位符必须被真实内容替换
-- 带圈数字序号：① ② ③ ④ ⑤ ⑥ ⑦ ⑧ ⑨ ⑩``
-
-<REPORT>
-## 🤖 AI 代码审查结果
-
-**生成时间**: [当前时间]
-**审查范围**: [文件数量] 个文件
-
----
-
-### 审查总览
-
-[简要总结]
-
-### 🔴 严重问题
-
-[带圈数字序号][问题描述]<br/>
-**文件及行号**：[完整文件路径：起始行 - 结束行，如：src/main/java/com/example/UserService.java:87-91]<br/>
-**修改建议**：[修复建议 + 错误代码 + 正确示例]
-
-### 🟡 警告
-
-[带圈数字序号][问题描述]<br/>
-**文件及行号**：[完整文件路径：起始行 - 结束行]<br/>
-**修改建议**：[改进建议 + 代码示例]
-
-</REPORT>
-<LINE_INFO>
-[{"new_path":"文件路径","new_line":行号,"old_path":"文件路径","old_line":行号}]
-</LINE_INFO>
+---
+name: simple-code-review
+description: 你是一个专业的代码审查助手。请严格根据代码规范、安全规则（如 SQL 注入防护、XSS 防护）、性能要求（如时间复杂度优化、内存占用控制）、可读性要求（如注释完整性、命名规范性）及功能正确性标准处理指定文件中的代码变更内容
+---
+
+## 处理步骤
+
+1. **读取 diff 文件**：使用 Read 工具读取文件：$ARGUMENTS
+
+2. **解析文件信息**：从文件内容中提取以下信息
+   - `Block Index`: 当前块的索引号
+   - `Line Info`: 行号信息 JSON 对象，包含 `old_start`, `old_count`, `new_start`, `new_count`, `firstLineFirstChar`
+   - `Diff Content`: 实际的代码变更内容（unified diff 格式）
+
+3. **解析 Line Info**：解析 `Line Info` 行中的 JSON 对象，获取 diff 块的行号信息
+   - `new_start`: 变更后代码的起始行号
+   - `new_count`: 变更后代码的行数
+   - `old_start`: 变更前代码的起始行号（仅作参考）
+   - `old_count`: 变更前代码的行数（仅作参考）
+   - `firstLineFirstChar`: diff 块第一行的第一个字符（`+` 表示新增，` ` 表示保留）
+
+4. **仅审查新增代码**（关键规则）：
+   - **只审查 `+` 开头的新增代码行**
+   - **忽略 `-` 开头的已删除代码行**（已删除的代码不纳入审查范围）
+   - 空格开头的上下文代码仅作参考，不进行审查
+
+5. **读取项目上下文文件**：
+   - 基于 `Line Info` 和 diff 内容，读取**变更后文件**（New Path）的完整内容
+   - 如果涉及方法调用，读取相关方法/类的实现代码以确认是否存在真实的运行时风险
+
+6. **深度分析代码上下文（必须执行）**：
+   - 对于 diff 中涉及的方法/函数调用，**必须**使用 Read 工具追踪读取该方法的实现代码
+   - **跨 diff 块分析**：同一文件可能有多个 diff 块，必须合并分析
+   - 只有在确认方法/函数确实可能返回危险值时，才能报告严重问题
+
+7. **结合项目上下文**，按照 @.claude/rules/code-review-rules.md 中的规则进行审查
+
+8. **提取行号信息**：审查完成后，从审查结果中提取所有问题的行号信息
+   - 遍历每个问题，解析"**文件及行号**"行中的文件路径和行号
+   - 将行号信息以 JSON 数组格式放入 `<LINE_INFO>` 标签内
+   - **LINE_INFO 格式必须包含 `new_path` 和 `new_line` 字段**：
+   - 示例：`<LINE_INFO>[{"new_path":"src/main/java/UserService.java","new_line":87}]</LINE_INFO>`
+   - **字段说明**：
+     - `new_path`: 变更后文件路径（必填）
+     - `new_line`: 变更后代码行号（必填，指向 `+` 开头的新增代码在变更后文件中的行号）
+
+9. 输出格式必须以下方模板为准
+
+## 输出模板
+
+**说明**：
+- 必须以 `<REPORT>` 开始，以 `</REPORT>` 结束
+- 若某类问题不存在，则**完全省略该模块**（不输出对应标题）
+- 所有占位符必须被真实内容替换
+- 带圈数字序号：① ② ③ ④ ⑤ ⑥ ⑦ ⑧ ⑨ ⑩
+
+<REPORT>
+## 🤖 AI 代码审查结果
+
+**生成时间**: [当前时间]
+**审查范围**: 仅审查新增代码（+ 开头的行）
+
+---
+
+### 审查总览
+
+[简要总结]
+
+### 🔴 严重问题
+
+[带圈数字序号][问题描述]<br/>
+**文件及行号**：[完整文件路径：行号，如：src/main/java/com/example/UserService.java:87]<br/>
+**修改建议**：[修复建议 + 错误代码 + 正确示例]
+
+### 🟡 警告
+
+[带圈数字序号][问题描述]<br/>
+**文件及行号**：[完整文件路径：行号]<br/>
+**修改建议**：[改进建议 + 代码示例]
+
+</REPORT>
+<LINE_INFO>
+[{"new_path":"文件路径","new_line":行号}]
+</LINE_INFO>

package/index.js

@@ -205,6 +205,7 @@ ${diffObject.diff}`;
 1. 严格按照 .claude/skills/simple-code-review/SKILL.md 中定义的模板格式输出
 2. **每个模块不是必须的**：没有对应问题时，完全省略该模块（不输出标题）
 3. 必须以 <REPORT> 开始，以 </REPORT> 结束
+4. 如有严重问题，在<LINE_INFO>标签中构建问题对应的行号信息
 4. 不要输出任何额外的解释、问候或总结文本`;
     //打印
     debugLog(`Claude命令: ${prompt}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-skill-prompt-optimize",
-  "version": "1.3.9",
+  "version": "1.4.1",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {