| refactor/reviewer优化编排流程, 修复了一些幻觉问题, 减少冗余提示词, pr 检视场景新增源码视野
Co-authored-by: Junren6415<joestranberg99@gmail.com>
# message auto-generated for no-merge-commit merge:
!199 merge reviewer_refactoring into master
refactor/reviewer优化编排流程, 修复了一些幻觉问题, 减少冗余提示词, pr 检视场景新增源码视野
Created-by: Junren6415
Commit-by: Junren6415
Merged-by: cann-robot
Description: # refactor/review优化编排流程, 修复了一些幻觉问题, 减少冗余提示词, pr 检视场景新增源码视野
## 描述
### 代码检视工作流全面优化
本轮改造围绕三个核心问题展开：PR 检视上下文缺失导致误报、条例重叠导致矛盾判定、子 Agent 越界检视产生幻觉。覆盖 9 个文件，涉及主 Agent
编排、子 Agent 流程、Skill 方法论、规范文档四层。
#### 新特性 New Features
- 【ops-code-reviewer】PR 检视新增完整源码上下文支持：新增 clone_pr_source.py 独立脚本，获取 PR 完整代码仓（git clone + checkout
merge ref，自动回退 head ref）。主 Agent 在阶段0 获取 diff 后同步克隆源码，将路径传递给概要子 Agent 和检视子 Agent，使子 Agent
能结合完整源码追溯变量定义和上游校验。
- 【ops-code-reviewer】代码概要子 Agent 新增 ascendc-code-summarizer.md：PR 模式支持按文件分别总结（变更文件概览 →
逐文件脉络/变量溯源/分支覆盖 → 跨文件关联），新增变量溯源（声明→初始化→校验链）、分支矩阵、设计模式识别。文件检视模式保留单体模板。
#### 特性增强
- 【ops-code-reviewer】条例重叠检测与合并派发：在 ascendc-topk.md 增加 5 条交叉引用指向对应 SEC 条款；主 Agent 阶段1 检测 TOPK 与 SEC
的重叠对，阶段2 合并到同一子 Agent 派发，合并后从原文档顺序移除避免重复检查。
- 【ops-code-reviewer】代码范围隔离：PR
混合侧别时，按条例适用范围（Kernel/Tiling/All）分别指定检视代码范围（op_kernel/op_host/全部），子 Agent 不再看到无关侧别的代码。
- 【ops-code-reviewer】行号校对 PR 模式增强：阶段3 对 PR 检视用 Grep 完整源码定位实际文件行号替换 diff 行号，同时校验行号是否在 diff
变更范围内，越界发现移出报告。
- 【ops-code-reviewer】检视文档列表新增 simt-api-analysis.md（SIMT API C风格化规范 13 条），Kernel 代码不再漏检。
#### 问题修复 Bug Fix
- 【ops-code-reviewer】修复 PR diff 上下文缺失导致的误报：子 Agent 阶段2 改为先读 diff 再读完整源码；阶段5 步骤4 增加 PR
模式交叉验证——判定 FAIL 前 grep 完整源码确认变量上游校验状态，已校验则降级为 PASS。
- 【ops-code-reviewer】修复子 Agent 越界检视：阶段5 步骤4 增加条款边界检查（代码模式不在当前条款适用范围 →
PASS），输出前自检（FAIL/SUSPICIOUS 必须能对应到条款描述或示例代码中的具体模式）。
- 【ops-code-reviewer】修复主 Agent 派发时编造条例标题：增加约束"标题必须从快速索引表原文复制，禁止自行概括"。
- 【ops-code-reviewer】修复重叠组合并后 SEC 条例被重复派发：增加"合并后从原文档位置移除，确保每个条例只派发一次"。
- 【ops-code-reviewer】修复 PR 模式子 Agent 在未变更代码上发起检视发现：阶段2 增加约束"检视意见只报告 diff 变更范围内的问题"。
- 【ops-code-reviewer】修复 SKILL.md 阶段三报告生成与 reviewer 禁止报告矛盾：增加"若调用方禁止生成报告，跳过本阶段"。
#### 性能优化
- 【ops-code-reviewer】主 Agent 阶段2 分组增加禁止枚举约束：分组改为纯位置切分，禁止在 thinking 中逐条列出条例 ID/标题，token 消耗从
2000+ 降至一行概括。
## 测试
- clone_pr_source.py 已在 PR #4356（已合入）和 PR #3258（已合入）两个 GitCode 场景验证成功，merge ref 和 head ref 回退逻辑正常。
- 文件检视全流程回归测试通过：对 flash_attention_block_cube_noquant_mla.h 执行全量检视，22 组子 Agent 分 3 波并行派发，产出报告 61
PASS / 1 FAIL / 2 SUSPICIOUS，结果合理。
## 类型标签
- [x] 🌟 新特性
- [x] ✨ 特性增强
- [x] 🐛 Bug 修复
- [x] ⚡ 性能优化
- [ ] 🧪 测试用例
- [ ] 📦 构建 / CI
- [ ] 📝 文档更新
- [ ] 🔧 配置变更
- [ ] ⬆️ 依赖升级
- [ ] 🔒 安全修复
- [x] ♻️ 代码重构
- [ ] 🧹 代码清理
- [ ] ❓ 其他，请描述：
See merge request: cann/cannbot-skills!199 | 16 天前 |
