fix(gui): address remaining pool hydration review nits

Author: Wibias

gui/src/clamp-draft.ts

@@ -6,7 +6,8 @@ export function clampNumberDraft(
   max: number,
   step = 1,
 ): string {
-  const parsed = Number(raw);
+  const trimmed = raw.trim();
+  const parsed = trimmed === "" ? NaN : Number(trimmed);
   const base = Number.isFinite(parsed) ? parsed : min;
   const next = Math.min(max, Math.max(min, base + delta));
   // Keep one decimal for GiB-style steps; integers otherwise.

gui/src/components/CodexPoolStrategySetting.tsx

@@ -48,6 +48,8 @@ export default function CodexPoolStrategySetting({
   const hydratedRef = useRef(false);
   const [saving, setSaving] = useState(false);
   const savingRef = useRef(false);
+  /** Set when an /active read arrives mid-save; triggers one post-save refresh. */
+  const deferredActiveRefreshRef = useRef(false);
   const revisionRef = useRef(0);
   const [loadError, setLoadError] = useState(false);
   const [error, setError] = useState<string | null>(null);
@@ -80,25 +82,47 @@ export default function CodexPoolStrategySetting({
     try {
       const res = await fetch(`${apiBase}/api/codex-auth/active`);
       if (!res.ok) throw new Error("load");
-      applyServer(await res.json() as {
+      const payload = await res.json() as {
         accountPoolStrategy?: unknown;
         accountPoolStickyLimit?: unknown;
-      });
+      };
+      // A save started while this GET was in flight — retry once after it settles.
+      if (savingRef.current) {
+        deferredActiveRefreshRef.current = true;
+        return;
+      }
+      applyServer(payload);
     } catch {
-      setLoadError(true);
+      if (!savingRef.current) setLoadError(true);
     }
   }, [apiBase, applyServer]);
 
+  const scheduleDeferredActiveRefresh = useCallback(() => {
+    if (!deferredActiveRefreshRef.current) return;
+    deferredActiveRefreshRef.current = false;
+    queueMicrotask(() => {
+      if (savingRef.current) {
+        deferredActiveRefreshRef.current = true;
+        return;
+      }
+      void load();
+    });
+  }, [load]);
+
   // Shared /active observer (preferred): same payload the pool already fetched.
-  // Ignore stale polls that started before a PUT bumped revision, and never apply while saving.
+  // Ignore stale polls that started before a PUT bumped revision; mid-save reads
+  // arm one post-save /active refresh instead of being dropped forever.
   // Replay readLastActive on subscribe so late mounts hydrate without waiting a poll.
   useEffect(() => {
     if (!subscribeLoadObserver) return;
     const observer: CodexAccountLoadObserver = {
       beginActiveRead: () => revisionRef.current,
       acceptActiveRead: (value, startedRevision) => {
         if (startedRevision !== revisionRef.current) return;
-        if (savingRef.current) return;
+        if (savingRef.current) {
+          deferredActiveRefreshRef.current = true;
+          return;
+        }
         applyActivePayload(value);
       },
       rejectActiveRead: () => {
@@ -154,7 +178,8 @@ export default function CodexPoolStrategySetting({
     }
     savingRef.current = false;
     setSaving(false);
-  }, [apiBase, stickyLimit, strategy, t]);
+    scheduleDeferredActiveRefresh();
+  }, [apiBase, scheduleDeferredActiveRefresh, stickyLimit, strategy, t]);
 
   // Block writes until /active confirms — defaults paint for CLS but must not overwrite server state.
   const controlsDisabled = saving || loadError || !hydrated;

gui/src/hooks/useCodexAccountPool.ts

@@ -1,4 +1,5 @@
 import { useCallback, useEffect, useRef, useState } from "react";
+import { extractAutoSwitchThresholdPayload } from "../codex-auto-switch";
 import type { AccountQuota } from "../codex-quota-utils";
 import { accountNeedsReauth } from "../oauth-health-display";
 
@@ -123,13 +124,10 @@ export function useCodexAccountPool(apiBase: string, enabled = true): CodexAccou
   }, []);
 
   /** Last threshold an actual read returned, or undefined when none has succeeded yet. */
-  const readLastThreshold = useCallback(() => {
-    const active = lastActiveRef.current?.value;
-    if (active && typeof active === "object" && active !== null && "autoSwitchThreshold" in active) {
-      return (active as { autoSwitchThreshold: unknown }).autoSwitchThreshold;
-    }
-    return active;
-  }, []);
+  const readLastThreshold = useCallback(
+    () => extractAutoSwitchThresholdPayload(lastActiveRef.current?.value),
+    [],
+  );
 
   /** Full last /active payload, or undefined when none has succeeded yet. */
   const readLastActive = useCallback(() => lastActiveRef.current?.value, []);