Repository navigation
fix(speckit-pro): detect Codex phase skills in the .agents/skills directory - #206
Merged
fgabelmannjr merged 2 commits intoJun 17, 2026
Conversation
…ectory Current spec-kit installs the Codex SDD phase skills natively under .agents/skills/<cmd>/SKILL.md (github/spec-kit#1906) instead of the legacy .codex/skills/ path. The autopilot preflight gate only looked in .claude/skills/ and .codex/skills/, so a fresh Codex install reported the phase commands as missing and the autopilot refused to start. Projects were working around this by hand-writing .codex/skills compatibility-marker stubs purely to pass the gate. Teach the gate to also accept .agents/skills/<cmd>/SKILL.md so no shims are needed. Adds a layer-4 test covering the .agents/skills layout. Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
Contributor
Author
|
ヾ(＾ ∇ ＾) Hi yah! Happy to iterate on this refinement. Perhaps I'm just doing something wrong, but this seems to address a tiny nitpick I've been running into, whereby these shims started to exist in my projects: 
And they seem entirely unnecessary. |
Contributor
|
@flyinprogrammer , thanks for the PR, and sorry this Spec Kit versioning change nipped you. I've also created a periodic Codex automation to keep things in line with all upstream Spec Kit version changes. 🙏 |
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.
What this fixes
Current spec-kit installs the Codex SDD phase skills natively under
.agents/skills/<cmd>/SKILL.md(per github/spec-kit#1906) instead of the legacy.codex/skills/path.The autopilot preflight gate (
check-prerequisites.sh, check 0.4) only looked in.claude/skills/and.codex/skills/. So on a fresh Codex install with current spec-kit, the gate reported the core phase commands (speckit-specify,speckit-plan,speckit-tasks,speckit-implement) as missing and the autopilot refused to start — even though the skills were correctly installed under.agents/skills/.The workaround people hit was hand-writing
.codex/skills/<cmd>/SKILL.mdcompatibility-marker stubs whose own body points back at the canonical.agents/skills/copy, purely to satisfy the gate. This PR removes the need for those shims.The change
Teach the gate to also accept
.agents/skills/<cmd>/SKILL.mdas a valid install location, alongside the existing.claude/skills/and.codex/skills/paths. A command counts as installed if it exists as a skill in any of the three project-local layouts. The plugin's own bundledcodex-skills/is still deliberately excluded so the gate can't false-pass.Validation
bash tests/speckit-pro/run-all.sh --layer 4→test-check-prerequisites16/16, including a new test covering the.agents/skills/layout.bash tests/speckit-pro/run-all.sh --layer 1→ 993/993 (regenerateddist/**payloads are in sync).🤖 Generated with Claude Code