Repository navigation
fix(models): correct Codestral 2508 context window - #1711
Open
Vinsy Moyya (vinsymoyya) wants to merge 1 commit into
Open
Vinsy Moyya (vinsymoyya) wants to merge 1 commit into
Vinsy Moyya (vinsymoyya) wants to merge 1 commit into
Conversation
|
Vinsy Moyya (@vinsymoyya) is attempting to deploy a commit to the Braintrust Team on Vercel. A member of the Team first needs to authorize it. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #745.
Correct
codestral-2508.max_input_tokensfrom 256,000 to 128,000. Mistral's current Codestral 25.08 model card documents a 128K context window; the existingcodestral-latestentry already uses 128,000.Only this field changes. Pricing, provider availability, parent references, and all other catalog entries are preserved. This follows the
model_list.jsonexception in AGENTS.md.Validation on Node.js 24.22.1, with the frozen lockfile and pnpm 10.33.0 (proxy and root test-tool workspaces):
vitest run packages/proxy/schema/models.test.ts --pool=threads --maxWorkers=1 --no-file-parallelism --reporter=verbose: 26 passed.ModelSchema; the built publicgetAvailableModels()API returns 128,000 forcodestral-2508.70286c232c3ab2df61f5894a0a8ba1044c8712e4confirms every other catalog value is unchanged.git diff --checkpasses; the working tree is clean.The proxy build produced all CJS/ESM bundles and schema declarations, but was interrupted during the remaining package declaration generation. A complete package build or full test-suite pass is not claimed. No model API calls were made.
Hosted Enforce pnpm, lint, and js workflows currently report
action_requiredand need maintainer approval before execution.AI assistance: OpenAI Codex helped research, implement, and validate this correction.