Skip to content

Expose model reasoning metadata for Codex#578

Merged
khaliqgant merged 1 commit intomainfrom
codex-reasoning-metadata
Mar 17, 2026
Merged

Expose model reasoning metadata for Codex#578
khaliqgant merged 1 commit intomainfrom
codex-reasoning-metadata

Conversation

@khaliqgant
Copy link
Member

@khaliqgant khaliqgant commented Mar 17, 2026

Summary

  • add reasoning-effort metadata for Codex models to the shared CLI registry
  • generate config metadata helpers and re-export them from the SDK
  • add SDK coverage for the Codex mini/high and frontier/xhigh defaults

Verification

  • npm run codegen:models:ts
  • npm --prefix packages/sdk run build
  • node --input-type=module -e "import { ModelMetadata, getDefaultReasoningEffort } from './packages/config/dist/index.js'; if (getDefaultReasoningEffort('codex', 'gpt-5.1-codex-mini') !== 'high') throw new Error('mini'); if (ModelMetadata.Codex['gpt-5.4']?.defaultReasoningEffort !== 'xhigh') throw new Error('frontier'); console.log('ok');"

Open with Devin

Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@khaliqgant khaliqgant merged commit c527e0f into main Mar 17, 2026
34 of 35 checks passed
@khaliqgant khaliqgant deleted the codex-reasoning-metadata branch March 17, 2026 10:12
khaliqgant added a commit that referenced this pull request Mar 25, 2026
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