master

cann-robotfeat: add MC² domain rules for ascendc-code-review

9ed68410创建于 10 天前历史提交

文件	最后提交记录	最后更新时间
references	feat: add MC² domain rules for ascendc-code-review Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !279 merge mc2-domain-rules into master feat: add MC² domain rules for ascendc-code-review Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 #### 新特性 New Features ## 关联的Issue MC2 领域专用检视规则开发需求 #178 ## 测试 UT ## 文档更新 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🌟 新特性 - [ ] ✨ 特性增强 - [ ] 🐛 Bug 修复 - [ ] ⚡ 性能优化 - [ ] 🧪 测试用例 - [ ] 📦 构建 / CI - [ ] 📝 文档更新 - [ ] 🔧 配置变更 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] ♻️ 代码重构 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： ## 流水线故障排查 <!-- 如果下方的流水线检查未通过，可参考以下指引进行自助修复 --> > **UT_Test 未通过？** > 请在下方流水线表格的 **UT_Test** 行，点击右侧的 >>>>> 下载「UT_Test 自检修复提示词模版」。 > 将下载的模版内容粘贴给 Agent，Agent 将根据日志自动定位并修复问题。 > > 示例： >  See merge request: cann/cannbot-skills!279	10 天前
scenarios	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	15 天前
scripts	升级 diff 工具以支持 cannbot 仓的 diff 获取 Co-authored-by: Junren6415<joestranberg99@gmail.com> # message auto-generated for no-merge-commit merge: !165 merge fix_git_diff_tool into master 升级 diff 工具以支持 cannbot 仓的 diff 获取 Created-by: Junren6415 Commit-by: Junren6415 Merged-by: cann-robot Description: diff 工具之前只能 获取 ops 仓的 diff，现在能支持获取 cannbot 仓的 diff，从而支撑AI在本地针对 cannbot 仓的提示词进行检视 See merge request: cann/cannbot-skills!165	15 天前
style	refactor skill directory with a more flatten structure Co-authored-by: jon_jin<jinjiong@hisilicon.com> Co-authored-by: yanhf<yanhaifeng5@huawei.com> # message auto-generated for no-merge-commit merge: !146 merge master-fork into master refactor skill directory with a more flatten structure Created-by: jon_jin Commit-by: yanhf;jon_jin Merged-by: cann-robot Description: ## 描述 将算子目录重构得更扁平 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单的链接。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🌟 新特性 - [ ] ✨ 特性增强 - [ ] 🐛 Bug 修复 - [ ] ⚡ 性能优化 - [ ] 🧪 测试用例 - [ ] 📦 构建 / CI - [ ] 📝 文档更新 - [ ] 🔧 配置变更 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] ♻️ 代码重构 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： See merge request: cann/skills!146	1 个月前
SKILL.md	feat: add MC² domain rules for ascendc-code-review Co-authored-by: libohao6<libohao3@huawei.com> # message auto-generated for no-merge-commit merge: !279 merge mc2-domain-rules into master feat: add MC² domain rules for ascendc-code-review Created-by: libohao6 Commit-by: libohao6 Merged-by: cann-robot Description: ## 描述 #### 新特性 New Features ## 关联的Issue MC2 领域专用检视规则开发需求 #178 ## 测试 UT ## 文档更新 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🌟 新特性 - [ ] ✨ 特性增强 - [ ] 🐛 Bug 修复 - [ ] ⚡ 性能优化 - [ ] 🧪 测试用例 - [ ] 📦 构建 / CI - [ ] 📝 文档更新 - [ ] 🔧 配置变更 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] ♻️ 代码重构 - [ ] 🧹 代码清理 - [ ] ❓ 其他，请描述： ## 流水线故障排查 <!-- 如果下方的流水线检查未通过，可参考以下指引进行自助修复 --> > **UT_Test 未通过？** > 请在下方流水线表格的 **UT_Test** 行，点击右侧的 >>>>> 下载「UT_Test 自检修复提示词模版」。 > 将下载的模版内容粘贴给 Agent，Agent 将根据日志自动定位并修复问题。 > > 示例： >  See merge request: cann/cannbot-skills!279	10 天前