maxpool 1.5.32 → 1.5.34

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "maxpool",
-  "version": "1.5.32",
+  "version": "1.5.34",
   "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

@@ -62,6 +62,11 @@ function emptyQuota() {
     lastProbeError: null,       // string
     lastProbeErrorAt: null,     // ms
     lastProbeErrorStatus: null, // http status (429, 500, …) or null
+    // Last time the DISPLAYED bars (unified5h/7d for OAuth, tokens/requests for
+    // API-key) were refreshed from a RESPONSE HEADER (updateQuota). Lets the TUI
+    // staleness marker tell "probe stale but bars fresh from live traffic" (a busy
+    // account — NOT stale) from "genuinely nothing refreshed it" (idle — stale).
+    lastHeaderQuotaAt: null,    // ms
   };
 }
 
@@ -2118,6 +2123,15 @@ export class AccountManager {
     if (genericRemaining != null) account.quota.genericRemaining = genericRemaining;
     if (genericReset != null) account.quota.genericReset = genericReset;
 
+    // Stamp header-freshness ONLY when a real quota header actually arrived — so a
+    // header-less response never falsely marks the bars fresh. Drives the TUI's
+    // "busy account isn't stale even if its background probe is 429-throttled".
+    const gotQuotaHeader = !isNaN(u5h) || !isNaN(u7d)
+      || !isNaN(tokensLimit) || !isNaN(tokensRemaining)
+      || !isNaN(requestsLimit) || !isNaN(requestsRemaining)
+      || genericLimit != null || genericRemaining != null;
+    if (gotQuotaHeader) account.quota.lastHeaderQuotaAt = Date.now();
+
     account.usage.totalRequests++;
     account.usage.lastUsed = new Date().toISOString();
 

package/src/tui.js

@@ -985,7 +985,7 @@ export class TUI {
       // Glossary FOOTER (expands the abbreviations the header + inline labels can't
       // spell out). Below the rows so it never breaks the header↔column alignment.
       if (W >= 88) {
-        lines.push(' ' + dim('Ses 5h · Wk 7d · Now in-flight (weight) · 15m/1h served (avg · f fails)'));
+        lines.push(' ' + dim('Legend  Ses = 5h · Wk = 7d · Now = in-flight (weight) · 15m/1h = served (avg · f = fails)'));
       }
     }
 
@@ -1040,13 +1040,16 @@ export class TUI {
 
   _renderAcct(idx, bw, showBoth) {
     const a = this.am.accounts[idx];
-    // Highlight the currently-active account. In manual mode that's the
-    // preferred account; in automatic mode it's the one most recently routed
-    // to (currentIndex). Previously only manual mode highlighted anything, so
-    // in automatic load-balancing no row was ever marked current.
+    // Highlight the ACTIVE account(s) — drives BOTH the ► marker and the green
+    // "active" status color. In manual mode that's the single pinned account (all
+    // traffic goes there). In automatic load-balancing there is no single "current"
+    // account, so mark every account SERVING a request right now (inFlight > 0) —
+    // an honest live view of what's working. When idle, nothing is marked; a
+    // throttled/idle provider is never falsely flagged (the old "most recently
+    // routed to" currentIndex could land the marker on an idle fallback).
     const isCur = this.am.routingMode === 'preferred'
       ? a.name === this.am.preferredAccountName
-      : idx === this.am.currentIndex;
+      : a.inFlight > 0;
     const isSel = this.mode === 'select' && idx === this.selIdx;
 
     // Prefix: selection marker + current marker
@@ -1174,8 +1177,24 @@ export class TUI {
     // Only annotate probe-staleness for LIVE accounts, where a failing probe
     // (e.g. a 429) is a real, actionable signal.
     if (a?.refreshDead || a?.enabled === false) return '';
-    if (!this.am._quotaProbeStale?.(a)) return '';
-    const s = a?.quota?.lastProbeErrorStatus;
+    if (!this.am._quotaProbeStale?.(a)) return '';   // probe fresh (or off) → nothing to flag; interval>0 after this
+    // The background probe IS stale — but only flag it if something DISPLAYED is
+    // actually stale. An OAuth account's Ses/Wk bars come from unified5h/7d, which
+    // response headers refresh on every served request (updateQuota → lastHeaderQuotaAt);
+    // so a BUSY oauth account with a header-fresh reading and no probe-only scoped
+    // cap is NOT stale, even if its background probe is 429-throttled. Provider bars
+    // and per-model scoped caps ARE probe-only, so for those a stale probe genuinely
+    // means stale data → keep flagging.
+    const q = a.quota || {};
+    const reserveFloor = this.am.scheduler?.weeklyReserveThreshold ?? 0.85; // mirrors _renderAcct's scoped-tag gate
+    const hasDisplayedScopedCap = Object.values(q.scopedWeekly || {}).some(
+      e => e && e.isActive !== false && e.utilization != null && e.utilization >= reserveFloor);
+    if (a.type !== 'provider' && !hasDisplayedScopedCap) {
+      const interval = this.am.quotaProbeIntervalMs; // > 0 here (else _quotaProbeStale returned false)
+      const headerFresh = q.lastHeaderQuotaAt && (Date.now() - q.lastHeaderQuotaAt) <= Math.max(3 * interval, 180_000);
+      if (headerFresh) return '';
+    }
+    const s = q.lastProbeErrorStatus;
     if (s === 429) return `  ${yellow('stale·rate-limited')}`;
     if (s) return `  ${yellow('stale·probe ' + s)}`;
     return `  ${dim('stale')}`;