Let the search dialog retry after a failed load

The request guard was set before the fetch and never cleared, so one failed
package load left "Could not load packages." in place until a full reload, even
after closing and reopening the dialog. Before the refactor the effect simply
ran again on the next open.

The guard now resets when the fetch throws, and opening the dialog clears a
stale error during render, so the spinner shows again while the retry is in
flight instead of the old message.

Author: Musiker15

components/search/SearchDialog.tsx

@@ -27,16 +27,20 @@ export function SearchDialog({ open, onClose }: Props) {
 
   // Packages beim ersten Öffnen laden. Der Fetch läuft komplett im Effect und
   // schreibt State nur nach dem await — kein zusätzlicher Render-Durchlauf.
+  // Nach einem Fehlschlag wird die Sperre wieder gelöst, damit das nächste
+  // Öffnen es erneut versucht, statt die alte Fehlermeldung bis zum Reload
+  // stehen zu lassen.
   useEffect(() => {
     if (!open || requested.current) return
     requested.current = true
     let alive = true
     async function run() {
       try {
         const list = await getPackages()
-        if (alive) setPackages(list)
+        if (alive) { setPackages(list); setError(null) }
       } catch (err) {
         console.error('[SearchDialog]', err)
+        requested.current = false
         if (alive) setError('Could not load packages.')
       }
     }
@@ -65,11 +69,14 @@ export function SearchDialog({ open, onClose }: Props) {
   }, [open, onClose])
 
   // Query bei Close resetten — während des Renders statt im Effect, damit der
-  // Dialog beim nächsten Öffnen nie kurz die alte Eingabe zeigt.
+  // Dialog beim nächsten Öffnen nie kurz die alte Eingabe zeigt. Beim Öffnen
+  // fällt eine alte Fehlermeldung weg, sonst zeigt der Dialog sie weiter,
+  // während der Effect darunter längst neu lädt.
   const [wasOpen, setWasOpen] = useState(open)
   if (wasOpen !== open) {
     setWasOpen(open)
-    if (!open) setQuery('')
+    if (open) setError(null)
+    else setQuery('')
   }
 
   if (!open) return null