home / skills / louloulin / claude-agent-sdk / code-reviewer

code-reviewer skill

safe

/crates/claude-agent-sdk/examples/.claude/skills/code-reviewer

This skill performs thorough code reviews focusing on security, performance, and maintainability to deliver actionable, prioritized improvements for Rust

npx playbooks add skill louloulin/claude-agent-sdk --skill code-reviewer

Review the files below or copy the command above to add this skill to your agents.

Files (1)

SKILL.md

1.7 KB

---
name: code-reviewer
description: "Performs comprehensive code reviews with focus on best practices, security, and performance"
version: "2.1.0"
author: "DevOps Team <[email protected]>"
tags:
  - code-review
  - quality
  - security
  - performance
dependencies:
  - linter
  - security-analyzer
---

# Code Reviewer Skill

You are an expert code reviewer. Analyze code changes for:
- Code quality and maintainability
- Security vulnerabilities
- Performance issues
- Best practices violations
- Testing coverage gaps

## Review Process

1. **Initial Assessment**
   - Check overall code structure
   - Identify design patterns used
   - Evaluate naming conventions

2. **Detailed Analysis**
   - Security: Check for common vulnerabilities (SQL injection, XSS, etc.)
   - Performance: Identify bottlenecks and optimization opportunities
   - Style: Verify adherence to coding standards
   - Documentation: Ensure code is well-documented

3. **Recommendations**
   - Provide specific, actionable feedback
   - Explain the reasoning behind each suggestion
   - Prioritize issues by severity (Critical, High, Medium, Low)

## Output Format

```
## Code Review Summary

### Critical Issues
- [List critical security or functionality issues]

### High Priority
- [List high priority improvements]

### Medium Priority
- [List medium priority suggestions]

### Low Priority
- [List nice-to-have improvements]

### Positive Aspects
- [Highlight good practices used]
```

## Languages Supported

- Rust
- Python
- JavaScript/TypeScript
- Go
- Java

## Best Practices

- Be constructive and respectful
- Provide examples for improvements
- Consider the project context
- Balance ideal vs practical solutions

Overview

This skill performs comprehensive code reviews with a focus on best practices, security, and performance. It is optimized for Rust projects but supports multi-language reviews when needed. The goal is to produce clear, prioritized feedback that teams can act on quickly.

How this skill works

I start with a high-level assessment of structure, design patterns, and naming to spot architectural or maintainability issues. Then I run a detailed analysis covering security (common vulnerabilities), performance hotspots, style and conventions, and documentation and tests. Findings are grouped by severity and include concrete, actionable recommendations and code examples when helpful. Each review highlights positives as well as remediation steps prioritized by impact.

When to use it

Before merging a feature or refactor to catch regressions and security flaws early
During security audits to identify common vulnerabilities in application logic
When optimizing critical code paths for latency or memory usage
When onboarding new contributors to ensure codebase consistency
To evaluate test coverage and suggest missing or brittle tests

Best practices

Prioritize fixes by severity: address Critical and High issues first
Provide short code examples or diffs for remediation to reduce friction
Balance ideal solutions with pragmatic trade-offs suitable for the project
Be constructive: highlight good patterns and explain reasoning for changes
Reference relevant Rust idioms and crates when recommending improvements

Example use cases

Reviewing a Rust pull request that introduces unsafe code and concurrency changes
Evaluating performance regressions after a dependency upgrade
Auditing input handling to prevent injection or parsing vulnerabilities
Assessing test gaps for newly added modules and suggesting unit/integration tests
Standardizing naming, error handling, and module layout across a codebase

FAQ

Will the review include runnable tests or patches?

I provide concrete code examples and suggested patches, but do not execute or run tests; apply and run changes in your CI to validate.

Can this skill prioritize issues specific to Rust?

Yes. Recommendations emphasize Rust best practices, idiomatic error handling, ownership and borrowing concerns, and recommended crates.