Skip to content

feat(skills): Remove "Invoke This Skill When" & improve skill frontmatter descriptions #255

Open
s1gr1d wants to merge 1 commit into
mainfrom
sig/update-frontmatter-description
Open

feat(skills): Remove "Invoke This Skill When" & improve skill frontmatter descriptions #255
s1gr1d wants to merge 1 commit into
mainfrom
sig/update-frontmatter-description

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Jul 6, 2026

Copy link
Copy Markdown
Member

Background: how skills work (progressive disclosure)

Skills load context in three levels

  1. YAML frontmatter: always loaded into the system prompt. Its job is to give the agent just enough to know when a skill applies, without loading the whole file.
  2. SKILL.md body: loaded only once the agent decides the skill is relevant.
  3. Linked files: fetched on demand.

Because the frontmatter description is the only part always in context, it must do the "when to use" work. A separate "Invoke This Skill When" section in the body just duplicated that info at level 2, polluting context without adding value.

@s1gr1d s1gr1d requested review from a team as code owners July 6, 2026 10:02
@s1gr1d s1gr1d requested review from JPeer264, Lms24, andreiborza, chargome, logaretm, mydea and nicohrubec and removed request for a team July 6, 2026 10:02
verify APIs against source code, studies existing skills for patterns, and
produces a full skill bundle (SKILL.md + reference files) with a PR.
Use when asked to "create a skill for <platform>" or "add SDK support for <framework>".
Use when asked to "create a skill for [platform]" or "add SDK support for [framework]".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed because the best practice is to not use XML tags in frontmatter descriptions

@chargome chargome left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks like a reasonable change, just two things:

  • Did you try this locally with some other agent than claude too?
  • Adding content to the description will add overall context if someone is using this full repo - but likely a good tradeoff

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.

2 participants