Transform @agent- syntax to Task() calls in Codex converter#118
Closed
terry-li-hm wants to merge 1 commit intoEveryInc:mainfrom
Closed
Transform @agent- syntax to Task() calls in Codex converter#118terry-li-hm wants to merge 1 commit intoEveryInc:mainfrom
terry-li-hm wants to merge 1 commit intoEveryInc:mainfrom
Conversation
- Add transformAgentSyntax() function to convert Claude Code @agent- mentions - Converts '@agent-name' to Task(subagent_type='name', ...) for Codex compatibility - Adds instructions for running multiple agents in parallel - Fixes /plan_review and other commands that use @agent- syntax in Codex Closes: Codex commands with @agent- syntax were failing with 'agents not available'
Contributor
Author
|
After deeper investigation, I realized this PR doesn't fully address the underlying issue. The real problem:
Better solution: This connects the already-converted agent skills to the commands that reference them. Closing in favor of #119 which implements this transformation in the Codex converter. |
Contributor
Author
|
Correction: The new PR is #120 (not #119). Also found and fixed an additional bug during testing - the converter was transforming the skill files but not the prompt files. Both now correctly transform Verified locally with Codex CLI 0.89.0 - both skill and prompt files now have correct syntax. |
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.
Summary
Fixes Codex commands that use
@agent-syntax by automatically transforming them toTask()calls during conversion.Problem
Commands like
/plan_reviewuse Claude Code's@agent-dhh-rails-reviewersyntax, which doesn't work in Codex. When these commands are converted via the Codex converter, the syntax is copied as-is, causing errors like "required review agents aren't available in this environment."Solution
Added
transformAgentSyntax()function to the Codex converter that:@agent-<name>patterns in command bodiesExample Transformation
Before (Claude Code syntax):
After (Codex-compatible):
Impact
/plan_reviewand any other commands using@agent-syntaxTesting
Ran the converter and verified
/plan_reviewnow includes proper Task() call instructions in Codex format.