code-review-and-quality

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

1|Updated Jul 5, 2026
One-click install
npx skills add https://github.com/yersonargotev/packy --skill code-review-and-quality-yersonargotev
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review-and-quality
Source: https://github.com/yersonargotev/packy/tree/main/bundle/skills/code-review-and-quality
Command: npx skills add https://github.com/yersonargotev/packy --skill code-review-and-quality-yersonargotev

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Code merged without structured review accumulates correctness bugs, security vulnerabilities, architectural drift, and unreadable logic. This Skill enforces a consistent multi-axis review process so every change is evaluated against the same quality gates before entering the main branch. ## Core Features & Use Cases - Five-Axis Review: Evaluates every change across correctness, readability, architecture, security, and performance with concrete checklists per axis. - Severity-Labeled Feedback: Categorizes findings as Critical, Required, Nit, Optional, or FYI so authors know exactly what must be addressed before merge. - Change Sizing and Dependency Discipline: Provides thresholds for splitting large changes, guidance for writing standalone change descriptions, and a workflow for reviewing dependency upgrades one package at a time with changelog and lockfile review. - Use Case: Before merging a pull request that adds a new API endpoint, run the review checklist to verify tests cover edge cases, inputs are validated at the boundary, no N+1 queries exist, and the change description stands alone in version control history. ## Quick Start Review the current uncommitted changes using the five-axis code review checklist and report findings with severity labels.

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 the five-step process: understand the change context, review tests first, walk the implementation across the five axes, categorize findings with severity labels, and verify the author's testing story. Approve only when the change improves overall code health.

What should a code review checklist include?▼

A review checklist should cover correctness against the spec, edge case and error handling, naming and readability, architectural fit, security checks like input validation and secrets, and performance concerns like N+1 queries and missing pagination. It should end with a clear approve or request-changes verdict.

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

Around 100 changed lines is ideal and reviewable in one sitting; 300 lines is acceptable for a single logical change; 1000 lines is too large and should be split. Splitting strategies include stacking dependent changes, grouping by file, or slicing horizontally or vertically.

How do I review dependency upgrades safely?▼

Read the changelog rather than trusting semver, upgrade one dependency per change so failures are attributable, verify with a green test suite before and after, and review the lockfile diff for transitive changes. Never merge bulk bump PRs without per-package isolation.

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, or when Required findings are unresolved. Approve when the change improves overall code health even if it is not perfect.

Does AI-generated code need the same review as human code?▼

AI-generated code needs more scrutiny, not less, because it is confident and plausible even when wrong. Apply the same five-axis review, verify tests actually test behavior, and never rubber-stamp with an unexamined approval.