perf(proxies): reduce delay-driven list and card churn

Changelog.md

@@ -48,6 +48,7 @@
 
 - 优化连接、规则和日志筛选：切换页面后保留过滤条件
 - 优化全局模式代理列表：仅显示 `GLOBAL` 节点，选中其中的策略组时才显示该组
+- 优化代理页面和首页节点卡片，减少延迟更新时的卡顿
 
 **🍎 macOS**
 

src/components/home/current-proxy-card.tsx

@@ -605,12 +605,14 @@ export const CurrentProxyCard = () => {
     [getProfileStorageKey],
   )
 
-  const { select } = useProxySelection({
+  const onSelectionError = useCallback(
-    onError: (error) => {
+    (error: unknown) => {
       console.error('代理切换失败', error)
       refreshProxy()
     },
-  })
+    [refreshProxy],
+  )
+  const { select } = useProxySelection({ onError: onSelectionError })
 
   const mode = clashConfig?.mode?.toLowerCase() || 'rule'
   const isGlobalMode = mode === 'global'
@@ -673,7 +675,6 @@ export const CurrentProxyCard = () => {
   if (proxyView && lastGroupName !== selectedGroupName) {
     setLastGroupName(selectedGroupName)
   }
-  const delays = useGroupDelays(selectedGroupName || null)
 
   const selectedGroup = useMemo<ProxyGroupView | null>(() => {
     if (!proxyView || isDirectMode) return null
@@ -767,10 +768,17 @@ export const CurrentProxyCard = () => {
   const currentProxy = currentMember ? memberDetails(currentMember) : undefined
   const selectedProxyName = currentMember?.ref.name ?? ''
 
+  const delays = useGroupDelays(
+    selectedGroupName,
+    currentMember,
+    openSelect === 'proxy',
+  )
   const currentDelay =
-    currentMember && selectedGroupName
+    typeof delays === 'number'
-      ? delayManager.getDelayFix(currentMember, selectedGroupName)
+      ? delays
-      : -1
+      : currentMember
+        ? delays.of(currentMember)
+        : -1
 
   const signalInfo =
     currentProxy && selectedGroupName
@@ -892,7 +900,7 @@ export const CurrentProxyCard = () => {
 
   const proxyOptions = useMemo(
     () =>
-      isDirectMode || openSelect !== 'proxy'
+      isDirectMode || openSelect !== 'proxy' || typeof delays === 'number'
         ? []
         : sortProxyOptions(
             unsortedProxyOptions,

src/components/proxy/proxy-groups.tsx

@@ -469,12 +469,14 @@ function NormalProxyGroups(props: { mode: string }) {
     }
   }, [handleScroll])
 
-  const { select } = useProxySelection({
+  const onSelectionError = useCallback(
-    onError: (error) => {
+    (error: unknown) => {
       console.error('代理切换失败', error)
       onProxies()
     },
-  })
+    [onProxies],
+  )
+  const { select } = useProxySelection({ onError: onSelectionError })
 
   const handleChangeProxy = useCallback(
     (group: ProxyGroupView, member: ResolvedProxyMember) => {

src/components/proxy/use-filter-sort.ts

@@ -29,6 +29,9 @@ export function filterSort(
 const regex1 = /delay([=<>])(\d+|timeout|error)/i
 const regex2 = /type=(.*)/i
 
+export const isDelayFilter = (filterText: string) =>
+  regex1.test(filterText.trim())
+
 function filterProxies(
   proxies: ResolvedMemberOccurrence[],
   groupName: string,

src/components/proxy/use-render-list.test.ts

@@ -0,0 +1,119 @@
+import { expect, it, vi } from 'vitest'
+
+import type { DelaySnapshot } from '@/services/delay'
+import { resolveMember, type ProxyViewV1 } from '@/types/proxy-view'
+
+import { filterSort } from './use-filter-sort'
+import { useRenderList as readRenderList } from './use-render-list'
+
+const hooks = vi.hoisted(() => {
+  const slots: { value: unknown; deps?: unknown[] }[] = []
+  let cursor = 0
+  return {
+    reset: () => {
+      cursor = 0
+    },
+    memo: (factory: () => unknown, deps: unknown[]) => {
+      const index = cursor++
+      const old = slots[index]
+      if (!old || deps.some((dep, i) => !Object.is(dep, old.deps?.[i]))) {
+        slots[index] = { value: factory(), deps }
+      }
+      return slots[index].value
+    },
+    ref: (current: unknown) => {
+      const index = cursor++
+      slots[index] ??= { value: { current } }
+      return slots[index].value
+    },
+  }
+})
+const state = vi.hoisted(() => ({
+  view: null as ProxyViewV1 | null,
+  delays: new Map<string, DelaySnapshot>(),
+}))
+
+vi.mock('react', () => ({
+  useMemo: hooks.memo,
+  useRef: hooks.ref,
+  useEffect: () => {},
+}))
+vi.mock('@/hooks/use-profiles', () => ({}))
+vi.mock('@/hooks/use-clash', () => ({
+  useRuntimeConfig: () => ({ data: null }),
+}))
+vi.mock('@/hooks/use-verge', () => ({
+  useVerge: () => ({ verge: { proxy_layout_column: 1 } }),
+}))
+vi.mock('@/providers/app-data-context', () => ({
+  useProxiesData: () => ({ proxyView: state.view }),
+  useAppRefreshers: () => ({ refreshProxy: () => {} }),
+}))
+vi.mock('@/hooks/use-group-delays', () => ({
+  useGroupsDelays: (names: string[]) =>
+    new Map(names.map((name) => [name, state.delays.get(name)])),
+}))
+vi.mock('./use-window-width', () => ({
+  useWindowWidth: () => ({ width: 800 }),
+}))
+vi.mock('./use-head-state', async (original) => {
+  const actual = await original<typeof import('./use-head-state')>()
+  const heads = {
+    plain: { ...actual.DEFAULT_STATE, open: true },
+    sorted: { ...actual.DEFAULT_STATE, open: true, sortType: 1 },
+  }
+  return { ...actual, useHeadStateNew: () => [heads, () => {}] }
+})
+vi.mock('./use-filter-sort', async (original) => {
+  const actual = await original<typeof import('./use-filter-sort')>()
+  return { ...actual, filterSort: vi.fn(actual.filterSort) }
+})
+vi.mock('@/types/proxy-view', async (original) => {
+  const actual = await original<typeof import('@/types/proxy-view')>()
+  return { ...actual, resolveMember: vi.fn(actual.resolveMember) }
+})
+
+it('skips delay-independent projections and reuses members for delay sorting', () => {
+  const group = (name: string) => ({
+    name,
+    type: 'Selector',
+    history: [],
+    udp: false,
+    xudp: false,
+    tfo: false,
+    mptcp: false,
+    smux: false,
+    members: Array.from({ length: 500 }, (_, i) => ({
+      kind: 'unresolved' as const,
+      name: `${i}`,
+      reason: 'missing' as const,
+    })),
+  })
+  state.view = {
+    schemaVersion: 1,
+    orderSource: 'runtime',
+    providerState: 'ready',
+    global: null,
+    direct: null,
+    groups: [group('plain'), group('sorted')],
+    records: {},
+    standalone: [],
+    providers: [],
+  }
+  const render = () => {
+    hooks.reset()
+    return readRenderList('rule')
+  }
+  state.delays.set('plain', { of: () => 42 })
+  state.delays.set('sorted', { of: () => 42 })
+  render()
+  vi.mocked(filterSort).mockClear()
+  vi.mocked(resolveMember).mockClear()
+  state.delays.set('plain', { of: () => 43 })
+  render()
+  expect(filterSort).toHaveBeenCalledTimes(0)
+  state.delays.set('sorted', { of: () => 43 })
+  render()
+  expect(filterSort).toHaveBeenCalledTimes(1)
+  expect(resolveMember).toHaveBeenCalledTimes(0)
+})

src/components/proxy/use-render-list.ts

@@ -17,7 +17,7 @@ import {
 } from '@/types/proxy-view'
 import { debugLog } from '@/utils/debug'
 
-import { filterSort } from './use-filter-sort'
+import { filterSort, isDelayFilter } from './use-filter-sort'
 import {
   DEFAULT_STATE,
   useHeadStateNew,
@@ -60,14 +60,14 @@ export const hasRenderableItems = (
 ): boolean => renderList.some((item) => item.type !== 0 || !item.group.hidden)
 
 type GroupCache = {
-  now: string | undefined
+  group: ProxyGroupView
-  members: ProxyGroupView['members']
   headState: HeadState
   col: number
   latencyTimeout: number | undefined
   /// This group's own delays. Compared by identity so that a test settling in one group
   /// does not throw away every other group's sorted order.
   delays: DelaySnapshot | undefined
+  occurrences: ResolvedMemberOccurrence[] | undefined
   items: IRenderItem[]
 }
 
@@ -222,19 +222,51 @@ export const useRenderList = (
     return current?.kind === 'group' ? [global, current.group] : [global]
   }, [mode, proxyView])
 
-  // Every group this list draws, so a test settling in any of them re-sorts that group.
+  const repeatedNameGroups = useMemo(() => {
+    const repeated = new Set<string>()
+    if (isChainMode) return repeated
+    for (const group of renderGroups) {
+      const names = new Set<string>()
+      if (
+        group.members.some(({ name }) => {
+          if (names.has(name)) return true
+          names.add(name)
+          return false
+        })
+      ) {
+        repeated.add(group.name)
+      }
+    }
+    return repeated
+  }, [isChainMode, renderGroups])
+
+  // Rows own delay updates unless repeated names share the current item listener.
   const renderedGroupNames = useMemo(() => {
-    if (!proxyView) return []
+    if (!proxyView || isChainMode) return []
-    if (isChainMode)
+    return renderGroups
-      return selectedGroup ? [selectedGroup] : [CHAIN_DELAY_GROUP]
+      .filter((group) => {
-    return renderGroups.map(({ name }) => name)
+        const state = headStates[group.name] || DEFAULT_STATE
-  }, [isChainMode, proxyView, renderGroups, selectedGroup])
+        return (
+          (state.open || group === proxyView.global) &&
+          (state.sortType === 1 ||
+            isDelayFilter(state.filterText) ||
+            repeatedNameGroups.has(group.name))
+        )
+      })
+      .map(({ name }) => name)
+  }, [headStates, isChainMode, proxyView, renderGroups, repeatedNameGroups])
   const groupDelays = useGroupsDelays(renderedGroupNames)
 
-  const groupCacheRef = useRef<Map<string, GroupCache>>(new Map())
+  const groupCacheRef = useRef<Map<string, GroupCache> | null>(null)
+  const cachedViewRef = useRef(proxyView)
   const prevListRef = useRef<IRenderItem[]>([])
 
   const renderList = useMemo<IRenderItem[]>(() => {
+    const cache = (groupCacheRef.current ??= new Map())
+    if (cachedViewRef.current !== proxyView) {
+      cache.clear()
+      cachedViewRef.current = proxyView
+    }
     if (!proxyView) return []
 
     if (isChainMode) {
@@ -269,16 +301,13 @@ export const useRenderList = (
       }))
     }
 
-    const cache = groupCacheRef.current
-
     const retList = renderGroups.flatMap((group) => {
       const isFlat = group === proxyView.global
       const headState = headStates[group.name] || DEFAULT_STATE
       const cached = cache.get(group.name)
       if (
         cached &&
-        cached.now === group.now &&
+        cached.group === group &&
-        cached.members === group.members &&
         cached.headState === headState &&
         cached.col === col &&
         cached.latencyTimeout === latencyTimeout &&
@@ -300,9 +329,18 @@ export const useRenderList = (
             },
       ]
 
+      // Duplicate names share one item listener until the delay-store fix lands.
+      const refreshRepeated =
+        repeatedNameGroups.has(group.name) &&
+        cached?.delays !== groupDelays.get(group.name)
+      let resolved =
+        cached?.group === group && !refreshRepeated
+          ? cached.occurrences
+          : undefined
       if (headState.open || isFlat) {
+        resolved ??= resolveOccurrences(proxyView, group)
         const occurrences = filterSort(
-          resolveOccurrences(proxyView, group),
+          resolved,
           group.name,
           headState.filterText,
           headState.sortType,
@@ -337,15 +375,17 @@ export const useRenderList = (
             })),
           )
         }
+      } else {
+        resolved = undefined
       }
 
       cache.set(group.name, {
-        now: group.now!,
+        group,
-        members: group.members,
         headState,
         col,
         latencyTimeout: latencyTimeout!,
         delays: groupDelays.get(group.name),
+        occurrences: resolved,
         items: ret,
       })
       return ret
@@ -371,6 +411,7 @@ export const useRenderList = (
     mode,
     proxyView,
     renderGroups,
+    repeatedNameGroups,
     selectedGroup,
   ])
 

src/hooks/use-group-delays.test.ts

@@ -0,0 +1,45 @@
+import { useSyncExternalStore } from 'react'
+import { expect, it, vi } from 'vitest'
+
+import delayManager from '@/services/delay'
+import type { ResolvedProxyMember } from '@/types/proxy-view'
+
+import { useGroupDelays } from './use-group-delays'
+
+vi.mock('react', () => ({
+  useCallback: (callback: unknown) => callback,
+  useSyncExternalStore: vi.fn((_subscribe, read) => read()),
+}))
+
+it('keeps the closed-menu snapshot unchanged when another member settles', () => {
+  const member: ResolvedProxyMember = {
+    kind: 'node',
+    ref: { kind: 'node', name: 'current', recordId: 'current' },
+    node: {
+      recordId: 'current',
+      name: 'current',
+      type: 'Shadowsocks',
+      source: { kind: 'core', proxyName: 'current' },
+      history: [],
+      udp: false,
+      xudp: false,
+      tfo: false,
+      mptcp: false,
+      smux: false,
+    },
+  }
+  vi.spyOn(delayManager, 'getDelayFix').mockReturnValue(42)
+  const snapshot = vi.spyOn(delayManager, 'groupDelays')
+  snapshot.mockReturnValue({ of: () => 42 })
+  useGroupDelays('group', member, false)
+  const [, read] = vi.mocked(useSyncExternalStore).mock.calls.at(-1)!
+  let previous = read()
+  let changes = 0
+  for (let i = 0; i < 120; i++) {
+    snapshot.mockReturnValue({ of: () => 42 })
+    const next = read()
+    if (!Object.is(next, previous)) changes++
+    previous = next
+  }
+  expect(changes).toBe(0)
+})

src/hooks/use-group-delays.ts

@@ -1,19 +1,27 @@
 import { useCallback, useSyncExternalStore } from 'react'
 
 import delayManager, { type DelaySnapshot } from '@/services/delay'
+import type { ResolvedProxyMember } from '@/types/proxy-view'
 
-const NO_DELAYS: DelaySnapshot = { of: () => -1 }
+/** A closed menu observes only its current member, not every settled group delay. */
-
+export const useGroupDelays = (
-/** Exposes the external delay store to React, updating sort order only after a test settles. */
+  group: string | null,
-export const useGroupDelays = (group: string | null): DelaySnapshot => {
+  member: ResolvedProxyMember | undefined,
+  menuOpen: boolean,
+): DelaySnapshot | number => {
   const subscribe = useCallback(
     (onSettle: () => void) =>
       group ? delayManager.addGroupListener(group, onSettle) : () => {},
     [group],
   )
   const read = useCallback(
-    () => (group ? delayManager.groupDelays(group) : NO_DELAYS),
+    () =>
-    [group],
+      group && menuOpen
+        ? delayManager.groupDelays(group)
+        : group && member
+          ? delayManager.getDelayFix(member, group)
+          : -1,
+    [group, member, menuOpen],
   )
 
   return useSyncExternalStore(subscribe, read, read)