From fb78722a79dcaf6e8ed81ab076ad7ea6d3dbebcf Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Wed, 2 Sep 2026 08:41:25 +0000 Subject: [PATCH] Replace reset-or-adjust-state Effect with render-time state adjustment Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- apps/web/src/repository-picker.tsx | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/apps/web/src/repository-picker.tsx b/apps/web/src/repository-picker.tsx index e2c9d31a..b6bf0d2a 100644 --- a/apps/web/src/repository-picker.tsx +++ b/apps/web/src/repository-picker.tsx @@ -70,6 +70,11 @@ export function RepositoryPicker( let [query, setQuery] = useState(""); let [active, setActive] = useState(0); let [avatarFailed, setAvatarFailed] = useState(false); + let [avatarUrlForFailure, setAvatarUrlForFailure] = useState(current?.ownerAvatarUrl); + if (avatarUrlForFailure !== current?.ownerAvatarUrl) { + setAvatarUrlForFailure(current?.ownerAvatarUrl); + setAvatarFailed(false); + } let normalized = query.trim().toLowerCase(); let repositories = installed?.flatMap(group => group.repositories) ?? []; @@ -97,8 +102,6 @@ export function RepositoryPicker( }; }, []); - useEffect(() => setAvatarFailed(false), [current?.ownerAvatarUrl]); - useEffect(() => { if (!open) { previousQuery.current = "";