code-reviewer

Review pull request changes for security, quality, performance, testing, and documentation.

1|1|Updated Feb 5, 2026
One-click install
npx skills add https://github.com/ntk148v/skills --skill code-reviewer-ntk148v
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-reviewer
Source: https://github.com/ntk148v/skills/tree/main/skills/code-reviewer
Command: npx skills add https://github.com/ntk148v/skills --skill code-reviewer-ntk148v

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Code Reviewer helps teams prevent defects and security weaknesses by applying a consistent, phase-based checklist that ensures changes are correct, safe, maintainable, and ready for merge.

Core Features & Use Cases

  • Security-first review: Identifies common vulnerability classes such as injection, XSS, CSRF, SSRF, IDOR, and path traversal patterns, and checks authentication/authorization and data exposure.
  • Quality, robustness, and maintainability checks: Evaluates structure (single responsibility, DRY), error handling, type safety, and naming conventions to reduce long-term maintenance cost.
  • Performance and testing validation: Flags inefficient algorithms and problematic database patterns (e.g., N+1), and verifies coverage and edge-case/error tests.
  • Documentation review and feedback format: Ensures public APIs and complex logic are documented and produces constructive, prioritized feedback templates.

Quick Start

Use the code-reviewer skill to review the pull request changes you are about to merge, focusing on security first, then quality, performance, testing, and documentation.

Frequently Asked Questions about code-reviewer

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

FAQPage Schema
How do I check a pull request for security vulnerabilities before merging?

To check a pull request for security vulnerabilities, apply a security-first review workflow that identifies injection, XSS, CSRF, SSRF, IDOR, and path traversal patterns, alongside verifying authentication and authorization logic to ensure changes are safe for merge.

What is the best way to automate code quality checks on proposed code changes?

The best way to automate code quality checks is applying structured criteria that evaluate single responsibility, DRY principles, error handling, type safety, and naming conventions, which systematically reduces long-term maintenance costs across multiple project structures.

How do I detect performance regressions and N+1 database issues during a code review?

Detect performance regressions and N+1 database issues by applying performance heuristics during code review that flag inefficient algorithms and problematic database usage patterns within the proposed changes before they reach production.

Can I use static analysis to verify test coverage and edge cases for merge requests?

Yes, you can use static analysis to verify test coverage by evaluating proposed changes for sufficient coverage, edge cases, and error scenarios, ensuring robust testing validation before approving merge requests.

Does a pre-merge code review ensure documentation and changelog validation?

A pre-merge code review ensures documentation validation by checking that public APIs and complex logic are properly documented, and by producing constructive, prioritized feedback templates to communicate missing changelog entries.

What should I review first when evaluating code changes for correctness and maintainability?

When evaluating code changes for correctness and maintainability, review security vulnerabilities first, then proceed through quality, performance, testing, and documentation phases to ensure comprehensive pre-merge assurance.