react-hooks-effects-review

Scope the review to your changed files or git diff, then classify each Effect by what external system it synchronizes with. Check dependency arrays for missing reactive values, verify cleanup mirrors setup, and confirm no side effects run during render.

Stale closures occur when delayed callbacks like setTimeout, promises, or subscriptions read values captured from an older render. Fixes include functional state updates, complete dependency arrays, useEffectEvent where supported, or abort/ignore guards for async races.

Yes, the review records whether the project uses SSR, Server Components, Strict Mode, or React Compiler before analyzing. It flags browser-only API reads during SSR render and hydration-unsafe nondeterministic output like Date.now or Math.random.

Skip it when changes involve only styling, static markup, TypeScript types, routing config, or server-only code with no hook or render lifecycle surface. The review targets lifecycle, timing, and synchronization bugs, not general style.

Loops happen when an Effect synchronously sets state it also lists as a dependency, or writes values that change unstable dependencies. Fixes include deriving values during render, splitting overloaded Effects, or moving logic into event handlers.