What to Put in a Claude Code Skill for Reviewing Your Team's Code

Re:What to Put in a Claude Code Skill for Reviewing Your Tea... by rgambee on Thursday March 05, 2026 @10:48AM (#47263029)

I definitely see a lot of these anti-patterns in the code that CC writes. Many of these can be caught at the time the code is written without needing to wait for a PR review. To me, it seems like most of these instructions belong in CLAUDE.md instead of or in addition to a specialized review skill. Are there things in the review skill that don't belong in CLAUDE.md?

[ Parent | on HN ]

Re:What to Put in a Claude Code Skill for Reviewing Your Tea... by parad0x0n on Thursday March 05, 2026 @11:03AM (#47263293)

fair point. I think they should ALSO be in your CLAUDE.md. Doing Claude reviews with the exact same instructions still makes sense because the PR itself is often a co-production between AI and the human who gives the instructions (and in the coding process the agent might be more sycophantic toward the user's ideas). Additionally, when writing code, Claude is more focused on writing code that works and puts a lot of your repo into context, while during reviewing it pays more attention to these guidelines

[ Parent | on HN ]

Re:What to Put in a Claude Code Skill for Reviewing Your Tea... by Bullhorn9268 on Thursday March 05, 2026 @12:37PM (#47264666)

Weirdly, CC still adds stupid comments to guide its next steps despite having that instruction in CLAUDE.md AFAIK.

[ Parent | on HN ]