code-review-expert

Identify SOLID violations, architectural smells, and security risks in git changes.

Updated Mar 14, 2026
One-click install
npx skills add https://github.com/akoken/one-piece --skill code-review-expert-akoken
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-expert
Source: https://github.com/akoken/one-piece/tree/main/skills/code-review-expert
Command: npx skills add https://github.com/akoken/one-piece --skill code-review-expert-akoken

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Identify and report SOLID violations, architectural smells, security risks, and actionable improvements in the current git changes.

Core Features & Use Cases

  • Structured review of changes with focus on SOLID principles, architecture, security, and maintainability.
  • Provides actionable improvements and recommended next steps, including risk levels and follow-up tasks.
  • Use during code reviews or pull requests to enforce consistency and quality.

Quick Start

Review the current PR changes for SOLID, architecture, and security risks and propose actionable improvements.

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 a pull request for SOLID violations and architectural smells?

To review a pull request for SOLID violations and architectural smells, analyze the current git changes to identify structural issues, categorize them by severity, and generate actionable improvements for maintainability and correctness.

What is the best way to check git changes for security risks before merging?

Checking git changes for security risks involves evaluating the diff to identify vulnerabilities, categorizing them by severity, and providing clear guidance with safe remediation steps to ensure correctness before merging.

Does this code review approach work across different programming languages and modules?

Yes, this code review approach works across different programming languages and modules by analyzing the structural impact of git changes on SOLID principles, architecture, and security rather than relying on language-specific syntax rules.

How do I plan the safe removal and iteration of code identified with architectural smells?

To plan the safe removal and iteration of code with architectural smells, use structured workflow prompts that define risk levels, outline follow-up tasks, and provide clear guidance for safe remediation of the identified violations.

Why does my pull request review need predefined severity levels for code violations?

Predefined severity levels are needed in a pull request review to prioritize SOLID violations, architectural smells, and security risks accurately, ensuring that critical correctness and maintainability issues are addressed before minor improvements.

When should I not rely on automated code review for my git diff?

You should not rely on automated code review for your git diff when evaluating highly specialized domain logic, as the review focuses broadly on SOLID principles, architectural smells, and security risks rather than business-specific correctness.