Repository navigation
Conversation
This was referenced Oct 13, 2026
Author
|
Superseded by #619, which carries this commit together with the other pstack frontmatter fix as one reviewable PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The skills docs say the
namefield is the skill identifier: "Lowercase letters, numbers, and hyphens only. Must match the parent folder name" (cursor.com/docs/context/skills, Frontmatter fields). The Custom Mode example on the same page usesname: tddfor a mode skill too.Two pstack skills carry a display-style name instead:
pstack/skills/poteto-mode/SKILL.md:name: Poteto Modepstack/skills/make-bot-ui/SKILL.md:name: Make Bot UIThe slash command and the mode are addressed by the directory name (
/poteto-mode,/make-bot-ui), so the frontmatter identifier and the thing users type disagree.What
Set
nameto the directory name in both files. No other frontmatter or body changes;disable-model-invocation,mode,iconand descriptions are untouched.node scripts/validate-plugins.mjsis green. Noplugin.jsonbump, following the pattern where pstack bumps land as separate chore PRs (#581, #584).Related
Two more skills in the repo have the same mismatch under
third_party/(x/skills/x-api-mcp-guideisX MCP guide,x-money/skills/x-money-guideisX Money guide); left out of this PR since those are vendor listings. A frontmatter check inscripts/validate-plugins.mjsis piece 3b of #531 and would catch all four.