fix: respect user-configured category model over fallbackChain defaults

When a user configures a custom model for a category (e.g. quick.model),
the hardcoded CATEGORY_MODEL_REQUIREMENTS fallbackChain was overriding it.
This caused the user's model to be ignored and replaced with the default
(e.g. openai/gpt-5.4-mini).

Fix:
- Use userModelOverride directly instead of potentially stale actualModel
- Suppress hardcoded fallbackChain when explicitCategoryModel is provided
- Add regression test verifying user category model takes precedence

Closes #3040
This commit is contained in:
YeonGyu-Kim
2026-04-03 18:27:26 +09:00
parent ed06428ba3
commit 8f449e1627
2 changed files with 35 additions and 3 deletions
+3 -3
View File
@@ -150,12 +150,12 @@ Available categories: ${allCategoryNames}`,
const userModelOverride = explicitCategoryModel ?? overrideModel
if (userModelOverride) {
actualModel = userModelOverride
const parsedModel = parseModelString(actualModel)
const parsedModel = parseModelString(userModelOverride)
const variantToUse = userCategories?.[args.category!]?.variant ?? resolved.config.variant
categoryModel = parsedModel
? applyCategoryParams({ ...parsedModel, variant: variantToUse ?? parsedModel.variant }, resolved.config)
: undefined
modelInfo = { model: actualModel, type: "user-defined", source: "override" }
modelInfo = { model: userModelOverride, type: "user-defined", source: "override" }
}
} else if (resolution) {
const {
@@ -275,6 +275,6 @@ Available categories: ${categoryNames.join(", ")}`,
actualModel,
isUnstableAgent,
// Don't use hardcoded fallback chain when resolution was skipped (cold cache)
fallbackChain: configuredFallbackChain ?? (isModelResolutionSkipped ? undefined : requirement?.fallbackChain),
fallbackChain: configuredFallbackChain ?? ((isModelResolutionSkipped || explicitCategoryModel) ? undefined : requirement?.fallbackChain),
}
}