job51-gitlab-cr-node-jt-1 2.3.0 → 2.3.2
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.
|
@@ -33,6 +33,12 @@
|
|
|
33
33
|
|
|
34
34
|
5. **⚠️ 特别强调**：当前 CR 流程处于版本开发的最后合并阶段，代码逻辑已经过开发自测和 QA 测试验证。审查时应**基于项目原有的代码风格**，不得强制要求添加项目惯例中不存在的内容。
|
|
35
35
|
> - **⚠️ 风格一致性原则**：参数校验、异常处理、注解使用等应与项目原有风格匹配，**不得报告"建议添加 XX 注解/XX 校验/XX 异常处理"** 这类问题。
|
|
36
|
+
> - **⚠️ Controller/Client 接口参数校验放宽**：
|
|
37
|
+
> - Controller 接口方法、Client 接口方法的参数校验**以项目原有风格为准**
|
|
38
|
+
> - 如果项目现有接口多数没有添加 `@NotNull`、`@NotBlank`、`@Valid` 等校验注解，**不得建议添加**
|
|
39
|
+
> - 如果项目现有接口多数没有显式参数判空逻辑，**不得建议添加**
|
|
40
|
+
> - 参数校验形式应参考同文件/同模块其他接口的写法，保持一致性
|
|
41
|
+
> - **仅在校验风格与同一文件中其他接口明显不一致时**，才需要提示
|
|
36
42
|
|
|
37
43
|
0. **Diff 数据结构与上下文读取规则**：
|
|
38
44
|
> - **临时文件格式说明**：
|
|
@@ -90,6 +96,14 @@
|
|
|
90
96
|
> - 基本类型（int, boolean）：不可能为 null
|
|
91
97
|
> - 明确返回空集合的方法（如 `return Collections.emptyList()`）
|
|
92
98
|
> - 有 `@NotNull` 注解或 `Objects.requireNonNull()` 保护的方法
|
|
99
|
+
> - **早期返回模式**：
|
|
100
|
+
> - 方法开头已有判空/边界检查并提前返回的代码，后续逻辑**不应报告**空指针问题
|
|
101
|
+
> - 示例：`if (query == null || list.isEmpty()) return emptyList;` 后续有正常业务逻辑，**不应报告**问题
|
|
102
|
+
> - **禁止报告"代码简化建议"类问题**：
|
|
103
|
+
> - **禁止**建议"移除变量声明，直接 return xxx.stream()..."
|
|
104
|
+
> - **禁止**建议"合并变量声明和返回语句"
|
|
105
|
+
> - **禁止**对正常的早期返回模式（Early Return）提出简化建议
|
|
106
|
+
> - 这类建议属于代码风格偏好，不是真正的缺陷
|
|
93
107
|
|
|
94
108
|
1. **仅输出一份代码审查报告**，以 `<REPORT>` 开始，`</REPORT>` 结束的 Markdown 格式生成；
|
|
95
109
|
2. **不得输出任何解释、问候、总结或额外文本**；
|
|
@@ -236,3 +250,8 @@
|
|
|
236
250
|
> - [ ] **已确认建议代码与项目原有风格一致**（参数校验方式、异常处理方式、注解使用习惯等）
|
|
237
251
|
> - [ ] **已执行跨块去重**：同一文件同一根本原因的问题只报告一次
|
|
238
252
|
> - [ ] **已验证问题位置**：问题报告在真正会执行出错的行上，而非 package/import/类定义/方法签名等位置
|
|
253
|
+
> - [ ] **已确认报告不自相矛盾**：
|
|
254
|
+
> - 如果代码已有 `if (obj != null)` 判空检查，**禁止**报告"未判空"问题
|
|
255
|
+
> - 如果分析结论包含"当前代码逻辑正确"、"无需修改"、"已有 XX 保护"等描述，**禁止**将该问题报告为严重问题
|
|
256
|
+
> - **报告前自检**：问题描述、错误代码、修改建议三者逻辑必须一致，不得出现"有判空但说未判空"的矛盾
|
|
257
|
+
> - **建议写法**：如果确实想提醒注意某行为，但代码已正确处理，应省略该问题，不输出报告
|