— Skill

Addy Code Review & Quality

“LGTM” is not a review — this skill forces multi-axis quality (correctness, readability, architecture, tests, security) with severity labels before code hits main.

What changes

Runs Addy Osmani’s five-axis code-review-and-quality skill so agents review diffs/PRs across correctness, readability, architecture, tests, and security — with severity-tagged findings.

When

Use Addy Osmani’s code-review-and-quality skill when a change needs a multi-axis quality review before merge — whether the author is you, another agent, or a human, and whether the diff is a PR or pasted inline.

Boundary vs cursor-pr-review: this card is multi-axis quality review (correctness / readability / architecture / tests / security) that works across Agent Skills hosts. cursor-pr-review is Cursor Agent’s GitHub PR review flow (/review, /review-bugbot, /review-security) inside Cursor. Use Addy’s skill for pack-based five-axis depth on Multi hosts; use Cursor’s built-ins when you are already in Cursor’s PR review commands.

Good fits:

• Pre-merge review of agent-written code that “works” but may be shallow.
• Pasted diffs without a full Cursor PR context.
• Teams standardizing severity-labeled comments.

Skip it for pure security deep-dives (addy-security-and-hardening) or live browser diagnosis (addy-browser-testing-devtools).

How to start

Install with `npx skills add addyosmani/agent-skills --skill code-review-and-quality` (or whole pack). Point at the diff/PR and require five-axis findings with severity labels.

Use the code-review-and-quality skill on [PR URL | branch | pasted diff].
Run the five-axis review: correctness, readability, architecture, tests, security.
Label every finding: blocker / should-fix / nit. Cite file paths. No vague praise.
Do not merge. Contrast: this is multi-axis quality — not Cursor’s /review command flow.