[#984][bugfix][cli] Normalize --yolo to --full-auto for Codex provider#986
Merged
ayazhankadessova merged 4 commits intomainfrom Apr 6, 2026
Merged
[#984][bugfix][cli] Normalize --yolo to --full-auto for Codex provider#986ayazhankadessova merged 4 commits intomainfrom
ayazhankadessova merged 4 commits intomainfrom
Conversation
added 2 commits
April 6, 2026 17:07
- docs/cli/acw.md: Update --yolo mapping note to include Codex --full-auto - docs/cli/lol.md: Update --yolo option description for lol impl - src/cli/acw/providers.md: Document Codex --yolo normalization Related: #984
…cit finalize example - src/cli/acw/providers.sh: Add --yolo → --full-auto normalization in _acw_invoke_codex(), matching Claude's --yolo → --dangerously-skip-permissions pattern. Without --full-auto, Codex runs in read-only sandbox and cannot write .tmp/finalize.txt, causing infinite FSM iteration loops. - python/agentize/workflow/impl/continue-prompt.md: Add explicit shell command example for writing the completion marker file - tests/cli/test-acw-codex-yolo-translation.sh: New test verifying --yolo is translated to --full-auto for Codex provider All shell tests pass. Related: #984
added 2 commits
April 6, 2026 19:26
… template - Restructure continue-prompt.md to make COMPLETION SIGNAL a prominent standalone section instead of buried in a bullet list - Add explicit cat heredoc example showing exact file structure - Explain that the orchestrator checks after every iteration and will loop if the marker is missing - Instruct agent to create finalize file in the SAME iteration as the fix Related: #984
- python/agentize/workflow/impl/kernels.py: Write AGENTS.md with explicit finalize.txt instructions before invoking non-claude impl agents (e.g. Codex). Codex reads AGENTS.md at startup as persistent system instructions, ensuring it sees the completion marker requirement before processing the prompt. Only written once (checked with exists()) and only for non-claude providers. Related: #984
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
2 participants
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
Fix Codex implementation backend completing in 1 iteration instead of looping 4+ times with "completion marker missing".
Root cause: Three compounding issues prevented Codex from writing
.tmp/finalize.txt:--yoloflag was passed to Codex unchanged, but Codex requires--full-autofor workspace write permissionsAGENTS.mdat startup as persistent system instructions, but no such file existedThree-layer fix:
--yolonormalizationproviders.sh--yolo→--full-autofor Codexcontinue-prompt.mdcatheredoc examplekernels.pyMeasured improvement:
Test plan
--yolotranslation test passes (existing)--yolo→--full-autotranslation test passes (new)--mode full --limit 1 --impl-backend codex:gpt-5.2-codexcompletes in 1 iteration withimpl_doneCloses #984
🤖 Generated with Claude Code