code-review-and-quality

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

2|Updated Jul 25, 2026
One-click install
npx skills add https://github.com/ankaboot-source/boucle --skill code-review-and-quality-ankaboot-source
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-and-quality
Source: https://github.com/ankaboot-source/boucle/tree/main/.jcode/skills/code-review-and-quality
Command: npx skills add https://github.com/ankaboot-source/boucle --skill code-review-and-quality-ankaboot-source

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Code merged without structured review accumulates defects, security vulnerabilities, and architectural debt. 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 for 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 Splitting Guidance: Defines thresholds for reviewable change sizes and provides strategies (stacking, horizontal, vertical) for splitting oversized PRs. - Use Case: Before merging a pull request that adds a new API endpoint, run this review to verify edge-case handling, check for N+1 queries and injection risks, confirm tests cover the behavior, and produce a structured verdict of approve or request changes. ## Quick Start Review this pull request across correctness, readability, architecture, security, and performance, and give me severity-labeled findings with an approve or request-changes 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?

Review a pull request by first understanding its intent, then checking tests, then evaluating the implementation across correctness, readability, architecture, security, and performance. Label each finding by severity and finish with an approve or request-changes verdict.

What should a code review checklist include?

A code review checklist should cover correctness against the spec, edge case and error handling, test coverage, naming and readability, architectural fit, security checks like input validation and secrets, and performance concerns like N+1 queries and missing pagination.

How large should a pull request be for effective review?

A pull request of around 100 changed lines is ideal, and up to 300 lines is acceptable for a single logical change. Around 1000 lines is too large and should be split using stacking, file-group, horizontal, or vertical splitting strategies.

How do I review dependency upgrades safely?

Review dependency upgrades by reading the changelog rather than trusting semver, upgrading one dependency per change, verifying with a green test suite before and after, and reviewing the lockfile diff for unexpected transitive changes.

When should a code review request changes instead of approving?

Request changes when there are critical issues like security vulnerabilities, data loss, or broken functionality, or when required findings are unresolved. Approve when the change improves overall code health even if it is not perfect.