code-reviewer_skill

Overview

This skill performs comprehensive code reviews with a focus on correctness, performance, security, and maintainability. It is optimized for reviewing pull requests, merge requests, or any code changes, with special attention to Rust code patterns and idioms. Output is structured, actionable, and prioritized so reviewers and authors can iterate quickly.

How this skill works

The skill inspects the change set and evaluates intent, matching commits and descriptions to the actual code. It runs a structured checklist across correctness, performance, security, maintainability, and testing, and then produces a clear review: overall summary, high-priority issues, suggestions, and nitpicks. It also flags breaking changes, dependency additions, and documentation gaps.

When to use it

• Reviewing pull requests or merge requests before merging to mainline
• Assessing security or performance regressions introduced by recent changes
• Validating new features or refactors for Rust-specific best practices
• Checking that tests and CI cover new or modified code paths
• Evaluating third-party dependency additions or license changes

Best practices

• Start with the PR description and run tests locally to reproduce issues
• Prioritize critical bugs, security flaws, and breaking API changes first
• Give concrete, example-based suggestions rather than vague criticisms
• Verify error handling, edge cases, and resource management in Rust (lifetimes, ownership)
• Confirm CI, formatting, linting, and test coverage are passing before approval

Example use cases

• A Rust refactor that touches core modules—verify correctness and performance impact
• A new endpoint or API—confirm authentication, input validation, and docs
• Dependency upgrade—check for security advisories, license compatibility, and API changes
• Bugfix PR—ensure edge cases are handled and add unit tests for regressions
• Performance optimization—confirm algorithmic improvements and measure resource usage