code-review

Review code changes for architectural conformance and quality gates.

17|1|Updated Dec 7, 2025
One-click install
npx skills add https://github.com/mhylle/claude-skills-collection --skill code-review-mhylle
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/mhylle/claude-skills-collection/tree/main/skills/code-review
Command: npx skills add https://github.com/mhylle/claude-skills-collection --skill code-review-mhylle

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

Code changes frequently drift away from architectural decisions and project standards. This Skill provides a formal, repeatable review to catch deviations before changes reach production.

Core Features & Use Cases

  • Service delegation verification: ensures controllers delegate to services and avoid business logic in controllers.
  • Framework standards compliance: checks patterns like NestJS DI, config usage, logging, and error handling.
  • ADR compliance and plan synchronization: verifies documentation of decisions and alignment with plan tasks.
  • Use Case: During a PR review or implement-phase gate, run the review to produce a pass/fail report.

Quick Start

  • Run the code-review skill on the changed files to verify architectural conformance, ADR alignment, and plan synchronization.

Frequently Asked Questions about code-review

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

FAQPage Schema
How do I enforce architectural decisions during a code review?

You can enforce architectural decisions during a code review by running a structured quality gate that verifies service delegation, framework standards, and ADR compliance against changed files to output a pass/fail report.

How do I verify NestJS dependency injection and framework standards in pull requests?

To verify NestJS dependency injection and framework standards in pull requests, run an automated review that checks controllers for proper service delegation, config usage, logging, and error handling patterns.

What is a quality gate pipeline for code implementation reviews?

A quality gate pipeline for code implementation reviews is a formal, repeatable process applied during the implement-phase to catch code deviations from project standards and architectural decisions before production.

Can I manually invoke a code review to check plan synchronization?

Yes, you can manually invoke a code review to check plan synchronization, verifying that code changes align with documented architectural decisions and planned implementation tasks.

How do I prevent business logic from being written in NestJS controllers?

To prevent business logic in NestJS controllers, apply a service delegation verification check during your code review to ensure controllers strictly delegate operations to underlying services.

What are the limitations of automated ADR compliance reviews?

Automated ADR compliance reviews are limited to verifying documented decisions and code alignment; they cannot detect undocumented architectural drift or evaluate the technical validity of the decisions themselves.