name	code-review
description	Review code for correctness, maintainability, security, accessibility, and test coverage.

Code Review

Use this skill when asked to review generated code or compare agent output.

Correctness: behavior matches the request and handles edge cases.
Architecture: responsibilities are separated and local conventions are followed.
Safety: inputs are validated and secrets are not exposed.
Maintainability: names are clear, functions stay focused, and duplication is justified.
Tests: happy paths, error paths, and regressions are covered.
Accessibility: UI code uses semantic markup, labels, keyboard support, and visible errors.

Group findings by severity:

Must Fix: bugs, security issues, broken tests, or violated requirements.
Should Fix: maintainability issues, missing tests, unclear naming, or brittle behavior.
Consider: optional improvements or tradeoffs.

For each finding, include the file, location if known, issue, and suggested fix.

name	code-review
description	Review code for correctness, maintainability, security, accessibility, and test coverage.

Use this skill when asked to review generated code or compare agent output.

Correctness: behavior matches the request and handles edge cases.
Architecture: responsibilities are separated and local conventions are followed.
Safety: inputs are validated and secrets are not exposed.
Maintainability: names are clear, functions stay focused, and duplication is justified.
Tests: happy paths, error paths, and regressions are covered.
Accessibility: UI code uses semantic markup, labels, keyboard support, and visible errors.

Group findings by severity:

Must Fix: bugs, security issues, broken tests, or violated requirements.
Should Fix: maintainability issues, missing tests, unclear naming, or brittle behavior.
Consider: optional improvements or tradeoffs.

For each finding, include the file, location if known, issue, and suggested fix.

code-review