diff --git a/src/shared/model-error-classifier.test.ts b/src/shared/model-error-classifier.test.ts index d359c26d3..88ba63dd5 100644 --- a/src/shared/model-error-classifier.test.ts +++ b/src/shared/model-error-classifier.test.ts @@ -82,4 +82,26 @@ describe("model-error-classifier", () => { //#then expect(provider).toBe("provider-x") }) + + test("treats FreeUsageLimitError (PascalCase name) as retryable by name", () => { + //#given + const error = { name: "FreeUsageLimitError" } + + //#when + const result = shouldRetryError(error) + + //#then + expect(result).toBe(true) + }) + + test("treats freeusagelimiterror (lowercase name) as retryable by name", () => { + //#given + const error = { name: "freeusagelimiterror" } + + //#when + const result = shouldRetryError(error) + + //#then + expect(result).toBe(true) + }) }) diff --git a/src/shared/model-error-classifier.ts b/src/shared/model-error-classifier.ts index 2bb7ef176..3c3af4982 100644 --- a/src/shared/model-error-classifier.ts +++ b/src/shared/model-error-classifier.ts @@ -6,13 +6,14 @@ import { readConnectedProvidersCache } from "./connected-providers-cache" * These errors completely halt the action loop and should trigger fallback retry. */ const RETRYABLE_ERROR_NAMES = new Set([ - "ProviderModelNotFoundError", - "RateLimitError", - "QuotaExceededError", - "InsufficientCreditsError", - "ModelUnavailableError", - "ProviderConnectionError", - "AuthenticationError", + "providermodelnotfounderror", + "ratelimiterror", + "quotaexceedederror", + "insufficientcreditserror", + "modelunavailableerror", + "providerconnectionerror", + "authenticationerror", + "freeusagelimiterror", ]) /** @@ -20,13 +21,13 @@ const RETRYABLE_ERROR_NAMES = new Set([ * These errors are typically user-induced or fixable without switching models. */ const NON_RETRYABLE_ERROR_NAMES = new Set([ - "MessageAbortedError", - "PermissionDeniedError", - "ContextLengthError", - "TimeoutError", - "ValidationError", - "SyntaxError", - "UserError", + "messageabortederror", + "permissiondeniederror", + "contextlengtherror", + "timeouterror", + "validationerror", + "syntaxerror", + "usererror", ]) /** @@ -97,12 +98,13 @@ export interface ErrorInfo { export function isRetryableModelError(error: ErrorInfo): boolean { // If we have an error name, check against known lists if (error.name) { + const errorNameLower = error.name.toLowerCase() // Explicit non-retryable takes precedence - if (NON_RETRYABLE_ERROR_NAMES.has(error.name)) { + if (NON_RETRYABLE_ERROR_NAMES.has(errorNameLower)) { return false } // Check if it's a known retryable error - if (RETRYABLE_ERROR_NAMES.has(error.name)) { + if (RETRYABLE_ERROR_NAMES.has(errorNameLower)) { return true } }