chore: create dedicated Claude and Codex skills, remove shared skill - #1409
Merged
Conversation
Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Setup dedicated skills for Claude and Codex
Create dedicated Claude and Codex skills, remove shared skill
Jan 3, 2026
tyler-dane
marked this pull request as ready for review
January 3, 2026 21:55
Contributor
There was a problem hiding this comment.
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 <175728472+Copilot@users.noreply.github.com>
Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
tyler-dane
approved these changes
Jan 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.