refactor(auth-profiles): reuse cooldown timestamp resolver

src/agents/auth-profiles/order.ts

@@ -2,20 +2,11 @@ import type { OpenClawConfig } from "../../config/config.js";
 import { findNormalizedProviderValue, normalizeProviderId } from "../model-selection.js";
 import { dedupeProfileIds, listProfilesForProvider } from "./profiles.js";
 import type { AuthProfileStore } from "./types.js";
-import { clearExpiredCooldowns, isProfileInCooldown } from "./usage.js";
-
-function resolveProfileUnusableUntil(stats: {
-  cooldownUntil?: number;
-  disabledUntil?: number;
-}): number | null {
-  const values = [stats.cooldownUntil, stats.disabledUntil]
-    .filter((value): value is number => typeof value === "number")
-    .filter((value) => Number.isFinite(value) && value > 0);
-  if (values.length === 0) {
-    return null;
-  }
-  return Math.max(...values);
-}
+import {
+  clearExpiredCooldowns,
+  isProfileInCooldown,
+  resolveProfileUnusableUntil,
+} from "./usage.js";
 
 export function resolveAuthProfileOrder(params: {
   cfg?: OpenClawConfig;

src/agents/auth-profiles/usage.test.ts

@@ -1,6 +1,10 @@
 import { describe, expect, it } from "vitest";
 import type { AuthProfileStore } from "./types.js";
-import { clearExpiredCooldowns, isProfileInCooldown } from "./usage.js";
+import {
+  clearExpiredCooldowns,
+  isProfileInCooldown,
+  resolveProfileUnusableUntil,
+} from "./usage.js";
 
 function makeStore(usageStats: AuthProfileStore["usageStats"]): AuthProfileStore {
   return {
@@ -13,6 +17,18 @@ function makeStore(usageStats: AuthProfileStore["usageStats"]): AuthProfileStore
   };
 }
 
+describe("resolveProfileUnusableUntil", () => {
+  it("returns null when both values are missing or invalid", () => {
+    expect(resolveProfileUnusableUntil({})).toBeNull();
+    expect(resolveProfileUnusableUntil({ cooldownUntil: 0, disabledUntil: Number.NaN })).toBeNull();
+  });
+
+  it("returns the latest active timestamp", () => {
+    expect(resolveProfileUnusableUntil({ cooldownUntil: 100, disabledUntil: 200 })).toBe(200);
+    expect(resolveProfileUnusableUntil({ cooldownUntil: 300 })).toBe(300);
+  });
+});
+
 // ---------------------------------------------------------------------------
 // isProfileInCooldown
 // ---------------------------------------------------------------------------

src/agents/auth-profiles/usage.ts

@@ -3,7 +3,9 @@ import { normalizeProviderId } from "../model-selection.js";
 import { saveAuthProfileStore, updateAuthProfileStoreWithLock } from "./store.js";
 import type { AuthProfileFailureReason, AuthProfileStore, ProfileUsageStats } from "./types.js";
 
-function resolveProfileUnusableUntil(stats: ProfileUsageStats): number | null {
+export function resolveProfileUnusableUntil(
+  stats: Pick<ProfileUsageStats, "cooldownUntil" | "disabledUntil">,
+): number | null {
   const values = [stats.cooldownUntil, stats.disabledUntil]
     .filter((value): value is number => typeof value === "number")
     .filter((value) => Number.isFinite(value) && value > 0);