job51-gitlab-cr-node-jt-1 2.7.6 → 2.7.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.
|
@@ -45,6 +45,17 @@
|
|
|
45
45
|
> - 如果项目现有接口多数没有显式参数判空逻辑，**不得建议添加**
|
|
46
46
|
> - 参数校验形式应参考同文件/同模块其他接口的写法，保持一致性
|
|
47
47
|
> - **仅在校验风格与同一文件中其他接口明显不一致时**，才需要提示
|
|
48
|
+
> - **⚠️ 老代码修改审查原则**（关键规则）：
|
|
49
|
+
> - **审查范围限制**：对于已有代码的修改（不是新增），**只针对修改的具体内容进行评审**
|
|
50
|
+
> - **禁止建议周边逻辑改动**：**不得报告整体逻辑或相关逻辑的修改优化建议**（如"建议重构整个方法"、"建议优化周边调用链"等）
|
|
51
|
+
> - **原因**：
|
|
52
|
+
> - 贸然修改原有逻辑会引入风险，可能破坏已验证的稳定功能
|
|
53
|
+
> - 周边逻辑的改动不在当前需求点范围内，可能不会经过有效测试
|
|
54
|
+
> - 原有代码已经过开发自测和 QA 测试验证，应保持最小改动原则
|
|
55
|
+
> - **正确做法**：
|
|
56
|
+
> - 只审查变更行本身是否有运行时异常风险、安全问题等严重缺陷
|
|
57
|
+
> - 不建议对方法的整体逻辑、调用链、数据流等进行优化
|
|
58
|
+
> - 即使周边逻辑看起来可以优化，也**保持沉默**，不在本次 CR 中报告
|
|
48
59
|
|
|
49
60
|
0. **Diff 数据结构与上下文读取规则**：
|
|
50
61
|
> - **临时文件格式说明**：
|
|
@@ -322,6 +333,7 @@
|
|
|
322
333
|
> - **禁止**在报告中输出错误代码示例（已经有行号定位了，不需要重复）
|
|
323
334
|
|
|
324
335
|
19. **误报防控检查清单**（报告前必须确认）：
|
|
336
|
+
> - [ ] **⚠️ 再次查阅代码确认（最高优先级）**：报告任何问题前，必须再次使用 Read 工具查阅相关代码文件。查阅范围包括：**代码所在方法、方法调用的上下游链路、相关依赖类/接口**。仔细确认代码逻辑，验证问题是否真实存在。禁止凭记忆或第一次分析就报告问题。
|
|
325
337
|
> - [ ] **已安全处理验证**：代码是否已有判空/异常处理/边界检查？如是，省略该问题
|
|
326
338
|
> - [ ] **逻辑一致性验证**：问题描述是否自相矛盾？（如"有判空但说未判空"、"逻辑正确但报告为问题"）
|
|
327
339
|
> - [ ] 已追踪读取相关方法实现，确认可能返回危险值
|