Merged
refactor(ops-direct-invoke): 以 Skill 承接主会话 PM #160
refactor(ops-direct-invoke): 以 Skill 承接主会话 PM #160
Merged
xutianzeAdd on 23 days ago
xutianze
xutianzeCollaborators
23 days ago

描述

主会话通过 ops-direct-invoke Skill 承接算子任务,负责需求对齐、流程编排与交付组织。子 Agent 按客户端角色配置和 harness 派发的 task 执行与验收。

  • PM 从用户原始需求开始,首次集中确认 conda/Python、CANN 与修改权限,代答四项启动输入,按整个任务沿用有效授权。
  • 每轮根据目标与有效产物准备 YAML;同轮中断恢复使用原 YAML 和运行目录。
  • 节点按自身角色与任务使用模板、脚本和知识 Skill,保持职责与权限边界。
  • 安装器同步 Skill、角色和配置记录,清理 AGENTS.md / CLAUDE.md 中本插件的完整管理区块,保留用户内容、其他插件区块和无完整标记的自定义内容;重复安装幂等。
  • README、QUICKSTART、入口 Skill 和 maintain 资料直接描述当前职责、调用关系与操作方法。

改动范围为 PM 入口、安装器、维护资料与测试。harness core、调度 Skill、三个子 Agent 角色和依赖 gitlink 保持不变;用户正在运行的算子工作区保持不变。

关联的Issue

无。后续重构,基于已合入的 PR #159。

测试

  • 按 ops-direct-invoke-maintain 完整运行 test/ut/test_all.py:6 类、61 项通过,本次文案整理后已完整重跑。
  • 两份修改后的 Skill 通过 skill-creator quick_validate.py;文档本地链接与 git diff --check 通过。
  • script/test/harness.test.js:36 项全部通过。覆盖源码/打包安装、五种客户端配置更新、两种项目指引的管理区块清理、重复安装、用户与其他插件内容保留、不完整标记保护。固定子图 template 字段的测试断言验证:组装输出应剥离该字段且子图文件实际落盘,未改组装器或 harness。
  • script/test/install.test.js 中指引重装与多插件共存两项通过,确保仍有 AGENTS.md 的其他插件保留原安装行为。
  • 实现阶段已完成临时工作区真实 OpenCode 只读冒烟(本次文案整理未重跑):主会话实际加载入口并确认 PM 身份与首次环境/权限问题;开发者角色读取执行记录模板后仍为 executor,缺输入时报告阻塞,不进行需求确认或派发。主会话初次加载后的响应超时,在同一会话缩小为身份确认后完成。未运行真实算子/NPU 开发,不将身份冒烟等同于端到端业务验收。
  • 全仓 npm test 在 pretest 打包阶段受既有外部依赖缺失阻塞:社区插件 ascendc-port-orchestrator 引用 ops/ascendc-cross-gen-port-light,但公共仓缺少该目录。使用公共仓 master 393564ae43f304426db2c206e361cd43b2811413 亦复现;未顺带改动该插件。上述相关安装测试使用该依赖版本构建 ops-direct-invoke 与 model-infer-optimize 后执行,测试后已恢复原 gitlink。

文档更新

更新 ops-direct-invoke 的 README、QUICKSTART、maintain Skill 及相关维护资料。QUICKSTART 仅提供首次安装、启动、任务示例和结果位置,README 的交叉链接同步。

类型标签

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for xutianze's contribution.)
xutianzexutianzeCollaborators
23 days ago Created pull request, commit cd50b6cd
CANN-robotCANN-robotCollaborators
23 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
23 days ago Comments:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cannbot ✅ dengguojie, 宋恺 (2/2) ✅ dengguojie (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

xutianze, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
xutianze
xutianzeCollaborators
23 days ago Comments:

compile

likedislike
xutianzexutianzeCollaborators
23 days ago Pre merge succeeded (commit_id: d5ba5f36574077dd098e46705dc33313d5d64571)
CANN-robot
CANN-robotCollaborators
23 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
23 days ago Added label: ci-pipeline-running
6 event messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
23 days ago Removed label: ci-pipeline-passed
CANN-robot
CANN-robotCollaborators
23 days ago Comments:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robotCollaborators
23 days ago Added label: cann-cla/yes
xutianzexutianzeCollaborators
23 days ago Change title to "refactor(ops-direct-invoke): 以 Skill 承接主会话 PM", originally titled "refactor(ops-direct-invoke): 将 PM 入口迁移到 Skill".
xutianzexutianzeCollaborators
23 days ago Modify description of pull request
xutianze
xutianzeCollaborators
23 days ago Comments:

compile

likedislike
xutianzexutianzeCollaborators
23 days ago Pre merge succeeded (commit_id: 7e973f9f9b64557301ac591ce6b4309ae49af890)
CANN-robot
CANN-robotCollaborators
23 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
23 days ago Added label: ci-pipeline-running
7 event messages are collapsed here。 See more
xutianzexutianzeCollaborators
23 days ago Modify description of pull request
xutianze
xutianzeCollaborators
23 days ago Comments:

compile

likedislike
xutianzexutianzeCollaborators
23 days ago Pre merge succeeded (commit_id: c5f082e8acf281e91967b1dac0363ad7e0a6b40c)
CANN-robot
CANN-robotCollaborators
23 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
23 days ago Removed label: ci-pipeline-passed
CANN-robotCANN-robotCollaborators
23 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
23 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
23 days ago Added label: ci-pipeline-passed
songkai111Collaborators
23 days ago Comments:

/lgtm

likedislike
sundayCollaborators
23 days ago Comments:

/lgtm
/approve

likedislike
dengguojieCollaborators
23 days ago Comments:

/approve
/lgtm

likedislike
CANN-robot
CANN-robotCollaborators
23 days ago Comments:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 周奇龙
/lgtm: 周奇龙

likedislike
CANN-robotCANN-robotCollaborators
23 days ago Added label: lgtmapproved
CANN-robotCANN-robotCollaborators
23 days ago Merged pull request