job51-gitlab-cr-node-jt-1 2.2.6 → 2.2.8
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
|
> - **修改建议中的代码示例**：
|
|
@@ -88,6 +90,9 @@
|
|
|
88
90
|
> - **代码风格一致性**：建议代码应遵循项目现有风格（工具类方法、函数式编程、命名约定等）
|
|
89
91
|
7. 若无某类问题（如无严重问题），则**完全省略该部分**；
|
|
90
92
|
8. 若同一级别问题有多个，在对应标题下按原代码顺序依次展示；
|
|
93
|
+
> - **⚠️ 跨块去重规则**：**同一文件中由同一根本原因导致的问题，只报告一次**
|
|
94
|
+
> - 示例：同一方法 `userTaskSDK.xxx()` 在文件中多处被调用，若都存在未判空问题，**合并为一条报告**
|
|
95
|
+
> - 判断标准：**文件 + 问题类型 + 根本原因**三者相同即为重复问题
|
|
91
96
|
9. **严重问题（🔴）**：
|
|
92
97
|
> - 安全漏洞（SQL 注入、命令注入、敏感信息泄露）
|
|
93
98
|
> - 可能导致数据丢失/损坏/不一致
|
|
@@ -139,21 +144,36 @@
|
|
|
139
144
|
|
|
140
145
|
16. **输出前验证流程**：
|
|
141
146
|
> - **去重检查**：检查是否存在行号重叠，若发现重叠仅保留最高优先级（严重问题 > 警告）
|
|
147
|
+
> - **跨块重复问题去重**：**同一文件同一问题在不同 diff 块中出现时，只报告一次**
|
|
148
|
+
> - 示例：同一方法 `userTaskSDK.userTaskModuleControl()` 在块 1、块 2、块 3 中被调用，未判空问题**只报告一次**
|
|
149
|
+
> - 方法：**按「文件 + 问题类型 + 根本原因」进行去重**，相同组合的问题只保留第一个
|
|
142
150
|
> - **警告验证**：检查警告中是否包含运行时异常关键词，若包含则升级为严重问题
|
|
143
151
|
> - **代码示例验证**：确保错误代码示例不包含 `-` 开头的已删除代码
|
|
152
|
+
> - **删除代码验证**：**逐条检查问题涉及的代码行，确认不是 `-` 开头的已删除代码**
|
|
144
153
|
> - **安全问题验证**：确保不针对已删除代码报告安全问题
|
|
145
154
|
|
|
146
155
|
17. **代码变更方向识别**：
|
|
147
|
-
> - `-`
|
|
148
|
-
> - `+`
|
|
156
|
+
> - `-` 开头：**已删除的代码**，**禁止**作为审查对象，**禁止**报告任何问题
|
|
157
|
+
> - `+` 开头：**新增的代码**，是审查的唯一重点
|
|
149
158
|
> - 空格开头：未变更的上下文代码，仅作参考
|
|
150
|
-
> -
|
|
151
|
-
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
> - **步骤
|
|
155
|
-
|
|
156
|
-
> - **步骤
|
|
159
|
+
> - **错误代码选取**：必须从变更后实际存在的代码中选取（`+` 开头或空格开头）
|
|
160
|
+
> - **⚠️ 违例后果**：对 `-` 开头的代码报告问题是严重误报，必须避免
|
|
161
|
+
|
|
162
|
+
18. **变更内容解析步骤**（分析前必须按顺序执行）：
|
|
163
|
+
> - **步骤 1：识别变更类型**
|
|
164
|
+
> - 遍历代码块，标记每行类型：`-`（删除）、`+`（新增）、空格（保留）
|
|
165
|
+
> - **步骤 2：丢弃已删除代码**（关键步骤）
|
|
166
|
+
> - **直接丢弃所有 `-` 开头的行**，不纳入审查范围
|
|
167
|
+
> - 这些行已从代码库中移除，审查它们没有任何意义
|
|
168
|
+
> - **步骤 3：构建变更后代码视图**
|
|
169
|
+
> - 仅保留 `+` 开头的行和空格开头的行
|
|
170
|
+
> - **步骤 4：基于变更后代码视图进行审查**
|
|
171
|
+
> - 所有问题识别、行号计算、错误代码选取均基于变更后代码视图
|
|
172
|
+
> - **步骤 5：验证错误代码示例**
|
|
173
|
+
> - 检查错误代码示例中是否包含 `-` 前缀
|
|
174
|
+
> - 若包含，必须重新选取变更后实际存在的代码
|
|
175
|
+
> - **步骤 6：最终验证**
|
|
176
|
+
> - 确认没有对已删除代码报告任何问题
|
|
157
177
|
|
|
158
178
|
19. **输出格式**：严格参照 SKILL.md 模板，无某类问题时省略对应部分；
|
|
159
179
|
|
|
@@ -166,3 +186,4 @@
|
|
|
166
186
|
> - [ ] 已确认不是返回空集合或有不返回 null 保证的方法
|
|
167
187
|
> - [ ] **已确认问题不是"风格偏好"类问题**（如建议添加项目惯例中不存在的注解/校验/异常处理）
|
|
168
188
|
> - [ ] **已确认建议代码与项目原有风格一致**（参数校验方式、异常处理方式、注解使用习惯等）
|
|
189
|
+
> - [ ] **已执行跨块去重**：同一文件同一根本原因的问题只报告一次
|