From 0d80f05e9e22e21dacfddbcf7f3f6b456d1003ec Mon Sep 17 00:00:00 2001 From: KooshaPari Date: Fri, 26 Jun 2026 05:02:40 -0700 Subject: [PATCH] chore(resilience): keep lockout files under size ratchet --- open-sse/services/accountFallback.ts | 2 -- src/sse/services/auth.ts | 10 ---------- 2 files changed, 12 deletions(-) diff --git a/open-sse/services/accountFallback.ts b/open-sse/services/accountFallback.ts index 3efe1f07fc..1263b31a4b 100644 --- a/open-sse/services/accountFallback.ts +++ b/open-sse/services/accountFallback.ts @@ -325,8 +325,6 @@ function buildProviderProfile( return { baseCooldownMs: connectionCooldown.baseCooldownMs, useUpstreamRetryHints: connectionCooldown.useUpstreamRetryHints, - // Issue #2100 follow-up: propagate stored override (boolean | undefined) - // so the runtime resolver picks user setting first, then per-provider default. useUpstream429BreakerHints: connectionCooldown.useUpstream429BreakerHints, maxCooldownMs: resolveModelLockoutSettings(settings).maxCooldownMs, maxBackoffSteps: connectionCooldown.maxBackoffSteps, diff --git a/src/sse/services/auth.ts b/src/sse/services/auth.ts index b14637d510..9ca5f83b74 100644 --- a/src/sse/services/auth.ts +++ b/src/sse/services/auth.ts @@ -1999,16 +1999,6 @@ export async function markAccountUnavailable( const cooldownMs = terminalStatus ? 0 : rawCooldownMs; // ── #3027: per-model subscription/permission 403 → model-only lockout ── - // Passthrough / per-model-quota providers (e.g. ollama-cloud with - // passthroughModels:true) multiplex many upstream models behind one key. - // A scoped 403 like "this model requires a subscription, upgrade for access" - // is about the paid model, not the key — cooling the whole connection would - // knock out the free models on the same key too and escalate backoff - // (#3001/#3027). This generalizes the grok-web 403 precedent above to every - // hasPerModelQuota provider. Terminal/credential 403s (banned/deactivated - // key, credits exhausted) are excluded here because - // resolveTerminalConnectionStatus() returns a non-null status for them, so - // they keep their existing connection-level cooldown/deactivation path. if (isPerModelQuotaProvider && status === 403 && provider && model && !terminalStatus) { const lockout = recordModelLockoutFailure( provider,