React Review Audit

[reactreview]

⚠️ 16 warnings

Copy as prompt

Fix the following React Review diagnostics in my codebase.

## Warnings (16)

1. [warning] no-unknown-property — src/app/components/Navbar.jsx:16
   Unknown property found

2. [warning] no-unknown-property — src/app/components/OutputDisplay.jsx:19
   Unknown property found

3. [warning] design-no-three-period-ellipsis — src/app/components/OutputDisplay.jsx:10
   Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`)

4. [warning] no-unknown-property — src/app/page.js:91
   Unknown property found

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

6. [warning] design-no-three-period-ellipsis — src/app/page.js:10
   Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`)

7. [warning] design-no-three-period-ellipsis — src/app/page.js:15
   Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`)

8. [warning] prefer-useReducer — src/app/page.js:21
   Component "Home" has 5 useState calls — consider useReducer for related state

9. [warning] rerender-state-only-in-handlers — src/app/page.js:22
   useState "code" is updated but never read in the component's return — use useRef so updates don't trigger re-renders

10. [warning] rendering-hydration-no-flicker — src/app/page.js:29
   useEffect(setState, []) on mount causes a flash — consider useSyncExternalStore or suppressHydrationWarning

11. [warning] no-unknown-property — src/app/components/CodeEditor.jsx:99
   Unknown property found

12. [warning] no-effect-chain — src/app/components/CodeEditor.jsx:33
   useEffect reacts to "hasInitialized" which is set by another useEffect — chains of effects add an extra render per link and become rigid as code evolves. Compute what you can during render and write all related state inside the event handler that originally fires the chain

13. [warning] rerender-state-only-in-handlers — src/app/components/CodeEditor.jsx:25
   useState "editorInstance" is updated but never read in the component's return — use useRef so updates don't trigger re-renders

14. [warning] rerender-state-only-in-handlers — src/app/components/CodeEditor.jsx:26
   useState "hasInitialized" is updated but never read in the component's return — use useRef so updates don't trigger re-renders

15. [warning] rendering-hydration-no-flicker — src/app/components/CodeEditor.jsx:28
   useEffect(setState, []) on mount causes a flash — consider useSyncExternalStore or suppressHydrationWarning

16. [warning] nextjs-no-font-link — src/app/layout.js:15
   Loading Google Fonts via <link> — use next/font instead for self-hosting, zero layout shift, and no render-blocking requests

---

⚠️ Warnings (16)

Unknown property found _{· 4 in 4 files}

_{no-unknown-property}

Remove unknown property

File	Lines
src/app/components/Navbar.jsx	16
src/app/components/OutputDisplay.jsx	19
src/app/page.js	91
src/app/components/CodeEditor.jsx	99

Three-period ellipsis ("...") in JSX text — use the actual ellipsis character "…" (or `…`) _{· 3 in 2 files}

_{design-no-three-period-ellipsis}

Use the typographic ellipsis "…" (or …) instead of three periods — pairs with action-with-followup labels ("Rename…", "Loading…")

File	Lines
src/app/page.js	10, 15
src/app/components/OutputDisplay.jsx	10

useState "code" is updated but never read in the component's return — use useRef so updates don't trigger re-renders _{· 3 in 2 files}

_{rerender-state-only-in-handlers}

Replace useState with useRef when the value is only mutated and never read in render — ref.current = ... updates without re-rendering the component

File	Lines
src/app/components/CodeEditor.jsx	25, 26
src/app/page.js	22

useEffect(setState, []) on mount causes a flash — consider useSyncExternalStore or suppressHydrationWarning _{· 2 in 2 files}

_{rendering-hydration-no-flicker}

Use useSyncExternalStore(subscribe, getSnapshot, getServerSnapshot) or add suppressHydrationWarning to the element

File	Lines
src/app/page.js	29
src/app/components/CodeEditor.jsx	28

Page without metadata or generateMetadata export — hurts SEO _{· 1 in 1 file}

_{nextjs-missing-metadata}

Add export const metadata = { title: '...', description: '...' } or export async function generateMetadata()

File	Lines
src/app/page.js	1

Component "Home" has 5 useState calls — consider useReducer for related state _{· 1 in 1 file}

_{prefer-useReducer}

Group related state: const [state, dispatch] = useReducer(reducer, { field1, field2, ... })

File	Lines
src/app/page.js	21

useEffect reacts to "hasInitialized" which is set by another useEffect — chains of effects add an extra render per link and become rigid as code evolves. Compute what you can during render and write all related state inside the event handler that originally fires the chain _{· 1 in 1 file}

_{no-effect-chain}

Compute as much as possible during render (e.g. const isGameOver = round > 5) and write all related state inside the event handler that originally fires the chain. Each effect link adds an extra render and makes the code rigid as requirements evolve

File	Lines
src/app/components/CodeEditor.jsx	33

Loading Google Fonts via <link> — use next/font instead for self-hosting, zero layout shift, and no render-blocking requests _{· 1 in 1 file}

_{nextjs-no-font-link}

import { Inter } from "next/font/google" — self-hosted, zero layout shift, no render-blocking requests

File	Lines
src/app/layout.js	15

---

^{Reviewed by reactreview for commit 149babd. Configure here.}