code-review-expert

Review git changes for SOLID violations, security risks, and race conditions.

Updated Feb 13, 2026
One-click install
npx skills add https://github.com/sanjitsaluja/zugzwang-puzzle-trainer --skill code-review-expert-sanjitsaluja
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-expert
Source: https://github.com/sanjitsaluja/zugzwang-puzzle-trainer/tree/main/.agents/skills/code-review-expert
Command: npx skills add https://github.com/sanjitsaluja/zugzwang-puzzle-trainer --skill code-review-expert-sanjitsaluja

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill provides a comprehensive code review, identifying critical issues like security vulnerabilities, SOLID principle violations, performance bottlenecks, and error handling flaws, ensuring higher code quality and maintainability.

Core Features & Use Cases

  • Deep Code Analysis: Scans code changes for adherence to SOLID principles, architectural smells, and potential removal candidates.
  • Security & Reliability: Detects common security risks (XSS, injection, auth issues) and reliability concerns (race conditions, resource exhaustion).
  • Use Case: Before merging a pull request, use this Skill to get a senior engineer's perspective on potential bugs, security holes, and areas for refactoring, preventing costly production issues.

Quick Start

Review the current git changes for SOLID violations, security risks, race conditions, error handling issues, performance problems, and boundary condition bugs.

Frequently Asked Questions about code-review-expert

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

FAQPage Schema
How do I perform a code review on git changes for SOLID violations and security risks?

Code review for SOLID violations and security risks analyzes git diff changes with a senior engineer's lens, identifying architectural smells, race conditions, and error handling flaws before merging pull requests.

What does a comprehensive security and reliability code review cover?

A comprehensive security and reliability code review covers vulnerabilities like XSS and injection, detects race conditions, and identifies resource exhaustion risks to prevent costly production issues.

Can I use grep or ripgrep to find boundary condition bugs during code review?

You can use grep or ripgrep to gather context for code review, analyzing boundary condition bugs and performance bottlenecks by scanning the codebase before providing structured findings with severity levels.

What is the best way to find refactoring candidates in a pull request?

The best way to find refactoring candidates in a pull request is scanning code changes for adherence to SOLID principles and architectural smells, flagging potential removal candidates for higher maintainability.

Does this code review approach require a git diff to check for error handling flaws?

This code review approach requires a git diff to check for error handling flaws, analyzing the exact code changes against checklists for specific checks and providing structured findings with severity levels.

When do I need an automated code review for performance and security?

You need automated code review for performance and security before merging a pull request, ensuring higher code quality by catching boundary condition bugs, performance bottlenecks, and security holes early.