code-review-assistant

AsiaOstrich's avatarfrom AsiaOstrich

Systematic code review checklist and pre-commit quality gates for PRs.Use when: reviewing pull requests, checking code quality, before committing code.Keywords: review, PR, pull request, checklist, quality, commit, 審查, 檢查, 簽入.

⭐43stars🔀8forks📁View on GitHub🕐Updated Dec 24, 2025
[Code Review]

When & Why to Use This Skill

The Code Review Assistant is a comprehensive Claude skill designed to streamline the pull request process through systematic checklists and pre-commit quality gates. It helps development teams maintain high code standards by providing structured feedback categories—covering functionality, security, and performance—alongside a standardized commenting system to clearly differentiate between blocking issues and minor suggestions.

Use Cases

  • Standardizing Pull Request Reviews: Utilize structured comment prefixes like BLOCKING, IMPORTANT, and SUGGESTION to provide clear, actionable feedback that reduces friction and improves communication between reviewers and authors.
  • Pre-Commit Quality Assurance: Implement a rigorous verification checklist before merging code to ensure builds succeed, all tests pass, and project-specific coding standards are strictly followed.
  • Security and Performance Auditing: Systematically evaluate code changes for potential vulnerabilities (e.g., SQL injection) and performance bottlenecks using targeted review categories and detailed guidelines.
  • Team Alignment and Onboarding: Establish a unified code review language and culture across the engineering team, ensuring consistent quality and architectural integrity regardless of the individual reviewer.
namecode-review-assistant
description|
Use whenreviewing pull requests, checking code quality, before committing code.
Keywordsreview, PR, pull request, checklist, quality, commit, 審查, 檢查, 簽入.

Code Review Assistant

This skill provides systematic checklists for code review and pre-commit verification.

Quick Reference

Comment Prefixes

Prefix Meaning Action Required
❗ BLOCKING Must fix before merge 🔴 Required
⚠️ IMPORTANT Should fix, but not blocking 🟡 Recommended
💡 SUGGESTION Nice-to-have improvement 🟢 Optional
❓ QUESTION Need clarification 🔵 Discuss
📝 NOTE Informational, no action ⚪ Informational

Review Checklist Categories

  1. Functionality - Does it work?
  2. Design - Right architecture?
  3. Quality - Clean code?
  4. Readability - Easy to understand?
  5. Tests - Adequate coverage?
  6. Security - No vulnerabilities?
  7. Performance - Efficient?
  8. Errors - Properly handled?
  9. Docs - Updated?
  10. Dependencies - Necessary?

Pre-Commit Checklist

  • Build succeeds (zero errors, zero warnings)
  • All tests pass
  • Code follows project standards
  • No security vulnerabilities
  • Documentation updated
  • Branch synced with target

Detailed Guidelines

For complete standards, see:

Example Review Comments

❗ BLOCKING: Potential SQL injection vulnerability here.
Please use parameterized queries instead of string concatenation.

⚠️ IMPORTANT: This method is doing too much (120 lines).
Consider extracting validation logic to a separate method.

💡 SUGGESTION: Consider using a Map here instead of an array for O(1) lookup.

❓ QUESTION: Why are we using setTimeout here instead of async/await?

📝 NOTE: This is a clever solution! Nice use of reduce here.

Core Principles

  1. Be Respectful - Review code, not the person
  2. Be Thorough - Check functionality, not just syntax
  3. Be Timely - Review within 24 hours
  4. Be Clear - Explain WHY, not just WHAT

Configuration Detection

This skill supports project-specific configuration.

Detection Order

  1. Check CONTRIBUTING.md for "Disabled Skills" section
    • If this skill is listed, it is disabled for this project
  2. Check CONTRIBUTING.md for "Code Review Language" section
  3. If not found, default to English

First-Time Setup

If no configuration found and context is unclear:

  1. Ask the user: "This project hasn't configured code review language. Which option would you like? (English / 中文)"
  2. After user selection, suggest documenting in CONTRIBUTING.md:
## Code Review Language

This project uses **[chosen option]** for code review comments.
<!-- Options: English | 中文 -->

Configuration Example

In project's CONTRIBUTING.md:

## Code Review Language

This project uses **English** for code review comments.
<!-- Options: English | 中文 -->

### Comment Prefixes
BLOCKING, IMPORTANT, SUGGESTION, QUESTION, NOTE

License: CC BY 4.0 | Source: universal-dev-standards