fix: move Haiku check before variant check for reliability
Addresses cubic-dev-ai review feedback. The Haiku early-return now runs immediately after model validation, ensuring it's always reached regardless of any upstream variant normalization.
This commit is contained in:
@@ -62,11 +62,11 @@ export function createAnthropicEffortHook() {
|
|||||||
): Promise<void> => {
|
): Promise<void> => {
|
||||||
const { agent, model, message } = input
|
const { agent, model, message } = input
|
||||||
if (!model?.modelID || !model?.providerID) return
|
if (!model?.modelID || !model?.providerID) return
|
||||||
|
if (isEffortUnsupportedModel(model.modelID)) return
|
||||||
if (message.variant !== "max") return
|
if (message.variant !== "max") return
|
||||||
if (!isClaudeProvider(model.providerID, model.modelID)) return
|
if (!isClaudeProvider(model.providerID, model.modelID)) return
|
||||||
if (shouldSkipForInternalAgent(agent?.name)) return
|
if (shouldSkipForInternalAgent(agent?.name)) return
|
||||||
if (output.options.effort !== undefined) return
|
if (output.options.effort !== undefined) return
|
||||||
if (isEffortUnsupportedModel(model.modelID)) return
|
|
||||||
|
|
||||||
const opus = isOpusModel(model.modelID)
|
const opus = isOpusModel(model.modelID)
|
||||||
const clamped = clampVariant(message.variant, opus)
|
const clamped = clampVariant(message.variant, opus)
|
||||||
|
|||||||
Reference in New Issue
Block a user