review

name	review
description	Review changed code in this repo against the project's review checklist (readability, maintainability, performance, security, consistency).

Context

• Current branch: !git branch --show-current
• Diff against base: !git diff --stat $(git merge-base HEAD v2)..HEAD
• Unstaged changes: !git diff --stat

How to review

Read the diff, then evaluate each changed file against the categories below. Report findings as a bulleted list grouped by category. Skip categories with no findings rather than padding with "looks good".

Readability

• Function length under ~50 lines.
• Nesting no deeper than 3–4 levels.
• Avoid let where const works. Avoid any.
• Extract repeated or multi-part conditions into named variables.
• Lift hard-coded values into constants.

Maintainability

• One responsibility per function/component.
• Watch for bloated props.

Performance

• No unnecessary re-renders or recomputation.
• Use early returns.

Security

• Validate user input at boundaries.
• Guard against XSS and similar OWASP Top 10 issues.
• No console.log or debug code left in.
• No hard-coded secrets.

Consistency

• Follow existing patterns in the codebase before introducing new ones.
• Verify the change adheres to AGENTS.md (component design principles, styling rules, code style).