job51-gitlab-cr-node-skill-prompt-optimize 1.3.4 → 1.3.6
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/mr-review-template.md +36 -15
- package/package.json +1 -1
package/mr-review-template.md
CHANGED
|
@@ -75,27 +75,48 @@
|
|
|
75
75
|
> - **第四步：验证警告中是否包含运行时异常风险**
|
|
76
76
|
> - 检查警告问题描述中是否包含"空指针"、"NullPointerException"、"自动拆箱"等关键词
|
|
77
77
|
> - 若包含，必须将该问题升级为严重问题
|
|
78
|
-
> -
|
|
79
|
-
> -
|
|
80
|
-
> -
|
|
81
|
-
>
|
|
82
|
-
|
|
83
|
-
|
|
78
|
+
> - **第五步：验证错误代码示例**
|
|
79
|
+
> - 检查所有错误代码示例中是否包含 `-` 开头的行
|
|
80
|
+
> - 若包含，必须重新选取变更后实际存在的代码
|
|
81
|
+
> - **第六步：验证安全问题是否针对已删除代码**
|
|
82
|
+
> - 检查安全问题涉及的敏感信息（如令牌、密码等）是否在变更中已被删除
|
|
83
|
+
> - 若已删除（`-` 开头），必须删除该安全问题条目
|
|
84
|
+
> - **最终验证**：生成报告前必须确认：
|
|
85
|
+
> - 严重问题、警告、优化建议中不存在任何行号重叠的条目
|
|
86
|
+
> - 所有错误代码示例展示的是变更后实际存在的代码（无 `-` 前缀）
|
|
87
|
+
> - 不存在针对已删除代码的安全问题报告
|
|
84
88
|
17. **代码变更方向识别规则**：
|
|
85
89
|
> - **变更内容格式说明**：
|
|
86
|
-
> - 以 `-`
|
|
87
|
-
|
|
88
|
-
|
|
90
|
+
> - 以 `-` 开头的行表示**已删除的代码**，**禁止**作为审查对象
|
|
91
|
+
> - 以 `+` 开头的行表示**新增的代码**，是审查的重点
|
|
92
|
+
> - 以空格开头的行表示**未变更的上下文代码**，仅作参考
|
|
89
93
|
> - **审查对象**：
|
|
90
94
|
> - 仅审查**变更后保留的代码**（新增代码 + 未删除的原有代码）
|
|
91
|
-
|
|
95
|
+
> - **禁止**对已删除的代码报告任何问题（包括安全问题、性能问题等）
|
|
92
96
|
> - **错误代码示例的选取**：
|
|
93
|
-
> -
|
|
94
|
-
|
|
97
|
+
> - 必须从**变更后实际存在的代码**中选取（`+` 开头的行或空格开头的保留行）
|
|
98
|
+
> - **禁止**展示以 `-` 开头的已删除代码作为错误示例
|
|
99
|
+
> - **安全问题特殊处理**：
|
|
100
|
+
> - 若敏感信息（如硬编码令牌、密码等）在变更中已被删除（`-` 开头），**不得**再报告此安全问题
|
|
101
|
+
> - 仅当敏感信息在变更后仍存在（`+` 开头或空格开头的保留行）时，才报告安全问题
|
|
95
102
|
> - **示例**：
|
|
96
|
-
> -
|
|
97
|
-
|
|
98
|
-
|
|
103
|
+
> - 若代码第一个位置以 `-` 开头表示已删除
|
|
104
|
+
> - **处理**：不报告此安全问题
|
|
105
|
+
> - 若该行以 `+` 开头表示新增或保留
|
|
106
|
+
> - **处理**：报告安全问题，错误代码展示该行（不带 `-` 前缀）
|
|
107
|
+
18. **变更内容解析步骤**（分析前必须执行）：
|
|
108
|
+
> - **步骤 1：识别变更类型**
|
|
109
|
+
> - 遍历 `<CHANGE_CONTENT>` 中的每一行
|
|
110
|
+
> - 标记每行的类型：`-`（删除）、`+`（新增）、空格（保留）
|
|
111
|
+
> - **步骤 2：构建变更后代码视图**
|
|
112
|
+
> - 仅保留 `+` 开头的行和空格开头的行
|
|
113
|
+
> - **丢弃**所有 `-` 开头的行
|
|
114
|
+
> - **步骤 3：基于变更后代码视图进行审查**
|
|
115
|
+
> - 所有问题识别、行号计算、错误代码选取均基于变更后代码视图
|
|
116
|
+
> - **禁止**引用已丢弃的 `-` 开头行
|
|
117
|
+
> - **步骤 4：验证错误代码示例**
|
|
118
|
+
> - 检查错误代码示例中是否包含 `-` 前缀
|
|
119
|
+
> - 若包含，必须移除 `-` 前缀或重新选取变更后实际存在的代码
|
|
99
120
|
|
|
100
121
|
<REPORT>
|
|
101
122
|
## 🤖 AI代码审查结果
|