Merge pull request #3868 from MoerAI/fix/openai-server-error-retryable
fix(model-error-classifier): mark OpenAI server_error patterns as retryable (fixes #3799)
This commit is contained in:
@@ -434,6 +434,34 @@ describe("model-error-classifier", () => {
|
||||
//#then
|
||||
expect(result).toBe(true)
|
||||
})
|
||||
|
||||
test("treats OpenAI streaming server_error envelopes as retryable (issue #3799)", () => {
|
||||
//#given: OpenAI surfaces its mid-stream error with type 'server_error'
|
||||
const error = {
|
||||
name: undefined,
|
||||
message: "{\"error\":{\"type\":\"server_error\",\"message\":\"server_error\"}}",
|
||||
}
|
||||
|
||||
//#when
|
||||
const result = shouldRetryError(error)
|
||||
|
||||
//#then
|
||||
expect(result).toBe(true)
|
||||
})
|
||||
|
||||
test("treats the OpenAI prose 'An error occurred while processing' message as retryable (issue #3799)", () => {
|
||||
//#given: the human-readable prose surfaced when OpenAI's stream fails
|
||||
const error = {
|
||||
name: undefined,
|
||||
message: "An error occurred while processing your request. Please try again later.",
|
||||
}
|
||||
|
||||
//#when
|
||||
const result = shouldRetryError(error)
|
||||
|
||||
//#then
|
||||
expect(result).toBe(true)
|
||||
})
|
||||
})
|
||||
|
||||
export {}
|
||||
|
||||
@@ -80,6 +80,11 @@ const RETRYABLE_MESSAGE_PATTERNS = [
|
||||
"请求过于频繁", // "too many requests"
|
||||
"暂时不可用", // "temporarily unavailable"
|
||||
"服务不可用", // "service unavailable"
|
||||
// OpenAI streaming server_error events surface either as a literal "server_error"
|
||||
// type or as the prose error sentence below. Without these patterns subagent
|
||||
// streams stall instead of being retried (issue #3799).
|
||||
"server_error",
|
||||
"an error occurred while processing",
|
||||
]
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user