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.

来源