Review fixes

Author: ScriptSmith

ui/src/components/Modal/Modal.tsx

@@ -80,7 +80,7 @@ export function Modal({ open, onClose, children, className }: ModalProps) {
           firstInput.focus();
         } else {
           const firstFocusable = contentRef.current?.querySelector<HTMLElement>(
-            'button, input, select, textarea, [tabindex]:not([tabindex="-1"])'
+            'button, [href], input, select, textarea, [tabindex]:not([tabindex="-1"])'
           );
           if (firstFocusable) {
             firstFocusable.focus();

ui/src/pages/AccountPage.tsx

@@ -32,6 +32,18 @@ function exportLocalStorageData(): Record<string, unknown> {
   return result;
 }
 
+/** Clear WASM SQLite database and unregister the service worker */
+async function clearWasmData(): Promise<void> {
+  await new Promise<void>((resolve) => {
+    const req = indexedDB.deleteDatabase("hadrian-wasm");
+    req.onsuccess = () => resolve();
+    req.onerror = () => resolve();
+    req.onblocked = () => resolve();
+  });
+  const registrations = await navigator.serviceWorker?.getRegistrations();
+  await Promise.all(registrations?.map((r) => r.unregister()) ?? []);
+}
+
 /** Clear all Hadrian-related localStorage data */
 function clearLocalStorageData(): void {
   for (const key of LOCAL_STORAGE_KEYS) {
@@ -130,17 +142,7 @@ export default function AccountPage() {
       try {
         // Clear IndexedDB (conversations)
         await deleteIndexedDBDatabase();
-        // Clear the WASM SQLite database (providers, users, etc.)
-        await new Promise<void>((resolve) => {
-          const req = indexedDB.deleteDatabase("hadrian-wasm");
-          req.onsuccess = () => resolve();
-          req.onerror = () => resolve();
-          req.onblocked = () => resolve();
-        });
-        // Unregister the service worker so its in-memory WASM gateway is
-        // discarded. A fresh worker will be registered on the next page load.
-        const registrations = await navigator.serviceWorker?.getRegistrations();
-        await Promise.all(registrations?.map((r) => r.unregister()) ?? []);
+        await clearWasmData();
         // Clear localStorage
         clearLocalStorageData();
 
@@ -185,14 +187,7 @@ export default function AccountPage() {
         // Clear local data first (before logout clears auth state)
         try {
           await deleteIndexedDBDatabase();
-          await new Promise<void>((resolve) => {
-            const req = indexedDB.deleteDatabase("hadrian-wasm");
-            req.onsuccess = () => resolve();
-            req.onerror = () => resolve();
-            req.onblocked = () => resolve();
-          });
-          const registrations = await navigator.serviceWorker?.getRegistrations();
-          await Promise.all(registrations?.map((r) => r.unregister()) ?? []);
+          await clearWasmData();
           clearLocalStorageData();
         } catch {
           // Continue with server deletion even if local clear fails