fix(agents): revert per-model thinkingDefault override

This commit is contained in:
Sebastian
2026-02-17 09:15:56 -05:00
parent 5d1bcc76cc
commit 610275f516
3 changed files with 0 additions and 32 deletions

View File

@@ -452,29 +452,10 @@ export function resolveThinkingDefault(params: {
model: string;
catalog?: ModelCatalogEntry[];
}): ThinkLevel {
// 1. Per-model thinkingDefault (highest priority)
// Normalize config keys via parseModelRef (consistent with buildModelAliasIndex,
// buildAllowedModelSet, etc.) so aliases like "anthropic/opus-4.6" resolve correctly.
const configModels = params.cfg.agents?.defaults?.models ?? {};
for (const [rawKey, entry] of Object.entries(configModels)) {
const parsed = parseModelRef(rawKey, params.provider);
if (
parsed &&
parsed.provider === params.provider &&
parsed.model === params.model &&
entry?.thinkingDefault
) {
return entry.thinkingDefault as ThinkLevel;
}
}
// 2. Global thinkingDefault
const configured = params.cfg.agents?.defaults?.thinkingDefault;
if (configured) {
return configured;
}
// 3. Auto-detect from model catalog (reasoning-capable → "low")
const candidate = params.catalog?.find(
(entry) => entry.provider === params.provider && entry.id === params.model,
);

View File

@@ -18,8 +18,6 @@ export type AgentModelEntryConfig = {
params?: Record<string, unknown>;
/** Enable streaming for this model (default: true, false for Ollama to avoid SDK issue #1205). */
streaming?: boolean;
/** Per-model default thinking level (overrides global thinkingDefault). */
thinkingDefault?: "off" | "minimal" | "low" | "medium" | "high" | "xhigh";
};
export type AgentModelListConfig = {

View File

@@ -38,17 +38,6 @@ export const AgentDefaultsSchema = z
params: z.record(z.string(), z.unknown()).optional(),
/** Enable streaming for this model (default: true, false for Ollama to avoid SDK issue #1205). */
streaming: z.boolean().optional(),
/** Per-model default thinking level (overrides global thinkingDefault). */
thinkingDefault: z
.union([
z.literal("off"),
z.literal("minimal"),
z.literal("low"),
z.literal("medium"),
z.literal("high"),
z.literal("xhigh"),
])
.optional(),
})
.strict(),
)