| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix: 修复 sh 脚本退出码规则合规性缺陷 Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !557 merge feat/rpm-deb-depends into master fix: 修复 sh 脚本退出码规则合规性缺陷 Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 修复 PR 557 中测试调度和安装脚本错误处理问题，并补充升级场景的权限回归检查。 ### 变更内容 - scripts/run_tests.sh：msaicerr 组件仅运行 msaicerr_ut，不再隐式调度 upgrade_st 和 uninstall_st。 - test/ut/asys/testcase/common/test_run_tests_parsing.py：新增回归测试，确保 msaicerr UT 不会调度包安装/升级/卸载 ST。 - test/st/upgrade/testcase/test_upgrade_st.py：补充升级前后脚本目录权限、owner/group 保持不变的检查，覆盖可选 bin 目录不存在和存在两种场景。 - .claude/skills/default-skills/scripts/install-default-skills.sh：检查 cp -r 失败并返回非 0，成功路径显式返回 0。 ### 目的 - 避免 msaicerr UT 混入安装、升级、卸载相关用例。 - 防止升级流程引入预期外的权限或属主变化。 - 确保 skill 安装失败不会被误报为成功。 ### 测试 - 新增 msaicerr 测试调度回归用例。 - 升级 ST 覆盖非 root/root 权限模型、owner/group 保持不变，以及可选 bin 目录缺失/存在场景。 - 具体执行结果以流水线报告为准。 ## 文档更新 无 ## 类型标签 - [x] 🐛 Bug 修复 See merge request: cann/oam-tools!557 | 23 小时前 | |
Update Asys/Msaicerr/Msprof Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !28 merge dev_master_syncwithdev into master Update Asys/Msaicerr/Msprof Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 更新Asys/Msacierr/Msprof，与发布的run包功能保持一致 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。例如：关联Issue #000--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/oam-tools/issues/13 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> LLT已通过 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/oam-tools!28 | 6 个月前 | |
fix: build.sh 禁用 rm -rf 并补非空判断，incremental_codecheck.py 声明 utf-8 编码 Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !309 merge fix/build-rm-rf-and-utf8-encoding into master fix: build.sh 禁用 rm -rf 并补非空判断，incremental_codecheck.py 声明 utf-8 编码 Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 1. scripts/incremental_codecheck.py：在 shebang 行后新增 # -*- coding: utf-8 -*- 编码声明，确保文件以 UTF-8 编码被解释器识别。 2. build.sh：封装 safe_rm_dir 统一安全删除函数，覆盖非空判断 + 存在性检查 + chmod -R u+w 恢复写权限后再 rm -r --，禁止 rm -rf。bundle、submodule、build_path、cpack staging、CMakeFiles、compare_dst 所有删除点均改为调用此函数。 3. build.sh：去除 BUILD_PATH 和 BUILD_OUT_PATH 末尾多余的 /，修复 CMake 拼接 protoc 路径时出现双斜杠导致 Error 127 的构建失败。 4. test/ut/asys/testcase/common/test_build_script.py：同步更新 UT，验证 safe_rm_dir 函数定义及 clean_cpack_staging 调用。 ## 变更类型 请选择本次引入的变更类型（勾选对应项）： - [x] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [ ] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [x] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue 无 ## 如何测试 1. 执行 bash build.sh --make_clean 验证清理流程不报错 2. 执行 python3 -m pytest test/ut/asys/ test/ut/msaicerr/ -q 全量 UT 通过（986 passed） ## 核对清单 - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如：feat:, fix:） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md） ## 其他信息 - 封装 safe_rm_dir 函数统一处理所有目录删除，替换原先分散的 chmod + rm -rf 重复逻辑（采纳评审建议） - BUILD_PATH/BUILD_OUT_PATH 去除末尾 /，修复 UT_Test 构建失败（protoc 双斜杠路径） - incremental_codecheck.py 仅新增一行编码声明，不影响运行逻辑 - UT 新增 test_safe_rm_dir_defined 验证函数定义，test_clean_cpack_staging_uses_safe_rm_dir 验证调用 See merge request: cann/oam-tools!309 | 2 个月前 | |
feat: gitcode-pr 创建PR后询问监控方式 & default-skills 不覆盖本仓已有skill Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !299 merge chore/skill-monitor-and-skip-existing into master feat: gitcode-pr 创建PR后询问监控方式 & default-skills 不覆盖本仓已有skill Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 完善 oam-tools 仓内置 agent skill（gitcode-pr / default-skills）与本地工程化能力。把 PR 全生命周期的机械操作固化为确定性脚本，大模型只做需判断力的事，减少反复踩坑与流水线失败轮次。累积改进： 1. **gitcode-pr：创建 PR 后询问监控方式（三选一）**：①不监控 ②本 agent 监控 ③脚本轮询+模型监听结果（推荐，省 token），不再创建后直接持续轮询。 2. **default-skills：本仓已有 skill 则跳过下载、不覆盖本地修改**。安装脚本先检测 .claude/skills/ 是否已存在，已存在跳过 clone，保护本地定制不被公共仓覆盖。 3. **PR 机械操作脚本化**（scripts/pr_ops.py 13 子命令 + poll_pipeline.sh）。固化分页(per_page=100)、流水线终态筛选(排除"触发成功")、引用回复、v4→v5 回退、fork 校验等踩坑逻辑。子命令：resolve-repo / get-state / get-pipeline(--since) / get-codecheck / get-reviews / get-files / reply / review / reply-review / update-pr / trigger / delete-comment / issue-prs。poll_pipeline.sh 后台轮询写 result 文件，配合 Monitor 实现多轮闭环。 4. **评论/评审三类操作明确区分**（实测固化）：reply=独立引用回复（机器人评审无法嵌套，用引用回复）；review=新发起行内评审意见（绑定 diff 文件:行号）；reply-review=回复进别人已有的行内评论线程（实测真人行内评论可线程回复，in_thread 回查确认）。 5. **get-codecheck / update-pr**：get-codecheck 拉 obs 公开 CI 结论(codecheck/precommit pass + commit_id)；update-pr 多轮修改后 PATCH 同步 PR 描述（本描述即由它更新）。 6. **SKILL.md 强约定**：读写 PR 一律走脚本、删除手写 curl 示例；每次推送后判断是否同步 PR 描述（通用规则）。 7. **本地 pre-commit 增加 Python 检查**：ruff(--fix)/ruff-format、pylint(放宽贴近 codecheck)、bandit(-ll)、codespell(白名单收录 cann 等专有词)。只查改动文件、阻断提交，push 前拦截 codecheck 类问题。 ## 关联的Issue 无（agent skill 与工程化改进）。 ## 测试 - pr_ops.py 各子命令对 #299 实测通过；reply-review 实测 in_thread=true；poll_pipeline.sh 后台跑通写 result。 - pre-commit 各 hook 对新增文件实测能拦真问题（F401/E0606/B602/拼写错），专有词不误判。 - 流水线 codecheck 已 pass（get-codecheck 按 commit_id 确认）。 ## 文档更新 更新 gitcode-pr SKILL.md（监控三选一、脚本化强约定、评论三类区分、get-codecheck/update-pr、pre-commit 与同步描述通用规则）、default-skills SKILL.md/SKILL_en.md、CLAUDE.md（pre-commit 章节）。 ## 类型标签 - [x] 🔧 配置变更 - [x] 📝 文档更新 See merge request: cann/oam-tools!299 | 2 个月前 | |
fix: 修复 sh 脚本退出码规则合规性缺陷 Co-authored-by: sinat_31531339<yuanyue23@huawei.com> # message auto-generated for no-merge-commit merge: !557 merge feat/rpm-deb-depends into master fix: 修复 sh 脚本退出码规则合规性缺陷 Created-by: sinat_31531339 Commit-by: sinat_31531339 Merged-by: cann-robot Description: ## 描述 修复 PR 557 中测试调度和安装脚本错误处理问题，并补充升级场景的权限回归检查。 ### 变更内容 - scripts/run_tests.sh：msaicerr 组件仅运行 msaicerr_ut，不再隐式调度 upgrade_st 和 uninstall_st。 - test/ut/asys/testcase/common/test_run_tests_parsing.py：新增回归测试，确保 msaicerr UT 不会调度包安装/升级/卸载 ST。 - test/st/upgrade/testcase/test_upgrade_st.py：补充升级前后脚本目录权限、owner/group 保持不变的检查，覆盖可选 bin 目录不存在和存在两种场景。 - .claude/skills/default-skills/scripts/install-default-skills.sh：检查 cp -r 失败并返回非 0，成功路径显式返回 0。 ### 目的 - 避免 msaicerr UT 混入安装、升级、卸载相关用例。 - 防止升级流程引入预期外的权限或属主变化。 - 确保 skill 安装失败不会被误报为成功。 ### 测试 - 新增 msaicerr 测试调度回归用例。 - 升级 ST 覆盖非 root/root 权限模型、owner/group 保持不变，以及可选 bin 目录缺失/存在场景。 - 具体执行结果以流水线报告为准。 ## 文档更新 无 ## 类型标签 - [x] 🐛 Bug 修复 See merge request: cann/oam-tools!557 | 23 小时前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 23 小时前 | ||
| 6 个月前 | ||
| 2 个月前 | ||
| 2 个月前 | ||
| 23 小时前 |