Skip to content

fix: 统一中国特色技能的自动触发说明,与 using-superpowers 路由规则保持一致 - #123

Open
jiutinglifeng wants to merge 1 commit into
jnMetaCode:mainfrom
jiutinglifeng:fix/chinese-skills-auto-trigger-description
Open

jiutinglifeng wants to merge 1 commit into
jnMetaCode:mainfrom
jiutinglifeng:fix/chinese-skills-auto-trigger-description

Conversation

@jiutinglifeng

Copy link
Copy Markdown

问题

using-superpowers/SKILL.md 中"中国特色技能路由"一节规定,检测到特定中文场景时必须优先自动调用对应的 chinese-* 系列技能(如"代码审查且团队使用中文沟通" → chinese-code-review)。

但这 4 个技能自身的 description 却写着"仅在用户显式 /xxx 时调用,不要根据上下文自动触发",与路由规则直接矛盾,导致实际触发行为不确定。

修改

统一 chinese-code-reviewchinese-git-workflowchinese-documentationchinese-commit-conventions 这 4 个技能的 description,使其同时覆盖两种触发路径:

  • 匹配 using-superpowers 路由表里对应的具体场景时自动触发
  • 用户显式输入 /技能名 时同样触发
  • 除以上两种情况外不依据上下文泛化触发,保留原有的防误触发边界

未改动

mcp-builder(路由表第五项)本身描述已是开放触发状态,未发现同样矛盾,未做改动。

jnMetaCode added a commit that referenced this pull request Sep 9, 2026
PR #123 指出的矛盾是真的,而且是已发布内容里的 bug:

  using-superpowers/SKILL.md:当检测到以下场景时,**必须**优先调用对应的中国特色技能
  chinese-*/SKILL.md:仅在用户显式 /xxx 时调用,不要根据上下文自动触发

两句话直接打架,模型读到哪句算哪句。

**但没有按 PR 的方向修。** 查这句「手动调用」的来源 392ff75:

  这 4 个本质是参考资料而非工作流,锁成手动调用更合适 —— 旧 description 的
  「卖点描述」句式会在修对之后过度触发污染上下文。

那次改动同时动了 4 个 SKILL.md、GEMINI.md、README 的「调用方式」列,是成体系的
决定。把 description 改回自动触发,等于在零 eval 证据下推翻它,后果很具体:任何
项目只要有中文 README,chinese-documentation 就会在每次写文档时被拉进上下文。

所以改路由表那一侧:改成说明这 4 个只在显式调用时加载,表头从「场景」改成「用户
显式说」,并把 392ff75 的理由和「改动需要 eval」写进注记,免得下次又有人来「修」它。

## 顺带两处

1. 路由表里的 mcp-builder 分类错了 —— 它不是中国特色技能,是本 fork 新增的通用
   技能(上游 15 个 skill 里没有它)。从「中国特色」表里移出。

2. 但直接移出会让它变成孤儿:它的 description 是「MCP 服务器构建方法论 — 系统化
   构建生产级 MCP 工具」这种卖点句式,不含任何触发条件,正是 392ff75 点名要改的
   那种。按仓库自己定的约定补成「在构建 MCP 服务器或 MCP 工具时使用 ——」,原路由
   行承载的触发条件不丢,且不再需要路由行。workflow-runner 的 description 本来就
   自带触发条件,无需处理。

核对过 20 个 skill 没有因此产生孤儿(description 无触发条件且路由未提及)。过程中
我的检测正则三次太窄,把 using-git-worktrees、verification-before-completion、
requesting-code-review 误报成孤儿 —— 逐个对照上游原文确认它们本来就有触发条件,
没有据此改动任何上游内容。

验证:audit 163 pass / 0 warn / 0 fail,verify-release 164 pass / 0 fail,
site build 66 页。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant