fix(codex): accept Codex 0.150 account plans - #8447
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a small, backward-compatible Codex 0.150 compatibility fix that accepts new account plan values and maps them to existing labels. Its runtime effect is limited to preventing valid authenticated provider probes from failing schema decoding, with focused regression coverage. You can add or adjust custom eligibility rules. Learn more. |
Codex 0.150.1 can return new account plan types such as
self_serve_business_prolite. The provider probe rejected thoseaccount/readresponses during payload decoding, which made an installed and authenticated Codex provider appear broken.Extend the existing Codex 0.150 compatibility schemas with the complete current plan type set, regenerate the affected schemas, and map the new Business, Enterprise, and Edu variants to their existing subscription labels. Add regression coverage for decoding each new plan type.
Tests:
vp test run packages/effect-codex-app-server/src/schema.test.tsvp test run apps/server/src/provider/Layers/CodexProvider.test.tsvp run --filter effect-codex-app-server typecheckvp run --filter t3 typecheckModel: gpt-5.6-sol. Harness: pi (openai-codex).
Note
Accept Codex 0.150 account plan types in
CodexProviderandPlanTypeschemaself_serve_business_prolite,self_serve_business_usage_based,business,ent26,enterprise_cbp_automation,edu_plus,edu_pro) to thePlanTypeenum in the generated Codex 0.150 schema and its generator scriptcodexAccountAuthLabelin CodexProvider.ts to map these new plan types to the correct subscription labels (Business, Enterprise, Edu)\n- Adds a test verifyingV2GetAccountResponseaccepts the new plan typesundefinedMacroscope summarized 2821f32.