SMOODEV-groq-deprecation: rename removed Groq Llama aliases to gpt-oss#115
Merged
Conversation
The smooai litellm gateway (PR #2097) deleted the deprecated Groq aliases `groq-llama-3.3-70b` and `groq-llama-3.1-8b`. The smooth repo's `smooth-judge` / `smooth-fast` slots routed to those names, so they would 404 once the gateway redeploys. - smooth-policy/smooth_alias.rs: judge slot -> groq-gpt-oss-120b, fast slot -> groq-gpt-oss-20b (concrete_default + doc table + tests). Also add a second migration step (migrate_deprecated_concrete) so a config ALREADY migrated onto the literal dead Groq names — which carry no `smooth-` prefix and so never re-hit the slot mapping — gets bumped to the live gpt-oss alias instead of 404ing. - smooth-cast/provider_migration.rs: update slot/rewrite tests; add a test proving an already-migrated registry pinned to groq-llama-3.3-70b / groq-llama-3.1-8b is bumped to gpt-oss (and is idempotent). - smooth-code/model_picker.rs: rename the offline catalog entries and fallback/post-migration test expectations. Historical CHANGELOG.md entries left intact (they describe a past release). No groq-llama-4 / groq-kimi / qwen3-32b references exist. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AxL4LdeChNivUCZM6No2NJ
|
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.
Problem
The smooai litellm gateway (PR #2097, merged) deleted the deprecated Groq aliases
groq-llama-3.3-70bandgroq-llama-3.1-8b. The smooth repo'ssmooth-judge/smooth-fastsemantic slots still routed to those names, so they would return HTTP 400 / 404 once the gateway redeploys.The rename
groq-llama-3.3-70b→groq-gpt-oss-120b(judge / reasoning slot)groq-llama-3.1-8b→groq-gpt-oss-20b(fast slot)No
groq-llama-4*,groq-kimi*, orqwen3-32breferences exist anywhere in the repo (verified).Changes
smooth-policy/src/smooth_alias.rs(source of truth):concrete_default()for Judge/Fast → gpt-oss; doc mapping table + tests updated. Added a second migration stepmigrate_deprecated_concrete(): a config that already ran thesmooth-*→ concrete migration is pinned to the literal dead Groq name (nosmooth-prefix left to re-trigger the slot map), so it would otherwise stay on a dead model. The step rewrites the dead concrete names → gpt-oss. This flows through every consumer (migrate_aliasis the shared entry point) — no new mechanism invented.smooth-cast/src/provider_migration.rs: slot/rewrite test expectations updated; new testmigrate_bumps_already_migrated_groq_llama_to_gpt_ossproving an already-migrated registry gets bumped + idempotent.smooth-code/src/model_picker.rs: offline catalog entries + fallback/post-migration test expectations renamed.Historical
CHANGELOG.mdentries left intact (they describe a past release).Validation
cargo check/cargo fmt --checkclean for all three crates.cargo test -p smooai-smooth-policy -p smooai-smooth-cast -p smooai-smooth-code— all green (incl. 2 new tests).smooth-pearlsclippysort_by_keyerror is onmaintoo (not touched here).Do NOT merge/deploy this before coordinating. smooth deploy must precede the litellm gateway redeploy that removes the old aliases — otherwise in-flight configs still on
smooth-judge/smooth-fastbriefly map to dead names. Main session reviews + coordinates.🤖 Generated with Claude Code
https://claude.ai/code/session_01AxL4LdeChNivUCZM6No2NJ