React Review Audit

Fix the following React Review diagnostics in my codebase.

## Errors (1)

1. [error] effect-needs-cleanup — src/app/chat/[roomId]/page.tsx:187
   useEffect schedules `setTimeout(...)` but never returns a cleanup — leaks the registration on every re-run and on unmount. Return a cleanup function that calls clearTimeout(...)

## Warnings (33)

2. [warning] async-await-in-loop — src/lib/db.ts:52
   await inside a while-loop runs the calls sequentially — for independent operations, collect them and use `await Promise.all(items.map(...))` to run them concurrently

3. [warning] async-await-in-loop — src/lib/db.ts:66
   await inside a while-loop runs the calls sequentially — for independent operations, collect them and use `await Promise.all(items.map(...))` to run them concurrently

4. [warning] design-no-three-period-ellipsis — src/components/StatusBanner.tsx:38
   Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`)

5. [warning] design-no-three-period-ellipsis — src/components/StatusBanner.tsx:56
   Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`)

6. [warning] no-generic-handler-names — src/components/GitHubStars.tsx:26
   Non-descriptive handler name "handleClick" — name should describe what it does, not when it runs

7. [warning] no-cascading-set-state — src/hooks/useSocket.ts:45
   46 setState calls in a single useEffect — consider using useReducer or deriving state

8. [warning] no-autofocus — src/components/GifPicker.tsx:132
   The `autoFocus` attribute is found here, which can cause usability issues for sighted and non-sighted users.

9. [warning] nextjs-no-img-element — src/components/GifPicker.tsx:171
   Use next/image instead of <img> — provides automatic optimization, lazy loading, and responsive srcset

10. [warning] async-await-in-loop — src/hooks/useMessages.ts:165
   await inside a for…of loop runs the calls sequentially — for independent operations, collect them and use `await Promise.all(items.map(...))` to run them concurrently

11. [warning] no-autofocus — src/app/chat/[roomId]/page.tsx:320
   The `autoFocus` attribute is found here, which can cause usability issues for sighted and non-sighted users.

12. [warning] nextjs-missing-metadata — src/app/chat/[roomId]/page.tsx:1
   Page without metadata or generateMetadata export — hurts SEO

13. [warning] rerender-state-only-in-handlers — src/app/chat/[roomId]/page.tsx:79
   useState "joined" is updated but never read in the component's return — use useRef so updates don't trigger re-renders

14. [warning] no-giant-component — src/app/chat/[roomId]/page.tsx:73
   Component "ChatPage" is 354 lines — consider breaking it into smaller focused components

15. [warning] nextjs-no-client-side-redirect — src/app/chat/[roomId]/page.tsx:192
   router.push() in useEffect — use redirect() from next/navigation or handle navigation in an event handler

16. [warning] react-compiler-destructure-method — src/app/chat/[roomId]/page.tsx:192
   Destructure for clarity: `const { push } = useRouter()` then call `push(...)` directly — easier for React Compiler to memoize and clearer about which methods this component depends on

17. [warning] react-compiler-destructure-method — src/app/chat/[roomId]/page.tsx:213
   Destructure for clarity: `const { push } = useRouter()` then call `push(...)` directly — easier for React Compiler to memoize and clearer about which methods this component depends on

18. [warning] react-compiler-destructure-method — src/app/chat/[roomId]/page.tsx:254
   Destructure for clarity: `const { push } = useRouter()` then call `push(...)` directly — easier for React Compiler to memoize and clearer about which methods this component depends on

19. [warning] design-no-redundant-padding-axes — src/app/chat/[roomId]/page.tsx:397
   px-4 py-4 → use the shorthand p-4

20. [warning] rerender-state-only-in-handlers — src/components/ChatInput.tsx:35
   useState "recording" is updated but never read in the component's return — use useRef so updates don't trigger re-renders

21. [warning] no-generic-handler-names — src/components/ChatInput.tsx:200
   Non-descriptive handler name "handleChange" — name should describe what it does, not when it runs

22. [warning] js-index-maps — server/roomManager.ts:106
   array.findIndex() in a loop is O(n*m) — build a Map for O(1) lookups

23. [warning] nextjs-no-img-element — src/components/ChatBubble.tsx:40
   Use next/image instead of <img> — provides automatic optimization, lazy loading, and responsive srcset

24. [warning] nextjs-no-img-element — src/components/ChatBubble.tsx:50
   Use next/image instead of <img> — provides automatic optimization, lazy loading, and responsive srcset

25. [warning] no-inline-bounce-easing — src/components/TypingIndicator.tsx:7
   animate-bounce feels dated and tacky — use a subtle ease-out transform for natural deceleration

26. [warning] no-inline-bounce-easing — src/components/TypingIndicator.tsx:11
   animate-bounce feels dated and tacky — use a subtle ease-out transform for natural deceleration

27. [warning] no-inline-bounce-easing — src/components/TypingIndicator.tsx:15
   animate-bounce feels dated and tacky — use a subtle ease-out transform for natural deceleration

28. [warning] no-unknown-property — src/components/Antigravity.tsx:219
   Unknown property found

29. [warning] no-unknown-property — src/components/Antigravity.tsx:221
   Unknown property found

30. [warning] no-unknown-property — src/components/Antigravity.tsx:224
   Unknown property found

31. [warning] no-unknown-property — src/components/Antigravity.tsx:226
   Unknown property found

32. [warning] no-unknown-property — src/components/Antigravity.tsx:228
   Unknown property found

33. [warning] nextjs-missing-metadata — src/app/page.tsx:1
   Page without metadata or generateMetadata export — hurts SEO

34. [warning] react-compiler-destructure-method — src/app/page.tsx:25
   Destructure for clarity: `const { push } = useRouter()` then call `push(...)` directly — easier for React Compiler to memoize and clearer about which methods this component depends on