feat: reshape Codex skills as a first-class product surface#909
Merged
jonnii merged 1 commit intoMay 5, 2026
Merged
Conversation
54d3a4d to
164d8d5
Compare
Codex skills (~/.codex/skills/) shipped today as Claude templates with $ARGUMENTS replaced — keeping model:, AskUserQuestion, !`...` macros, and ~/.claude/skills paths Codex either ignores or treats as noise. Reshape Codex output without touching Claude (which stays byte-identical): - Add codexDescriptions corpus with trigger-phrase routing descriptions for all 17 commands plus the stackit router. - Extend renderCodexSkillContent to strip model:, filter Claude-only entries from allowed-tools, rewrite ~/.claude paths, strip !`...` macros, rewrite cross-skill 'Skill tool' prose to direct instructions, and rewrite AskUserQuestion prose to plain 'ask the user'. - Apply the same frontmatter + body transforms to file-group installs so router SKILL.md, references/, and subagent files all get the treatment. - Add a templates/codex_overrides/ tree with per-file precedence: slim router SKILL.md, three references (workflows, commit-style, stack-plan-recovery), and execution-checklist bodies for stack-create and stack-plan. - Wire format through installFileGroup and installCommandSkills so the override loader is keyed off agentSkillFormatCodex. - Add tests: override precedence, full-tree banned-substring lint, Claude byte-identical guard, and per-transform unit cases.
164d8d5 to
019a362
Compare
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.
Codex skills (~/.codex/skills/) shipped today as Claude templates with
$ARGUMENTS replaced — keeping model:, AskUserQuestion, !
...macros, and~/.claude/skills paths Codex either ignores or treats as noise.
Reshape Codex output without touching Claude (which stays byte-identical):
for all 17 commands plus the stackit router.
from allowed-tools, rewrite ~/.claude paths, strip !
...macros, rewritecross-skill 'Skill tool' prose to direct instructions, and rewrite
AskUserQuestion prose to plain 'ask the user'.
router SKILL.md, references/, and subagent files all get the treatment.
router SKILL.md, three references (workflows, commit-style,
stack-plan-recovery), and execution-checklist bodies for stack-create
and stack-plan.
override loader is keyed off agentSkillFormatCodex.
byte-identical guard, and per-transform unit cases.