code-review

Reviews code changes for structural quality, abstraction design, and maintainability regressions.

2|Updated Mar 29, 2015
One-click install
npx skills add https://github.com/ovisan/dotfiles --skill code-review-ovisan
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/ovisan/dotfiles/tree/main/.grok/skills/code-review
Command: npx skills add https://github.com/ovisan/dotfiles --skill code-review-ovisan

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Code reviews often rubber-stamp working implementations that quietly degrade a codebase through file sprawl, scattered conditionals, and leaky abstractions. This Skill enforces a strict, structure-first review standard that catches maintainability regressions before they merge. ## Core Features & Use Cases - Structural Quality Audit: Evaluates diffs for abstraction quality, modularity, type-boundary cleanliness, and architectural layering rather than surface-level style. - Spaghetti-Growth Detection: Flags ad-hoc conditionals, special-case branches, and feature logic leaking into shared paths as design problems. - File-Size Guardrails: Treats pushing a file past 1000 lines as a presumptive blocker requiring decomposition or explicit justification. - Use Case: Before merging a large feature branch, run this review to get high-conviction feedback on missed simplification opportunities, wrong-layer logic, and unnecessary wrappers, with concrete restructuring suggestions. ## Quick Start Ask the AI to run a strict code quality review of the current branch's changes using the code-review skill.

Frequently Asked Questions about code-review

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

FAQPage Schema
How do I run a strict code review on my pull request?

Invoke the code-review skill against your current branch's changes. It audits the diff for structural regressions, abstraction quality, and spaghetti growth, then returns prioritized findings with concrete restructuring suggestions.

What does a maintainability-focused code review check for?

It checks for files crossing 1000 lines, ad-hoc conditionals bolted onto existing flows, feature logic leaking into shared modules, unnecessary wrappers and casts, and missed opportunities to delete complexity through restructuring.

When should I use a strict structural review instead of a standard review?

Use it for deep quality audits of significant changes where long-term maintainability matters more than quick approval. Standard reviews catch bugs; this review catches design debt like tangled control flow and wrong-layer logic.

Why does the review block PRs that push files past 1000 lines?

Files crossing the 1000-line threshold are treated as a strong code-quality smell because large files resist decomposition and become harder to reason about. The reviewer must either decompose the code first or provide a compelling structural justification.

What are the limitations of an automated maintainability review?

It evaluates structure and design quality, not runtime correctness, security vulnerabilities, or test coverage. It also cannot verify whether a proposed restructuring truly preserves behavior, so suggested refactors still need test validation.