fix: preserve server retry policy fidelity

Author: ndycode

lib/request/failure-policy.ts

@@ -135,7 +135,7 @@ export function evaluateFailurePolicy(
 				markRateLimited: false,
 				removeAccount: false,
 				cooldownMs,
-				cooldownReason: cooldownMs > 0 ? "network-error" : undefined,
+				cooldownReason: cooldownMs > 0 ? "server-error" : undefined,
 				retrySameAccount,
 				retryDelayMs: retrySameAccount ? 500 : undefined,
 				handoffStrategy: "hard",

lib/request/response-metadata.ts

@@ -1,4 +1,4 @@
-const MAX_RETRY_HINT_MS = 5 * 60 * 1000;
+const MAX_RETRY_HINT_MS = 24 * 60 * 60 * 1000;
 
 function clampRetryHintMs(value: number): number | null {
 	if (!Number.isFinite(value)) return null;

lib/schemas.ts

@@ -81,7 +81,7 @@ export type AccountIdSourceFromSchema = z.infer<typeof AccountIdSourceSchema>;
 /**
  * Cooldown reason for temporary account suspension.
  */
-export const CooldownReasonSchema = z.enum(["auth-failure", "network-error", "rate-limit"]);
+export const CooldownReasonSchema = z.enum(["auth-failure", "network-error", "server-error", "rate-limit"]);
 
 export type CooldownReasonFromSchema = z.infer<typeof CooldownReasonSchema>;
 

lib/storage/migrations.ts

@@ -7,7 +7,7 @@ import { MODEL_FAMILIES, type ModelFamily } from "../prompts/codex.js";
 import type { AccountIdSource } from "../types.js";
 import type { Workspace } from "../accounts.js";
 
-export type CooldownReason = "auth-failure" | "network-error" | "rate-limit";
+export type CooldownReason = "auth-failure" | "network-error" | "server-error" | "rate-limit";
 
 export interface RateLimitStateV3 {
 	[key: string]: number | undefined;

test/failure-policy.test.ts

@@ -65,6 +65,7 @@ describe("failure policy", () => {
 		expect(decision.retryDelayMs).toBe(500);
 		expect(decision.rotateAccount).toBe(false);
 		expect(decision.handoffStrategy).toBe("hard");
+		expect(decision.cooldownReason).toBe("server-error");
 	});
 
 	it("marks rate limit without cooldown mutation", () => {
@@ -146,6 +147,7 @@ describe("failure policy", () => {
 		expect(decision.retrySameAccount).toBe(false);
 		expect(decision.rotateAccount).toBe(true);
 		expect(decision.cooldownMs).toBe(3_000);
+		expect(decision.cooldownReason).toBe("server-error");
 	});
 
 	it("uses override cooldowns for network and server kinds", () => {

test/response-metadata.test.ts

@@ -23,10 +23,10 @@ describe("response metadata helpers", () => {
 		expect(parseRetryAfterHintMs(headers)).toBe(1200);
 	});
 
-	it("parses retry-after seconds and caps large values", () => {
+	it("parses retry-after seconds and caps extreme values to one day", () => {
 		const headers = new Headers({ "retry-after": "999999" });
 
-		expect(parseRetryAfterHintMs(headers)).toBe(300000);
+		expect(parseRetryAfterHintMs(headers)).toBe(86_400_000);
 	});
 
 	it("parses retry-after dates and x-ratelimit-reset timestamps", () => {