code-review-expert

Reviews git changes for architectural, security, reliability and code-quality issues.

Updated Aug 23, 2026
One-click install
npx skills add https://github.com/edupazogle/agentic_cockpit --skill code-review-expert-edupazogle
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-expert
Source: https://github.com/edupazogle/agentic_cockpit/tree/main/.agents/skills/agent-toolkit/code-review-expert
Command: npx skills add https://github.com/edupazogle/agentic_cockpit --skill code-review-expert-edupazogle

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

It helps prevent production incidents by performing a structured, senior-engineer style review of your current git changes, focusing on correctness, security, reliability, performance, and maintainability.

Core Features & Use Cases

  • SOLID + Architecture Review: Detect SRP/OCP/LSP/ISP/DIP violations and recommend minimal, safe refactors.
  • Security & Reliability Scan: Surface common web/runtime risks including injection, SSRF, auth gaps, secrets leakage, and race conditions.
  • Code Quality & Risk Checks: Identify error-handling failures, performance anti-patterns (e.g., N+1), and boundary-condition bugs.
  • Removal/Iteration Planning: Propose safe deletion candidates and incremental follow-up plans with verification steps.
  • Use Case: Before merging a PR, review the exact set of changed files to get prioritized findings (P0–P3) and a next-steps plan that defaults to review-only until you confirm.

Quick Start

Run /code-review-expert to review your current git changes and receive prioritized P0–P3 findings with an implementation confirmation prompt.

Frequently Asked Questions about code-review-expert

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

FAQPage Schema
How do I review git changes for security risks and SOLID violations before merging a PR?

Reviewing git changes for security risks and SOLID violations involves analyzing your current diff to detect architectural smells like SRP/OCP/LSP/ISP/DIP violations and common web vulnerabilities. The process outputs severity-ranked findings from P0 to P3 with safe refactor recommendations before merging a PR.

What is the best way to scan a git diff for performance anti-patterns and boundary-condition bugs?

Scanning a git diff for performance anti-patterns and boundary-condition bugs involves checking changed files against internal reference checklists to identify N+1 queries and error-handling failures. This yields prioritized code-quality findings and an incremental follow-up plan to ensure reliability.

How does a structured code review help prevent production incidents?

A structured code review prevents production incidents by applying a senior-engineer style analysis to current git changes, focusing on correctness, security, reliability, and performance. It systematically surfaces race conditions, auth gaps, and secrets leakage before the code reaches production.

Can I get safe removal and refactor guidance directly from my current commit changes?

Yes, you can get safe removal and refactor guidance directly from commit changes by analyzing the diff for architectural violations. The review proposes safe deletion candidates and minimal refactors with verification steps, defaulting to review-only until you explicitly confirm the implementation.

Does automated PR review cover reliability issues like race conditions and error handling?

Automated PR review covers reliability issues like race conditions and error-handling failures by scanning the git diff for runtime risks. It consistently applies internal checklists to produce severity-ranked findings and a structured next-steps plan for resolving these specific defects.