Code Review Expert

Review code changes for logic, security, and architecture issues.

51|6|Updated Mar 28, 2019
One-click install
npx skills add https://github.com/Mte90/dotfiles --skill code-review-expert-mte90
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: Code Review Expert
Source: https://github.com/Mte90/dotfiles/tree/main/.config/opencode/skills/common/code-review
Command: npx skills add https://github.com/Mte90/dotfiles --skill code-review-expert-mte90

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill provides a structured and expert-driven approach to code reviews, ensuring higher quality, security, and maintainability of software.

Core Features & Use Cases

  • Persona-Driven Reviews: Adopts a Principal Engineer mindset, focusing on critical aspects like logic, security, and architecture.
  • Structured Feedback: Organizes feedback into BLOCKER, MAJOR, and NIT categories for clarity.
  • Use Case: When submitting a Pull Request, use this Skill to get a thorough review that goes beyond superficial checks, identifying potential bugs, security vulnerabilities, and architectural flaws.

Quick Start

Use the Code Review Expert skill to review the attached code changes for security vulnerabilities and logic errors.

Frequently Asked Questions about Code Review Expert

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I perform a code review for security vulnerabilities and logic errors?

To perform a code review for security vulnerabilities and logic errors, adopt a Principal Engineer mindset to systematically analyze code changes, prioritizing architectural flaws and security risks over superficial style checks.

What is the best way to structure pull request feedback for high software quality?

The best way to structure pull request feedback for high software quality is to categorize identified issues into BLOCKER, MAJOR, and NIT severity levels, prioritizing substance and using guiding questions to prompt constructive improvements.

Does a persona-driven code review approach catch architectural flaws?

Yes, a persona-driven code review approach catches architectural flaws by enforcing best practices from a Principal Engineer perspective, focusing deeply on logic, security, and overall system architecture rather than just formatting.

How do I enforce best practices during a software engineering code review?

You enforce best practices during a software engineering code review by applying a structured checklist that requires specific output formats for identified issues, ensuring reviews prioritize architectural logic and security over stylistic preferences.

When do I need a structured checklist for code reviews?

You need a structured checklist for code reviews when submitting a pull request that requires thorough validation beyond superficial checks, specifically to identify potential bugs, security vulnerabilities, and architectural flaws.

Why prioritize architectural logic over style in a code review?

You prioritize architectural logic over style in a code review to ensure the long-term maintainability and security of the software, focusing expert-driven feedback on critical functional aspects rather than superficial formatting choices.