Skip to content

Adhere to claude agent skills spec format#9

Open
forefy wants to merge 3 commits intoalt-research:mainfrom
forefy:patch-1
Open

Adhere to claude agent skills spec format#9
forefy wants to merge 3 commits intoalt-research:mainfrom
forefy:patch-1

Conversation

@forefy
Copy link
Copy Markdown

@forefy forefy commented Mar 26, 2026

PR to make this skill better pickable by claude code and similar

Skill.md must have a yaml frontmatter, where name is lowercase, and name needs to match the directory name.

This allows for correct extension of longer skills as when the file line count is over 500 we need to separate instructions to reference files. this is all from the claude docs.

Need this skills format compliance to list the skills in this repo on the Auditor Skill Registry ( https://forefy.com/skills )

Copy link
Copy Markdown

@renlulu renlulu left a comment

Choose a reason for hiding this comment

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

Codex batch review found one spec-compliance issue on this head.

Finding 1

[P2] The skill name now matches its directory, but the frontmatter still uses allowed-tools as a YAML sequence. The Agent Skills spec defines allowed-tools as a space-separated string, so registry/spec consumers expecting that schema can reject or mishandle this skill. Since this PR is specifically about Agent Skills/registry compliance, please keep the touched skill fully spec-shaped, for example allowed-tools: Read Grep Glob Bash Write.

@forefy
Copy link
Copy Markdown
Author

forefy commented Apr 28, 2026

@renlulu see fixes !

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.

2 participants