Skip to content

refactor(skills): move yuanbao to optional-skills - #70456

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-a4be898e
Jul 24, 2026
Merged

refactor(skills): move yuanbao to optional-skills#70456
teknium1 merged 1 commit into
mainfrom
hermes/hermes-a4be898e

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Moves the yuanbao skill from the always-installed bundled set (skills/) to optional-skills/ — it is platform-specific guidance for Tencent Yuanbao group chats, too niche for every default install.

Install via:

hermes skills install official/yuanbao/yuanbao

Changes

  • skills/yuanbao/optional-skills/yuanbao/ (pure git mv, no content changes)
  • Auto-gen docs page moved bundled/optional/ (EN + zh-Hans), source/path metadata flipped to the optional install command
  • skills-catalog.md row removed, optional-skills-catalog.md row added (EN + zh-Hans)
  • website/sidebars.ts: entry moved from the bundled group to the optional group

Validation

Check Result
OptionalSkillSource E2E (isolated HERMES_HOME): scan, fetch("official/yuanbao"), fetch("official/yuanbao/yuanbao"), inspect all resolve to the moved skill
sync_skills stale-manifest cleanup existing installs get the entry cleaned automatically (built-in behavior, no code change needed)
npx docusaurus build SUCCESS (EN + zh-Hans)
Diff scope 8 files, +26/−26, zero Python changes

Infographic

yuanbao-optional-move

The yuanbao skill is platform-specific guidance for Tencent Yuanbao
group chats — niche for the default bundled set. Per the
'when in doubt, optional' rule, ship it as an optional skill.

Install via: hermes skills install official/yuanbao/yuanbao
@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on b6f2175

all good!

@alt-glitch alt-glitch added type/refactor Code restructuring, no behavior change tool/skills Skills system (list, view, manage) P3 Low — cosmetic, nice to have sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades labels Jul 24, 2026
@teknium1
teknium1 merged commit 9a4d1a0 into main Jul 24, 2026
68 of 71 checks passed
@teknium1
teknium1 deleted the hermes/hermes-a4be898e branch July 24, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades tool/skills Skills system (list, view, manage) type/refactor Code restructuring, no behavior change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants