code-review-and-quality

Reviews code changes across correctness, readability, architecture, security, and performance axes.

Updated Apr 21, 2026
One-click install
npx skills add https://github.com/alissonpef/copilot_agent_skills --skill code-review-and-quality-alissonpef
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review-and-quality
Source: https://github.com/alissonpef/copilot_agent_skills/tree/main/.github/skills/code-review-and-quality
Command: npx skills add https://github.com/alissonpef/copilot_agent_skills --skill code-review-and-quality-alissonpef

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Code changes often get merged with superficial reviews that only check whether tests pass, letting bugs, security vulnerabilities, and architectural problems slip into the main branch. This Skill enforces a structured, multi-dimensional review process so every change is evaluated consistently before merge. ## Core Features & Use Cases - Five-Axis Review: Evaluates every change across correctness, readability, architecture, security, and performance with concrete checklists for each dimension. - Severity-Labeled Feedback: Categorizes findings as Critical, Nit, Optional, or FYI so authors know exactly what must be fixed versus what is a suggestion. - Change Sizing and Splitting Guidance: Defines target change sizes (~100 lines) and provides stacking, horizontal, and vertical splitting strategies for oversized PRs. - Use Case: Before merging a pull request, run this review to verify tests cover the change, check for N+1 queries and injection vulnerabilities, confirm the change description stands alone in version control history, and produce a verdict of approve or request changes. ## Quick Start Review the current uncommitted changes or the specified pull request across all five quality axes and produce a severity-labeled review report with a merge verdict.

Frequently Asked Questions about code-review-and-quality

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review a pull request before merging?▼

Follow a five-step process: understand the change's intent, review the tests first, walk through the implementation across five quality axes, categorize findings by severity, and verify the author's testing story. Approve only when the change improves overall code health.

What should a code review checklist include?▼

A thorough checklist covers correctness (edge cases, error paths), readability (clear naming, simple logic), architecture (pattern consistency, no circular dependencies), security (input validation, no secrets), and performance (no N+1 queries, pagination on lists).

How large should a pull request be for effective review?▼

Target around 100 lines changed for a change reviewable in one sitting. Up to 300 lines is acceptable for a single logical change, but 1000 lines should be split using stacking, file-group, horizontal, or vertical splitting strategies.

How do I handle disagreements during code review?▼

Resolve disputes with a hierarchy: technical facts and data override opinions, style guides are authoritative on style, design is judged on engineering principles, and codebase consistency is acceptable if it does not degrade health. Comment on code, not people.

When should a code review request changes instead of approving?▼

Request changes when Critical issues exist, such as security vulnerabilities, data loss risks, or broken functionality. Do not accept promises to fix issues later; require cleanup before merge unless it is a genuine emergency.