-
Notifications
You must be signed in to change notification settings - Fork 51
chore: create dedicated Claude and Codex skills, remove shared skill #1409
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: tyler-dane <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR successfully migrates from a shared accessibility audit skill to dedicated Claude and Codex skills, reducing verbosity in the codebase root and enabling LLM-specific optimizations in the future.
Key changes:
- Created separate skill files for Claude (
.claude/skills/a11y-audit/SKILL.md) and Codex (.codex/skills/a11y-audit/SKILL.md) with platform-specific frontmatter - Updated all references in
.cursorrules/accessibility.mdto point to the new LLM-specific skill locations - Removed the shared
skills/a11y-audit/SKILL.mdfile
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
.claude/skills/a11y-audit/SKILL.md |
Created Claude-specific skill with minimal frontmatter (name + description) per Claude agent skills spec |
.codex/skills/a11y-audit/SKILL.md |
Created Codex-specific skill with extended frontmatter including version and tags per agentskills.io spec |
.cursorrules/accessibility.md |
Updated all four references to point to both new skill locations |
skills/a11y-audit/SKILL.md |
Removed shared skill file |
Co-authored-by: Copilot <[email protected]>
tyler-dane
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Co-authored-by: tyler-dane <[email protected]>
Both
.claude/skills/and.codex/skills/were pointing to a sharedskills/a11y-audit/SKILL.md, adding unnecessary verbosity to the codebase root and preventing LLM-specific optimization.Changes
Claude skill (
.claude/skills/a11y-audit/SKILL.md)Codex skill (
.codex/skills/a11y-audit/SKILL.md)version: 1.0.0and tagsUpdated references
.cursorrules/accessibility.mdnow points exclusively to.codex/skills/a11y-audit/SKILL.md(since Cursor IDE uses Codex)Removed
skills/a11y-audit/SKILL.mdand empty parent directoryOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.