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.