mirror of
https://github.com/DayuanJiang/next-ai-draw-io.git
synced 2026-09-01 17:10:24 +08:00
* fix: raise the output budget so reasoning models reach the tool call A reasoning model spends the output budget in order: thinking first, then prose, then the tool call. With 16000 the thinking alone can consume all of it, so the turn ends with finishReason "length" before display_diagram is ever called. The canvas stays empty and nothing surfaces in the UI, because no tool call means no tool error, and the client never reads finishReason. Measured on openrouter deepseek/deepseek-v4-flash, the model from the report: - max_tokens=800 with reasoning on returns reasoning_tokens=800, empty content, finish_reason length. So reasoning is billed against this budget, not exempt. - refining an existing diagram (19k chars of XML in the input) produced 49142 chars of reasoning, zero tool calls, finishReason "length" at 16000 - the same request at 40000 finished and called edit_diagram with 12 operations 64000 cannot just be sent to every model: bedrock claude-3-haiku caps at 4096, nova-lite at 10000, and the openrouter deepseek-r1 endpoint counts input and output against one 64000 ceiling. All three name the real limit in the 400, so parse it and retry once. Verified: nova-lite logs "64000 rejected, retrying with 10000" and then completes its tool call. Also expose the budget in Settings. It is sent as a header rather than read from env only, so desktop users can raise it themselves without an env file. vercel.json goes back to the 300s it had before #238 traded it for $2-4/month. That is now Vercel's own default, and billing pauses while the function waits on the model, so the saving that motivated 120s no longer applies. edgeone.json is left alone: its 120 may be that platform's actual ceiling. * fix: only reinterpret an error as a budget rejection when it says so Review of the first commit found the retry could fire on errors that have nothing to do with the budget, which would replace a readable provider error with a truncated response: exactly the symptom this PR exists to remove. - Drop the generic "lower than N" pattern. For the Bedrock message it was dead code, since "model limit of N" matches first with the same number. Left live, it would read a number out of any message shaped like "must be lower than 2". - Skip errors whose status is not 400 or 422, so auth and rate-limit failures are never reinterpreted. - Require the parsed ceiling to be at least 1024. Below that a diagram cannot come out whole, so retrying would hide the error behind broken XML. - Validate MAX_OUTPUT_TOKENS from env the same way as the header, so a stray "-1" falls back instead of reaching the provider. Adds tests for the retry wrapper itself, which had none: it retries once with the named ceiling, leaves a 401 alone, does not retry when the ceiling is not smaller, propagates a second rejection, and preserves the other call options. Re-verified against the live APIs: bedrock nova-lite still logs "64000 rejected, retrying with 10000" and completes its tool call, and deepseek-v4-flash still finishes normally at 64000.
146 lines
5.2 KiB
TypeScript
146 lines
5.2 KiB
TypeScript
import { wrapLanguageModel } from "ai"
|
|
|
|
type WrappedModel = ReturnType<typeof wrapLanguageModel>
|
|
|
|
/**
|
|
* Default output budget for a chat turn.
|
|
*
|
|
* This has to cover thinking + prose + the tool call, because reasoning models
|
|
* spend it in that order. Measured on deepseek-v4-flash: refining an existing
|
|
* diagram burned 16000 tokens on thinking alone and the request ended with
|
|
* finishReason "length" before display_diagram was ever called (issue #924).
|
|
* 64000 leaves room for the plan and the XML in one turn.
|
|
*/
|
|
export const DEFAULT_MAX_OUTPUT_TOKENS = 64000
|
|
|
|
/** Ceiling for the user-supplied override, to catch typos like an extra zero. */
|
|
export const MAX_OUTPUT_TOKENS_LIMIT = 200000
|
|
|
|
/**
|
|
* Below this a diagram cannot come out whole, so a retry would just produce
|
|
* truncated XML instead of the provider's error. Better to surface the error.
|
|
*/
|
|
const MIN_USABLE_OUTPUT_TOKENS = 1024
|
|
|
|
/** Status codes that can carry a complaint about the requested budget. */
|
|
const BUDGET_REJECTION_STATUSES = new Set([400, 422])
|
|
|
|
function usableLimit(value: number): number | null {
|
|
return value >= MIN_USABLE_OUTPUT_TOKENS ? value : null
|
|
}
|
|
|
|
/**
|
|
* A budget this large exceeds what some models accept. Providers reject it with a
|
|
* 400 that names the real limit, so we parse the number out and retry once
|
|
* instead of failing the turn.
|
|
*
|
|
* Formats seen in the wild:
|
|
* - Bedrock: "The maximum tokens you requested exceeds the model limit of 4096."
|
|
* - OpenRouter: "This endpoint's maximum context length is 64000 tokens. However,
|
|
* you requested about 64025 tokens (25 of text input, 64000 in the output)."
|
|
* Note this one is an input+output ceiling, so the input has to be subtracted.
|
|
* - Anthropic: "max_tokens: 200000 > 64000, which is the maximum allowed..."
|
|
* - OpenAI: "This model supports at most 16384 completion tokens"
|
|
*
|
|
* Every pattern names tokens explicitly. A generic one (an earlier draft matched
|
|
* "lower than N") would reinterpret unrelated failures, and retrying on a bogus
|
|
* number turns a readable error into an empty diagram.
|
|
*/
|
|
export function parseOutputTokenLimit(error: unknown): number | null {
|
|
const err = error as {
|
|
message?: unknown
|
|
responseBody?: unknown
|
|
statusCode?: unknown
|
|
}
|
|
|
|
// An auth or rate-limit failure is not about the budget, so leave it alone.
|
|
if (
|
|
typeof err?.statusCode === "number" &&
|
|
!BUDGET_REJECTION_STATUSES.has(err.statusCode)
|
|
) {
|
|
return null
|
|
}
|
|
|
|
const text = [
|
|
typeof err?.message === "string" ? err.message : "",
|
|
typeof err?.responseBody === "string" ? err.responseBody : "",
|
|
].join(" ")
|
|
|
|
if (!text) return null
|
|
|
|
// Combined input+output ceiling: subtract the input the provider counted,
|
|
// plus a small margin because its estimate is approximate.
|
|
const context = text.match(/maximum context length is (\d+)/i)
|
|
if (context) {
|
|
const input = text.match(/(\d+) of text input/i)
|
|
return usableLimit(
|
|
Number(context[1]) - (input ? Number(input[1]) : 0) - 1024,
|
|
)
|
|
}
|
|
|
|
const output =
|
|
text.match(/model limit of (\d+)/i) ||
|
|
text.match(/> (\d+), which is the maximum/i) ||
|
|
text.match(/at most (\d+) completion tokens/i)
|
|
|
|
return output ? usableLimit(Number(output[1])) : null
|
|
}
|
|
|
|
/**
|
|
* Retry the stream once with a smaller budget when the provider rejects the
|
|
* requested one. Without this, raising the default breaks every model whose
|
|
* ceiling is below it (measured: bedrock claude-3-haiku 4096, nova-lite 10000,
|
|
* openrouter deepseek-r1 64000 shared with the input).
|
|
*/
|
|
export function withOutputTokenLimitFallback(
|
|
model: WrappedModel,
|
|
): WrappedModel {
|
|
return wrapLanguageModel({
|
|
model,
|
|
middleware: {
|
|
specificationVersion: "v3",
|
|
async wrapStream({ doStream, params, model: inner }) {
|
|
try {
|
|
return await doStream()
|
|
} catch (error) {
|
|
const limit = parseOutputTokenLimit(error)
|
|
const requested = params.maxOutputTokens
|
|
|
|
if (!limit || !requested || limit >= requested) throw error
|
|
|
|
console.warn(
|
|
`[maxOutputTokens] ${requested} rejected, retrying with ${limit}`,
|
|
)
|
|
return await inner.doStream({
|
|
...params,
|
|
maxOutputTokens: limit,
|
|
})
|
|
}
|
|
},
|
|
},
|
|
})
|
|
}
|
|
|
|
function validBudget(value: string | null | undefined): number | null {
|
|
const parsed = Number(value)
|
|
return Number.isInteger(parsed) &&
|
|
parsed > 0 &&
|
|
parsed <= MAX_OUTPUT_TOKENS_LIMIT
|
|
? parsed
|
|
: null
|
|
}
|
|
|
|
/**
|
|
* Resolve the output budget: user setting (sent as a header so it works in the
|
|
* desktop app too), then server env, then the default. Both sources go through
|
|
* the same validation, so a typo in either falls back instead of reaching the
|
|
* provider.
|
|
*/
|
|
export function resolveMaxOutputTokens(headerValue: string | null): number {
|
|
return (
|
|
validBudget(headerValue) ??
|
|
validBudget(process.env.MAX_OUTPUT_TOKENS) ??
|
|
DEFAULT_MAX_OUTPUT_TOKENS
|
|
)
|
|
}
|