job51-gitlab-cr-node-jt-1 2.2.6 → 2.2.7
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.
|
@@ -79,6 +79,8 @@
|
|
|
79
79
|
> - **配置文件内容变更**(如 JSON/YAML/Properties 等),仅检查文件格式正确性,不检查业务逻辑
|
|
80
80
|
> - **构建配置文件版本号变更**(如 pom.xml 中的版本号升级),不检查版本依赖兼容性
|
|
81
81
|
> - **没有修改的代码**(即以空格开头的上下文代码),仅作为参考,不得审查未变更的代码行
|
|
82
|
+
> - **已删除的代码**(以 `-` 开头的行),**禁止**对已删除的代码报告任何问题
|
|
83
|
+
> - **⚠️ 重点强调**:审查前必须先丢弃所有 `-` 开头的行,只审查 `+` 开头的新增代码
|
|
82
84
|
5. 使用当前系统时间填充 `[当前时间]`,并根据实际变更内容计算 `[文件数量]`;
|
|
83
85
|
6. 所有占位符必须被真实内容替换;
|
|
84
86
|
> - **修改建议中的代码示例**:
|
|
@@ -141,19 +143,31 @@
|
|
|
141
143
|
> - **去重检查**:检查是否存在行号重叠,若发现重叠仅保留最高优先级(严重问题 > 警告)
|
|
142
144
|
> - **警告验证**:检查警告中是否包含运行时异常关键词,若包含则升级为严重问题
|
|
143
145
|
> - **代码示例验证**:确保错误代码示例不包含 `-` 开头的已删除代码
|
|
146
|
+
> - **删除代码验证**:**逐条检查问题涉及的代码行,确认不是 `-` 开头的已删除代码**
|
|
144
147
|
> - **安全问题验证**:确保不针对已删除代码报告安全问题
|
|
145
148
|
|
|
146
149
|
17. **代码变更方向识别**:
|
|
147
|
-
> - `-`
|
|
148
|
-
> - `+`
|
|
150
|
+
> - `-` 开头:**已删除的代码**,**禁止**作为审查对象,**禁止**报告任何问题
|
|
151
|
+
> - `+` 开头:**新增的代码**,是审查的唯一重点
|
|
149
152
|
> - 空格开头:未变更的上下文代码,仅作参考
|
|
150
|
-
> -
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
> - **步骤
|
|
155
|
-
|
|
156
|
-
> - **步骤
|
|
153
|
+
> - **错误代码选取**:必须从变更后实际存在的代码中选取(`+` 开头或空格开头)
|
|
154
|
+
> - **⚠️ 违例后果**:对 `-` 开头的代码报告问题是严重误报,必须避免
|
|
155
|
+
|
|
156
|
+
18. **变更内容解析步骤**(分析前必须按顺序执行):
|
|
157
|
+
> - **步骤 1:识别变更类型**
|
|
158
|
+
> - 遍历代码块,标记每行类型:`-`(删除)、`+`(新增)、空格(保留)
|
|
159
|
+
> - **步骤 2:丢弃已删除代码**(关键步骤)
|
|
160
|
+
> - **直接丢弃所有 `-` 开头的行**,不纳入审查范围
|
|
161
|
+
> - 这些行已从代码库中移除,审查它们没有任何意义
|
|
162
|
+
> - **步骤 3:构建变更后代码视图**
|
|
163
|
+
> - 仅保留 `+` 开头的行和空格开头的行
|
|
164
|
+
> - **步骤 4:基于变更后代码视图进行审查**
|
|
165
|
+
> - 所有问题识别、行号计算、错误代码选取均基于变更后代码视图
|
|
166
|
+
> - **步骤 5:验证错误代码示例**
|
|
167
|
+
> - 检查错误代码示例中是否包含 `-` 前缀
|
|
168
|
+
> - 若包含,必须重新选取变更后实际存在的代码
|
|
169
|
+
> - **步骤 6:最终验证**
|
|
170
|
+
> - 确认没有对已删除代码报告任何问题
|
|
157
171
|
|
|
158
172
|
19. **输出格式**:严格参照 SKILL.md 模板,无某类问题时省略对应部分;
|
|
159
173
|
|