Skip to content

hardening: Codex runtime fixes for 1.0.x - #1

Draft
loong10k wants to merge 4 commits into
feature/1.0.xfrom
hardening/1.0.x
Draft

loong10k wants to merge 4 commits into
feature/1.0.xfrom
hardening/1.0.x

Conversation

@loong10k

Copy link
Copy Markdown
Collaborator

Implementation branch for the approved Codex runtime hardening plan. Work is in progress and follows TDD task-by-task.

Copy link
Copy Markdown
Collaborator Author

Task 2 Ruling: the planned execAndParse force-JSON test is a regression guard rather than a RED test on the current baseline, because defaultOptions() already hardcodes json(true). RED is established by the new jsonOutput=false normal-exec test and noAltScreen interactive-default test. We still keep the execAndParse test so the refactor cannot accidentally make it inherit jsonOutput=false. Cost if wrong: parsed JSONL could silently stop working when users disable normal JSON output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant