pstack: fix poteto-mode skill name to satisfy identifier constraint - #241
Open
austinchennn wants to merge 1 commit into
Open
pstack: fix poteto-mode skill name to satisfy identifier constraint#241austinchennn wants to merge 1 commit into
austinchennn wants to merge 1 commit into
Conversation
Cursor's Agent Skills spec requires the frontmatter name field to be lowercase letters, numbers, and hyphens only, and to match the parent folder name. poteto-mode declared name: Poteto Mode, which breaks on other spec-conformant harnesses (skill fails to register silently) and fails the reference conformance validator, even though it loads fine in Cursor. Fixes cursor#237.
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
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.
Summary
Fixes #237.
pstack/skills/poteto-mode/SKILL.mddeclaresname: Poteto Mode, which violates Cursor's own documented Agent Skills frontmatter constraint (name must be lowercase letters/numbers/hyphens only, and must match the parent folder name —poteto-mode). It's the only one of pstack's skills that doesn't conform.This was introduced deliberately in #149 to give the skill a nicer slash-menu label, on the reasoning that routing derives from the folder path so the field is "just" a display label inside Cursor. That holds inside Cursor, but the Agent Skills spec defines
nameas the identifier, not a display label, and other spec-conformant harnesses (verified on Kiro) fail to register the skill at all when it doesn't match — silently, with no error surfaced. It also fails theskills-refconformance validator.Fix
Applying suggested fix option 1 from the issue: set
name: poteto-mode(matches the folder, conformant) and move the human-readable label intometadata, which the documented frontmatter table designates for client-specific extras:Note
Per the issue:
metadata.cursor-display-namefixes the portability break (the identifier is now spec-conformant everywhere), but I don't know whether Cursor's client currently reads that key to render the slash-menu label — if not, the menu entry may showpoteto-modeinstead ofPoteto Modeuntil/unless Cursor adds that lookup (the issue's suggested fix option 2, first-classdisplayNamesupport for skill frontmatter, would need Cursor-side work beyond this PR's scope).Testing
namefield value, so this is a self-contained, non-breaking change..github/workflows/validate-plugins.ymltriggers on (.cursor-plugin/marketplace.json,**/plugin.json,schemas/**), so no CI job runs against this diff; verified the YAML frontmatter parses correctly by hand.Note
Low Risk
Frontmatter-only identifier fix on one skill file; no runtime, auth, or data-handling changes.
Overview
Makes the
poteto-modeskill register on spec-conformant harnesses by settingnametopoteto-mode(matching the folder) instead of the display stringPoteto Mode.The human-readable label moves to
metadata.cursor-display-name. Skill body is unchanged. Cursor may still show the hyphenated identifier in the slash menu until it honors that metadata key.Reviewed by Cursor Bugbot for commit 797d638. Bugbot is set up for automated code reviews on this repo. Configure here.