job51-gitlab-cr-node-jt-1 2.3.1 → 2.3.3
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.
|
@@ -66,6 +66,10 @@
|
|
|
66
66
|
> - **错误处理**：如果某些上下文文件不存在或无法读取，跳过并继续审查
|
|
67
67
|
|
|
68
68
|
> - **⚠️ 代码上下文深度分析要求（强制执行）**：
|
|
69
|
+
> - **必须读取完整方法上下文**：
|
|
70
|
+
> - 审查前必须使用 Read 工具读取**整个方法**的代码，不只是 diff 块
|
|
71
|
+
> - 分析方法开始处是否有判空保护、早期返回、断言等逻辑
|
|
72
|
+
> - 示例：方法开头有 `if (accountInfo == null) return;`，后续 `accountInfo.getXxx()` **不应报告 NPE**
|
|
69
73
|
> - **必须跨 diff 块分析同一文件/方法的上下文**：
|
|
70
74
|
> - 同一个文件/方法可能被拆分成多个 diff 块，**必须合并所有块一起分析**
|
|
71
75
|
> - 示例：块 A 显示删除了 `obj.setValue(xxx)`，块 B 显示新增了 `if(condition) obj.setValue(yyy)`
|
|
@@ -96,6 +100,14 @@
|
|
|
96
100
|
> - 基本类型（int, boolean）：不可能为 null
|
|
97
101
|
> - 明确返回空集合的方法（如 `return Collections.emptyList()`）
|
|
98
102
|
> - 有 `@NotNull` 注解或 `Objects.requireNonNull()` 保护的方法
|
|
103
|
+
> - **早期返回模式**：
|
|
104
|
+
> - 方法开头已有判空/边界检查并提前返回的代码，后续逻辑**不应报告**空指针问题
|
|
105
|
+
> - 示例：`if (query == null || list.isEmpty()) return emptyList;` 后续有正常业务逻辑，**不应报告**问题
|
|
106
|
+
> - **禁止报告"代码简化建议"类问题**：
|
|
107
|
+
> - **禁止**建议"移除变量声明，直接 return xxx.stream()..."
|
|
108
|
+
> - **禁止**建议"合并变量声明和返回语句"
|
|
109
|
+
> - **禁止**对正常的早期返回模式（Early Return）提出简化建议
|
|
110
|
+
> - 这类建议属于代码风格偏好，不是真正的缺陷
|
|
99
111
|
|
|
100
112
|
1. **仅输出一份代码审查报告**，以 `<REPORT>` 开始，`</REPORT>` 结束的 Markdown 格式生成；
|
|
101
113
|
2. **不得输出任何解释、问候、总结或额外文本**；
|
|
@@ -242,3 +254,8 @@
|
|
|
242
254
|
> - [ ] **已确认建议代码与项目原有风格一致**（参数校验方式、异常处理方式、注解使用习惯等）
|
|
243
255
|
> - [ ] **已执行跨块去重**：同一文件同一根本原因的问题只报告一次
|
|
244
256
|
> - [ ] **已验证问题位置**：问题报告在真正会执行出错的行上，而非 package/import/类定义/方法签名等位置
|
|
257
|
+
> - [ ] **已确认报告不自相矛盾**：
|
|
258
|
+
> - 如果代码已有 `if (obj != null)` 判空检查，**禁止**报告"未判空"问题
|
|
259
|
+
> - 如果分析结论包含"当前代码逻辑正确"、"无需修改"、"已有 XX 保护"等描述，**禁止**将该问题报告为严重问题
|
|
260
|
+
> - **报告前自检**：问题描述、错误代码、修改建议三者逻辑必须一致，不得出现"有判空但说未判空"的矛盾
|
|
261
|
+
> - **建议写法**：如果确实想提醒注意某行为，但代码已正确处理，应省略该问题，不输出报告
|