diff --git a/libs/utils/src/error-handler.ts b/libs/utils/src/error-handler.ts index 303a21ce..eddab383 100644 --- a/libs/utils/src/error-handler.ts +++ b/libs/utils/src/error-handler.ts @@ -83,11 +83,6 @@ export function extractRetryAfter(error: unknown): number | undefined { return parseInt(waitMatch[1], 10); } - // Default retry-after for rate limit errors - if (isRateLimitError(error)) { - return 60; // Default to 60 seconds for rate limit errors - } - return undefined; } @@ -103,7 +98,7 @@ export function classifyError(error: unknown): ErrorInfo { const isAuth = isAuthenticationError(message); const isCancellation = isCancellationError(message); const isRateLimit = isRateLimitError(error); - const retryAfter = isRateLimit ? extractRetryAfter(error) : undefined; + const retryAfter = isRateLimit ? (extractRetryAfter(error) ?? 60) : undefined; let type: ErrorType; if (isAuth) { diff --git a/libs/utils/tests/error-handler.test.ts b/libs/utils/tests/error-handler.test.ts index 77f32d67..816cfdbf 100644 --- a/libs/utils/tests/error-handler.test.ts +++ b/libs/utils/tests/error-handler.test.ts @@ -145,9 +145,9 @@ describe('error-handler.ts', () => { expect(extractRetryAfter(error)).toBe(30); }); - it('should return default 60 for rate limit errors without explicit retry-after', () => { + it('should return undefined for rate limit errors without explicit retry-after', () => { const error = new Error('429 rate_limit_error'); - expect(extractRetryAfter(error)).toBe(60); + expect(extractRetryAfter(error)).toBeUndefined(); }); it('should return undefined for non-rate-limit errors', () => {