chore(models): drop retired grok-4-1-fast from metadata, tests, and provider docs (#29523) - #32260
Merged
Conversation
xAI retired grok-4-1-fast. hermes_cli/models.py already removed it from the static fallback in an earlier commit, but the context-length metadata, the tests pinning those values, and the provider doc still referenced the retired ID. Clean those up so retired model names stop appearing in user-facing output. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Maps the machine-local commit email (ronhi@buildabear1.localdomain) to the GitHub login RonHillDev so the attribution check passes.
Contributor
🔎 Lint report:
|
alt-glitch
added
type/refactor
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
2 participants
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.
Summary
Drops the retired
grok-4-1-fastmodel fromagent/model_metadata.py'sDEFAULT_CONTEXT_LENGTHS, the corresponding test assertions, and the xAI provider docs. Pure metadata + docs cleanup, no runtime behavior change for any non-retired model.Salvages PR #29523 (@RonHillDev). Closes #29523's original issue scope (retired model still surfacing in user-facing lists, error messages, and docs).
Changes
agent/model_metadata.py— remove"grok-4-1-fast": 2000000fromDEFAULT_CONTEXT_LENGTHS. Thegrok-4-fastentry already covers the live successor at the same 2M context.tests/agent/test_model_metadata.py— remove thegrok-4-1-fastassertion fromtest_grok_models_context_lengthsand thegrok-4-1-fast-reasoning/-non-reasoningcases fromtest_grok_substring_matching.website/docs/integrations/providers.md—/model grok-4-1-fast-reasoning→/model grok-4-fast-reasoningin the xAI Grok section.scripts/release.py(follow-up) — AUTHOR_MAP entry forronhi@buildabear1.localdomain→RonHillDevso the attribution check passes (the original commit was authored from a machine-local email, not a GitHub noreply).What's preserved
agent/model_metadata.py:257-258still referencesgrok-4-1-fast-(non-)reasoningin the "REJECTS effort" list — that's documenting what was tested live on 2026-05-10, not asserting current capability. Untouched.hermes_cli/models.py_XAI_STATIC_FALLBACKis also untouched — fix(xai): drop models being retired May 15, 2026 from pickers #23291 already collapsed it to live models, re-adding entries here would regress the upstream sweep.Validation
Credit
@RonHillDev did the work. Salvage just adds the AUTHOR_MAP entry needed for CI.