maxpool 1.5.13 → 1.5.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "maxpool",
-  "version": "1.5.13",
+  "version": "1.5.14",
   "description": "Multi-account Claude Code proxy with adaptive, rate-aware load balancing across Claude accounts",
   "type": "module",
   "main": "src/index.js",

package/src/account-manager.js

@@ -1617,10 +1617,27 @@ export class AccountManager {
     return 'normal';
   }
 
+  // True only when the account is REJECTED ACCOUNT-WIDE — a 'rejected' unified
+  // status CORROBORATED by an actually-exhausted unified bucket. A bare 'rejected'
+  // with healthy unified buckets is a PER-MODEL sub-limit rejection (e.g. Fable
+  // weekly, tracked in scopedWeekly) that Anthropic reports on the unified-status
+  // header — it must NOT bench or "block" the whole account, which still has
+  // headroom for its other models. Guards both routing (_weeklyRawState) and the
+  // TUI "blocked" label against the per-model→account-wide conflation. NOTE: this
+  // is a display/inter-429 hint, not the primary bench — a genuinely-dead account
+  // is benched account-wide by markRateLimited (status='throttled') on its next 429.
+  _isAccountWideRejected(account) {
+    const q = account?.quota || {};
+    if (q.unifiedStatus !== 'rejected') return false;
+    const floor = this.scheduler.weeklyExhaustedThreshold;
+    return (Number.isFinite(q.unified5h) && q.unified5h >= floor)
+      || (Number.isFinite(q.unified7d) && q.unified7d >= floor);
+  }
+
   _weeklyRawState(account) {
     const q = account.quota;
     this._clearExpiredQuotas(account);
-    if (q.unifiedStatus === 'rejected') return 'exhausted';
+    if (this._isAccountWideRejected(account)) return 'exhausted';
     if (q.unified7d == null) return 'unknown';
 
     const used = clamp01(q.unified7d);
@@ -1727,7 +1744,18 @@ export class AccountManager {
     }
 
     const uStatus = headers['anthropic-ratelimit-unified-status'];
-    if (uStatus) account.quota.unifiedStatus = uStatus;
+    if (uStatus) {
+      // 'rejected' with HEALTHY unified buckets is a PER-MODEL sub-limit rejection
+      // (e.g. Fable weekly) — Anthropic surfaces it on the account-wide status
+      // header, but recording it account-wide would bench the account for EVERY
+      // model and render it "blocked" (the per-model cap is tracked in scopedWeekly).
+      // Only honor 'rejected' when a unified bucket corroborates a real account block.
+      const floor = this.scheduler.weeklyExhaustedThreshold;
+      const unifiedExhausted =
+        (Number.isFinite(account.quota.unified5h) && account.quota.unified5h >= floor)
+        || (Number.isFinite(account.quota.unified7d) && account.quota.unified7d >= floor);
+      account.quota.unifiedStatus = (uStatus === 'rejected' && !unifiedExhausted) ? 'allowed' : uStatus;
+    }
 
     // Standard rate limits (API key accounts)
     const tokensLimit = parseInt(headers['anthropic-ratelimit-tokens-limit'], 10);

package/src/tui.js

@@ -925,14 +925,12 @@ export class TUI {
     if (a.enabled !== false && upstreamBlocking && a.status === 'active') {
       effectiveStatus = a.inFlight > 0 ? 'probing' : 'waiting';
     }
-    // Anthropic is actively REJECTING this account right now (e.g. a per-model weekly
-    // sub-limit the general utilization % doesn't expose). It's unusable — surface
-    // that instead of a benign green "active", so a low weekly % (the bar keeps its
-    // true value) is never misread as available headroom.
-    // Keys on a.status (not effectiveStatus) so a rejected account reads 'blocked'
-    // even inside an upstream-throttle window (where it would otherwise show
-    // probing/waiting) — a rejected account is unusable, not part of the recovery.
-    if (a.enabled !== false && a.quota?.unifiedStatus === 'rejected' && a.status === 'active') {
+    // "blocked" = Anthropic is rejecting the WHOLE account (a 'rejected' unified
+    // status corroborated by an exhausted unified bucket). A per-model cap (Fable)
+    // is NOT account-wide — it shows as the separate "… maxed" tag, leaving the
+    // status column truthful. Keys on a.status (not effectiveStatus) so a genuine
+    // block wins even inside an upstream-throttle window.
+    if (a.enabled !== false && a.status === 'active' && this.am._isAccountWideRejected?.(a)) {
       effectiveStatus = 'blocked';
     }
     switch (effectiveStatus) {