Skip to content

fix: add missing name frontmatter to 5 slash commands#17

Open
xiaolai wants to merge 1 commit intoChrisWiles:mainfrom
xiaolai:fix/nlpm-add-name-frontmatter
Open

fix: add missing name frontmatter to 5 slash commands#17
xiaolai wants to merge 1 commit intoChrisWiles:mainfrom
xiaolai:fix/nlpm-add-name-frontmatter

Conversation

@xiaolai
Copy link
Copy Markdown

@xiaolai xiaolai commented Apr 22, 2026

Automated audit: This PR was generated by NLPM, a natural language programming linter, running via claude-code-action. Please evaluate the diff on its merits.

Bug

Five slash commands are missing the name frontmatter field:

  • .claude/commands/code-quality.md
  • .claude/commands/docs-sync.md
  • .claude/commands/pr-review.md
  • .claude/commands/pr-summary.md
  • .claude/commands/ticket.md

Claude Code uses the name field to register a command so it can be invoked as /name. Without it, the command cannot be discovered or called by name — the slash command simply doesn't work.

Fix

Added name: <command> to each file's frontmatter, using the same value as the filename (the conventional default). For example:

 ---
+name: code-quality
 description: Run code quality checks on a directory
 allowed-tools: Read, Glob, Grep, Bash(npm:*), Bash(npx:*)
 ---

No behavior is changed — this is a pure registration fix.

Commands without a name field cannot be registered or invoked by name.
Adds name: <command> to code-quality, docs-sync, pr-review, pr-summary,
and ticket — derived from each file's existing filename convention.

Co-Authored-By: Claude Code <noreply@anthropic.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