code-review

Review code and pull requests for security, architecture, performance, and compliance.

1|Updated Nov 12, 2024

Search Tags:#code review #performance #pull request #security #compliance #remix #nestJS

Ready to run

npx skills add https://github.com/ak125/nestjs-remix-monorepo --skill code-review-ak125

💬 Prompt for ChatGPT / Claude Chat▼

Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/ak125/nestjs-remix-monorepo/tree/main/.claude/skills/code-review
Command: npx skills add https://github.com/ak125/nestjs-remix-monorepo --skill code-review-ak125

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes scripts (resource) and references (resource) components.

What problem does it solve?

This Skill streamlines the code review process, ensuring code adheres to security, architectural, performance, and business compliance standards, reducing the risk of bugs and vulnerabilities.

Core Features & Use Cases

Systematic Review: Applies a comprehensive checklist tailored to different code domains (Backend, Frontend, Payments, Migrations, Config).
Risk Assessment: Classifies changes by file count, modules touched, and critical areas like payments, assigning a risk level (Low, Medium, High).
Security & Performance Focus: Explicitly checks for common security vulnerabilities (SQL Injection, XSS) and performance anti-patterns (N+1 queries).
Use Case: When a Pull Request is opened, this Skill can be invoked to perform an automated review, flagging critical issues before human review, ensuring faster and safer merges.

Quick Start

Initiate a code review for the current Pull Request by running /code-review.

Frequently Asked Questions about code-review

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

FAQPage Schema

How do I automate a pull request review for security and performance issues?▼

Automated pull request review applies a domain-specific checklist to modified files, categorizing identified security and performance issues into blocking, high, and suggestion levels for safer merges.

What is the best way to assess risk in a monorepo pull request?▼

Assessing monorepo pull request risk classifies changes by file count, touched modules, and critical areas like payments, assigning Low, Medium, or High risk levels to prioritize review efforts effectively.

Does this code review checklist work with NestJS and Remix?▼

Yes, the code review checklist adapts to frameworks like NestJS and Remix, applying domain-specific checks for Backend, Frontend, Payments, Migrations, and Config to ensure architectural compliance.

How do I check for N+1 queries and bundle size in my code changes?▼

Checking for N+1 queries and bundle size involves applying performance-focused checks during pull request review, explicitly identifying performance anti-patterns and architectural risks based on modified files.

When do I need an automated code review for business compliance?▼

You need an automated code review for business compliance when a pull request touches critical areas like payments or config, ensuring code adheres to domain-specific standards and reducing vulnerability risks.