refactor(gui): drop dead refs and duplicate Desktop/Debug cache reads

Remove unused SectionTabs stripRef and ClaudeDesktop hasCache, seed Desktop from one cache read, inline ClaudeCode session seed, and reuse debugResourceKey for optimistic writes.

Author: Wibias

gui/src/components/section-tabs.tsx

@@ -28,7 +28,6 @@ export function SectionTabs({
   ariaLabel: string;
 }) {
   const [active, setActive] = useState(items[0]?.id ?? "");
-  const stripRef = useRef<HTMLDivElement | null>(null);
   /** While set, scroll-spy ignores intermediate sections during smooth scroll-to-click. */
   const scrollLockRef = useRef<string | null>(null);
   const scrollLockTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
@@ -110,7 +109,6 @@ export function SectionTabs({
 
   return (
     <div
-      ref={stripRef}
       className="page-tabs section-tabs"
       role="tablist"
       aria-label={ariaLabel}

gui/src/pages/ClaudeCode.tsx

@@ -22,17 +22,13 @@ export { AutoConnectSetting, SmallFastModelSetting } from "./claude-code-setting
 
 type CachedClaudeCode = { state: ClaudeCodeState; rows: MapRow[] };
 
-function seedClaudeCode(cacheKey: string): CachedClaudeCode | null {
-  return readSessionListCache<CachedClaudeCode>(cacheKey);
-}
-
 export default function ClaudeCode({ apiBase, active = true }: { apiBase: string; active?: boolean }) {
   const t = useT();
   const { locale } = useI18n();
   const localeTag = LOCALES.find(l => l.code === locale)?.htmlLang ?? "en";
   const cacheKey = `ocx.claude-code.v1:${apiBase}`;
   const resourceKey = `claude-code:${apiBase}`;
-  const cached = useMemo(() => seedClaudeCode(cacheKey), [cacheKey]);
+  const cached = useMemo(() => readSessionListCache<CachedClaudeCode>(cacheKey), [cacheKey]);
   const [draftState, setState] = useState<ClaudeCodeState | null>(() => cached?.state ?? null);
   const [draftRows, setRows] = useState<MapRow[]>(() => cached?.rows ?? []);
   const [hasDraftRows, setHasDraftRows] = useState(Boolean(cached));

gui/src/pages/ClaudeDesktop.tsx

@@ -131,6 +131,7 @@ function readDesktopCache(cacheKey: string): CachedDesktop | null {
 function seedDesktop(cacheKey: string) {
   const cached = readDesktopCache(cacheKey);
   return {
+    held: cached,
     data: cached?.data ?? null,
     profile: cached?.profile ?? null,
     savedProfile: cached?.profile ? cloneProfile(cached.profile) : null,
@@ -139,7 +140,6 @@ function seedDesktop(cacheKey: string) {
         cached.data.models.map(model => [model.route, cached.profile.assignments[model.route]?.family ?? "opus"]),
       )
       : {} as Record<string, Family>,
-    hasCache: Boolean(cached?.data),
   };
 }
 
@@ -158,7 +158,6 @@ export default function ClaudeDesktop({
   const cacheKey = `ocx.claude-desktop.v1:${apiBase}`;
   const resourceKey = `claude-desktop:${apiBase}`;
   const cached = useMemo(() => seedDesktop(cacheKey), [cacheKey]);
-  const heldDesktop = useMemo(() => readDesktopCache(cacheKey), [cacheKey]);
   const [draftProfile, setProfile] = useState<DesktopProfile | null>(() => cached.profile);
   const [savedDraftProfile, setSavedProfile] = useState<DesktopProfile | null>(() => cached.savedProfile);
   const [draftDestinations, setDestinations] = useState<Record<string, Family>>(() => cached.destinations);
@@ -212,7 +211,7 @@ export default function ClaudeDesktop({
     resourceKey,
     [apiBase],
     fetchDesktop,
-    { isEmpty: () => false, enabled: active, initialData: heldDesktop ?? undefined },
+    { isEmpty: () => false, enabled: active, initialData: cached.held ?? undefined },
   );
   const loadState = desktopResource.state;
   const resourceData = loadState.data ?? (cached.data && cached.profile ? { data: cached.data, profile: cached.profile } : null);

gui/src/pages/Debug.tsx

@@ -179,7 +179,7 @@ export default function Debug({ apiBase, embedded, active = true }: { apiBase: s
         const next = await res.json() as DebugSettings;
         if (generation !== mutationGenerationRef.current) return;
         writeSessionListCache(settingsCacheKey, next);
-        setClientResourceData(debugSettingsKey(apiBase), next);
+        setClientResourceData(debugResourceKey, next);
       } catch { /* ignore */ }
     };
     const previous = mutationQueueRef.current ?? Promise.resolve();