Skip to content

pstack: give poteto-mode a human display name - #149

Merged
poteto merged 1 commit into
mainfrom
pstack/poteto-display-name
Jul 11, 2026
Merged

pstack: give poteto-mode a human display name#149
poteto merged 1 commit into
mainfrom
pstack/poteto-display-name

Conversation

@poteto

@poteto poteto commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Sets name: Poteto Mode in poteto-mode's frontmatter. The name field is the display label in the slash menu; skill identity and /poteto-mode resolution derive from the folder path, so routing is unchanged. Unquoted bare string, same pattern as similar display names elsewhere.

Note

Low Risk
Single frontmatter label change with no runtime, routing, or data logic.

Overview
Updates the name field in poteto-mode skill frontmatter from poteto-mode to Poteto Mode so the slash menu shows a readable label.

Skill routing still uses the folder path and /poteto-mode; only the UI display string changes.

Reviewed by Cursor Bugbot for commit 90ac966. Bugbot is set up for automated code reviews on this repo. Configure here.

@poteto
poteto merged commit 9d2a3f2 into main Jul 11, 2026
1 check passed

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Invalid skill identifier name
    • Restored the lowercase kebab-case identifier poteto-mode so it matches the skill directory and validates correctly.

Create PR

Or push these changes by commenting:

@cursor push 6fa9062db2
Preview (6fa9062db2)
diff --git a/pstack/skills/poteto-mode/SKILL.md b/pstack/skills/poteto-mode/SKILL.md
--- a/pstack/skills/poteto-mode/SKILL.md
+++ b/pstack/skills/poteto-mode/SKILL.md
@@ -1,5 +1,5 @@
 ---
-name: Poteto Mode
+name: poteto-mode
 description: poteto's agent style for concise, detailed responses, deliberate subagents, unslopped prose, simple code, and verified work. Use for poteto, /poteto-mode, or requests to work in this style.
 disable-model-invocation: true
 mode: true

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit 90ac966. Configure here.

@@ -1,5 +1,5 @@
---
name: poteto-mode
name: Poteto Mode

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Invalid skill identifier name

Medium Severity · Logic Bug

The name field was changed to Poteto Mode, but this field is an identifier that requires lowercase kebab-case and must match its directory (poteto-mode). Spaces and capitals make this identifier invalid, which can prevent the skill from being discovered, resolved, or loaded correctly. Other skills use displayName for human labels.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 90ac966. Configure here.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Dismissing: frontmatter name is a display label, not the resolution identifier. The client computes slash-command identity from the SKILL.md's parent directory (getSkillDisplayName in slashMenuData.ts) and uses frontmatter name only as displayName = configuredName || commandName — plugin-loaded skills (rule.plugin path) flow through the same code. There is no displayName frontmatter field in that parser. Empirically: skills with spaced/capitalized names ("Fredrika Mode", "Lauren Mode") have shipped for weeks and their slash commands resolve normally.

JMak-Security added a commit to JMak-Security/plugins that referenced this pull request Aug 21, 2026
"name: Poteto Mode" has capitals and a space, violating the documented
identifier constraint (lowercase letters, numbers, hyphens only; must
match the parent folder) and the agentskills.io spec's identical rule.
It is the only one of pstack's 44 skills that does not conform, and it
fails to register on other spec-compliant harnesses (verified on Kiro).

Sets "name: poteto-mode" to match the folder and the documented pattern,
and moves the capitalized label into "metadata.cursor-display-name",
the field the frontmatter spec designates for client-specific display
extras, so the slash-menu label PR cursor#149 introduced is preserved.

Fixes cursor#237

Signed-off-by: Jason Mak <squrrielbro@gmail.com>
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