Senior Code Review Skill

This skill reviews code with the mindset of an experienced engineer who cares about production quality, not just style feedback.

It looks for the issues that usually matter most before code gets merged, including security risks, logic bugs, performance problems, weak error handling, and code that may be hard to maintain later.

What it does

• Reviews diffs, pull requests, and changed files
• Checks security, correctness, performance, and readability
• Prioritizes serious issues before minor cleanup
• Suppresses small nits when there are blocking problems
• Gives clear merge guidance such as approve, fix before merge, or block
• Calls out at least one thing the code does well

Why it is useful

AI generated code can look correct while still hiding risky edge cases. This skill helps reduce review time by focusing attention on the parts of the code that deserve a closer look.

It is especially useful when you want a second set of eyes before merging work, shipping a feature, or trusting code that came from an AI coding assistant.

Best for

• Pull request review
• AI generated code review
• Security focused review
• Performance sensitive code
• Production readiness checks
• Teams that want clearer review priorities

Review style

The review is direct, practical, and organized by importance. It is not designed to nitpick. It is designed to help developers find real problems faster and ship better code with more confidence.