Teach Your Coding Agent to Review Code Without the Style Nitpicks

Описание и применение

Скрипт-способность "code-review-checklist" заменяет хаотический обзор кода, фокусируясь на пяти ключевых осях и ограничивая комментарии до десяти, чтобы выявлять реальные ошибки, а не стилистические нюансы.

Задача

A coding agent returned 23 comments on a 120‑line PR, most about style, while missing critical bugs such as an IndexError, SQL injection, and N+1 query issues.

Результат

The skill reduces review comments from 23 to 4, all actionable and focused on real defects, while delegating style to a linter.

Рабочий процесс

• 1. Create a skill folder with a SKILL.md checklist.
• 2. Clone the skill repository and copy it to ~/.claude/skills/
• 3. Run the agent with the prompt "Review this diff."
• 4. The agent reads the skill, applies the five axes, and generates comments in the format "[axis] file:line problem → suggested fix".
• 5. Merge‑blocking comments are labeled "Must fix"; others are "Should fix".
• 6. The agent limits to 10 comments and requests a split for diffs >400 lines.

Доказательства

I asked my coding agent to review a 120-line pull request last month. It came back with 23 comments.

Источник

The review was technically thorough and completely useless. The dangerous stuff sailed through while the agent graded my taste in quotation marks.

Источник

The skill — code-review-checklist — reduces a review to one question: "Will this code become someone else's problem within three months?"

Источник

Every comment has a fixed format — [axis] file:line problem → suggested fix — and every comment must cite a concrete line.

Источник

If you want the whole team on it, commit the folder to .claude/skills/ in the repo — now every review in the project runs the same checklist.

Источник