review-code

Review changed source files for design and structural quality issues.

Updated Jun 3, 2026
One-click install
npx skills add https://github.com/0xdeafcafe/skills --skill review-code-0xdeafcafe
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review-code
Source: https://github.com/0xdeafcafe/skills/tree/main/skills/review-code
Command: npx skills add https://github.com/0xdeafcafe/skills --skill review-code-0xdeafcafe

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill helps you review the architectural quality of changed code, focusing on issues that linters and formatters usually miss. It highlights design problems in touched files such as mixed responsibilities, weak modular boundaries, poor naming, dead code, duplication, and files that have grown too large or dense.

Core Features & Use Cases

  • Design-focused code review: Audits changed files for single-responsibility violations, layering problems, structural smells, and readability issues.
  • Scoped to real changes: Prioritizes PR diffs, working tree changes, or an explicit file list so the review stays relevant to the current work.
  • Structured findings output: Emits findings in a strict schema suitable for orchestration by downstream review and fix workflows.
  • Use case: When a pull request passes linting but still feels messy or overgrown, use this Skill to identify which files should be split, renamed, simplified, or cleaned up before merge.

Quick Start

Ask the assistant to run review-code on the current pull request to produce design findings for the changed files only.

Frequently Asked Questions about review-code

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

FAQPage Schema
How do I review pull request code for architecture and design issues that linters miss?

To review pull request code for architecture and design issues that linters miss, use a design-focused code review tool to audit changed files for single-responsibility violations, poor modularity, and structural smells. It outputs structured findings without applying fixes.

What is the best way to find files that need to be split or simplified before merging a working tree diff?

The best way to find files that need to be split or simplified before merging a working tree diff is to run a design audit on the changed files. It identifies dense files, mixed responsibilities, and duplication, helping you clean up code before merge.

Can I generate structured code review findings for downstream automation workflows?

Yes, you can generate structured code review findings for downstream automation workflows. This audit process emits results in a strict schema format, allowing downstream review and fix workflows to consume and orchestrate the identified design issues.

Does this code review approach work with explicit file lists and exclude generated or vendored files?

Yes, this code review approach works with explicit file lists and excludes generated or vendored files. It performs read-only analysis on targeted files, maintains repository convention awareness, and focuses only on the structural quality of the provided source files.

When should I use a design-focused code review instead of an automated linter?

You should use a design-focused code review instead of an automated linter when your pull request passes linting but still feels messy or overgrown. It catches judgment-based issues like weak modular boundaries, poor naming, and dead code that automated formatters miss.

Why does my pull request pass automated linting but still have readability and modularity problems?

Your pull request passes automated linting but still has readability and modularity problems because linters do not reliably catch judgment-based design flaws. A dedicated code design audit is required to identify responsibility boundary violations, layering issues, and structural smells.