feat(codex): hand off plans to implementation#3341
Conversation
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
There was a problem hiding this comment.
Zero reviews on a T1d-complex PR with a behavioral API contract change: plan mode is now passed through to the cloud API as initial_permission_mode: "plan" for Codex task runs instead of being degraded to read-only. The generated.ts edit adds "plan" to the server enum, but it's unclear whether the backend has deployed support for this value. If not, cloud plan-mode task creation will fail with a server validation error.
|
8de4f0d to
94954c3
Compare
94954c3 to
2bc836d
Compare
a6444d6 to
cbcb0b5
Compare
|
Merging to
After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here |
cbcb0b5 to
f661c53
Compare
|
Could not verify team-posthog-code membership (GitHub API error). Auto-release is limited to Team PostHog Code members, so this will ship with the next scheduled release instead. |
Problem
Codex Plan mode can stream its proposal outside regular assistant-message deltas, so users could see no plan and never receive the prompt to begin implementation. Cloud runs also downgraded Codex Plan to Read only because the backend contract did not yet accept the native mode.
Stacked on #3340. The cloud API contract is added by PostHog/posthog#69947.
Changes