Skip to content

fix: correct invalid skill name fields to kebab-case#6

Merged
sjnims merged 1 commit intomainfrom
fix/1-correct-skill-name-fields
Dec 7, 2025
Merged

fix: correct invalid skill name fields to kebab-case#6
sjnims merged 1 commit intomainfrom
fix/1-correct-skill-name-fields

Conversation

@sjnims
Copy link
Owner

@sjnims sjnims commented Dec 7, 2025

Summary

  • Fix 6 skill SKILL.md files with invalid name fields that used Title Case instead of kebab-case
  • Update names to match directory names exactly per official Claude Code skills documentation

Problem

Fixes #1

Per the official Claude Code skills documentation:

Name: Must use lowercase letters, numbers, and hyphens only (max 64 characters)

Six skills violated this specification with Title Case names.

Solution

Changed the name field in each affected SKILL.md to kebab-case:

Skill Directory Before After
command-development Command Development command-development
mcp-integration MCP Integration mcp-integration
plugin-structure Plugin Structure plugin-structure
skill-development Skill Development skill-development
plugin-settings Plugin Settings plugin-settings
hook-development Hook Development hook-development

Alternatives Considered

None - this is a straightforward spec compliance fix. The correct names are the directory names themselves.

Changes

  • plugins/plugin-dev/skills/command-development/SKILL.md: name field updated
  • plugins/plugin-dev/skills/mcp-integration/SKILL.md: name field updated
  • plugins/plugin-dev/skills/plugin-structure/SKILL.md: name field updated
  • plugins/plugin-dev/skills/skill-development/SKILL.md: name field updated
  • plugins/plugin-dev/skills/plugin-settings/SKILL.md: name field updated
  • plugins/plugin-dev/skills/hook-development/SKILL.md: name field updated

Testing

  • All 6 affected skill name fields updated to kebab-case
  • Names match their directory names exactly
  • No other changes to SKILL.md files
  • Linting passes (markdownlint)

🤖 Generated with Claude Code

Six skill SKILL.md files had invalid name fields using Title Case
instead of kebab-case. Per the official Claude Code skills
documentation, name fields must use lowercase letters, numbers,
and hyphens only.

Changed:
- Command Development → command-development
- MCP Integration → mcp-integration
- Plugin Structure → plugin-structure
- Skill Development → skill-development
- Plugin Settings → plugin-settings
- Hook Development → hook-development

Fixes #1

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Dec 7, 2025

🎉 Thanks for opening your first pull request! We appreciate your contribution to plugin-dev.

Please make sure:

  • You've filled out the PR template completely
  • All markdown is properly linted
  • You've tested the changes locally

A maintainer will review your PR soon. Feel free to ask questions if you need help!

@claude
Copy link

claude bot commented Dec 7, 2025

Claude finished @sjnims's task —— View job


PR Review: fix: correct invalid skill name fields to kebab-case

Summary

Looks good - approve and merge

This is a straightforward compliance fix that correctly addresses a specification violation. All 6 skill name fields have been properly updated from Title Case to kebab-case format.


What's Good

  1. Precise spec compliance: The changes correctly implement the Claude Code skills documentation requirement that skill names "Must use lowercase letters, numbers, and hyphens only (max 64 characters)".

  2. Consistent approach: All 6 affected skills follow the same pattern - the name field now matches the directory name exactly:

    • command-development/name: command-development
    • hook-development/name: hook-development
    • mcp-integration/name: mcp-integration
    • plugin-settings/name: plugin-settings
    • plugin-structure/name: plugin-structure
    • skill-development/name: skill-development
  3. Surgical changes: Each change is a single-line modification to the YAML frontmatter name field - no other content was altered.

  4. Clean markdown: Markdownlint passes with no issues on all 6 changed files.

  5. Well-documented: The PR description clearly explains the problem, references the official spec, and provides a comparison table showing before/after values.


Checklist Verification

  • ✅ All 6 skill name fields updated to kebab-case
  • ✅ Names match directory names exactly
  • ✅ No unintended changes to other content
  • ✅ Markdownlint passes (no linting errors)
  • ✅ Changes align with official Claude Code specification
  • ✅ PR description is clear and complete

Recommendations

None - this PR is ready to merge. The fix is correct, complete, and follows the specification exactly.


@sjnims sjnims self-assigned this Dec 7, 2025
@claude claude bot added bug Something isn't working component:skill Skills layer effort:small < 1 hour labels Dec 7, 2025
@sjnims sjnims merged commit 936be68 into main Dec 7, 2025
8 checks passed
@sjnims sjnims deleted the fix/1-correct-skill-name-fields branch December 7, 2025 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component:skill Skills layer effort:small < 1 hour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: correct invalid skill name fields to kebab-case

1 participant