refactor(web): migrate NEED_REFRESH_APP_LIST_KEY to useLocalStorage/useSetLocalStorage

[ZongrongLi]

Summary

Migrates all direct localStorage.getItem/setItem/removeItem calls for NEED_REFRESH_APP_LIST_KEY to useLocalStorage and useSetLocalStorage hooks from @/hooks/use-local-storage.

This is part of the broader refactor tracked in #36898 — one storage concern at a time.

Changes

9 source files touched, all using the same { raw: true } option for raw string storage:

Setter-only (useSetLocalStorage)

• hooks/use-import-dsl.ts — 2 occurrences
• app/components/app-sidebar/app-info/use-app-info-actions.ts
• app/components/app/create-from-dsl-modal/index.tsx — 2 occurrences
• app/components/app/create-app-modal/index.tsx
• app/components/app/create-app-dialog/app-list/index.tsx
• app/components/app/switch-app-modal/index.tsx
• app/components/apps/app-card.tsx
• app/(commonLayout)/app/(appDetailLayout)/[appId]/overview/card-view.tsx

Getter + remover (useLocalStorage)

• app/components/apps/list.tsx — migrated the getItem + removeItem check-and-clear pattern to useLocalStorage<string>(key, '0', { raw: true }), calling setNeedRefresh(null) to clear.

Checklist

• TypeScript compiles with zero errors
• All localStorage.*NEED_REFRESH_APP_LIST_KEY references in source files have been migrated

Part of #36898

[lyzno1]

I think this changes the behavior of NEED_REFRESH_APP_LIST_KEY in web/app/components/apps/list.tsx.

Before this PR, the app list only performed a one-shot check on mount:

if (localStorage.getItem(NEED_REFRESH_APP_LIST_KEY) === '1') {
  localStorage.removeItem(NEED_REFRESH_APP_LIST_KEY)
  refetch()
}

After the change, useLocalStorage turns the key into subscribed state:

const [needRefresh, setNeedRefresh] = useLocalStorage<string>(NEED_REFRESH_APP_LIST_KEY, '0', { raw: true })

That changes two things:

1. When the key is missing, the hook writes the default '0' into localStorage. The old code did not write anything.
2. The list now reacts to same-tab storage updates from useSetLocalStorage('1'), because the hook dispatches/subscribes to its custom storage event. The old code only checked the flag once on mount. In cases like app duplication from an already-mounted app list, this can add another refetch() path on top of the existing onRefresh={refetch} path.

The writer-side replacements with useSetLocalStorage(..., { raw: true }) look fine. The issue is only the reader migration in apps/list.tsx: this key is a one-shot refresh signal, not a persisted UI preference. To keep behavior equivalent, please keep the one-shot read/remove behavior here, or use a storage boundary that does not write a default value and does not subscribe to live updates.