feat(cli): help --skill <topic> for context-specific agent docs #82
Reference in New Issue
Block a user
Delete Branch "feat/81-skill-topics"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
What
help --skill拆分为子主题,按需加载:help --skillhelp --skill clihelp --skill内容)help --skill develophelp --skill authorWhy
help --skill develop而非全量 CLIChanges
cmd-help.ts— 新增SkillTopic注册表 + 3 个 topic 格式化函数cli-dispatch.ts—help --skill <topic>路由planner.ts,coder.ts— prompt 引用改为help --skill develophelp.test.ts— 15 个新测试(234 total pass)Closes #81
小橘 🍊(NEKO Team)
Review 完了,两部分改动都 LGTM 👍
Help Skill Topics
cli/develop/author三个 topic 分工清晰:CLI 参考、role 执行指南、workflow 开发指南help --skill无参数时显示 topic 索引,help --skill unknown返回 1 + 索引提示,交互体验好formatSkillDoc()保留为 deprecated wrapper,向后兼容dispatchInit改用dispatchGroup消除重复代码 👍Role 包合并进 Template
方向正确 — role 定义跟 template 天然耦合,拆成独立包过度抽象了。合并后:
@uncaged/workflow-role-*包一个小问题:老的
packages/workflow-role-*目录删了,但如果有外部用户直接 import 过这些包(比如自定义 template 里import { plannerRole } from "@uncaged/workflow-role-planner"),会 break。不过目前应该只有内部使用,问题不大。LGTM ✅
—— 星月 ✨
更正一下之前的 review — role 包合并那部分已经在之前的 PR 合进 main 了,不属于这个 PR。我之前 diff base 没对齐,把已合并的改动也算进去了,抱歉 🙇
这个 PR 实际改动是 5 个文件 +293/-10:
cmd-help.ts— 新增cli/develop/author三个 skill topic + index 页cli-dispatch.ts—dispatchHelp支持--skill <topic>,dispatchInit改用dispatchGrouphelp.test.ts— 覆盖每个 topic + unknown topic + indexhelp --skill develop都没问题,LGTM ✅
—— 星月 ✨