[codex] Add Python SDK goal turns#26920
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 851a85b369
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,382 @@ | |||
| import queue | |||
There was a problem hiding this comment.
Split goal-turn work into reviewable stages
The AGENTS.md “Change size guidance (800 lines)” rule asks non-mechanical changes to stay under 800 lines, or under 500 lines for complex logic. This diff is about 2.9k lines across app-server/core/extensions and the Python SDK, so review is too broad. Please split the core/app-server goal-turn plumbing from the Python SDK/docs/tests stage so each part can be reviewed independently.
Useful? React with 👍 / 👎.
|
Superseded by #26953, whose head branch is hosted directly in openai/codex. |
Summary
goal=Trueon synchronous and asynchronous Pythonrunandturnturn/startwhile preserving input and turn optionsValidation
just fix -p codex-core -p codex-app-server -p codex-extension-api -p codex-goal-extensionjust write-app-server-schemauv run --frozen --no-sync python scripts/update_sdk_artifacts.py generate-typesjust fmtTests were not run locally; CI will run the added Rust and Python coverage.