code-review-expert

Review Git diffs for structural regressions, security risks, and maintainability issues.

2|Updated Aug 4, 2025
One-click install
npx skills add https://github.com/zhu327/go-clean-arch --skill code-review-expert-zhu327
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review-expert
Source: https://github.com/zhu327/go-clean-arch/tree/main/.cursor/skills/code-review-expert
Command: npx skills add https://github.com/zhu327/go-clean-arch --skill code-review-expert-zhu327

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill helps you review code changes with a senior engineer lens so you can catch structural regressions, security risks, and avoidable complexity before merge.

Core Features & Use Cases

  • Diff-focused review: Analyze changed files, surface high-conviction findings, and separate critical issues from low-value nits.
  • Architecture and simplification checks: Identify spaghetti growth, boundary leaks, unnecessary wrappers, and missed code-judo opportunities that could delete complexity.
  • Security and reliability review: Flag auth gaps, unsafe data access, leaked internals, swallowed errors, race conditions, and other diff-visible production risks.

Quick Start

Ask for a code review of the current branch or a specific commit range and include any special context the reviewer should prioritize.

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 git diff for structural regressions and security risks?

To review a git diff for structural regressions, provide the branch comparison or commit range to surface security risks and maintainability issues before merging the pull request into production.

What is the best way to catch architectural boundary leaks in a pull request?

Catching architectural boundary leaks in a pull request requires analyzing the git diff context and repository structure to identify spaghetti growth, unnecessary wrappers, and missed code simplification opportunities.

Can I use a code review tool to flag auth gaps and race conditions in my branch?

Yes, you can use a code review tool to flag auth gaps, unsafe data access, swallowed errors, and race conditions by applying engineering judgment to the visible production risks in your branch diff.

Does diff-focused code review separate critical issues from low-value nits?

Diff-focused code review separates critical structural and security issues from low-value nits by surfacing high-conviction findings specifically from changed files within the specified commit range.

Do I need repository structure awareness to review hot code paths in a commit range?

Yes, you need repository structure awareness to review hot code paths in a commit range, as understanding architectural boundaries is required to apply correct engineering judgment about coupling and reliability.

When should I not use automated git diff analysis for code review?

You should not use automated git diff analysis alone when a code review requires deep runtime context or testing, as diff-visible production risks only cover structural, security, and maintainability boundaries.