fix: address CTO review feedback on quick-find search (GH #97, GRO-134)

- Remove unused makeSelectChain function from search.test.ts (lint blocker)
- Fix handleClientClick/handlePetClick to navigate to /admin/clients?highlight={id}
  so the target client is identified in the URL rather than silently ignored
- Add console.warn for fetch errors in GlobalSearch instead of swallowing silently

Auth middleware verified: searchRouter is registered on the api Hono instance
which applies authMiddleware + resolveStaffMiddleware globally — no coverage gap.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

apps/api/src/__tests__/search.test.ts

@@ -34,16 +34,6 @@ vi.mock("@groombook/db", () => {
   const clients = tableProxy("clients");
   const pets = tableProxy("pets");
 
-  function makeSelectChain(results: unknown[]): unknown {
-    const chain: Record<string, unknown> = {};
-    const terminal = () => Promise.resolve(results);
-    chain.from = () => chain;
-    chain.innerJoin = () => chain;
-    chain.where = () => chain;
-    chain.limit = terminal;
-    return chain;
-  }
-
   return {
     getDb: () => ({
       select: (_fields?: unknown) => {

apps/web/src/components/GlobalSearch.tsx

@@ -52,8 +52,8 @@ export function GlobalSearch() {
           setResults(data);
           setOpen(true);
         }
-      } catch {
+      } catch (err) {
-        // ignore fetch errors
+        console.warn("GlobalSearch: fetch error", err);
       } finally {
         setLoading(false);
       }
@@ -83,13 +83,13 @@ export function GlobalSearch() {
   function handleClientClick(client: ClientResult) {
     setOpen(false);
     setQuery("");
-    navigate("/admin/clients");
+    navigate(`/admin/clients?highlight=${client.id}`);
   }
 
   function handlePetClick(pet: PetResult) {
     setOpen(false);
     setQuery("");
-    navigate("/admin/clients");
+    navigate(`/admin/clients?highlight=${pet.clientId}`);
   }
 
   const hasResults = results && (results.clients.length > 0 || results.pets.length > 0);