Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use Set instead of Array for fallback status codes

Browse Source 
Oxlint rule: should be a Set and use .has() for existence checks
This commit is contained in:
subaochen 2026-03-21 11:33:28 +08:00
parent 775a8169b6
commit 02e23d4468
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/agents/failover-error.ts
View File

@ -337,7 +337,7 @@ export function coerceToFailoverError(
* - Timeouts: 408
* - Not found: 404 (model may have been removed)
*/
const DEFAULT_FALLBACK_STATUS_CODES = [408, 429, 500, 502, 503, 504, 404];
const DEFAULT_FALLBACK_STATUS_CODES = new Set([408, 429, 500, 502, 503, 504, 404]);
/**
* Check if an error should trigger fallback based on the configured error codes.
@ -365,13 +365,13 @@ export function shouldTriggerFallback(
// Determine if status code should trigger fallback
if (fallbackOnErrors === undefined || fallbackOnErrors === "default") {
return DEFAULT_FALLBACK_STATUS_CODES.includes(status);
return DEFAULT_FALLBACK_STATUS_CODES.has(status);
} else if (fallbackOnErrors === "all") {
// "all" means all HTTP errors (4xx and 5xx)
return status >= 400;
} else {
// Custom list of status codes
return fallbackOnErrors.includes(status);
return new Set(fallbackOnErrors).has(status);
}
}