Learn-skills.dev modern-best-practice-react-components
Build clean, modern React components that apply common best practices and avoid common pitfalls like unnecessary state management or useEffect usage
install
source · Clone the upstream repo
git clone https://github.com/NeverSight/learn-skills.dev
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/NeverSight/learn-skills.dev "$T" && mkdir -p ~/.claude/skills && cp -r "$T/data/skills-md/academind/ai-config/modern-best-practice-react-components" ~/.claude/skills/neversight-learn-skills-dev-modern-best-practice-react-components && rm -rf "$T"
manifest:
data/skills-md/academind/ai-config/modern-best-practice-react-components/SKILL.mdsource content
Writing React Components
We're using modern React (19+) and we're following common best practices focused on clarity, correctness, and maintainability.
Component Structure & Style
- PREFER small, focused components with a single responsibility
- PREFER named
components over arrow functionsfunction- Exception: anonymous callbacks, inline render props, and closures
- PREFER explicit return types and props typing (TypeScript) where applicable
- Keep components flat and readable; avoid deeply nested JSX
- Group related logic together (event handlers, derived values, helpers)
State Management
- AVOID
useEffect()- See the "You Might Not Need An Effect" guide for detailed guidance
- PREFER deriving values during render instead of synchronizing state
- Fetch data via TanStack Query (
)@tanstack/react-query
- AVOID unnecessary
oruseState()
usageuseReducer()- Derive state from props or other state when possible
- Localize state to the lowest possible component
- DO NOT mirror props in state unless absolutely necessary
- Prefer controlled components over syncing uncontrolled state
Rendering & Derivation
- PREFER computing derived values inline or via helper functions
- Use
sparingly and only for proven performance issuesuseMemo() - AVOID premature optimization
- Keep render logic deterministic and free of side effects
Event Handling
-
AVOID in-line event handlers in JSX
-
PREFER:
function handleClick() { // ... } <button onClick={handleClick} />; -
Over:
<button onClick={() => { /* ... */ }} />
-
-
Name handlers clearly (
,handleSubmit
,handleChange
)handleClose -
Keep handlers small; extract complex logic into helpers
Effects, Data, and Side Effects
- AVOID effects for:
- Derived state
- Data transformations
- Event-based logic that can live in handlers
- If side effects are unavoidable, keep them minimal, isolated, and well-documented
- Prefer framework-level or external abstractions (routers, data libraries) over raw effects
Props & Composition
- PREFER composition over configuration
- AVOID excessive boolean props; prefer expressive APIs
- Use
intentionally and document expected structurechildren - Keep prop names semantic and predictable
Performance & Stability
- PREFER stable references only when required (not by default)
- AVOID unnecessary memoization (
,memo
) unless absolutely requireduseCallback - Keep keys stable and meaningful when rendering lists
General Principles
- Write code for humans first, compilers second
- Prefer explicitness over cleverness
- Optimize for readability and long-term maintenance
- If a pattern feels complex, reconsider the component boundary