Skip to content

fix: pricing for Daybreak - #576

Merged
kmbroai merged 2 commits into
mainfrom
codex/fix-model-alias-pricing
Aug 20, 2026
Merged

fix: pricing for Daybreak #576
kmbroai merged 2 commits into
mainfrom
codex/fix-model-alias-pricing

Conversation

@ianw-oai

@ianw-oai ianw-oai commented Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

Cost estimates and cost limits are not available when a scan uses gpt-daybreak-blue-latest or gpt-daybreak-red-latest. The price table does not include these model names.

Changes

  • Add separate price-table entries for Blue and Red using the public price list.
  • Remove the special-case model-name rewrite.
  • Keep the requested model name in scan results.
  • Extend the existing tests for token rates, model-name prefixes, cost limits, and live and saved costs.

Testing

  • Focused cost and API tests: 153 passed, 2 skipped.
  • pnpm run test --seed 12345: 1,438 passed, 23 skipped.
  • pnpm run types: passed.
  • pnpm run format: passed.
  • pnpm run test: 1,438 passed, 23 skipped.
  • git diff --check: passed.

Risk and rollout

This changes only the local price table. It does not change the selected model, default model, or scan behavior. The aliases can point to different models in the future. Update their table entries when the published prices change.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@ianw-oai ianw-oai changed the title [codex] Fix cost estimates for Daybreak Blue fix: [codex] use Sol prices for Daybreak Blue Aug 20, 2026
@github-actions github-actions Bot added the bug Something isn't working label Aug 20, 2026
@ianw-oai
ianw-oai marked this pull request as ready for review August 20, 2026 04:24
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review Completed 2026-08-20T04:26:54.363565Z bd3cae1 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@ianw-oai ianw-oai changed the title fix: [codex] use Sol prices for Daybreak Blue fix: [codex] add Daybreak Blue and Red prices Aug 20, 2026
@ianw-oai ianw-oai changed the title fix: [codex] add Daybreak Blue and Red prices fix: pricing for Daybreak Aug 20, 2026
@kmbroai
kmbroai merged commit 73f8559 into main Aug 20, 2026
37 checks passed
@kmbroai
kmbroai deleted the codex/fix-model-alias-pricing branch August 20, 2026 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants